Skip to content

refactor(web): workspace feature folders and sidebar split - #383

Merged
kinqsradio merged 7 commits into
mainfrom
refactor/workspace-folders
Nov 9, 2026
Merged

kinqsradio merged 7 commits into
mainfrom
refactor/workspace-folders

Conversation

@kinqsradio

Copy link
Copy Markdown
Member

Summary

  • features/workspace/ becomes sidebar/, palette/ and app-chrome/; the front door index.ts exports the same names.
  • Sidebar.tsx 1,199 to 590 lines and leaves the long-files baseline (11 to 10 long TypeScript files, 21 direction entries unchanged).
  • No behaviour, copy, class-name or CSS change; addProject.css stays put.

Commits: pure git mv (8678212, in blame-ignore), import fixes, two cut commits, baseline shrink, docs.

Edits that are not pure moves:

  • Import paths and vi.mock paths in moved files, app/, thread/, onboarding/; three renamed baseline keys.
  • Sidebar cuts are verbatim except: useSidebarCatalog (settings and catalog state moved with their effects into one hook at the same call position), useSelectionDismiss (adds the stable setSelection to deps), finishDrop and mentionInComposer take their refs, store actions and selectedThread as arguments, handleTreeKey and folderMenuItems take what they read as arguments, SearchField and DeleteThreadsDialog are module-level components with no wrapper element.
  • docs/refactor.md status and moved paths in docs/design-inventory.md.

Parity rows

none

Frames built

Frames built: none

Testing

  • CI covers the rest.
  • Local: npm run check, npm test (1,714 passed), npm run build; mock Playwright workspace, pane-resize, titlebar-line, search-fields, selection specs (59 passed).
  • Not tested: sidebar drag-and-drop reorder and the command palette have no mock Playwright spec; only unit tests cover them. Rendered DOM not diffed.

Co-authors

none

AI usage

  • Written by hand, with no AI
  • Harness and version: Claude Code 2.1.318
  • Models: claude-sonnet-5-5 (all)
  • Effort: medium
  • Human role: set the target layout and rules, and will review the diff
  • Sources: none

Checklist

  • Works on Windows and macOS
  • web/src/generated with any contract change; docs/architecture.md with any rule change (none needed)
  • No copied source, secrets, credentials or personal data; no AI attribution in commits, title or branch

@github-actions github-actions Bot added area: web web: React UI area: docs docs and Markdown files labels Nov 9, 2026
@kinqsradio
kinqsradio merged commit 8c6feb1 into main Nov 9, 2026
29 checks passed
@kinqsradio
kinqsradio deleted the refactor/workspace-folders branch November 9, 2026 04:00
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

area: docs docs and Markdown files area: web web: React UI

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant