AngelDann/code-quality

Code Quality Review

Review code quality in the directory provided by the user.

Instructions

1. Identify files to review:

   • Find all .py files in the directory
   • Exclude migrations, __pycache__, and generated files

2. Run automated checks (use the project's toolchain; prefer uv run if uv is configured, otherwise activate the venv and run):

   uv run ruff check <directory>
   uv run ruff format --check <directory>
   uv run pyright <directory>
   uv run pytest <directory> -v
   

   Equivalents without uv:

   ruff check <directory>
   ruff format --check <directory>
   pyright <directory>
   pytest <directory> -v
   

3. Manual review checklist:

   • [ ] No Any types without justification
   • [ ] Proper error handling (no silent exceptions)
   • [ ] N+1 queries avoided (select_related/prefetch_related)
   • [ ] Forms have proper validation
   • [ ] Views return correct HTTP status codes
   • [ ] HTMX partials handle HX-Request header
   • [ ] Celery tasks are idempotent
   • [ ] Tests use factories, not raw object creation

4. Report findings organized by severity:

   • Critical (must fix)
   • Warning (should fix)
   • Suggestion (could improve)

Source

Adapted from claude-code-django (.claude/skills/code-quality).