Agent Skills: Code Review Skill

Review code changes against hard rules and conventions

UncategorizedID: wellapp-ai/well/code-review

Install this agent skill to your local

pnpm dlx add-skill https://github.com/WellApp-ai/Well/tree/HEAD/cursor-rules/skills/code-review

Skill Files

Browse the full folder contents for code-review.

Download Skill

Loading file tree…

cursor-rules/skills/code-review/SKILL.md

Skill Metadata

Name
code-review
Description
Review code changes against hard rules and conventions

Code Review Skill

Systematically review code for quality, conventions, and potential issues.

When to Use

  • Before committing changes (auto-triggered)
  • Reviewing others' PRs
  • Self-review before pushing
  • Pre-push validation (Phase 0)

Phase 0: Pre-Push Validation

Before reviewing code, ensure build passes:

  • [ ] npm run typecheck - No type errors
  • [ ] npm run lint - No lint errors
  • [ ] No console.log in changed files
  • [ ] Tests pass (if applicable): npm run test

If any check fails, fix before proceeding to code review.

Quick check command:

npm run typecheck && npm run lint

Check for console.log:

git diff --cached --name-only | xargs grep -l "console.log" 2>/dev/null

Review Checklist

Hard Rules Check

  • [ ] No any type usage
  • [ ] No console.log statements
  • [ ] Components under 200 lines
  • [ ] No inline styles (Tailwind only)
  • [ ] No arbitrary values (px-[13px])
  • [ ] API follows 3-layer pattern

Code Quality Check

  • [ ] Clear variable/function names
  • [ ] No duplicated code
  • [ ] Proper error handling
  • [ ] Types are specific (not unknown everywhere)

Conventions Check

  • [ ] Follows existing patterns in codebase
  • [ ] Imports organized correctly
  • [ ] File in correct location (feature folder)

Security Check

  • [ ] No hardcoded secrets/API keys
  • [ ] No exposed sensitive data
  • [ ] Proper input validation

Output Format

Present findings as:


Code Review Summary

Files Changed: [count] Issues Found: [count by severity]

Critical (must fix)

  • [ ] file.ts:L42 - [issue description]

Warnings (should fix)

  • [ ] file.ts:L15 - [issue description]

Suggestions (nice to have)

  • [ ] file.ts:L88 - [suggestion]

Approved

  • [x] No blocking issues found

Proceed with commit? (yes / fix issues first)


Auto-Trigger

This skill is automatically invoked before every commit.