ci: declare explicit permissions on the shared workflow callers - #1147
ci: declare explicit permissions on the shared workflow callers#1147prisis wants to merge 1 commit into
Conversation
Co-Authored-By: Claude Opus 5 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01CLftRo14oumJr2GEa8rPgf
|
Thank you for following the naming conventions! 🙏 |
|
No actionable comments were generated in the recent review. 🎉 ℹ️ Recent review info⚙️ Run configurationConfiguration used: defaults Review profile: CHILL Plan: Advanced Run ID: 📒 Files selected for processing (3)
Included review availability: Your plan provides up to 1 included review per hour; 0 remain after this review. 📝 WalkthroughWalkthroughThree GitHub Actions workflows now declare explicit permissions for repository access, issue and pull request updates, and cache deletion. ChangesWorkflow permissions
Estimated code review effort: 1 (Trivial) | ~3 minutes Merge Risk: ⚪ Minimal · up to The workflows retain the access needed to manage issues, pull requests, and caches while supporting read-only default token permissions. No current merge-blocking risk remains. 🚥 Pre-merge checks | ✅ 5✅ Passed checks (5 passed)
✨ Finishing Touches🧪 Generate unit tests (beta)
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 |
These workflows call reusable workflows in
anolilab/workflowsand declare nopermissionsof their own, so the token they receive is whatever the repository default happens to be — currently write.That default is the only thing keeping this repository from running Actions with a read-only token. A caller cannot grant a reusable workflow more than it holds, so each block below is exactly what the callee declares, read from the callee rather than guessed:
allo-allo.yaml— contents: read, issues: write, pull-requests: writecache-clear.yml— actions: write, contents: readstale-issues.yml— contents: read, issues: write, pull-requests: writeOnce this merges the repository default can drop to read-only with nothing losing access it was using.
🤖 Generated with Claude Code
https://claude.ai/code/session_01CLftRo14oumJr2GEa8rPgf
Summary by CodeRabbit