Skip to content

chore(hydra-gates): vendor manifest schema 2.41.0 - #794

Open
github-actions[bot] wants to merge 1 commit into
mainfrom
chore/vendored-manifest-schema-2.41.0
Open

github-actions[bot] wants to merge 1 commit into
mainfrom
chore/vendored-manifest-schema-2.41.0

Conversation

@github-actions

Copy link
Copy Markdown
Contributor

The vendored manifest schema was at 2.37.0. @conduction/nextcloud-vue 2.57.1 ships 2.41.0.

Gates 22 and 53 validate every app's manifest against the vendored copy, so until this merges an app that adopts a key 2.41.0 added is rejected by the gate while its own npm run check:manifest passes.

Before merging

CI resolves these gates at @main, so this reaches all 21 swept apps the minute it lands. Constraints that got stricter in this diff:

/$defs/page/allOf[0]/then/not/anyOf[2]/required: newly required ["savedViewTree"]
/$defs/page/allOf[0]/then/not/anyOf[3]/required: newly required ["board"]
/$defs/page/allOf[0]/then/not/anyOf[4]/required: newly required ["dateAxis"]
/$defs/page/properties/savedViewPlaces: property removed while additionalProperties is false, so the key is now refused
/$defs/widgetEntry/allOf[10]/if/properties/widgetKey/enum: enum no longer allows ["stat","delta"]
/$defs/widgetEntry/allOf[1]/if/required: newly required ["ncDashboard"]
/$defs/widgetEntry/allOf[1]/then/required: newly required ["id"]
/$defs/widgetEntry/allOf[3]/if/required: newly required ["slot"]
/$defs/widgetEntry/allOf[5]/then/properties/props/required: newly required ["entries"]
/$defs/widgetEntry/allOf[5]/then/required: newly required ["props"]
/$defs/widgetEntry/allOf[9]/if/properties/widgetKey/const: new const constraint "chart"

none means the diff is additive and cannot redden a manifest that passes today. Anything else means it can, and the fleet's effective manifests need validating against both schemas first.

Opened by .github/workflows/hydra-gates-schema-sync.yml. See #785 for why the vendored copy needs a keeper.

This branch has not been deployed

No deployments
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.

0 participants