Skip to content

Add argument group support#25

Open
BlakeASmith wants to merge 6 commits intomainfrom
cursor/add-argument-group-support-a28d
Open

Add argument group support#25
BlakeASmith wants to merge 6 commits intomainfrom
cursor/add-argument-group-support-a28d

Conversation

@BlakeASmith
Copy link
Contributor

Add support for argument groups and mutually exclusive groups to enhance CLI organization and enforce argument constraints.


Open in Cursor Open in Web

Co-authored-by: blakeinvictoria <blakeinvictoria@gmail.com>
@cursor
Copy link

cursor bot commented Aug 26, 2025

Cursor Agent can help with this pull request. Just @cursor in comments and I'll start working on changes in this branch.
Learn more about Cursor Agents

cursoragent and others added 5 commits August 26, 2025 14:53
Co-authored-by: blakeinvictoria <blakeinvictoria@gmail.com>
Co-authored-by: blakeinvictoria <blakeinvictoria@gmail.com>
Co-authored-by: blakeinvictoria <blakeinvictoria@gmail.com>
…ations

Co-authored-by: blakeinvictoria <blakeinvictoria@gmail.com>
Co-authored-by: blakeinvictoria <blakeinvictoria@gmail.com>
@BlakeASmith BlakeASmith marked this pull request as ready for review August 26, 2025 22:51
Copy link

@cursor cursor bot left a comment

Choose a reason for hiding this comment

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

Bugbot free trial expires on September 9, 2025
Learn more in the Cursor dashboard.

echo "Verbose: $VERBOSE"
echo "Quiet: $QUIET"
"""

Copy link

Choose a reason for hiding this comment

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

Bug: Bracket Parsing Ignores Group Annotations

The parse_arg_annotations function's bracket parsing logic only extracts alias from annotations, silently ignoring [group: ...] and [exclusive_group: ...] declarations. This means tests like test_integration_script_with_groups that use this bracket syntax for groups will not have their groups parsed, leading to unexpected behavior or silent failures where group functionality isn't truly tested.

Additional Locations (1)

Fix in Cursor Fix in Web

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