Skip to content

refactor(web): size the knowledge panes with PixelPane, plus certifi bump - #2606

Merged
0xPlayerOne merged 5 commits into
mainfrom
dependabot/uv/uv-5b18fe017b
Oct 21, 2026
Merged

0xPlayerOne merged 5 commits into
mainfrom
dependabot/uv/uv-5b18fe017b

Conversation

@dependabot

@dependabot dependabot Bot commented on behalf of github Oct 21, 2026 •

Copy link
Copy Markdown
Contributor

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 in uv.lock.
  • refactor(web): the knowledge workspace sizes its three panes with the shared PixelPane and PixelResizeHandle instead 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, so PixelPane carries the computed width.
  • chore(web): @adea-ai/ui moves to 0.146.0 (package.json files, bun.lock with its release markers, NOTICE), the release that ships PixelPane (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

  • Commands run (local, on the head below):
    • bun run lint (UI tailwind sources, web UI contract, oxlint --deny-warnings), oxfmt --check ., tsc --noEmit in apps/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 build plus 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.
  • CI or deployment links: this PR is still a draft, so hosted lanes have not run on this head.
  • Screenshots/recordings, when applicable: none. Cortana has no pixel baselines. Visible difference: the resize grip is now a quiet bar and the panes sit 1px differently than the corvu handle did.

Impact and rollout

  • No migration or rollout action is required
  • Migration or rollout steps are documented below
  • Environment or secret changes are documented below
  • Security, privacy, compatibility, or performance impact is documented below
  • Rollback or follow-up work is documented below

Performance: initial JavaScript drops by roughly 20 KB (the corvu resizable code leaves the startup graph).

Review notes

  • Draft, auto-merge off; this branch was Dependabot's and has been edited, so Dependabot will no longer rebase it.
  • Not verified here: hosted CI on this head, and the macOS visual lane.

Generated by Claude Code

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>
@dependabot
dependabot Bot requested a review from 0xPlayerOne as a code owner October 21, 2026 03:59
@dependabot dependabot Bot added dependencies Pull requests that update a dependency file python:uv Pull requests that update python:uv code labels Oct 21, 2026
@0xPlayerOne
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 0xPlayerOne changed the title chore(deps): bump certifi from 2026.10.18 to 2026.10.19 in the uv group across 1 directory refactor(web): size the knowledge panes with PixelPane, plus certifi bump Oct 21, 2026
@0xPlayerOne
0xPlayerOne marked this pull request as ready for review October 21, 2026 20:11
@0xPlayerOne
0xPlayerOne enabled auto-merge (squash) October 21, 2026 20:12
@0xPlayerOne
0xPlayerOne merged commit 19a4031 into main Oct 21, 2026
62 of 63 checks passed
@0xPlayerOne
0xPlayerOne deleted the dependabot/uv/uv-5b18fe017b branch October 21, 2026 20:26
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

dependencies Pull requests that update a dependency file python:uv Pull requests that update python:uv code

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants