Repository navigation
[Schema Consistency] 🔍 Schema Consistency Check - 2026-11-13 #77360
Closed
Replies: 1 comment
|
This discussion has been marked as outdated by Schema Consistency Checker. A newer discussion is available at Discussion #77667. |
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
Key findings
Daily AI-credit default is documented as disabled, but the compiler supplies 5,000. The schema description (main_workflow_schema.json:5028) and frontmatter reference:578 say omission disables the guardrail. Instead, daily_aic_workflow.go:208 falls back to an Actions variable and the 5,000-credit constant, matching the enterprise controls reference. Workflows omitting the field can therefore experience unexpected admission checks or skips. Update the schema and frontmatter reference to describe precedence and explicit
-1disabling, and test that the documented default matches compilation.The schema accepts
max-tool-denials: 0, but the compiler drops it. The field references the non-negative templatable integer. The parser requires a minimum of 1 and returns an empty value for zero; support validation then treats it as unset. An author can pass schema validation while the requested threshold is ignored. Use the positive-integer schema and reject zero explicitly, with a schema-to-compiler regression test.Generated frontmatter examples violate their own schema. The generated reference shows empty
connectors: {}, despite the minimum of one property; it also shows an invalid source SHA and invalid expression placeholders. The generator falls back toexample-valuewithout respecting patterns. Generate valid representative values and validate individual alternatives against their schema in generator tests.Glossary excludes supported Pi tool limits. The denial entry says Copilot SDK only, and the calls entry excludes other engines. Compiler validation and the Pi reference support both limits for built-in Pi CLI, SDK and RPC drivers. The unsupported-engine error also incorrectly says denials are Copilot-only. Align the glossary and error message with supported Pi drivers.
Scope, workflow usage, and limitations
skip-if-decisionand explicit daily credit limits; this was a targeted review, not a full workflow validation.FrontmatterConfigare not automatically defects: targeted code inspection found raw-frontmatter handling for connectors, Docker pull policy, skip decisions, and the deprecated work-queue policy.Next actions
Strategy performance: 4 actionable findings; high effectiveness; reuse for constrained-value and generated-example checks.
References: §43371724452
All reactions