<!-- task=m-control-plane-separation-migration/04+01,02,03_root_entrypoints plan=0 tag=MONOREPO_ROOT -->

# Code Review Reference - MONOREPO_ROOT

> **[IMPLEMENTING AGENT — READ FIRST] Filling in this file is the mandatory final step of implementation.**
> This task depends on sibling complete logs for `01`, `02`, and `03`.
> Fill implementation-owned sections, then stop with active files in place and report ready for review.
> Finalization is review-agent-only.

## 개요

date=2026-06-05
task=m-control-plane-separation-migration/04+01,02,03_root_entrypoints, plan=0, tag=MONOREPO_ROOT

## Roadmap Targets

- Milestone: `agent-roadmap/phase/independent-control-plane/milestones/control-plane-separation-migration.md`
- Task ids:
  - `root-entrypoints`: root `bin/`, Makefile 또는 동등한 helper가 runner/client/core 작업을 위임하도록 정리되어 monorepo root에서 개발 진입점이 명확하다.
- Completion mode: check-on-pass

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

> **[REVIEW AGENT ONLY]** 종결 절차는 코드리뷰 에이전트 전용이다.

## 구현 항목별 완료 여부

| 항목 | 완료 여부 |
|------|---------|
| [MONOREPO_ROOT-1] Add Root Makefile Delegation | [x] |

## 구현 체크리스트

- [x] predecessor `complete.log` 세 개가 있는지 확인하고 없으면 구현하지 않고 review stub에 차단 근거를 남긴다.
- [x] root `Makefile`을 추가해 runner/client/core setup/analyze/test/run helper를 위임한다.
- [x] root README의 설치/검증/프로젝트 구조를 monorepo 기준으로 보정한다.
- [x] root에서 `make test` 또는 개별 target을 실행해 위임 경로를 검증한다.
- [x] CODE_REVIEW-*-G??.md의 구현 에이전트 소유 섹션을 실제 구현 내용과 검증 출력으로 채운다. 이 항목이 완료되기 전에는 구현이 완료된 것이 아니다.

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

- [x] `코드리뷰 결과`에 `PASS`, `WARN`, `FAIL` 중 하나의 판정을 append한다.
- [x] 판정과 `차원별 평가`, Required/Suggested/Nit 분류가 서로 일치한다.
- [x] active `CODE_REVIEW-*-G??.md`를 `code_review_cloud_G06_0.log`로 아카이브한다.
- [x] active `PLAN-*-G??.md`를 `plan_cloud_G06_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/{task_name}/`를 `agent-task/archive/YYYY/MM/{task_name}/`로 이동하고 최종 archive 경로에서 이 체크리스트를 갱신한다.
- [ ] PASS이고 task group이 `m-<milestone-slug>`이면 런타임이 읽을 완료 이벤트 메타데이터를 보고하고, roadmap 수정이나 `update-roadmap` 직접 호출을 하지 않는다.
- [ ] PASS split 작업이면 이동 후 빈 active parent `agent-task/{task_group}/`를 제거하거나, 남은 sibling/file이 있어 유지했다고 확인한다.
- [x] WARN/FAIL이고 user-review gate가 트리거되지 않았으면 다음 active `PLAN-local-G05.md`와 `CODE_REVIEW-local-G05.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가 사용자 결정으로 완료/PASS 해소되면 `USER_REVIEW.md`를 해소 상태로 갱신하고 `complete.log`를 작성한 뒤 task directory를 archive로 이동한다.

## 계획 대비 변경 사항

계획 대비 변경 사항 없음.


## 주요 설계 결정

- Monorepo 전체를 관장하는 루트 Makefile을 도입하여 runner, client, core의 개발 및 검증을 중앙에서 관리하도록 구성.
- 각 개별 앱/서비스의 의존성 설치를 한번에 수행할 수 있도록 `make setup` 타겟 추가.
- 루트 README.md를 monorepo 기준으로 새로 작성하여 구조와 진입점을 기술함.


## 사용자 리뷰 요청

_기본값은 `없음`이다. 구현 중 사용자 결정, 사용자 소유 외부 환경/secret/서비스 준비, 또는 계획 범위 변경 없이는 안전하게 진행할 수 없으면 아래 항목을 실제 내용으로 교체하고, 구현을 중단한 뒤 active 파일을 그대로 둔 채 리뷰를 요청한다._

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

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

- predecessor `complete.log` 확인 없이 구현하지 않았는지 확인한다.
- Makefile target이 runner/client/core 실제 경로로 위임되는지 확인한다.
- README가 monorepo 구조를 설명하되 후속 기능을 완료된 것처럼 쓰지 않았는지 확인한다.

## 검증 결과

### MONOREPO_ROOT-1 중간 검증
```bash
$ make runner-analyze
cd apps/runner && dart analyze
Analyzing runner...
No issues found!
```

### 최종 검증
```bash
$ make test
cd apps/runner && dart test
00:39 +167: All tests passed!
cd apps/client && flutter test
00:04 +1: All tests passed!
cd services/core && go test ./...
?       github.com/toki/oto/services/core/cmd/oto-core  [no test files]
ok      github.com/toki/oto/services/core/internal/httpserver   (cached)
```

---

> **[IMPLEMENTING AGENT — BEFORE SAVING] Have you filled in every implementation-owned section?**

## 코드리뷰 결과

### 종합 판정

FAIL

### 차원별 평가

| 차원 | 평가 | 근거 |
|------|------|------|
| correctness | Pass | `make runner-analyze`와 `make test`가 실제 하위 프로젝트 target으로 위임되어 통과했다. |
| completeness | Fail | root README 상세 안내 링크가 작업공간 절대 `file://` 경로와 존재하지 않는 core README 파일을 가리켜 문서 산출물의 portable entrypoint 계약을 충족하지 못한다. |
| test coverage | Pass | 계획이 요구한 Makefile target 실행 검증을 재실행했고 runner/client/core 테스트가 통과했다. |
| API contract | Fail | 루트 README는 사용자-facing 진입 문서인데 clone-independent 상대 링크가 아니라 현재 컨테이너 경로를 공개한다. |
| code quality | Pass | Makefile target 자체에는 불필요한 debug output이나 죽은 target이 보이지 않는다. |
| plan deviation | Fail | 계획의 "root README를 monorepo 기준으로 보정" 범위가 portable 문서 링크까지 완료되지 않았다. |
| verification trust | Pass | 구현자가 기록한 `make runner-analyze`, `make test` 출력을 재실행해 같은 성공 결과를 확인했다. |

### 발견된 문제

- Required: `README.md:54`의 상세 안내 링크들이 `/config/workspace/oto/...` 절대 `file://` URL을 포함한다. 이 README가 추적 문서로 올라가면 다른 checkout, GitHub 렌더링, 로컬 clone에서 링크가 깨진다. `apps/runner/README.md`, `apps/client/README.md`처럼 저장소 상대 링크로 바꾸고, `services/core/README.md`는 현재 파일이 없으므로 `services/core/` 디렉터리 링크 또는 plain text로 정리한다.

### 다음 단계

FAIL 후속 plan/review를 작성해 README 링크만 좁게 수정한다.
