Skip to content

Refactor feedback-panel.tsx: extract detail panel and finding row#739

Merged
selfcontained merged 1 commit into
mainfrom
agt_6c3ade22d844/job-componentizer-af406ca1
Jul 9, 2026
Merged

Refactor feedback-panel.tsx: extract detail panel and finding row#739
selfcontained merged 1 commit into
mainfrom
agt_6c3ade22d844/job-componentizer-af406ca1

Conversation

@selfcontained

Copy link
Copy Markdown
Owner

Summary

  • Extracted FeedbackDetailPanel (~230 lines) into feedback-detail-panel.tsx — the feedback detail view with navigation, actions, and content rendering is now a self-contained module.
  • Extracted FeedbackFindingRow (~90 lines) into feedback-finding-row.tsx — the per-finding row rendering (severity dot, file path, status icon, round dividers) is now a focused component.
  • feedback-panel.tsx drops from 660 → 327 lines. Re-exports preserved so no consumer changes needed.

New file structure

feedback-panel.tsx          327 lines  (was 660)  — ParentFeedbackPanel + re-exports
feedback-detail-panel.tsx   253 lines  (new)      — FeedbackDetailPanel
feedback-finding-row.tsx     94 lines  (new)      — FeedbackFindingRow

Next up

media-sidebar.tsx (629 lines) — extract media item renderers, pin display, and media list into subcomponents.

Test plan

  • pnpm run finalize:web — type check + production build passes
  • pnpm run test:e2e — all 172 tests pass
  • No behavior change — purely structural refactoring with re-exports for backwards compatibility

🤖 Generated with Claude Code

@selfcontained
selfcontained merged commit cd163cb into main Jul 9, 2026
1 check passed
@selfcontained
selfcontained deleted the agt_6c3ade22d844/job-componentizer-af406ca1 branch July 9, 2026 08:14
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