Skip to content

Add cargo-vet supply-chain security#44

Open
jerrysxie wants to merge 10 commits into
OpenDevicePartnership:mainfrom
jerrysxie:add-supply-chain-security
Open

Add cargo-vet supply-chain security#44
jerrysxie wants to merge 10 commits into
OpenDevicePartnership:mainfrom
jerrysxie:add-supply-chain-security

Conversation

@jerrysxie

@jerrysxie jerrysxie commented May 12, 2026

Copy link
Copy Markdown
Contributor

This PR adds supply-chain security tooling based on the
embedded-rust-template:

  • cargo-vet (supply-chain/) – dependency audit tracking with imports
    from ODP shared audits, Google, and Mozilla.
  • CI workflowscargo-vet.yml + PR comment workflow

Copilot AI review requested due to automatic review settings May 12, 2026 20:21
@jerrysxie jerrysxie requested a review from a team as a code owner May 12, 2026 20:21

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

Adds Rust supply-chain tooling centered around cargo-vet, including a repo-local supply-chain/ configuration and CI automation to run cargo vet on PRs and (optionally) comment results back on the PR.

Changes:

  • Introduces supply-chain/ cargo-vet configuration scaffolding (config, audits, and imports lock).
  • Adds a cargo-vet PR workflow to run cargo vet --locked.
  • Adds a follow-up workflow_run workflow to post/update a PR comment and apply a label based on the cargo-vet result.

Reviewed changes

Copilot reviewed 4 out of 5 changed files in this pull request and generated 7 comments.

Show a summary per file
File Description
supply-chain/imports.lock Initializes cargo-vet imports lock sections for upstream audit sources.
supply-chain/config.toml Configures cargo-vet and upstream audit import URLs.
supply-chain/audits.toml Adds the local audits file scaffold.
.github/workflows/cargo-vet.yml Runs cargo-vet in CI and uploads the PR number artifact.
.github/workflows/cargo-vet-pr-comment.yml Posts/updates PR comments (and labels) based on cargo-vet workflow results.

Comment thread .github/workflows/cargo-vet.yml
Comment thread .github/workflows/cargo-vet-pr-comment.yml
Comment thread .github/workflows/cargo-vet-pr-comment.yml
Comment thread .github/workflows/cargo-vet-pr-comment.yml
Comment thread .github/workflows/cargo-vet-pr-comment.yml
Comment thread .github/workflows/cargo-vet-pr-comment.yml
Comment thread .github/workflows/cargo-vet.yml
felipebalbi
felipebalbi previously approved these changes May 18, 2026
RobertZ2011
RobertZ2011 previously approved these changes May 28, 2026
@jerrysxie jerrysxie changed the title Add cargo-vet and cargo-deny supply-chain security Add cargo-vet supply-chain security May 28, 2026
@jerrysxie jerrysxie self-assigned this May 28, 2026
@github-project-automation github-project-automation Bot moved this to In progress in ODP v0.2 May 28, 2026
Add explicit audit-as-crates-io policy entries for workspace crates that share published versions on crates.io.

Import bytecode-alliance audits and refresh imports.lock so cargo-vet can resolve bitflags coverage and pass in CI.

Assisted-by: GitHub Copilot:GPT-5.3-Codex
@jerrysxie jerrysxie dismissed stale reviews from RobertZ2011 and felipebalbi via fb010de June 9, 2026 20:29
jerrysxie added 2 commits June 9, 2026 15:34
The all-features CI build pulls proc-macro-error2 via defmt-macros, which triggers RUSTSEC-2026-0173 (unmaintained). No safe upgrade is available and migration must happen upstream, so add it to the advisories ignore list alongside the existing RUSTSEC-2024-0370 entry.

Assisted-by: GitHub Copilot:claude-opus-4.8
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Status: In progress

Development

Successfully merging this pull request may close these issues.

4 participants