Skip to content

fix(chat): add JSON schema type and validation for chatLanguageModels configuration#317881

Merged
vritant24 merged 2 commits into
mainfrom
dev/vritant24/chatLanguageModels-intellisense
May 21, 2026
Merged

fix(chat): add JSON schema type and validation for chatLanguageModels configuration#317881
vritant24 merged 2 commits into
mainfrom
dev/vritant24/chatLanguageModels-intellisense

Conversation

@vritant24
Copy link
Copy Markdown
Member

Adds JSON schema associations for chatLanguageModels.json profile files so the language models configuration file receives schema-backed IntelliSense, validation, and completions when opened directly.

Also normalizes native BYOK provider configuration schemas in the Copilot extension by declaring their API-key configuration blocks as explicit objects.

Copilot AI review requested due to automatic review settings May 21, 2026 23:02
@vritant24 vritant24 self-assigned this May 21, 2026
@vritant24 vritant24 enabled auto-merge May 21, 2026 23:04
Copy link
Copy Markdown
Contributor

Copilot AI left a comment

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 improves JSON schema support for the chatLanguageModels.json user/profile configuration file and normalizes Copilot BYOK provider configuration schemas so they validate as explicit objects.

Changes:

  • Adds JSON schema associations so chatLanguageModels.json (default + per-profile) gets IntelliSense/validation when opened directly.
  • Updates Copilot extension BYOK provider configuration schemas (Anthropic, xAI, Google Gemini, OpenRouter, OpenAI) to explicitly declare type: "object".
Show a summary per file
File Description
extensions/copilot/package.json Declares BYOK provider configuration blocks as JSON schema objects to improve validation/consistency.
extensions/configuration-editing/package.json Associates chatLanguageModels.json (default and profiles) with vscode://schemas/language-models for schema-backed editing.

Copilot's findings

  • Files reviewed: 2/2 changed files
  • Comments generated: 0

@vritant24 vritant24 merged commit 3a54038 into main May 21, 2026
25 checks passed
@vritant24 vritant24 deleted the dev/vritant24/chatLanguageModels-intellisense branch May 21, 2026 23:43
@vs-code-engineering vs-code-engineering Bot added this to the 1.122.0 milestone May 21, 2026
@vritant24 vritant24 mentioned this pull request May 26, 2026
1 task
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.

3 participants