원클릭으로
reviewing-code
Use when reviewing code changes, checking PRs, or evaluating code quality.
Codex 또는 Claude로 설치 이 Prompt를 복사해 Codex, Claude 또는 다른 어시스턴트에 붙여 넣으면 Skill 페이지를 검토하고 설치를 진행할 수 있습니다.
메뉴
Use when reviewing code changes, checking PRs, or evaluating code quality.
Codex 또는 Claude로 설치 이 Prompt를 복사해 Codex, Claude 또는 다른 어시스턴트에 붙여 넣으면 Skill 페이지를 검토하고 설치를 진행할 수 있습니다.
SOC 직업 분류 기준
Use when receiving feature requests, architectural discussions, or multi-step implementation needs that require design before coding.
Manages task lifecycle transitions including starting, completing, and blocking tasks with enforcement gates and Trello synchronization.
Manages feature planning workflow with provider-agnostic PM operations. Detects active provider and adapts commands accordingly.
Manages feature planning workflow including budget validation, task creation, and Trello synchronization.
Use when promoting a new tranche, modifying the manifest, or otherwise about to push changes that affect the deploy-side build. Runs the same deterministic checks as the release-gate CI workflow.
Guides module architecture decisions including file size limits, function boundaries, and modular design patterns to maintain code quality and prevent complexity creep.
| name | reviewing-code |
| description | Use when reviewing code changes, checking PRs, or evaluating code quality. |
| skills | ["reviewing-code"] |
| agent-roles | ["nayru","laverna","vaivora"] |
| context | fork |
The review pipeline dispatches three specialized agents:
Use the review command to get the proper 3-agent pipeline:
bpsai-pair review pr <number>
bpsai-pair review branch
IMPORTANT: Do NOT dispatch generic agents for review. The review command handles agent dispatch with proper mythology names, severity-aware output, and size-scaled Vaivora dispatch. If you are already inside a review command invocation, do not duplicate the dispatch -- the command handles it.
# Run ALL checks at once (tests + linting)
bpsai-pair ci
# Validate project structure
bpsai-pair validate
# See what changed
git diff main...HEAD --stat
git diff main...HEAD
## Code Review: [Description]
### Summary
Brief assessment.
### P0 (blocks merge -- breaking state, auto-reject)
1. **[File:Line]** - Issue and fix
### P1 (fix before merge -- quality issue, not breaking)
1. **[File:Line]** - Suggestion
### P2 (fix before merge -- lower priority improvement)
1. **[File:Line]** - Optional improvement
### Positive Notes
- What was done well
### Verdict
- [ ] Approve
- [ ] Approve with comments
- [ ] Request changes
# Find debug statements
git diff main...HEAD | grep -E "print\(|breakpoint|pdb"
# Find TODOs in changes
git diff main...HEAD --name-only | xargs grep -n "TODO\|FIXME"
# Check for secrets
git diff main...HEAD | grep -iE "password|secret|api.?key|token"