Skip to content

fix(cli): accept model prefixes without breaking utility commands - #62423

Closed
YangLiuWillow wants to merge 1 commit into
anomalyco:v1from
YangLiuWillow:global-model-option
Closed

YangLiuWillow wants to merge 1 commit into
anomalyco:v1from
YangLiuWillow:global-model-option

Conversation

@YangLiuWillow

@YangLiuWillow YangLiuWillow commented Nov 6, 2026 •

Copy link
Copy Markdown

Launchers that prefix --model provider/model cannot pass utility commands such as export through OpenCode. A config default alone also changes resumed-session model selection. Inherit the long model option globally, share its definition with model-driving commands, and let a later explicit model override the prefix. Keep existing command-local -m meanings.

Fixes #62417. Downstream reproduction: https://github.com/liszt-ai/etude/issues/4032.

Validated on Bun 1.4.4 / Node 24.15.0: 16 tests passed (2 model-option, 13 run-process, 1 help; 34 snapshots), package typecheck, Prettier and default changed-file lint. New files pass strict lint with zero warnings. The 20 legacy warnings also occur on unpatched v1. TUI/agent-create coverage is source/help only; live coverage exercises run, resume/fork, export, explicit overrides and literal arguments.

The downstream Etude pin remains unchanged until a corrected v1 artifact is released and its digest verified.

@github-actions github-actions Bot added the needs:compliance This means the issue will auto-close after 2 hours. label Nov 6, 2026
@github-actions

github-actions Bot commented Nov 6, 2026

Copy link
Copy Markdown
Contributor

This PR doesn't fully meet our contributing guidelines and PR template.

What needs to be fixed:

  • PR description is missing required template sections. Please use the PR template.

Please edit this PR description to address the above within 72 hours, or it will be automatically closed.

If you believe this was flagged incorrectly, please let a maintainer know.

@YangLiuWillow
YangLiuWillow marked this pull request as ready for review November 6, 2026 20:44
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

needs:compliance This means the issue will auto-close after 2 hours.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant