Thanks to visit codestin.com
Credit goes to github.com

Skip to content

fix(chat): add /models and stop /model from dumping full model list #1398

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom
Merged
steipete merged 3 commits into from
Jan 21, 2026
Merged

fix(chat): add /models and stop /model from dumping full model list #1398

steipete merged 3 commits into from
Jan 21, 2026

Conversation

@vignesh07
Copy link
Contributor

@vignesh07 vignesh07 commented Jan 21, 2026 •
edited
Loading

Summary

Chat surfaces currently treat /model (no args) as "list all models", which can dump 500+ lines and is especially painful when users make typos.

This PR:

  • Adds a dedicated /models command for discovery
  • /models lists providers (+ counts)
  • /models <provider> [page] [all] lists models for that provider (default 20/page)
  • Changes /model (no args) to show the current model + guidance (no large lists)
  • Updates /model list to point users at /models
  • Improves fuzzy selection to tolerate small typos (bounded Levenshtein) and applies a minimum score threshold

Notes

  • This only affects chat command/directive handling; the TUI picker remains unchanged.

Tests

pnpm vitest run --reporter dot src/auto-reply/reply/commands-models.test.ts src/auto-reply/reply/directive-handling.model.chat-ux.test.ts src/auto-reply/commands-registry.test.ts

AI Assisted

openai-codex/gpt-5.2

@vignesh07 vignesh07 marked this pull request as draft January 21, 2026 19:57
@vignesh07 vignesh07 marked this pull request as ready for review January 21, 2026 20:20
@steipete steipete self-assigned this Jan 21, 2026
@steipete
Copy link
Contributor

steipete commented Jan 21, 2026

  • P1: /models omits configured non‑catalog/default models; custom provider can be active yet show “Unknown provider.” Map only pulls curated catalog + allowlist keys, not configured defaults/ fallbacks/image models. src/auto-reply/reply/commands-models.ts:100
  • P3: all flag still applies page; /models anthropic 2 all yields empty list (startIndex skips all models). Suggest force page=1 when all=true. src/auto-reply/reply/commands-models.ts:146

steipete added a commit that referenced this pull request Jan 21, 2026
@steipete
fix: improve /models listings and model directive UX (#1398) (thanks @…
f4917c2 
…vignesh07)
@steipete steipete merged commit 529372f into openclaw:main Jan 21, 2026
9 of 22 checks passed
@steipete
Copy link
Contributor

steipete commented Jan 21, 2026 •
edited
Loading

Landed via temp rebase onto main.

  • Gate: pnpm lint && pnpm build && pnpm test
  • Land commit: f4917c2
  • Merge commit: 529372f

Thanks @vignesh07!

Jackten pushed a commit to Jackten/clawdbot that referenced this pull request Jan 29, 2026
@claude
docs: remove numeric model selection references and document /models
fe1b0c7 
- Update `/model` docs to reflect summary view and alias/provider syntax
- Remove numbered picker and `/model <#>` references
- Add `/models` and `/models <provider>` guidance

PR openclaw#1398 (Jan 21, 2026) removed numeric model selection from chat,
but the docs were never updated. Users see `/model 3` in docs but get
"Numeric model selection is not supported in chat" when they try it.

🤖 AI Disclosure: Drafted with Claude (Opus 4.5) assistance.

Co-Authored-By: Claude Opus 4.5 <[email protected]>
@Jackten Jackten mentioned this pull request Jan 29, 2026
Open
2 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

@steipete steipete

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@vignesh07 @steipete