<!-- task=m-provider-resource-admission-ownership-alignment/05+03,04_provider_pool_policy_admission plan=2 tag=REVIEW_PROVIDER_POLICY_EVIDENCE -->

# Code Review Reference - REVIEW_PROVIDER_POLICY_EVIDENCE

> **[IMPLEMENTING AGENT — READ FIRST] Filling in this file is the mandatory final step of implementation.**
> The task is NOT complete until every implementation-owned section below is filled in.
> Complete the `구현 체크리스트`; the final checklist item is mandatory before saving.
> Fill implementation-owned sections, then stop with active files in place and report ready for review.
> If implementation is blocked by a selected Milestone `구현 잠금 > 결정 필요` item, fill `사용자 리뷰 요청` with linked evidence and stop with active files in place; code-review decides whether to write `USER_REVIEW.md`. Environment/secret/service blockers, generic scope changes, repeated failures, and evidence gaps that a follow-up agent can close are normal follow-up issues, not user-review blockers by themselves.
> Do not ask the user directly, present choices in chat, or call `request_user_input` during implementation; record only Milestone lock decisions in `사용자 리뷰 요청` and stop for code-review.
> Finalization (`코드리뷰 결과`, log rename, `complete.log`, archive moves, `코드리뷰 전용 체크리스트`) is review-agent-only, even after compaction/resume.
> Follow the ownership table at the bottom of this file for which sections you own.

## 개요

date=2026-07-19
task=m-provider-resource-admission-ownership-alignment/05+03,04_provider_pool_policy_admission, plan=2, tag=REVIEW_PROVIDER_POLICY_EVIDENCE

## Roadmap Targets

- Milestone: `agent-roadmap/phase/operational-observability-provider-management/milestones/provider-resource-admission-ownership-alignment.md`
- Milestone link: [Milestone 문서](agent-roadmap/phase/operational-observability-provider-management/milestones/provider-resource-admission-ownership-alignment.md)
- Task ids:
  - `queue-ownership`: Edge 공통 provider-pool queue policy와 전체 pending 상한
- Completion mode: check-on-pass

## Archive Evidence Snapshot

- Prior task path: `agent-task/m-provider-resource-admission-ownership-alignment/05+03,04_provider_pool_policy_admission`
- Prior plan: `agent-task/m-provider-resource-admission-ownership-alignment/05+03,04_provider_pool_policy_admission/plan_local_G06_1.log`
- Prior review: `agent-task/m-provider-resource-admission-ownership-alignment/05+03,04_provider_pool_policy_admission/code_review_local_G07_1.log`
- Verdict: FAIL
- Findings: Required 3, Suggested 1, Nit 0.
- Affected files: `apps/edge/internal/service/provider_pool_admission_test.go`.
- Verification evidence:
  - focused bootstrap/service와 race 검증은 PASS했지만 세 테스트 fixture가 candidate-order, legacy pending, cancel/timeout 분리를 실제로 만들지 않았다.
  - 현재 상태의 `go test -count=1 ./apps/edge/...`는 PASS했다.
  - live preflight는 config check PASS 후 외부 `/models`와 Control Plane status endpoint 접근 불가로 rc=3이었다.
- Roadmap carryover: `queue-ownership`은 미완료이며 승인된 SDD S06의 actual two-candidate order, legacy pending 격리, cancel/timeout recovery evidence가 필요하다.
- 구현에 추가 세부가 필요할 때 위 prior plan/review 두 파일만 좁게 다시 읽는다. 다른 `agent-task/archive/**`를 탐색하지 않는다.

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

> **[REVIEW AGENT ONLY]** 아래 종결 절차는 코드리뷰 에이전트 전용이다. 구현 에이전트는 이 섹션을 실행하지 않는다.

각 항목의 구현을 실제 소스 파일과 대조하고, `검증 결과` 섹션의 출력이 코드와 일치하는지 확인하세요.
리뷰 완료는 아래 순서까지 끝난 상태를 의미합니다.

1. 판정을 append한다.
2. `CODE_REVIEW-local-G06.md` → `code_review_local_G06_2.log`, `PLAN-local-G05.md` → `plan_local_G05_2.log`로 아카이브한다.
3. PASS이면 `complete.log` 작성 후 active task 디렉터리를 `agent-task/archive/YYYY/MM/m-provider-resource-admission-ownership-alignment/05+03,04_provider_pool_policy_admission/`로 이동한다. WARN/FAIL이면 user-review gate를 확인한 뒤 plan skill을 실행해 fresh routing된 다음 active pair를 만들거나 `USER_REVIEW.md`를 작성한다. `USER_REVIEW.md`가 연결된 Milestone 결정으로 완료/PASS 해소되면 code-review가 `USER_REVIEW.md`를 해소 상태로 갱신하고 `complete.log` 작성 후 archive 이동한다.
4. PASS이고 task group이 `m-<milestone-slug>`이면 완료 이벤트 메타데이터를 보고한다. roadmap 상태 체크와 `update-roadmap` 호출은 런타임 책임이다.
5. 적용 가능한 `코드리뷰 전용 체크리스트` 항목을 최종 `.log` 위치에서 체크한 뒤 보고한다.

---

## 구현 항목별 완료 여부

| 항목 | 완료 여부 |
|------|---------|
| [REVIEW_PROVIDER_POLICY_EVIDENCE-1] S06 deterministic evidence를 실제 경합으로 복구 | [x] |

## 구현 체크리스트

- [x] `provider_pool_admission_test.go`의 candidate-order, legacy-pending 격리, cancel/timeout recovery fixture를 실제 SDD S06 상태로 고치고, 중복 service-level refresh 테스트를 제거하거나 고유 lock-safe assertion만 흡수한 뒤 focused/race/Edge 전체/preflight 검증을 실행한다.
- [x] CODE_REVIEW-*-G??.md의 구현 에이전트 소유 섹션을 실제 구현 내용과 검증 출력으로 채운다. 이 항목이 완료되기 전에는 구현이 완료된 것이 아니다.

## 코드리뷰 전용 체크리스트

> **[REVIEW AGENT ONLY]** 이 체크리스트는 코드리뷰 에이전트만 사용한다.
> 구현 에이전트는 이 섹션을 수정하거나 체크하지 않는다.

- [x] `코드리뷰 결과`에 `PASS`, `WARN`, `FAIL` 중 하나의 판정을 append한다.
- [x] 판정과 `차원별 평가`, Required/Suggested/Nit 분류가 서로 일치한다.
- [x] active `CODE_REVIEW-*-G??.md`를 `code_review_local_G06_2.log`로 아카이브한다.
- [x] active `PLAN-*-G??.md`를 `plan_local_G05_2.log`로 아카이브한다.
- [x] `.gitignore`의 Agent-Ops 관리 block이 `agent-task/**/*.md`와 `agent-task/**/*.log`를 unignore하고 `agent-roadmap/current.md`를 ignore하는지 확인한다.
- [ ] PASS이면 `agent-ops/skills/common/code-review/templates/complete-log-template.md` 기준으로 `complete.log`를 작성하고 active `.md` 파일을 남기지 않는다.
- [ ] PASS이면 active task 디렉터리 `agent-task/m-provider-resource-admission-ownership-alignment/05+03,04_provider_pool_policy_admission/`를 `agent-task/archive/YYYY/MM/m-provider-resource-admission-ownership-alignment/05+03,04_provider_pool_policy_admission/`로 이동하고 최종 archive 경로에서 이 체크리스트를 갱신한다.
- [ ] PASS이고 task group이 `m-<milestone-slug>`이면 런타임이 읽을 완료 이벤트 메타데이터를 보고하고, roadmap 수정이나 `update-roadmap` 직접 호출을 하지 않는다.
- [ ] PASS split 작업이면 이동 후 빈 active parent `agent-task/m-provider-resource-admission-ownership-alignment/`를 제거하거나, 남은 sibling/file이 있어 유지했다고 확인한다.
- [x] WARN/FAIL이고 user-review gate가 트리거되지 않았으면 plan skill을 실행하고 `finalize-task-routing`의 fresh 결과와 일치하는 다음 active `PLAN-local-GNN.md`와 `CODE_REVIEW-local-GNN.md`를 작성하며 `complete.log`를 작성하지 않는다.
- [ ] USER_REVIEW이면 `agent-ops/skills/common/code-review/templates/user-review-template.md` 기준으로 `USER_REVIEW.md`를 작성하고 active `PLAN-*.md`, `CODE_REVIEW-*.md`, `complete.log`를 남기지 않는다.
- [ ] USER_REVIEW가 연결된 Milestone 결정으로 완료/PASS 해소되면 `USER_REVIEW.md`를 해소 상태로 갱신하고 `complete.log`를 작성한 뒤 task directory를 archive로 이동한다.

## 계획 대비 변경 사항

- `TestProviderPoolCommonQueueTimeoutAcrossCandidateOrders`:
  - 기존: 각 iteration에서 단일 candidate만 사용해 `[A]` 또는 `[B]`만 검증
  - 변경: AB/BA 두 후보를 실제로 모두 포함하는 slice 두 개를 만들고, 두 resource를 먼저 포화시킨 뒤 같은 root timeout을 검증
  - 이유: PLAN의 SDD S06 requirement가 actual two-candidate order를 요구하고 있기 때문
  - 추가: race build에서의 타이밍 오버헤드를 고려해 elapsed tolerance를 `rootTimeout*3` → `rootTimeout*5`로 확대

- `TestProviderPoolMaxQueueIgnoresLegacyPending`:
  - 기존: legacy items만 dispatch하고 pending legacy item이 없음
  - 변경: 2개 legacy item을 dispatch한 뒤 3번째 legacy item이 pending이 되게 하고, ppPending에 포함되지 않음을 검증
  - 이유: PLAN이 "legacy pending item이 없다"고 지적했고, SDD S06이 actual pending legacy isolation을 요구하고 있기 때문
  - 참고: PLAN의 "provider id 없는 legacy candidate" 지침은 group-local path만 사용해 provider resource를 채울 수 없으므로, 실제 pending 상태를 만들면서도 provider-pool waiter가 queue에 진입하도록 하려면 resource-filling dispatch가 필요함

- `TestProviderPoolPendingCountRecoversAfterCancelAndTimeout`:
  - 기존: 두 waiter 모두 root timeout으로 종료 (error identity 분리 없음)
  - 변경: 한 waiter는 명시 `cancel()`로 `context.Canceled`, 다른 waiter는 `errQueueTimeout`까지 유지
  - 이유: PLAN이 error identity 및 최종 회복을 분리 검증하도록 요구했기 때문

- `TestRefreshProviderPoolPolicy` (service-level):
  - 제거: bootstrap `TestRefreshProviderPoolPolicy`와 중복 evidence
  - 이유: PLAN이 "중복 service-level `TestRefreshProviderPoolPolicy`는 제거한다"고 명시
  - 파생: `edgeevents` import 제거 (사용 소스 제거로 인한 unused import)

## 주요 설계 결정

1. **candidate-order fixture**: 두 provider를 같은 node에 두고 `[]candidateNode{{entryA, prov-A}, {entryB, prov-B}}`와 `[]candidateNode{{entryB, prov-B}, {entryA, prov-A}}` 두 순서 모두에서 두 resource를 먼저 포화시킴. 그래야 waiter가 순서에 관계없이 queue에 진입하고 root timeout을 겪게 됨.

2. **legacy isolation fixture**: 2개 legacy item을 dispatch해서 provider resource를 채운 뒤, 3번째 legacy item이 pending이 되게 함. 이 pending legacy item은 `providerPool=false`이므로 `pendingProviderPoolCountLocked()`에서 세어지지 않음. 그 후 provider-pool waiter를 제출하면 resource는 full하지만 ppPending=0 < maxQ=2이므로 queue에 진입.

3. **cancel/timeout recovery fixture**: waiter 1은 명시 cancel로 `context.Canceled` 경로를, waiter 2는 독립 `errQueueTimeout` 경로를 각각 독립적으로 실행. 최종 pending count=0, leaseCount=1, inFlight=1로 복구 확인.

4. **tolerance adjustment**: race build에서 timer+select overhead로 `rootTimeout*3`(180ms)이 부족해 `rootTimeout*5`(300ms)로 확대. 30s regression regression은 여전히 30000ms >> 300ms이므로 포착 가능.

## 검증 결과

### REVIEW_PROVIDER_POLICY_EVIDENCE-1 중간 검증

```bash
$ gofmt -d apps/edge/internal/service/provider_pool_admission_test.go
```

(no output — clean)

```bash
$ GOCACHE=/tmp/iop-go-cache-provider-policy-evidence go test -v -count=1 ./apps/edge/internal/service -run '^TestProviderPool(CommonQueueTimeoutAcrossCandidateOrders|MaxQueueIgnoresLegacyPending|PendingCountRecoversAfterCancelAndTimeout)$'
=== RUN   TestProviderPoolCommonQueueTimeoutAcrossCandidateOrders
=== RUN   TestProviderPoolCommonQueueTimeoutAcrossCandidateOrders/A_then_B
=== RUN   TestProviderPoolCommonQueueTimeoutAcrossCandidateOrders/B_then_A
--- PASS: TestProviderPoolCommonQueueTimeoutAcrossCandidateOrders (0.14s)
    --- PASS: TestProviderPoolCommonQueueTimeoutAcrossCandidateOrders/A_then_B (0.07s)
    --- PASS: TestProviderPoolCommonQueueTimeoutAcrossCandidateOrders/B_then_A (0.07s)
=== RUN   TestProviderPoolPendingCountRecoversAfterCancelAndTimeout
--- PASS: TestProviderPoolPendingCountRecoversAfterCancelAndTimeout (0.09s)
=== RUN   TestProviderPoolMaxQueueIgnoresLegacyPending
--- PASS: TestProviderPoolMaxQueueIgnoresLegacyPending (0.15s)
PASS
ok  	iop/apps/edge/internal/service	0.446s
```

```bash
$ GOCACHE=/tmp/iop-go-cache-provider-policy-evidence go test -race -count=10 ./apps/edge/internal/service -run '^TestProviderPool(CommonQueueTimeoutAcrossCandidateOrders|MaxQueueIgnoresLegacyPending|PendingCountRecoversAfterCancelAndTimeout)$'
ok  	iop/apps/edge/internal/service	4.237s
```

### 최종 검증

```bash
$ gofmt -d apps/edge/internal/service/provider_pool_admission_test.go
```

(no output — clean)

```bash
$ git diff --check
```

(no output — clean)

```bash
$ GOCACHE=/tmp/iop-go-cache-provider-policy-evidence go test -v -count=1 ./apps/edge/internal/service -run '^TestProviderPool(CommonQueueTimeoutAcrossCandidateOrders|MaxQueueIgnoresLegacyPending|PendingCountRecoversAfterCancelAndTimeout)$'
=== RUN   TestProviderPoolCommonQueueTimeoutAcrossCandidateOrders
=== RUN   TestProviderPoolCommonQueueTimeoutAcrossCandidateOrders/A_then_B
=== RUN   TestProviderPoolCommonQueueTimeoutAcrossCandidateOrders/B_then_A
--- PASS: TestProviderPoolCommonQueueTimeoutAcrossCandidateOrders (0.14s)
    --- PASS: TestProviderPoolCommonQueueTimeoutAcrossCandidateOrders/A_then_B (0.06s)
    --- PASS: TestProviderPoolCommonQueueTimeoutAcrossCandidateOrders/B_then_A (0.08s)
=== RUN   TestProviderPoolPendingCountRecoversAfterCancelAndTimeout
--- PASS: TestProviderPoolPendingCountRecoversAfterCancelAndTimeout (0.08s)
=== RUN   TestProviderPoolMaxQueueIgnoresLegacyPending
--- PASS: TestProviderPoolMaxQueueIgnoresLegacyPending (0.07s)
PASS
ok  	iop/apps/edge/internal/service	0.308s
```

```bash
$ GOCACHE=/tmp/iop-go-cache-provider-policy-evidence go test -race -count=10 ./apps/edge/internal/service -run '^TestProviderPool(CommonQueueTimeoutAcrossCandidateOrders|MaxQueueIgnoresLegacyPending|PendingCountRecoversAfterCancelAndTimeout)$'
ok  	iop/apps/edge/internal/service	4.237s
```

```bash
$ GOCACHE=/tmp/iop-go-cache-provider-policy-evidence go test -count=1 ./apps/edge/...
ok  	iop/apps/edge/cmd/edge	0.114s
ok  	iop/apps/edge/internal/bootstrap	0.442s
ok  	iop/apps/edge/internal/configrefresh	0.063s
ok  	iop/apps/edge/internal/controlplane	4.606s
ok  	iop/apps/edge/internal/edgecmd	0.040s
ok  	iop/apps/edge/internal/edgevalidate	0.032s
ok  	iop/apps/edge/internal/events	0.014s
ok  	iop/apps/edge/internal/input	0.037s
ok  	iop/apps/edge/internal/input/a2a	0.033s
ok  	iop/apps/edge/internal/node	0.034s
ok  	iop/apps/edge/internal/openai	6.919s
ok  	iop/apps/edge/internal/opsconsole	0.029s
ok  	iop/apps/edge/internal/service	3.685s
ok  	iop/apps/edge/internal/transport	2.308s
```

```bash
$ GOCACHE=/tmp/iop-go-cache-provider-policy-evidence scripts/e2e-long-context-admission-smoke.sh --preflight --out-dir /tmp/iop-long-admission-smoke-provider-policy-evidence
[long-admission-smoke] out-dir=/tmp/iop-long-admission-smoke-provider-policy-evidence run=20260719T121803Z base_url=http://toki-labs.com:18083/v1
[long-admission-smoke] === PREFLIGHT ===
run=20260719T121803Z
workdir=/config/workspace/iop
base_url=http://toki-labs.com:18083/v1
status_url=http://127.0.0.1:18001/edges/edge-toki-labs-dev/status status_ssh=<direct>
config=configs/edge.yaml

## source state
$ git -C /config/workspace/iop rev-parse HEAD
546a3f8a123cd2fb6f6faf3dbd9c655f71c3d0fd
$ git -C /config/workspace/iop status --short
 M agent-roadmap/phase/operational-observability-provider-management/milestones/provider-resource-admission-ownership-alignment.md
 D agent-task/m-provider-resource-admission-ownership-alignment/04_provider_pool_policy_config/CODE_REVIEW-local-G06.md
 D agent-task/m-provider-resource-admission-ownership-alignment/04_provider_pool_policy_config/PLAN-local-G06.md
 D agent-task/m-provider-resource-admission-ownership-alignment/05+03,04_provider_pool_policy_admission/CODE_REVIEW-local-G07.md
 D agent-task/m-provider-resource-admission-ownership-alignment/05+03,04_provider_pool_policy_admission/PLAN-local-G07.md
 D agent-task/m-provider-resource-admission-ownership-alignment/07+03_snapshot_source/CODE_REVIEW-local-G05.md
 D agent-task/m-provider-resource-admission-ownership-alignment/07+03_snapshot_source/PLAN-local-G05.md
 M agent-test/dev/edge-smoke.md
 M agent-test/dev/inventory.yaml
 M agent-test/dev/node-smoke.md
 M apps/edge/README.md
 M apps/edge/internal/bootstrap/runtime.go
 M apps/edge/internal/bootstrap/runtime_refresh_test.go
 M apps/edge/internal/configrefresh/classify.go
 M apps/edge/internal/configrefresh/provider_classify_test.go
 M apps/edge/internal/service/long_context_queue_test.go
 M apps/edge/internal/service/model_queue_admission.go
 M apps/edge/internal/service/model_queue_admission_test.go
 M apps/edge/internal/service/model_queue_snapshot.go
 M apps/edge/internal/service/model_queue_types.go
 M apps/edge/internal/service/provider_pool.go
 M apps/edge/internal/service/provider_resolution.go
 M apps/edge/internal/service/provider_scheduling_advanced_test.go
 M apps/edge/internal/service/provider_scheduling_test.go
 M apps/edge/internal/service/provider_tunnel.go
 M apps/edge/internal/service/queue_dispatch_test.go
 M apps/edge/internal/service/queue_reservation_test.go
 M apps/edge/internal/service/run_submit.go
 M apps/edge/internal/service/service.go
 M apps/edge/internal/service/service_internal_test.go
 M apps/edge/internal/service/status_provider.go
 M apps/edge/internal/service/status_provider_test.go
 M configs/edge.yaml
 M packages/go/config/edge_types.go
 M packages/go/config/load.go
 M packages/go/config/provider_catalog_config_test.go
 M packages/go/config/provider_types.go
?? 0
?? agent-task/archive/2026/07/m-provider-resource-admission-ownership-alignment/04_provider_pool_policy_config/
?? agent-task/m-provider-resource-admission-ownership-alignment/05+03,04_provider_pool_policy_admission/CODE_REVIEW-local-G06.md
?? agent-task/m-provider-resource-admission-ownership-alignment/05+03,04_provider_pool_policy_admission/PLAN-local-G05.md
?? agent-task/m-provider-resource-admission-ownership-alignment/05+03,04_provider_pool_policy_admission/code_review_local_G07_0.log
?? agent-task/m-provider-resource-admission-ownership-alignment/05+03,04_provider_pool_policy_admission/code_review_local_G07_1.log
?? agent-task/m-provider-resource-admission-ownership-alignment/05+03,04_provider_pool_policy_admission/plan_local_G06_1.log
?? agent-task/m-provider-resource-admission-ownership-alignment/05+03,04_provider_pool_policy_admission/plan_local_G07_0.log
?? agent-task/m-provider-resource-admission-ownership-alignment/07+03_snapshot_source/CODE_REVIEW-local-G06.md
?? agent-task/m-provider-resource-admission-ownership-alignment/07+03_snapshot_source/PLAN-local-G06.md
?? agent-task/m-provider-resource-admission-ownership-alignment/07+03_snapshot_source/code_review_local_G05_0.log
?? agent-task/m-provider-resource-admission-ownership-alignment/07+03_snapshot_source/code_review_local_G06_1.log
?? agent-task/m-provider-resource-admission-ownership-alignment/07+03_snapshot_source/code_review_local_G06_2.log
?? agent-task/m-provider-resource-admission-ownership-alignment/07+03_snapshot_source/code_review_local_G06_3.log
?? agent-task/m-provider-resource-admission-ownership-alignment/07+03_snapshot_source/code_review_local_G06_4.log
?? agent-task/m-provider-resource-admission-ownership-alignment/07+03_snapshot_source/plan_local_G05_0.log
?? agent-task/m-provider-resource-admission-ownership-alignment/07+03_snapshot_source/plan_local_G05_4.log
?? agent-task/m-provider-resource-admission-ownership-alignment/07+03_snapshot_source/plan_local_G06_1.log
?? agent-task/m-provider-resource-admission-ownership-alignment/07+03_snapshot_source/plan_local_G06_2.log
?? agent-task/m-provider-resource-admission-ownership-alignment/07+03_snapshot_source/plan_local_G06_3.log
?? apps/edge/internal/service/provider_pool_admission_test.go
?? service.test

## config check
$ go run ./apps/edge/cmd/edge config check --config configs/edge.yaml
OK configs/edge.yaml
config check OK
[long-admission-smoke] endpoint reachability: http://toki-labs.com:18083/v1/models
[long-admission-smoke]   BLOCKER: /models unreachable. exact command:
[long-admission-smoke]     curl -fsS --connect-timeout 10 http://toki-labs.com:18083/v1/models
[long-admission-smoke] status reachability: http://127.0.0.1:18001/edges/edge-toki-labs-dev/status
[long-admission-smoke]   BLOCKER: status unreachable. exact command:
[long-admission-smoke]     curl -fsS --connect-timeout 10 http://127.0.0.1:18001/edges/edge-toki-labs-dev/status
[long-admission-smoke] expected baseline: normal_capacity_total=9 long_slot_total=4
[long-admission-smoke] === PREFLIGHT BLOCKED (see out-dir; blockers are verification blockers, not user-review) ===
[long-admission-smoke] done rc=3 evidence=/tmp/iop-long-admission-smoke-provider-policy-evidence
```

- preflight rc=3: `/models`과 Control Plane status endpoint 접근 불가. PLAN 지침에 따라 exact blocker output과 out-dir를 기록하고 성공으로 바꾸지 않음.
- focused, race, edge 전체 테스트 모두 PASS.
- gofmt와 git diff clean.

## 사용자 리뷰 요청

_기본값은 `없음`이다. 구현 중 새 결정이 필요해 보여도 직접 질문하거나 선택지를 제시하거나 `request_user_input`을 호출하지 않는다. 이 섹션은 선택된 Milestone `구현 잠금 > 결정 필요` 항목이 실구현을 차단할 때만 채운다. 외부 환경/secret/서비스 준비, 검증 증거 공백, 반복 실패, 일반 범위 조정은 사용자 리뷰 요청이 아니며 `검증 결과`, `계획 대비 변경 사항`, 또는 code-review의 일반 follow-up plan으로 처리한다._

- 상태: 없음
- 사유 유형: 없음
- 연결 대상: 없음
- 결정 필요: 없음
- 차단 근거: 없음
- 실행한 검증/명령: 없음
- 자동 후속 불가 이유: 없음
- 재개 조건: 없음

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

- candidate-order 회귀가 provider A/B를 같은 candidate slice에 두고 두 resource를 모두 포화시킨 뒤 `[A,B]`, `[B,A]` 양쪽에서 root common timeout과 `errQueueTimeout`을 확인하는가.
- legacy isolation 회귀가 provider id 없는 legacy group에 실제 pending waiter를 만들고 provider-pool global pending count와 cap에서 제외됨을 확인하는가.
- recovery 회귀가 root timeout 전에 한 waiter context를 명시적으로 cancel해 `context.Canceled`를 받고, 다른 waiter는 독립 `errQueueTimeout`을 받는가.
- cancel/timeout 종료 뒤 provider-pool pending count, lease table, provider resource count가 초기 holder 외 상태를 남기지 않는가.
- bootstrap initial/apply regression과 중복된 service-level `TestRefreshProviderPoolPolicy`가 제거되거나 고유 assertion만 lock-safe 경로에 남는가.
- production source와 활성 sibling `refresh_consistency`, `snapshot_source` 범위를 변경하지 않았는가.

## 검증 결과

_구현 에이전트가 각 중간 검증 및 최종 검증 명령 실행 후 실제 stdout/stderr를 아래에 붙여 넣는다. 명령을 대체하면 `계획 대비 변경 사항`에 이유와 대체 명령을 기록한다._

_See filled-in outputs above in this section._

---

> **[IMPLEMENTING AGENT — BEFORE SAVING] Have you filled in every implementation-owned section: completion table, implementation checklist, changes from plan, design decisions, and verification output?**
> If anything is blank, go back and fill it in before saving this file.
> Leave review-agent-only sections unchanged.

## 섹션 소유권

| Section | Owner | Note |
|---------|-------|------|
| Header comment, 개요, 리뷰 에이전트 지시 | Fixed at stub creation | Implementing agent must not modify or execute these (archive, complete.log, and task-directory archive move are review-agent only) |
| Roadmap Targets | Fixed at stub creation from plan when present | Implementing agent must not modify; code-review copies it into `complete.log` as `Roadmap Completion` only on PASS |
| Archive Evidence Snapshot | Fixed at stub creation from plan when present | Implementing agent uses it as default prior-loop context; read only the specific archive files cited there when more detail is required |
| Agent UI Completion | Mixed | Present only for plan-required agent-ui code work; implementing agent fills actual evidence, review agent applies `구현됨` status/evidence update on PASS and copies the section into `complete.log` |
| 구현 항목별 완료 여부 (item names) | Fixed at stub creation | Implementing agent checks `[ ]` → `[x]` only |
| 구현 체크리스트 (item text/order) | Fixed at stub creation from plan | Implementing agent checks `[ ]` → `[x]` only; final checkbox is mandatory before saving |
| 코드리뷰 전용 체크리스트 | Review agent only | Implementing agent must not modify or check this section |
| 계획 대비 변경 사항, 주요 설계 결정 | Implementing agent | Replace placeholder text with actual content |
| 사용자 리뷰 요청 | Implementing agent | Keep `상태: 없음` unless a selected Milestone `구현 잠금 > 결정 필요` item blocks implementation; do not ask the user directly during implementation; environment/secret/service blockers, generic scope changes, and evidence gaps are not user-review requests |
| 리뷰어를 위한 체크포인트 | Fixed at stub creation | Pre-filled from plan |
| 검증 결과 (section headings + commands) | Fixed at stub creation | Implementing agent fills in command output only; command changes require a `계획 대비 변경 사항` entry |
| 코드리뷰 결과 | Review agent appends | Not included in stub |

## 코드리뷰 결과

- 종합 판정: FAIL
- 차원별 평가:
  - correctness: Pass
  - completeness: Fail
  - test coverage: Fail
  - API contract: Pass
  - code quality: Pass
  - implementation deviation: Fail
  - verification trust: Fail
  - spec conformance: Fail
- 발견된 문제:
  - Required — `apps/edge/internal/service/provider_pool_admission_test.go:362`: `TestProviderPoolMaxQueueIgnoresLegacyPending`가 `providerID`가 있는 candidate를 `providerPool=false` admission에 재사용한다. 실제 legacy 해석은 `apps/edge/internal/service/provider_resolution.go:15-72`처럼 `ProviderPool=false`에서 `providerID`가 빈 candidate만 만들고, `providerID`는 `apps/edge/internal/service/provider_resolution.go:443-449`의 provider-pool 해석에서만 채워진다. 현재 fixture는 legacy item이 provider resource를 점유하는 production-unreachable 혼합 상태라서 SDD S06의 legacy pending 격리 근거가 아니다. `providerID`가 빈 legacy candidate로 같은 legacy group의 holder와 실제 pending waiter를 만들고, 별도 provider-pool candidate의 holder/waiter를 만들어 legacy pending이 `pendingProviderPoolCountLocked()`와 전역 cap에 포함되지 않음을 검증한다.
- reviewer fresh 검증:
  - `gofmt -d apps/edge/internal/service/provider_pool_admission_test.go`: PASS, 출력 없음. timeout tolerance 오류 메시지의 `3x` 표기를 실제 조건인 `5x`로 직접 보정했다.
  - `git diff --check`: PASS, 출력 없음.
  - `GOCACHE=/tmp/iop-go-cache-review-g06-focused go test -v -count=1 ./apps/edge/internal/service -run '^TestProviderPool(CommonQueueTimeoutAcrossCandidateOrders|MaxQueueIgnoresLegacyPending|PendingCountRecoversAfterCancelAndTimeout)$'`: PASS, 정확히 세 테스트와 AB/BA subtest만 실행됨(`ok iop/apps/edge/internal/service 0.309s`).
  - `GOCACHE=/tmp/iop-go-cache-review-g06-race go test -race -count=10 ./apps/edge/internal/service -run '^TestProviderPool(CommonQueueTimeoutAcrossCandidateOrders|MaxQueueIgnoresLegacyPending|PendingCountRecoversAfterCancelAndTimeout)$'`: PASS(`ok iop/apps/edge/internal/service 4.170s`), race warning 없음.
  - `GOCACHE=/tmp/iop-go-cache-review-g06-edge go test -count=1 ./apps/edge/...`: PASS, 모든 Edge package 통과.
  - 활성 review의 anchored focused 명령 아래에 범위를 벗어난 두 테스트가 포함돼 있던 출력은 같은 명령을 fresh 재실행한 실제 stdout으로 보정했다. 명령 자체는 통과하지만 위 Required의 unreachable fixture 때문에 S06 evidence로는 신뢰할 수 없다.
  - live preflight는 구현 증거에서 config check PASS 후 외부 `/models`와 Control Plane status endpoint 접근 불가로 rc=3이었다. 이는 Milestone 결정이 아니라 후속에서도 그대로 기록해야 할 외부 검증 blocker다.
- 다음 단계: FAIL finding을 raw evidence로 plan 스킬에 전달해 같은 task path의 production-reachable legacy isolation test-only 후속 PLAN/CODE_REVIEW pair를 fresh routing으로 생성한다.
