Skip to content

refactor: split large files into focused modules#688

Merged
max-sixty merged 3 commits intomainfrom
file-sizes
Jan 16, 2026
Merged

refactor: split large files into focused modules#688
max-sixty merged 3 commits intomainfrom
file-sizes

Conversation

@max-sixty
Copy link
Owner

Summary

  • Split 6 large files (~11,500 lines total) into smaller, focused modules
  • Updated .pre-commit-config.yaml exclude patterns to match new module paths
Original Lines New Module
shell.rs 1,709 src/shell/ (detection, paths, utils)
ci_status.rs 1,710 src/commands/list/ci_status/ (cache, github, gitlab, platform)
worktree.rs 1,632 src/commands/worktree/ (hooks, push, remove, resolve, switch, types)
config.rs 1,904 src/commands/config/ (create, hints, show, state)
select.rs 1,850 src/commands/select/ (items, log_formatter, pager, preview)
collect.rs 2,639 src/commands/list/collect/ (execution, results, tasks, types)

Test plan

  • All 428 unit tests pass
  • All 854 integration tests pass
  • All pre-commit lints pass

🤖 Generated with Claude Code

max-sixty and others added 3 commits January 16, 2026 13:44
Split 6 large files (~11,500 lines total) into smaller, focused modules:

- shell.rs (1,709 lines) → src/shell/ (detection, paths, utils)
- ci_status.rs (1,710 lines) → src/commands/list/ci_status/ (cache, github, gitlab, platform)
- worktree.rs (1,632 lines) → src/commands/worktree/ (hooks, push, remove, resolve, switch, types)
- config.rs (1,904 lines) → src/commands/config/ (create, hints, show, state)
- select.rs (1,850 lines) → src/commands/select/ (items, log_formatter, pager, preview)
- collect.rs (2,639 lines) → src/commands/list/collect/ (execution, results, tasks, types)

Also updated .pre-commit-config.yaml exclude patterns to match new module paths.

Co-Authored-By: Claude <noreply@anthropic.com>
Co-Authored-By: Claude <noreply@anthropic.com>
@max-sixty max-sixty merged commit 51c65bd into main Jan 16, 2026
20 of 21 checks passed
@max-sixty max-sixty deleted the file-sizes branch January 16, 2026 22:15
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