Repository navigation
refactor(web): size the knowledge panes with PixelPane, plus certifi bump - #2606
Merged
Merged
Conversation
Bumps the uv group with 1 update in the / directory: [certifi](https://github.com/certifi/python-certifi). Updates `certifi` from 2026.10.18 to 2026.10.19 - [Commits](certifi/python-certifi@2026.10.18...2026.10.19) --- updated-dependencies: - dependency-name: certifi dependency-version: 2026.10.19 dependency-type: indirect dependency-group: uv ... Signed-off-by: dependabot[bot] <support@github.com>
0xPlayerOne
marked this pull request as draft
October 21, 2026 04:18
The Dependabot lock rewrite dropped the x-release-please-version marker on the cortana-brain entry and wrote revision 3, which fails tests/test_package.py and release-please. Restore both; the certifi bump itself is unchanged. Co-Authored-By: Claude Sonnet 5.5 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01XSokH4xCdEiUXMpwiAfHgq
…eHandle Drops @corvu/resizable from the startup graph (about 23 KB). Pane widths were already kept in pixels by the host; the panes now take them directly. Needs the ui release that adds PixelPane before it can be pushed. Co-Authored-By: Claude Sonnet 5.5 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01XSokH4xCdEiUXMpwiAfHgq
Moves the pin, lockfile and published notice to 0.146.0, which ships PixelPane, so the knowledge panes no longer need the local primitive or @corvu/resizable at startup. The release markers in bun.lock are kept. Co-Authored-By: Claude Sonnet 5.5 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01XSokH4xCdEiUXMpwiAfHgq
0xPlayerOne
marked this pull request as ready for review
October 21, 2026 20:11
0xPlayerOne
enabled auto-merge (squash)
October 21, 2026 20:12
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
Takes over Dependabot's certifi bump (2026.10.18 to 2026.10.19) and adds the UI adoption that gets the web app's initial JavaScript back under its budget.
chore(deps): certifi 2026.10.19, with the release marker and lock revision kept inuv.lock.refactor(web): the knowledge workspace sizes its three panes with the sharedPixelPaneandPixelResizeHandleinstead of@corvu/resizable, which no longer loads at startup. Pane widths stay host-owned (paneWidths), with the same minimums and maximums as before. Cortana's web UI contract still forbids JSX styles, soPixelPanecarries the computed width.chore(web):@adea-ai/uimoves to 0.146.0 (package.json files, bun.lock with its release markers, NOTICE), the release that shipsPixelPane(feat(ui): add PixelPane for host-owned pixel widths ui#478) and the lazily loaded reorder dialog (perf(ui): load the workspace reorder dialog on demand ui#477).Validation
bun run lint(UI tailwind sources, web UI contract,oxlint --deny-warnings),oxfmt --check .,tsc --noEmitinapps/web.bun test --conditions=browser apps/web/src/App.test.tsx apps/web/src/App.desktop.test.tsx: 178 pass. The context splitter test now expects the pane to grow when dragged.bun scripts/run-js-tests.mjs: all 79 JavaScript test files pass in 4 groups. (An earlier attempt hit the runner's 420 s group timeout; the same group passes alone in seconds, and a rerun passed in full.)bun test scripts/shared-notice-contract.test.mjs(2 pass),uv run pytest tests/test_package.py(4 pass),bun install --frozen-lockfile(no changes).vite buildplus the bundle budget: initial application JavaScript 503,923 of 525,000 bytes (headroom 21,077), complete JavaScript 887,211 of 940,000, CSS 242,860 of 285,000. The cap is unchanged.Impact and rollout
Performance: initial JavaScript drops by roughly 20 KB (the corvu resizable code leaves the startup graph).
Review notes
Generated by Claude Code