Code Review Excellence

Code Review Excellence

Transform code reviews from gatekeeping to knowledge sharing through constructive feedback, systematic analysis, and collaborative improvement.

Use this skill when

• Reviewing pull requests and code changes
• Establishing code review standards
• Mentoring developers through review feedback
• Auditing for correctness, security, or performance

Do not use this skill when

• There are no code changes to review
• The task is a design-only discussion without code
• You need to implement fixes instead of reviewing

Instructions

• Read context, requirements, and test signals first.
• Review for correctness, security, performance, and maintainability.
• Provide actionable feedback with severity and rationale.
• Ask clarifying questions when intent is unclear.
• If detailed checklists are required, open resources/implementation-playbook.md.

Output Format

• High-level summary of findings
• Issues grouped by severity (blocking, important, minor)
• Suggestions and questions
• Test and coverage notes

Resources

• resources/implementation-playbook.md for detailed review patterns and templates.

AGI Framework Integration

Adapted for @techwavedev/agi-agent-kit Original source: antigravity-awesome-skills

Memory-First Protocol

Retrieve prior test strategies, known flaky tests, and coverage gaps. Cache test infrastructure setup to avoid re-configuring test environments.

# Check for prior testing/QA context before starting
python3 execution/memory_manager.py auto --query "test patterns and coverage strategies for Code Review Excellence"

Storing Results

After completing work, store testing/QA decisions for future sessions:

python3 execution/memory_manager.py store \
  --content "Testing strategy: integration tests hit real DB (no mocks), 85% line coverage, mutation testing on critical paths" \
  --type technical --project <project> \
  --tags code-review-excellence testing

Multi-Agent Collaboration

Share test results and coverage reports with code review agents so they can verify adequate coverage on changed code.

python3 execution/cross_agent_context.py store \
  --agent "<your-agent>" \
  --action "QA complete — test suite expanded with 12 new integration tests, all passing" \
  --project <project>

TDD Enforcement

This skill integrates with the framework's iron-law RED-GREEN-REFACTOR cycle. No production code without a failing test first.

Agent Team: QA

Dispatch qa_team to generate tests and verify they pass before marking implementation complete.