self-review

name	self-review
description	구현 완료 후 thinking mode 심층 리뷰 — 즉시 반영 항목을 수정하고 결과를 보고한다
user-invocable	true

Self Review

절차(대상 수집, 수정 적용, 중단, 보고)는 .agents/skills/_shared/review-protocol.md를 읽고 그대로 따른다. 이 문서는 무엇을 볼지만 정한다.

현재 작업의 scope 문서가 있으면 함께 읽어 범위를 확인한다.

분석 렌즈 (extended thinking)

정합성

• 호출하는 함수, 의존하는 타입, 참조하는 상수가 실제로 존재하고 올바른지.
• 새 인터페이스/타입과 기존 구현체 간 계약이 맞는지.
• import 경로, export 누락, 순환 참조.

로직

• 분기의 완전성 (switch/if-else).
• 에러 경로에서의 리소스 정리와 상태 롤백.
• 경계 조건 (null, empty, 0, max).
• 비동기 코드의 await 누락, 에러 전파 누락.

설계 정합성

• docs/architecture.md의 계층 책임과 일치하는지.
• .agents/rules/의 규칙을 위반하지 않는지.
• scope 문서가 있으면 그 범위 내인지.
• 모듈 간 의존 방향이 설계 의도와 맞는지.
• 문서 간 충돌은 Architecture > Rules > Scope 우선순위로 해소한다.

테스트 충분성

• 변경된 로직의 주요 경로와 에러 경로에 대응하는 테스트가 있는지.
• 테스트가 구현 세부사항이 아니라 계약/동작을 검증하는지.

분류 경계

• 즉시 반영: 버그, 누락된 에러 처리, 계약 불일치, 명확한 설계 위반.
• 사용자 판단 필요: 설계 트레이드오프, 의도적일 수 있는 선택, scope 경계 항목.
• 무시: 이전 리뷰에서 이미 다뤄진 항목, 스타일 선호, 범위 밖 개선 제안.

추가 중단 기준

• 심각도 높은 버그(보안, 데이터 손실, 인증/권한 우회)가 즉시 반영으로 해결되지 않은 경우
• 동일 변경에서 버그가 2개 이상 발견되고 즉시 반영으로 해결되지 않은 경우
• 설계 원칙 위반이 발견된 경우