# Workbench Code Review

> Findings-first review discipline for code, diffs, task plans, live workflow changes, and implementation evidence.

- Skill: `fearvox/workbench-code-review` (Agent Skill)
- Install (CLI): `npx skillmds@latest add fearvox/workbench-code-review`
- Raw SKILL.md: https://api.skillmd.com/api/skills/fearvox/workbench-code-review/raw
- Safety review: pending
- Works with: Claude Code, Claude.ai, OpenAI Codex
- Category: Productivity
- Author: Fearvox (https://skillmd.com/u/fearvox)
- Updated: 2026-09-17
- Page: https://skillmd.com/skills/fearvox/workbench-code-review

---


# Workbench Code Review

Use this skill for reviewing code, diffs, task plans, live workflow changes, and implementation evidence before merge or closure.

## Review Stance

- Findings first. Do not lead with praise or a broad summary.
- Order findings by severity and user impact.
- Tie every finding to a file, command, issue/comment ID, run ID, or observable behavior.
- Prefer concrete repro or blast-radius evidence over style opinions.
- If no issues are found, say that clearly and name remaining test gaps.

## Severity

- `P0`: security, data loss, irreversible action, or broken primary workflow.
- `P1`: likely user-facing regression or unsafe automation.
- `P2`: correctness gap, missing edge case, or weak verification.
- `P3`: maintainability, clarity, or low-risk cleanup.

## Checklist

- Does the implementation match the promised objective?
- Are owner boundaries preserved?
- Are live mutations backed up and reversible?
- Are commands, IDs, and paths reproducible?
- Are tests or real-path checks sufficient for the blast radius?
- Did the change avoid unrelated refactors and hidden policy changes?

## Output Contract

For findings:

- `SEVERITY`
- `LOCATION`
- `ISSUE`
- `WHY IT MATTERS`
- `REQUIRED FIX`

For a clean review:

- `VERDICT: PASS`
- `EVIDENCE CHECKED`
- `TEST GAPS`
- `RESIDUAL RISK`

