<!-- task=m-agent-readable-repository-refactor/05_core_node_tests plan=0 tag=TEST -->

# Core Node 테스트 책임별 분해

## 이 파일을 읽는 구현 에이전트에게

> 구현 후 계획의 검증 명령을 그대로 실행하고 CODE_REVIEW의 구현 에이전트 소유 섹션에 실제 메모와 stdout/stderr를 채운 뒤 active 파일을 유지한 채 리뷰 준비 완료를 보고한다. 종결 판정, log rename, complete.log와 archive 이동은 code-review 전용이다. 선택된 Milestone의 미해결 구현 잠금 결정이 실제 구현을 차단할 때만 review stub의 사용자 리뷰 요청에 연결 근거를 기록하고 중단한다. 직접 사용자 질문, chat 선택지, request_user_input, USER_REVIEW.md/complete.log 작성은 금지한다. 환경·secret·service, 일반 범위 조정, 후속으로 보완 가능한 증거 공백은 사용자 리뷰 사유가 아니다.

## 배경

apps/node/internal/node/node_test.go는 3,346줄에서 run/cancel, command, refresh, tunnel을 함께 검증한다. Node package 경계를 유지하면서 실행 책임별 파일과 최소 fixture로 나눈다.

## 사용자 리뷰 요청 흐름

사용자 리뷰 요청은 선택된 Milestone의 구현 잠금 결정만 active review stub의 사용자 리뷰 요청 섹션에 agent-ops/skills/common/_templates/implementation-user-review-request-section.md 형식으로 기록한다. 구현 중 직접 사용자 prompt는 금지하며, 타당성 검증과 USER_REVIEW.md 작성은 code-review가 소유한다.

## 분석 결과

### 읽은 파일

- agent-roadmap/phase/automation-runtime-bridge/milestones/agent-readable-repository-refactor.md
- agent-test/local/rules.md
- agent-test/local/profiles/node-smoke.md
- agent-spec/runtime/edge-node-execution.md
- agent-spec/runtime/provider-pool-config-refresh.md
- apps/node/internal/node/node_test.go

### SDD 기준

선택 Milestone은 SDD: 불필요다. 외부 API/proto/config schema와 runtime 책임 경계를 바꾸지 않는 동작 보존형 테스트 재배치라는 기록 사유를 적용한다.

### 테스트 환경 규칙

test_env=local. local rules와 node-smoke profile을 읽었다. 현재 checkout에서 GOCACHE=/tmp/iop-go-build-cache go test -count=1 ./apps/node/internal/node가 통과했다. cache를 쓰지 않는 package test와 Test 목록 66개를 계약으로 사용한다.

### 테스트 커버리지 공백

Node 동작 변경은 없다. 기존 66개 top-level Test가 run/cancel/command/refresh/tunnel 경로를 덮으며 모두 그대로 이동한다.

### 심볼 참조

운영 심볼의 rename/remove는 없다. 기존 Test* 이름과 package 경계를 유지한다.

### 분할 판단

독립 05_core_node_tests다. 04_core_config_tests도 독립이고 06+04,05_core_edge_tests가 두 complete.log 뒤 core-tests를 최종 완료한다.

### 범위 결정 근거

node_test.go와 test-only helper만 수정한다. node.go, wire, adapter API는 제외한다.

### 빌드 등급

local-G07: 3,300줄, concurrency/refresh/tunnel fixture를 책임별로 옮기며 공유 double 축소가 필요하다.

## 구현 체크리스트

- [ ] [TEST-1] node_test.go를 run/cancel, command, refresh, tunnel 파일로 옮기고 Test 66개, 원본 제거, package test 통과를 보존한다.
- [ ] CODE_REVIEW-*-G??.md의 구현 에이전트 소유 섹션을 실제 구현 내용과 검증 출력으로 채운다. 이 항목이 완료되기 전에는 구현이 완료된 것이 아니다.

### [TEST-1] Node run/command/refresh/tunnel 테스트 분리

- 문제: apps/node/internal/node/node_test.go:283~3346의 run/cancel, command, refresh, tunnel 시나리오가 공통 fake와 한 파일에 결합되어 있다.
- 해결 방법:

~~~text
Before
apps/node/internal/node/node_test.go:283  func TestOnRunRequest_RouterError(...)
apps/node/internal/node/node_test.go:501  func TestOnCommandRequest_Success(...)
apps/node/internal/node/node_test.go:1563 func TestNodeConfigRefreshWithoutApplyManagerReportsRestartRequired(...)
apps/node/internal/node/node_test.go:3097 func TestNodeOnProviderTunnelRequest_Success(...)

After
node_test_support_test.go   최소 registry/adapter fixture
run_cancel_test.go          run/cancel/terminal event
command_test.go             command/capabilities
concurrency_gate_test.go    concurrency gate (reject/store/event)
gate_refresh_test.go        admission gate capacity + runtime concurrency metadata
registry_refresh_test.go    registry lifecycle (deferred stop, rollback)
provider_tunnel_test.go     provider tunnel
~~~

- 수정 파일 및 체크리스트:
  - [ ] apps/node/internal/node/node_test.go: 모든 테스트를 이동한 뒤 삭제한다.
  - [ ] apps/node/internal/node/node_test_support_test.go: 공통 최소 fixture를 둔다.
  - [ ] apps/node/internal/node/run_cancel_test.go: run/cancel/terminal event 테스트를 둔다.
  - [ ] apps/node/internal/node/command_test.go: command/capabilities 테스트를 둔다.
  - [ ] apps/node/internal/node/gate_refresh_test.go: admission/config refresh 테스트를 둔다.
  - [ ] apps/node/internal/node/provider_tunnel_test.go: tunnel 테스트를 둔다.
- 테스트 작성: 추가 테스트는 작성하지 않는다. concurrency assertion과 timing 값을 포함한 기존 Test 66개를 그대로 보존한다.
- 중간 검증:

~~~bash
gofmt -w apps/node/internal/node/*_test.go
test ! -e apps/node/internal/node/node_test.go
[ "$(GOCACHE=/tmp/iop-go-build-cache go test -list '^Test' ./apps/node/internal/node | sed '/^ok[[:space:]]/d' | wc -l)" -eq 66 ]
~~~

기대 결과: 원본 삭제, gofmt 성공, Test 66개.

## 수정 파일 요약

| 파일 | 항목 |
|---|---|
| apps/node/internal/node/node_test.go | TEST-1 |
| apps/node/internal/node/node_test_support_test.go | TEST-1 |
| apps/node/internal/node/run_cancel_test.go | TEST-1 |
| apps/node/internal/node/command_test.go | TEST-1 |
| apps/node/internal/node/gate_refresh_test.go | TEST-1 |
| apps/node/internal/node/concurrency_gate_test.go | TEST-1 |
| apps/node/internal/node/registry_refresh_test.go | TEST-1 |
| apps/node/internal/node/provider_tunnel_test.go | TEST-1 |

## 최종 검증

~~~bash
GOCACHE=/tmp/iop-go-build-cache go test -count=1 ./apps/node/internal/node
[ "$(GOCACHE=/tmp/iop-go-build-cache go test -list '^Test' ./apps/node/internal/node | sed '/^ok[[:space:]]/d' | wc -l)" -eq 66 ]
python3 -c "from pathlib import Path; p=Path('apps/node/internal/node'); bad=sorted((sum(1 for _ in f.open()),str(f)) for f in p.glob('*_test.go') if sum(1 for _ in f.open())>1000); print(bad); raise SystemExit(bool(bad))"
~~~

기대 결과: fresh Node test PASS, Test 66개, 1,000줄 초과 test 없음.

모든 코드 변경 완료 후 반드시 `CODE_REVIEW-*-G??.md`의 구현 에이전트 소유 섹션을 채운다. 이 파일 작성이 구현의 마지막 단계다.

