AI Summary1. Run to identify changed files 2. Read modified files to understand the intent of changes 3. Review against the checklist below
Install
Copy this and paste it into Claude Code, Cursor, or any AI assistant:
I want to install the "code-review" skill in my project. Please run this command in my terminal: # Install skill into your project (2 files) mkdir -p .claude/skills/unknown-jcdoll-agents && curl --retry 3 --retry-delay 2 --retry-all-errors -o .claude/skills/unknown-jcdoll-agents/SKILL.md "https://raw.githubusercontent.com/majiayu000/claude-skill-registry/main/skills/development/unknown-jcdoll-agents/SKILL.md" && curl --retry 3 --retry-delay 2 --retry-all-errors -o .claude/skills/unknown-jcdoll-agents/metadata.json "https://raw.githubusercontent.com/majiayu000/claude-skill-registry/main/skills/development/unknown-jcdoll-agents/metadata.json" Then restart Claude Code (or reload the window in Cursor) so the skill is picked up.
Description
Reviews code for quality, security, and maintainability. Use after implementing features, completing significant work, or when user asks for code review.
Instructions
• Run git diff to identify changed files • Read modified files to understand the intent of changes • Review against the checklist below • Provide structured feedback organized by priority
Security
• No exposed credentials, API keys, or secrets • Input validation implemented where needed • File paths are sanitized (no path traversal) • External commands are safely constructed (no injection)
Code Quality
• Variables and functions are clearly named • Functions have single responsibility • No code duplication (DRY principle) • Complex logic has explanatory comments • Error handling is comprehensive • No unused imports or variables • Clean separation of concerns
Parameters
• Use enums rather than bare strings for parameters • Do not spread "magic strings" throughout the code, use a single source of truth for parameter defaults (config.py) • Do not use the "if (check config) then (value = from config) else (value = default)" pattern - this breaks the single source of truth rule
Discussion
Health Signals
My Fox Den
Community Rating
Sign in to rate this booster