[Schema Consistency] Schema Consistency Check — 2026-06-11 #38531
Replies: 3 comments
-
|
Smoke bot grunt. Run 27329723424 stomp here. Fire still warm. Warning Firewall blocked 6 domainsThe following domains were blocked by the firewall during workflow execution:
network:
allowed:
- defaults
- "accounts.google.com"
- "android.clients.google.com"
- "clients2.google.com"
- "contentautofill.googleapis.com"
- "safebrowsingohttpgateway.googleapis.com"
- "www.google.com"See Network Configuration for more information.
|
Beta Was this translation helpful? Give feedback.
-
|
Smoke ping: discussion comment path works. ✅ Warning Firewall blocked 6 domainsThe following domains were blocked by the firewall during workflow execution:
network:
allowed:
- defaults
- "accounts.google.com"
- "android.clients.google.com"
- "clients2.google.com"
- "contentautofill.googleapis.com"
- "safebrowsingohttpgateway.googleapis.com"
- "www.google.com"See Network Configuration for more information.
|
Beta Was this translation helpful? Give feedback.
-
|
This discussion has been marked as outdated by Schema Consistency Checker. A newer discussion is available at Discussion #38798. |
Beta Was this translation helpful? Give feedback.
Uh oh!
There was an error while loading. Please reload this page.
-
Summary
This run focused on re-verifying the established standing findings from the strategy cache and probing two fresh leads (a deprecation flag and an enum-vs-docs comparison). The headline result is positive: the two strongest open findings have been fixed upstream since the last run, and both fresh leads turned out to be fully consistent.
Critical Issues
None. No correctness-affecting schema/parser mismatches were found.
Resolved Since Last Run ✅
These were long-standing findings in the strategy cache and are now cleared:
roles:exact-match behavior is now documented (was a security-relevant gap)actions/setup/js/check_permissions_utils.cjs:254uses exact equality (normalizedRequired === effectiveRole) with no privilege hierarchy — soroles: [write]rejectsadminandmaintaineractors.docs/src/content/docs/reference/compilation-process.md:217andfrontmatter-full.md:953-954("matching is exact, not a minimum threshold. List every role you want to accept.").pkg/workflow/js/→actions/setup/js/.IgnoredFrontmatterFieldsundocumented entry cleareduser-invokablelived inIgnoredFrontmatterFieldsbut was documented nowhere. The slice is now empty (pkg/constants/constants.go:321:var IgnoredFrontmatterFields = []string{}), so the finding no longer applies.Verified-consistent checks (no action needed)
create-agent-taskdeprecation is fully consistent: the property carriesdeprecated: true+x-deprecation-messageinside itsoneOfbranch (main_workflow_schema.json:5104-5105), the compiler emits a runtime warning (pkg/workflow/create_agent_session.go:52), and it is wired as an alias ofcreate-agent-session(pkg/workflow/safe_output_handlers.go:36).deprecated: truefrequently lives inside aoneOf/anyOfsub-schema rather than the property root — traversal scripts must descend into branches before flagging a property as "missing deprecation".experiments.*.analysis_typeenum vs docs: schema enum[t_test, mann_whitney, proportion_test, bayesian_ab]matches the documentation table exactly (experiments-specification.md:657-660).analysis_typedecorative-enum concern downgraded: the bundled analyzer stores and prints the value but never branches on it (the onlyswitchatpkg/cli/experiments_analyze_statistics.go:345is onRecommendation). However, the spec explicitly documents this —experiments-specification.md:1184statesanalysis_typeis "advisory only" and R-STAT-003 uses normative SHOULD. So this is documented behavior, not a gap.Documentation Gaps
None newly found.
Schema Improvements Needed
Low priority — naming-convention inconsistency in
dispatch-workflowsafe-outputThe
dispatch-workflowsafe-output (kebab-case,main_workflow_schema.json:9013) uses snake_case for its child properties, unlike every sibling safe-output which uses kebab-case:dispatch_repository(main_workflow_schema.json:9095)allowed_repositories(main_workflow_schema.json:9121)event_typeThis is internally consistent (the Go handler yaml tags and the docs both use snake_case), so it is not a functional bug — but it diverges from the kebab-case convention used across the rest of the safe-outputs surface. Aligning it would require an alias to preserve backward compatibility, so this is genuinely low priority.
Parser Updates Required
None.
Workflow Violations
None. No workflow files use fields absent from the schema (the
in_used_not_schemagaps in the pre-computed diff are all nested keys or expression fragments, not top-level frontmatter keys).Recommendations
dispatch-repository,allowed-repositories,event-type) for thedispatch-workflowchild keys to match the convention used by sibling safe-outputs, while keeping the snake_case forms for backward compatibility.Strategy Performance
Next Steps
dispatch-workflowchild keysroles:exact-match documentation — already presentIgnoredFrontmatterFieldscleanup — already emptycreate-agent-taskdeprecation wiring — fully consistentReferences: §27329587938
Beta Was this translation helpful? Give feedback.
All reactions