Agent skill
review
Use after implementation is complete and before creating a PR — agent-driven code review checking spec compliance, code quality, security, and regression risk
Install this agent skill to your Project
npx add-skill https://github.com/majiayu000/claude-skill-registry/tree/main/skills/other/other/review-dodi-hq-dodi-skills
SKILL.md
Review
Comprehensive agent-driven code review. Run after implementation, before PR.
What to Check
| Category | What to Look For |
|---|---|
| Spec compliance | Does the code implement what was specified? Nothing missing, nothing extra |
| Code quality | Clean, idiomatic, follows project conventions (CLAUDE.md, style guides) |
| Security | Injection, auth bypass, data leaks, OWASP top 10 |
| Regression risk | Does this change break assumptions in callers/consumers? |
| Error handling | Silent failures, swallowed exceptions, missing edge cases |
| API contracts | If touching APIs — are request/response shapes backwards-compatible? |
Process
- Identify the spec/plan and the diff (
git diff <base>...HEAD) - Dispatch review subagent (see review-prompt.md) with:
- The spec/plan path
- The diff or list of changed files
- Project conventions (CLAUDE.md path)
- If issues found: fix them, re-run review
- When approved: proceed to
dodi-dev:submit
Don't Skip This
"Tests pass" is not a review. Tests verify behavior; review verifies intent, quality, and risk.
Recommended Agent Skills
Expand your agent's capabilities with these related and highly-rated skills.
majiayu000/claude-skill-registry
agent-ops-spec
Manage specification documents in .agent/specs/. Use when user provides requirements, acceptance criteria, or feature descriptions that need to be tracked and validated against implementation.
majiayu000/claude-skill-registry
agent-ops-state
Maintain .agent state files. Use at session start, after meaningful steps, and before concluding: read/update constitution/memory/focus/issues/baseline consistently.
majiayu000/claude-skill-registry
agent-ops-spec
Manage specification documents in .agent/specs/. Use when user provides requirements, acceptance criteria, or feature descriptions that need to be tracked and validated against implementation.
majiayu000/claude-skill-registry
agent-ops-testing
Test strategy, execution, and coverage analysis. Use when designing tests, running test suites, or analyzing test results beyond baseline checks.
majiayu000/claude-skill-registry
agent-ops-testing
Test strategy, execution, and coverage analysis. Use when designing tests, running test suites, or analyzing test results beyond baseline checks.
majiayu000/claude-skill-registry
agent-ops-state
Maintain .agent state files. Use at session start, after meaningful steps, and before concluding: read/update constitution/memory/focus/issues/baseline consistently.
Didn't find tool you were looking for?