建议添加供应商搜索

[046081-dotcom]

随着供应商越来越多，寻找供应商变得不那么方便了

[clawsweeper]

Codex review: keeping this open for maintainer follow-up; there is still a little grit to resolve. Reviewed May 27, 2026, 3:05 AM ET / 07:05 UTC.

Summary
Keep open: current main and the latest release still show the Providers pane as an unfiltered ordered list, so the requested provider search remains useful but needs maintainer sign-off as a small new UI feature.

Reproducibility: not applicable. as a bug reproduction; this is a provider-settings UX feature request. Source inspection confirms the current Providers pane has no search or filter state.

Ways to help us reproduce this

• Add a screenshot or short recording showing the behavior.
• Add expected vs actual behavior.

Next step
A fix shape is clear, but VISION.md treats new features as sign-off work, so maintainers should approve the provider-search UX before a repair lane implements it.

Review details

Best possible solution:

Add a localized search field above the Providers sidebar that filters by provider display name and raw id while preserving existing order, selection, enable toggles, and drag reordering behavior.

Do we have a high-confidence way to reproduce the issue?

Not applicable as a bug reproduction; this is a provider-settings UX feature request. Source inspection confirms the current Providers pane has no search or filter state.

Is this the best way to solve the issue?

Yes, after maintainer sign-off, a narrow Providers-pane search/filter is the most maintainable path. A broader provider-management redesign would be unnecessary for the reported pain point.

AGENTS.md: found and applied where relevant.

Remaining risk / open question:

• The exact UX placement, matching rules, and interaction with drag reordering need maintainer sign-off before implementation.

Codex review notes: model gpt-5.5, reasoning high; reviewed against 79f61e63ac2d.

Label changes

Label changes:

• add P3: This is a low-risk provider settings ergonomics request rather than a regression or broken workflow.
• add impact:auth-provider: The issue concerns finding and selecting providers in the Providers pane as the provider catalog grows.
• add issue-rating: 🌊 off-meta tidepool: Current issue advisory state selects this label.
• add clawsweeper:no-new-fix-pr: Current issue advisory state selects this label.
• add clawsweeper:needs-maintainer-review: Current issue advisory state selects this label.
• add clawsweeper:needs-product-decision: Current issue advisory state selects this label.

Label justifications:

• P3: This is a low-risk provider settings ergonomics request rather than a regression or broken workflow.
• impact:auth-provider: The issue concerns finding and selecting providers in the Providers pane as the provider catalog grows.

Evidence reviewed

What I checked:

• Issue request: The issue asks for supplier/provider search because the growing number of providers makes finding one inconvenient; the current GitHub issue is open with only the ClawSweeper placeholder comment.
• Provider pane source: The Providers pane builds its sidebar input from settings.orderedProviders() and passes that full list directly into ProviderSidebarListView, with no search or filter state in this view. (Sources/CodexBar/PreferencesProvidersPane.swift:21, 79f61e63ac2d)
• Sidebar source: ProviderSidebarListView renders every provider in the supplied array via ForEach inside a ScrollView; there is no TextField, searchable modifier, or filtered collection in the sidebar. (Sources/CodexBar/PreferencesProviderSidebarView.swift:15, 79f61e63ac2d)
• Provider count pressure: UsageProvider currently enumerates 48 provider cases, which supports the report that provider discovery is becoming harder. (Sources/CodexBarCore/Providers/Providers.swift:5, 79f61e63ac2d)
• No existing search implementation found: Focused source search found provider filtering in unrelated runtime paths but no provider-list search/filter UI in the Providers pane. (79f61e63ac2d)
• Release/current-main check: The current main delta from v0.30.0 only changes CHANGELOG.md and version.env, and no relevant provider settings files changed, so the latest release and current main both lack this feature. (79f61e63ac2d)

Likely related people:

• Peter Steinberger: The provider pane redesign commit introduced ProviderSidebarListView, and current blame for the provider pane/sidebar source also points to Peter-authored source snapshots in the checked-out history. (role: provider pane introducer and current source owner; confidence: high; commits: 90e608be824b, 81cf30a6d5e1; files: Sources/CodexBar/PreferencesProvidersPane.swift, Sources/CodexBar/PreferencesProviderSidebarView.swift)
• Perry Story: The history search shows Perry's recent performance/refactor commit touched or carried the same provider pane/sidebar files into the current source lineage, though the commit scope was not specifically provider-search UX. (role: recent adjacent contributor; confidence: medium; commits: c2b63a01bae4; files: Sources/CodexBar/PreferencesProvidersPane.swift, Sources/CodexBar/PreferencesProviderSidebarView.swift, Sources/CodexBarCore/Providers/Providers.swift)

How this review workflow works

• ClawSweeper keeps one durable marker-backed review comment per issue or PR.
• Re-runs edit this comment so the latest verdict, findings, and automation markers stay together instead of adding duplicate bot comments.
• A fresh review can be triggered by eligible @clawsweeper re-review comments, exact-item GitHub events, scheduled/background review runs, or manual workflow dispatch.
• PR/issue authors and users with repository write access can comment @clawsweeper re-review or @clawsweeper re-run on an open PR or issue to request a fresh review only.
• Maintainers can also comment @clawsweeper review to request a fresh review only.
• Fresh-review commands do not start repair, autofix, rebase, CI repair, or automerge.
• Maintainer-only repair and merge flows require explicit commands such as @clawsweeper autofix, @clawsweeper automerge, @clawsweeper fix ci, or @clawsweeper address review.
• Maintainers can comment @clawsweeper explain to ask for more context, or @clawsweeper stop to stop active automation.

[Yuxin-Qiao]

This makes sense to me as the provider list keeps growing.

Maybe a small first version could just add a lightweight filter in Settings → Providers: case-insensitive matching by provider display name, empty query shows all providers, and no changes to provider ordering/auth logic.

[046081-dotcom]

This makes sense to me as the provider list keeps growing.

Maybe a small first version could just add a lightweight filter in Settings → Providers: case-insensitive matching by provider display name, empty query shows all providers, and no changes to provider ordering/auth logic.

Yes, I agree.

[steipete]

Implemented and shipped in:

• 4c9e6a8 feat: add provider search
• b3c0d57 docs: update changelog for provider search

The Providers pane now has a localized search field that filters by provider display name or raw provider id, keeps empty search showing all providers, and preserves drag reordering against the full provider order.

Validation run:

• swiftformat Sources Tests
• swift test --filter ProvidersPaneCoverageTests
• make check
• swift test
• autoreview --mode local