Skip to content

fix(workbench): require models when adding providers - #458

Open
lucas77778 wants to merge 1 commit into
masterfrom
chenyu/code-589
Open

fix(workbench): require models when adding providers#458
lucas77778 wants to merge 1 commit into
masterfrom
chenyu/code-589

Conversation

@lucas77778

Copy link
Copy Markdown
Member

Summary

  • require at least one selected model before manually adding OAuth, catalog API, gateway, or custom endpoint providers
  • persist the selected models on every newly created account and automatically probe/select all models returned by LinkCode Gateway
  • keep local auto-detected providers unchanged: they already select every model in the harness's curated model set

Linear: CODE-589

Verification

  • pnpm check:ci
  • pnpm test — 2964 passed, 1 skipped
  • provider add-flow tests — 15 passed
  • exercised the provider form in the running webview and recorded the empty-selection validation followed by successful model selection and provider creation

Checklist

  • pnpm check:ci and pnpm test both pass (plus cargo fmt / clippy / test for Rust changes)
  • I ran the affected surface and observed the change working
  • If a wire message changed: WIRE_PROTOCOL_VERSION is bumped (not applicable; no wire change)
  • New code and assets are my own work, or their origin and license compatibility are noted above
  • Docs and comments are updated where behavior changed (not applicable; UI copy and tests updated)

Copilot AI lite review requested due to automatic review settings August 18, 2026 08:56
@linear-code

linear-code Bot commented Aug 18, 2026

Copy link
Copy Markdown

CODE-589

@pullfrog

pullfrog Bot commented Aug 18, 2026

Copy link
Copy Markdown

Your Claude subscription has hit its usage limit. It resets at 11am (UTC). Re-trigger Pullfrog after the reset, or add an ANTHROPIC_API_KEY repo secret — Pullfrog routes around an exhausted subscription automatically when one is present.

Add repo secret → · Model settings → · Setup docs → · Ask in Discord →

Pullfrog  | Rerun failed job ➔View workflow run | via Pullfrog | Using Claude Opus𝕏

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Copilot was unable to review this pull request because the user who requested the review has reached their quota limit.

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