# Phase 1 오디오 콘텐츠 첫 답글 작성 진입 리뷰

## 1. 리뷰 정보

| 항목 | 내용 |
|---|---|
| 리뷰 대상 | Phase 1 / `P1-T1`, `P1-GATE` |
| 기준 commit 또는 working tree | 2026-08-05 현재 uncommitted working tree |
| 리뷰 일자 | 2026-08-05 |
| 리뷰어 | Codex |
| 기준 문서 | [prd.md](../prd.md), [api-contract.md](../api-contract.md), [plan-task.md](../plan-task.md) |
| 리뷰 상태 | 수정 검증 완료 |

## 2. 리뷰 목적과 범위

### 목적

- `ACR-001~005`와 기존 Audio 댓글 API 계약이 실제 코드·test에서 충족되는지 확인한다.
- `P1-T1`, `P1-GATE` 완료 기록과 현재 working tree가 일치하는지 확인한다.

### 포함 범위

- 코드: `src/features/comments/components/{CommentItem,CommentThread}.tsx`, Comments API·model과 Audio/Community 소비 경로
- 테스트: `src/features/comments/tests`, `tests/e2e/comments.spec.ts`
- 문서: `ACR-001~005`, 기능 API Contract, `P1-T1`, `P1-GATE`, Progress
- 브라우저 검증: Chromium·mobile Chrome mock E2E와 keyboard-only 흐름

### 제외 범위

- 실제 개발 API와 ADMIN credential을 사용하는 server integration
- 댓글 수정·삭제·pagination의 기존 기능 자체 재설계
- exact 320px Audio 첫 답글 수동 시각 QA 재수행. 현재 리뷰에서는 mobile Chrome 첫 답글과 320px Community overflow를 자동 검증했다.

## 3. 판정 기준

### 심각도

| 심각도 | 기준 |
|---|---|
| Blocker | 보안·데이터 손실 위험 또는 핵심 첫 답글 흐름 불능 |
| High | `ACR-001~005` 또는 API Contract 위반 |
| Medium | 제한된 target·viewport·권한·재시도 조건의 기능 회귀 |
| Low | 문서 추적, 유지보수성 또는 비핵심 UX 불일치 |

### 상태

| 상태 | 의미 | 후속 처리 |
|---|---|---|
| 후보 | 근거 발견 후 판정 전 | 재현·대조 |
| 확정 | 코드·test·문서 근거로 문제 확인 | 후속 Task 후보 |
| 오탐 | 요구사항과 실행 결과상 문제 아님 | 근거 보존 후 종료 |
| 보류 | 외부 조건이 필요함 | 담당·재개 조건 기록 |
| 수정 완료 | 수정과 회귀 검증 완료 | 검증 기록 누적 |

## 4. 검토한 근거

### 문서와 코드

- 요구사항: `ACR-001~005`, PRD §10·§14
- API Contract: 직접 답글 GET, Audio 댓글·답글 POST, `NullSuccess`
- 계획: `P1-T1`, `P1-GATE`, 2026-08-05 구현·검증 Progress
- 코드: `CommentItem.tsx:6,28`, `CommentThread.tsx:113-125,127-147,167-175`
- 테스트: `comment-thread.test.tsx:138-176`, `comments.spec.ts:33-145`

### 실행 환경

```text
OS: Darwin 25.0.0 x86_64
Node: v24.12.0
npm: 11.7.0
Browser/viewport: Desktop Chrome, Pixel 5 mobile Chrome, spec 내부 320x640·1280x900
API mode: mock
```

### 실행한 검증

| 명령 또는 수동 검증 | 결과 | 핵심 증거 |
|---|---|---|
| `npm run test:run -- src/features/comments/tests/comment-thread.test.tsx` | 성공 | exit 0, 1 file / 8 tests passed |
| `npm run test:run -- src/features/comments` | 성공 | exit 0, 3 files / 15 tests passed |
| `npm run typecheck` | 성공 | exit 0, TypeScript 오류 0건 |
| `npm run lint` | 성공 | exit 0, ESLint 오류 0건 |
| `git diff --check` | 성공 | exit 0 |
| sandbox의 `npm run e2e:mock -- tests/e2e/comments.spec.ts` | 실행 불가 | 로컬 `127.0.0.1:8889` listen `EPERM` |
| 승인된 동일 E2E 재실행 | 성공 | exit 0, Chromium 3 + mobile Chrome 3 = 6 tests passed |

## 5. 발견 사항 요약

| ID | 심각도 | 상태 | 제목 | 소유 Task | 후속 goal |
|---|---|---|---|---|---|
| `REV-P1-001` | Low | 수정 완료 | 완료 상태와 현재 상태·수용 체크박스가 서로 모순된다 | `P1-T1`, `P1-GATE` | `P1-R1` |

기능·API·권한·반응형 범위에서는 확정 발견 사항이 없다.

## 6. 발견 사항 상세

### REV-P1-001 — 완료 상태와 현재 상태·수용 체크박스가 서로 모순된다

- **심각도:** Low
- **상태:** 수정 완료
- **관련 요구사항:** `ACR-001~005`, PRD §14
- **관련 계약:** 없음
- **소유 Task:** `P1-T1`, `P1-GATE`; 후속 문서 Task `P1-R1`

**관찰 내용**

`plan-task.md`는 상태, Phase 표, Task·Gate 체크박스와 Progress에서 구현·검증
완료라고 기록한다. 그러나 같은 문서의 현재 상태는 답글 진입 버튼이 없고
코드·test가 변경되지 않았다고 적으며, 발견된 문제도 기능 부재를 현재형으로
유지한다. `prd.md` §14의 수용·추적 체크박스도 모두 미완료다.

**근거**

- 완료 기록: `plan-task.md:5,21,84-100,159-163`
- 미구현 기록: `plan-task.md:24-25,175`
- 미완료 수용 기준: `prd.md:173-185`
- 실제 구현: `CommentThread.tsx:170`, `CommentItem.tsx:28`
- 검증: focused 8/8, Comments 15/15, mock E2E 6/6, typecheck·lint 통과

**재현 또는 검증 절차**

1. `plan-task.md`의 문서 상태와 Phase 표를 확인한다.
2. 같은 문서의 현재 상태·발견된 문제와 `prd.md` §14를 확인한다.
3. 완료와 미구현·미완료 표기가 동시에 존재함을 확인한다.
4. 실제 code·test와 재실행 결과는 완료 쪽 기록과 일치한다.

**영향**

런타임 기능에는 영향이 없지만, 다음 작업자와 문서 검증 자동화가 구현 완료
여부를 일관되게 판정할 수 없다.

**권장 조치**

코드 변경 없이 `plan-task.md`의 현재 상태와 발견된 문제를 실제 완료 상태로
정정하고, `prd.md` §14 체크박스를 이번 리뷰 증거에 따라 완료 처리한다. 기존
설계·구현 Progress는 삭제하지 않고 문서 정정 기록을 추가한다.

**판정 기록**

- 2026-08-05 — 문서 내부 대조와 현재 검증 결과로 Low 문서 정합성 문제를 확정했다.
- 2026-08-05 정정 — 확정 finding을 초안으로만 남긴 처리는 review 가이드 §4·§5와 맞지 않아 [plan-task.md](../plan-task.md)의 `P1-R1` 후속 Task로 전환했다.
- 2026-08-05 수정 완료 — `P1-R1`에서 stale 현재 상태·발견된 문제와 PRD §14 체크박스를 실제 구현·검증 상태에 맞게 정정했다.

## 7. 확정 항목의 plan·goal 전환

최초 리뷰에서는 진단 범위라는 이유로 아래 Task를 초안으로만 남겼다.

**정정 — 2026-08-05:** 확정 finding은 코드 수정 여부와 별개로
`plan-task.md`의 후속 Task로 전환해야 하므로 [P1-R1](../plan-task.md)을
추가했다. 아래 초안은 실제 Task의 입력으로 보존한다.

### 신규 회귀 수정 Task 초안

```markdown
### Task R1.1 완료 문서 상태 정합성 복구

**Goal 실행 `P1-R1`:** `REV-P1-001`의 완료·미구현 상태 모순을 제거하고 실제 검증 증거와 PRD·plan을 일치시킨다.

- **시작 조건:** `REV-P1-001`, 완료된 `P1-T1`, `P1-GATE`.
- **완료 증거:** 현재 상태·발견된 문제·PRD §14 정정, 기존 Progress 보존, 문서 링크·diff 검증.
- **범위 밖:** 애플리케이션 코드·test·API Contract 변경.
- **TDD 예외 사유:** 구현 동작이 아닌 완료 문서 정합성 수정이다.
- **대체 검증:** 완료/미구현 marker 대조, Markdown link 확인, `git diff --check`.

- [x] 현재 상태와 발견된 문제를 실제 완료 상태로 정정한다.
- [x] PRD §14 수용·추적 체크박스를 검증 증거에 맞게 갱신한다.
- [x] 기존 Progress를 보존하고 정정 기록을 누적한다.
- [x] Markdown link와 `git diff --check`를 실행해 결과를 기록한다.
```

### create_goal objective 초안

```text
[P1-R1]의 확정 review 항목 REV-P1-001을 문서에서 수정한다.
애플리케이션 코드·test·API Contract는 변경하지 않는다.
현재 상태, PRD 수용 기준, 정정 기록과 문서 검증이 모두 끝나기 전에는 complete로 표시하지 않는다.
```

## 8. 리뷰 종료 판정

| 판정 항목 | 결과 | 근거 |
|---|---|---|
| 리뷰 범위 전체 확인 | 충족 | PRD·계약·plan·code·unit·E2E 대조 완료 |
| 후보 항목 판정 완료 | 충족 | `REV-P1-001` Low 수정 완료 |
| 확정 항목 plan 반영 | 충족 | [plan-task.md](../plan-task.md)에 `P1-R1` 추가 |
| 보류 항목의 담당·재개 조건 기록 | 해당 없음 | 외부 의존 finding 없음 |
| 검증 명령과 결과 기록 | 충족 | unit 15/15, E2E 6/6, typecheck·lint와 `P1-R1` 문서 검증 통과 |

**최종 결론:** 기능 구현 승인, `REV-P1-001` 수정 검증 완료.

**남은 항목:** 없음.

## 9. 수정 후 검증 기록

수정 검증 결과는 아래에 기존 기록을 보존하며 누적한다.

### 1차 수정 검증 — 2026-08-05

- 무엇을: `REV-P1-001`의 plan stale 상태·PRD §14 미완료 체크박스와 review 상태를 실제 구현·검증 완료 상태에 맞췄다.
- 왜: 런타임 기능은 승인됐지만 완료·미구현 표기가 공존해 문서 추적과 자동 판정이 모순됐다.
- 어떻게:
  - `! rg -n '^- 현재 .*답글 진입 버튼이 없다|^- 애플리케이션 코드와 test는 아직 변경하지 않았다|^- 확정: .*첫 답글 작성 진입이 없다' docs/20260805_오디오콘텐츠댓글답글/plan-task.md` — 성공, exit 0, stale marker 0건
  - `! sed -n '/## 14\./,/## 15\./p' docs/20260805_오디오콘텐츠댓글답글/prd.md | rg -n '^- \[ \]'` — 성공, exit 0, 미완료 체크박스 0건
  - `test -f docs/20260805_오디오콘텐츠댓글답글/reviews/phase1-audio-comment-first-reply.md` — 성공, exit 0
  - `npm run test:run -- src/features/comments` — 성공, exit 0, 3 files / 15 tests passed
  - `npm run e2e:mock -- tests/e2e/comments.spec.ts --project=chromium` — 성공, exit 0, Chromium 3 tests passed
  - `npm run typecheck` — 성공, exit 0, TypeScript 오류 0건
  - `npm run lint` — 성공, exit 0, ESLint 오류 0건
  - `git diff --check` — 성공, exit 0
- 남은 항목: 없음.

### 2차 수정 재검증 — 2026-08-05

- 무엇을: 사용자가 반영한 `P1-R1`의 PRD·plan·review 정합성과 Comments 기능 회귀를 현재 working tree에서 다시 판정했다.
- 왜: 완료 체크박스와 과거 검증 기록만 신뢰하지 않고 실제 반영 여부와 신규 문제를 독립적으로 확인하기 위해서다.
- 어떻게:
  - `! rg -n '^- 현재 .*답글 진입 버튼이 없다|^- 애플리케이션 코드와 test는 아직 변경하지 않았다|^- 확정: .*첫 답글 작성 진입이 없다' docs/20260805_오디오콘텐츠댓글답글/plan-task.md` — 성공, exit 0, stale marker 0건
  - `! sed -n '/## 14\./,/## 15\./p' docs/20260805_오디오콘텐츠댓글답글/prd.md | rg -n '^- \[ \]'` — 성공, exit 0, 미완료 체크박스 0건
  - `test -f docs/20260805_오디오콘텐츠댓글답글/reviews/phase1-audio-comment-first-reply.md` — 성공, exit 0
  - `npm run test:run -- src/features/comments` — 성공, exit 0, 3 files / 15 tests passed
  - `npm run e2e:mock -- tests/e2e/comments.spec.ts` — 성공, exit 0, Chromium 3 + mobile Chrome 3 = 6 tests passed
  - `npm run typecheck` — 성공, exit 0, TypeScript 오류 0건
  - `npm run lint` — 성공, exit 0, ESLint 오류 0건
  - `git diff --check` — 성공, exit 0
- 판정: `REV-P1-001` 수정 완료를 재확인했고 검토 범위의 신규 확정 발견 사항은 없다.
- 남은 항목: 실제 개발 API와 ADMIN credential을 사용하는 server integration은 기존 제외 범위로 유지한다.