Repository navigation
refactor: split pages into src/pages and components into layout/ui - #748
Conversation
Move route-level pages (landing, create, manage, calls) to src/pages and split src/components into layout/ (header, page header, containers, auth guard, error boundaries) and ui/ (reusable and feature components). All relative imports and vi.mock paths updated; no behavior changes. Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>
There was a problem hiding this comment.
Code Review — LGTM
Automated review (daily-backlog-pr Phase 3). Pure structural refactor (#515): route-level pages moved to src/pages/, src/components/ split into layout/ and ui/, done with git mv. Verified the non-rename content changes are only import-path and vi.mock rewrites plus prettier re-wraps — no logic changes; a full resolve of every relative import / dynamic import() / vi.mock / require path across src found zero dangling references; no tsconfig/vite path aliases to update; nothing duplicated or left in old locations; double quotes / es5 trailing commas / 2-space indent preserved; no yarn. CI is green (lint, prettier, builds, tests, e2e ×2).
Non-blocking follow-up: docs/mute-state-model.md:27,43,50,56 still reference the old src/components/production-line/... path (now src/components/ui/production-line/); worth fixing in a follow-up (doc-only, no build impact).
Summary
src/pages/.src/components/intolayout/(header, page header, containers, auth guard, error handling) andui/(reusable and feature components).git mv; every relative import andvi.mockpath updated, no behavior changes.Test plan
npm run typechecknpm run lintnpm run buildnpm test(61 files, 594 tests)Closes #515
What a human does next
Nothing, beyond reviewing. Use
git diff -Mor the GitHub rename view, since the diff is almost entirely renames plus import-path edits.🤖 Generated with Claude Code