Skip to content

docs: stop describing two deleted CI workflows as live - #298

Merged
hyperpolymath merged 1 commit into
mainfrom
docs/retire-blocker-workflow-references
Jul 29, 2026
Merged

docs: stop describing two deleted CI workflows as live#298
hyperpolymath merged 1 commit into
mainfrom
docs/retire-blocker-workflow-references

Conversation

@hyperpolymath

Copy link
Copy Markdown
Owner

npm-bun-blocker.yml was replaced by runtime-policy.yml, and ts-blocker.yml was deleted outright, across 55 repositories. These documents still described both as active CI.

Old New
npm-bun-blocker runtime-policy — renamed in place; it has a successor
ts-blocker removed — no successor; the rule it enforced ("use ReScript instead") is itself retired

Why they went: the old blocker failed any build carrying bun.lockb with "Use Deno instead" — blocking what is now the first-choice runtime under LANGUAGE-POLICY.adoc §1. And ts-blocker could not fail: it diffed HEAD~1 on a depth-1 checkout, sent the error to /dev/null, and swallowed it with || true. 20/20 runs green; it never once fired.

Deliberately untouched: dated records (CRG-AUDIT-2026-04-18.adoc, SESSION-CLOSE-*, CHANGELOGs, .audittraining/) — they record the tree as it was, and rewriting them falsifies the record. Also standards/LANGUAGE-POLICY.adoc, which names both on purpose to document their retirement.

Full scan: dev-notes/SCAN-RESULTS-2026-07-29.adoc.

🤖 Generated with Claude Code

npm-bun-blocker.yml was REPLACED by runtime-policy.yml, and ts-blocker.yml was
DELETED outright, across 55 repositories on 2026-07-29. These documents still
described both as active CI.

  npm-bun-blocker -> runtime-policy   (renamed in place; it has a successor)
  ts-blocker      -> removed          (no successor; the rule it enforced,
                                       "use ReScript instead", is itself retired)

Why they went, briefly: the old npm-bun-blocker failed any build carrying
bun.lockb with "Use Deno instead", which blocked what is now the FIRST-choice
runtime under standards/LANGUAGE-POLICY.adoc §1 and mandated the second.
ts-blocker could not fail at all -- it diffed HEAD~1 on a depth-1 checkout, sent
the error to /dev/null and swallowed it with || true. 20 of 20 runs green; it
never once fired.

Deliberately NOT touched, and excluded from the sweep that produced this:
  * dated records -- CRG-AUDIT-2026-04-18.adoc, SESSION-CLOSE-*,
    STATE-ASSESSMENT-*, CHANGELOGs, .audittraining/. They record the tree AS IT
    WAS; rewriting them would falsify the record.
  * standards/LANGUAGE-POLICY.adoc, which names both workflows on purpose in
    order to document their retirement.

Full scan: dev-notes/SCAN-RESULTS-2026-07-29.adoc.

Co-Authored-By: Claude Opus 5 <noreply@anthropic.com>
@gitar-bot

gitar-bot Bot commented Jul 29, 2026

Copy link
Copy Markdown

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.
Learn more

Code Review 👍 Approved with suggestions 0 resolved / 1 findings

Removes references to deleted CI workflows across documentation files. Update the '17 Mandatory Workflows' header count to match the reduced bullet list.

Auto-approved and auto-merge armed: No blocking issues found.
Please see Auto-approve Docs for details on setting custom approval criteria. — merges when pipeline and required approvals pass.

💡 Quality: Workflow bullet count no longer matches "17 Mandatory Workflows" header

📄 docs/RSR_OUTLINE.adoc:175-184

This change nets -1 bullet (removes npm-bun-blocker.yml and ts-blocker.yml, adds runtime-policy.yml), leaving 16 items under the "=== 17 Mandatory Workflows" header (line 165), which still claims 17. The heading and the list now disagree. Update the header count (or reconcile the list) so the stated number matches the bullets.

🤖 Prompt for agents
Code Review: Removes references to deleted CI workflows across documentation files. Update the '17 Mandatory Workflows' header count to match the reduced bullet list.

1. 💡 Quality: Workflow bullet count no longer matches "17 Mandatory Workflows" header
   Files: docs/RSR_OUTLINE.adoc:175-184

   This change nets -1 bullet (removes `npm-bun-blocker.yml` and `ts-blocker.yml`, adds `runtime-policy.yml`), leaving 16 items under the "=== 17 Mandatory Workflows" header (line 165), which still claims 17. The heading and the list now disagree. Update the header count (or reconcile the list) so the stated number matches the bullets.

Options

Display: compact → Showing less information.

Comment with these commands to change the behavior for this request:

Compact
gitar display:verbose         

Was this helpful? React with 👍 / 👎 | Gitar

@gitar-bot

gitar-bot Bot commented Jul 29, 2026

Copy link
Copy Markdown

⚠️ Gitar auto-approved this PR but could not enable auto-merge: auto-merge is disabled for this repository — enable "Allow auto-merge" in the repository settings.

@gitar-bot gitar-bot Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Gitar has auto-approved this PR and enabled auto-merge (configure)

@gitar-bot gitar-bot Bot added the gitar-approved Added by Gitar label Jul 29, 2026
@github-actions

Copy link
Copy Markdown

🔍 Hypatia Security Scan

Findings: 244 issues detected

Severity Count
🔴 Critical 15
🟠 High 163
🟡 Medium 66

⚠️ Action Required: Critical security issues found!

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

@hyperpolymath
hyperpolymath merged commit 908a5c5 into main Jul 29, 2026
51 of 52 checks passed
@hyperpolymath
hyperpolymath deleted the docs/retire-blocker-workflow-references branch July 29, 2026 07:10
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

gitar-approved Added by Gitar

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant