crush/skills/ecc-code-review/SKILL.md
# ECC Code Review Review checklist: - Correctness: logic matches requirements; handles errors; no silent failure - Readability: naming, small functions, clear invariants - Tests: coverage for new behavior; no brittle tests - Performance: no obvious N+1, unbounded loops, large allocations - DX: helpful logs; clear errors; docs updated if needed Provide: - Must-fix items - Nice-to-have improvements - Suggested patch/diff if practical
npx skillsauth add e2ec-it/pub-xifan-crush-best-practices crush/skills/ecc-code-reviewInstall this skill globally with one command. Works with Claude Code, Cursor, and Windsurf.
3 of 9 scanners reported clean
Some scanners were skipped, did not run, or reported a non-clean status. Review each row below.
Review checklist:
Provide:
content-media
SUPERPOWERS-style implementation planning: turn an approved design into 2–5 minute tasks with exact file paths, commands, verifications, and frequent commits.
development
SUPERPOWERS TDD enforcement: NO production code without a failing test first. Delete code written before tests.
development
Execute plan per task with role separation and review loops (spec then quality). In Crush, simulate subagents via role prompts / new sessions.
development
Request review between tasks and before merge. Use git SHAs to scope changes and produce actionable findings.