Skip to content

Add 'augur test models' command alias with improved argument handling#125

Merged
ManuelKugelmann merged 1 commit into
mainfrom
claude/augur-ops-shortcuts-EkUjw
Mar 19, 2026
Merged

Add 'augur test models' command alias with improved argument handling#125
ManuelKugelmann merged 1 commit into
mainfrom
claude/augur-ops-shortcuts-EkUjw

Conversation

@ManuelKugelmann
Copy link
Copy Markdown
Owner

Summary

This change adds support for the augur test models command as an alternative to the existing augur testmodels command, while maintaining backward compatibility and improving argument handling.

Key Changes

  • Added test as an alternative command trigger alongside testmodels for the model testing functionality
  • Implemented validation to ensure augur test is only used with the models subcommand, displaying usage information if called incorrectly
  • Updated argument passing logic to correctly handle the different command formats:
    • augur testmodels [args] passes arguments starting from position 2 (${@:2})
    • augur test models [args] passes arguments starting from position 3 (${@:3}) to skip the "models" keyword
  • Maintained full backward compatibility with the existing augur testmodels command

Implementation Details

  • The validation check ensures users get clear feedback if they attempt to use augur test without the models subcommand
  • Conditional logic branches based on which command format was used to correctly offset the argument array when invoking the Python test script
  • All existing functionality and configuration file requirements remain unchanged

https://claude.ai/code/session_01LEpw5r69g77Gow7vjvNDXJ

The case pattern "test models" never matched because CMD only captures $1.
Changed to match on 'test' and check $2 for 'models' subcommand, consistent
with how other two-word commands like 'safemode on/off' work.

https://claude.ai/code/session_01LEpw5r69g77Gow7vjvNDXJ
@ManuelKugelmann ManuelKugelmann merged commit 4f836ca into main Mar 19, 2026
9 checks passed
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