🤖 refactor: enforce components/features boundary cleanup

[jaaydenh]

Summary

Enforce a cleaner components/ vs features/ boundary: pure-UI primitives live in components/, domain-aware modules (store/context/IPC consumers) live in features/. Also standardizes icons → Icons casing and eliminates the orphan features/Hooks/ folder.

Background

The existing layout mixed domain-aware components (importing stores, contexts, window.api) into components/ alongside pure-UI primitives, while some pure-UI files lived under features/. This made navigation harder and blocked Figma Code Connect integration which expects a clear UI-primitive vs domain-feature boundary.

Implementation

Structural-only — every commit is a git mv + import rewrite with zero behavior changes.

Workstream 1 — Pure-UI features → components/

• Shimmer, PowerModeOverlay, CoderControls relocated from features/ to components/

Workstream 3 — Orphan features/Hooks/ eliminated

• useGitBranchDetails moved to hooks/

Workstream 4 — Icons casing standardized

• components/icons/ → components/Icons/ (PascalCase, matching all sibling dirs)

Workstream 2a — Workspace + Project/Git domain → features/

• 8 workspace-domain folders → features/Workspace/
• 5 project/git folders → features/Project/

Workstream 2b — Chat/command/system domain → features/

• 3 chat folders → features/Chat/
• CommandPalette/ → features/CommandPalette/
• 4 app-shell folders → features/AppShell/

New features/ tree

features/
├── About/           (existing)
├── Analytics/       (existing)
├── AppShell/        ← NEW
├── Chat/            ← NEW
├── ChatInput/       (existing)
├── CommandPalette/  ← NEW
├── Messages/        (existing)
├── Project/         ← NEW
├── RightSidebar/    (existing)
├── Settings/        (existing)
├── Shared/          (existing)
├── SplashScreens/   (existing)
├── Tools/           (existing)
└── Workspace/       ← NEW

Validation

• make static-check passes (typecheck, lint, fmt, broken-links)
• 28 path hygiene greps — all return 0 hits (no stale import paths remain)

Risks

• Low: rename-only changes preserve all runtime behavior. git mv retains file history. All imports confirmed clean via both typecheck and grep audit.

---

Generated with mux • Model: anthropic:claude-opus-4-6 • Thinking: xhigh • Cost: $5.21

[jaaydenh]

@codex review

[jaaydenh]

@codex review

[chatgpt-codex-connector]

💡 Codex Review

Here are some automated review suggestions for this pull request.

Reviewed commit: 7d2998b05a

ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

• Open a pull request for review
• Mark a draft as ready
• Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".

[jaaydenh]

@codex review

The Shimmer import was already fixed in the latest push (commit ccabe7e). Please take another look.

[chatgpt-codex-connector]

Codex Review: Didn't find any major issues. Swish!

ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

• Open a pull request for review
• Mark a draft as ready
• Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".

[jaaydenh]

@codex review

Fixed the AppLoader.auth.test.tsx CI failure by mocking @/browser/App to prevent react-dnd module evaluation in happy-dom. Also squashed the empty CI retrigger commit.

[jaaydenh]

@codex review

Rebased on main (resolved conflict in stories/meta.tsx). All static checks pass locally.

[chatgpt-codex-connector]

Codex Review: Didn't find any major issues. You're on a roll.

ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

• Open a pull request for review
• Mark a draft as ready
• Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".

[jaaydenh]

@codex review

Added react-dnd and react-dnd-html5-backend module mocks to AppLoader auth test to prevent transitive @react-dnd/asap module evaluation in CI's happy-dom environment. Root cause: AppLoader's store/context imports transitively reach react-dnd even with the App mock in place.

[chatgpt-codex-connector]

💡 Codex Review

Here are some automated review suggestions for this pull request.

Reviewed commit: 5facdd6bce

ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

• Open a pull request for review
• Mark a draft as ready
• Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".

[jaaydenh]

@codex review

Expanded react-dnd and react-dnd-html5-backend mocks to include all commonly imported symbols (useDragLayer, getEmptyImage) to prevent leaked-mock issues across test files.

[chatgpt-codex-connector]

Codex Review: Didn't find any major issues. Already looking forward to the next diff.

ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

• Open a pull request for review
• Mark a draft as ready
• Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".

[jaaydenh]

@codex review

Cleaned up commit history — restored AppLoader auth test to match main baseline (only updated relative paths for new location). All previous test over-modifications reverted.

[chatgpt-codex-connector]

Codex Review: Didn't find any major issues. You're on a roll.

ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

• Open a pull request for review
• Mark a draft as ready
• Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".

[jaaydenh]

@codex review

Fixed AppLoader auth test: added module-scope installDom() before the deferred import("./AppLoader") so the DOM is available when react-dnd evaluates document.createTextNode during module loading. This is needed because the test's new path changes Bun's test execution order.

[chatgpt-codex-connector]

Codex Review: Didn't find any major issues. You're on a roll.

ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

• Open a pull request for review
• Mark a draft as ready
• Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".

[jaaydenh]

@codex review

Made AppLoader auth test fully self-contained: mocked all context providers (PolicyContext, RouterContext, ProjectContext, WorkspaceContext) and stores (WorkspaceStore, GitStatusStore, BackgroundBashStore, PRStatusStore) to prevent dependence on test execution order. This fixes the CI failure caused by the test's new path changing Bun's alphabetical test ordering.

[chatgpt-codex-connector]

💡 Codex Review

Here are some automated review suggestions for this pull request.

Reviewed commit: 5244350dd0

ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

• Open a pull request for review
• Mark a draft as ready
• Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".

[jaaydenh]

@codex review

Resolved mock teardown thread: Bun's mock.module() is process-global and cannot be restored — there's no unmockModule() API. The test already mitigates leak impact by exporting all commonly imported symbols from each mocked module (same pattern used in the existing AuthTokenModal mock which documents this limitation).

[chatgpt-codex-connector]

Codex Review: Didn't find any major issues. Can't wait for the next one!

ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

• Open a pull request for review
• Mark a draft as ready
• Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".