# Review

> Pre-commit review pipeline. Runs tests, build check, then reviews the diff.

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

---


## Pre-flight

!`git diff --name-only main...HEAD 2>/dev/null || git diff --name-only HEAD~1 2>/dev/null || echo "No diff available"`

!`pytest tests/ --tb=short 2>&1 | tail -20`

!`cd frontend && npm run build 2>&1 | tail -15`

## Diff

!`git diff main...HEAD 2>/dev/null || git diff HEAD~1 2>/dev/null || git diff --cached`

## Instructions

1. If any pre-flight check failed, list failures first with exact fixes.

2. Review the diff using the reviewer agent's checklist:
   - **Bugs**: Logic errors, null risks, race conditions, silent error swallowing, fetchJSON header pattern
   - **Security**: Unvalidated input, missing auth, token leakage, IDOR
   - **Performance**: Three.js object allocation in loops, DOM thrashing, unbounded state serialization
   - **Test gaps**: Untested critical paths

3. For each issue: file, line, what's wrong, how to fix it. Be specific.

4. Verdict: SHIP IT / NEEDS WORK / BLOCKED.

5. If SHIP IT: suggest the commit message in conventional commits format.