Skip to content

ci: add cargo-audit dependency scan (Bundle C) - #35

Merged
Liohtml merged 2 commits into
mainfrom
claude/bundle-c-ci-audit
Jun 16, 2026
Merged

ci: add cargo-audit dependency scan (Bundle C)#35
Liohtml merged 2 commits into
mainfrom
claude/bundle-c-ci-audit

Conversation

@Liohtml

@Liohtml Liohtml commented Jun 15, 2026

Copy link
Copy Markdown
Owner

Overview

Bundle C — adds a dependency vulnerability scan to CI (#28).

A new audit job installs cargo-audit and runs cargo audit against the resolved dependency tree, so a known RUSTSEC advisory in chromiumoxide, tokio, regex, etc. surfaces on every push/PR instead of silently landing.

Details

  • Because Cargo.lock is gitignored (library crate), the job runs cargo generate-lockfile first so it audits the exact resolved versions.
  • Actions are SHA-pinned to match the existing test / lint jobs (least-privilege permissions, persist-credentials: false).
  • Verified locally: cargo audit reports no advisories in the current tree (150 deps scanned), so the job is green on day one.

Test plan

  • cargo generate-lockfile + cargo audit run locally → clean (exit 0)
  • YAML validated; actions reuse the already-verified pinned SHAs
  • Job runs green in CI on this PR

Closes #28

https://claude.ai/code/session_01TPpTHPokxsZ3dQpRzg4NkD


Generated by Claude Code

Closes #28. Adds an 'audit' job to the CI workflow that installs cargo-audit
and runs 'cargo audit' against the resolved dependency tree. Because Cargo.lock
is gitignored, the job generates one first. Actions are SHA-pinned, matching the
existing jobs. Verified locally: no advisories in the current tree.

https://claude.ai/code/session_01TPpTHPokxsZ3dQpRzg4NkD
@coderabbitai

coderabbitai Bot commented Jun 15, 2026

Copy link
Copy Markdown

Warning

Review limit reached

@Liohtml, we couldn't start this review because you've reached your PR review rate limit.

More reviews will be available in 32 minutes and 1 second. Learn how PR review limits work.

Your organization has used up its prepaid credits, and credit purchases are no longer available. Enable the review add-on in the billing tab to keep reviews running — you're only billed for reviews past your plan's rate limits ($0.25/file).

⌛ How to resolve this issue?

After more reviews become available, 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 include higher PR review limits than trial, open-source, and free plans. In all cases, reviews become available again over time. During sustained high-volume PR review activity, CodeRabbit may temporarily slow when the next review becomes available.

Please see our Fair Usage Limits Policy for further information.

ℹ️ Review info
⚙️ Run configuration

Configuration used: defaults

Review profile: CHILL

Plan: Pro Plus

Run ID: e0c20953-7ebc-48b4-b3ff-c7d9d17d5039

📥 Commits

Reviewing files that changed from the base of the PR and between 3bdd0a1 and 8430cfb.

📒 Files selected for processing (2)
  • .github/workflows/ci.yml
  • CHANGELOG.md
✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch claude/bundle-c-ci-audit

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

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

Make the audit deterministic and avoid clobbering a committed lockfile: only
run 'cargo generate-lockfile' when no Cargo.lock exists, otherwise audit the
committed one. Resolves the review note about non-deterministic resolution and
the interaction with committing Cargo.lock (Bundle D).

https://claude.ai/code/session_01TPpTHPokxsZ3dQpRzg4NkD
@Liohtml
Liohtml merged commit 3e9a46d into main Jun 16, 2026
4 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[repo-health] Medium: no dependency vulnerability audit in CI (cargo-audit / cargo-deny missing)

2 participants