Skip to content

ci: add dependency-review-action#2257

Open
16francej wants to merge 2 commits into
mainfrom
dependency-review-action
Open

ci: add dependency-review-action#2257
16francej wants to merge 2 commits into
mainfrom
dependency-review-action

Conversation

@16francej

Copy link
Copy Markdown
Collaborator

Summary

  • Adds actions/dependency-review-action@v4 on every PR, scanning dependency manifest changes (package.json / bun.lock) against the GitHub Advisory Database.
  • Fails the check when a newly introduced dependency has a known high or critical vulnerability; posts a summary comment on the PR only when it fails.
  • Repo is public so the dependency graph is already enabled — no settings change needed.

Notes

  • Runner matches existing workflows (ubicloud-standard-8); the job is a single API-backed diff check, so it's fast and free of repo build steps.
  • Doesn't touch VERSION/CHANGELOG (CI-only change, no user-facing behavior).

🤖 Generated with Claude Code

Fails PRs that introduce dependencies with known high/critical
vulnerabilities; posts a summary comment on failure.

Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
@trunk-io

trunk-io Bot commented Jul 13, 2026

Copy link
Copy Markdown

Merging to main in this repository is managed by Trunk.

  • To merge this pull request, check the box to the left or comment /trunk merge below.

After your PR is submitted to the merge queue, this comment will be automatically updated with its status. If the PR fails, failure details will also be posted here

@github-actions

Copy link
Copy Markdown

E2E Evals: ✅ PASS

0/0 tests passed | $0 total cost | 13 parallel runners

Suite Result Status Cost

13x ubicloud-standard-8 (Docker: pre-baked toolchain + deps) | wall clock ≈ slowest suite

Fixes applied:
- fail-on-scopes covers development deps, not just runtime (default)
- path-filter to dependency manifests + workflow files (GitHub Actions
  ecosystem changes are also reviewed)
- concurrency cancel-in-progress + 10min timeout, matching version-gate

Auto-fixed by Claude watch-pipeline

Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
@16francej
16francej requested a review from garrytan July 14, 2026 00:27
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.

1 participant