Agent skill

code-reviewer

Assists with code review by analyzing code changes for quality, best practices, security, and potential issues. Activates after implementing code features, bug fixes, or refactorings. Provides structured feedback with critical issues, suggestions, and positive highlights.

View SKILL.md on GitHub Repository
Stars 1
Forks 0

Install this agent skill to your Project

npx add-skill https://github.com/fubira/claude-code-settings/tree/main/skills/code-reviewer

SKILL.md

Code Reviewer Skill

Review code changes for quality, security, and performance. Provide structured, actionable feedback.

Activation Triggers

  • After completing a feature, bug fix, or refactoring (automatic)
  • "review this code" (manual)
  • Before PR creation

Review Areas

  1. Correctness: Logic, bugs, edge cases, boundary values
  2. Quality: Language idioms, DRY, early return, duplication (per CLAUDE.md)
  3. Type Safety: Type annotations, null/undefined, off-by-one
  4. Performance: Unnecessary allocations, parallelization opportunities, data structure choice
  5. Security: Input validation, SQLi/XSS, secrets handling
  6. Testing: Coverage for new code paths, edge case tests
  7. Project Compliance: CLAUDE.md standards, consistency with existing patterns

Workflow

  1. Context: git diff to understand changes, check project CLAUDE.md, identify related tests
  2. Analysis: Review against above areas. Run mcp__ide__getDiagnostics for lint/type errors
  3. Test Verification: Check coverage and test quality
  4. Feedback: Report using the format below

Output Format

  1. Critical Issues (must fix): Bugs, vulnerabilities, breaking changes → file:line + fix suggestion
  2. Important Suggestions (should address): Performance, maintainability
  3. Minor Improvements (nice to have): Style, documentation
  4. Positive Highlights: Good implementations
  5. Next Steps: Prioritized recommended actions

Template details: templates/review-report.md

Decision Criteria

  • Correctness > cleverness. CLAUDE.md standards > general best practices
  • Provide specific, actionable feedback (file:line + code examples)
  • Investigate why unusual approaches pass tests before flagging them

Recommended Agent Skills

Expand your agent's capabilities with these related and highly-rated skills.

fubira/claude-code-settings

refactoring-assistant

Assists with code refactoring by detecting code smells, suggesting improvements, and providing refactoring patterns. Activates when writing/editing code, explicitly requested refactoring, or when code quality issues are detected. Maintains awareness of core principles while providing detailed patterns and examples.

1 0
Explore
fubira/claude-code-settings

context-compactor

Analyzes and compacts context-affecting documents (project memory, CLAUDE.md, skill files) to reduce token usage and compaction frequency. Manual trigger only. Always requires user approval before making changes.

1 0
Explore
fubira/claude-code-settings

test-executor

Executes tests, analyzes test results, checks test coverage, and provides comprehensive testing status overview. Primarily for Go projects. Activates after implementing/modifying code to verify correctness, or when explicitly requested to assess test suite health.

1 0
Explore
fubira/claude-code-settings

knowledge-manager

Manages a structured knowledge base of patterns, troubleshooting guides, best practices, and workflows. Activates when discovering reusable insights, solving technical problems, or establishing new standards. Records knowledge in categorized files for future reference without bloating global CLAUDE.md.

1 0
Explore
fubira/claude-code-settings

release-assistant

Automates and ensures reliable release workflows with automatic version bump based on commit history, mandatory lint/build/test execution before release, and safe tag creation and push.

1 0
Explore
fubira/claude-code-settings

git-commit-assistant

Assists with careful Git commits in any repository. Activates when committing changes, checking .gitignore, or generating commit messages. Ensures proper file exclusion (credentials, MCP configs, personal settings), identifies untracked files, and generates Conventional Commits messages with Japanese explanations.

1 0
Explore

Didn't find tool you were looking for?

Be as detailed as possible for better results