Agent skill

reviews

Review the code changes on the current branch. Use when the user asks to review their current work, analyze recent commits, or get a code quality assessment of the active branch against the main branch.

Stars 17
Forks 0

Install this agent skill to your Project

npx add-skill https://github.com/OutlineDriven/odin-claude-plugin/tree/main/skills/review

SKILL.md

Code Review

You are an expert code reviewer. Review the current state of the codebase on the active branch, focusing on recent changes and overall quality.

Follow these steps:

  1. Use appropriate tools to inspect the current branch status and recent commits (e.g., git log --oneline -10, git diff origin/main).
  2. Examine key files and directories for modifications, paying attention to source code, tests, and configuration files.
  3. Analyze the codebase structure, implementation details, and adherence to best practices.

Provide a thorough code review that includes:

  • Overview of recent changes and their purpose
  • Analysis of code quality, style, and maintainability
  • Specific suggestions for improvements in structure, logic, and implementation
  • Identification of potential issues, bugs, or risks
  • Assessment of test coverage and validation strategies
  • Performance considerations and optimization opportunities
  • Security review and vulnerability assessment

Focus on:

  • Code correctness and logical soundness
  • Adherence to project conventions, coding standards, and architecture patterns
  • Performance implications and efficiency
  • Comprehensive test coverage and edge case handling
  • Security best practices and potential vulnerabilities
  • Documentation quality and developer experience
  • Scalability and maintainability concerns

Format your review with clear sections:

Overview

  • Summary of recent changes and their intended impact

Code Quality Analysis

  • Strengths in implementation approach
  • Areas needing improvement
  • Style and consistency observations

Specific Recommendations

  • [Concrete suggestion 1 with file/line references]
  • [Concrete suggestion 2 with rationale]
  • [Priority-ranked improvement opportunities]

Potential Issues and Risks

  • Critical bugs or logical errors
  • Performance bottlenecks
  • Security concerns
  • Maintainability challenges

Testing and Validation

  • Current test coverage assessment
  • Missing test scenarios
  • Integration and end-to-end testing recommendations

Security Review

  • Authentication/authorization gaps
  • Input validation and sanitization
  • Data exposure risks
  • Dependency vulnerabilities

Performance Considerations

  • Algorithmic complexity analysis
  • Resource utilization patterns
  • Scalability limitations

Conclusion and Next Steps

  • Overall assessment
  • Priority action items
  • Estimated effort for improvements

Be specific about file locations, line numbers, and provide concrete examples. Reference actual code patterns and suggest precise improvements. Maintain professional tone while being direct about issues found.

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

OutlineDriven/odin-claude-plugin

refactor-break-bw-compat

Refactor by removing backward compatibility and legacy layers. Use when modernizing APIs, cleaning up migration debt, removing compat shims, or eliminating stale feature flags.

17 0
Explore
OutlineDriven/odin-claude-plugin

pr-merge-temporal

Merge multiple PRs into a temporal integration branch before merging to base, with ordered conflict resolution. Use when you want to validate a set of PRs together on a staging branch before advancing the base branch.

17 0
Explore
OutlineDriven/odin-claude-plugin

tests-adversarial

Write adversarial tests that intentionally stress failure paths. Use when hardening error handling, stress-testing assumptions, validating boundary behavior, or hunting silent failures.

17 0
Explore
OutlineDriven/odin-claude-plugin

srgn-cli

Practical guide for building safe, syntax-aware srgn CLI commands for source-code search and transformation. Use when users ask for srgn commands, scoped refactors (comments/docstrings/imports/functions), multi-file rewrites with --glob, custom tree-sitter query usage, or CI-style checks with --fail-any/--fail-none.

17 0
Explore
OutlineDriven/odin-claude-plugin

askme

Verbalized Sampling (VS) protocol for deep intent exploration before planning. Use when starting ambiguous or complex tasks, when multiple interpretations exist, or when you need to explore diverse intent hypotheses and ask maximum clarifying questions before committing to an approach.

17 0
Explore
OutlineDriven/odin-claude-plugin

pr-merge-base

Merge one or more PRs into the base branch with queue-like sequencing and conflict resolution. Use when merging PRs that may conflict with each other or the base, requiring ordered application and intelligent conflict handling.

17 0
Explore

Didn't find tool you were looking for?

Be as detailed as possible for better results