Skip to content

[cli] Update codebase to argmojo v0.5.0 and use declarative API#192

Merged
forfudan merged 3 commits intodevfrom
cli
Apr 7, 2026
Merged

[cli] Update codebase to argmojo v0.5.0 and use declarative API#192
forfudan merged 3 commits intodevfrom
cli

Conversation

@forfudan
Copy link
Copy Markdown
Owner

@forfudan forfudan commented Apr 7, 2026

This PR updates the CLI calculator to use ArgMojo v0.5.0 and migrates the argument parsing code to ArgMojo’s declarative Parsable API (hybrid, with builder API).

Changes:

  • Replaced imperative Command/Arg setup with a DecimoArgs(Parsable) definition and parsing via to_command() / parse_from_command().
  • Switched the project to depend on argmojo >=0.5.0,<0.6.0 and simplified CLI build steps to rely on the packaged dependency.
  • Removed the CI “fetch argmojo” step now that ArgMojo is pulled in via pixi.

Copy link
Copy Markdown

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

Updates the CLI calculator to use ArgMojo v0.5.0 and migrates the argument parsing code to ArgMojo’s declarative Parsable API.

Changes:

  • Replaced imperative Command/Arg setup with a DecimoArgs(Parsable) definition and parsing via to_command() / parse_from_command().
  • Switched the project to depend on argmojo >=0.5.0,<0.6.0 and simplified CLI build steps to rely on the packaged dependency.
  • Removed the CI “fetch argmojo” step now that ArgMojo is pulled in via pixi.

Reviewed changes

Copilot reviewed 3 out of 4 changed files in this pull request and generated 2 comments.

File Description
src/cli/main.mojo Migrates CLI arg parsing to ArgMojo’s declarative Parsable model and adds help tips.
pixi.toml Adds ArgMojo v0.5 dependency and updates tasks/build steps to stop using a local clone by default.
pixi.lock Locks ArgMojo v0.5.0 for supported platforms.
.github/workflows/run_tests.yaml Removes the step that cloned ArgMojo in CI.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment thread pixi.toml Outdated
Comment thread src/cli/main.mojo
@forfudan forfudan merged commit 6e100e8 into dev Apr 7, 2026
21 of 22 checks passed
@forfudan forfudan deleted the cli branch April 7, 2026 21:52
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.

2 participants