Conversation
Fully pin all workflow actions to commit SHAs with version comments, add persist-credentials: false to checkout steps, scope down workflow/job permissions (default-deny plus per-job grants with justification comments), and pass the pixi environment through an env var instead of directly interpolating the matrix value into a run: command. Assisted-by: Claude (Anthropic)
Freeze all pre-commit hook revs to their commit SHAs (frozen comment retains the version) and add the zizmor hook to catch GitHub Actions workflow security issues locally. Assisted-by: Claude (Anthropic)
Add a 7-day cooldown to both dependabot update groups and switch from weekly to monthly updates to reduce noise and avoid picking up just-published, unvetted releases. Assisted-by: Claude (Anthropic)
Set exclude-newer on the pixi workspace so dependency resolution matches the 7-day cooldown used elsewhere in this hardening pass. Assisted-by: Claude (Anthropic)
|
Warning Review limit reached
Next review available in: 93 minutes You've used all free OSS reviews for now. Wait for the free limit to reset to keep reviewing this public repository. How can I continue?After more reviews become available, a review can be triggered using the To avoid repeated limits, reduce automatic review volume by pausing incremental auto-reviews earlier, using label-based review opt-in, excluding WIP or generated PR titles, or requesting reviews manually when the PR is ready. If your team needs uninterrupted high-volume reviews, an organization admin can enable usage-based reviews. How do review limits work?CodeRabbit enforces per-developer PR review limits for each organization. Most developers receive the normal plan review availability. For paid Pro and Pro+ PR reviews, CodeRabbit uses adaptive limits for sustained high-volume activity. When a developer's recent PR review activity reaches the 95th percentile or higher among CodeRabbit users, additional reviews become available more gradually as earlier reviews age out of the rolling window. Please refer docs for additional details. Review details⚙️ Run configurationConfiguration used: Organization UI Review profile: ASSERTIVE Plan: Pro Plus Run ID: 📒 Files selected for processing (6)
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 |
Changes
ci.yml,cd.yml, anddocs.ymlto commit SHAs with version comments (actions-up --min-age=7, then hand-pinned the one actionactions-upskipped due to cooldown).persist-credentials: falseto allactions/checkoutsteps (none of these jobs need to push).permissions: {}at the workflow level forci.ymlandcd.yml, with explicit least-privilegepermissions:per job (all commented with why).pages/id-tokenpermissions indocs.yml's deploy job.matrix.pixi-environmentinterpolation in arun:step with an env var, to avoid template-injection risk.zizmorpre-commit hook (pinned to a SHA) to catch these issues locally going forward.dependabot.ymlupdate groups and switch from weekly to monthly cadence.exclude-newer = "7d"to the pixi workspace so dependency resolution also respects the 7-day cooldown.Major version changes
hynek/build-and-inspect-python-packagev2 to v3.0.1 major bump and theactions/attest-build-provenancev4.1.1 to v4.2.2 bump were already merged intomainvia separate PRs before this branch was rebased; this PR only adds the SHA pin on top of those already-merged refs.Held-back items
zizmor ignores
dependabot-cooldown,artipacked,excessive-permissions,undocumented-permissions,template-injection) were fixed directly rather than ignored.Verification
uvx zizmor --persona=pedantic .githubreports no findings.uvx prek run -apasses all hooks, includingzizmor.pixi infosucceeds with the newexclude-newersetting.