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.
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:
- Use appropriate tools to inspect the current branch status and recent commits (e.g., git log --oneline -10, git diff origin/main).
- Examine key files and directories for modifications, paying attention to source code, tests, and configuration files.
- 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.
Recommended Agent Skills
Expand your agent's capabilities with these related and highly-rated skills.
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.
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.
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.
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.
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.
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.
Didn't find tool you were looking for?