<!-- task=m-agent-task-runtime-target-selector/02+01_quota_input plan=1 tag=REVIEW_API -->

# Code Review Reference - REVIEW_API

> **[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, record the exact blocker, attempted commands/output, and resume condition only in implementation-owned evidence fields.
> Do not ask the user directly, present choices, call user-input tools, create control-plane stop files, or classify the next state.
> 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-25
task=m-agent-task-runtime-target-selector/02+01_quota_input, plan=1, tag=REVIEW_API

## Roadmap Targets

- Milestone: `agent-roadmap/phase/automation-runtime-bridge/milestones/agent-task-runtime-target-selector.md`
- Milestone link: [Milestone 문서](agent-roadmap/phase/automation-runtime-bridge/milestones/agent-task-runtime-target-selector.md)
- Task ids:
  - `quota-input`: Go usage checker 기반 required-cap quota snapshot과 selector admission
- Completion mode: check-on-pass

## Archive Evidence Snapshot

- Prior task: `agent-task/m-agent-task-runtime-target-selector/02+01_quota_input`
- Archived plan: `agent-task/m-agent-task-runtime-target-selector/02+01_quota_input/plan_cloud_G07_0.log`
- Archived review: `agent-task/m-agent-task-runtime-target-selector/02+01_quota_input/code_review_cloud_G09_0.log`
- Verdict: `FAIL`
- Findings: Required 4, Suggested 0, Nit 0
  - exhausted/ineligible 첫 후보가 계속 `selected`로 반환된다.
  - `NaN%`가 유효한 available evidence로 통과한 뒤 JSON 인코딩을 깨뜨린다.
  - 명시적 빈 `--target=`/`--command=`가 selector-invalid snapshot으로 이어질 수 있다.
  - 필수 Go 패키지 테스트가 `crypto/internal/fips140cache` 누락으로 setup 실패했다.
- Affected files: `agent-ops/skills/project/orchestrate-agent-task-loop/scripts/select_execution_target.py`, `agent-ops/skills/project/orchestrate-agent-task-loop/tests/test_select_execution_target.py`, `apps/node/internal/adapters/cli/status/quota.go`, `apps/node/internal/adapters/cli/status/quota_test.go`, `apps/node/cmd/node/quota_probe.go`, `apps/node/cmd/node/quota_probe_test.go`
- Verification evidence:
  - focused selector reproducer: selected=`agy/Gemini 3.6 Flash Medium`, candidates=`agy exhausted/ineligible`, `pi not_applicable/eligible`
  - Python selector/policy: 5 discovery tests와 30 selector tests PASS
  - Go packages: setup FAIL; `/config/opt/go/src/crypto/internal/fips140cache` missing
- Roadmap carryover: SDD S04와 Milestone Task `quota-input`은 미완료 상태를 유지한다.

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

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

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

1. 판정을 append한다.
2. `CODE_REVIEW-cloud-G09.md` → `code_review_cloud_G09_1.log`, `PLAN-cloud-G07.md` → `plan_cloud_G07_1.log`로 아카이브한다.
3. PASS이면 `complete.log` 작성 후 active task 디렉터리를 `agent-task/archive/YYYY/MM/m-agent-task-runtime-target-selector/02+01_quota_input/`로 이동한다. WARN/FAIL이면 code-review skill이 요구하는 다음 filesystem state를 완전히 작성한다.
4. PASS이고 task group이 `m-<milestone-slug>`이면 완료 이벤트 메타데이터를 보고한다. roadmap 상태 체크와 `update-roadmap` 호출은 런타임 책임이다.
5. 적용 가능한 `코드리뷰 전용 체크리스트` 항목을 최종 `.log` 위치에서 체크한 뒤 보고한다.

---

## 구현 항목별 완료 여부

| 항목 | 완료 여부 |
|------|---------|
| REVIEW_API-1 Exhausted 후보 선택 차단 | [x] |
| REVIEW_API-2 Quota producer 유한성 및 identity 검증 | [x] |
| REVIEW_API-3 Fresh 검증 신뢰 회복 | [ ] host Go/GOROOT 표준 라이브러리 불일치로 차단 |

## 구현 체크리스트

- [x] exhausted/ineligible 첫 후보를 `selected`에서 제외하고, eligible 대체 후보 선택과 no-eligible 결과를 selector 회귀 테스트로 고정한다.
- [x] overall/model 비유한 percent를 `unknown`으로 정규화하고 명시적 빈 target/command를 checker 호출 전 입력 오류로 거부하는 Go 테스트를 추가한다.
- [ ] 동일 Go 환경 preflight를 통과한 뒤 fresh Go package 테스트와 selector/policy 테스트 및 정적 검증을 모두 통과시킨다. host Go/GOROOT 표준 라이브러리 불일치로 차단됨.
- [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_cloud_G09_1.log`로 아카이브한다.
- [x] active `PLAN-*-G??.md`를 `plan_cloud_G07_1.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-agent-task-runtime-target-selector/02+01_quota_input/`를 `agent-task/archive/YYYY/MM/m-agent-task-runtime-target-selector/02+01_quota_input/`로 이동하고 최종 archive 경로에서 이 체크리스트를 갱신한다.
- [ ] PASS이고 task group이 `m-<milestone-slug>`이면 런타임이 읽을 완료 이벤트 메타데이터를 보고하고, roadmap 수정이나 `update-roadmap` 직접 호출을 하지 않는다.
- [ ] PASS split 작업이면 이동 후 빈 active parent `agent-task/m-agent-task-runtime-target-selector/`를 제거하거나, 남은 sibling/file이 있어 유지했다고 확인한다.
- [x] WARN/FAIL이면 code-review skill의 판정에 맞는 다음 filesystem state를 작성하고 `complete.log`를 작성하지 않는다.

## 계획 대비 변경 사항

없음. 계획한 selector admission 및 Go producer 입력 경계를 그대로 구현했다. host Go 설치는 repository 범위 밖이므로 다른 Go binary 또는 `GOTOOLCHAIN`으로 우회하지 않았다.

## 주요 설계 결정

- selector는 policy 순위를 보존하면서 `eligibility=eligible`인 첫 후보만 `selected`로 채운다. 후보가 모두 `exhausted`면 실행 가능한 target을 만들지 않고 `no_eligible_target` JSON 오류로 종료한다.
- non-finite percent(`NaN`, `Inf`, `-Inf`)는 remaining evidence가 아니라 `unknown`으로 정규화해 JSON encoder에 비유한 float가 도달하지 않게 했다.
- `quota-probe`는 target과 command를 trim한 뒤 둘 중 하나라도 비면 usage checker 호출 전에 입력 오류로 반환한다.

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

- exhausted first candidate가 selected에 남지 않고 policy order의 첫 eligible 후보로 이동하는가
- 모든 candidate가 exhausted일 때 실행 가능한 selected를 만들지 않고 안정적인 no-eligible 오류로 종료하는가
- unknown은 eligible 1회 admission 의미를 유지하고 local은 unbounded/not_applicable인가
- overall/model의 NaN/Inf가 available 또는 encoder failure가 아니라 unknown인가
- 빈 target/command가 checker 호출 전에 입력 오류로 종료되는가
- raw PTY output, checker error, generic stderr가 quota JSON에 노출되거나 exhausted 근거로 쓰이지 않는가
- 동일 Go/GOROOT source에서 fresh Go package tests가 실제로 실행됐는가

## 검증 결과

실행한 각 명령의 실제 stdout/stderr를 아래 코드 블록에 기록한다. 명령을 바꾸면 `계획 대비 변경 사항`에 대체 명령과 이유를 먼저 적는다.

### REVIEW_API-1 중간 검증

명령:

```bash
python3 agent-ops/skills/project/orchestrate-agent-task-loop/tests/test_select_execution_target.py -v -k SelectorQuotaRepresentationTests
```

실제 stdout/stderr:

```text
test_all_candidates_exhausted_returns_no_eligible_target (__main__.SelectorQuotaRepresentationTests.test_all_candidates_exhausted_returns_no_eligible_target) ... ok
test_exhausted_primary_selects_eligible_alternate (__main__.SelectorQuotaRepresentationTests.test_exhausted_primary_selects_eligible_alternate) ... ok
test_generic_stderr_is_not_quota_evidence (__main__.SelectorQuotaRepresentationTests.test_generic_stderr_is_not_quota_evidence) ... ok
test_injected_snapshot_is_reflected (__main__.SelectorQuotaRepresentationTests.test_injected_snapshot_is_reflected) ... ok
test_local_route_does_not_call_probe (__main__.SelectorQuotaRepresentationTests.test_local_route_does_not_call_probe) ... ok
test_quota_probe_tri_state (__main__.SelectorQuotaRepresentationTests.test_quota_probe_tri_state) ... ok
test_quota_representation_without_snapshot (__main__.SelectorQuotaRepresentationTests.test_quota_representation_without_snapshot) ... ok
test_unknown_is_admitted_once_per_work_unit (__main__.SelectorQuotaRepresentationTests.test_unknown_is_admitted_once_per_work_unit) ... ok

----------------------------------------------------------------------
Ran 8 tests in 0.042s

OK
```

### REVIEW_API-2 중간 검증

명령:

```bash
go test -count=1 ./apps/node/internal/adapters/cli/status ./apps/node/cmd/node
```

실제 stdout/stderr:

```text
# iop/apps/node/internal/adapters/cli/status
../../opt/go/src/crypto/ecdsa/ecdsa.go:27:2: package crypto/internal/fips140cache is not in std (/config/opt/go/src/crypto/internal/fips140cache)
# iop/apps/node/cmd/node
../../opt/go/src/crypto/ecdsa/ecdsa.go:27:2: package crypto/internal/fips140cache is not in std (/config/opt/go/src/crypto/internal/fips140cache)
FAIL	iop/apps/node/internal/adapters/cli/status [setup failed]
FAIL	iop/apps/node/cmd/node [setup failed]
FAIL
```

### REVIEW_API-3 Go 환경 프리플라이트

명령:

```bash
command -v go && readlink -f "$(command -v go)" && go version && go env GOROOT
test -d "$(go env GOROOT)/src/crypto/internal/fips140cache"
```

실제 stdout/stderr:

```text
/config/.local/bin/go
/config/opt/go/bin/go
go version go1.26.2 linux/arm64
/config/opt/go

`test -d "$(go env GOROOT)/src/crypto/internal/fips140cache"` exited 1 because the directory is absent. Resume after the `/config/opt/go` standard-library source is restored to match go1.26.2; then rerun this preflight and the required fresh Go package test without changing toolchains.
```

### 최종 검증

명령:

```bash
command -v go && readlink -f "$(command -v go)" && go version && go env GOROOT
test -d "$(go env GOROOT)/src/crypto/internal/fips140cache"
go test -count=1 ./apps/node/internal/adapters/cli/status ./apps/node/cmd/node
python3 -m unittest discover -s agent-ops/skills/project/orchestrate-agent-task-loop/tests -p 'test_execution_target*.py'
python3 agent-ops/skills/project/orchestrate-agent-task-loop/tests/test_select_execution_target.py -v
python3 -m py_compile agent-ops/skills/project/orchestrate-agent-task-loop/scripts/execution_target_policy.py agent-ops/skills/project/orchestrate-agent-task-loop/scripts/select_execution_target.py agent-ops/skills/project/orchestrate-agent-task-loop/tests/test_execution_target_policy.py agent-ops/skills/project/orchestrate-agent-task-loop/tests/test_select_execution_target.py
gofmt -d apps/node/internal/adapters/cli/status/quota.go apps/node/internal/adapters/cli/status/quota_test.go apps/node/cmd/node/quota_probe.go apps/node/cmd/node/quota_probe_test.go apps/node/cmd/node/main.go
git diff --check
```

실제 stdout/stderr:

```text
python3 -m unittest discover -s agent-ops/skills/project/orchestrate-agent-task-loop/tests -p 'test_execution_target*.py'
.....
----------------------------------------------------------------------
Ran 5 tests in 0.000s

OK

python3 agent-ops/skills/project/orchestrate-agent-task-loop/tests/test_select_execution_target.py -v
----------------------------------------------------------------------
Ran 32 tests in 0.415s

OK

python3 -m py_compile ...
gofmt -d ...
git diff --check

All three static commands exited 0 with no stdout/stderr.

Go preflight and `go test -count=1 ./apps/node/internal/adapters/cli/status ./apps/node/cmd/node` remain blocked as recorded above; this is not a pass. Required resume condition: restore `/config/opt/go/src/crypto/internal/fips140cache` in the host Go 1.26.2 GOROOT, retain `/config/.local/bin/go` as the first PATH Go binary, then rerun the complete final verification command block.
```

---

> **[IMPLEMENTING AGENT — BEFORE SAVING] Have you filled in every implementation-owned section?**
> 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 |
| 코드리뷰 전용 체크리스트 | Review agent only | Implementing agent must not modify or check this section |
| 계획 대비 변경 사항, 주요 설계 결정 | Implementing agent | Replace placeholder text with actual content |
| 리뷰어를 위한 체크포인트 | 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: Pass
  - verification trust: Fail
  - spec conformance: Fail
- 발견된 문제:
  - Required — `agent-task/m-agent-task-runtime-target-selector/02+01_quota_input/CODE_REVIEW-cloud-G09.md:72`: 필수 fresh Go 검증을 리뷰에서 재실행했지만 `/config/opt/go/src/crypto/internal/fips140cache`가 여전히 없어 두 대상 패키지가 setup 단계에서 실패했다. selector 32개 테스트, policy 5개 테스트, `py_compile`, `gofmt -d`, `git diff --check`는 통과했고 이전 세 코드 결함의 수정도 소스에서 확인했지만, 새 Go producer/command 코드의 컴파일과 회귀 assertion이 실행되지 않아 SDD S04 Evidence Map과 plan의 통합 검증을 충족하지 못한다. `/config/opt/go`를 `/config/opt/go/bin/go`의 go1.26.2 표준 라이브러리와 일치하도록 복구한 뒤 동일 PATH/GOROOT에서 `go test -count=1 ./apps/node/internal/adapters/cli/status ./apps/node/cmd/node`와 전체 최종 검증을 통과시켜야 한다.
- 다음 단계: FAIL 후속을 plan 스킬의 `prepare-follow-up` 모드와 격리 재라우팅으로 준비한다.
