Skip to content

Fix theme hydration and terminal/file navigation#333

Merged
BunsDev merged 3 commits into
mainfrom
okcode/fix-failing-tests
Apr 7, 2026
Merged

Fix theme hydration and terminal/file navigation#333
BunsDev merged 3 commits into
mainfrom
okcode/fix-failing-tests

Conversation

@BunsDev
Copy link
Copy Markdown
Member

@BunsDev BunsDev commented Apr 7, 2026

Summary

  • Fix server-side theme hydration by providing a stable useSyncExternalStore server snapshot.
  • Lazy-load the terminal drawer and add a loading fallback to reduce initial chat UI cost.
  • Open terminal file links in the in-app code viewer instead of jumping to an external editor when appropriate.
  • Update the header panels menu to prefetch the terminal on hover/focus and wire panel toggles through more explicit props.
  • Adjust tests to cover the new file-open path and updated sidebar behavior.

Testing

  • Not run (not requested).
  • Updated/added coverage in apps/web/src/fileOpen.test.ts for in-app file opening.
  • Updated apps/web/src/components/Sidebar.test.ts to match the new right-panel file tree behavior.
  • Adjusted apps/server/src/git/Layers/GitManager.test.ts to keep the rebase scenario self-contained without a remote.

- add a stable server snapshot for `useTheme`
- make chat header preview toggle safe when missing
- update tests for the sidebar file tree shortcut and Git rebase flow
@vercel
Copy link
Copy Markdown

vercel Bot commented Apr 7, 2026

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated (UTC)
okcode-marketing Ready Ready Preview Apr 7, 2026 4:24am
v0-compute-the-platform-to-build Ready Ready Preview, Open in v0 Apr 7, 2026 4:24am

- Stop destructuring unused viewer and issue-thread props
- Attach `githubRef` after building projection snapshots
- Simplify validation and query helpers
- Build thread snapshots with an optional `githubRef`
- Keep the projection query aligned with `OrchestrationThread`
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

size:L vouch:trusted PR author is trusted by repo permissions or the VOUCHED list.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant