Skip to content

fix(args): register --yes and --async in GLOBAL_OPTIONS#41

Closed
EElaineYIN wants to merge 1 commit intomainfrom
fix/register-yes-async-flags
Closed

fix(args): register --yes and --async in GLOBAL_OPTIONS#41
EElaineYIN wants to merge 1 commit intomainfrom
fix/register-yes-async-flags

Conversation

@EElaineYIN
Copy link
Copy Markdown

Summary

  • --yes and --async were defined in GlobalFlags and used in loadConfig but missing from GLOBAL_OPTIONS
  • The parser didn't know they were booleans → consumed the next argument as their value
  • minimax text chat --yes --message "hello" would eat --message

Test plan

  • minimax text chat --yes --message "hello" --dry-run → works (--message not eaten)
  • minimax auth logout --yes → no "requires a value" error

🤖 Generated with Claude Code

These flags were in GlobalFlags and loadConfig but not in GLOBAL_OPTIONS,
causing the parser to treat them as string flags that consume the next arg.

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
@RyanLee-Dev RyanLee-Dev closed this Apr 6, 2026
@RyanLee-Dev RyanLee-Dev deleted the fix/register-yes-async-flags branch April 8, 2026 21:44
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