sniper-review-pr-
Team Composition
| Member | Process | Technical | Cognitive | Domain |
|---|---|---|---|---|
| code-reviewer | code-reviewer | -- | devils-advocate | -- |
| security-reviewer | code-reviewer | -- | security-first | -- |
| test-reviewer | qa-engineer | -- | systems-thinker | -- |
Tasks
code-reviewer
Code Quality Review
- ID:
code-review - Output:
docs/reviews/PR-{pr_number}-review.md (code quality section)
Review the PR diff for code quality: logic errors, naming clarity, pattern adherence, error handling, complexity, DRY violations, and architecture compliance. Reference docs/conventions.md for project patterns. Each finding must include severity, file:line, description, and suggested fix.
security-reviewer
Security Review
- ID:
security-review - Output:
docs/reviews/PR-{pr_number}-review.md (security section)
Review the PR diff for security issues: OWASP top 10, input validation, authentication checks, authorization checks, secrets handling, SQL injection, XSS, CSRF, insecure dependencies. Each finding must include severity, file:line, description, and suggested fix.
test-reviewer
Test Coverage Review
- ID:
test-review - Output:
docs/reviews/PR-{pr_number}-review.md (test coverage section)
Review the PR for test quality and coverage: missing tests for new code paths, uncovered edge cases, test naming, mock patterns, assertion quality. Each finding must include severity and suggested test additions.
Review Gate
- Mode: auto