--- name: review description: Use when reviewing repository changes against AGENTS.md, architecture decisions, tests, and build requirements. --- # Repository Review 먼저 다음 문서를 읽는다. - `/AGENTS.md` - `/docs/ARCHITECTURE.md` - `/docs/ADR.md` 사용자가 범위를 지정하지 않으면 현재 작업 트리의 변경을 리뷰한다. 관련 diff를 확인하고 가능한 빌드와 테스트 command를 실제로 실행한다. ## 체크리스트 1. MSVC toolset와 C++ 표준이 AGENTS.md/ADR과 일치하는가? 2. CMake 또는 MSBuild Debug/x64 빌드가 통과하는가? 3. CTest 또는 `.harness/config.json`의 명시적 test command가 통과하는가? 4. 새 C/C++ 소스와 헤더에 대응 테스트가 있는가? 5. CRITICAL 아키텍처 규칙과 public header 경계를 지키는가? ## 출력 형식 실제 결함을 심각도순으로 먼저 제시한다. 각 finding에 파일과 줄 번호, 영향, 재현 또는 근거, 구체적인 수정 방안을 포함한다. 그 뒤 다음 표를 제공한다. | 항목 | 결과 | 비고 | |------|------|------| | 아키텍처 준수 | ✅/❌/미검증 | 상세 | | 기술 스택 준수 | ✅/❌/미검증 | 상세 | | 테스트 존재 | ✅/❌/미검증 | 상세 | | CRITICAL 규칙 | ✅/❌/미검증 | 상세 | | 빌드 가능 | ✅/❌/미검증 | 상세 | 실행할 수 없는 검사는 성공으로 추정하지 말고 `미검증`과 이유를 적는다. finding이 없으면 발견된 문제가 없다고 명시하고 남아 있는 검증 공백을 설명한다.