Skip to content

refactor(web): compose app from route modules#126

Merged
xingkaixin merged 1 commit into
mainfrom
feat/cs-45-app-route-composition
Jul 11, 2026
Merged

refactor(web): compose app from route modules#126
xingkaixin merged 1 commit into
mainfrom
feat/cs-45-app-route-composition

Conversation

@xingkaixin

Copy link
Copy Markdown
Owner

Summary

  • move route-specific content and derived state into AppRouteContent
  • extract route header titles and breadcrumbs into a pure model builder
  • replace the wide sidebar prop surface with explicit model and action interfaces
  • reduce App.tsx from 915 to 596 lines while reusing canonical session and agent indexes

Impact

This is an internal composition refactor. Existing routes, search, bookmarks, live refresh, keyboard navigation, and sidebar behavior remain unchanged.

Validation

  • pnpm format
  • pnpm lint
  • pnpm format:check
  • pnpm build
  • pnpm test (662 tests)
  • pnpm test:e2e (6 tests)

Closes CS-45

@xingkaixin xingkaixin marked this pull request as ready for review July 11, 2026 12:37
@xingkaixin xingkaixin merged commit 373ec77 into main Jul 11, 2026
7 checks passed
@xingkaixin xingkaixin deleted the feat/cs-45-app-route-composition branch July 11, 2026 12:37
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