Skip to content

feat: support CLI default model option#52

Merged
Ark0N merged 1 commit intoArk0N:masterfrom
TeigenZhang:feat/default-model-support
Mar 30, 2026
Merged

feat: support CLI default model option#52
Ark0N merged 1 commit intoArk0N:masterfrom
TeigenZhang:feat/default-model-support

Conversation

@TeigenZhang
Copy link
Copy Markdown
Contributor

Summary

  • Add a "Default (CLI default)" option to the model selection dropdown, allowing sessions to inherit the CLI's own default model instead of forcing a specific one
  • Treat empty string as undefined when passing model to Session, applied consistently across session creation, quick-start, and Ralph

Test plan

  • Open Settings → Model Config → verify "Default (CLI default)" appears as first option
  • Select "Default (CLI default)", create a new session → verify no --model flag is passed to CLI
  • Select a specific model (e.g. Opus), create session → verify --model opus is passed
  • Quick-start a session with default model → verify same behavior
  • Start a Ralph loop with default model → verify no model override

🤖 Generated with Claude Code

Allow users to leave the default model unset, so sessions use whatever
the Claude CLI defaults to rather than forcing a specific model.

- Add empty-value "Default (CLI default)" option to the model dropdown
- Treat empty string as undefined when passing model to Session
- Apply consistently across session creation, quick-start, and Ralph
@Ark0N Ark0N merged commit 5743af3 into Ark0N:master Mar 30, 2026
1 check passed
@Ark0N
Copy link
Copy Markdown
Owner

Ark0N commented Mar 30, 2026

Thanks for the contribution @TeigenZhang! Clean, well-scoped change. The || undefined guards are the right approach to ensure empty string doesn't pass through as a model name.

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