fix(ci): pin dependency-audit reusable workflow to SHA

[don-petry]

Summary

• Pins petry-projects/.github/.github/workflows/dependency-audit-reusable.yml from @v1 to SHA 208ec2d69b75227d375edf8745d84fbac05a76b2 (which is the v1 tag) to comply with the org action-pinning policy.

Motivation

Weekly compliance audit flagged dependency-audit.yml for using an unpinned action reference (@v1 tag). Tags are mutable and can be moved; pinning to a SHA ensures the workflow always runs the exact same code.

Closes #89

---

Generated with Claude Code

[coderabbitai]

Warning

Rate limit exceeded

@github-actions[bot] has exceeded the limit for the number of commits that can be reviewed per hour. Please wait 51 minutes and 37 seconds before requesting another review.

Your organization is not enrolled in usage-based pricing. Contact your admin to enable usage-based pricing to continue reviews beyond the rate limit, or try again in 51 minutes and 37 seconds.

⌛ How to resolve this issue?

After the wait time has elapsed, a review can be triggered using the @coderabbitai review command as a PR comment. Alternatively, push new commits to this PR.

We recommend that you space out your commits to avoid hitting the rate limit.

🚦 How do rate limits work?

CodeRabbit enforces hourly rate limits for each developer per organization.

Our paid plans have higher rate limits than the trial, open-source and free plans. In all cases, we re-allow further reviews after a brief timeout.

Please see our FAQ for further information.

ℹ️ Review info

⚙️ Run configuration

Configuration used: Path: .coderabbit.yaml

Review profile: CHILL

Plan: Pro

Run ID: f92fc20e-76f0-4c49-bbcd-2158d69295eb

📥 Commits

Reviewing files that changed from the base of the PR and between 051f051 and 5e8d013.

📒 Files selected for processing (1)

• .github/workflows/dependency-audit.yml

✨ Finishing Touches

🧪 Generate unit tests (beta)

• Create PR with unit tests
• Commit unit tests in branch claude/issue-89-20260414-1321

---

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

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

[Copilot]

Pull request overview

This PR updates the repository’s dependency audit GitHub Actions workflow to comply with the org “pin reusable workflows to a SHA” policy by replacing a mutable @v1 reference with an immutable commit SHA.

Changes:

• Pin petry-projects/.github/.github/workflows/dependency-audit-reusable.yml from @v1 to commit 208ec2d69b75227d375edf8745d84fbac05a76b2 (annotated as # v1).

---

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

[sonarqubecloud]

Quality Gate passed

Issues
0 New issues
0 Accepted issues

Measures
0 Security Hotspots
0.0% Coverage on New Code
0.0% Duplication on New Code

See analysis details on SonarQube Cloud

[don-petry]

CI is green. @don-petry — please review and merge when ready.

[don-petry]

Automated review — APPROVED

Risk: MEDIUM
Reviewed commit: f01b2364ade0bf3b959285796404cc2b8ca8f2ab
Cascade: triage → deep (see triage: haiku 4.5 → deep: sonnet 4.6 + duck: gpt-5.4 → audit: opus 4.6 for models)

Summary

This PR pins a reusable workflow call from a mutable @v1 tag to an immutable SHA, which is a security improvement that addresses org compliance issue #89. All CI checks pass (CodeQL, SonarCloud, AgentShield). One minor correctness note: the pinned SHA (208ec2d69b75227d375edf8745d84fbac05a76b2) is the annotated tag object SHA, not the underlying commit SHA (ae9709f4466dec60a5733c9e7487f69dcd004e05); however, GitHub Actions resolves this correctly as evidenced by passing CI, and the immutability guarantee of SHA pinning is preserved either way.

Findings

Minor

• [minor] .github/workflows/dependency-audit.yml:33 — The SHA pinned (208ec2d69b75227d375edf8745d84fbac05a76b2) is the annotated tag object SHA, not the underlying commit SHA (ae9709f4466dec60a5733c9e7487f69dcd004e05). GitHub Actions resolves annotated tag objects to the underlying commit, so this works in practice and CI confirms it. The preferred approach is to pin to the commit SHA directly, as that is the canonical immutable reference. The security goal (preventing tag mutation supply chain attacks) is achieved by either approach.

Info

• [info] .github/workflows/dependency-audit.yml — The workflow header explicitly warns "You MUST NOT change: trigger events, the uses: line, or job name". This PR modifies the uses: line for compliance/pinning purposes, which is a legitimate exception to that rule. No trigger events or job names were changed.
• [info] CI status — All status checks pass: CodeQL (actions + python), SonarCloud (0 new issues, 0 security hotspots), AgentShield, Claude Code. Merge state is BLOCKED only because a human review approval is required.

CI status

All status checks pass: CodeQL (actions + python), SonarCloud (0 new issues, 0 security hotspots), AgentShield, Claude Code.

---

Note: GitHub prevents self-approval of PRs. Review verdict is APPROVED but a second reviewer must formally approve via GitHub.

Reviewed by the don-petry PR-review cascade (triage: haiku 4.5 → deep: sonnet 4.6 + duck: gpt-5.4 → audit: opus 4.6). Reply with @don-petry if you need a human.