reviewing-dependency-changes
Use this skill when a PR diff contains changes to dependency manifest files (package.json, .csproj, Cargo.toml, go.mod, requirements.txt, etc.) or when reviewing Renovate/Dependabot bot PRs. Evaluates new dependencies for AppSec approval process compliance, major version bump significance, lock file hygiene, and dependency removal completeness. Does NOT perform deep security or license analysis — that is handled by the bitwarden-security-engineer plugin's reviewing-dependencies skill.
- Installs
- 70
- Last 24h
- —
- Trending rank
- —
- Security
- Not audited
Install
Works with every agent the skills CLI supports.
npx skills add https://github.com/bitwarden/ai-plugins --skill reviewing-dependency-changes- 01
Install
Run the command in your project. The CLI asks which agents to install for.
- 02
Describe the task
Ask your agent for something this skill covers. It loads reviewing-dependency-changes automatically when relevant.
- 03
Review the result
The agent follows the SKILL.md steps, scripts and references bundled with the skill.
SKILL.md
The instructions your agent reads when the skill is triggered.
Reviewing Dependency Changes
Manifest File Detection
Flag this skill when any of these files appear in the diff:
package.json,package-lock.json*.csproj,Directory.Packages.props,packages.lock.jsonCargo.toml,Cargo.lockgo.mod,go.sumrequirements.txt,pyproject.toml,poetry.lockGemfile,Gemfile.lock
Area 1: New Dependencies
When a PR adds a dependency that was not previously in the codebase, Bitwarden's Dependency Review and Approval process requires AppSec review and approval before integration. This applies to all new dependencies — production, dev, and test.
The submitter must provide the package name/version, ecosystem, justification, scope, affected products, and what it replaces. A security engineer creates a VULN task in Jira and evaluates the dependency across security (known CVEs, exploitability), license compatibility (permissive licenses like MIT/Apache-2.0 are acceptable; copyleft licenses like GPL/AGPL are flagged), maintenance health (active maintainers, recent releases, security policy), supply chain risk (typosquatting, ownership changes, obfuscated install scripts), and transitive dependencies before rendering an approval decision.
What to Check
- Is this a net-new dependency (not already present in the codebase)?
- Does the PR description contain an approval signal indicating the process was followed?
Approval Signals
Evidence that the dependency approval process was followed:
- PR description references a VULN task (e.g.,
VULN-1234) - PR description explicitly mentions AppSec approval or the dependency review process
Severity
When emitting a finding that references the Dependency Review and Approval process, always link the process name to https://bitwarden.atlassian.net/wiki/spaces/APPSEC/pages/2774466657/Dependency+Review+and+Approval so the posted review comment points reviewers to the canonical documentation.
- No approval signal found → ⚠️ IMPORTANT: New dependency
<package>added. Bitwarden requires AppSec approval before introducing new dependencies. The submitter should reach out to the AppSec team to initiate the Dependency Review and Approval process. - Unclear whether approval was obtained → ❓ QUESTION: Was AppSec approval obtained for the new
<package>dependency?
What NOT to Flag
- Dependencies that already exist in the codebase (version updates are not new dependencies)
- Dependencies added by Renovate/Dependabot as transitive dependency updates (these are part of Stage 5 monitoring for existing approved dependencies)
Area 2: Major Version Bumps
A major version bump (e.g., v2 → v3) may introduce breaking changes that affect Bitwarden's codebase.
What to Check
- Is this a SemVer major version change?
- Does the PR description discuss breaking changes or migration steps?
Severity
- Major bump without migration discussion → ❓ QUESTION: This bumps
<package>from vXto vY(major). Were breaking changes evaluated? - Version downgrade → ⚠️ IMPORTANT:
<package>is being downgraded from vXto vY. This is unusual and may reintroduce resolved vulnerabilities.
Area 3: Lock File Hygiene
Lock files ensure reproducible builds. Inconsistencies between manifests and lock files are a build reliability and security concern.
What to Check
| Scenario | Finding |
|---|---|
| Manifest changed, lock file not updated | ⚠️ IMPORTANT: Lock file not updated to reflect manifest changes |
| Lock file changed, no manifest change | ❓ QUESTION: Lock file changed without a corresponding manifest change — was this intentional (e.g., npm audit fix)? |
| Lock file deleted | ⚠️ IMPORTANT: Lock file removal breaks reproducible builds |
What NOT to Flag
- Large lock file diffs from a small manifest change — this is normal behavior. Lock files can change significantly from a single dependency addition or version bump.
- Lock file-only changes that accompany a clear manifest change in the same PR.
Area 4: Automated Dependency PRs
Renovate and Dependabot PRs are part of Bitwarden's Stage 5 (Monitoring) process. These automated updates to existing approved dependencies require different review treatment.
How to Detect
- PR author:
renovate[bot],dependabot[bot], or similar bot accounts - PR title pattern: "Update ...", "Bump ...", "chore(deps): ..."
Review Guidance
| Scenario | Action |
|---|---|
| Minor/patch update to existing dependency | No approval-process finding needed. Focus on lock file hygiene and CI status. |
| Major version bump from bot | Flag per Area 2 — major bumps warrant human review regardless of source. |
| Bot PR introduces a net-new dependency | Flag per Area 1 — new dependencies require the approval process regardless of source. |
Area 5: Dependency Removal
When a dependency is removed from a manifest, verify the removal is complete.
What to Check
- Are there remaining code references to the removed package?
- JavaScript/TypeScript:
import ... from '<package>',require('<package>') - C#/.NET:
using <namespace>, references in other.csprojfiles - Rust:
use <crate>::,extern crate <crate> - Python:
import <package>,from <package> import
- JavaScript/TypeScript:
- Are there references in build or infrastructure files?
Dockerfile,docker-compose.yml- CI workflow files (
.github/workflows/*.yml) - Build scripts,
Makefile, task runners
Severity
- Dead imports or references remain → ♻️ DEBT:
<package>removed from manifest but still referenced in code.
FAQ
What does the reviewing-dependency-changes skill do?
Use this skill when a PR diff contains changes to dependency manifest files (package.json, .csproj, Cargo.toml, go.mod, requirements.txt, etc.) or when reviewing Renovate/Dependabot bot PRs. Evaluates new dependencies for AppSec approval process compliance, major version bump significance, lock file hygiene, and dependency removal completeness. Does NOT perform deep security or license analysis — that is handled by the bitwarden-security-engineer plugin's reviewing-dependencies skill.
How do I install reviewing-dependency-changes?
Run npx skills add https://github.com/bitwarden/ai-plugins --skill reviewing-dependency-changes in your project. The skills CLI detects the agents you use and copies the skill into the right folder. For Claude Code you can also place the skill folder at ~/.claude/skills/reviewing-dependency-changes/ (personal) or .claude/skills/reviewing-dependency-changes/ (project).
Which AI agents can use reviewing-dependency-changes?
Any agent that supports the open Agent Skills format (a folder with a SKILL.md file), including Claude Code, Codex, Cursor, GitHub Copilot, Gemini CLI, Windsurf, Cline, OpenCode and more.
Is reviewing-dependency-changes safe to install?
No partner security audit is available yet. Skills can run commands and scripts, so read the SKILL.md and any bundled files before installing.
Who maintains reviewing-dependency-changes?
It is published by bitwarden in bitwarden/ai-plugins, and it is an official skill from the maker of the technology it covers. It has 70 recorded installs.
More from bitwarden/ai-plugins
reviewing-security-architecture
bitwarden/ai-plugins
Open the skill to read its SKILL.md instructions.
threat-modeling
bitwarden/ai-plugins
Open the skill to read its SKILL.md instructions.
retrospecting
bitwarden/ai-plugins
Open the skill to read its SKILL.md instructions.
avoiding-false-positives
bitwarden/ai-plugins
Use this skill to validate findings during a code review. For each finding, run the rejection criteria and verification checks. If a finding fails any check, drop it. In PR mode it also holds the once-per-review stacked-PR gate that decides whether completeness findings apply to the pull request at all.
reviewing-claude-config
bitwarden/ai-plugins
Open the skill to read its SKILL.md instructions.
posting-review-summary
bitwarden/ai-plugins
Open the skill to read its SKILL.md instructions.
Popular security skills
azure-compliance
microsoft/azure-skills
Run Azure compliance and security audits with azqr plus Key Vault expiration checks. Covers best-practice assessment, resource review, policy/compliance validation, and security posture checks. WHEN: compliance scan, security audit, BEFORE running azqr (compliance cli tool), Azure best practices, Key Vault expiration check, expired certificates, expiring secrets, orphaned resources, compliance assessment.
minimal-run-and-audit
lllllllama/rigorpilot-skills
Rigor Run skill for README-first deep learning repo reproduction. Use when the task is specifically to capture or normalize evidence from the selected smoke test or documented inference or evaluation command and write standardized reprooutputs/ files, including patch notes when repository files changed. Do not use for training execution, initial repo intake, generic environment setup, paper lookup, target selection, hidden scientific-meaning changes, or end-to-end orchestration by itself.
seo-audit
coreyhaines31/marketingskills
When the user wants to audit, review, or diagnose SEO issues on their site. Also use when the user mentions "SEO audit," "technical SEO," "why am I not ranking," "SEO issues," "on-page SEO," "meta tags review," "SEO health check," "my traffic dropped," "lost rankings," "not showing up in Google," "site isn't ranking," "Google update hit me," "page speed," "core web vitals," "crawl errors," "indexing issues," "stuck on page 2," "striking distance keywords," "title tag rewrite," "low CTR," "local SEO," "Google Business Profile," or "map pack." Use this even if the user just says something vague like "my SEO is bad" or "help with SEO" — start with an audit. For building pages at scale to target keywords, see programmatic-seo. For adding structured data, see schema. For AI search optimization, see ai-seo.
firebase-security-rules-auditor
firebase/agent-skills
Audits Firebase (Firestore, Cloud Storage) security rules for vulnerabilities, privilege escalation, role bypasses, create vs update inconsistencies, resource exhaustion, type safety, size limits, and hasOnly ownership checks. Use when auditing/reviewing rules, running red-team rule assessments, or scoring against auditor checklists. Don't use for Firebase CLI (login, deploy), Auth, Crashlytics, Remote Config, or database queries.
audit
pbakaus/impeccable
Run technical quality checks across accessibility, performance, theming, responsive design, and anti-patterns. Generates a scored report with P0-P3 severity ratings and actionable plan. Use when the user wants an accessibility check, performance audit, or technical quality review.
competitors
coreyhaines31/marketingskills
When the user wants to create competitor comparison or alternative pages for SEO and buyer-facing use. Also use when the user mentions 'alternative page,' 'vs page,' 'competitor comparison,' 'comparison page,' '[Product] vs [Product],' '[Product] alternative,' 'competitive landing pages,' 'how do we compare to X,' 'competitor teardown,' 'audit our competitor pages,' 'are our comparison pages out of date,' or 'competitive asset audit.' Use this for any content that positions your product against competitors. Covers four formats: singular alternative, plural alternatives, you vs competitor, and competitor vs competitor. For auditing existing claims (not researching competitors from scratch, which is competitor-profiling; not technical SEO on these pages, which is seo-audit), use the asset audit here. For internal battle cards and sales-specific competitor docs, see sales-enablement.