Skip to content

Add adapted resource schema files - #1461

Merged
Mikey Lombardi (He/Him) (michaeltlombardi) merged 3 commits into
PowerShell:mainfrom
Gijsreyn:gh-1460/main/add-adaptedresource-schema
Jul 6, 2026
Merged

Add adapted resource schema files#1461
Mikey Lombardi (He/Him) (michaeltlombardi) merged 3 commits into
PowerShell:mainfrom
Gijsreyn:gh-1460/main/add-adaptedresource-schema

Conversation

@Gijsreyn

@Gijsreyn Gijs Reijn (Gijsreyn) commented Apr 3, 2026

Copy link
Copy Markdown
Collaborator

PR Summary

This pull request adds the adapted resource schema files. I can remember from #1083 that it wasn't required to generate the .json files.

PR Context

Partially fixes #1460.

Copilot AI review requested due to automatic review settings April 3, 2026 06:25

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 schema sources for adapted resource manifests to the repository’s schema set, and wires the new manifest schema into the schema bundling configuration (to support publishing bundled/enhanced variants alongside existing resource/extension schemas).

Changes:

  • Added schemas/src/adaptedresource/manifest.yaml describing the adapted resource manifest shape and authoring helpers.
  • Added schemas/src/adaptedresource/manifest.schema.yaml defining the embedded instance schema shape for adapted resources.
  • Updated schemas/schemas.config.yaml to generate bundled (Json + JsonVSCode) outputs for the adapted resource manifest schema.

Reviewed changes

Copilot reviewed 3 out of 3 changed files in this pull request and generated 4 comments.

File Description
schemas/src/adaptedresource/manifest.yaml New adapted resource manifest schema (incl. docs/snippets and $schema URI set).
schemas/src/adaptedresource/manifest.schema.yaml New schema describing the embedded instance-schema object used by adapted resources.
schemas/schemas.config.yaml Adds bundling config entry for adapted resource manifest schema outputs.

Comment thread schemas/src/adaptedresource/manifest.yaml Outdated
Comment thread schemas/src/adaptedresource/manifest.yaml
Comment thread schemas/schemas.config.yaml
Comment thread schemas/schemas.config.yaml
@Gijsreyn
Gijs Reijn (Gijsreyn) force-pushed the gh-1460/main/add-adaptedresource-schema branch from d041afa to 577582c Compare July 3, 2026 02:34

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

I'm approving this for merge to avoid slowing things down further, but I think in a future PR we should probably start decomposing the schemas for more references instead of duplication. A lot of these fields are now shared across resource/extension/adapted resource manifests.

Merged via the queue into PowerShell:main with commit 951b032 Jul 6, 2026
20 checks passed
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.

Adapted Resource Manifest Schema

3 participants