<!-- task=m-iop-agent-cli-runtime/01_common_runtime_node_bridge plan=0 tag=API -->

# Code Review Reference - 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-28
task=m-iop-agent-cli-runtime/01_common_runtime_node_bridge, plan=0, tag=API

## Roadmap Targets

- Milestone: `agent-roadmap/phase/automation-runtime-bridge/milestones/iop-agent-cli-runtime.md`
- Milestone link: [Milestone 문서](agent-roadmap/phase/automation-runtime-bridge/milestones/iop-agent-cli-runtime.md)
- Task ids:
  - `node-consumer`: Node의 공통 runtime bridge 전환
- Completion mode: check-on-pass

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

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

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

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

---

## 구현 항목별 완료 여부

| 항목 | 완료 여부 |
|------|---------|
| API-1 공통 runtime 계약과 API 확정 | [x] |
| API-2 CLI runtime 단일 구현 추출 | [x] |
| API-3 Node consumer bridge와 호환성 보존 | [x] |
| API-4 통합 및 중복 제거 검증 | [x] |

## 구현 체크리스트

- [x] API-1 공통 agent runtime inner contract와 public Go API를 확정한다.
- [x] API-2 CLI provider, emitter/stream/session, status/quota, failure codec의 단일 공통 구현과 conformance tests를 만든다.
- [x] API-3 Node를 공통 runtime consumer bridge로 전환하고 기존 wire/config/provider 회귀 tests를 통과시킨다.
- [x] API-4 전체 fresh/race 검증과 duplicate implementation search를 실행한다.
- [x] CODE_REVIEW-*-G??.md의 구현 에이전트 소유 섹션을 실제 구현 내용과 검증 출력으로 채운다.

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

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

- [x] `코드리뷰 결과`에 `PASS`, `WARN`, `FAIL` 중 하나의 판정과 검증된 `review_rework_count`, `evidence_integrity_failure`를 append한다.
- [x] 판정과 `차원별 평가`, Required/Suggested/Nit 분류가 서로 일치한다.
- [x] active `CODE_REVIEW-*-G??.md`를 `code_review_cloud_G10_0.log`로 아카이브한다.
- [x] active `PLAN-*-G??.md`를 `plan_cloud_G10_0.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-iop-agent-cli-runtime/01_common_runtime_node_bridge/`를 `agent-task/archive/YYYY/MM/m-iop-agent-cli-runtime/01_common_runtime_node_bridge/`로 이동하고 최종 archive 경로에서 이 체크리스트를 갱신한다.
- [ ] PASS이고 task group이 `m-<milestone-slug>`이면 런타임이 읽을 완료 이벤트 메타데이터를 보고하고, roadmap 수정이나 `update-roadmap` 직접 호출을 하지 않는다.
- [ ] PASS split 작업이면 이동 후 빈 active parent `agent-task/m-iop-agent-cli-runtime/`를 제거하거나, 남은 sibling/file이 있어 유지했다고 확인한다.
- [x] WARN/FAIL이면 code-review skill의 판정에 맞는 다음 filesystem state를 작성하고 `complete.log`를 작성하지 않는다.

## 계획 대비 변경 사항

- 계획의 별도 `ProfileSet` 예시 대신 기존 공통 config 계약인 `config.CLIConf`를 `agentprovider/cli.New` 입력으로 유지했다. host-neutral provider package가 이미 `packages/go/config`를 안전하게 소비하므로 동일 profile schema와 변환을 다시 만들지 않기 위한 선택이다.
- 구조 이동 뒤 현재 구현 문서가 삭제된 Node-owned runtime/CLI/terminal 경로를 계속 가리키지 않도록 `runtime/edge-node-execution` living spec과 node/platform-common project domain rule을 최소 동기화했다. 새 기능 범위는 추가하지 않았다.
- PLAN의 고정 Go 검증 외에 testing domain rule에 따라 `IOP_E2E_BIND_TIMEOUT=60 IOP_E2E_PROFILE=mock ./scripts/e2e-smoke.sh`를 추가 실행했다. 이 smoke는 임시 config와 실제 Edge/Node process, 공통 CLI provider를 사용해 등록, 동일 session 메시지 2회, background run, capabilities/transport/sessions/terminate-session을 검증했다.
- 최초 non-race CLI aggregate 실행에서 시간 기반 `TestCLIStartPartialRollbackWithMarkers`가 1회 timeout 실패했다. 단일 fresh 재실행은 PASS했고, 이후 전체 fresh 2회와 race 2회가 모두 PASS했다. 구현 오류를 숨기지 않기 위해 이 최초 결과를 기록한다.
- 실제 로그인된 `claude`/`antigravity`/`codex`/`opencode` 외부 profile 호출은 PLAN이 checkout 내부 Go test/search만 요구하고 외부 account/provider를 요구하지 않는다고 고정했으므로 실행하지 않았다. 이 task의 S01/S04 증거는 common conformance, 기존 CLI fixture suite, Node wire/config suite와 mock process cycle로 한정했다.

## 주요 설계 결정

- `packages/go/agentruntime`가 `Provider`, request/spec/event, typed `Failure` codec, terminal guard, terminal session과 lifecycle `Registry`를 소유하고 `packages/go/agentprovider/cli`가 기존 CLI process/emitter/session/status/quota 구현을 단일 source of truth로 소유한다. 두 package는 Node internal과 protobuf를 import하지 않는다.
- Node-owned `apps/node/internal/runtime`, `adapters/cli`, `terminal`, `adapters.Registry` compatibility alias를 남기지 않았다. Node의 mock/Ollama/OpenAI-compatible/vLLM provider도 같은 공통 contract/registry를 직접 소비한다.
- `apps/node/internal/node/runtime_bridge.go`만 protobuf ↔ common runtime 변환을 소유한다. typed failure는 legacy `RunEvent.error`에 JSON을 강제로 싣지 않고 기존 문자열을 우선하며, 문자열이 없을 때만 typed failure message를 fallback으로 사용해 wire 호환을 보존한다.
- 공통 `TerminalEmitter`가 provider terminal 이후 event를 차단하고 Node의 terminal-deferring sink도 duplicate terminal을 억제한다. Node admission ticket release 뒤 terminal flush 순서는 유지된다.
- durable failure codec은 versioned JSON envelope를 사용하고 알 수 없는 미래 code를 `unknown`으로 정규화하면서 원래 code를 metadata에 보존한다. cancellation/deadline boundary는 table test로 고정했다.
- 공통 CLI의 Codex app-server client identity와 quota snapshot source에서 Node 전용 이름을 제거하고 `iop-agent-runtime`으로 통일했다.

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

- 공통 contract가 provider run/stream/resume/cancel/status/quota/failure 의미를 고정하는가.
- Node bridge가 Edge-Node proto/config/event 의미와 terminal exactly-once를 보존하는가.
- Node internal에 provider/session/emitter 독립 구현이 남지 않고 common package가 Node internal을 import하지 않는가.
- public API 정상·경계 tests와 race/conformance evidence가 실제 변경에 대응하는가.

## 검증 결과

> 구현 에이전트는 각 명령을 그대로 실행하고 stdout/stderr를 아래에 원문으로 기록한다. 명령을 바꾸면 `계획 대비 변경 사항`에 이유와 대체 명령을 적는다.

### API-1 중간 검증

```bash
/config/.local/bin/go test -count=1 ./packages/go/agentruntime/...
```

_실제 stdout/stderr:_

```text
ok  	iop/packages/go/agentruntime	0.885s
```

### API-2 중간 검증

```bash
/config/.local/bin/go test -race -count=1 ./packages/go/agentruntime/... ./packages/go/agentprovider/cli/...
```

_실제 stdout/stderr:_

```text
ok  	iop/packages/go/agentruntime	2.020s
ok  	iop/packages/go/agentprovider/cli	32.439s
?   	iop/packages/go/agentprovider/cli/internal/testutil	[no test files]
ok  	iop/packages/go/agentprovider/cli/status	40.972s
```

### API-3 중간 검증

```bash
/config/.local/bin/go test -count=1 ./apps/node/internal/...
```

_실제 stdout/stderr:_

```text
ok  	iop/apps/node/internal/adapters	0.024s
?   	iop/apps/node/internal/adapters/mock	[no test files]
ok  	iop/apps/node/internal/adapters/ollama	0.038s
ok  	iop/apps/node/internal/adapters/openai_compat	0.164s
ok  	iop/apps/node/internal/adapters/vllm	0.145s
ok  	iop/apps/node/internal/bootstrap	1.417s
ok  	iop/apps/node/internal/node	0.863s
ok  	iop/apps/node/internal/router	0.511s
ok  	iop/apps/node/internal/store	0.060s
ok  	iop/apps/node/internal/transport	5.563s
```

### API-4 및 최종 검증

```bash
test "$(git rev-parse --show-toplevel)" = "/config/workspace/iop-s0"
/config/.local/bin/go test -count=1 ./packages/go/agentruntime/... ./packages/go/agentprovider/cli/...
/config/.local/bin/go test -count=1 ./apps/node/internal/...
/config/.local/bin/go test -race -count=1 ./packages/go/agentruntime/... ./packages/go/agentprovider/cli/... ./apps/node/internal/adapters/... ./apps/node/internal/router/...
/config/.local/bin/go test -count=1 ./packages/go/... ./apps/node/...
rg --sort path -n 'type (Adapter|RuntimeEvent|ExecutionSpec|RunRequest|Capabilities|EventSink|Registry) ' packages/go apps/node/internal
rg --sort path -n 'apps/node/internal/(runtime|adapters/cli|terminal)' packages/go || true
git diff --check
```

_실제 stdout/stderr:_

```text
# test "$(git rev-parse --show-toplevel)" = "/config/workspace/iop-s0"
# success; no stdout/stderr

# /config/.local/bin/go test -count=1 ./packages/go/agentruntime/... ./packages/go/agentprovider/cli/...
ok  	iop/packages/go/agentruntime	0.572s
ok  	iop/packages/go/agentprovider/cli	29.917s
?   	iop/packages/go/agentprovider/cli/internal/testutil	[no test files]
ok  	iop/packages/go/agentprovider/cli/status	39.852s

# /config/.local/bin/go test -count=1 ./apps/node/internal/...
ok  	iop/apps/node/internal/adapters	0.013s
?   	iop/apps/node/internal/adapters/mock	[no test files]
ok  	iop/apps/node/internal/adapters/ollama	0.016s
ok  	iop/apps/node/internal/adapters/openai_compat	0.133s
ok  	iop/apps/node/internal/adapters/vllm	0.140s
ok  	iop/apps/node/internal/bootstrap	1.377s
ok  	iop/apps/node/internal/node	0.823s
ok  	iop/apps/node/internal/router	0.504s
ok  	iop/apps/node/internal/store	0.052s
ok  	iop/apps/node/internal/transport	5.553s

# /config/.local/bin/go test -race -count=1 ./packages/go/agentruntime/... ./packages/go/agentprovider/cli/... ./apps/node/internal/adapters/... ./apps/node/internal/router/...
ok  	iop/packages/go/agentruntime	1.729s
ok  	iop/packages/go/agentprovider/cli	32.468s
?   	iop/packages/go/agentprovider/cli/internal/testutil	[no test files]
ok  	iop/packages/go/agentprovider/cli/status	40.885s
ok  	iop/apps/node/internal/adapters	1.037s
?   	iop/apps/node/internal/adapters/mock	[no test files]
ok  	iop/apps/node/internal/adapters/ollama	1.036s
ok  	iop/apps/node/internal/adapters/openai_compat	1.164s
ok  	iop/apps/node/internal/adapters/vllm	1.155s
ok  	iop/apps/node/internal/router	1.513s

# /config/.local/bin/go test -count=1 ./packages/go/... ./apps/node/...
ok  	iop/packages/go/agentprovider/cli	29.719s
?   	iop/packages/go/agentprovider/cli/internal/testutil	[no test files]
ok  	iop/packages/go/agentprovider/cli/status	39.935s
ok  	iop/packages/go/agentruntime	0.638s
ok  	iop/packages/go/audit	0.003s
?   	iop/packages/go/auth	[no test files]
ok  	iop/packages/go/config	0.090s
?   	iop/packages/go/events	[no test files]
ok  	iop/packages/go/hostsetup	0.014s
?   	iop/packages/go/jobs	[no test files]
?   	iop/packages/go/metadata	[no test files]
ok  	iop/packages/go/observability	0.034s
?   	iop/packages/go/policy	[no test files]
ok  	iop/packages/go/streamgate	0.891s
?   	iop/packages/go/version	[no test files]
ok  	iop/apps/node/cmd/node	0.015s
ok  	iop/apps/node/internal/adapters	0.014s
?   	iop/apps/node/internal/adapters/mock	[no test files]
ok  	iop/apps/node/internal/adapters/ollama	0.014s
ok  	iop/apps/node/internal/adapters/openai_compat	0.128s
ok  	iop/apps/node/internal/adapters/vllm	0.125s
ok  	iop/apps/node/internal/bootstrap	1.397s
ok  	iop/apps/node/internal/node	0.809s
ok  	iop/apps/node/internal/router	0.504s
ok  	iop/apps/node/internal/store	0.055s
ok  	iop/apps/node/internal/transport	5.573s

# rg --sort path -n 'type (Adapter|RuntimeEvent|ExecutionSpec|RunRequest|Capabilities|EventSink|Registry) ' packages/go apps/node/internal
packages/go/agentruntime/registry.go:20:type Registry struct {
packages/go/agentruntime/types.go:34:type ExecutionSpec struct {
packages/go/agentruntime/types.go:61:type RuntimeEvent struct {
packages/go/agentruntime/types.go:104:type Capabilities struct {
packages/go/agentruntime/types.go:116:type RunRequest struct {
packages/go/agentruntime/types.go:188:type EventSink interface {
apps/node/internal/adapters/openai_compat/adapter.go:17:type Adapter struct {

# rg --sort path -n 'apps/node/internal/(runtime|adapters/cli|terminal)' packages/go || true
# no matches

# git diff --check
# no stdout/stderr
```

추가 환경/실행 검증:

```text
$ command -v go
/config/.local/bin/go
$ readlink -f "$(command -v go)"
/config/opt/go/bin/go
$ go version
go version go1.26.2 linux/arm64
$ go env GOROOT
/config/opt/go

$ IOP_E2E_BIND_TIMEOUT=60 IOP_E2E_PROFILE=mock ./scripts/e2e-smoke.sh
[node0-evt] connected reason="registered"
[node0-capabilities] adapter=cli target=fake-cli session=default
[node0-transport] adapter=cli target=fake-cli session=default
[edge] sent run_id=manual-1785221609781763418 node=node0 adapter=cli target=fake-cli session=default background=false
[node0-evt] start run_id=manual-1785221609781763418
[node0-msg] IOP_E2E_BYE_SHORT
[node0-msg] IOP_E2E_BYE_SHORT_TAIL
[node0-evt] complete run_id=manual-1785221609781763418 detail="idle-timeout"
[edge] sent run_id=manual-1785221611008745044 node=node0 adapter=cli target=fake-cli session=default background=false
[node0-evt] start run_id=manual-1785221611008745044
[node0-msg] IOP_E2E_YES_SHORT
[node0-msg] IOP_E2E_YES_SHORT_TAIL
[node0-evt] complete run_id=manual-1785221611008745044 detail="idle-timeout"
[node0-evt] start run_id=manual-1785221612658368670 session=session2 background=true
[node0-msg] IOP_E2E_HELLO_BASIC
[node0-msg] IOP_E2E_HELLO_BASIC_TAIL
[node0-evt] complete run_id=manual-1785221612658368670 detail="idle-timeout"
[node0-sessions] adapter=cli target=fake-cli session=session2
sessions: 2
terminated session session2 node=node0
[e2e] Auxiliary smoke test PASSED.
```

---

> **[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 | Fail | concurrent `Emit`의 승인 순서와 실제 sink 전달 순서가 역전될 수 있고, Node deferring sink가 terminal 뒤 non-terminal event를 flush한다. |
| completeness | Pass | reviewer가 별도 Edge/Node two-process full-cycle 진단을 실행해 mock smoke와 구분된 통합 evidence를 보완했다. |
| test coverage | Fail | terminal concurrent ordering과 post-terminal non-terminal suppression 회귀 test가 없다. |
| API contract | Fail | terminal 이후 event를 host에 노출하지 않는 agent-runtime 계약을 위반한다. |
| code quality | Pass | 공통 package 추출, Node bridge 경계, 중복 구현 제거 구조는 계획과 일치한다. |
| implementation deviation | Pass | 계획 변경 사항은 근거가 있으며 reviewer 보완 검증까지 포함해 프로젝트 테스트 규칙과 정합하다. |
| verification trust | Pass | 구현 기록의 Go test/search/mock-smoke 출력은 fresh reviewer 실행 결과와 일치했다. |
| spec conformance | Fail | S04의 기존 provider behavior 보존과 terminal ordering 조건을 현재 구현/evidence로 닫을 수 없다. |

### 발견된 문제

- **Required** — `packages/go/agentruntime/emitter.go:34`, `apps/node/internal/node/runtime_sink.go:38`: `TerminalEmitter.Emit`은 terminal 상태를 lock 안에서 확정한 뒤 실제 sink 호출 전에 lock을 풀어, 먼저 승인된 delta의 sink 호출이 막힌 사이 뒤의 complete가 먼저 전달될 수 있다. reviewer의 deterministic reproducer는 `[complete, delta]`를 관측했다. 또한 `terminalDeferringSink`는 terminal 뒤 duplicate terminal만 버리고 late delta는 `deferred`에 추가해 flush하므로 `agent-contract/inner/agent-runtime.md:36,60`의 “terminal 이후 event 비노출” 계약을 직접 위반한다. accepted event 전달을 mutex 또는 ordered queue로 직렬화하고, Node sink는 terminal 관측 뒤 모든 event를 버리도록 수정한 뒤 concurrent ordering 및 post-terminal delta 회귀 test를 추가한다.

### 리뷰 중 직접 정리

- 이동된 CLI package 경로를 반영하도록 `scripts/readability_read_sets.json`과 `scripts/readability_baseline.json`의 stale 경로를 갱신했다.
- `agent-ops/rules/project/rules.md`의 central runtime 경로를 `packages/go/agentruntime`와 Node protobuf bridge 경계로 동기화했다.
- `IOP_DEV_RECONNECT_BIND_TIMEOUT=60 ./scripts/dev/edge-node-reconnect-diagnostic.sh`를 fresh 실행해 실제 `scripts/dev/edge.sh`/`scripts/dev/node.sh` 분리 프로세스, 3개 run의 Node==Edge payload 순서, terminal-last exactly-once, reconnect와 다섯 command 응답을 확인했다.
- `make readability-audit`의 missing-path configuration failure는 해소됐다. 남은 ratchet failure는 현재 worktree의 이 task 밖 변경에서 발생해 이번 판정 수에는 포함하지 않았다.

### 라우팅 신호

- `review_rework_count=1`
- `evidence_integrity_failure=false`

### 다음 단계

- plan 스킬 `prepare-follow-up`으로 위 Required를 닫는 최소 후속 계획을 새로 라우팅하고, 현재 plan/review 쌍을 로그로 아카이브한 뒤 새 active pair를 작성한다.
