Skip to content

fix(provider): mark DeepSeek models as BYOK#162

Merged
Vizards merged 1 commit into
mainfrom
fix/mark-deepseek-models-byok
Jun 16, 2026
Merged

fix(provider): mark DeepSeek models as BYOK#162
Vizards merged 1 commit into
mainfrom
fix/mark-deepseek-models-byok

Conversation

@Vizards

@Vizards Vizards commented Jun 15, 2026

Copy link
Copy Markdown
Owner

Summary

  • Mark DeepSeek language model metadata with isBYOK: true so new VS Code BYOK-aware logic classifies the provider correctly.
  • Keep the local non-public metadata typing in sync with the runtime shape.

Why

VS Code now distinguishes BYOK models from Copilot/CAPI models using LanguageModelChatInformation.isBYOK. DeepSeek requests are served with the user supplied API key, so the provider should opt into that signal.

Upstream VS Code PR: microsoft/vscode#321090

Validation

  • npm run format:check
  • npm run compile

@Vizards Vizards marked this pull request as ready for review June 15, 2026 16:25
@Vizards Vizards requested a review from Copilot June 15, 2026 16:25

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Pull request overview

This PR updates the DeepSeek chat model metadata returned to VS Code so the models are explicitly classified as BYOK (Bring Your Own Key) via LanguageModelChatInformation.isBYOK, aligning with new VS Code logic that differentiates BYOK from Copilot/CAPI-backed models.

Changes:

  • Extend the local ModelPickerChatInformation metadata type to include isBYOK.
  • Set isBYOK: true on the model chat info objects returned by toChatInfo.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@Vizards Vizards merged commit 50ff6ba into main Jun 16, 2026
2 checks passed
@Vizards Vizards deleted the fix/mark-deepseek-models-byok branch June 16, 2026 03:22
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