Analyzed GitHub issue #844 root causePublic

Agent

Model

GPT-5.2 Codex

Started

Jan 19, 1:22 PM

Messages49

Downloads0

Summary

In Progress

For each issue:

Read the issue in full, including all comments and linked issues/PRs.
For bugs:
- Ignore any root cause analysis in the issue (likely wrong)
- Read all related code files in full (no truncation)
- Trace the code path and identify the actual root cause
- Propose a fix
For feature requests:
- Read all related code files in full (no truncation)
- Propose the most concise implementation approach
- List affected files and changes needed

Do NOT implement unless explicitly asked. Analyze and propose only.

Issue 844 (bug)

Summary of code path and root cause

/scoped-models persists selections via SettingsManager.setEnabledModels in packages/coding-agent/src/modes/interactive/interactive-mode.ts.
On startup, packages/coding-agent/src/main.ts sets modelPatterns = parsed.models ?? settingsManager.getEnabledModels() and resolves them into scopedModels, which are passed into createAgentSession.
AgentSession exposes these scoped models via session.scopedModels.
/model uses session.scopedModels as the candidate list in:
- InteractiveMode.getModelCandidates and showModelSelector
- ModelSelectorComponent.loadModels
- Autocomplete for /model in setupAutocomplete
As soon as settings enabledModels exist, session.scopedModels is non-empty, so /model only shows the scoped list and the UI hints “Showing models from --models scope.” There is no path to show all models.

Proposed fix

Decouple “model cycling scope” from /model selection. /model should always list all available models, regardless of scoped models saved via /scoped-models.
Keep scoped models only for Ctrl+P cycling and for the startup “Model scope” message.
Update the model selector hint to reflect this, for example “Only showing models with configured API keys. Model cycling is scoped to N models.”

Most concise implementation approach

In packages/coding-agent/src/modes/interactive/interactive-mode.ts:
- setupAutocomplete: always use session.modelRegistry.getAvailable() for /model completions.
- getModelCandidates: always return modelRegistry.getAvailable(), not session.scopedModels.
- showModelSelector: pass only the model registry and current model. Do not pass scoped models as a filter.
In packages/coding-agent/src/modes/interactive/components/model-selector.ts:
- Remove scopedModels filtering from loadModels.
- Update hint text to reflect availability and optional cycling scope.
No changes needed in settings or session logic unless you want to add a dedicated “scope source” flag later.

Affected files

No linked issues or PRs were referenced in the issue.