Skip to content

chore: disable spellcheck in model search#8889

Merged
kalvinnchau merged 1 commit intomainfrom
kalvin/disable-spellcheck-model-selector-1
Apr 29, 2026
Merged

chore: disable spellcheck in model search#8889
kalvinnchau merged 1 commit intomainfrom
kalvin/disable-spellcheck-model-selector-1

Conversation

@kalvinnchau
Copy link
Copy Markdown
Collaborator

@kalvinnchau kalvinnchau commented Apr 29, 2026

Category: fix
User Impact: Users no longer see spelling suggestions while searching model IDs in the desktop app.
Problem: The all-models search field could trigger browser spellcheck even though users are often typing provider-specific model IDs. That made technical search terms look incorrect and added visual noise in the model picker.
Solution: The picker now uses the shared search component, which carries the intended search-input behavior consistently across the UI, and the behavior is covered by a focused test.

File changes

ui/goose2/src/features/chat/ui/AgentModelPicker.tsx
Replaced the custom all-models search input with the shared SearchBar component so the model picker inherits the app's standard search behavior, including disabled spellcheck.

ui/goose2/src/features/chat/ui/tests/AgentModelPicker.test.tsx
Added coverage that opens the all-models view and verifies the search field disables spellcheck.

Reproduction Steps

  1. Open the agent and model picker in the desktop UI.
  2. Select "Browse all models" to open the full model list.
  3. Focus the "Search models..." field and type a model-like identifier such as gpt-4o-mini-2024-07-18.
  4. Confirm the search input does not show spelling suggestions or correction UI.

before:

image

after:

image

@kalvinnchau kalvinnchau enabled auto-merge April 29, 2026 00:21
@kalvinnchau kalvinnchau changed the title disable spellcheck in model search chore: disable spellcheck in model search Apr 29, 2026
@kalvinnchau kalvinnchau added this pull request to the merge queue Apr 29, 2026
Merged via the queue into main with commit fffe425 Apr 29, 2026
23 of 24 checks passed
@kalvinnchau kalvinnchau deleted the kalvin/disable-spellcheck-model-selector-1 branch April 29, 2026 00:47
lifeizhou-ap added a commit that referenced this pull request Apr 30, 2026
* main: (24 commits)
  fix: copy and content improvements in goose2 (#8886)
  feat: make ollama host configurable in goose2 (#8912)
  polish sidebar navigation and project icons (#8896)
  fix: model picker stays usable during provider loading (#8900)
  feat: update provider row after saving credentials (#8914)
  feat: support google model inventory refresh (#8913)
  chore: Added goose 2 UI refactor review skill (#8903)
  blog: goose with peekaboo (#8884)
  blog: Built-in Local Inference blogpost. (#8808)
  perf: parallelize provider resolution and eagerly init SQLite pool (#8899)
  refactor: update goose2 credential management behind provider-scoped ACP/core API (#8887)
  fix: handle acp requests concurrently (#8781)
  build: set LLAMA_STATIC_CRT for Windows CUDA (#8901)
  perf: deduplicate _goose/providers/list RPC call at startup (#8873)
  chore: add a bit more instructions in the release pr (#8890)
  chore: disable spellcheck in model search (#8889)
  add skills to the chat composer (#8881)
  mergeable configs + cleanup (#8378)
  refactor: agent provider to use explicit type states (#8879)
  [goose2] MCP Apps: hydrate and replay app payloads in Goose2 (#8632)
  ...
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