Skip to content

fix: preserve Copilot catalog variants#30143

Closed
he-yufeng wants to merge 1 commit into
anomalyco:devfrom
he-yufeng:fix/copilot-preserve-static-variants
Closed

fix: preserve Copilot catalog variants#30143
he-yufeng wants to merge 1 commit into
anomalyco:devfrom
he-yufeng:fix/copilot-preserve-static-variants

Conversation

@he-yufeng
Copy link
Copy Markdown

Summary

  • keep existing catalog variants when refreshing GitHub Copilot models from the /models endpoint
  • still let the endpoint-provided variants override matching catalog keys
  • keep message-API models on their provider-specific variant path so stale Copilot variants are not carried across

Fixes #30092.

To verify

  • bun --cwd packages\opencode test test\plugin\github-copilot-models.test.ts
  • bunx prettier --check packages\opencode\src\plugin\github-copilot\models.ts packages\opencode\test\plugin\github-copilot-models.test.ts
  • bun --cwd packages\opencode typecheck
  • bunx oxlint packages\opencode\src\plugin\github-copilot\models.ts packages\opencode\test\plugin\github-copilot-models.test.ts

@github-actions github-actions Bot added the needs:compliance This means the issue will auto-close after 2 hours. label May 31, 2026
@github-actions
Copy link
Copy Markdown
Contributor

This PR doesn't fully meet our contributing guidelines and PR template.

What needs to be fixed:

  • PR description is missing required template sections. Please use the PR template.

Please edit this PR description to address the above within 2 hours, or it will be automatically closed.

If you believe this was flagged incorrectly, please let a maintainer know.

@rekram1-node
Copy link
Copy Markdown
Collaborator

This is expected, we trust the api here.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

needs:compliance This means the issue will auto-close after 2 hours.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[BUG]: fix(github-copilot): xhigh reasoning variant missing from GPT-5.x models

2 participants