chore: land local compliance sweep + YAML repair - #299
Conversation
…flows) The estate compliance sweep inserted an indented 'actions: read' beneath 'permissions: read-all', which is invalid YAML (scalar followed by mapping continuation) and would parse-kill all four workflows. read-all already grants actions:read, so removal is semantically neutral. Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
| @@ -0,0 +1 @@ | |||
| Subproject commit 7add7bcac05ea4cb2fc0af62ec106dcd4779d73e | |||
| @@ -0,0 +1 @@ | |||
| Subproject commit a1517a3677eed03e96e859d5aad0e957d5f156ac | |||
🔍 Hypatia Security ScanFindings: 246 issues detected
View findings[
{
"reason": "Issue in build.yml",
"type": "missing_timeout_minutes",
"file": "build.yml",
"action": "flag",
"rule_module": "workflow_audit",
"severity": "medium"
},
{
"reason": "Issue in pages-deploy.yml",
"type": "missing_timeout_minutes",
"file": "pages-deploy.yml",
"action": "flag",
"rule_module": "workflow_audit",
"severity": "medium"
},
{
"reason": "Issue in push-email-notify.yml",
"type": "missing_timeout_minutes",
"file": "push-email-notify.yml",
"action": "flag",
"rule_module": "workflow_audit",
"severity": "medium"
},
{
"reason": "Issue in instant-sync.yml",
"type": "secret_action_without_presence_gate",
"file": "instant-sync.yml",
"action": "peter-evans/repository-dispatch",
"rule_module": "workflow_audit",
"severity": "high"
},
{
"reason": "Issue in codeql.yml",
"type": "codeql_missing_actions_language",
"file": "codeql.yml",
"action": "flag",
"rule_module": "workflow_audit",
"severity": "medium"
},
{
"reason": "TypeScript file detected -- banned language",
"type": "banned_language_file",
"file": "/home/runner/work/boj-server/boj-server/cartridges/academic-workflow-mcp/adapter/mod.ts",
"action": "flag",
"rule_module": "cicd_rules",
"severity": "critical"
},
{
"reason": "TypeScript file detected -- banned language",
"type": "banned_language_file",
"file": "/home/runner/work/boj-server/boj-server/cartridges/sanctify-mcp/adapter/mod.ts",
"action": "flag",
"rule_module": "cicd_rules",
"severity": "critical"
},
{
"reason": "TypeScript file detected -- banned language",
"type": "banned_language_file",
"file": "/home/runner/work/boj-server/boj-server/cartridges/bofig-mcp/adapter/mod.ts",
"action": "flag",
"rule_module": "cicd_rules",
"severity": "critical"
},
{
"reason": "TypeScript file detected -- banned language",
"type": "banned_language_file",
"file": "/home/runner/work/boj-server/boj-server/cartridges/ephapax-mcp/adapter/mod.ts",
"action": "flag",
"rule_module": "cicd_rules",
"severity": "critical"
},
{
"reason": "TypeScript file detected -- banned language",
"type": "banned_language_file",
"file": "/home/runner/work/boj-server/boj-server/cartridges/hesiod-mcp/adapter/mod.ts",
"action": "flag",
"rule_module": "cicd_rules",
"severity": "critical"
}
]Powered by Hypatia Neurosymbolic CI/CD Intelligence |
|
Note Automatic reviews are paused because your trial's included automatic processing has been used for this period. Upgrade now, or comment "Gitar review" to run a review anytime. Code Review ✅ ApprovedLands estate-wide compliance updates, consolidates duplicate governance files, and fixes invalid YAML syntax across four CI workflows. No issues found.
OptionsDisplay: compact → Showing less information. Comment with these commands to change the behavior for this request:
Important Your trial ends in 7 days — upgrade now to keep code review, CI analysis, auto-apply, custom automations, and more. Was this helpful? React with 👍 / 👎 | Gitar |
Lands three commits stranded on the local main (main is PR-only):
20dc1d7e): addsactions: readto workflow permission maps + GOVERNANCE.md content.ba42404d): drops.github/GOVERNANCE.mdin favor of rootGOVERNANCE.md.actions: readunder scalar permissions (f2b7cd1b): the sweep inserted an indentedactions: readbeneathpermissions: read-allin 4 workflows (truthfulness, e2e, lsp-dap-bsp, container-publish) — invalid YAML that would parse-kill all four. Removed;read-allalready implies it. All 26 workflows now pass a YAML parse check.Note: a fourth local commit ("update guix.scm from squisher-corpus") was dropped — it was the known squisher-corpus guix.scm clobber (replaced the boj-server package definition with a generic squisher-corpus stub,
source #f). The repo's realguix.scmis untouched by this PR.🤖 Generated with Claude Code