ci(todos): stop the scanner filing its own permission comment as a work item - #93
Conversation
…rk item The job's permission comment carried the uppercase marker the TODO scanner keys on, so the scanner filed the comment itself as issue #14. Spell it in lowercase, matching the convention devantler-tech/actions uses in the scanner's own sources. Fixes #14 Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_011Aut24ni8XbKPYyY5mYrbb
Root cause. #14's body cites RED / GREEN (a whole-word search for the uppercase marker, the token the scanner keys on per
Validation. Behaviour. The runtime surface is the hosted scanner on pushes to Generated by Claude Code |
✅
|
| Descriptor | Linter | Files | Fixed | Errors | Max errors | Warnings | Elapsed time |
|---|---|---|---|---|---|---|---|
| ✅ ACTION | actionlint | 6 | 0 | 0 | 0.09s | ||
| ✅ ACTION | zizmor | 6 | 0 | 0 | 0 | 1.37s | |
| ✅ BASH | bash-exec | 13 | 0 | 0 | 0.1s | ||
| ✅ BASH | shellcheck | 13 | 0 | 0 | 1.12s | ||
| ✅ BASH | shfmt | 13 | 0 | 0 | 0 | 0.05s | |
| ✅ COPYPASTE | jscpd | yes | no | no | 1.01s | ||
| ✅ CSS | stylelint | 2 | 0 | 0 | 0 | 1.78s | |
| ✅ DOCKERFILE | hadolint | 2 | 0 | 0 | 0.09s | ||
| ✅ EDITORCONFIG | editorconfig-checker | 124 | 0 | 0 | 0.29s | ||
| ✅ HTML | djlint | 2 | 0 | 0 | 1.56s | ||
| ✅ HTML | htmlhint | 2 | 0 | 0 | 0.32s | ||
| ✅ JSON | jsonlint | 2 | 0 | 0 | 0.11s | ||
| ✅ JSON | prettier | 2 | 0 | 0 | 0 | 0.69s | |
| ✅ JSON | v8r | 2 | 0 | 0 | 1.81s | ||
| markdownlint | 13 | 0 | 2 | 0 | 1.5s | ||
| ✅ MARKDOWN | markdown-table-formatter | 13 | 0 | 0 | 0 | 0.26s | |
| ✅ REPOSITORY | betterleaks | yes | no | no | 1.01s | ||
| ✅ REPOSITORY | checkov | yes | no | no | 24.94s | ||
| ✅ REPOSITORY | git_diff | yes | no | no | 0.03s | ||
| ✅ REPOSITORY | grype | yes | no | no | 76.34s | ||
| ✅ REPOSITORY | osv-scanner | yes | no | no | 0.39s | ||
| ✅ REPOSITORY | secretlint | yes | no | no | 1.46s | ||
| ✅ REPOSITORY | syft | yes | no | no | 2.61s | ||
| ✅ REPOSITORY | trivy | yes | no | no | 13.73s | ||
| ✅ REPOSITORY | trivy-sbom | yes | no | no | 0.87s | ||
| ✅ REPOSITORY | trufflehog | yes | no | no | 3.81s | ||
| ✅ SPELL | lychee | 55 | 0 | 0 | 1.65s | ||
| ✅ YAML | prettier | 25 | 0 | 0 | 0 | 1.39s | |
| ✅ YAML | v8r | 38 | 0 | 0 | 10.32s | ||
| ✅ YAML | yamllint | 29 | 0 | 0 | 0.95s |
Detailed Issues
⚠️ MARKDOWN / markdownlint - 2 errors
CLAUDE.md:1 error MD041/first-line-heading/first-line-h1 First line in a file should be a top-level heading [Context: "@AGENTS.md"]
GEMINI.md:1 error MD041/first-line-heading/first-line-h1 First line in a file should be a top-level heading [Context: "@AGENTS.md"]
See detailed reports in MegaLinter artifacts

Show us your support by starring ⭐ the repository
@coderabbitai review Generated by Claude Code |
|
✅ Action performedReview finished.
|
|
Navigate logical layers of code changes, visualize relationships, and explore their blast radius. No actionable comments were generated in the recent review. 🎉 ℹ️ Recent review info⚙️ Run configurationConfiguration used: Organization UI Review profile: ASSERTIVE Plan: Advanced Run ID: 📒 Files selected for processing (1)
Included review availability: Your plan provides up to 1 included review per hour; 0 remain after this review. 📜 Recent review details🔇 Additional comments (1)
📝 WalkthroughWalkthroughThe workflow’s Priority: ⬇️ Low Merge Risk: ⚪ Minimal · up to The revised comment should no longer create a work item, and the scanner is configured to close the matching prior issue. Its live closure was not verified, but no material merge risk remains. 🚥 Pre-merge checks | ✅ 5✅ Passed checks (5 passed)
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. Comment |
Why
The to-do scanner opened issue #14, "tracking issues.", because the comment explaining the scanner's own permissions contained the exact marker word it searches for. The issue describes no real work and adds noise to the backlog and project board.
What
Rewords that comment in lowercase, the same convention the shared actions repository already uses in the scanner's own sources, so the scanner no longer treats it as a work item. The scanner's permissions and behaviour are unchanged; on merge the removed marker also lets the scanner close the stray issue.
Fixes #14
🤖 Generated with Claude Code
https://claude.ai/code/session_011Aut24ni8XbKPYyY5mYrbb
Generated by Claude Code