Review

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

vibeforge1111 46948e7 1.1 KB Updated

File contents

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.

vibeforge1111/founder-arena/tree/main/.claude/skills/review commit 46948e78ab

Frequently asked questions

npx skillmds@latest add vibeforge1111/review