Skip to content

refactor: delete 7 dead component files#791

Draft
jSydorowicz21 wants to merge 2 commits intoRunMaestro:rcfrom
jSydorowicz21:dedup/phase-01a-dead-components
Draft

refactor: delete 7 dead component files#791
jSydorowicz21 wants to merge 2 commits intoRunMaestro:rcfrom
jSydorowicz21:dedup/phase-01a-dead-components

Conversation

@jSydorowicz21
Copy link
Copy Markdown
Contributor

@jSydorowicz21 jSydorowicz21 commented Apr 11, 2026

Removes 7 component files with zero production imports and their 3 associated test files. Verified via grep that only references remaining are comments, type aliases, or self-references.

  • AgentSessionsModal: superseded by AgentSessionsBrowser
  • GitWorktreeSection: unreferenced worktree UI
  • GroupChatParticipants: superseded by GroupChatRightPanel
  • MergeProgressModal: superseded by TransferProgressModal
  • ShortcutEditor: unreferenced shortcut editor
  • SummarizeProgressModal: unreferenced progress modal
  • ThemePicker: unreferenced theme picker

Lint passes clean. Targeted tests pass

Summary by CodeRabbit

Release Notes

  • Tests

    • Removed test suites for shortcut editor, theme picker, and agent session management components
  • Removed Components

    • Removed UI components for agent session management, shortcut editing, theme selection, git worktree configuration, group chat participants, merge progress tracking, and summarization features

@jSydorowicz21 jSydorowicz21 self-assigned this Apr 11, 2026
@coderabbitai
Copy link
Copy Markdown

coderabbitai bot commented Apr 11, 2026

Important

Review skipped

Draft detected.

Please check the settings in the CodeRabbit UI or the .coderabbit.yaml file in this repository. To trigger a single review, invoke the @coderabbitai review command.

⚙️ Run configuration

Configuration used: Path: .coderabbit.yaml

Review profile: CHILL

Plan: Pro

Run ID: f9739d2b-be9f-4852-93ef-a406dc8d2958

You can disable this status message by setting the reviews.review_status to false in the CodeRabbit configuration file.

Use the checkbox below for a quick retry:

  • 🔍 Trigger review
✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

Removes 7 component files with zero production imports and their 3
associated test files. Verified via grep that only references remaining
are comments, type aliases, or self-references.

- AgentSessionsModal: superseded by AgentSessionsBrowser
- GitWorktreeSection: unreferenced worktree UI
- GroupChatParticipants: superseded by GroupChatRightPanel
- MergeProgressModal: superseded by TransferProgressModal
- ShortcutEditor: unreferenced shortcut editor
- SummarizeProgressModal: unreferenced progress modal
- ThemePicker: unreferenced theme picker

Lint passes clean. Targeted tests pass; pre-existing baseline failure
in BatchRunnerModal.test.tsx is unchanged.
Three surviving files had header-docstring or inline comments naming
components deleted in this PR. Clean them up:

- GroupChatRightPanel: drop 'Replaces direct use of GroupChatParticipants'
- TransferProgressModal: drop 'Based on MergeProgressModal patterns'
- BatchRunnerModal.test: drop 'Git Worktree tests were removed' NOTE
  pointing to GitWorktreeSection.test.tsx (which never existed)
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