dependabot-review

name

dependabot-review

description

Triage, review, and merge Dependabot dependency-bump PRs + clear Dependabot security alerts for the whiteboard repo. Security-first, semver x ecosystem classification, verify-CI + `pnpm audit --prod` gate awareness, conflict-cascade-safe batch merging by the single integrator. Adapted for this repo's constraints — NO GitHub Issues (backlog -> Tasks/tmp-issues), release-please conventional commits, npm-published runtime-dep priority. Pairs with the dependabot-triage workflow (which does the read-only analysis fan-out).

dependabot-review (whiteboard)

Triage Dependabot PRs by semver level + ecosystem and safely merge them, and keep the security-alert backlog clearing. The dependabot-triage workflow does the read-only analysis (classify + changelog + repo-impact + CI + adversarial "safe to merge?" + alert coverage); this skill is the integrator's execution loop — judgement + merge mechanics.

How this repo differs from a generic dependabot flow

Read these before applying the reference playbook — they change several steps:

• NO GitHub Issues. This project uses native Tasks (live board) + tmp/issues/ (durable private backlog) — see the ticketing skill. So never gh issue create for a migration/feature follow-up; file a tmp/issues/<slug>.md or a TaskCreate instead.
• Single integrator owns git/CI/merge. The main session merges; there is no team of reviewers approving on GitHub. Merges go to the working branch / main per the push model.
• release-please reads merged commits. Dependabot already titles PRs chore(deps): bump … / chore(deps-dev): … — keep that on squash merge so release-please classifies them as no-bump chores. Do not rewrite the title to fix:/feat:.
• The CI gate is verify (.github/workflows/ci.yml): pnpm audit --prod --audit-level=high → validate-skills → noConsole → typecheck → test → stdio/template/packaged smokes → build → web-app smokes. A prod high+ vuln BLOCKS merge via that audit step. dev-only vulns do not.
• npm-published artifact. @kamiazya/whiteboard-mcp ships its runtime deps. Prioritize runtime-scope bumps/alerts (they reach users AND gate pnpm audit --prod) over dev-only ones.
• Local pre-push gate (lefthook) runs typecheck + mcp-node + noconsole before push; CI is authoritative. For the post-merge verification run, see Step 5.

Load-bearing runtime deps (extra scrutiny)

These ship in the published package and/or anchor a cross-process contract — read the changelog even for a minor, and run the relevant smoke after merging:

Package	Why it is load-bearing	Verify after bump
zod	schema single source of truth (z.infer, parse at every boundary)	pnpm smoke:e2e + typecheck; a minor can shift inference/parse semantics
loro-crdt	CRDT persistence/merge core	reconcile-elements + useWhiteboardSync + export-json tests green
@modelcontextprotocol/sdk	gates every MCP tool contract	initialize negotiation + tools/list + pnpm smoke:e2e; re-check the protocol matrix in docs/contributing/mcp-debugging.md
hono / @hono/node-server	HTTP routes + daemon transport	route tests + pnpm mcp:http:dev reachable
kysely / @libsql/*	persistence query layer	store/route tests
@excalidraw/*	canvas rendering surface	web-app smokes / browser tests
jose, ws, pino, nanoid	auth tokens / websocket / logging / ids	nearest tests

@types/node must match .node-version (currently 24) — if a @types/node PR bumps the major past the runtime, close it rather than merge.

Flow

Step 1 — Triage (run the workflow)

Workflow({ scriptPath: '.claude/workflows/dependabot-triage.workflow.mjs', args: { includeAlerts: true } })

Returns plan.mergeOrder (Security > patch > minor > major; supersedes resolved first), plan.needsMigration (backlog candidates), and plan.alertCoverage (each alert → fixed-by-pr / needs-manual-bump / transitive / dev-only). Scope to specific PRs with args:{prs:[58,53]}.

The alerts API needs security_events/admin scope; if it 404s, run gh auth refresh -s security_events or read the Security tab and note alerts as unavailable — do not block PR triage on it.

Step 2 — Resolve supersedes & close stale PRs first

Dependabot leaves stale PRs when a newer bump of the same package opens (e.g. an earlier hono → 4.12.16 superseded by a later hono → 4.12.21). Close the superseded one so it does not consume a merge/rebase cycle:

gh pr close <stale-number> --comment "Superseded by #<newer> (bumps <pkg> further)."

Step 3 — Merge loop (conflict-cascade-safe)

Every PR touches pnpm-lock.yaml, so merging one almost always conflicts the rest — that is expected. Rebase only the one PR you will merge next (rebasing all triggers O(N²) CI runs).

Repeat until the plan is drained:

1. Confirm the next PR's verify is green (not just mergeable): gh pr checks <n>. mergeStateStatus: BLOCKED with a passing verify is just branch-protection awaiting the merge action — fine to merge. A failing verify is not.
2. Merge (squash, keep the chore(deps): title for release-please):

   gh pr merge <n> --squash
   

   Do not use --auto (errors when repo auto-merge is off; only consider it for a pending CI after confirming with the human).
3. Rebase only the next PR in the plan: gh pr comment <next> --body "@dependabot rebase". Wait — Dependabot takes a few minutes and CI re-runs after rebase; pre-rebase CI is void.
4. Back to 1.

CodeRabbit/AccessLint do not need to pass to merge a dep bump — verify is the gate. (CodeRabbit skips Draft/WIP PRs anyway; see the ci-triage skill.)

Step 4 — Majors / breaking bumps → backlog (NOT a GitHub issue)

For plan.needsMigration items (adversarial verify found a breaking change reaching our code): do NOT merge. File a backlog entry per the ticketing skill:

• tmp/issues/deps-migrate-<pkg>-<from>-to-<to>.md (frontmatter: id/status/severity/owner/ blocked-by/related/created) with: breaking changes, affected paths, official migration guide link, and an action checklist. Reference the Dependabot PR number in the body.
• Leave a pointer on the PR: gh pr comment <n> --body "Migration tracked locally; holding this bump." (Do not close it — Dependabot will keep it rebased until the migration lands.)

A major is not automatically a migration: if the only breaking change is a runtime floor we already meet (e.g. "requires Node ≥ 20") or a GitHub Actions runner bump, and verify is green, merge it (mark merge-with-care and smoke after).

Step 5 — Post-merge verification

After draining the merge loop, on the integration branch:

pnpm install
pnpm -r typecheck
pnpm test
pnpm smoke:e2e
pnpm audit --prod --audit-level=high   # mirrors the CI gate — must be clean to merge

If a load-bearing dep moved (zod / loro-crdt / MCP SDK / hono), also run its smoke from the table above. If tests/audit fail, bisect to the offending bump and either pin it back or fix the call site; report which bump caused it.

Step 6 — Clear the alert backlog

From plan.alertCoverage:

• fixed-by-pr → cleared once that PR merges; re-run pnpm audit --prod to confirm.
• needs-manual-bump (direct dep, no PR) → bump it yourself (pnpm up <pkg>@<firstPatched> or edit package.json + pnpm install), run Step 5, commit fix(deps): bump <pkg> to <ver> (GHSA-…). Use fix(deps) (not chore) when it closes a real vuln so release-please records a patch bump.
• transitive-no-fix → add a pnpm.overrides pin to the patched version if one exists; if not, file tmp/issues/ and wait for upstream.
• dev-only-nonci → does NOT gate pnpm audit --prod; lowest priority. Batch with the next weekly Dependabot run rather than churning CI now.

pnpm split-instance gotcha (React Context)

This is a pnpm monorepo with React in both apps/web and packages/mcp-server. If a React-family bump (react / react-dom) lands and tests start failing with Cannot read properties of null (reading 'useContext') or No QueryClient set, the virtual store has two React versions (Provider and Consumer became different instances). Diagnose:

node -e "const c=require('fs').readFileSync('pnpm-lock.yaml','utf8');console.log([...new Set(c.match(/react@[0-9.]+/g))].sort())"

Fix by unifying the version across workspaces (align both package.json specs; add a pnpm.overrides pin if a transitive dep pulls a second copy), then @dependabot rebase and re-run CI.

Notes

• Always read the changelog — even patches. "It's a patch" / "the diff is only the lockfile" is not a reason to skip. The workflow does this; spot-check load-bearing ones yourself.
• Squash-merge, keep the chore(deps): title (release-please depends on it).
• Rate limits: leave intervals when rebasing/merging many PRs; the alerts API + WebFetch can rate-limit on the free tier.
• GitHub Security alerts ⇄ pnpm audit --prod can diverge: a dev-only path vuln shows in the Security tab but not in pnpm audit --prod, so it does not gate CI. Track those in tmp/issues/ rather than churning the lockfile.