| name | review |
| description | Pre-landing code review. Finds bugs that pass CI but break in prod. Runs after implementation and before ship. Maps to Gate โก (mutual review) for code artifacts. |
Review
๊ตฌํ์ด ๋๋ diff ๋ฅผ ์ฐฉ๋ฅ ์ ์ต์ข
๊ฒํ ํ๋ skill. Gate โก ์ ์ค์
์ด์์.
์ธ์ ์ฐ๋๊ฐ
runtime.json โ gates.implementation_verified == true
- PR ์ด ์ค๋น๋๊ณ
ship ์ ๋ง์ง๋ง ์ ๊ฒ
- tier ๊ฐ normal ์ด์
์ธ์ ์ฐ์ง ์๋๊ฐ
- trivial tier ๋ณ๊ฒฝ (์ง์ merge)
- ์์ง ๊ตฌํ ์ค (
Implement.md ์ validation loop ๊ฐ ๋จผ์ )
๋ฐฉ์
.claude/sealed-prompts/review-code.md ๋ฅผ ๋ฐ๋ฅธ๋ค. ์๋๋ ์ด์ ๊ฐ์ด๋.
1. ๋๊ฐ review ํ๋๊ฐ
- ์์ฑ์ actor ๊ฐ
claude ๋ฉด reviewer ๋ user ๋๋ isolated claude-reviewer ์ธ์
- ์ด ํ๋ ์์ํฌ๋ Codex ๊ฒฝ๋ก๋ฅผ ์ฌ์ฉํ์ง ์๋๋ค (CLAUDE.md / AGENTS.md ์ฐธ์กฐ)
- Gate โก P2-F ๋ ๊ฐ์ actor ๊ฐ self-review ๋ฅผ ๊ธฐ๋กํ๋ ๊ฒ์ ๊ธฐ๊ณ ์ฐจ๋จ
2. ๋ฌด์์ ์ฝ๋๊ฐ
- diff ์ ์ฒด
- ์ํฅ๋ฐ์ ํ์ผ ์ฃผ๋ณ 20์ค
- ์ด ๋ณ๊ฒฝ์ด import/call ํ๋ ํ์ผ
- ์ต๊ทผ
.claude/test-runs/run-*.log
.claude/learnings.jsonl ๋ง์ง๋ง 20์ค
3. Rollback triggers ์ฒดํฌ (ํ์)
review-code.md ์ 7๊ฐ์ง ํญ๋ชฉ์ explicit ํ๊ฒ ํ์ธ:
- Public API signature ๋ณ๊ฒฝ โ migration path?
- Test ์ ๊ฐ์ / skip / xfail?
- Performance ๊ฒฝ๋ก regression?
- Security ๊ฒฝ๋ก ๋ณ๊ฒฝ + ๋์ test?
- ์ TODO/FIXME?
- Commented-out code?
- One-way schema migration?
4. Test oracle ๋ฌด๊ฒฐ์ฑ
๊ตฌํ + ํ
์คํธ๊ฐ ๊ฐ์ ์ปค๋ฐ ์์ ๋ฐ๋๋ฉด red flag. reviewer ๋:
- Fail-before ์ฆ๊ฑฐ ์๊ตฌ (๊ธฐ์กด test ๊ฐ ์ด ๋ฒ๊ทธ ์ก์๋์ง)
- Pass-after ์ฆ๊ฑฐ ์๊ตฌ
- ์์ผ๋ฉด ๊ธฐ๋ณธ verdict =
revise
5. Verdict ๊ธฐ๋ก
python3 scripts/harness/event_log.py 02 pass <reviewer-actor> <reviewed-file> <<JSON
{
"reviewer_file": ".claude/reviews/<ts>.md",
"summary": "<verdict + top finding>",
"reviewed_file": "<path>",
"verdict": "accept|revise|reject"
}
JSON
์ถ๋ ฅ ํ์
review-code.md sealed prompt ํ์ ๊ทธ๋๋ก .claude/reviews/<ts>.md ์
์์ฑ. ์ฃผ์ ์น์
:
- Verdict (accept / revise / reject [Rollback recommended: ...])
- Severity of worst finding
- Objections (ranked, worst first) โ ์ต์ 2๊ฐ (normal) / 3๊ฐ (high-risk)
- Tests actually exercising this change
- Assumptions made to review
- Rollback triggers (fired or none)
๊ฐ๋๋ ์ผ
- "LGTM" ๊ธ์ง
- Objection ์ ์๊ฑด ๋ฏธ๋ง์ด๋ฉด
<INSUFFICIENT_OBJECTIONS> ๋ก stop
- Rollback triggers ์ฒดํฌ ์น์
๋๋ฝ ์ Gate โก Pre ๊ฐ ๋ค์ ํธ์ง์ ์ฐจ๋จ
- Reviewer_file ์ด 800B ๋ฏธ๋ง์ด๊ฑฐ๋ ๊ตฌ์กฐํ๋ Objection block ์ด ์์ผ๋ฉด ์๋ ๊ฑฐ๋ถ (Gate โก ๋ด๋ถ ๊ฒ์ฆ)
๋จผ์ ์ฝ์ ๊ฒ
.claude/sealed-prompts/review-code.md
- ๋์ diff (
git diff)
Plan.md (์๋ scope ์ ์ผ์นํ๋์ง)