Skip to content

feat: enables schema store support for the json language server - #325970

Open
Vincent Biret (baywet) wants to merge 3 commits into
microsoft:mainfrom
baywet:feat/schema-store
Open

feat: enables schema store support for the json language server#325970
Vincent Biret (baywet) wants to merge 3 commits into
microsoft:mainfrom
baywet:feat/schema-store

Conversation

@baywet

Copy link
Copy Markdown
Member

Copilot AI review requested due to automatic review settings July 15, 2026 14:45

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.

Pull request overview

Adds SchemaStore catalog support to JSON language features.

Changes:

  • Adds an online-service setting for SchemaStore.
  • Passes the setting to the JSON language service.
  • Refreshes language status after schema diagnostics.

Reviewed changes

Copilot reviewed 5 out of 5 changed files in this pull request and generated 1 comment.

Show a summary per file
File Description
jsonServer.ts Configures SchemaStore support.
package.nls.json Adds the setting description.
package.json Registers the SchemaStore setting.
languageStatus.ts Exposes manual status refresh.
jsonClient.ts Synchronizes settings and refreshes status.

validate: validateEnabled,
allowComments: true,
schemas: new Array<SchemaConfiguration>()
schemaStore: { enable: schemaStoreEnabled },

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

that's expected for now, we'll need to merge/publish the related PR, and update the reference here.

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>

Copilot-Session: c5e9b306-5d8c-4cc8-ac55-1b1175cfcedf
Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>

Copilot-Session: c5e9b306-5d8c-4cc8-ac55-1b1175cfcedf
@aeschli

Copy link
Copy Markdown
Contributor

Th PR touches many things that should not be necessary. Only change needed in getSchemaCatalogAssociations
(I cleaned that up in #327104)

Note that in jsonClient we can use node or fetch. So the http fetching needs t be delegated to Runtime.schemaRequests)

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.

4 participants