fix: clean up CLI test layout and add blackbox test logs

agent-task/02_cli_test_layout_cleanup/CODE_REVIEW.md

@@ -1,59 +0,0 @@
-<!-- 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 부모 디렉토리로 이동 | [ ] |
-
-## 계획 대비 변경 사항
-
-_구현 에이전트가 계획과 다르게 구현한 부분을 이유와 함께 기록한다._
-
-## 주요 설계 결정
-
-_구현 에이전트가 주요 설계 결정 사항을 기록한다._
-
-## 리뷰어를 위한 체크포인트
-
-- 빈 sub-디렉토리(`oneshot`, `persistent`, `lifecycle`)가 모두 제거되었는지
-- 이동된 4개 파일의 `package` 선언이 모두 `cli_test`인지
-- 동일 이름의 테스트 헬퍼/함수가 충돌 없이 모두 단일 패키지 안에서 컴파일되는지
-- `apps/node/internal/adapters/cli/internal/testutil`는 그대로 유지되었는지
-
-## 검증 결과
-
-_구현 에이전트가 각 중간 검증 및 최종 검증 명령 실행 후 출력을 여기에 붙여 넣는다._
-
-### REFACTOR-1 중간 검증
-```
-$ ls apps/node/internal/adapters/cli/
-$ go test ./apps/node/internal/adapters/cli/...
-(output)
-```
-
-### 최종 검증
-```
-$ go build ./...
-$ go test ./apps/node/...
-$ find apps/node/internal/adapters/cli -type d
-(output)
-```

agent-task/02_cli_test_layout_cleanup/code_review_0.log

@@ -0,0 +1,124 @@
+<!-- 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` 작성 후 종료.

agent-task/02_cli_test_layout_cleanup/complete.log

@@ -0,0 +1,13 @@
+완료 일시: 2026-05-04
+요약: CLI 어댑터 테스트 파일을 부모 디렉토리로 이동하고 black-box `cli_test` 패키지로 통합한 1회 루프 작업 완료.
+
+루프 이력:
+| plan log | code review log | verdict |
+|----------|-----------------|---------|
+| `plan_0.log` | `code_review_0.log` | PASS |
+
+최종 리뷰 요약:
+- `apps/node/internal/adapters/cli/oneshot/`, `persistent/`, `lifecycle/`의 테스트 파일 4개가 부모 `cli` 디렉토리로 이동되었다.
+- 이동된 테스트 파일의 패키지 선언이 모두 `cli_test`로 통일되어 기존 `cli_test.go`와 같은 black-box 테스트 패키지로 정리되었다.
+- 빈 `oneshot/`, `persistent/`, `lifecycle/` 디렉토리가 제거되었고 `internal/testutil` 및 기존 `status` sub-package는 유지되었다.
+- `go test ./apps/node/internal/adapters/cli/...`, `go build ./...`, `go test ./apps/node/...` 검증이 모두 통과했다.

apps/node/internal/adapters/cli/codex_exec_blackbox_test.go

@@ -1,4 +1,4 @@
-package persistent_test
+package cli_test
 
 import (
 	"context"

apps/node/internal/adapters/cli/lifecycle_blackbox_test.go

@@ -1,4 +1,4 @@
-package lifecycle_test
+package cli_test
 
 import (
 	"context"

apps/node/internal/adapters/cli/oneshot_blackbox_test.go

@@ -1,4 +1,4 @@
-package oneshot_test
+package cli_test
 
 import (
 	"context"

apps/node/internal/adapters/cli/persistent_execute_blackbox_test.go

@@ -1,4 +1,4 @@
-package persistent_test
+package cli_test
 
 import (
 	"context"