Skip to content

refactor(cli): decompose monolithic cli/index.ts into modular command modules - #2286

Merged
mbiernacik merged 7 commits into
mainfrom
split-cli-index
Sep 8, 2026
Merged

refactor(cli): decompose monolithic cli/index.ts into modular command modules#2286
mbiernacik merged 7 commits into
mainfrom
split-cli-index

Conversation

@mbiernacik

Copy link
Copy Markdown
Contributor

Splits the large cli/index.ts file into separate files for each command under cli/commands/.
Common options and project config flags were also moved into helper files. Now cli/index.ts only sets up the CLI and registers each command.
There are no behavior or API changes—all commands and options work exactly the same as before.

@mbiernacik
mbiernacik requested a review from a team as a code owner September 7, 2026 14:23
@mbiernacik
mbiernacik requested review from Tuseeq1, apilaskowski and rafal-hawrylak and removed request for a team September 7, 2026 14:23
Comment thread cli/commands/run_command.ts
Comment thread cli/project_config_options.ts Outdated
Comment thread cli/commands/run_command.ts Outdated

@apilaskowski apilaskowski left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

Overall LGTM from me, lets just fix those small things before submitting.

@rafal-hawrylak rafal-hawrylak left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

only pre-existing nits

Comment thread cli/common_options.ts Outdated
Comment thread cli/common_options.ts Outdated
Comment thread cli/common_options.ts Outdated
@mbiernacik
mbiernacik merged commit 9ce9612 into main Sep 8, 2026
11 checks passed
@mbiernacik
mbiernacik deleted the split-cli-index branch September 8, 2026 11:38
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.

3 participants