← ClaudeAtlas

forge-reviewlisted

This skill should be used to run a comprehensive, multi-lens code review of a pull request, a branch, or the local working changes in a Context Forge methodology project — phrases like "forge-review", "review this PR", "review my diff", "review the branch before I push", or "review PR 42". It resolves the review scope, loads the project's context files, builds a mechanical inventory of the diff, reviews across quality lenses (spec, standards, tests, errors, types, comments, simplicity) with per-item coverage tracked in a persistent review ledger, gates on confidence, and reports findings ranked by severity. With --until-clean it loops review → fix → re-review until one full pass finds nothing. Read-only — reviews and reports, never fixes.
yerros/context-forge · ★ 0 · Code & Development · score 75
Install: claude install-skill yerros/context-forge
# forge-review A comprehensive, multi-perspective review for a change that is about to ship — a PR, a branch, or the uncommitted working tree. Where `forge-verify` is the pre-close gate for **one spec'd unit** (checklist + build + tests + tiered review, hard PASS/FAIL), `forge-review` is the broader **quality sweep** over an arbitrary diff: many lenses, confidence-gated, severity-ranked, no unit required. Read-only. This skill reviews and reports; it never edits code. Route fixes back through `forge-fix` / `forge-build`. ## Argument Text after the command sets the scope and the lenses: - **PR** — a number or URL (`/forge-review 42`, `/forge-review https://github.com/…/pull/42`) → review that PR's diff. - **No PR** → review the current branch's open PR if one exists (`gh pr view`); otherwise review the branch's diff vs its base; otherwise the local working changes (`git diff` + staged + untracked). - **`--focus=<lenses>`** — comma-separated, limits the review to those lenses (see the lens table). No focus → run every applicable lens. - **`parallel`** — fan the lenses out across subagents instead of one in-session pass (see Execution). Default is the cheaper single pass. - **`--until-clean`** — don't stop at the report: loop review → fix (via `forge-fix`) → re-review until one full pass finds no Critical/Important (see Convergence). The flag is the user's consent to route fixes automatically. Confirm the resolved scope (which diff, which base) before spendin