[Schema Consistency] 🔍 Schema Consistency Check - 2026-06-26 #41619
Closed
Replies: 1 comment
-
|
This discussion has been marked as outdated by Schema Consistency Checker. A newer discussion is available at Discussion #41857. |
Beta Was this translation helpful? Give feedback.
0 replies
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Uh oh!
There was an error while loading. Please reload this page.
-
Summary
Critical Issues
engine.extensionsis implemented and documented, but missing from the main frontmatter schema andfrontmatter-full.md.pkg/workflow/engine.go:521readsengine.extensions;pkg/workflow/pi_engine.go:210and:237generate install steps from it.docs/src/content/docs/reference/engines.md:424and glossary documentengine.extensionswith examples.pkg/parser/schemas/main_workflow_schema.jsonengineobject (around line 11729 onward) has noextensionsproperty, anddocs/src/content/docs/reference/frontmatter-full.mdengine section similarly omits it.engine.extensionswill fail schema-based validation/editor assistance despite being supported at compile/runtime.Engine catalog/runtime supports
antigravity, but schema/docs still describe the built-in engine set as excluding it.pkg/workflow/antigravity_engine.go:20-24registers engine idantigravity..github/workflows/smoke-antigravity.md:36-37usesengine.id: antigravity.pkg/parser/schemas/main_workflow_schema.jsonengine descriptions list built-ins asclaude, codex, copilot, gemini, opencode, crush, pi;docs/src/content/docs/reference/frontmatter-full.md:2093-2100,2421-2422, anddocs/src/content/docs/reference/engines.mdengine comments use the same list.pkg/workflow/engine_definition.go:18stale comment also omits bothpiandantigravity.antigravityis not a built-in option even though repo workflows and runtime support it; schema remains too lax (type: string) yet its descriptive guidance is outdated.Parser accepts
safe-outputs.dispatch-repository, but schema/docs only allowdispatch_repository.pkg/workflow/dispatch_repository.go:30-39explicitly accepts bothdispatch_repositoryanddispatch-repository.pkg/parser/schemas/main_workflow_schema.json:9157,docs/src/content/docs/reference/safe-outputs.md:1343+, andfrontmatter-full.md:7508document onlydispatch_repository.Documentation Gaps
frontmatter-full.mdis missingengine.extensionseven though the dedicated engines reference documents it.antigravitymissing; one code comment also missespi).Schema Improvements Needed
engine.extensionsto theengineobject branch inpkg/parser/schemas/main_workflow_schema.jsonwithtype: array,items: {"type":"string"}, and Pi-specific description.antigravity(and keeppi).dispatch-repositoryis a supported alias:Parser Updates Required
engine.extensions.pkg/workflow/engine_definition.goso code comments match runtime registration.Workflow Violations
.github/workflows/smoke-antigravity.mduses a real runtime-supported engine id that current schema/docs do not list as built-in guidance. This is not a runtime failure today, but it is a documentation/schema drift signal inside the repository itself.Recommendations
engine.extensionsto both schema andfrontmatter-full.md, reusing the wording already present indocs/src/content/docs/reference/engines.md.antigravity/pi.dispatch_repositoryvsdispatch-repositorynaming split with a single supported public spelling and a clear deprecation path if needed.Strategy Performance
Next Steps
engine.extensionsReferences:
.github/workflows/smoke-antigravity.md,pkg/workflow/antigravity_engine.go,pkg/workflow/engine.gopkg/parser/schemas/main_workflow_schema.jsondocs/src/content/docs/reference/frontmatter-full.md,docs/src/content/docs/reference/engines.md,docs/src/content/docs/reference/safe-outputs.mdBeta Was this translation helpful? Give feedback.
All reactions