[AAASM-4656] ✨ (cli): Add aasm --version and fix README error-handling example#260
Conversation
Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01HMnPRm9T3fdrS4uNYXkzg6
Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01HMnPRm9T3fdrS4uNYXkzg6
The example implied an empty gateway_url raises ConfigurationError; in reality an empty/unset gateway_url triggers local auto-discovery. Show a genuinely-invalid config (bad mode) that validation rejects up front, and note the empty/unset behavior. Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01HMnPRm9T3fdrS4uNYXkzg6
Codecov Report✅ All modified and coverable lines are covered by tests. 📢 Thoughts on this report? Let us know! |
|
🤖 Claude Code — PR reviewVerdict: ✅ Ready to approve & merge. 1. CI: Green — unit + integration tests pass (3.13/ubuntu), 2. Scope vs AAASM-4656: Both AC items covered — (a) 3. Side effects: None. 4. Front-end: N/A. No changes requested. |



Description
Two small doc/UX fixes for the
aasmCLI and the README:aasm --version— previously argparse rejected--version(exit 2). Added an argparseaction="version"flag that printsaasm <version>sourced fromagent_assembly.__version__and exits 0.gateway_url=""inside atry/except ConfigurationError, implying an emptygateway_urlraisesConfigurationError. It does not:resolve_gateway_urltreats an empty/unset value as unset and falls through to local auto-discovery (probe localhost, then auto-start), which surfacesGatewayError(orConfigurationErroronly if theaasmbinary is absent). Rewrote the example to demonstrateConfigurationErrorwith a genuinely-invalid config (an invalidmode), which validation rejects up front before any network I/O, and added a note on the real empty/unset behavior.Type of Change
Breaking Changes
Related Issues
Testing
Describe the testing performed for this PR:
TestCliMainVersionassertsaasm --versionprints the expected version and exits 0..venv/bin/aasm --versionprintsaasm 0.0.1rc5, exit 0.ruff check/ruff format --checkclean on changed files;mypy agent_assembly/cli/main.pyclean; CLI test suite passes (4 passed).ci.yamlpaths).Checklist
🤖 Generated with Claude Code
https://claude.ai/code/session_01HMnPRm9T3fdrS4uNYXkzg6