fix(ci): repair unparseable permissions scalar (3 workflow file(s))

[hyperpolymath]

permissions: read-all followed by an indented actions: read is a YAML
parse error. The workflow never starts, so it emits no check run — a
required context silently never reports, and this repository currently looks
greener than a repaired one.

read-all already grants actions: read, so removing the stray line is a
semantic no-op. Each file was verified with a real YAML parser: it fails
yq before the change and parses after it. Every diff is exactly one deleted line.

Files changed: cargo-audit.yml e2e.yml workflow-linter.yml

Measured estate-wide: 42 such files across 28 repositories on main, all
carrying the identical stray key. Zero overlap with the TokenPermissions sweep.

🤖 Generated with Claude Code

https://claude.ai/code/session_0178nN4Nm3neFRy5K9StZKnB

[coderabbitai]

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info

⚙️ Run configuration

Configuration used: Organization UI

Review profile: ASSERTIVE

Plan: Advanced

Run ID: 7e525353-e075-468c-b42c-9a14432903f6

📥 Commits

Reviewing files that changed from the base of the PR and between 210beb3 and 2e90ff6.

📒 Files selected for processing (3)

• .github/workflows/cargo-audit.yml
• .github/workflows/e2e.yml
• .github/workflows/workflow-linter.yml

💤 Files with no reviewable changes (3)

• .github/workflows/e2e.yml
• .github/workflows/workflow-linter.yml
• .github/workflows/cargo-audit.yml

Included review availability: Your plan provides up to 1 included review per hour; 0 remain after this review.

📜 Recent review details

⏰ Context from checks skipped due to timeout. (30)

• GitHub Check: governance / Workflow security linter
• GitHub Check: governance / Licence consistency
• GitHub Check: governance / Code quality + docs
• GitHub Check: governance / Allowlist Preflight
• GitHub Check: governance / Debt ratchet
• GitHub Check: governance / Check Workflow Staleness
• GitHub Check: scan / gitleaks
• GitHub Check: governance / Well-Known (RFC 9116 + RSR)
• GitHub Check: governance / Exemption ratchet
• GitHub Check: scan / rust-secrets
• GitHub Check: governance / Security policy checks
• GitHub Check: governance / Guix packaging policy (Nix retired)
• GitHub Check: scan / shell-secrets
• GitHub Check: governance / Trusted-base reduction policy
• GitHub Check: governance / Language / package anti-pattern policy
• GitHub Check: hypatia / Hypatia Neurosymbolic Analysis
• GitHub Check: Zig FFI Build + Test
• GitHub Check: E2E Lifecycle Test
• GitHub Check: Validate K9 contracts
• GitHub Check: Aspect Tests (Cross-Cutting)
• GitHub Check: Groove manifest check
• GitHub Check: Empty-linter (invisible characters)
• GitHub Check: analyze (actions, none)
• GitHub Check: Criterion Benchmarks
• GitHub Check: STATE vs dashboard reconciliation
• GitHub Check: Validate A2ML manifests
• GitHub Check: Rust Build + Unit Tests
• GitHub Check: semgrep
• GitHub Check: lint-workflows
• GitHub Check: lint-workflows

---

📝 Summary

Summary by CodeRabbit

• Chores
  • Updated automated workflow permission settings to use read-only access across all available scopes.
  • Standardised permission configuration across audit, end-to-end testing, and workflow validation processes.

Walkthrough

The pull request updates permission declarations in three GitHub Actions workflows. It removes an explicit actions: read permission from one workflow and changes two workflows to use read-all.

Changes

Workflow permission declarations

Layer / File(s)	Summary
Update workflow read permissions .github/workflows/cargo-audit.yml, .github/workflows/e2e.yml, .github/workflows/workflow-linter.yml	The cargo audit workflow removes its explicit actions: read permission. The end-to-end and workflow linter workflows change their permission declarations to read-all.

Priority: ⬇️ Low

Estimated code review effort: 1 (Trivial) | ~5 minutes

Change: Bug fix

Merge Risk: ⚪ Minimal · up to 2e90f

The workflows can parse with their intended read permissions, and no regression from this cleanup was identified.

🚥 Pre-merge checks | ✅ 5

✅ Passed checks (5 passed)

Check name	Status	Explanation
Title check	✅ Passed	The title clearly identifies the CI fix, the unparseable permissions scalar, and the three affected workflow files.
Description check	✅ Passed	The description directly explains the YAML parse error, the removed lines, the affected files, and the verification performed.
Docstring Coverage	✅ Passed	No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check. Docstring coverage is scoped to functions touched by this diff. Analyzed 0 functions across 0…
Linked Issues check	✅ Passed	Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check	✅ Passed	Check skipped because no linked issues were found for this pull request.

---

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

• X
• Mastodon
• Reddit
• LinkedIn

_{A rabbit checks the workflow gate
Read permissions now state their weight
Three files keep the rules in line
Tokens follow a clearer sign
The audit trail hops on time}

_{Comment @coderabbitai help to get the list of available commands.}