From 90d33db7c30e966f58c9b08697d5e2a11b17c214 Mon Sep 17 00:00:00 2001 From: "Jonathan D.A. Jewell" <6759885+hyperpolymath@users.noreply.github.com> Date: Thu, 6 Aug 2026 04:04:49 +0100 Subject: [PATCH] fix(ci): repair workflows made unparseable by a blind permissions insertion MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit These workflow files are not valid YAML, so they have NEVER run. GitHub Actions rejects the file before creating any job: the run is recorded as `failure` with no jobs, no log and no check run, and `gh pr checks` shows no row at all. A red mark with nothing behind it to read. The cause is a sweep that added permission declarations by LINE POSITION rather than by parsing the document. Two invalid shapes resulted. permissions: read-all actions: read <- a mapping indented under a SCALAR value `read-all` is a scalar and cannot take children. It also already grants everything `actions: read` would, so the orphaned line is dropped and nothing is lost. on: permissions: contents: read <- two colons on one line, and push: `permissions` is not a valid key here Dropped entirely. VERIFIED, not assumed. Every workflow in this repository parses after the change, and the repairer refuses to write any file that does not parse and still contain jobs afterwards. Where a job-level permissions line was removed, a top-level `permissions:` remains and is read-only, so nothing is widened — and if none remained the tool reports that rather than inventing one, because guessing a permission set is how you silently over-grant. ESTATE CONTEXT: 67 repositories and 100 workflow files are in this state. The most frequently broken file is workflow-linter.yml, in 22 repositories — the workflow whose job is to lint workflows was itself unparseable, so it never ran and never caught this. Detection is being added upstream so it cannot recur invisibly: a strict YAML loader in the governance reusable (hyperpolymath/standards#582). Ordinary validation cannot see this class of fault, because yaml.safe_load accepts duplicate keys and only a full parse catches the malformed indentation. EXPECT THIS REPOSITORY TO GET LOUDER. Workflows that have been failing silently will now actually run, and some will find real problems. Co-Authored-By: Claude Fable 5 Signed-off-by: Jonathan D.A. Jewell <6759885+hyperpolymath@users.noreply.github.com> --- .github/workflows/cargo-audit.yml | 1 - .github/workflows/ci.yml | 1 - .github/workflows/release.yml | 1 - .github/workflows/workflow-linter.yml | 1 - 4 files changed, 4 deletions(-) diff --git a/.github/workflows/cargo-audit.yml b/.github/workflows/cargo-audit.yml index af81f8a..963e6e6 100644 --- a/.github/workflows/cargo-audit.yml +++ b/.github/workflows/cargo-audit.yml @@ -16,7 +16,6 @@ on: - cron: '0 6 * * 1' # Weekly on Monday permissions: read-all - actions: read jobs: audit: diff --git a/.github/workflows/ci.yml b/.github/workflows/ci.yml index 12fae8c..e786238 100644 --- a/.github/workflows/ci.yml +++ b/.github/workflows/ci.yml @@ -8,7 +8,6 @@ on: branches: [ main, develop ] permissions: read-all - actions: read env: CARGO_TERM_COLOR: always diff --git a/.github/workflows/release.yml b/.github/workflows/release.yml index 2c82379..e7fabe6 100644 --- a/.github/workflows/release.yml +++ b/.github/workflows/release.yml @@ -7,7 +7,6 @@ on: - 'v*' permissions: read-all - actions: read env: CARGO_TERM_COLOR: always diff --git a/.github/workflows/workflow-linter.yml b/.github/workflows/workflow-linter.yml index 9ad08b3..67b6a82 100644 --- a/.github/workflows/workflow-linter.yml +++ b/.github/workflows/workflow-linter.yml @@ -11,7 +11,6 @@ on: - '.github/workflows/**' permissions: read-all - actions: read jobs: lint-workflows: