From a4fbc83c954ace069fc4795d97cdd1603cc03d31 Mon Sep 17 00:00:00 2001 From: "Jonathan D.A. Jewell" <6759885+hyperpolymath@users.noreply.github.com> Date: Thu, 6 Aug 2026 04:01:16 +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/codeql.yml | 1 - .github/workflows/guix-nix-policy.yml | 1 - .github/workflows/quality.yml | 1 - .github/workflows/scorecard.yml | 1 - .github/workflows/security-policy.yml | 1 - .github/workflows/wellknown-enforcement.yml | 1 - .github/workflows/workflow-linter.yml | 1 - 7 files changed, 7 deletions(-) diff --git a/.github/workflows/codeql.yml b/.github/workflows/codeql.yml index c1e3ccc..2f99df1 100644 --- a/.github/workflows/codeql.yml +++ b/.github/workflows/codeql.yml @@ -18,7 +18,6 @@ concurrency: cancel-in-progress: true permissions: read-all - actions: read jobs: analyze: diff --git a/.github/workflows/guix-nix-policy.yml b/.github/workflows/guix-nix-policy.yml index bf4aa64..0d0b946 100644 --- a/.github/workflows/guix-nix-policy.yml +++ b/.github/workflows/guix-nix-policy.yml @@ -12,7 +12,6 @@ concurrency: cancel-in-progress: true permissions: read-all - actions: read jobs: check: diff --git a/.github/workflows/quality.yml b/.github/workflows/quality.yml index 3bccf7e..7acb4b0 100644 --- a/.github/workflows/quality.yml +++ b/.github/workflows/quality.yml @@ -13,7 +13,6 @@ concurrency: permissions: read-all - actions: read jobs: lint: diff --git a/.github/workflows/scorecard.yml b/.github/workflows/scorecard.yml index f0c8699..64ac2ac 100644 --- a/.github/workflows/scorecard.yml +++ b/.github/workflows/scorecard.yml @@ -9,7 +9,6 @@ on: branches: [main] permissions: read-all - actions: read jobs: analysis: diff --git a/.github/workflows/security-policy.yml b/.github/workflows/security-policy.yml index b6cece2..54860dc 100644 --- a/.github/workflows/security-policy.yml +++ b/.github/workflows/security-policy.yml @@ -12,7 +12,6 @@ concurrency: cancel-in-progress: true permissions: read-all - actions: read jobs: check: diff --git a/.github/workflows/wellknown-enforcement.yml b/.github/workflows/wellknown-enforcement.yml index 883b80b..5c9fe19 100644 --- a/.github/workflows/wellknown-enforcement.yml +++ b/.github/workflows/wellknown-enforcement.yml @@ -16,7 +16,6 @@ on: permissions: read-all - actions: read jobs: validate: diff --git a/.github/workflows/workflow-linter.yml b/.github/workflows/workflow-linter.yml index d7d5a40..f355fc3 100644 --- a/.github/workflows/workflow-linter.yml +++ b/.github/workflows/workflow-linter.yml @@ -13,7 +13,6 @@ on: workflow_dispatch: permissions: read-all - actions: read jobs: lint-workflows: