Repository navigation
[Schema Consistency] Schema consistency check — 2026-11-09 #76152
Closed
Replies: 1 comment
|
This discussion has been marked as outdated by Schema Consistency Checker. A newer discussion is available at Discussion #76412. |
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.
🔍 Schema consistency check — 2026-11-09
Four actionable inconsistencies were found in a targeted comparison of the schema, compiler, documentation, and repository workflows. Strategy: precomputed field-gap triage with targeted validation and documentation checks (reused). No critical implementation defect was established.
AWF comparison limitation: Canonical AWF configuration sources were unavailable, so the precomputed AWF comparison is degraded. This is a non-authoritative warning; no AWF drift finding is asserted.
Findings and evidence
Schema accepts skip questions that the compiler rejects.
pkg/parser/schemas/main_workflow_schema.json:10permits any nonblank string, including a question containing the literal${{marker.pkg/workflow/skip_if_decision.go:25rejects that marker, anddocs/src/content/docs/reference/frontmatter.md:89documents the restriction. Schema-based editors can therefore accept input that fails compilation. Add a schema constraint excluding the marker and a regression case matching the compiler rule.Sandbox guide misstates the timeout scope.
docs/src/content/docs/reference/sandbox.md:359and:363describe top-leveltimeout-minutesas the entire agent job timeout with a 20-minute default.docs/src/content/docs/reference/frontmatter.md:485anddocs/src/content/docs/reference/steps-jobs.md:257distinguish the agentic execution step, normally 20 minutes, from the agent job, normally 60 minutes. Correct the guide and explainjobs.agent.timeout-minutesfor setup and post-processing, including the documented adjustment for longer literal step budgets.Reusable timeout example splits its frontmatter. The example at
docs/src/content/docs/reference/sandbox.md:386places---betweenon.workflow_callandtimeout-minutes, rather than surrounding both fields with opening and closing frontmatter delimiters. A reader copying it as a workflow will not get the intended combined frontmatter. Move the delimiter to the beginning and add a closing delimiter after the timeout, with a workflow body if presenting a complete file.MCP scripts schema description omits Go.
pkg/parser/schemas/main_workflow_schema.json:14319lists only JavaScript, shell, and Python and says one of those three is required, while the same schema definesgoat:14390and the reference atdocs/src/content/docs/reference/mcp-scripts.md:65correctly includes it. Add Go to the parent description and its exactly-one list so schema-generated guidance matches the accepted options.Workflow review: The precomputed usage diff reports no used top-level keys missing from the schema. Sampled
skip-if-decisionworkflows, including.github/workflows/daily-graft-intelligence.md:18, use the documented list form. The three top-level fields absent from the typed frontmatter structure are handled through raw frontmatter; their absence from that struct alone is not a defect. This targeted review did not validate every workflow or execute workflows.Strategy performance: High for separating false-positive field gaps from actionable cross-layer mismatches; reuse with focused source checks.
Next actions
References: §42567100532
All reactions