| name | review-checklist |
| description | Review a diff against the team's checklist. Use when the user asks for a code review, diff review, or pre-merge check. |
Review checklist
Review the diff (git diff main... or the given PR) against exactly this list. One line per finding: file:line — problem — fix. No essays.
The list
- Escaping — every string rendered into HTML goes through
esc(). Any raw interpolation into innerHTML is a blocker.
- Tokens — no hard-coded colors/fonts in CSS; must use
var(--…) tokens.
- Data invariants — habits still 100 total, 10 per pack, card shape respected (
node --test must pass).
- Dependencies — diff adds no npm package, CDN script, or build step. Any new dependency is a blocker.
- Mobile & keyboard — new UI works at 360px and via arrows/j/k; interactive elements are real
<button>s.
- Motion — new animation respects
prefers-reduced-motion.
- Leftovers — no
console.log, debugger, commented-out code, or TODO without an owner.
- Voice — new card copy matches house voice (short, imperative, no hype).
Finish with a verdict: merge, merge after nits, or blocked.
