Repository navigation
ci(workflow-linter): find the SPDX header in the leading comment block - #94
Conversation
The SPDX check read only line 1 of each workflow, but gh actions-lock puts '# This workflow is managed by gh actions-lock.' there, so all 30 workflows failed although each carries its SPDX header in the opening comment block. The check now looks through that block, as the RSR template does. A file with no SPDX line, or one after the workflow body, still fails (checked under gawk and mawk). push also runs only on main now, as in the template (rsr-template-repo#322), so a PR push is linted once. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_0136P7ZnmBahbhfkFEP6ark4
|
Warning Review limit reachedYou've used all free OSS reviews for now. Wait for the free limit to reset to keep reviewing this public repository. Next included review available in 41 minutes. View limit details
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. |
|
Codex Review Summary: ❌ FailedThis comment shows the latest Codex review activity on this pull request.
ℹ️ About Codex in GitHubYour team has set up Codex to review pull requests in this repo. Reviews are triggered when you
Codex reacts with 👀 while any review is running, comments if it has suggestions, and reacts with 👍 once all reviews finish with no findings. |
|
You have reached your Codex usage limits for code reviews. You can see your limits in the Codex usage dashboard. |
|
CI status on 21c8797. None of the 4 red checks comes from this diff.
main has the same drift: Then commit the updated Validate K9 contracts, SonarQube, Hypatia neurosymbolic scan: these failures are already on main and come from other causes: unfilled K9 mint tokens, no Auto-merge stays blocked until the lockfile is fixed on main and merged into this branch. Generated by Claude Code |



Summary
Workflow Security Linterfailed on all 30 workflows withmissing SPDX header. Every one of them does carry# SPDX-License-Identifier: MPL-2.0, but on line 2 or later:gh actions-lockwrites# This workflow is managed by gh actions-lock.on line 1. The check read only line 1.Changes
.github/workflows/workflow-linter.yml:pushruns only onmain, as in ci(workflow-linter): run push only on main rsr-template-repo#322. A PR push is then linted once, throughpull_request.No licence header is changed; this PR only corrects where the check looks.
Testing
Not in this PR
The K9 failures in
dogfood-gate.ymlare a separate matter. The current validator, standardse064539, rejects 15 of this repo's 18 K9 files on content: unfilledTODO/{{PORT}}mint placeholders, missingpedigreefields, andcoordination.k9/session/custom-checks.k9not being in the coordination dialect. Swapping the validator alone would not turn that job green. It is left for the owner's decision.Generated by Claude Code