<!-- task=m-control-plane-api-data-binding/01_runner_surface plan=0 tag=API -->

# Plan - API Runner Surface

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

구현 완료의 마지막 단계는 active `CODE_REVIEW-*-G??.md`의 구현 에이전트 소유 섹션을 실제 구현 내용과 검증 출력으로 채우는 것이다. 검증을 실행하고, 실제 출력과 설계 결정을 기록한 뒤 active 파일을 그대로 둔 채 리뷰 준비를 보고한다. 종결 처리, log rename, `complete.log` 작성, archive 이동은 code-review 스킬 전용이다. 구현 중 사용자만 결정할 항목, 사용자 소유 외부 환경, 범위 충돌이 발견되면 review stub의 `사용자 리뷰 요청` 섹션에 정확한 근거를 남기고 멈춘다. 구현 중 채팅으로 직접 질문하거나 `request_user_input`을 호출하지 않는다.

## 배경

현재 runner 화면은 `OtoConsoleShell`의 기본 empty surface만 보여준다. 작은 선행 작업에서 `OtoHttpCoreReadClient`가 `OtoConsoleConfig.serverHttpUrl`을 기준으로 `/api/v1/runners/{id}`와 `/api/v1/runners/{id}/status`를 읽도록 추가되었다. 이 계획은 Core API schema를 바꾸지 않고, 알려진 runner id가 있을 때 loading/empty/error/data 상태를 화면에 표시하는 runner surface를 만든다.

## 사용자 리뷰 요청 흐름

구현 중 차단 사항은 active review stub의 `사용자 리뷰 요청` 섹션에 기록한다. 해당 섹션은 `agent-ops/skills/common/_templates/implementation-user-review-request-section.md` 형식을 따른다. 직접 사용자 프롬프트는 금지되며, code-review가 요청 타당성을 검토하고 실제 `USER_REVIEW.md` 작성 여부를 결정한다.

## Roadmap Targets

- Milestone: `agent-roadmap/phase/control-plane-product-surface/milestones/control-plane-api-data-binding.md`
- Task ids:
  - `runner-surface`: Runners 화면이 Core runner registry 응답을 loading/empty/error/data 상태로 표시한다.
- Completion mode: check-on-pass

## 분석 결과

### 읽은 파일

- `agent-roadmap/current.md`
- `agent-roadmap/phase/control-plane-product-surface/PHASE.md`
- `agent-roadmap/phase/control-plane-product-surface/milestones/control-plane-api-data-binding.md`
- `agent-test/local/rules.md`
- `apps/client/lib/src/app/core_connection_client.dart`
- `apps/client/lib/src/app/oto_client_app.dart`
- `apps/client/test/widget_test.dart`
- `apps/client/pubspec.yaml`
- `apps/client/analysis_options.yaml`
- `packages/flutter/oto_console/lib/oto_console.dart`
- `packages/flutter/oto_console/lib/src/oto_console_contract.dart`
- `packages/flutter/oto_console/lib/src/oto_console_models.dart`
- `packages/flutter/oto_console/lib/src/oto_console_overview.dart`
- `packages/flutter/oto_console/lib/src/oto_console_shell.dart`
- `packages/flutter/oto_console/test/oto_console_test.dart`
- `packages/flutter/oto_console/pubspec.yaml`
- `packages/flutter/oto_console/analysis_options.yaml`
- `services/core/internal/httpserver/routes.go`
- `services/core/internal/httpserver/dto.go`
- `services/core/internal/httpserver/runner_handlers.go`
- `services/core/internal/httpserver/job_handlers.go`
- `services/core/internal/httpserver/execution_handlers.go`
- `services/core/internal/cicdstate/store.go`
- `services/core/internal/runnerregistry/registry.go`

### 테스트 환경 규칙

- `test_env=local`; `agent-test/local/rules.md`를 읽었다.
- matched agent-test profile: 없음. `apps/client/**`, `packages/flutter/oto_console/**`, `services/core/internal/**`에는 전용 domain rule/test profile이 아직 없다.
- 적용할 fallback source: `agent-test/local/rules.md`의 Monorepo Targets.
- 검증 명령: `cd packages/flutter/oto_console && flutter test`, `cd apps/client && flutter test`.
- 도구 확인: `command -v flutter`, `command -v dart`, `command -v go` 성공.

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

- runner surface contract와 loading/empty/error/data 렌더링은 기존 `packages/flutter/oto_console/test/oto_console_test.dart`에 없다. 새 widget test가 필요하다.
- app이 runner read 상태를 만들고 shell에 주입하는 흐름은 기존 `apps/client/test/widget_test.dart`에 없다. fake read client 기반 테스트가 필요하다.

### 심볼 참조

- renamed/removed symbols: none.

### 분할 판단

- split decision policy를 먼저 평가했다.
- shared task group: `agent-task/m-control-plane-api-data-binding/`.
- `01_runner_surface`: 독립 실행. runner 화면 contract와 client runner 상태 주입을 먼저 만든다.
- `02+01_execution_surface`: `01_runner_surface`의 공통 surface state contract 완료 후 진행한다. predecessor `01`: missing active/archive `complete.log`.
- `03+01,02_preview_smoke`: runner/execution surface 완료 후 원격 preview smoke를 수집한다. predecessor `01`: missing, predecessor `02`: missing.

### 범위 결정 근거

- Core HTTP API route/schema는 수정하지 않는다. `routes.go:63-127`은 id 기반 read path만 노출하며, 이 계획은 그 제약을 화면 empty/data 상태에 반영한다.
- jobs/executions/logs/artifacts 화면은 `02+01_execution_surface`에서 다룬다.
- remote preview screenshot/curl evidence는 `03+01,02_preview_smoke`에서 다룬다.

### 빌드 등급

- build: `local-G05`, review: `local-G05`. public widget contract와 app state wiring이 있지만 범위가 Flutter client/console로 제한되고 widget tests로 검증 가능하다.

## 구현 체크리스트

- [ ] `packages/flutter/oto_console`에 runner surface용 상태 contract와 `OtoRunnersSurface`를 추가하고 loading/empty/error/data widget test를 작성한다. 검증: `cd packages/flutter/oto_console && flutter test`에서 runners surface 테스트가 통과한다.
- [ ] `apps/client`가 `OtoCoreReadClient.fetchRunner`와 `fetchRunnerStatus` 결과를 runner surface 상태로 변환해 `OtoConsoleShell.runners`에 주입하고 fake read client widget test를 작성한다. 검증: `cd apps/client && flutter test`가 통과한다.
- [ ] CODE_REVIEW-*-G??.md의 구현 에이전트 소유 섹션을 실제 구현 내용과 검증 출력으로 채운다. 이 항목이 완료되기 전에는 구현이 완료된 것이 아니다.

### [API-1] Console runner surface contract

문제: `OtoConsoleShell`은 runners section에서 기본 empty surface만 렌더링한다.

Before:

```dart
// packages/flutter/oto_console/lib/src/oto_console_shell.dart:74
OtoConsoleSection.runners =>
  widget.runners ??
      OtoConsoleSectionSurface(
        icon: Icons.precision_manufacturing_outlined,
        title: 'Runners',
        emptyTitle: 'No runners connected',
        emptyMessage: 'Runner registry is empty.',
        theme: theme,
      ),
```

해결 방법: `oto_console_contract.dart`에 transport-free `OtoSurfaceLoadState`, `OtoSurfaceSnapshot<T>`, `OtoRunnerViewModel`을 추가한다. 새 `OtoRunnersSurface` 위젯은 `snapshot.state`에 따라 progress, empty panel, error panel, runner rows를 렌더링한다. `oto_console.dart`에서 새 surface를 export한다.

수정 파일 및 체크리스트:

- [ ] `packages/flutter/oto_console/lib/src/oto_console_contract.dart`에 상태 contract와 runner view model 추가.
- [ ] `packages/flutter/oto_console/lib/src/oto_runners_surface.dart` 생성.
- [ ] `packages/flutter/oto_console/lib/oto_console.dart` export 추가.
- [ ] `packages/flutter/oto_console/test/oto_console_test.dart`에 runner loading/empty/error/data 테스트 추가.

테스트 작성: 작성. `OtoRunnersSurface`를 직접 pump하고 상태별 텍스트, runner id/status/current execution 노출을 검증한다.

중간 검증:

```bash
cd packages/flutter/oto_console && flutter test
```

기대 결과: runners surface 테스트 포함 전체 통과.

### [API-2] Client runner read wiring

문제: `OtoClientApp`은 overview만 shell에 제공하고 runners slot은 비어 있다.

Before:

```dart
// apps/client/lib/src/app/oto_client_app.dart:111
home: OtoConsoleShell(
  config: widget.config,
  capabilities: widget.capabilities,
  themeAdapter: themeAdapter,
  overview: OtoConsoleOverview(
    config: widget.config,
    connection: _connection,
    onRefresh: _refreshCoreStatus,
    themeAdapter: themeAdapter,
  ),
),
```

해결 방법: `OtoClientApp`에 optional known runner id list와 `OtoCoreReadClient` 주입점을 추가한다. runner id가 없으면 empty snapshot을 만든다. id가 있으면 `fetchRunner`와 `fetchRunnerStatus`를 병렬 호출하고, adapter의 loading/empty/error/data 결과를 console surface snapshot으로 변환한다. `Refresh`는 Core status와 runner snapshot을 함께 갱신한다.

수정 파일 및 체크리스트:

- [ ] `apps/client/lib/src/app/oto_client_app.dart`에 read client, runner id seed, runner snapshot state 추가.
- [ ] `apps/client/lib/src/app/core_connection_client.dart`의 `OtoCoreReadClient`를 그대로 사용하고 Core API route를 추가하지 않는다.
- [ ] `apps/client/test/widget_test.dart`에 fake read client로 runner data/error/empty 주입 테스트 추가.

테스트 작성: 작성. fake read client가 `config.serverHttpUrl`을 받는지, runner 화면이 data/error/empty를 표시하는지 검증한다.

중간 검증:

```bash
cd apps/client && flutter test
```

기대 결과: runner wiring 테스트 포함 전체 통과.

## 수정 파일 요약

| 파일 | 항목 |
|------|------|
| `packages/flutter/oto_console/lib/src/oto_console_contract.dart` | API-1 |
| `packages/flutter/oto_console/lib/src/oto_runners_surface.dart` | API-1 |
| `packages/flutter/oto_console/lib/oto_console.dart` | API-1 |
| `packages/flutter/oto_console/test/oto_console_test.dart` | API-1 |
| `apps/client/lib/src/app/oto_client_app.dart` | API-2 |
| `apps/client/lib/src/app/core_connection_client.dart` | API-2 |
| `apps/client/test/widget_test.dart` | API-2 |

## 최종 검증

```bash
cd packages/flutter/oto_console && flutter test
cd apps/client && flutter test
```

기대 결과: 두 명령 모두 통과한다. Flutter test cache 여부는 Dart/Flutter test runner 기본 동작을 따른다.

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