Host the cockpit with Bijou 7 by flyingrobots · Pull Request #109 · git-stunts/git-cas

flyingrobots · 2026-07-27T16:40:32Z

Closes #105
Closes #106

Result

Upgrades the coherent Bijou family to 7.2.0 and hosts the interactive cockpit in createFramedApp, preserving static output and domain behavior.

Design / proof

docs/design/0056-bijou-7-framed-cockpit/

Validation

npm test                         # 2,095 passed
npx eslint .                     # passed
npm run test:integration:node    # 199 passed
npm pack --dry-run               # passed

Release impact

Patch release: v6.5.6
Breaking public API change

coderabbitai · 2026-07-27T16:40:57Z

📝 Walkthrough

Walkthrough

The PR upgrades Bijou packages to 7.2.0, moves the interactive dashboard to createFramedApp, updates themed rendering and cursor ownership, adds dashboard and lower-mode tests, and documents the v6.5.6 release cycle.

Changes

Bijou 7 cockpit migration

Layer / File(s)	Summary
Runtime, theme, and progress contracts `package.json`, `bin/ui/context.js`, `bin/ui/theme.js`, `bin/ui/progress.js`, `test/unit/cli/`, `test/unit/package/`	Bijou dependencies, context wiring, theme tokens, cursor handling, and related regression tests are updated for 7.2.0.
Framed dashboard integration `bin/ui/dashboard.js`, `bin/ui/dashboard-view.js`, `test/unit/cli/dashboard*`	Dashboard state, rendering, key routing, overlays, commands, search, launch execution, and hosted tests are refactored around `createFramedApp`.
Cycle, usage, and validation records `BEARING.md`, `CHANGELOG.md`, `README.md`, `ROADMAP.md`, `docs/design/*`	Release status, cockpit guidance, design decisions, witness evidence, validation results, and roadmap references are updated for v6.5.6.

Estimated code review effort: 4 (Complex) | ~60 minutes

Possibly related PRs

git-stunts/git-cas#23: Related CLI context and dashboard launch mode handling.

Suggested reviewers: git-stunts

Poem

A rabbit frames the cockpit bright,
With guarded cursors tucked from sight.
Themes glow clear, commands take flight,
Bijou seven makes things right.
The burrow cheers the shipped delight!

🚥 Pre-merge checks | ✅ 4 | ❌ 1

❌ Failed checks (1 warning)

Check name	Status	Explanation	Resolution
Docstring Coverage	⚠️ Warning	Docstring coverage is 5.68% which is insufficient. The required threshold is 80.00%.	Write docstrings for the functions missing them to satisfy the coverage threshold.

✅ Passed checks (4 passed)

Check name	Status	Explanation
Linked Issues check	✅ Passed	The changes match `#105` and `#106` by upgrading Bijou 7.2.0, hosting the cockpit with createFramedApp, and preserving static/output behavior.
Out of Scope Changes check	✅ Passed	The documented and code changes stay focused on the Bijou 7 cockpit migration, tests, and release notes, with no clear unrelated additions.
Title check	✅ Passed	The title clearly summarizes the main change: hosting the cockpit with Bijou 7.
Description check	✅ Passed	The description covers the required sections and includes issue refs, design/proof, validation, and release impact.

✨ Finishing Touches

📝 Generate docstrings

Create stacked PR
Commit on current branch

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

_{Comment @coderabbitai help to get the list of available commands.}

coderabbitai

Actionable comments posted: 9

🤖 Prompt for all review comments with AI agents

Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.

Inline comments:
In `@bin/ui/dashboard-view.js`:
- Around line 529-547: Update renderAtlasBriefing so filtering removes only
absent sidebar section values while preserving the intentional empty-string
separators between sections. Keep the existing section order and newline joining
unchanged.

In `@bin/ui/dashboard.js`:
- Around line 666-705: Update handleOperationsKey to delegate its n, s, and x
branches to handleOpenStoreWizardMsg, handleRefreshStatisticsMsg, and
handleRunDoctorMsg respectively, passing the current model and dependencies as
required. Remove the duplicated state and command construction from
handleOperationsKey so the message handlers remain the single source of truth,
including their workspace: 'operations' behavior.
- Around line 1183-1185: The global dashboard keymap should not bind escape or
backspace because only modal-specific handlers process them. Remove the escape
and backspace bindings from the shared keymap while preserving their bindings in
the password and store-wizard modal configurations; leave the enter binding
unchanged.
- Around line 1207-1226: Update createDashboardModalKeyMap to preserve the
KeyMap instance returned by createKeyMap instead of spreading helpMap into a
plain object. Delegate the returned handle(msg) implementation to helpMap.handle
while retaining the existing passthrough behavior for unmatched input and the
current bindings.

In `@docs/design/0056-bijou-7-framed-cockpit/bijou-7-framed-cockpit.md`:
- Around line 365-370: Replace the unrelated “Done Does Not Mean” bullets in the
design document with git-cas-specific scope boundaries, including TUI
modernization `#39`, light-theme work, and the store-wizard rewrite. Retain the
boundary about internal cockpit panels only if it applies to this design, and
ensure the resulting contract contains no references to git-warp, retained
substrates, or the Think mind.

In `@README.md`:
- Around line 73-78: Update the quick-start keybinding list in the framed
cockpit README to match Bijou 7.2.0: remove the incorrect Shift+N
notification-history binding and unqualified “:” command binding, while
retaining the valid Ctrl+P command palette, ? help, F2 settings, and confirmed q
exit bindings.

In `@test/unit/cli/dashboard.launch.test.js`:
- Around line 39-57: The launchDashboard static fallback test should verify the
promised tab-separated format, not merely the presence of both values. In the
“retains the tab-separated static fallback outside a TTY” test, add an assertion
on the output string from output.write that contains “alpha” followed by a tab
and “abc123”.

In `@test/unit/cli/dashboard.test.js`:
- Around line 283-288: Strengthen the “renders themed dashboard headers” test by
retaining the existing exclusion assertion and adding a positive assertion for
workspace-owned header content, such as “1 Explorer” or “3 Operations.” Update
the assertion in the test using createDashboardApp and renderView so it verifies
the surface actually renders.
- Around line 12-24: Rename the local createDashboardApp test shim to a name
such as createPageHarness, and update its call sites in the legacy dashboard
tests. Keep the imported production createDashboardApp alias
createFramedDashboardApp unchanged so the two implementations remain clearly
distinguished.

🪄 Autofix (Beta)

Fix all unresolved CodeRabbit comments on this PR:

Push a commit to this branch (recommended)
Create a new PR with the fixes

ℹ️ Review info

⚙️ Run configuration

Configuration used: Organization UI

Review profile: ASSERTIVE

Plan: Pro Plus

Run ID: 07b633f7-8902-4c33-8c57-4605b9d12058

📥 Commits

Reviewing files that changed from the base of the PR and between 9ea91a7 and 88de976.

⛔ Files ignored due to path filters (1)

pnpm-lock.yaml is excluded by !**/pnpm-lock.yaml

📒 Files selected for processing (18)

BEARING.md
CHANGELOG.md
README.md
ROADMAP.md
bin/ui/context.js
bin/ui/dashboard-view.js
bin/ui/dashboard.js
bin/ui/progress.js
bin/ui/theme.js
docs/design/0056-bijou-7-framed-cockpit/bijou-7-framed-cockpit.md
docs/design/0056-bijou-7-framed-cockpit/witness/validation.md
docs/design/README.md
package.json
test/unit/cli/dashboard.launch.test.js
test/unit/cli/dashboard.test.js
test/unit/cli/progress.test.js
test/unit/cli/theme.test.js
test/unit/package/dependency-pins.test.js

📜 Review details

⏰ Context from checks skipped due to timeout. (3)

GitHub Check: test-docker (bun)
GitHub Check: test-docker (node)
GitHub Check: test-docker (deno)

🧰 Additional context used

📓 Path-based instructions (5)

docs/design/**