Skip to content

refactor(cli): split src/cli.ts for 450-line rule#39

Closed
hmmhmmhm wants to merge 2 commits into
mainfrom
chore/split-cli
Closed

refactor(cli): split src/cli.ts for 450-line rule#39
hmmhmmhm wants to merge 2 commits into
mainfrom
chore/split-cli

Conversation

@hmmhmmhm

@hmmhmmhm hmmhmmhm commented May 5, 2026

Copy link
Copy Markdown
Owner

src/cli.ts was 943 lines, exceeding the 450-line rule from AGENTS.md.

Split into smaller modules under src/cli/:

  • src/cli.ts: 181 lines (orchestrator)
  • src/cli/args.ts: 66 lines
  • src/cli/commands/convenience.ts: 335 lines
  • src/cli/commands/daiso.ts: 165 lines
  • src/cli/constants.ts: 6 lines
  • src/cli/deps.ts: 57 lines
  • src/cli/http.ts: 120 lines
  • src/cli/types.ts: 20 lines

All files now under 450 lines.

Verification:

  • pnpm typecheck: passed
  • pnpm test: 1 pre-existing failure (직접 실행 여부를 올바르게 판별한다), unrelated to this refactor

No public CLI commands or runtime behavior changed.

@hmmhmmhm hmmhmmhm marked this pull request as ready for review May 5, 2026 06:27
@hmmhmmhm

Copy link
Copy Markdown
Owner Author

Superseded by #43. The CLI split was merged there, with the coverage configuration updated so the 100% coverage gate stays green.

@hmmhmmhm hmmhmmhm closed this May 14, 2026
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