Add Dependency Review GitHub Action#1031
Conversation
Adds the dependency-review-action workflow to scan PRs for dependency changes that introduce known vulnerabilities. Co-Authored-By: Itamar Hartstein <haritamar@gmail.com>
🤖 Devin AI EngineerI'll be helping with this pull request! Here's what you should know: ✅ I will automatically:
Note: I can only respond to comments from users who have write access to this repository. ⚙️ Control Options:
|
|
👋 @haritamar |
|
Note Reviews pausedIt looks like this branch is under active development. To avoid overwhelming you with review comments due to an influx of new commits, CodeRabbit has automatically paused this review. You can configure this behavior by changing the Use the following commands to manage reviews:
Use the checkboxes below for quick actions:
📝 WalkthroughWalkthroughAdds a dependency review config and GitHub Actions workflow for pull requests and manual runs, with restricted permissions and license-based review rules. ChangesDependency Review Workflow
Estimated code review effort🎯 2 (Simple) | ⏱️ ~10 minutes Poem
🚥 Pre-merge checks | ✅ 5✅ Passed checks (5 passed)
✨ Finishing Touches🧪 Generate unit tests (beta)
Comment |
There was a problem hiding this comment.
Actionable comments posted: 2
🤖 Prompt for all review comments with AI agents
Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.
Inline comments:
In @.github/workflows/dependency-review.yml:
- Around line 14-15: The Dependency Review workflow is using a mutable action
reference, so update the uses entry in the Dependency Review step to pin
actions/dependency-review-action from the workflow to a specific commit SHA
instead of `@v4`. Keep the step name unchanged, and replace the version tag with
the immutable SHA that corresponds to the intended release.
- Around line 12-13: The workflow still uses floating major tags for
actions/checkout and actions/dependency-review-action, so update both to
immutable commit SHAs in the dependency-review workflow. In the checkout step,
also set persist-credentials to false so the token is not left in git config
after checkout. Use the existing action names to locate the two steps and
replace only the action references and checkout options.
🪄 Autofix (Beta)
Fix all unresolved CodeRabbit comments on this PR:
- Push a commit to this branch (recommended)
- Create a new PR with the fixes
ℹ️ Review info
⚙️ Run configuration
Configuration used: defaults
Review profile: CHILL
Plan: Pro
Run ID: 37a4bdb1-b425-4d50-9f37-2b051e5f033e
📒 Files selected for processing (1)
.github/workflows/dependency-review.yml
Co-Authored-By: Itamar Hartstein <haritamar@gmail.com>
Co-Authored-By: Itamar Hartstein <haritamar@gmail.com>
Co-Authored-By: Itamar Hartstein <haritamar@gmail.com>
There was a problem hiding this comment.
Actionable comments posted: 1
🤖 Prompt for all review comments with AI agents
Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.
Inline comments:
In @.github/workflows/dependency-review.yml:
- Line 2: The dependency review workflow currently allows workflow_dispatch
without any comparison refs, so manual runs have no base/head range to evaluate.
Update the dependency-review workflow by adding explicit base-ref and head-ref
inputs for workflow_dispatch and pass them through to
actions/dependency-review-action, using the existing workflow trigger and action
wiring to locate the manual-run configuration. If you do not want to support
manual runs, remove workflow_dispatch entirely.
🪄 Autofix (Beta)
Fix all unresolved CodeRabbit comments on this PR:
- Push a commit to this branch (recommended)
- Create a new PR with the fixes
ℹ️ Review info
⚙️ Run configuration
Configuration used: defaults
Review profile: CHILL
Plan: Pro
Run ID: e96d6d25-de96-4cc8-8e7b-1b22586e3109
📒 Files selected for processing (1)
.github/workflows/dependency-review.yml
Co-Authored-By: Itamar Hartstein <haritamar@gmail.com>
Co-Authored-By: Itamar Hartstein <haritamar@gmail.com>
Add deny-licenses for AGPL, SSPL, BUSL, Elastic. Co-Authored-By: Itamar Hartstein <haritamar@gmail.com>
Co-Authored-By: Itamar Hartstein <haritamar@gmail.com>
Co-Authored-By: Itamar Hartstein <haritamar@gmail.com>
Co-Authored-By: Itamar Hartstein <haritamar@gmail.com>
Co-Authored-By: Itamar Hartstein <haritamar@gmail.com>
Co-Authored-By: Itamar Hartstein <haritamar@gmail.com>
Co-Authored-By: Itamar Hartstein <haritamar@gmail.com>
Summary
Adds
actions/dependency-review-action@v4workflow to scan PRs for vulnerable or improperly-licensed dependency changes.pull_requestandworkflow_dispatch(manual runs requirebase-refinput, e.g. a tag or commit SHA, to define the comparison range;head-refdefaults to HEAD)fail-on-severity: high— only blocks PRs on HIGH/CRITICAL vulnerabilities (moderate/low reported but don't fail)persist-credentials: falseon checkout for securityallow-licensesallowlist covering permissive (MIT, Apache-2.0, BSD, ISC, etc.) and weak copyleft (LGPL, MPL-2.0) — blocks strong copyleft (GPL, AGPL), proprietary, SSPL, BUSL, Elastic, and any other unlisted licenseLink to Devin session: https://app.devin.ai/sessions/0f830b140bd8488797ab340ef05dc88f
Requested by: @haritamar
Summary by CodeRabbit