Skip to content

chore(skills): bump-transitive skill - #2367

Merged
r0b1n merged 1 commit into
mainfrom
ai/deps-update
Jul 30, 2026
Merged

chore(skills): bump-transitive skill#2367
r0b1n merged 1 commit into
mainfrom
ai/deps-update

Conversation

@r0b1n

@r0b1n r0b1n commented Jul 30, 2026

Copy link
Copy Markdown
Collaborator

No description provided.

@r0b1n
r0b1n requested a review from a team as a code owner July 30, 2026 13:27
@r0b1n r0b1n changed the title chore(skills): rename fix-vuln-transitive to bump-transitive chore(skills): bump-transitive skill Jul 30, 2026
@github-actions

Copy link
Copy Markdown
Contributor

AI Code Review

✅ Approved — no issues found


What was reviewed

File Change
.agents/skills/README.md Added bump-transitive to directory listing and slash command table
.agents/skills/bump-transitive/SKILL.md New skill — full process for surgically bumping a vulnerable transitive dep via pnpm.overrides

Skipped (out of scope): dist/, pnpm-lock.yaml


Positives

  • Skill frontmatter (name, description) is present and accurate — description precisely matches what the skill does.
  • The two-install pattern (add overrides → install → remove overrides → install) is correctly documented; the rationale for each step is explained so an agent can reason about edge cases rather than blindly follow steps.
  • "What to flag" equivalent is captured concisely in the Rules section with clear negative examples (pnpm update, pnpm upgrade, --frozen-lockfile).
  • The verification step (git diff … | grep) gives the agent a concrete pass/fail signal rather than asking it to eyeball the lockfile diff.
  • The guard for direct dependencies (last rule) prevents the skill from being misused when an override is unnecessary.
  • README directory listing and slash command table kept in sync — no stale references to the old fix-vuln-transitive name.

@r0b1n
r0b1n merged commit a5f06c0 into main Jul 30, 2026
14 of 15 checks passed
@r0b1n
r0b1n deleted the ai/deps-update branch July 30, 2026 14:07
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.

3 participants