Skip to content

refactor: split pages into src/pages and components into layout/ui - #748

Merged
eyevinn-developer[bot] merged 1 commit into
mainfrom
frontend/515-components-folder-cleanup
Nov 3, 2026
Merged

eyevinn-developer[bot] merged 1 commit into
mainfrom
frontend/515-components-folder-cleanup

Conversation

@eyevinn-developer

Copy link
Copy Markdown
Contributor

Summary

  • Move route-level pages (landing, create production, manage productions, calls) into new src/pages/.
  • Split src/components/ into layout/ (header, page header, containers, auth guard, error handling) and ui/ (reusable and feature components).
  • Pure structural refactor via git mv; every relative import and vi.mock path updated, no behavior changes.

Test plan

  • npm run typecheck
  • npm run lint
  • npm run build
  • npm test (61 files, 594 tests)

Closes #515

What a human does next

Nothing, beyond reviewing. Use git diff -M or the GitHub rename view, since the diff is almost entirely renames plus import-path edits.

🤖 Generated with Claude Code

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>

@eyevinn-pr-reviewer eyevinn-pr-reviewer Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

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).

@eyevinn-developer
eyevinn-developer Bot merged commit 283b94c into main Nov 3, 2026
6 checks passed
@eyevinn-developer
eyevinn-developer Bot deleted the frontend/515-components-folder-cleanup branch November 3, 2026 06:51
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.

Frontend: Refactor / cleanup of components folder

1 participant