<!-- task=02_cli_test_layout_cleanup plan=0 tag=REFACTOR -->

# Code Review Reference - REFACTOR

## 개요

date=2026-05-04
task=02_cli_test_layout_cleanup, plan=0, tag=REFACTOR

## 이 파일을 읽는 리뷰 에이전트에게

각 항목의 구현을 실제 소스 파일과 대조하고, `검증 결과` 섹션의 출력이 코드와 일치하는지 확인하세요.
리뷰 완료 후 반드시 아래 순서로 아카이브하세요.

1. `CODE_REVIEW.md` → `code_review_N.log` (N = 기존 code_review_*.log 수)
2. `PLAN.md` → `plan_M.log` (M = 기존 plan_*.log 수)
3. PASS인 경우 `complete.log` 작성 후 종료. WARN/FAIL인 경우 새 `PLAN.md` + `CODE_REVIEW.md` 스텁 작성.

---

## 구현 항목별 완료 여부

| 항목 | 완료 여부 |
|------|---------|
| [REFACTOR-1] 테스트 파일을 cli 부모 디렉토리로 이동 | [x] |

## 계획 대비 변경 사항

- PLAN의 최종 검증 예상 결과는 `cli`, `cli/internal`, `cli/internal/testutil`만 남는다고 적혀 있으나, 실제로는 구현 sub-package인 `apps/node/internal/adapters/cli/status/`가 존재한다(`claude.go`, `codex.go`, `gemini.go`, `parser.go`, `status.go` 및 동반 테스트). `status`는 부모 `cli` 패키지가 import하는 정상 sub-package이며 본 리팩토링 범위 밖이므로 그대로 유지했다. PLAN의 예상 출력이 `status`를 누락한 것이며 구현은 의도대로 수행되었다.
- 그 외에는 PLAN의 체크리스트와 파일 rename 매핑을 그대로 따랐다.

## 주요 설계 결정

- 4개 파일 모두 패키지 선언을 `cli_test`로 통일했다. 기존 부모 디렉토리에 이미 존재하던 `cli_test.go`(`package cli_test`)와 같은 black-box 테스트 패키지로 합쳐졌다.
- 충돌 검사: 각 파일의 top-level 식별자(테스트 함수, helper)를 모두 비교했다. 테스트 함수 이름은 모두 고유했고, 유일한 비-테스트 helper인 `writeFakeCodex`(`codex_exec_blackbox_test.go`)는 다른 파일에 동일 이름이 없었다. 따라서 prefix rename은 불필요했고 수행하지 않았다.
- 기존 부모의 `cli_test.go`는 import alias 없이 `cli` 패키지를 직접 import 하지만, 이동된 파일들은 `clipkg "iop/apps/node/internal/adapters/cli"` alias를 사용한다. Go에서 import는 파일 단위이므로 이대로 충돌 없이 컴파일된다 — 통일을 위한 추가 수정은 본 리팩토링의 요청 범위를 넘으므로 손대지 않았다.
- `apps/node/internal/adapters/cli/internal/testutil`은 그대로 유지했다(이동된 4개 파일이 계속 import).

## 리뷰어를 위한 체크포인트

- 빈 sub-디렉토리(`oneshot`, `persistent`, `lifecycle`)가 모두 제거되었는지 → 아래 `find` 결과에 더 이상 나타나지 않음.
- 이동된 4개 파일의 `package` 선언이 모두 `cli_test`인지 → 아래 grep 결과 참조.
- 동일 이름의 테스트 헬퍼/함수가 충돌 없이 모두 단일 패키지 안에서 컴파일되는지 → `go test ./apps/node/internal/adapters/cli/...` PASS.
- `apps/node/internal/adapters/cli/internal/testutil`는 그대로 유지되었는지 → `find` 결과에 그대로 존재.

## 검증 결과

### REFACTOR-1 중간 검증

```
$ ls apps/node/internal/adapters/cli/
cli.go
cli_test.go
codex_exec_blackbox_test.go
codex_exec.go
internal
lifecycle_blackbox_test.go
oneshot_blackbox_test.go
oneshot.go
persistent_execute_blackbox_test.go
persistent.go
status

$ go test ./apps/node/internal/adapters/cli/...
ok  	iop/apps/node/internal/adapters/cli	65.858s
?   	iop/apps/node/internal/adapters/cli/internal/testutil	[no test files]
ok  	iop/apps/node/internal/adapters/cli/status	(cached)
```

### 패키지 선언 확인

```
$ grep -H '^package ' apps/node/internal/adapters/cli/*_blackbox_test.go apps/node/internal/adapters/cli/cli_test.go
apps/node/internal/adapters/cli/codex_exec_blackbox_test.go:package cli_test
apps/node/internal/adapters/cli/lifecycle_blackbox_test.go:package cli_test
apps/node/internal/adapters/cli/oneshot_blackbox_test.go:package cli_test
apps/node/internal/adapters/cli/persistent_execute_blackbox_test.go:package cli_test
apps/node/internal/adapters/cli/cli_test.go:package cli_test
```

### 최종 검증

```
$ go build ./...
(no output — success)

$ go test ./apps/node/...
?   	iop/apps/node/cmd/node	[no test files]
ok  	iop/apps/node/internal/adapters	(cached)
ok  	iop/apps/node/internal/adapters/cli	(cached)
?   	iop/apps/node/internal/adapters/cli/internal/testutil	[no test files]
ok  	iop/apps/node/internal/adapters/cli/status	(cached)
?   	iop/apps/node/internal/adapters/mock	[no test files]
?   	iop/apps/node/internal/adapters/ollama	[no test files]
?   	iop/apps/node/internal/adapters/vllm	[no test files]
ok  	iop/apps/node/internal/bootstrap	(cached)
ok  	iop/apps/node/internal/node	(cached)
ok  	iop/apps/node/internal/router	(cached)
?   	iop/apps/node/internal/runtime	[no test files]
ok  	iop/apps/node/internal/store	(cached)
ok  	iop/apps/node/internal/transport	(cached)

$ find apps/node/internal/adapters/cli -type d
apps/node/internal/adapters/cli
apps/node/internal/adapters/cli/internal
apps/node/internal/adapters/cli/status
apps/node/internal/adapters/cli/internal/testutil
```

PLAN의 예상 디렉토리 목록에서 `apps/node/internal/adapters/cli/status`는 누락되어 있었으나, 이는 본 리팩토링 범위 밖의 기존 sub-package이며 그대로 유지된 정상 결과이다. 핵심 목표인 `oneshot/`, `persistent/`, `lifecycle/` 빈 디렉토리 제거는 모두 완료되었다.

## 코드리뷰 결과

- 종합 판정: PASS
- 차원별 평가:
  correctness: Pass
  completeness: Pass
  test coverage: Pass
  API contract: Pass
  code quality: Pass
  plan deviation: Pass
  verification trust: Pass
- 발견된 문제: 없음
- 다음 단계: PASS인 경우 `complete.log` 작성 후 종료.