Repository navigation
[Schema Consistency] 🔍 Schema Consistency Check - 2026-10-20 #70443
Closed
Replies: 1 comment
|
This discussion has been marked as outdated by Schema Consistency Checker. A newer discussion is available at Discussion #70798. |
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.
Overview
Found 3 inconsistencies in a focused schema, compiler, documentation, and workflow comparison. No critical issue or workflow violation was confirmed. Strategy: cached schema spot checks with targeted parser/doc/workflow verification (strategy-2); new strategy: no.
Key findings
stop-after.docs/src/content/docs/reference/rate-limiting-controls.md:49shows top-levelstop-after: +48h. The schema rejects this as an additional property;pkg/workflow/stop_after.go:25reads it from theonmap. Move it underon, alongside the trigger. Users copying the example cannot compile it.docs/src/content/docs/reference/mcp-scripts.md:187showschoice-param.enum. The parameter schema atpkg/parser/schemas/main_workflow_schema.json:13893excludesenum; a complete minimal configuration fails validation specifically at that property.pkg/workflow/mcp_scripts_parser.go:158does not extract enum values, andpkg/workflow/mcp_scripts_generator.go:91does not emit them. Remove the unsupported example or implement enum support across schema, types, extraction, and generated input validation together. Merely accepting the field in the schema would not enforce the advertised restriction.pkg/parser/schemas/main_workflow_schema.json:13944says Python input parameters arrive asINPUT_*environment variables.pkg/workflow/mcp_scripts_generator.go:244documents JSON stdin, and line 263 generatesjson.loads(sys.stdin.read()). Correct the schema description to explain the generatedinputsdictionary; explicitenvconfiguration is separate. This affects schema-derived guidance and editor help.Evidence, scope, and exclusions
docker-image-pull-policyis consumed throughRawFrontmatterinpkg/workflow/docker.go:294, andwork-queue-policyinpkg/workflow/work_queue_policy.go:184. Absence from typed frontmatter is insufficient evidence of an unsupported field.stop-afterand MCP-script parameterenum; the corresponding nestedon.stop-afterconfiguration validates. No workflow code was executed.mcp-servers. This focused check does not establish full nested-schema validity for every workflow.degraded: true; canonical config-source retrieval failed. AWF drift conclusions are excluded. No required-check failure, corrective PR, or drift issue is proposed from that comparison.Recommendations and next steps
on.stop-after.Strategy performance: 3 findings; high effectiveness; reuse recommended. No workflow changes are indicated by the evidence collected.
References: §39375393940
All reactions