Skip to content

Fix onboard intro capture and Git instructions routing#297

Merged
0xroylee merged 2 commits into
mainfrom
codex/fix-onboard-intro-capture
Jun 3, 2026
Merged

Fix onboard intro capture and Git instructions routing#297
0xroylee merged 2 commits into
mainfrom
codex/fix-onboard-intro-capture

Conversation

@0xroylee
Copy link
Copy Markdown
Owner

@0xroylee 0xroylee commented Jun 3, 2026

What Changed

  • Added a top-level /git page for the Git instruction editor and wired the workspace navigation to it.
  • Restored the Git instruction panel as a dedicated component backed by the existing GitHub settings API.
  • Updated the shell navigation tests to cover the new /git route contract.

Why

  • The old Git settings page lived under /settings/git, but the current shell now surfaces Git as its own operator page.
  • This keeps commit and pull request instruction editing reachable from the main navigation without relying on the removed settings route.

How to Test

  • bun test packages/web/tests/web-shell-navigation.test.ts packages/web/tests/settings-navigation.test.ts
  • bun test
  • Open http://127.0.0.1:3000/git and confirm the Commit and Pull request fields render with the saved values.

Additional Notes

  • bun run --filter web build passed and confirmed the /git route is included in the app build.
  • A repo-wide bun run check still reports unrelated pre-existing formatting issues in chat-room files outside this change.

@0xroylee 0xroylee merged commit 0847afe into main Jun 3, 2026
4 of 6 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant