jforksy

review

Comprehensive code review checking logging, error handling, TypeScript, and production readiness

jforksy 2 1 Updated 7mo ago
GitHub

Install

npx skillscat add jforksy/claude-skills/review

Install via the SkillsCat registry.

About this skill

Here's a thinking process: 1. **Analyze the Request:** - **Input:** An AI agent skill description from a public software directory. - **Author-provided short description:** "Comprehensive code review checking logging, error handling, TypeScript, and production readiness" - **SKILL.md excerpt:** Provided, showing a detailed code review task with specific checks and output format. - **Task:** Write a 2-3 sentence plain-text summary in English.

SKILL.md

Code Review Task

Perform comprehensive code review. Be thorough but concise.

Check For:

Logging - No console.log statements, uses proper logger with context
Error Handling - Try-catch for async, centralized handlers, helpful messages
TypeScript - No any types, proper interfaces, no @ts-ignore
Production Readiness - No debug statements, no TODOs, no hardcoded secrets
React/Hooks - Effects have cleanup, dependencies complete, no infinite loops
Performance - No unnecessary re-renders, expensive calcs memoized
Security - Auth checked, inputs validated, RLS policies in place
Architecture - Follows existing patterns, code in correct directory

Output Format

✅ Looks Good

  • [Item 1]
  • [Item 2]

⚠️ Issues Found

  • [Severity] [File:line] - [Issue description]
    • Fix: [Suggested fix]

📊 Summary

  • Files reviewed: X
  • Critical issues: X
  • Warnings: X

Severity Levels

  • CRITICAL - Security, data loss, crashes
  • HIGH - Bugs, performance issues, bad UX
  • MEDIUM - Code quality, maintainability
  • LOW - Style, minor improvements