[Schema Consistency] Schema consistency check report - 2026-08-22 #54737
Closed
Replies: 1 comment
|
This discussion was automatically closed because it expired on 2026-08-23T05:37:00.588Z.
|
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-08-22
🔍 Schema Consistency Check - 2026-08-22
Summary
Critical Issues
1.
permissionsschema omits parser-supported scopespkg/parser/schemas/main_workflow_schema.jsondefinespermissionsvia$defs.github_actions_permissions, but that schema does not includesecret-scanning-alerts.pkg/workflow/permissions.godefinesPermissionSecretScanningAlerts, andpkg/workflow/safe_outputs_app_config.goemitspermission-secret-scanning-alerts.pkg/workflow/permissions.go:101pkg/workflow/safe_outputs_app_config.go:563$defs.github_actions_permissions2.
permissionsdocs omit several schema/parser-supported scopesattestations,models, andcopilot-requestsunderpermissions.docs/src/content/docs/reference/permissions.mddocumentscopilot-requests, but the main “Key read permission scopes” list and GitHub App-only section do not mentionattestations,models, orsecret-scanning-alerts.$defs.github_actions_permissions.properties.attestations/models/copilot-requestspkg/workflow/permissions.go:62,:78,:101docs/src/content/docs/reference/permissions.mdDocumentation Gaps
3.
cache-memorydocs claim defaultallowed-extensionsis empty/all files allowed, but code/schema default to a restricted allowlistdocs/src/content/docs/reference/cache-memory.mdsays default is “empty/all files allowed”.tools.cache-memory.allowed-extensionsdescription says default is[".json", ".jsonl", ".txt", ".md", ".csv"].pkg/workflow/cache_config.goappliesconstants.DefaultAllowedMemoryExtensionswhen none are provided.docs/src/content/docs/reference/cache-memory.md:29,43pkg/parser/schemas/main_workflow_schema.jsonaroundtools.cache-memory.allowed-extensionspkg/workflow/cache_config.go:292-2954.
cache-memorydocs implyretention-daysextends cache lifetime, but schema/compiler treat it as artifact retention onlyretention-dayscan “extend access beyond cache expiration”.retention-daysis described as uploaded artifact retention.retention-daysis parsed as artifact retention metadata; the underlying Actions cache lifetime is still governed by GitHub cache behavior.docs/src/content/docs/reference/cache-memory.md:8,28,95tools.cache-memory.retention-dayspkg/workflow/cache_config.goparsesretention-daysbut does not alter cache eviction semanticsWorkflow Violations / Real-World Checks
field_gaps.in_used_not_schemawas empty in/tmp/gh-aw/agent/schema-diff.json)..github/workflows/release.mdusesattestations: write.Parser ↔ Documentation Notes
network: {}deny-all semantics are consistent between docs and parser (docs/src/content/docs/reference/network.md:21,pkg/workflow/engine.gocomments,pkg/workflow/engine_firewall_support.go)./tmp/gh-aw/agent/awf-config-drift.jsonreporteddegraded: true.Recommendations
secret-scanning-alertsto$defs.github_actions_permissionsinpkg/parser/schemas/main_workflow_schema.json.permissions.mdagainstpkg/workflow/permissions.goand document every currently supported scope, includingattestations,models, andsecret-scanning-alerts.cache-memory.mdto state the true defaultallowed-extensionsallowlist and explain that omitted config is not equivalent to “all files allowed”.cache-memory.mdsoretention-daysis clearly artifact-retention-only and does not extend GitHub Actions cache lifetime.permissions.goconstants against$defs.github_actions_permissionsto prevent future schema drift.Strategy Performance
Next Steps
All reactions