Skip to content

refactor: organize artifact CLI commands - #50

Closed
mnriem wants to merge 4 commits into
mainfrom
mnriem-chore/artifact-cli-refactor
Closed

mnriem wants to merge 4 commits into
mainfrom
mnriem-chore/artifact-cli-refactor

Conversation

@mnriem

@mnriem mnriem commented Sep 22, 2026

Copy link
Copy Markdown
Owner

Summary

  • move each specify artifact command into its paired command_<name>.py module
  • keep shared Typer registration and CLI infrastructure in artifacts/_commands.py
  • reorganize artifact command tests to mirror the source hierarchy while retaining domain and cross-domain contracts
  • add an explicit command registration order/uniqueness test

Validation

  • full repository suite: 8,358 passed, 16 skipped (8,374 collected)
  • focused artifact suite: 149 passed
  • exact uvx ruff@0.15.0 check src tests: passed
  • git diff --check: passed
  • full specify artifact help surface: byte-identical before/after
  • wheel packaging audit: all new command modules included and no obsolete grouped modules present
  • original artifact test function names preserved; collection increased by one intentional registration test

AI disclosure

Authored autonomously by GitHub Copilot (model: GPT-5.6 Sol) on behalf of @mnriem. The implementation, tests, validation, commit, and this pull request description were fully AI-generated.

mnriem and others added 4 commits September 21, 2026 16:44
* refactor: organize preset CLI commands

Assisted-by: GitHub Copilot (model: GPT-5.6 Sol, autonomous)

Co-authored-by: Copilot App <223556219+Copilot@users.noreply.github.com>

* fix: accept project root in Amp dispatch

Assisted-by: GitHub Copilot (model: GPT-5.6 Sol, autonomous)

Co-authored-by: Copilot App <223556219+Copilot@users.noreply.github.com>

* fix: preserve preset refactor seams

Assisted-by: GitHub Copilot (model: GPT-5.6 Sol, autonomous)

Co-authored-by: Copilot App <223556219+Copilot@users.noreply.github.com>

---------

Co-authored-by: Copilot App <223556219+Copilot@users.noreply.github.com>
* refactor: organize integration CLI commands

Assisted-by: GitHub Copilot (model: GPT-5.6 Sol, autonomous)

Co-authored-by: Copilot App <223556219+Copilot@users.noreply.github.com>

* chore: keep integration refactor scoped

Assisted-by: GitHub Copilot (model: GPT-5.6 Sol, autonomous)

Co-authored-by: Copilot App <223556219+Copilot@users.noreply.github.com>

* refactor: finish integration CLI hierarchy

Assisted-by: GitHub Copilot (model: GPT-5.6 Sol, autonomous)

Co-authored-by: Copilot App <223556219+Copilot@users.noreply.github.com>

---------

Co-authored-by: Copilot App <223556219+Copilot@users.noreply.github.com>
Assisted-by: GitHub Copilot (model: GPT-5.6 Sol, autonomous)

Co-authored-by: Copilot App <223556219+Copilot@users.noreply.github.com>
Assisted-by: GitHub Copilot (model: GPT-5.6 Sol, autonomous)

Co-authored-by: Copilot App <223556219+Copilot@users.noreply.github.com>
@mnriem

mnriem commented Sep 22, 2026

Copy link
Copy Markdown
Owner Author

Closed because this pull request was accidentally opened against the fork instead of the intended upstream repository. Posted on behalf of @mnriem by GitHub Copilot (model: GPT-5.6 Sol, autonomous); comment fully AI-drafted.

@mnriem mnriem closed this Sep 22, 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