Skip to content

fix: show available catalog models in chat picker#1046

Open
AWL-Gaming wants to merge 1 commit into
openclaw:mainfrom
AWL-Gaming:fix/chat-model-catalog
Open

fix: show available catalog models in chat picker#1046
AWL-Gaming wants to merge 1 commit into
openclaw:mainfrom
AWL-Gaming:fix/chat-model-catalog

Conversation

@AWL-Gaming

Copy link
Copy Markdown

Summary

  • Load the configured model list first so the existing safe choices remain immediately available.
  • Query the broader model catalog and append available or auth-needed entries to the chat model picker.
  • Keep catalog-only rows visible but disabled with a not configured marker, preventing model not allowed failures.
  • Persist only selectable model IDs in the legacy reconnect fallback.

Validation

  • dotnet build .\src\OpenClaw.Tray.WinUI\OpenClaw.Tray.WinUI.csproj -c Debug -r win-x64 --no-restore: passed, 0 warnings, 0 errors.
  • dotnet test .\tests\OpenClaw.Shared.Tests\OpenClaw.Shared.Tests.csproj --no-restore --filter FullyQualifiedName~ModelCatalogMergeTests: passed 2/2.
  • dotnet test .\tests\OpenClaw.Tray.Tests\OpenClaw.Tray.Tests.csproj --no-restore: passed 1917/1917.
  • Full shared test suite: 3171 passed, 31 skipped, 1 unrelated environment-specific failure in TokenSanitizerTests.SanitizeLogMessage_RedactsCurrentUserAndMachineNames because this machine's host name equals its user name.
  • git diff --check: passed.
  • Committed SHA: aaf8dea1fd853417736713808afbb25282a08fe8
  • git log -1 --oneline: aaf8dea1 fix: show available catalog models in chat picker

Real behavior proof

  • Environment tested: Windows 10, OpenClaw gateway 2026.7.2-beta.1.
  • Before: the live models.list configured view returns 5 models, exactly matching the tray's limited dropdown.
  • Live catalog observation: the configured view returns 5 models while the all view returns 93 catalog entries, including additional available OpenAI models.
  • Current-head behavior: configured rows are emitted immediately, then merged with available catalog rows. Catalog-only rows remain visible but disabled and labeled not configured.
  • Focused tests verify provider-qualified deduplication, configured-first ordering, disabled catalog-only rows, and reconnect safety.
  • Screenshot or artifact links verified? N/A.
  • Not verified or blocked: the source build was not installed over the existing production tray 0.6.12, because current upstream main contains unrelated newer changes. Replacing the live binary before review would mix this fix with a broader version upgrade.

Configuration changes

  • None.

Checklist

  • Builds successfully.
  • Focused and tray test suites pass.
  • Existing configured models remain immediately selectable.
  • Catalog-only models cannot trigger unsupported model switches.
  • No credentials, tokens, or configuration files changed.

@clawsweeper

clawsweeper Bot commented Jul 22, 2026

Copy link
Copy Markdown

ClawSweeper status: review started.

I am starting a fresh review of this pull request: fix: show available catalog models in chat picker This is item 1/1 in the current shard. Shard 0/1.

This placeholder means the worker is alive and reading the current context. I will edit this same comment with the actual review when the claws are done clicking.

Crustacean status: shell secured, claws on keyboard, evidence pebbles being sorted.

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.

1 participant