diff --git a/Makefile b/Makefile index 8181383..5a9bd71 100644 --- a/Makefile +++ b/Makefile @@ -1,4 +1,4 @@ -.PHONY: all build build-local build-edge build-edge-host build-node build-node-target build-node-targets pack-node-target pack-edge archive-edge tidy test test-e2e test-control-plane-edge-wire test-openai-ollama proto proto-dart client-test client-build-web clean +.PHONY: all build build-local build-edge build-edge-host build-node build-node-target build-node-targets pack-node-target pack-edge archive-edge tidy test test-e2e test-control-plane-edge-wire test-openai-ollama test-oto-registration-online-smoke proto proto-dart client-test client-build-web clean GOFLAGS ?= -trimpath BUILD_DIR ?= build @@ -86,6 +86,15 @@ test-control-plane-edge-wire: test-openai-ollama: ./scripts/e2e-openai-ollama.sh +# OTO oto-agent registration online smoke. Runs the OTO Dart proto-socket +# client against this iop checkout, asserting registration accepted plus the +# first-heartbeat/online transition. Used as OTO unlock handoff evidence. +OTO_REPO ?= ../oto +test-oto-registration-online-smoke: + @echo "==> OTO registration online smoke" + @echo "command: cd $(OTO_REPO) && IOP_REPO_ROOT=$(CURDIR) dart test test/oto_iop_connection_smoke_test.dart" + cd $(OTO_REPO) && IOP_REPO_ROOT=$(CURDIR) dart test test/oto_iop_connection_smoke_test.dart + # Requires: protoc + protoc-gen-go (go install google.golang.org/protobuf/cmd/protoc-gen-go@latest) proto: protoc \ diff --git a/agent-roadmap/phase/automation-runtime-bridge/milestones/oto-bootstrap-command-issue.md b/agent-roadmap/archive/phase/automation-runtime-bridge/milestones/oto-bootstrap-command-issue.md similarity index 68% rename from agent-roadmap/phase/automation-runtime-bridge/milestones/oto-bootstrap-command-issue.md rename to agent-roadmap/archive/phase/automation-runtime-bridge/milestones/oto-bootstrap-command-issue.md index 4eee5ee..cf7c2f1 100644 --- a/agent-roadmap/phase/automation-runtime-bridge/milestones/oto-bootstrap-command-issue.md +++ b/agent-roadmap/archive/phase/automation-runtime-bridge/milestones/oto-bootstrap-command-issue.md @@ -12,7 +12,7 @@ Edge는 OTO bootstrap script를 구현하지 않고, OTO가 소유한 script 인 ## 상태 -[계획] +[완료] ## 구현 잠금 @@ -33,21 +33,24 @@ Edge는 OTO bootstrap script를 구현하지 않고, OTO가 소유한 script 인 Edge-local field UX에서 OTO agent를 생성하고 bootstrap command를 출력한다. -- [ ] [agent-register-cmd] `iop-edge`가 OTO agent record를 만들거나 갱신하는 command 표면을 제공한다. -- [ ] [node-interface-parity] OTO agent command/config 표면이 기존 `iop-node` bootstrap의 필드명, flag 의미, 출력 형식, 진단 흐름을 최대한 유지한다. -- [ ] [bootstrap-output] command 출력은 한 줄 bootstrap command만 안정적으로 제공하고 token을 불필요하게 재출력하지 않는다. -- [ ] [oto-args] 출력 command가 OTO bootstrap 필수 인자 `--edge-url`, `--agent-id`, `--enrollment-token`, `--release-base-url`을 포함한다. -- [ ] [config-roundtrip] 생성된 edge config를 `config check`와 runtime load가 읽을 수 있다. 검증: `go test ./apps/edge/cmd/edge ./packages/go/config` +- [x] [agent-register-cmd] `iop-edge`가 OTO agent record를 만들거나 갱신하는 command 표면을 제공한다. +- [x] [node-interface-parity] OTO agent command/config 표면이 기존 `iop-node` bootstrap의 필드명, flag 의미, 출력 형식, 진단 흐름을 최대한 유지한다. +- [x] [bootstrap-output] command 출력은 한 줄 bootstrap command만 안정적으로 제공하고 token을 불필요하게 재출력하지 않는다. +- [x] [oto-args] 출력 command가 OTO bootstrap 필수 인자 `--edge-url`, `--agent-id`, `--enrollment-token`, `--release-base-url`을 포함한다. +- [x] [config-roundtrip] 생성된 edge config를 `config check`와 runtime load가 읽을 수 있다. 검증: `go test ./apps/edge/cmd/edge ./packages/go/config` ## 완료 리뷰 -- 상태: 없음 -- 요청일: 없음 -- 완료 근거: 모든 기능 Task가 아직 충족되지 않았다. +- 상태: 승인됨 +- 요청일: 2026-06-03 +- 완료 근거: + - `iop-edge agent register`가 OTO agent record를 upsert하고 `agent_kind: oto-agent`, alias, token, OTO bootstrap URL, release URL을 edge config에 반영한다. + - 출력 command는 OTO bootstrap 필수 인자를 포함한 한 줄 command로 검증되며, 기본 HTTPS 우선과 명시적 local HTTP 허용 flag가 분리되어 있다. + - 검증 명령 `go test -count=1 ./apps/edge/cmd/edge ./packages/go/config` 통과, 임시 config CLI smoke의 `config init -> agent register -> config check` 통과. - 리뷰 필요: - - [ ] 사용자가 완료 결과를 확인했다 - - [ ] archive 이동을 승인했다 -- 리뷰 코멘트: 없음 + - [x] 사용자가 완료 결과를 확인했다 + - [x] archive 이동을 승인했다 +- 리뷰 코멘트: 2026-06-03 사용자 요청으로 모든 기능 Task 완료 상태를 확인하고 Milestone을 완료/archive 처리한다. ## 범위 제외 @@ -64,4 +67,4 @@ Edge-local field UX에서 OTO agent를 생성하고 bootstrap command를 출력 - 표준선(선택): MVP에서는 one-time enrollment token을 사용하고 장기 credential 재발급은 후속으로 둔다. - 선행 작업: OTO Agent Registry 구현 - 후속 작업: OTO Registration Online Smoke -- 확인 필요: local smoke에서 사용할 OTO bootstrap script URL과 release base URL 기본값 +- 확인 결과: local smoke용 OTO bootstrap script URL과 release base URL은 기본값을 두지 않고, command flag 또는 `bootstrap.*` config로 명시 입력한다. diff --git a/agent-roadmap/phase/automation-runtime-bridge/PHASE.md b/agent-roadmap/phase/automation-runtime-bridge/PHASE.md index 08f53ee..c8b234c 100644 --- a/agent-roadmap/phase/automation-runtime-bridge/PHASE.md +++ b/agent-roadmap/phase/automation-runtime-bridge/PHASE.md @@ -26,21 +26,21 @@ CLI 실행, specialized agent 등록, bootstrap/enrollment, 원격 터미널 브 - 경로: `agent-roadmap/archive/phase/automation-runtime-bridge/milestones/oto-agent-registry-implementation.md` - 요약: Edge-side `generic-node`/`oto-agent` 구분을 config/store, registry/transport, service snapshot에 반영하고 검증을 완료했다. +- [완료] OTO Bootstrap Command 발급 + - 경로: `agent-roadmap/archive/phase/automation-runtime-bridge/milestones/oto-bootstrap-command-issue.md` + - 요약: `iop-edge agent register`가 OTO agent record와 OTO Linux bootstrap script 인자를 포함한 one-line command를 발급하고 config roundtrip 검증을 완료했다. + - [검토중] Specialized Agent proto-socket 연결 기반 - 경로: `agent-roadmap/phase/automation-runtime-bridge/milestones/specialized-agent-proto-socket-foundation.md` - 요약: OTO의 node-compatible proto-socket registration smoke를 바탕으로 Edge의 specialized agent kind, registry/config, message boundary, reconnect 정책과 runtime 경로 구분을 완료 후보로 정리했다. -- [계획] OTO Bootstrap Command 발급 - - 경로: `agent-roadmap/phase/automation-runtime-bridge/milestones/oto-bootstrap-command-issue.md` - - 요약: `iop-edge`가 OTO agent record를 만들고 OTO Linux bootstrap script가 소비할 one-line command를 발급하는 CLI/config 표면을 구현한다. - -- [계획] OTO Registration Online Smoke +- [진행중] OTO Registration Online Smoke - 경로: `agent-roadmap/phase/automation-runtime-bridge/milestones/oto-registration-online-smoke.md` - 요약: OTO Dart `proto_socket` client와 iop Edge를 함께 띄워 registration accepted, 최초 heartbeat, online/failed 상태 구분을 검증한다. -- [진행중] OTO Registration Unlock Handoff +- [계획] OTO Registration Unlock Handoff - 경로: `agent-roadmap/phase/automation-runtime-bridge/milestones/oto-registration-unlock-handoff.md` - - 요약: iop2oto의 OTO registry, bootstrap command, online smoke 결과를 OTO `oto-agent` 등록 흐름 잠금 해제 evidence로 묶어 넘긴다. + - 요약: OTO bootstrap command와 online smoke가 완료된 뒤 iop2oto evidence를 OTO `oto-agent` 등록 흐름 잠금 해제 근거로 묶어 넘긴다. - [계획] OTO Agent Message Boundary - 경로: `agent-roadmap/phase/automation-runtime-bridge/milestones/oto-agent-message-boundary.md` diff --git a/agent-roadmap/phase/automation-runtime-bridge/milestones/oto-registration-online-smoke.md b/agent-roadmap/phase/automation-runtime-bridge/milestones/oto-registration-online-smoke.md index 3f2862f..67255c0 100644 --- a/agent-roadmap/phase/automation-runtime-bridge/milestones/oto-registration-online-smoke.md +++ b/agent-roadmap/phase/automation-runtime-bridge/milestones/oto-registration-online-smoke.md @@ -12,7 +12,7 @@ iop Edge와 OTO `oto-agent`를 실제로 함께 띄워 Edge 직접 outbound 등 ## 상태 -[계획] +[진행중] ## 구현 잠금 @@ -32,16 +32,16 @@ iop Edge와 OTO `oto-agent`를 실제로 함께 띄워 Edge 직접 outbound 등 Edge와 OTO가 같은 기준으로 online/failed 상태를 판정한다. -- [ ] [accepted-state] OTO agent registration accepted 상태가 generic node와 구분되어 기록된다. +- [x] [accepted-state] OTO agent registration accepted 상태가 generic node와 구분되어 기록된다. - [ ] [heartbeat-online] 최초 heartbeat 도착 후 online으로 전환된다. 검증: OTO Dart smoke 또는 통합 smoke에서 registration accepted 뒤 heartbeat 도착까지 확인한다. -- [ ] [failure-reason] unknown token, duplicate connection, heartbeat timeout이 서로 다른 실패 reason으로 확인된다. +- [x] [failure-reason] unknown token, duplicate connection, heartbeat timeout이 서로 다른 실패 reason으로 확인된다. - [ ] [unlock-evidence] OTO `oto-agent` 등록 흐름 잠금 해제에 사용할 smoke 결과와 실행 명령이 문서 또는 test log에 남는다. ## 완료 리뷰 - 상태: 없음 - 요청일: 없음 -- 완료 근거: 모든 기능 Task가 아직 충족되지 않았다. +- 완료 근거: `01_iop_edge_lifecycle` PASS로 `accepted-state`, `failure-reason`은 충족되었고, `heartbeat-online`, `unlock-evidence`가 아직 남아 있다. - 리뷰 필요: - [ ] 사용자가 완료 결과를 확인했다 - [ ] archive 이동을 승인했다 @@ -61,4 +61,5 @@ Edge와 OTO가 같은 기준으로 online/failed 상태를 판정한다. - 표준선(선택): heartbeat timeout은 transport close reason과 lifecycle event metadata로 남긴다. - 선행 작업: OTO Agent Registry 구현, OTO Bootstrap Command 발급 - 후속 작업: OTO Agent Message Boundary, OTO repo `oto-agent` 등록 흐름 잠금 해제 검토 -- 확인 필요: smoke 실행 시 iop Edge와 OTO를 같은 host에서 띄울지, split host로 띄울지 +- 확인 결과: local smoke의 기본 기준은 같은 host에서 iop Edge와 OTO를 띄우는 흐름으로 두고, split host는 주소/포트 override가 필요한 후속 또는 수동 검증 경로로 분리한다. +- 실행 계획: 큰 작업은 `agent-task/m-oto-registration-online-smoke/01_iop_edge_lifecycle`, `agent-task/m-oto-registration-online-smoke/02+01_oto_agent_session`, `agent-task/m-oto-registration-online-smoke/03+01,02_smoke_evidence`로 분리해 plan/code-review 기준을 작성했다. diff --git a/agent-roadmap/phase/automation-runtime-bridge/milestones/oto-registration-unlock-handoff.md b/agent-roadmap/phase/automation-runtime-bridge/milestones/oto-registration-unlock-handoff.md index 01fefd6..34195cb 100644 --- a/agent-roadmap/phase/automation-runtime-bridge/milestones/oto-registration-unlock-handoff.md +++ b/agent-roadmap/phase/automation-runtime-bridge/milestones/oto-registration-unlock-handoff.md @@ -12,7 +12,7 @@ iop2oto의 OTO registry, bootstrap command, registration online smoke 결과를 ## 상태 -[진행중] +[계획] ## 구현 잠금 @@ -74,9 +74,9 @@ ORU02 handoff plan의 upstream completion gate 평가 결과다. 상태는 각 - predecessor `01_node_interface_parity`: PASS. `agent-task/archive/2026/06/m-oto-registration-unlock-handoff/01_node_interface_parity/complete.log` 확인 결과 `node-interface-parity`가 PASS 완료되었고, `rg --sort path -n 'node-interface-parity|iop-node|agent_kind' ...`와 `git diff --check` 검증이 통과했다. - `OTO Agent Registry 구현`: [완료] — archive 문서 `agent-roadmap/archive/phase/automation-runtime-bridge/milestones/oto-agent-registry-implementation.md`로 이동되었고, 기능 Task(`agent-kind-model`, `compat-node`, `connected-entry`, `duplicate-policy`)와 registry task group의 3개 `complete.log`가 모두 PASS evidence를 갖는다. -- `OTO Bootstrap Command 발급`: [계획] — 구현/완료 evidence 없음. -- `OTO Registration Online Smoke`: [계획] — registration accepted/최초 heartbeat online PASS 로그 또는 smoke evidence 없음. -- **upstream-complete gate: 미충족.** `OTO Bootstrap Command 발급`과 `OTO Registration Online Smoke`가 아직 [검토중] 또는 [완료]가 아니므로 online-evidence(accepted/heartbeat)와 failure-retry(unknown token, duplicate connection, heartbeat timeout, bootstrap/registration 실패 reason과 retry 기준) evidence를 OTO `registration-state`로 옮길 수 있는 형태로 수집할 수 없다. 따라서 `upstream-complete`, `online-evidence`, `failure-retry`, `lock-sync-ready`는 아직 `[x]` 처리하지 않는다. 이 gate는 사용자 결정이 아니라 선행 Milestone 구현이 완료되어야 닫히는 repo-owned evidence gap이다. +- `OTO Bootstrap Command 발급`: [완료] — archive 문서 `agent-roadmap/archive/phase/automation-runtime-bridge/milestones/oto-bootstrap-command-issue.md`로 이동되었고, `agent register` command, one-line bootstrap output, OTO args, config roundtrip evidence가 PASS로 남아 있다. +- `OTO Registration Online Smoke`: [진행중] — 현재 실행 가능한 다음 Milestone이며 registration accepted/최초 heartbeat online PASS 로그 또는 smoke evidence는 아직 없다. +- **upstream-complete gate: 미충족.** `OTO Registration Online Smoke`가 아직 [검토중] 또는 [완료]가 아니므로 online-evidence(accepted/heartbeat)와 failure-retry(unknown token, duplicate connection, heartbeat timeout, bootstrap/registration 실패 reason과 retry 기준) evidence를 OTO `registration-state`로 옮길 수 있는 형태로 수집할 수 없다. 따라서 `upstream-complete`, `online-evidence`, `failure-retry`, `lock-sync-ready`는 아직 `[x]` 처리하지 않는다. 이 gate는 사용자 결정이 아니라 선행 Milestone 구현이 완료되어야 닫히는 repo-owned evidence gap이다. ### lock sync readiness diff --git a/agent-task/archive/2026/06/m-oto-bootstrap-command-issue/01_oto_bootstrap_config/code_review_local_G05_0.log b/agent-task/archive/2026/06/m-oto-bootstrap-command-issue/01_oto_bootstrap_config/code_review_local_G05_0.log new file mode 100644 index 0000000..38ab4ac --- /dev/null +++ b/agent-task/archive/2026/06/m-oto-bootstrap-command-issue/01_oto_bootstrap_config/code_review_local_G05_0.log @@ -0,0 +1,188 @@ + + +# Code Review Reference - CONFIG + +> **[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 user-only decision, user-owned external environment prerequisite, or scope conflict, fill `사용자 리뷰 요청` with evidence and stop with active files in place; code-review decides whether to write `USER_REVIEW.md`. Evidence gaps that a follow-up agent can close by rerunning commands or collecting artifacts are normal follow-up issues, not user-review blockers by themselves. +> 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-06-03 +task=m-oto-bootstrap-command-issue/01_oto_bootstrap_config, plan=0, tag=CONFIG + +## 이 파일을 읽는 리뷰 에이전트에게 + +> **[REVIEW AGENT ONLY]** 아래 종결 절차는 코드리뷰 에이전트 전용이다. 구현 에이전트는 이 섹션을 실행하지 않는다. + +각 항목의 구현을 실제 소스 파일과 대조하고, `검증 결과` 섹션의 출력이 코드와 일치하는지 확인하세요. +리뷰 완료는 아래 순서까지 끝난 상태를 의미합니다. + +1. 판정을 append한다. +2. `CODE_REVIEW-{review_lane}-GNN.md` -> `code_review_{review_lane}_GNN_N.log`, `PLAN-{build_lane}-GNN.md` -> `plan_{build_lane}_GNN_M.log`로 아카이브한다. +3. PASS이면 `complete.log` 작성 후 active task 디렉터리를 `agent-task/archive/YYYY/MM/{task_name}/`로 이동한다. WARN/FAIL이면 user-review gate를 확인한 뒤 다음 active plan/review 파일 또는 `USER_REVIEW.md`를 작성한다. `USER_REVIEW.md`가 사용자 결정으로 완료/PASS 해소되면 code-review가 `USER_REVIEW.md`를 해소 상태로 갱신하고 `complete.log` 작성 후 archive 이동한다. +4. PASS이고 task group이 `m-`이면 완료 이벤트 메타데이터를 보고한다. roadmap 상태 체크와 `update-roadmap` 호출은 런타임 책임이다. +5. 적용 가능한 `코드리뷰 전용 체크리스트` 항목을 최종 `.log` 위치에서 체크한 뒤 보고한다. + +--- + +## 구현 항목별 완료 여부 + +| 항목 | 완료 여부 | +|------|---------| +| [CONFIG-1] Edge bootstrap config에 OTO script/release URL 필드를 추가하고 load/default/example 테스트를 작성한다. | [x] | +| [CONFIG-2] YAML patch helper가 `artifact_base_url`, `oto_bootstrap_script_url`, `oto_release_base_url`을 같은 방식으로 보존/갱신하게 만든다. | [x] | + +## 구현 체크리스트 + +- [x] [CONFIG-1] Edge bootstrap config에 OTO script/release URL 필드를 추가하고 load/default/example 테스트를 작성한다. +- [x] [CONFIG-2] YAML patch helper가 `artifact_base_url`, `oto_bootstrap_script_url`, `oto_release_base_url`을 같은 방식으로 보존/갱신하게 만든다. +- [x] [CONFIG-3] `go test -count=1 ./packages/go/config ./apps/edge/cmd/edge`를 실행한다. +- [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_{review_lane}_GNN_N.log`로 아카이브한다. +- [x] active `PLAN-*-G??.md`를 `plan_{build_lane}_GNN_M.log`로 아카이브한다. +- [x] `.gitignore`의 Agent-Ops 관리 block이 `agent-task/**/*.md`와 `agent-task/**/*.log`를 unignore하여 plan/review/archive 산출물이 추적 가능한지 확인한다. +- [x] PASS이면 `agent-ops/skills/common/code-review/templates/complete-log-template.md` 기준으로 `complete.log`를 작성하고 active `.md` 파일을 남기지 않는다. +- [x] PASS이면 active task 디렉터리 `agent-task/m-oto-bootstrap-command-issue/01_oto_bootstrap_config/`를 `agent-task/archive/YYYY/MM/m-oto-bootstrap-command-issue/01_oto_bootstrap_config/`로 이동하고 최종 archive 경로에서 이 체크리스트를 갱신한다. +- [x] PASS이고 task group이 `m-oto-bootstrap-command-issue`이면 런타임이 읽을 완료 이벤트 메타데이터를 보고하고, roadmap 수정이나 `update-roadmap` 직접 호출을 하지 않는다. +- [x] PASS split 작업이면 이동 후 빈 active parent `agent-task/m-oto-bootstrap-command-issue/`를 제거하거나, 남은 sibling/file이 있어 유지했다고 확인한다. +- [ ] WARN/FAIL이고 user-review gate가 트리거되지 않았으면 다음 active `PLAN-{build_lane}-GNN.md`와 `CODE_REVIEW-{review_lane}-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가 사용자 결정으로 완료/PASS 해소되면 `USER_REVIEW.md`를 해소 상태로 갱신하고 `complete.log`를 작성한 뒤 task directory를 archive로 이동한다. + +## 계획 대비 변경 사항 + +없음. + +## 주요 설계 결정 + +- `patchBootstrapStringFields` 헬퍼 함수를 추가하여 `artifact_base_url`, `oto_bootstrap_script_url`, `oto_release_base_url`을 순서대로 일관되고 결정론적(deterministic order)인 방식으로 YAML 노드에 패치하도록 설계했습니다. 이를 통해 map의 순서 무작위성으로 인해 YAML이 불필요하게 변경되는 현상을 막았습니다. + +## 사용자 리뷰 요청 + +- 상태: 없음 +- 사유 유형: 없음 +- 결정 필요: 없음 +- 차단 근거: 없음 +- 실행한 검증/명령: 없음 +- 자동 후속 불가 이유: 없음 +- 재개 조건: 없음 + +## 리뷰어를 위한 체크포인트 + +- `bootstrap.oto_bootstrap_script_url`과 `bootstrap.oto_release_base_url`이 loader/default/example/test에 모두 반영됐는가. +- YAML patch helper가 기존 `artifact_base_url` 동작을 깨지 않고 새 OTO 필드를 보존형으로 추가/갱신하는가. +- 이 subtask가 `agent register` command 구현으로 scope를 넓히지 않았는가. + +## 검증 결과 + +### CONFIG-1 중간 검증 +```bash +$ go test -count=1 ./packages/go/config +ok iop/packages/go/config 0.019s +``` + +### CONFIG-2 중간 검증 +```bash +$ go test -count=1 ./apps/edge/cmd/edge +ok iop/apps/edge/cmd/edge 0.054s +``` + +### 최종 검증 +```bash +$ go test -count=1 ./packages/go/config ./apps/edge/cmd/edge +ok iop/packages/go/config 0.018s +ok iop/apps/edge/cmd/edge 0.078s +``` + +```bash +$ go test -count=1 ./packages/go/... ./apps/edge/... +? iop/packages/go/auth [no test files] +ok iop/packages/go/config 0.021s +? iop/packages/go/events [no test files] +? iop/packages/go/hostsetup [no test files] +? iop/packages/go/jobs [no test files] +? iop/packages/go/metadata [no test files] +? iop/packages/go/observability [no test files] +? iop/packages/go/policy [no test files] +? iop/packages/go/version [no test files] +ok iop/apps/edge/cmd/edge 0.054s +? iop/apps/edge/internal/bootstrap [no test files] +? iop/apps/edge/internal/controlplane [no test files] +? iop/apps/edge/internal/events [no test files] +? iop/apps/edge/internal/input [no test files] +ok iop/apps/edge/internal/node 0.015s +? iop/apps/edge/internal/openai [no test files] +? iop/apps/edge/internal/opsconsole [no test files] +? iop/apps/edge/internal/service [no test files] +ok iop/apps/edge/internal/transport 0.134s +``` + +--- + +> **[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. + +## 코드리뷰 결과 + +- 종합 판정: PASS +- 차원별 평가: + - correctness: Pass + - completeness: Pass + - test coverage: Pass + - API contract: Pass + - code quality: Pass + - plan deviation: Pass + - verification trust: Pass +- 발견된 문제: 없음 +- 다음 단계: PASS 종결. `complete.log` 작성 후 task archive로 이동한다. + +리뷰 검증: + +```bash +$ go test -count=1 ./packages/go/config ./apps/edge/cmd/edge +ok iop/packages/go/config 0.049s +ok iop/apps/edge/cmd/edge 0.093s +``` + +```bash +$ go test -count=1 ./apps/edge/internal/controlplane -run TestConnectorStopClosesConnection -v +=== RUN TestConnectorStopClosesConnection +--- PASS: TestConnectorStopClosesConnection (0.01s) +PASS +ok iop/apps/edge/internal/controlplane 0.018s +``` + +```bash +$ go test -count=1 ./packages/go/... ./apps/edge/... +ok iop/packages/go/audit 0.005s +ok iop/packages/go/config 0.016s +ok iop/packages/go/hostsetup 0.006s +ok iop/packages/go/observability 0.014s +ok iop/apps/edge/cmd/edge 0.057s +ok iop/apps/edge/internal/bootstrap 0.032s +ok iop/apps/edge/internal/controlplane 4.451s +ok iop/apps/edge/internal/events 0.009s +ok iop/apps/edge/internal/input 0.008s +ok iop/apps/edge/internal/input/a2a 0.008s +ok iop/apps/edge/internal/node 0.015s +ok iop/apps/edge/internal/openai 1.516s +ok iop/apps/edge/internal/opsconsole 0.008s +ok iop/apps/edge/internal/service 0.007s +ok iop/apps/edge/internal/transport 0.018s +``` + +참고: 첫 `go test -count=1 ./packages/go/... ./apps/edge/...` 재실행에서 이번 변경과 무관한 `apps/edge/internal/controlplane`의 `TestConnectorStopClosesConnection`이 1회 실패했으나, 해당 단일 테스트와 전체 범위 재실행이 이어서 통과했다. diff --git a/agent-task/archive/2026/06/m-oto-bootstrap-command-issue/01_oto_bootstrap_config/complete.log b/agent-task/archive/2026/06/m-oto-bootstrap-command-issue/01_oto_bootstrap_config/complete.log new file mode 100644 index 0000000..974d9c7 --- /dev/null +++ b/agent-task/archive/2026/06/m-oto-bootstrap-command-issue/01_oto_bootstrap_config/complete.log @@ -0,0 +1,36 @@ +# Complete - m-oto-bootstrap-command-issue/01_oto_bootstrap_config + +## 완료 일시 + +2026-06-03 + +## 요약 + +CONFIG plan 0/code-review 0 loop를 PASS로 종결했다. Edge bootstrap OTO URL config 계약과 YAML patch helper 변경이 계획 범위와 검증 기준을 충족했다. + +## 루프 이력 + +| Plan | Review | Verdict | 메모 | +|------|--------|---------|------| +| `plan_local_G05_0.log` | `code_review_local_G05_0.log` | PASS | Required/Suggested 이슈 없음 | + +## 구현/정리 내용 + +- `EdgeBootstrapConf`에 `oto_bootstrap_script_url`, `oto_release_base_url` 필드와 empty default를 추가했다. +- `configs/edge.yaml` bootstrap 예시에 OTO bootstrap script/release base URL 필드를 추가했다. +- `patchBootstrapStringFields` helper로 `artifact_base_url`, `oto_bootstrap_script_url`, `oto_release_base_url` YAML patch 경로를 결정론적으로 통합했다. +- config load/default 테스트와 Edge YAML patch 테스트를 추가했다. + +## 최종 검증 + +- `go test -count=1 ./packages/go/config ./apps/edge/cmd/edge` - PASS; `iop/packages/go/config`, `iop/apps/edge/cmd/edge` 모두 통과. +- `go test -count=1 ./apps/edge/internal/controlplane -run TestConnectorStopClosesConnection -v` - PASS; 첫 전체 범위 실행에서 1회 실패한 무관 control-plane 테스트를 단독 재확인. +- `go test -count=1 ./packages/go/... ./apps/edge/...` - PASS; packages/go 및 apps/edge 범위 재실행 통과. + +## 잔여 Nit + +- 없음 + +## 후속 작업 + +- 없음 diff --git a/agent-task/archive/2026/06/m-oto-bootstrap-command-issue/01_oto_bootstrap_config/plan_local_G05_0.log b/agent-task/archive/2026/06/m-oto-bootstrap-command-issue/01_oto_bootstrap_config/plan_local_G05_0.log new file mode 100644 index 0000000..86a28a1 --- /dev/null +++ b/agent-task/archive/2026/06/m-oto-bootstrap-command-issue/01_oto_bootstrap_config/plan_local_G05_0.log @@ -0,0 +1,181 @@ + + +# Plan - CONFIG + +## 이 파일을 읽는 구현 에이전트에게 + +`CODE_REVIEW-*-G??.md`의 구현 에이전트 소유 섹션 작성은 필수다. 구현 후 검증을 실행하고 실제 구현 내용과 stdout/stderr를 review stub에 기록한 뒤 active 파일을 그대로 두고 리뷰 준비 상태를 보고한다. 최종 판정, log rename, `complete.log`, archive 이동은 code-review 전용이다. 사용자 결정, 사용자 소유 외부 환경, scope 충돌로 막히면 review stub의 `사용자 리뷰 요청` 섹션에 정확한 결정과 증거를 남기고 멈춘다. 후속 에이전트가 재실행이나 증거 수집으로 해소할 수 있는 검증 공백은 사용자 리뷰 요청이 아니다. + +## 배경 + +현재 `node register`는 Node artifact URL만 `bootstrap.artifact_base_url`에 보존한다. OTO bootstrap command는 OTO 소유 script URL과 release base URL을 함께 소비해야 하므로 Edge config가 두 값을 읽고 YAML patch가 보존할 수 있어야 한다. 이 선행 작업은 command 구현 전에 config 계약만 작게 고정한다. + +## 사용자 리뷰 요청 흐름 + +구현 중 blocker는 active review stub의 `사용자 리뷰 요청` 섹션에 기록한다. 섹션 형식은 `agent-ops/skills/common/_templates/implementation-user-review-request-section.md`를 따른다. code-review가 요청 정당성을 검증하고 실제 `USER_REVIEW.md` 작성 여부를 결정한다. + +## 분석 결과 + +### 읽은 파일 + +- `agent-roadmap/current.md` +- `agent-roadmap/phase/automation-runtime-bridge/PHASE.md` +- `agent-roadmap/phase/automation-runtime-bridge/milestones/oto-bootstrap-command-issue.md` +- `agent-ops/rules/project/domain/edge/rules.md` +- `agent-ops/rules/project/domain/platform-common/rules.md` +- `agent-ops/rules/project/domain/testing/rules.md` +- `agent-test/local/rules.md` +- `agent-test/local/edge-smoke.md` +- `agent-test/local/platform-common-smoke.md` +- `agent-test/local/testing-smoke.md` +- `apps/edge/cmd/edge/main.go` +- `apps/edge/cmd/edge/main_test.go` +- `packages/go/config/config.go` +- `packages/go/config/config_test.go` +- `configs/edge.yaml` +- `apps/edge/internal/node/mapper.go` +- `apps/edge/internal/node/store.go` +- `apps/edge/internal/transport/server.go` +- `../oto/assets/script/shell/oto_agent_bootstrap.sh` + +### 테스트 환경 규칙 + +- `test_env=local`; `agent-test/local/rules.md`를 읽었다. +- 적용 profile: `agent-test/local/edge-smoke.md`, `agent-test/local/platform-common-smoke.md`, `agent-test/local/testing-smoke.md`. +- 적용 명령: edge 변경은 `go test ./apps/edge/...`, platform-common 변경은 `go test ./packages/go/...`, bootstrap UX 변경은 변경 entrypoint full-cycle 확인을 요구한다. +- 이 subtask의 최종 명령은 fresh 실행을 위해 `-count=1`을 사용한다. fallback 없음. + +### 테스트 커버리지 공백 + +- `bootstrap.oto_bootstrap_script_url`, `bootstrap.oto_release_base_url` 로딩: 기존 테스트 없음. `packages/go/config/config_test.go`에 default/explicit load 테스트를 추가한다. +- YAML patch가 새 bootstrap 필드를 보존/갱신하는 동작: 기존 `artifact_base_url`만 다룬다. `apps/edge/cmd/edge/main_test.go`에 patch roundtrip 테스트를 추가한다. +- OTO agent adapterless config check는 작은 선행 변경으로 `TestConfigCheckAcceptsOTOAgentWithoutAdapters`가 추가되어 통과했다. + +### 심볼 참조 + +- renamed/removed symbol: none. + +### 분할 판단 + +- split decision policy를 plan 파일 선택 전에 평가했다. +- shared task group: `m-oto-bootstrap-command-issue`. +- `01_oto_bootstrap_config`: 독립 선행 작업. config struct/default/example/YAML patch helper를 다룬다. +- `02+01_agent_register_command`: `01` 완료 후 command surface가 새 config 계약을 소비한다. +- 이 plan은 predecessor가 없다. sibling `02+01_agent_register_command`는 이 subtask의 `complete.log`를 기다린다. + +### 범위 결정 근거 + +- 포함: `packages/go/config`, `configs/edge.yaml`, `apps/edge/cmd/edge`의 YAML patch helper와 테스트. +- 제외: `agent register` command 구현, OTO bootstrap script 구현, release asset build/upload, checksum/signature, Control Plane UI. +- `../oto`는 script 인자 계약 확인용으로만 읽고 수정하지 않는다. + +### 빌드 등급 + +- build=`local-G05`, review=`local-G05`. Config 계약과 YAML patch helper 중심의 bounded 변경이고 Go 테스트로 검증 가능하다. + +## 구현 체크리스트 + +- [ ] [CONFIG-1] Edge bootstrap config에 OTO script/release URL 필드를 추가하고 load/default/example 테스트를 작성한다. +- [ ] [CONFIG-2] YAML patch helper가 `artifact_base_url`, `oto_bootstrap_script_url`, `oto_release_base_url`을 같은 방식으로 보존/갱신하게 만든다. +- [ ] [CONFIG-3] `go test -count=1 ./packages/go/config ./apps/edge/cmd/edge`를 실행한다. +- [ ] CODE_REVIEW-*-G??.md의 구현 에이전트 소유 섹션을 실제 구현 내용과 검증 출력으로 채운다. 이 항목이 완료되기 전에는 구현이 완료된 것이 아니다. + +## CONFIG-1 Edge bootstrap config 필드 + +### 문제 + +`packages/go/config/config.go:86`의 `EdgeBootstrapConf`는 Node artifact URL만 가진다. + +```go +86 type EdgeBootstrapConf struct { +87 ArtifactBaseURL string `mapstructure:"artifact_base_url" yaml:"artifact_base_url"` +88 Listen string `mapstructure:"listen" yaml:"listen"` +89 ArtifactDir string `mapstructure:"artifact_dir" yaml:"artifact_dir"` +90 } +``` + +OTO script는 필수 `--release-base-url`을 요구하고 HTTPS를 검증한다: `../oto/assets/script/shell/oto_agent_bootstrap.sh:16`, `../oto/assets/script/shell/oto_agent_bootstrap.sh:82`. + +### 해결 방법 + +`EdgeBootstrapConf`에 아래 필드를 추가한다. + +```go +OTOBootstrapScriptURL string `mapstructure:"oto_bootstrap_script_url" yaml:"oto_bootstrap_script_url"` +OTOReleaseBaseURL string `mapstructure:"oto_release_base_url" yaml:"oto_release_base_url"` +``` + +`setEdgeDefaults`에는 두 값을 빈 문자열 default로 둔다. `configs/edge.yaml`의 `bootstrap:` 주석과 예시에 두 필드를 추가한다. + +### 수정 파일 및 체크리스트 + +- [ ] `packages/go/config/config.go`에 필드와 defaults를 추가한다. +- [ ] `packages/go/config/config_test.go`에 default empty와 explicit load 테스트를 추가한다. +- [ ] `configs/edge.yaml`의 `bootstrap:` 예시에 두 값을 빈 문자열로 추가한다. + +### 테스트 작성 + +작성한다. `packages/go/config/config_test.go`에 `TestLoadEdge_OTOBootstrapDefaults`와 `TestLoadEdge_OTOBootstrapExplicit`를 추가한다. 목표는 default empty와 YAML explicit 값이 loader/runtime load에서 유지되는지 확인하는 것이다. + +### 중간 검증 + +```bash +go test -count=1 ./packages/go/config +``` + +기대 결과: `ok iop/packages/go/config`. + +## CONFIG-2 YAML patch helper 확장 + +### 문제 + +`apps/edge/cmd/edge/main.go:1163` 이후 `patchYAML`은 `bootstrap.artifact_base_url` 하나만 patch한다. 새 command가 OTO URL을 config에 기록하려면 같은 보존형 patch 경로가 필요하다. + +```go +1163 // 2. Patch "bootstrap.artifact_base_url" if bootstrap config has artifact URL +1164 if cfg.Bootstrap.ArtifactBaseURL != "" { +... +1192 urlKey := &yaml.Node{Kind: yaml.ScalarNode, Tag: "!!str", Value: "artifact_base_url"} +1193 urlVal := &yaml.Node{Kind: yaml.ScalarNode, Tag: "!!str", Value: cfg.Bootstrap.ArtifactBaseURL} +``` + +### 해결 방법 + +`patchBootstrapStringFields(rootMap, map[string]string)` helper를 추가하고 `artifact_base_url`, `oto_bootstrap_script_url`, `oto_release_base_url` 중 값이 비어 있지 않은 필드만 patch한다. 기존 `artifact_base_url` 동작은 유지한다. + +### 수정 파일 및 체크리스트 + +- [ ] `apps/edge/cmd/edge/main.go`에서 bootstrap patch 로직을 helper로 분리한다. +- [ ] `apps/edge/cmd/edge/main_test.go`에 OTO bootstrap fields patch 테스트를 추가한다. + +### 테스트 작성 + +작성한다. `apps/edge/cmd/edge/main_test.go`에 `TestPatchYAMLUpdatesOTOBootstrapFields`를 추가한다. 목표는 기존 YAML layout을 보존하면서 `bootstrap.oto_bootstrap_script_url`과 `bootstrap.oto_release_base_url`이 추가/갱신되는지 확인하는 것이다. + +### 중간 검증 + +```bash +go test -count=1 ./apps/edge/cmd/edge +``` + +기대 결과: `ok iop/apps/edge/cmd/edge`. + +## 수정 파일 요약 + +| 파일 | 항목 | +|------|------| +| `packages/go/config/config.go` | CONFIG-1 | +| `packages/go/config/config_test.go` | CONFIG-1 | +| `configs/edge.yaml` | CONFIG-1 | +| `apps/edge/cmd/edge/main.go` | CONFIG-2 | +| `apps/edge/cmd/edge/main_test.go` | CONFIG-2 | + +## 최종 검증 + +```bash +go test -count=1 ./packages/go/config ./apps/edge/cmd/edge +go test -count=1 ./packages/go/... ./apps/edge/... +``` + +기대 결과: 모든 package가 exit code 0으로 종료한다. +모든 코드 변경 완료 후 반드시 `CODE_REVIEW-*-G??.md`의 구현 에이전트 소유 섹션을 채운다. 이 파일 작성이 구현의 마지막 단계다. diff --git a/agent-task/archive/2026/06/m-oto-bootstrap-command-issue/02+01_agent_register_command/code_review_cloud_G07_0.log b/agent-task/archive/2026/06/m-oto-bootstrap-command-issue/02+01_agent_register_command/code_review_cloud_G07_0.log new file mode 100644 index 0000000..3c8acca --- /dev/null +++ b/agent-task/archive/2026/06/m-oto-bootstrap-command-issue/02+01_agent_register_command/code_review_cloud_G07_0.log @@ -0,0 +1,258 @@ + + +# Code Review Reference - CLI + +> **[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 user-only decision, user-owned external environment prerequisite, or scope conflict, fill `사용자 리뷰 요청` with evidence and stop with active files in place; code-review decides whether to write `USER_REVIEW.md`. Evidence gaps that a follow-up agent can close by rerunning commands or collecting artifacts are normal follow-up issues, not user-review blockers by themselves. +> 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-06-03 +task=m-oto-bootstrap-command-issue/02+01_agent_register_command, plan=0, tag=CLI + +## Roadmap Targets + +- Milestone: `agent-roadmap/phase/automation-runtime-bridge/milestones/oto-bootstrap-command-issue.md` +- Task ids: + - `agent-register-cmd`: `iop-edge`가 OTO agent record를 만들거나 갱신하는 command 표면을 제공한다. + - `node-interface-parity`: OTO agent command/config 표면이 기존 `iop-node` bootstrap의 필드명, flag 의미, 출력 형식, 진단 흐름을 최대한 유지한다. + - `bootstrap-output`: command 출력은 한 줄 bootstrap command만 안정적으로 제공하고 token을 불필요하게 재출력하지 않는다. + - `oto-args`: 출력 command가 OTO bootstrap 필수 인자 `--edge-url`, `--agent-id`, `--enrollment-token`, `--release-base-url`을 포함한다. + - `config-roundtrip`: 생성된 edge config를 `config check`와 runtime load가 읽을 수 있다. +- Completion mode: check-on-pass + +## 이 파일을 읽는 리뷰 에이전트에게 + +> **[REVIEW AGENT ONLY]** 아래 종결 절차는 코드리뷰 에이전트 전용이다. 구현 에이전트는 이 섹션을 실행하지 않는다. + +각 항목의 구현을 실제 소스 파일과 대조하고, `검증 결과` 섹션의 출력이 코드와 일치하는지 확인하세요. +리뷰 완료는 아래 순서까지 끝난 상태를 의미합니다. + +1. 판정을 append한다. +2. `CODE_REVIEW-{review_lane}-GNN.md` -> `code_review_{review_lane}_GNN_N.log`, `PLAN-{build_lane}-GNN.md` -> `plan_{build_lane}_GNN_M.log`로 아카이브한다. +3. PASS이면 `complete.log` 작성 후 active task 디렉터리를 `agent-task/archive/YYYY/MM/{task_name}/`로 이동한다. WARN/FAIL이면 user-review gate를 확인한 뒤 다음 active plan/review 파일 또는 `USER_REVIEW.md`를 작성한다. `USER_REVIEW.md`가 사용자 결정으로 완료/PASS 해소되면 code-review가 `USER_REVIEW.md`를 해소 상태로 갱신하고 `complete.log` 작성 후 archive 이동한다. +4. PASS이고 task group이 `m-`이면 완료 이벤트 메타데이터를 보고한다. roadmap 상태 체크와 `update-roadmap` 호출은 런타임 책임이다. +5. 적용 가능한 `코드리뷰 전용 체크리스트` 항목을 최종 `.log` 위치에서 체크한 뒤 보고한다. + +--- + +## 구현 항목별 완료 여부 + +| 항목 | 완료 여부 | +|------|---------| +| [CLI-1] `iop-edge agent register [agent-id]` command를 추가하고 OTO agent record를 upsert한다. | [x] | +| [CLI-2] OTO bootstrap URL/release URL/edge URL 정책과 one-line command renderer를 구현한다. | [x] | +| [CLI-3] command help, output, config roundtrip, URL boundary, full-cycle command 검증 테스트를 작성한다. | [x] | + +## 구현 체크리스트 + +- [x] [CLI-1] `iop-edge agent register [agent-id]` command를 추가하고 OTO agent record를 upsert한다. +- [x] [CLI-2] OTO bootstrap URL/release URL/edge URL 정책과 one-line command renderer를 구현한다. +- [x] [CLI-3] command help, output, config roundtrip, URL boundary, full-cycle command 검증 테스트를 작성한다. +- [x] [CLI-4] `go test -count=1 ./apps/edge/cmd/edge ./packages/go/config`와 실제 `go run ./apps/edge/cmd/edge ... agent register ...` full-cycle command를 실행한다. +- [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_{review_lane}_GNN_N.log`로 아카이브한다. +- [x] active `PLAN-*-G??.md`를 `plan_{build_lane}_GNN_M.log`로 아카이브한다. +- [x] `.gitignore`의 Agent-Ops 관리 block이 `agent-task/**/*.md`와 `agent-task/**/*.log`를 unignore하여 plan/review/archive 산출물이 추적 가능한지 확인한다. +- [x] PASS이면 `agent-ops/skills/common/code-review/templates/complete-log-template.md` 기준으로 `complete.log`를 작성하고 active `.md` 파일을 남기지 않는다. +- [x] PASS이면 active task 디렉터리 `agent-task/m-oto-bootstrap-command-issue/02+01_agent_register_command/`를 `agent-task/archive/YYYY/MM/m-oto-bootstrap-command-issue/02+01_agent_register_command/`로 이동하고 최종 archive 경로에서 이 체크리스트를 갱신한다. +- [x] PASS이고 task group이 `m-oto-bootstrap-command-issue`이면 런타임이 읽을 완료 이벤트 메타데이터를 보고하고, roadmap 수정이나 `update-roadmap` 직접 호출을 하지 않는다. +- [x] PASS split 작업이면 이동 후 빈 active parent `agent-task/m-oto-bootstrap-command-issue/`를 제거하거나, 남은 sibling/file이 있어 유지했다고 확인한다. +- [ ] WARN/FAIL이고 user-review gate가 트리거되지 않았으면 다음 active `PLAN-{build_lane}-GNN.md`와 `CODE_REVIEW-{review_lane}-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가 사용자 결정으로 완료/PASS 해소되면 `USER_REVIEW.md`를 해소 상태로 갱신하고 `complete.log`를 작성한 뒤 task directory를 archive로 이동한다. + +## 계획 대비 변경 사항 + +계획에 명시된 모든 설계와 기능을 준수하여 구현하였으며, 특이 사항이나 계획 대비 변경 사항은 없습니다. + +## 주요 설계 결정 + +- **URL 스키마 검증 강제화**: `release-base-url`은 `https://`로 시작하지 않을 시 에러를 유발하여 보안 정책을 고수했습니다. `bootstrap-script-url`은 기본으로 `https://`를 사용하되, 로컬/개발 편의성을 고려해 `--allow-insecure-bootstrap-url` 플래그가 수동으로 지정되었을 경우에만 `http://`도 사용 가능하게 통제하였습니다. +- **OTO Node 구조 통일**: OTO agent record가 upsert될 때, `agent_kind`는 `oto-agent`로 저장되며 adapters는 전부 disabled/empty 상태가 됩니다. 이는 edge-node 공통 validation 흐름에 영향을 주지 않으면서 Specialized Agent의 요구사항을 반영하는 최소 침습적 구조 설계입니다. + +## 사용자 리뷰 요청 + +- 상태: 없음 +- 사유 유형: 없음 +- 결정 필요: 없음 +- 차단 근거: 없음 +- 실행한 검증/명령: 없음 +- 자동 후속 불가 이유: 없음 +- 재개 조건: 없음 + +## 리뷰어를 위한 체크포인트 + +- `agent-task/m-oto-bootstrap-command-issue/01_oto_bootstrap_config/complete.log`가 존재한 뒤 구현됐는가. +- `agent register`가 OTO record를 `agent_kind=oto-agent`로 저장하고 generic node path를 깨지 않는가. +- 출력이 정확히 한 줄이고 OTO 필수 인자 4개를 포함하며 token을 추가로 출력하지 않는가. +- HTTPS 기본 정책과 explicit local/dev HTTP override가 테스트로 고정됐는가. +- `Roadmap Targets`의 5개 task id가 PASS 근거와 일치하는가. + +## 검증 결과 + +### CLI-1 중간 검증 +```bash +$ go test -count=1 ./apps/edge/cmd/edge +ok iop/apps/edge/cmd/edge 0.069s +``` + +### CLI-2 중간 검증 +```bash +$ go test -count=1 ./apps/edge/cmd/edge +ok iop/apps/edge/cmd/edge 0.069s +``` + +### CLI-3 중간 검증 +```bash +$ go test -count=1 ./apps/edge/cmd/edge ./packages/go/config +ok iop/apps/edge/cmd/edge 0.069s +ok iop/packages/go/config 0.018s +``` + +### 최종 검증 +```bash +$ go test -count=1 ./apps/edge/cmd/edge ./packages/go/config +ok iop/apps/edge/cmd/edge 0.069s +ok iop/packages/go/config 0.018s +``` + +```bash +$ go test -count=1 ./apps/edge/... ./packages/go/... +ok iop/apps/edge/cmd/edge 0.098s +ok iop/apps/edge/internal/bootstrap 0.033s +ok iop/apps/edge/internal/controlplane 4.454s +ok iop/apps/edge/internal/events 0.016s +ok iop/apps/edge/internal/input 0.028s +ok iop/apps/edge/internal/input/a2a 0.016s +ok iop/apps/edge/internal/node 0.013s +ok iop/apps/edge/internal/openai 1.517s +ok iop/apps/edge/internal/opsconsole 0.035s +ok iop/apps/edge/internal/service 0.013s +ok iop/apps/edge/internal/transport 0.014s +ok iop/packages/go/audit 0.003s +? iop/packages/go/auth [no test files] +ok iop/packages/go/config 0.015s +? iop/packages/go/events [no test files] +ok iop/packages/go/hostsetup 0.012s +? iop/packages/go/jobs [no test files] +? iop/packages/go/metadata [no test files] +ok iop/packages/go/observability 0.024s +? iop/packages/go/policy [no test files] +? iop/packages/go/version [no test files] +``` + +```bash +$ tmpdir="$(mktemp -d)"; printf 'server:\n listen: "127.0.0.1:19090"\n advertise_host: "edge.example.test"\nbootstrap:\n oto_bootstrap_script_url: "https://oto.example.test/bootstrap/oto_agent_bootstrap.sh"\n oto_release_base_url: "https://oto.example.test/releases"\nnodes: []\n' > "$tmpdir/edge.yaml"; go run ./apps/edge/cmd/edge --config "$tmpdir/edge.yaml" agent register oto-agent-a --token test-token; go run ./apps/edge/cmd/edge --config "$tmpdir/edge.yaml" config check +curl -fsSL https://oto.example.test/bootstrap/oto_agent_bootstrap.sh | bash -s -- --edge-url edge.example.test:19090 --agent-id oto-agent-a --enrollment-token test-token --release-base-url https://oto.example.test/releases --agent-alias oto-agent-a +OK /tmp/tmp.CYgEwfuROj/edge.yaml +``` + +```bash +$ rg --sort path -n "agent register|oto_bootstrap_script_url|oto_release_base_url|--edge-url|--release-base-url" apps/edge/cmd/edge packages/go/config configs agent-roadmap/phase/automation-runtime-bridge/milestones/oto-bootstrap-command-issue.md +apps/edge/cmd/edge/main.go +52: 3. node register - Add a node record to edge.yaml and generate its one-line bootstrap command. (Use 'agent register' for specialized OTO agents.) +60: iop-edge agent register my-agent --bootstrap-script-url https://example.com/boot.sh --release-base-url https://example.com/release +907:Use agent register to create or update the agent definition and print the +922: Example: ` iop-edge agent register my-agent --bootstrap-script-url https://example.com/boot.sh --release-base-url https://example.com/release +952: return errors.New("validate agent register: bootstrap-script-url must not be empty (set via flag or config)") +957: return errors.New("validate agent register: bootstrap-script-url must start with https:// unless --allow-insecure-bootstrap-url is set") +960: return errors.New("validate agent register: bootstrap-script-url must start with https:// (or http:// if allowed)") +971: return errors.New("validate agent register: release-base-url must not be empty (set via flag or config)") +975: return errors.New("validate agent register: release-base-url must start with https://") +1024: return fmt.Errorf("validate agent register: %w", err) +1035: // curl -fsSL | bash -s -- --edge-url --agent-id --enrollment-token --release-base-url --agent-alias +1036: fmt.Fprintf(cmd.OutOrStdout(), "curl -fsSL %s | bash -s -- --edge-url %s --agent-id %s --enrollment-token %s --release-base-url %s --agent-alias %s\n", +1326: fieldsToPatch["oto_bootstrap_script_url"] = cfg.Bootstrap.OTOBootstrapScriptURL +1329: fieldsToPatch["oto_release_base_url"] = cfg.Bootstrap.OTOReleaseBaseURL +1366: keys := []string{"artifact_base_url", "oto_bootstrap_script_url", "oto_release_base_url"} +apps/edge/cmd/edge/main_test.go +1145: want: []string{"agent register", "edge.yaml"}, +1148: name: "agent register", +1150: want: []string{"one-line bootstrap command", "edge.yaml", "--alias", "--token", "--bootstrap-script-url", "--release-base-url"}, +1372: OTOBootstrapScriptURL string `yaml:"oto_bootstrap_script_url"` +1373: OTOReleaseBaseURL string `yaml:"oto_release_base_url"` +1388: t.Errorf("oto_bootstrap_script_url = %q", doc.Bootstrap.OTOBootstrapScriptURL) +1391: t.Errorf("oto_release_base_url = %q", doc.Bootstrap.OTOReleaseBaseURL) +1402: oto_bootstrap_script_url: "https://example.com/boot.sh" +1403: oto_release_base_url: "https://example.com/releases" +1417: t.Fatalf("agent register failed: %v", err) +1454: oto_bootstrap_script_url: "https://example.com/boot.sh" +1455: oto_release_base_url: "https://example.com/releases" +1466: root.SetArgs([]string{"--config", cfgPath, "agent", "register", "oto-agent-1", "--alias", "custom-alias", "--token", "custom-token", "--edge-url", "http://edge.test:9090"}) +1469: t.Fatalf("agent register failed: %v", err) +1473: want := "curl -fsSL https://example.com/boot.sh | bash -s -- --edge-url http://edge.test:9090 --agent-id oto-agent-1 --enrollment-token custom-token --release-base-url https://example.com/releases --agent-alias custom-alias\n" +1486: oto_release_base_url: "https://example.com/releases" +1515: oto_bootstrap_script_url: "https://example.com/boot.sh" +1526: root.SetArgs([]string{"--config", cfgPath, "agent", "register", "oto-agent-1", "--release-base-url", "http://example.com/releases"}) +1544: oto_release_base_url: "https://example.com/releases" +1558: t.Fatalf("agent register failed with flag: %v", err) +1582: // 2. agent register +1591: "--release-base-url", "https://oto.example.test/releases", +1593: "--edge-url", "localhost:9090", +1596: t.Fatalf("agent register failed: %v", err) +1601: wantCmd := "curl -fsSL https://oto.example.test/bootstrap/oto_agent_bootstrap.sh | bash -s -- --edge-url localhost:9090 --agent-id oto-agent-test --enrollment-token test-token --release-base-url https://oto.example.test/releases --agent-alias oto-agent-test\n" +packages/go/config/config.go +90: OTOBootstrapScriptURL string `mapstructure:"oto_bootstrap_script_url" yaml:"oto_bootstrap_script_url"` +91: OTOReleaseBaseURL string `mapstructure:"oto_release_base_url" yaml:"oto_release_base_url"` +272: v.SetDefault("bootstrap.oto_bootstrap_script_url", "") +273: v.SetDefault("bootstrap.oto_release_base_url", "") +packages/go/config/config_test.go +736: t.Errorf("expected empty bootstrap.oto_bootstrap_script_url by default, got %q", cfg.Bootstrap.OTOBootstrapScriptURL) +739: t.Errorf("expected empty bootstrap.oto_release_base_url by default, got %q", cfg.Bootstrap.OTOReleaseBaseURL) +749: oto_bootstrap_script_url: "https://example.com/bootstrap.sh" +750: oto_release_base_url: "https://example.com/releases" +760: t.Errorf("bootstrap.oto_bootstrap_script_url=%q", cfg.Bootstrap.OTOBootstrapScriptURL) +763: t.Errorf("bootstrap.oto_release_base_url=%q", cfg.Bootstrap.OTOReleaseBaseURL) +configs/edge.yaml +16: oto_bootstrap_script_url: "" +17: oto_release_base_url: "" +agent-roadmap/phase/automation-runtime-bridge/milestones/oto-bootstrap-command-issue.md +39:- [ ] [oto-args] 출력 command가 OTO bootstrap 필수 인자 `--edge-url`, `--agent-id`, `--enrollment-token`, `--release-base-url`을 포함한다. +63:- 표준선(선택): command 형식은 `curl -fsSL "${OTO_BOOTSTRAP_URL}" | bash -s -- --edge-url "${EDGE_URL}" --agent-id "${AGENT_ID}" --enrollment-token "${ENROLLMENT_TOKEN}" --release-base-url "${OTO_RELEASE_BASE_URL}" --agent-alias "${AGENT_ALIAS}"`를 기준으로 한다. +``` + +--- + +> **[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. + +## 코드리뷰 결과 + +### 종합 판정 + +PASS + +### 차원별 평가 + +| 차원 | 평가 | 근거 | +|------|------|------| +| correctness | Pass | `agent register`가 OTO record를 upsert하고 `agent_kind=oto-agent`, adapter disabled 상태, one-line OTO bootstrap command를 계획대로 생성한다. | +| completeness | Pass | CLI-1~CLI-4 범위와 Roadmap Targets의 command/config/output 요구사항을 모두 충족했다. | +| test coverage | Pass | command, help, URL boundary, YAML patch, generated config roundtrip 테스트가 있으며 review에서 대상/확장/전체 Go 테스트와 실제 CLI full-cycle을 재실행했다. | +| API contract | Pass | 기존 `node register` 경로를 유지하고 OTO script/parser의 필수 인자 계약과 config loader/default 계약을 유지한다. | +| code quality | Pass | 기존 Cobra command, YAML patch, config validation 패턴을 확장했고 불필요한 구조 변경이 없다. | +| plan deviation | Pass | 계획 대비 범위 이탈 없음. | +| verification trust | Pass | 구현 문서의 검증 출력과 review 재실행 결과가 일치하고, 추가로 generated config persistence를 실제 command로 확인했다. | + +### 발견된 문제 + +없음 + +### 다음 단계 + +PASS: `complete.log` 작성 후 active task 디렉터리를 archive로 이동한다. diff --git a/agent-task/archive/2026/06/m-oto-bootstrap-command-issue/02+01_agent_register_command/complete.log b/agent-task/archive/2026/06/m-oto-bootstrap-command-issue/02+01_agent_register_command/complete.log new file mode 100644 index 0000000..c6c39cc --- /dev/null +++ b/agent-task/archive/2026/06/m-oto-bootstrap-command-issue/02+01_agent_register_command/complete.log @@ -0,0 +1,48 @@ +# Complete - m-oto-bootstrap-command-issue/02+01_agent_register_command + +## 완료 일시 + +2026-06-03 + +## 요약 + +CLI plan 0/code-review 0 loop를 PASS로 종결했다. `iop-edge agent register`가 OTO agent record와 one-line bootstrap command를 생성하고, URL 정책과 config roundtrip 검증을 충족했다. + +## 루프 이력 + +| Plan | Review | Verdict | 메모 | +|------|--------|---------|------| +| `plan_cloud_G07_0.log` | `code_review_cloud_G07_0.log` | PASS | Required/Suggested 이슈 없음 | + +## 구현/정리 내용 + +- `iop-edge agent register [agent-id]` command를 추가해 `agent_kind=oto-agent` record를 upsert하고 generic node adapter 설정을 활성화하지 않도록 했다. +- OTO bootstrap script URL, release base URL, edge URL, agent id, enrollment token, alias를 포함하는 one-line command renderer와 HTTPS/default URL validation을 구현했다. +- Edge CLI help, OTO output, URL boundary, YAML patch, generated config roundtrip 테스트를 추가했다. + +## 최종 검증 + +- `go test -count=1 ./apps/edge/cmd/edge ./packages/go/config` - PASS; `iop/apps/edge/cmd/edge`, `iop/packages/go/config` 통과. +- `go test -count=1 ./apps/edge/... ./packages/go/...` - PASS; Edge와 shared Go packages 범위 통과. +- `go run ./apps/edge/cmd/edge --config "$tmpdir/edge.yaml" agent register oto-agent-a --token test-token` + `go run ./apps/edge/cmd/edge --config "$tmpdir/edge.yaml" config check` - PASS; one-line OTO bootstrap command 출력 후 `OK /edge.yaml`. +- `rg -n "oto_bootstrap_script_url|oto_release_base_url|agent_kind|token" "$tmpdir/edge.yaml"` - PASS; generated config에 OTO bootstrap URL, release URL, `agent_kind: oto-agent`, token이 persisted됨을 확인. +- `go test ./...` - PASS; 전체 Go 테스트 통과. + +## Roadmap Completion + +- Milestone: `agent-roadmap/phase/automation-runtime-bridge/milestones/oto-bootstrap-command-issue.md` +- Completed task ids: + - `agent-register-cmd`: PASS; evidence=`agent-task/archive/2026/06/m-oto-bootstrap-command-issue/02+01_agent_register_command/plan_cloud_G07_0.log`, `agent-task/archive/2026/06/m-oto-bootstrap-command-issue/02+01_agent_register_command/code_review_cloud_G07_0.log`; verification=`go test -count=1 ./apps/edge/cmd/edge ./packages/go/config` + - `node-interface-parity`: PASS; evidence=`agent-task/archive/2026/06/m-oto-bootstrap-command-issue/02+01_agent_register_command/plan_cloud_G07_0.log`, `agent-task/archive/2026/06/m-oto-bootstrap-command-issue/02+01_agent_register_command/code_review_cloud_G07_0.log`; verification=`go run ./apps/edge/cmd/edge --config "$tmpdir/edge.yaml" agent register oto-agent-a --token test-token` + - `bootstrap-output`: PASS; evidence=`agent-task/archive/2026/06/m-oto-bootstrap-command-issue/02+01_agent_register_command/plan_cloud_G07_0.log`, `agent-task/archive/2026/06/m-oto-bootstrap-command-issue/02+01_agent_register_command/code_review_cloud_G07_0.log`; verification=`go run ./apps/edge/cmd/edge --config "$tmpdir/edge.yaml" agent register oto-agent-a --token test-token` + - `oto-args`: PASS; evidence=`agent-task/archive/2026/06/m-oto-bootstrap-command-issue/02+01_agent_register_command/plan_cloud_G07_0.log`, `agent-task/archive/2026/06/m-oto-bootstrap-command-issue/02+01_agent_register_command/code_review_cloud_G07_0.log`; verification=`go run ./apps/edge/cmd/edge --config "$tmpdir/edge.yaml" agent register oto-agent-a --token test-token` + - `config-roundtrip`: PASS; evidence=`agent-task/archive/2026/06/m-oto-bootstrap-command-issue/02+01_agent_register_command/plan_cloud_G07_0.log`, `agent-task/archive/2026/06/m-oto-bootstrap-command-issue/02+01_agent_register_command/code_review_cloud_G07_0.log`; verification=`go run ./apps/edge/cmd/edge --config "$tmpdir/edge.yaml" config check` +- Not completed task ids: 없음 + +## 잔여 Nit + +- 없음 + +## 후속 작업 + +- 없음 diff --git a/agent-task/archive/2026/06/m-oto-bootstrap-command-issue/02+01_agent_register_command/plan_cloud_G07_0.log b/agent-task/archive/2026/06/m-oto-bootstrap-command-issue/02+01_agent_register_command/plan_cloud_G07_0.log new file mode 100644 index 0000000..7f6828f --- /dev/null +++ b/agent-task/archive/2026/06/m-oto-bootstrap-command-issue/02+01_agent_register_command/plan_cloud_G07_0.log @@ -0,0 +1,267 @@ + + +# Plan - CLI + +## 이 파일을 읽는 구현 에이전트에게 + +`CODE_REVIEW-*-G??.md`의 구현 에이전트 소유 섹션 작성은 필수다. 구현 후 검증을 실행하고 실제 구현 내용과 stdout/stderr를 review stub에 기록한 뒤 active 파일을 그대로 두고 리뷰 준비 상태를 보고한다. 최종 판정, log rename, `complete.log`, archive 이동은 code-review 전용이다. 사용자 결정, 사용자 소유 외부 환경, scope 충돌로 막히면 review stub의 `사용자 리뷰 요청` 섹션에 정확한 결정과 증거를 남기고 멈춘다. 후속 에이전트가 재실행이나 증거 수집으로 해소할 수 있는 검증 공백은 사용자 리뷰 요청이 아니다. + +## 배경 + +현재 Edge CLI는 generic node용 `node register`만 제공한다. OTO는 같은 token 기반 proto-socket registration path를 재사용하지만 bootstrap command는 OTO script와 release URL 인자를 소비해야 한다. 이 작업은 선행 config 계약이 완료된 뒤 Edge-local `agent register` command를 추가해 OTO agent record와 one-line bootstrap command를 만든다. + +## 사용자 리뷰 요청 흐름 + +구현 중 blocker는 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/automation-runtime-bridge/milestones/oto-bootstrap-command-issue.md` +- Task ids: + - `agent-register-cmd`: `iop-edge`가 OTO agent record를 만들거나 갱신하는 command 표면을 제공한다. + - `node-interface-parity`: OTO agent command/config 표면이 기존 `iop-node` bootstrap의 필드명, flag 의미, 출력 형식, 진단 흐름을 최대한 유지한다. + - `bootstrap-output`: command 출력은 한 줄 bootstrap command만 안정적으로 제공하고 token을 불필요하게 재출력하지 않는다. + - `oto-args`: 출력 command가 OTO bootstrap 필수 인자 `--edge-url`, `--agent-id`, `--enrollment-token`, `--release-base-url`을 포함한다. + - `config-roundtrip`: 생성된 edge config를 `config check`와 runtime load가 읽을 수 있다. +- Completion mode: check-on-pass + +## 분석 결과 + +### 읽은 파일 + +- `agent-roadmap/current.md` +- `agent-roadmap/phase/automation-runtime-bridge/PHASE.md` +- `agent-roadmap/phase/automation-runtime-bridge/milestones/oto-bootstrap-command-issue.md` +- `agent-ops/rules/project/domain/edge/rules.md` +- `agent-ops/rules/project/domain/platform-common/rules.md` +- `agent-ops/rules/project/domain/testing/rules.md` +- `agent-test/local/rules.md` +- `agent-test/local/edge-smoke.md` +- `agent-test/local/platform-common-smoke.md` +- `agent-test/local/testing-smoke.md` +- `apps/edge/cmd/edge/main.go` +- `apps/edge/cmd/edge/main_test.go` +- `packages/go/config/config.go` +- `packages/go/config/config_test.go` +- `configs/edge.yaml` +- `apps/edge/internal/node/mapper.go` +- `apps/edge/internal/node/store.go` +- `apps/edge/internal/transport/server.go` +- `../oto/assets/script/shell/oto_agent_bootstrap.sh` +- `../oto/lib/oto/agent/edge_registration_client.dart` +- `../oto/test/oto_agent_config_test.dart` + +### 테스트 환경 규칙 + +- `test_env=local`; `agent-test/local/rules.md`를 읽었다. +- 적용 profile: `agent-test/local/edge-smoke.md`, `agent-test/local/platform-common-smoke.md`, `agent-test/local/testing-smoke.md`. +- 적용 명령: `go test ./apps/edge/...`, `go test ./packages/go/...`, bootstrap/install UX 변경의 실제 entrypoint full-cycle command. +- 이 plan의 Go 검증은 fresh 실행을 위해 `-count=1`을 사용한다. fallback 없음. + +### 테스트 커버리지 공백 + +- OTO `agent register` command: 기존 테스트 없음. 새 CLI tests가 필요하다. +- OTO bootstrap one-line output: 기존 node output tests만 있다. 새 exact-output test가 필요하다. +- OTO 필수 인자 포함: 기존 테스트 없음. command output assertion으로 검증한다. +- HTTP script URL gating: 기존 테스트 없음. boundary test가 필요하다. +- Config roundtrip: 작은 선행 변경으로 adapterless OTO config check는 통과했고, 이 plan은 generated config가 `config check`와 `config.LoadEdge`를 통과하는 테스트를 추가해야 한다. + +### 심볼 참조 + +- renamed/removed symbol: none. + +### 분할 판단 + +- split decision policy를 plan 파일 선택 전에 평가했다. +- shared task group: `m-oto-bootstrap-command-issue`. +- sibling subtasks: + - `01_oto_bootstrap_config`: config struct/default/example/YAML patch 선행 계약. + - `02+01_agent_register_command`: 이 plan. command surface가 `01`의 config fields와 patch helper를 소비한다. +- predecessor `01`: active plan exists at `agent-task/m-oto-bootstrap-command-issue/01_oto_bootstrap_config/PLAN-local-G05.md`; `complete.log` is not present yet. Runtime must wait for predecessor completion before implementing this task. + +### 범위 결정 근거 + +- 포함: `apps/edge/cmd/edge` command surface, CLI tests, generated temp config full-cycle check. +- 제외: OTO bootstrap script 변경, OTO release asset build/upload, checksum/signature enforcement, Control Plane remote issue UI, OTO online smoke. Online smoke는 후속 Milestone이다. +- `../oto`는 script and config parser contract 확인용으로만 읽고 수정하지 않는다. + +### 빌드 등급 + +- build=`cloud-G07`, review=`cloud-G07`. Public CLI workflow, one-line bootstrap output, URL policy, config mutation, full-cycle command evidence가 함께 걸려 terminal/CLI reasoning과 verification trust가 중요하다. + +## 의존 관계 및 구현 순서 + +`02+01_agent_register_command`는 디렉터리 이름 기준으로 predecessor index `01`에 의존한다. `agent-task/m-oto-bootstrap-command-issue/01_oto_bootstrap_config/complete.log`가 생긴 뒤 구현한다. 추가 dependency는 없다. + +## 구현 체크리스트 + +- [ ] [CLI-1] `iop-edge agent register [agent-id]` command를 추가하고 OTO agent record를 upsert한다. +- [ ] [CLI-2] OTO bootstrap URL/release URL/edge URL 정책과 one-line command renderer를 구현한다. +- [ ] [CLI-3] command help, output, config roundtrip, URL boundary, full-cycle command 검증 테스트를 작성한다. +- [ ] [CLI-4] `go test -count=1 ./apps/edge/cmd/edge ./packages/go/config`와 실제 `go run ./apps/edge/cmd/edge ... agent register ...` full-cycle command를 실행한다. +- [ ] CODE_REVIEW-*-G??.md의 구현 에이전트 소유 섹션을 실제 구현 내용과 검증 출력으로 채운다. 이 항목이 완료되기 전에는 구현이 완료된 것이 아니다. + +## CLI-1 Agent register command surface + +### 문제 + +`apps/edge/cmd/edge/main.go:66`의 root command는 `nodeCmd()`만 등록하고 specialized agent command가 없다. + +```go +66 root.AddCommand(serveCmd(), consoleCmd(), versionCmd(), configCmd(), envCmd(), setupCmd(), bootstrapCmd(), nodeCmd(), nodesCmd(), smokeCmd()) +``` + +`nodeRegisterCmd`는 generic node adapter/runtime 설정을 전제로 한다: `apps/edge/cmd/edge/main.go:913`. + +### 해결 방법 + +`agentCmd()`와 `agentRegisterCmd()`를 추가하고 root에 `agentCmd()`를 등록한다. Command shape: + +```text +iop-edge agent register --bootstrap-script-url --release-base-url +``` + +Flags: + +- `--alias` default `` +- `--token` default generated UUID, same meaning as `node register --token` +- `--edge-url` default `addressFor(resolveAdvertiseHost(cfg.Server.AdvertiseHost).Host, cfg.Server.Listen)` +- `--bootstrap-script-url` overrides and persists `bootstrap.oto_bootstrap_script_url` +- `--release-base-url` overrides and persists `bootstrap.oto_release_base_url` +- `--allow-insecure-bootstrap-url` allows local/dev `http://` script URL only when explicit + +Upsert record: + +```go +config.NodeDefinition{ + ID: agentID, + Alias: alias, + Token: token, + AgentKind: config.AgentKindOTOAgent, +} +``` + +Do not enable Node adapters for OTO records. + +### 수정 파일 및 체크리스트 + +- [ ] `apps/edge/cmd/edge/main.go`에 `agentCmd`, `agentRegisterCmd`, OTO-specific flag variables를 추가한다. +- [ ] `apps/edge/cmd/edge/main.go` root command에 `agentCmd()`를 등록한다. +- [ ] Existing `node register` behavior는 변경하지 않는다. + +### 테스트 작성 + +작성한다. `apps/edge/cmd/edge/main_test.go`에 `TestAgentRegisterUpsertsOTOAgentRecord`를 추가한다. 목표는 OTO record가 `agent_kind: oto-agent`, alias, token을 보존하고 adapter 없이 config check/load를 통과하는지 확인하는 것이다. + +### 중간 검증 + +```bash +go test -count=1 ./apps/edge/cmd/edge +``` + +기대 결과: `ok iop/apps/edge/cmd/edge`. + +## CLI-2 One-line OTO bootstrap renderer and URL policy + +### 문제 + +기존 `node register` 출력은 Node script와 positional token만 렌더링한다: `apps/edge/cmd/edge/main.go:1101`. + +```go +1101 bootstrapURL := universalBootstrapScriptURL(artifactURL) +1102 fmt.Fprintf(cmd.OutOrStdout(), "curl -fsSL %s | bash -s %s\n", bootstrapURL, nodeDef.Token) +``` + +OTO script는 named 필수 인자 4개를 요구한다: `../oto/assets/script/shell/oto_agent_bootstrap.sh:16`, `../oto/assets/script/shell/oto_agent_bootstrap.sh:76`. + +### 해결 방법 + +`renderOTOBootstrapCommand(opts)` helper를 추가한다. Output must be exactly one line and include: + +```text +curl -fsSL | bash -s -- --edge-url --agent-id --enrollment-token --release-base-url --agent-alias +``` + +Validation: + +- script URL and release base URL must come from flag or config; missing values fail before YAML write. +- `release-base-url` must start with `https://`. +- script URL must start with `https://` unless `--allow-insecure-bootstrap-url` is set. `http://` is accepted only with that explicit flag. +- token appears only as the `--enrollment-token` value in the one-line command. + +### 수정 파일 및 체크리스트 + +- [ ] `apps/edge/cmd/edge/main.go`에 OTO URL resolver/validator를 추가한다. +- [ ] `apps/edge/cmd/edge/main.go`에 one-line renderer를 추가한다. +- [ ] `apps/edge/cmd/edge/main_test.go`에 exact output and URL boundary tests를 추가한다. + +### 테스트 작성 + +작성한다. Tests: + +- `TestAgentRegisterPrintsOneLineOTOCommand` +- `TestAgentRegisterRejectsInsecureBootstrapURLWithoutFlag` +- `TestAgentRegisterRejectsInsecureReleaseBaseURL` +- `TestAgentRegisterAllowsInsecureBootstrapURLWithExplicitFlag` + +### 중간 검증 + +```bash +go test -count=1 ./apps/edge/cmd/edge +``` + +기대 결과: `ok iop/apps/edge/cmd/edge`. + +## CLI-3 Help, config roundtrip, and full-cycle command evidence + +### 문제 + +Root help currently advertises only `node register`: `apps/edge/cmd/edge/main.go:49`, and help tests assert the same at `apps/edge/cmd/edge/main_test.go:1099`. The milestone requires OTO agent command/config parity and generated config roundtrip. + +### 해결 방법 + +Update help without replacing the existing generic node flow. Add OTO agent register as an optional specialized agent path. Tests must assert command discovery and no legacy script path leakage. + +Generated config roundtrip test should run command against a temp `edge.yaml`, then: + +- parse YAML and assert OTO URL config fields are persisted, +- call `config.LoadEdge`, +- run `config check`, +- assert output is exactly one command line. + +### 수정 파일 및 체크리스트 + +- [ ] `apps/edge/cmd/edge/main.go` root/help examples mention `agent register` as OTO specialized path. +- [ ] `apps/edge/cmd/edge/main_test.go` updates command discovery/help tests. +- [ ] `apps/edge/cmd/edge/main_test.go` adds generated config roundtrip test. + +### 테스트 작성 + +작성한다. `TestHelpDiscoversEdgeLocalDevFlow`에 `agent register` command discovery를 추가하고, a full roundtrip test를 새로 추가한다. + +### 중간 검증 + +```bash +go test -count=1 ./apps/edge/cmd/edge ./packages/go/config +``` + +기대 결과: both packages pass. + +## 수정 파일 요약 + +| 파일 | 항목 | +|------|------| +| `apps/edge/cmd/edge/main.go` | CLI-1, CLI-2, CLI-3 | +| `apps/edge/cmd/edge/main_test.go` | CLI-1, CLI-2, CLI-3 | + +## 최종 검증 + +```bash +go test -count=1 ./apps/edge/cmd/edge ./packages/go/config +go test -count=1 ./apps/edge/... ./packages/go/... +tmpdir="$(mktemp -d)"; printf 'server:\n listen: "127.0.0.1:19090"\n advertise_host: "edge.example.test"\nbootstrap:\n oto_bootstrap_script_url: "https://oto.example.test/bootstrap/oto_agent_bootstrap.sh"\n oto_release_base_url: "https://oto.example.test/releases"\nnodes: []\n' > "$tmpdir/edge.yaml"; go run ./apps/edge/cmd/edge --config "$tmpdir/edge.yaml" agent register oto-agent-a --token test-token; go run ./apps/edge/cmd/edge --config "$tmpdir/edge.yaml" config check +rg --sort path -n "agent register|oto_bootstrap_script_url|oto_release_base_url|--edge-url|--release-base-url" apps/edge/cmd/edge packages/go/config configs agent-roadmap/phase/automation-runtime-bridge/milestones/oto-bootstrap-command-issue.md +``` + +기대 결과: Go tests pass; full-cycle command prints one bootstrap line then config check prints `OK`; search output shows command/config/milestone anchors. +모든 코드 변경 완료 후 반드시 `CODE_REVIEW-*-G??.md`의 구현 에이전트 소유 섹션을 채운다. 이 파일 작성이 구현의 마지막 단계다. diff --git a/agent-task/archive/2026/06/m-oto-registration-online-smoke/01_iop_edge_lifecycle/code_review_cloud_G07_0.log b/agent-task/archive/2026/06/m-oto-registration-online-smoke/01_iop_edge_lifecycle/code_review_cloud_G07_0.log new file mode 100644 index 0000000..392cb2d --- /dev/null +++ b/agent-task/archive/2026/06/m-oto-registration-online-smoke/01_iop_edge_lifecycle/code_review_cloud_G07_0.log @@ -0,0 +1,171 @@ + + +# 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 by a user-only decision, user-owned external environment prerequisite, or scope conflict, fill `사용자 리뷰 요청` with evidence and stop with active files in place; code-review decides whether to write `USER_REVIEW.md`. Evidence gaps that a follow-up agent can close by rerunning commands or collecting artifacts are normal follow-up issues, not user-review blockers by themselves. +> 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-06-03 +task=m-oto-registration-online-smoke/01_iop_edge_lifecycle, plan=0, tag=API + +## Roadmap Targets + +- Milestone: `agent-roadmap/phase/automation-runtime-bridge/milestones/oto-registration-online-smoke.md` +- Task ids: + - `accepted-state`: OTO agent registration accepted 상태가 generic node와 구분되어 기록된다. + - `failure-reason`: unknown token, duplicate connection, heartbeat timeout이 서로 다른 실패 reason으로 확인된다. +- Completion mode: check-on-pass + +## 이 파일을 읽는 리뷰 에이전트에게 + +> **[REVIEW AGENT ONLY]** 아래 종결 절차는 코드리뷰 에이전트 전용이다. 구현 에이전트는 이 섹션을 실행하지 않는다. + +각 항목의 구현을 실제 소스 파일과 대조하고, `검증 결과` 섹션의 출력이 코드와 일치하는지 확인하세요. +리뷰 완료는 아래 순서까지 끝난 상태를 의미합니다. + +1. 판정을 append한다. +2. `CODE_REVIEW-cloud-G07.md` → `code_review_cloud_G07_N.log`, `PLAN-cloud-G07.md` → `plan_cloud_G07_M.log`로 아카이브한다. +3. PASS이면 `complete.log` 작성 후 active task 디렉터리를 `agent-task/archive/YYYY/MM/m-oto-registration-online-smoke/01_iop_edge_lifecycle/`로 이동한다. WARN/FAIL이면 user-review gate를 확인한 뒤 다음 active plan/review 파일 또는 `USER_REVIEW.md`를 작성한다. +4. PASS이고 task group이 `m-`이면 완료 이벤트 메타데이터를 보고한다. roadmap 상태 체크와 `update-roadmap` 호출은 런타임 책임이다. +5. 적용 가능한 `코드리뷰 전용 체크리스트` 항목을 최종 `.log` 위치에서 체크한 뒤 보고한다. + +--- + +## 구현 항목별 완료 여부 + +| 항목 | 완료 여부 | +|------|---------| +| [API-1] Lifecycle Constants and Registry Update | [x] | +| [API-2] Registration Accepted Versus Online | [x] | +| [API-3] First Heartbeat Online Transition | [x] | +| [API-4] Failure Reasons | [x] | + +## 구현 체크리스트 + +- [x] OTO agent 등록 직후 lifecycle을 accepted로 기록하고 generic node 기본 connected 동작은 유지한다. +- [x] 첫 inbound `packets.HeartBeat`를 감지하면 OTO agent lifecycle을 online으로 한 번만 전환하고 node event를 발행한다. +- [x] unknown token, duplicate connection, heartbeat timeout/transport close가 서로 다른 reason/metadata로 관측되게 한다. +- [x] service snapshot과 관련 테스트가 accepted/online/failed lifecycle 값을 보존하는지 검증한다. +- [x] `go test -count=1 ./apps/edge/internal/transport ./apps/edge/internal/node ./apps/edge/internal/service ./packages/go/events`를 실행한다. +- [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_G07_N.log`로 아카이브한다. +- [x] active `PLAN-*-G??.md`를 `plan_cloud_G07_M.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-oto-registration-online-smoke/01_iop_edge_lifecycle/`를 `agent-task/archive/YYYY/MM/m-oto-registration-online-smoke/01_iop_edge_lifecycle/`로 이동하고 최종 archive 경로에서 이 체크리스트를 갱신한다. +- [ ] PASS이고 task group이 `m-`이면 런타임이 읽을 완료 이벤트 메타데이터를 보고하고, roadmap 수정이나 `update-roadmap` 직접 호출을 하지 않는다. +- [ ] PASS split 작업이면 이동 후 빈 active parent `agent-task/m-oto-registration-online-smoke/`를 제거하거나, 남은 sibling/file이 있어 유지했다고 확인한다. +- [x] WARN/FAIL이고 user-review gate가 트리거되지 않았으면 다음 active `PLAN-{build_lane}-GNN.md`와 `CODE_REVIEW-{review_lane}-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가 사용자 결정으로 완료/PASS 해소되면 `USER_REVIEW.md`를 해소 상태로 갱신하고 `complete.log`를 작성한 뒤 task directory를 archive로 이동한다. + +## 계획 대비 변경 사항 + +- 테스트 시 heartbeat timeout의 빠른 유도를 위해 `server.go`에 `HeartbeatInterval`과 `HeartbeatWait` 필드를 Export하여 테스트 코드가 1초로 짧은 타임아웃 주기를 지정할 수 있도록 필드를 노출함. +- `TestEdgeServerHeartbeatTimeoutDisconnect` 테스트 시, 클라이언트 측의 `packets.HeartBeat` 수신 리스너를 제거(`client.RemoveListeners`)하여 서버 측의 HeartBeat 송신에 응답하지 않게 함으로써 heartbeat timeout을 안정적이고 빠르게 유도함. 최초 connection 시 automatically 송신된 heartbeat 이후, 클라이언트가 수신 대기를 무시하므로 바로 disconnected(failed)로 넘어가게 되며, 이에 맞춰 online event 확인 블록은 제외함. + +## 주요 설계 결정 + +- OTO agent 등록 시 generic node의 connected 기본값과 명시적으로 분리하여 `LifecycleAccepted` 상태로 최초 등록 처리. +- heartbeat timeout 테스트의 빠른 실행과 E2E smoke 관측을 위해 `Server` 구조체 내에 heartbeat interval/wait 필드를 Export함으로써 테스트용 override가 용이하도록 개선함. +- 클라이언트 측의 `RemoveListeners` 메서드를 사용하여 mock 커넥션을 짜지 않고도 실제 `toki.TcpClient` 연결 내에서 heartbeat 응답 거부를 성공적으로 구현함. + +## 사용자 리뷰 요청 + +- 상태: 없음 +- 사유 유형: 없음 +- 결정 필요: 없음 +- 차단 근거: 없음 +- 실행한 검증/명령: 없음 +- 자동 후속 불가 이유: 없음 +- 재개 조건: 없음 + +## 리뷰어를 위한 체크포인트 + +- OTO registration accepted와 online이 lifecycle state와 event에서 분리되는가? +- `LifecycleConnected` generic node 호환이 깨지지 않는가? +- HeartBeat listener가 첫 heartbeat에만 online event를 발행하고 중복 발행하지 않는가? +- failure reason metadata가 token 원문을 노출하지 않는가? + +## 검증 결과 + +### API-1 중간 검증 +```bash +$ go test -count=1 ./apps/edge/internal/node +``` +``` +ok iop/apps/edge/internal/node 0.008s +``` + +### API-2 중간 검증 +```bash +$ go test -count=1 ./apps/edge/internal/transport +``` +``` +ok iop/apps/edge/internal/transport 0.011s +``` + +### API-3 중간 검증 +```bash +$ go test -count=1 ./apps/edge/internal/transport ./apps/edge/internal/service ./packages/go/events +``` +``` +ok iop/apps/edge/internal/transport 2.018s +ok iop/apps/edge/internal/service 0.007s +? iop/packages/go/events [no test files] +``` + +### API-4 중간 검증 +```bash +$ go test -count=1 ./apps/edge/internal/transport +``` +``` +ok iop/apps/edge/internal/transport 2.018s +``` + +### 최종 검증 +```bash +$ go test -count=1 ./apps/edge/internal/transport ./apps/edge/internal/node ./apps/edge/internal/service ./packages/go/events +``` +``` +ok iop/apps/edge/internal/transport 2.018s +ok iop/apps/edge/internal/node 0.010s +ok iop/apps/edge/internal/service 0.007s +? iop/packages/go/events [no test files] +``` + +--- + +> **[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. + +## 코드리뷰 결과 + +- 종합 판정: WARN +- 차원별 평가: + - correctness: Pass + - completeness: Pass + - test coverage: Pass + - API contract: Pass + - code quality: Warn + - plan deviation: Pass + - verification trust: Pass +- 발견된 문제: + - Suggested: [registry_test.go](/config/workspace/iop2oto/apps/edge/internal/node/registry_test.go:132) and [integration_test.go](/config/workspace/iop2oto/apps/edge/internal/transport/integration_test.go:359) are not gofmt-clean. `gofmt -l apps/edge/internal/node/registry_test.go apps/edge/internal/transport/integration_test.go` returns both files, so run gofmt on the touched Go files and rerun the targeted verification. +- 다음 단계: WARN 후속으로 `PLAN-local-G01.md`와 `CODE_REVIEW-local-G01.md`를 작성한다. diff --git a/agent-task/archive/2026/06/m-oto-registration-online-smoke/01_iop_edge_lifecycle/code_review_local_G01_1.log b/agent-task/archive/2026/06/m-oto-registration-online-smoke/01_iop_edge_lifecycle/code_review_local_G01_1.log new file mode 100644 index 0000000..3f0590e --- /dev/null +++ b/agent-task/archive/2026/06/m-oto-registration-online-smoke/01_iop_edge_lifecycle/code_review_local_G01_1.log @@ -0,0 +1,133 @@ + + +# 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 by a user-only decision, user-owned external environment prerequisite, or scope conflict, fill `사용자 리뷰 요청` with evidence and stop with active files in place; code-review decides whether to write `USER_REVIEW.md`. Evidence gaps that a follow-up agent can close by rerunning commands or collecting artifacts are normal follow-up issues, not user-review blockers by themselves. +> 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-06-03 +task=m-oto-registration-online-smoke/01_iop_edge_lifecycle, plan=1, tag=REVIEW_API + +## Roadmap Targets + +- Milestone: `agent-roadmap/phase/automation-runtime-bridge/milestones/oto-registration-online-smoke.md` +- Task ids: + - `accepted-state`: OTO agent registration accepted 상태가 generic node와 구분되어 기록된다. + - `failure-reason`: unknown token, duplicate connection, heartbeat timeout이 서로 다른 실패 reason으로 확인된다. +- Completion mode: check-on-pass + +## 이 파일을 읽는 리뷰 에이전트에게 + +> **[REVIEW AGENT ONLY]** 아래 종결 절차는 코드리뷰 에이전트 전용이다. 구현 에이전트는 이 섹션을 실행하지 않는다. + +각 항목의 구현을 실제 소스 파일과 대조하고, `검증 결과` 섹션의 출력이 코드와 일치하는지 확인하세요. +리뷰 완료는 아래 순서까지 끝난 상태를 의미합니다. + +1. 판정을 append한다. +2. `CODE_REVIEW-local-G01.md` -> `code_review_local_G01_N.log`, `PLAN-local-G01.md` -> `plan_local_G01_M.log`로 아카이브한다. +3. PASS이면 `complete.log` 작성 후 active task 디렉터리를 `agent-task/archive/YYYY/MM/m-oto-registration-online-smoke/01_iop_edge_lifecycle/`로 이동한다. WARN/FAIL이면 user-review gate를 확인한 뒤 다음 active plan/review 파일 또는 `USER_REVIEW.md`를 작성한다. `USER_REVIEW.md`가 사용자 결정으로 완료/PASS 해소되면 `USER_REVIEW.md`를 해소 상태로 갱신하고 `complete.log` 작성 후 archive 이동한다. +4. PASS이고 task group이 `m-`이면 완료 이벤트 메타데이터를 보고한다. roadmap 상태 체크와 `update-roadmap` 호출은 런타임 책임이다. +5. 적용 가능한 `코드리뷰 전용 체크리스트` 항목을 최종 `.log` 위치에서 체크한 뒤 보고한다. + +--- + +## 구현 항목별 완료 여부 + +| 항목 | 완료 여부 | +|------|---------| +| [REVIEW_API-1] gofmt Cleanup | [x] | + +## 구현 체크리스트 + +- [x] `gofmt -w apps/edge/internal/node/registry_test.go apps/edge/internal/transport/integration_test.go`를 실행한다. +- [x] `gofmt -l apps/edge/internal/node/registry.go apps/edge/internal/node/registry_test.go apps/edge/internal/transport/server.go apps/edge/internal/transport/integration_test.go packages/go/events/events.go`가 빈 출력인지 확인한다. +- [x] `go test -count=1 ./apps/edge/internal/transport ./apps/edge/internal/node ./apps/edge/internal/service ./packages/go/events`를 실행한다. +- [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_G01_N.log`로 아카이브한다. +- [x] active `PLAN-*-G??.md`를 `plan_local_G01_M.log`로 아카이브한다. +- [x] `.gitignore`의 Agent-Ops 관리 block이 `agent-task/**/*.md`와 `agent-task/**/*.log`를 unignore하고 `agent-roadmap/current.md`를 ignore하는지 확인한다. +- [x] PASS이면 `agent-ops/skills/common/code-review/templates/complete-log-template.md` 기준으로 `complete.log`를 작성하고 active `.md` 파일을 남기지 않는다. +- [x] PASS이면 active task 디렉터리 `agent-task/m-oto-registration-online-smoke/01_iop_edge_lifecycle/`를 `agent-task/archive/YYYY/MM/m-oto-registration-online-smoke/01_iop_edge_lifecycle/`로 이동하고 최종 archive 경로에서 이 체크리스트를 갱신한다. +- [x] PASS이고 task group이 `m-`이면 런타임이 읽을 완료 이벤트 메타데이터를 보고하고, roadmap 수정이나 `update-roadmap` 직접 호출을 하지 않는다. +- [x] PASS split 작업이면 이동 후 빈 active parent `agent-task/m-oto-registration-online-smoke/`를 제거하거나, 남은 sibling/file이 있어 유지했다고 확인한다. +- [ ] WARN/FAIL이고 user-review gate가 트리거되지 않았으면 다음 active `PLAN-{build_lane}-GNN.md`와 `CODE_REVIEW-{review_lane}-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가 사용자 결정으로 완료/PASS 해소되면 `USER_REVIEW.md`를 해소 상태로 갱신하고 `complete.log`를 작성한 뒤 task directory를 archive로 이동한다. + +## 계획 대비 변경 사항 + +계획 그대로 `apps/edge/internal/node/registry_test.go`와 `apps/edge/internal/transport/integration_test.go`에 `gofmt -w`를 적용했다. source behavior, test assertions, event constants, registry/transport logic은 변경하지 않았다. + +## 주요 설계 결정 + +formatting-only 변경이므로 설계 결정은 없다. + +## 사용자 리뷰 요청 + +- 상태: 없음 +- 사유 유형: 없음 +- 결정 필요: 없음 +- 차단 근거: 없음 +- 실행한 검증/명령: 없음 +- 자동 후속 불가 이유: 없음 +- 재개 조건: 없음 + +## 리뷰어를 위한 체크포인트 + +- `gofmt -l`이 빈 출력인가? 예, 빈 출력 확인. +- formatting-only 변경 외에 동작 변경이 끼어들지 않았는가? 예, formatting-only. +- targeted Go 테스트가 통과했는가? 예. + +## 검증 결과 + +### gofmt -l (변경된 파일들 기준) + +``` +$ gofmt -l apps/edge/internal/node/registry.go apps/edge/internal/node/registry_test.go apps/edge/internal/transport/server.go apps/edge/internal/transport/integration_test.go packages/go/events/events.go +(no output) +``` + +### go test -count=1 + +``` +$ go test -count=1 ./apps/edge/internal/transport ./apps/edge/internal/node ./apps/edge/internal/service ./packages/go/events +ok iop/apps/edge/internal/transport 2.026s +ok iop/apps/edge/internal/node 0.006s +ok iop/apps/edge/internal/service 0.008s +? iop/packages/go/events [no test files] +``` + +--- + +> **[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. + +## 코드리뷰 결과 + +- 종합 판정: PASS +- 차원별 평가: + - correctness: Pass + - completeness: Pass + - test coverage: Pass + - API contract: Pass + - code quality: Pass + - plan deviation: Pass + - verification trust: Pass +- 발견된 문제: 없음 +- 다음 단계: PASS 종결. `complete.log` 작성 후 task directory를 archive로 이동한다. diff --git a/agent-task/archive/2026/06/m-oto-registration-online-smoke/01_iop_edge_lifecycle/complete.log b/agent-task/archive/2026/06/m-oto-registration-online-smoke/01_iop_edge_lifecycle/complete.log new file mode 100644 index 0000000..3434fa7 --- /dev/null +++ b/agent-task/archive/2026/06/m-oto-registration-online-smoke/01_iop_edge_lifecycle/complete.log @@ -0,0 +1,45 @@ +# Complete - m-oto-registration-online-smoke/01_iop_edge_lifecycle + +## 완료 일시 + +2026-06-03 + +## 요약 + +OTO Edge lifecycle foundation subtask completed after 2 review loops; final verdict PASS. + +## 루프 이력 + +| Plan | Review | Verdict | 메모 | +|------|--------|---------|------| +| `plan_cloud_G07_0.log` | `code_review_cloud_G07_0.log` | WARN | Lifecycle/event implementation and verification passed, but touched Go test files were not gofmt-clean. | +| `plan_local_G01_1.log` | `code_review_local_G01_1.log` | PASS | gofmt cleanup completed; formatter, whitespace, and targeted Go verification passed. | + +## 구현/정리 내용 + +- Added accepted/online/failed lifecycle states while preserving generic node connected behavior. +- Added OTO accepted registration metadata, first-heartbeat online transition, and distinct registration/heartbeat failure reason metadata. +- Added registry, transport, and service-adjacent tests for lifecycle state preservation and failure reason observability. +- Applied gofmt cleanup for the follow-up WARN finding. + +## 최종 검증 + +- `gofmt -l apps/edge/internal/node/registry.go apps/edge/internal/node/registry_test.go apps/edge/internal/transport/server.go apps/edge/internal/transport/integration_test.go packages/go/events/events.go` - PASS; empty output. +- `git diff --check` - PASS; empty output. +- `go test -count=1 ./apps/edge/internal/transport ./apps/edge/internal/node ./apps/edge/internal/service ./packages/go/events` - PASS; `ok` for transport, node, service and `[no test files]` for events. + +## Roadmap Completion + +- Milestone: `agent-roadmap/phase/automation-runtime-bridge/milestones/oto-registration-online-smoke.md` +- Completed task ids: + - `accepted-state`: PASS; evidence=`agent-task/archive/2026/06/m-oto-registration-online-smoke/01_iop_edge_lifecycle/plan_cloud_G07_0.log`, `agent-task/archive/2026/06/m-oto-registration-online-smoke/01_iop_edge_lifecycle/code_review_cloud_G07_0.log`, `agent-task/archive/2026/06/m-oto-registration-online-smoke/01_iop_edge_lifecycle/plan_local_G01_1.log`, `agent-task/archive/2026/06/m-oto-registration-online-smoke/01_iop_edge_lifecycle/code_review_local_G01_1.log`; verification=`go test -count=1 ./apps/edge/internal/transport ./apps/edge/internal/node ./apps/edge/internal/service ./packages/go/events` + - `failure-reason`: PASS; evidence=`agent-task/archive/2026/06/m-oto-registration-online-smoke/01_iop_edge_lifecycle/plan_cloud_G07_0.log`, `agent-task/archive/2026/06/m-oto-registration-online-smoke/01_iop_edge_lifecycle/code_review_cloud_G07_0.log`, `agent-task/archive/2026/06/m-oto-registration-online-smoke/01_iop_edge_lifecycle/plan_local_G01_1.log`, `agent-task/archive/2026/06/m-oto-registration-online-smoke/01_iop_edge_lifecycle/code_review_local_G01_1.log`; verification=`go test -count=1 ./apps/edge/internal/transport ./apps/edge/internal/node ./apps/edge/internal/service ./packages/go/events` +- Not completed task ids: 없음 + +## 잔여 Nit + +- 없음 + +## 후속 작업 + +- 없음 diff --git a/agent-task/archive/2026/06/m-oto-registration-online-smoke/01_iop_edge_lifecycle/plan_cloud_G07_0.log b/agent-task/archive/2026/06/m-oto-registration-online-smoke/01_iop_edge_lifecycle/plan_cloud_G07_0.log new file mode 100644 index 0000000..5cf66c9 --- /dev/null +++ b/agent-task/archive/2026/06/m-oto-registration-online-smoke/01_iop_edge_lifecycle/plan_cloud_G07_0.log @@ -0,0 +1,289 @@ + + +# OTO Edge Lifecycle State Foundation + +## 이 파일을 읽는 구현 에이전트에게 + +`CODE_REVIEW-*-G??.md`의 구현 에이전트 소유 섹션을 채우는 것까지가 구현입니다. 코드를 바꾼 뒤 검증을 실행하고, 실제 변경 내용과 stdout/stderr를 리뷰 stub에 기록한 상태로 멈추세요. 종결 판정, log rename, `complete.log`, archive 이동은 code-review 스킬 전용입니다. 구현 중 사용자 결정, 사용자 소유 외부 환경, 범위 충돌이 없으면 `사용자 리뷰 요청`을 만들지 않습니다. + +## 배경 + +현재 iop Edge registry는 등록 직후 `LifecycleConnected`를 기본값으로 넣는다. OTO Milestone은 registration accepted와 최초 heartbeat 도착을 분리해 accepted/online/failed 상태와 실패 reason을 확인해야 한다. 이 subtask는 iop Edge 내부 상태와 event metadata를 먼저 정리해 후속 OTO smoke가 믿을 수 있는 기준을 만든다. + +## 사용자 리뷰 요청 흐름 + +구현 중 사용자-only blocker가 있으면 active `CODE_REVIEW-*-G??.md`의 `사용자 리뷰 요청` 섹션을 실제 내용으로 교체하고 멈춘다. code-review가 요청 정당성을 검증하고 실제 `USER_REVIEW.md` 작성 여부를 결정한다. + +## Roadmap Targets + +- Milestone: `agent-roadmap/phase/automation-runtime-bridge/milestones/oto-registration-online-smoke.md` +- Task ids: + - `accepted-state`: OTO agent registration accepted 상태가 generic node와 구분되어 기록된다. + - `failure-reason`: unknown token, duplicate connection, heartbeat timeout이 서로 다른 실패 reason으로 확인된다. +- Completion mode: check-on-pass + +## 분석 결과 + +### 읽은 파일 + +- `agent-roadmap/current.md` +- `agent-roadmap/phase/automation-runtime-bridge/PHASE.md` +- `agent-roadmap/phase/automation-runtime-bridge/milestones/oto-registration-online-smoke.md` +- `agent-ops/rules/project/domain/edge/rules.md` +- `agent-ops/rules/project/domain/platform-common/rules.md` +- `agent-ops/rules/project/domain/testing/rules.md` +- `agent-test/local/rules.md` +- `agent-test/local/edge-smoke.md` +- `agent-test/local/platform-common-smoke.md` +- `apps/edge/internal/transport/server.go` +- `apps/edge/internal/transport/server_test.go` +- `apps/edge/internal/transport/integration_test.go` +- `apps/edge/internal/transport/heartbeat_test.go` +- `apps/edge/internal/node/registry.go` +- `apps/edge/internal/node/registry_test.go` +- `apps/edge/internal/node/store.go` +- `apps/edge/internal/node/store_test.go` +- `apps/edge/internal/service/service.go` +- `apps/edge/internal/service/service_test.go` +- `apps/edge/internal/events/bus.go` +- `packages/go/events/events.go` +- `../proto-socket/go/base_client.go` +- `../proto-socket/go/tcp_client.go` +- `../proto-socket/go/communicator.go` + +### 테스트 환경 규칙 + +- test_env: `local`. +- iop `agent-test/local/rules.md` 존재 및 읽음. +- 적용 profile: `agent-test/local/edge-smoke.md`, `agent-test/local/platform-common-smoke.md`. +- 적용 명령: `go test -count=1 ./apps/edge/internal/transport ./apps/edge/internal/node ./apps/edge/internal/service ./packages/go/events`. +- command preflight 결과: 위 명령은 현재 기준 통과했고 `packages/go/events`는 `[no test files]`다. + +### 테스트 커버리지 공백 + +- accepted 상태: `TestEdgeServerOTORegistrationKind`는 OTO kind와 `LifecycleConnected`만 확인한다. accepted와 online 분리는 미커버. +- first heartbeat online: iop transport에는 heartbeat listener가 없다. `proto_socket` 내부 `packets.HeartBeat` listener를 추가로 붙일 수 있음은 확인했지만 Edge 상태 전환 테스트는 없다. +- failure reason: duplicate reason 테스트는 있으나 unknown token event/metadata는 없고, heartbeat timeout은 disconnect metadata만 존재한다. +- service snapshot: `ListNodeSnapshots`는 lifecycle을 전달하지만 accepted/online/failed 값 전환은 미커버. + +### 심볼 참조 + +- 변경/추가 후보: `LifecycleConnected` 참조는 `apps/edge/internal/node/registry.go`, `apps/edge/internal/node/registry_test.go`, `apps/edge/internal/service/service_test.go`, `apps/edge/internal/transport/server_test.go`에 있다. +- 제거하지 말 것: `LifecycleConnected`는 generic node 호환을 위해 유지한다. + +### 분할 판단 + +- split 정책 평가 완료. +- shared task group: `agent-task/m-oto-registration-online-smoke/`. +- `01_iop_edge_lifecycle`: iop Edge lifecycle/event foundation. 의존 없음. +- `02+01_oto_agent_session`: OTO Dart agent session은 Edge lifecycle 기준이 필요하므로 `01`에 의존. +- `03+01,02_smoke_evidence`: 실제 smoke/evidence는 Edge와 OTO 양쪽 구현이 필요하므로 `01`, `02`에 의존. + +### 범위 결정 근거 + +- proto 파일은 변경하지 않는다. `EdgeNodeEvent.type`, `reason`, `metadata` 문자열로 충분하다. +- `../proto-socket/go`는 변경하지 않는다. HeartBeat listener 추가는 소비자인 Edge transport에서 처리한다. +- Control Plane/Client UI 반영은 Milestone 범위 제외다. +- OTO Dart agent 유지 연결은 다음 subtask에서 다룬다. + +### 빌드 등급 + +- build: `cloud-G07`, review: `cloud-G07`. proto-socket heartbeat와 Edge registry lifecycle이 결합된 protocol-adjacent 상태 전환이라 신중한 리뷰가 필요하다. + +## 구현 체크리스트 + +- [ ] OTO agent 등록 직후 lifecycle을 accepted로 기록하고 generic node 기본 connected 동작은 유지한다. +- [ ] 첫 inbound `packets.HeartBeat`를 감지하면 OTO agent lifecycle을 online으로 한 번만 전환하고 node event를 발행한다. +- [ ] unknown token, duplicate connection, heartbeat timeout/transport close가 서로 다른 reason/metadata로 관측되게 한다. +- [ ] service snapshot과 관련 테스트가 accepted/online/failed lifecycle 값을 보존하는지 검증한다. +- [ ] `go test -count=1 ./apps/edge/internal/transport ./apps/edge/internal/node ./apps/edge/internal/service ./packages/go/events`를 실행한다. +- [ ] CODE_REVIEW-*-G??.md의 구현 에이전트 소유 섹션을 실제 구현 내용과 검증 출력으로 채운다. 이 항목이 완료되기 전에는 구현이 완료된 것이 아니다. + +### [API-1] Lifecycle Constants and Registry Update + +문제: [registry.go](/config/workspace/iop2oto/apps/edge/internal/node/registry.go:15)는 `LifecycleConnected`만 정의하고, [registry.go](/config/workspace/iop2oto/apps/edge/internal/node/registry.go:54)는 빈 lifecycle을 모두 connected로 채운다. + +해결 방법: `LifecycleAccepted`, `LifecycleOnline`, `LifecycleFailed`를 추가하고 `UpdateLifecycle(nodeID, state string) bool`를 Registry에 추가한다. `LifecycleConnected`는 generic node 호환 alias로 유지한다. + +Before: + +```go +// apps/edge/internal/node/registry.go:15 +const LifecycleConnected = "connected" +``` + +After: + +```go +const ( + LifecycleAccepted = "accepted" + LifecycleOnline = "online" + LifecycleFailed = "failed" + LifecycleConnected = "connected" +) +``` + +수정 파일 및 체크리스트: + +- [ ] `apps/edge/internal/node/registry.go` lifecycle constants 추가. +- [ ] `apps/edge/internal/node/registry.go` `UpdateLifecycle` 추가. +- [ ] `apps/edge/internal/node/registry_test.go` default lifecycle과 update lifecycle 테스트 추가/수정. + +테스트 작성: `TestRegistryUpdateLifecycle` 추가. unknown node는 false, existing node는 state 변경과 snapshot 보존을 확인한다. + +중간 검증: + +```bash +go test -count=1 ./apps/edge/internal/node +``` + +예상 결과: exit code 0. + +### [API-2] Registration Accepted Versus Online + +문제: [server.go](/config/workspace/iop2oto/apps/edge/internal/transport/server.go:164)는 OTO entry를 만들 때 lifecycle을 지정하지 않아 registry default가 connected가 된다. [server.go](/config/workspace/iop2oto/apps/edge/internal/transport/server.go:205)는 등록 event에 lifecycle metadata도 남기지 않는다. + +해결 방법: OTO agent는 registration response 직전 `LifecycleAccepted`, generic node는 기존 `LifecycleConnected`로 등록한다. 등록 event metadata에 `agent_kind`와 `lifecycle_state`를 넣는다. + +Before: + +```go +// apps/edge/internal/transport/server.go:164 +entry := &edgenode.NodeEntry{ + NodeID: rec.ID, + Alias: rec.Alias, + AgentKind: rec.AgentKind, + Client: client, + Index: rec.Index, + HasIndex: true, +} +``` + +After: + +```go +lifecycle := edgenode.LifecycleConnected +if rec.AgentKind == config.AgentKindOTOAgent { + lifecycle = edgenode.LifecycleAccepted +} +entry := &edgenode.NodeEntry{ + NodeID: rec.ID, + Alias: rec.Alias, + AgentKind: rec.AgentKind, + LifecycleState: lifecycle, + Client: client, + Index: rec.Index, + HasIndex: true, +} +``` + +수정 파일 및 체크리스트: + +- [ ] `apps/edge/internal/transport/server.go` lifecycle 결정 helper 추가. +- [ ] `apps/edge/internal/transport/server.go` registration event metadata 추가. +- [ ] `apps/edge/internal/transport/server_test.go` OTO registration lifecycle 기대값을 accepted로 변경. +- [ ] `apps/edge/internal/transport/integration_test.go` generic node는 connected 유지 검증. + +테스트 작성: `TestEdgeServerOTORegistrationKind`를 accepted 기대값으로 수정하고 connected generic integration 회귀를 유지한다. + +중간 검증: + +```bash +go test -count=1 ./apps/edge/internal/transport +``` + +예상 결과: exit code 0. + +### [API-3] First Heartbeat Online Transition + +문제: `../proto-socket/go/tcp_client.go:40`에서 내부 `packets.HeartBeat` listener가 등록되지만 Edge transport는 별도 listener가 없어 첫 heartbeat를 상태 전환으로 관측하지 않는다. + +해결 방법: `git.toki-labs.com/toki/proto-socket/go/packets`를 import하고 OTO registration 이후 `toki.AddListenerTyped[*packets.HeartBeat]`를 추가한다. 첫 heartbeat에서만 registry lifecycle을 `online`으로 바꾸고 `node.online` event를 emit한다. + +Before: + +```go +// apps/edge/internal/transport/server.go:172 +toki.AddListenerTyped[*iop.EdgeNodeEvent](&client.Communicator, func(e *iop.EdgeNodeEvent) { + if e.NodeId == "" { + e.NodeId = rec.ID + } +``` + +After: + +```go +if rec.AgentKind == config.AgentKindOTOAgent { + var onlineOnce sync.Once + toki.AddListenerTyped[*packets.HeartBeat](&client.Communicator, func(*packets.HeartBeat) { + onlineOnce.Do(func() { + if s.registry.UpdateLifecycle(rec.ID, edgenode.LifecycleOnline) { + s.emitNodeEvent(events.NewEdgeNodeEvent(events.SourceEdge, events.TypeNodeOnline, rec.ID, rec.Alias, events.ReasonFirstHeartbeat, lifecycleMetadata(rec, edgenode.LifecycleOnline))) + } + }) + }) +} +``` + +수정 파일 및 체크리스트: + +- [ ] `packages/go/events/events.go` `TypeNodeOnline`, `ReasonFirstHeartbeat`, lifecycle metadata key 상수 추가. +- [ ] `apps/edge/internal/transport/server.go` heartbeat listener 추가. +- [ ] `apps/edge/internal/transport/server_test.go` 또는 `integration_test.go`에서 raw heartbeat frame을 보내 online 전환과 event 확인. +- [ ] `apps/edge/internal/service/service_test.go` snapshot이 online lifecycle을 전달하는지 확인. + +테스트 작성: `TestEdgeServerOTOFirstHeartbeatMarksOnline` 추가. OTO register 후 entry lifecycle accepted, heartbeat frame 송신 후 lifecycle online 및 `node.online` event를 확인한다. + +중간 검증: + +```bash +go test -count=1 ./apps/edge/internal/transport ./apps/edge/internal/service ./packages/go/events +``` + +예상 결과: exit code 0. + +### [API-4] Failure Reasons + +문제: [server.go](/config/workspace/iop2oto/apps/edge/internal/transport/server.go:137)는 unknown token을 response reason으로만 돌려주고 lifecycle event를 남기지 않는다. duplicate는 OTO/generic reason이 다르지만 event evidence가 없다. heartbeat timeout은 disconnect metadata로만 있다. + +해결 방법: rejected registration은 registry에 넣지 않되 `node.registration_failed` event를 emit한다. metadata에는 `failure_reason=unknown_token|duplicate_connection|heartbeat_timeout`, `agent_kind` 가능한 경우, token prefix만 포함한다. heartbeat timeout disconnect는 lifecycle failed로 전환하고 metadata에 lifecycle/failure reason을 보강한다. + +수정 파일 및 체크리스트: + +- [ ] `packages/go/events/events.go` `TypeNodeRegistrationFailed`, `ReasonUnknownToken`, `ReasonDuplicateConnection`, metadata key 상수 추가. +- [ ] `apps/edge/internal/transport/server.go` unknown token/duplicate reject event emit. +- [ ] `apps/edge/internal/transport/server.go` disconnect path에서 heartbeat timeout이면 lifecycle failed로 전환. +- [ ] `apps/edge/internal/transport/server_test.go` unknown token, duplicate OTO, heartbeat timeout metadata 테스트 추가. + +테스트 작성: unknown token reject event, duplicate reject event, heartbeat timeout disconnect metadata를 각각 검증한다. heartbeat timeout은 interval 조절이 어려우면 `transportDisconnectMetadata`/disconnect info helper 단위 테스트로 시작하되 integration TODO를 남기지 말고 실제 observable event path를 검증한다. + +중간 검증: + +```bash +go test -count=1 ./apps/edge/internal/transport +``` + +예상 결과: exit code 0. + +## 수정 파일 요약 + +| 파일 | 항목 | +|------|------| +| `apps/edge/internal/node/registry.go` | API-1 | +| `apps/edge/internal/node/registry_test.go` | API-1 | +| `apps/edge/internal/transport/server.go` | API-2, API-3, API-4 | +| `apps/edge/internal/transport/server_test.go` | API-2, API-3, API-4 | +| `apps/edge/internal/transport/integration_test.go` | API-2 | +| `apps/edge/internal/service/service_test.go` | API-3 | +| `packages/go/events/events.go` | API-3, API-4 | +| `packages/go/events/events_test.go` | API-3, API-4 | + +## 최종 검증 + +```bash +go test -count=1 ./apps/edge/internal/transport ./apps/edge/internal/node ./apps/edge/internal/service ./packages/go/events +``` + +예상 결과: exit code 0. Go test cache output은 허용하지 않는다. + +모든 코드 변경 완료 후 반드시 `CODE_REVIEW-*-G??.md`의 구현 에이전트 소유 섹션을 채운다. 이 파일 작성이 구현의 마지막 단계다. diff --git a/agent-task/archive/2026/06/m-oto-registration-online-smoke/01_iop_edge_lifecycle/plan_local_G01_1.log b/agent-task/archive/2026/06/m-oto-registration-online-smoke/01_iop_edge_lifecycle/plan_local_G01_1.log new file mode 100644 index 0000000..4f9792a --- /dev/null +++ b/agent-task/archive/2026/06/m-oto-registration-online-smoke/01_iop_edge_lifecycle/plan_local_G01_1.log @@ -0,0 +1,58 @@ + + +# Follow-up Plan - REVIEW_API + +## 이 파일을 읽는 구현 에이전트에게 + +`CODE_REVIEW-*-G??.md`의 구현 에이전트 소유 섹션을 채우는 것까지가 구현입니다. 코드를 바꾼 뒤 검증을 실행하고, 실제 변경 내용과 stdout/stderr를 리뷰 stub에 기록한 상태로 멈추세요. 종결 판정, log rename, `complete.log`, archive 이동은 code-review 스킬 전용입니다. 구현 중 사용자 결정, 사용자 소유 외부 환경, 범위 충돌이 없으면 `사용자 리뷰 요청`을 만들지 않습니다. + +## 배경 + +이전 리뷰 `code_review_cloud_G07_0.log`는 lifecycle/event 구현과 targeted/Edge/package 회귀 검증은 통과했지만, `gofmt -l`가 변경된 Go test 파일 두 개를 반환해 WARN으로 종료했다. 이번 후속은 동작 변경 없이 formatting을 정리하고 같은 targeted 검증을 재실행하는 작업이다. + +## 사용자 리뷰 요청 흐름 + +구현 중 사용자-only blocker가 있으면 active `CODE_REVIEW-*-G??.md`의 `사용자 리뷰 요청` 섹션을 실제 내용으로 교체하고 멈춘다. code-review가 요청 정당성을 검증하고 실제 `USER_REVIEW.md` 작성 여부를 결정한다. + +## Roadmap Targets + +- Milestone: `agent-roadmap/phase/automation-runtime-bridge/milestones/oto-registration-online-smoke.md` +- Task ids: + - `accepted-state`: OTO agent registration accepted 상태가 generic node와 구분되어 기록된다. + - `failure-reason`: unknown token, duplicate connection, heartbeat timeout이 서로 다른 실패 reason으로 확인된다. +- Completion mode: check-on-pass + +## 범위 결정 근거 + +- source behavior, test assertions, event constants, registry/transport logic은 변경하지 않는다. +- 이전 리뷰의 Suggested issue만 해소한다. +- `agent-roadmap/**`는 수정하지 않는다. + +## 구현 체크리스트 + +- [ ] `gofmt -w apps/edge/internal/node/registry_test.go apps/edge/internal/transport/integration_test.go`를 실행한다. +- [ ] `gofmt -l apps/edge/internal/node/registry.go apps/edge/internal/node/registry_test.go apps/edge/internal/transport/server.go apps/edge/internal/transport/integration_test.go packages/go/events/events.go`가 빈 출력인지 확인한다. +- [ ] `go test -count=1 ./apps/edge/internal/transport ./apps/edge/internal/node ./apps/edge/internal/service ./packages/go/events`를 실행한다. +- [ ] CODE_REVIEW-*-G??.md의 구현 에이전트 소유 섹션을 실제 변경 내용과 검증 출력으로 채운다. 이 항목이 완료되기 전에는 구현이 완료된 것이 아니다. + +### [REVIEW_API-1] gofmt Cleanup + +문제: `gofmt -l`가 [registry_test.go](/config/workspace/iop2oto/apps/edge/internal/node/registry_test.go:132)와 [integration_test.go](/config/workspace/iop2oto/apps/edge/internal/transport/integration_test.go:359)를 반환한다. + +해결 방법: touched Go files 중 formatter가 지적한 두 파일에 `gofmt -w`를 적용한다. + +수정 파일 및 체크리스트: + +- [ ] `apps/edge/internal/node/registry_test.go` gofmt 적용. +- [ ] `apps/edge/internal/transport/integration_test.go` gofmt 적용. + +테스트 작성: 없음. formatting-only 후속이다. + +중간 검증: + +```bash +gofmt -l apps/edge/internal/node/registry.go apps/edge/internal/node/registry_test.go apps/edge/internal/transport/server.go apps/edge/internal/transport/integration_test.go packages/go/events/events.go +go test -count=1 ./apps/edge/internal/transport ./apps/edge/internal/node ./apps/edge/internal/service ./packages/go/events +``` + +예상 결과: `gofmt -l`은 빈 출력, `go test`는 exit code 0. diff --git a/agent-task/archive/2026/06/m-oto-registration-online-smoke/02+01_oto_agent_session/code_review_cloud_G06_0.log b/agent-task/archive/2026/06/m-oto-registration-online-smoke/02+01_oto_agent_session/code_review_cloud_G06_0.log new file mode 100644 index 0000000..3b979ba --- /dev/null +++ b/agent-task/archive/2026/06/m-oto-registration-online-smoke/02+01_oto_agent_session/code_review_cloud_G06_0.log @@ -0,0 +1,212 @@ + + +# 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 by a user-only decision, user-owned external environment prerequisite, or scope conflict, fill `사용자 리뷰 요청` with evidence and stop with active files in place; code-review decides whether to write `USER_REVIEW.md`. +> Finalization (`코드리뷰 결과`, log rename, `complete.log`, archive moves, `코드리뷰 전용 체크리스트`) is review-agent-only. + +## 개요 + +date=2026-06-03 +task=m-oto-registration-online-smoke/02+01_oto_agent_session, plan=0, tag=API + +## 이 파일을 읽는 리뷰 에이전트에게 + +> **[REVIEW AGENT ONLY]** 아래 종결 절차는 코드리뷰 에이전트 전용이다. 구현 에이전트는 이 섹션을 실행하지 않는다. + +각 항목의 구현을 실제 소스 파일과 대조하고, `검증 결과` 섹션의 출력이 코드와 일치하는지 확인하세요. +PASS이면 `complete.log` 작성 후 `agent-task/archive/YYYY/MM/m-oto-registration-online-smoke/02+01_oto_agent_session/`로 이동합니다. roadmap 수정은 런타임 책임입니다. + +--- + +## 구현 항목별 완료 여부 + +| 항목　　　　　　　　　　　　　　　　| 완료 여부 | +| -------------------------------------| -----------| +| [API-1] Dependency Alignment　　　　| [x]　　　 | +| [API-2] Persistent Edge Session API | [x]　　　 | +| [API-3] Runner Keeps Agent Alive　　| [x]　　　 | + +## 구현 체크리스트 + +- [x] `../oto/pubspec.yaml`의 protobuf constraint를 path dependency `proto_socket`과 맞춰 tests가 resolve되게 한다. (`^3.1.0` → `^6.0.0`) +- [x] 기존 `register()` 호환을 유지하면서 close하지 않는 OTO registration session API를 추가한다. (`EdgeAgentSession` + `EdgeRegistrationClient.openSession()`, 기존 `register()`는 wrapper로 보존) +- [x] `DefaultAgentRunner.run()`이 registration 성공 후 agent connection을 유지하고 종료 신호 또는 cancellation에서 닫도록 조정한다. (SIGINT/SIGTERM 대기, `finally`에서 session close) +- [x] OTO agent unit tests가 persistent session API와 rejected registration behavior를 검증하게 한다. +- [x] `cd ../oto && dart test test/oto_agent_registration_test.dart test/oto_agent_config_test.dart`를 실행한다. (12 tests passed, exit 0) +- [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_G06_N.log`로 아카이브한다. +- [x] active `PLAN-*-G??.md`를 `plan_cloud_G06_M.log`로 아카이브한다. +- [x] `.gitignore`의 Agent-Ops 관리 block이 `agent-task/**/*.md`와 `agent-task/**/*.log`를 unignore하고 `agent-roadmap/current.md`를 ignore하는지 확인한다. +- [ ] PASS이면 `complete.log`를 작성하고 active `.md` 파일을 남기지 않는다. +- [ ] PASS이면 active task 디렉터리 `agent-task/m-oto-registration-online-smoke/02+01_oto_agent_session/`를 archive로 이동한다. +- [ ] PASS split 작업이면 이동 후 parent에 남은 sibling/file이 있어 유지했다고 확인한다. +- [x] WARN/FAIL이고 user-review gate가 트리거되지 않았으면 다음 active `PLAN-{build_lane}-GNN.md`와 `CODE_REVIEW-{review_lane}-GNN.md`를 작성하고 `complete.log`를 작성하지 않는다. + +## 계획 대비 변경 사항 + +- **generated protobuf 코드 재생성 (계획 범위 밖에서 추가 발생).** 계획 API-1은 `pubspec.yaml`의 `protobuf` constraint 상향과 `dart pub get`만 예상했으나, constraint를 `^6.0.0`으로 올리자 기존 generated `*.pb.dart`가 protobuf 3.x 시절 API(`$pb.PbList()` unnamed ctor, `toProto3Json`/`fromProto3Json` named param)를 사용해 `dart analyze`에서 49개 error가 발생했다. agent/framework domain rule이 "generated protobuf 파일은 직접 수정하지 않고 원본 proto와 생성 절차를 확인해 재생성한다"를 명시하므로, 손으로 고치지 않고 원본 proto(`../iop/proto/iop/runtime.proto`)에서 `protoc` + `protoc-gen-dart`(protoc_plugin 25.0.0, protobuf 6.x 호환)로 재생성했다. 이는 "tests가 resolve되게 한다"는 API-1 목표를 충족하기 위한 dependency-solving 작업이며, 사용자 결정이 필요한 blocker가 아니다. +- 재생성된 코드 차이: + - well-known type `google/protobuf/struct`를 로컬 사본이 아니라 `package:protobuf/well_known_types/...`에서 import하도록 바뀌어, 로컬 `google/protobuf/struct.*` 사본이 더 이상 generated set에 포함되지 않는다. + - 새 plugin은 `*.pbserver.dart`를 생성하지 않는다. + - 위 두 가지로 orphan이 된 파일(로컬 `google/protobuf/*` 4개, `iop/runtime.pbserver.dart`)을 `lib/oto/agent/`와 `test/support/` 양쪽에서 삭제했다. 삭제 전 `grep`으로 외부 참조가 `edge_registration_client.dart`(lib runtime.pb.dart 하나)뿐이고 삭제 대상은 어디에서도 import되지 않음을 확인했다. +- `test/oto_iop_connection_smoke_test.dart`는 변경하지 않았다. `EdgeRegistrationClient().register()`(one-shot wrapper)를 그대로 사용하므로 기존 호환이 유지된다. +- `command_agent.dart`는 변경하지 않았다. runner 호출 시그니처(`runner.run(config)`)가 그대로이고 daemon wait는 runner 내부에 캡슐화되어 call-site 조정이 불필요했다. + +## 주요 설계 결정 + +- **`EdgeAgentSession` 추상 타입 도입.** persistent 연결을 표현하는 인터페이스(`result` getter + `close()`)를 두고, 실 구현 `_OtoEdgeAgentSession`은 `_OtoIopClient`(이미 heartbeat mixin 보유)를 감싼다. abstract로 둔 이유는 runner 단위 테스트가 `FakeEdgeAgentSession`으로 close 호출 여부를 hang 없이 검증할 수 있게 하기 위함이다. +- **`register()`는 `openSession()` 위의 one-shot wrapper로 보존.** `openSession()`이 connect+register 후 연결을 유지하고, `register()`는 그 위에서 `finally`로 `session.close()`를 호출한다. 기존 unit test/smoke test API와 runner injection 호환을 깨지 않는다. +- **failure leak 방지.** 연결/등록 단계 예외는 `openSession()`의 `catch`에서 `client.close()` 후 rethrow한다. registration rejected는 session을 정상 반환하되 runner의 `finally`가 항상 `session.close()`를 호출하므로, accepted/rejected/예외 모든 경로에서 socket leak이 없다. `close()`는 `BaseClient._closeCalled` 가드로 idempotent하다. +- **runner의 test-injectable wait hook.** `DefaultAgentRunner`에 `Future Function()? waitForShutdown`을 추가했다. 주입 없으면 production 기본값으로 `SIGINT`/`SIGTERM`을 감시하다 신호 수신 시 종료하고 구독을 정리한다(daemon 동작). 테스트는 즉시 완료 함수나 `Completer`를 주입해 "유지 중 미close → 신호 후 close"를 검증한다. + +## 사용자 리뷰 요청 + +_기본값은 `없음`이다. 구현 중 사용자 결정, 사용자 소유 외부 환경/secret/서비스 준비, 또는 계획 범위 변경 없이는 안전하게 진행할 수 없으면 아래 항목을 실제 내용으로 교체하고, 구현을 중단한 뒤 active 파일을 그대로 둔 채 리뷰를 요청한다. 후속 에이전트가 명령 재실행이나 산출물 수집으로 해소할 수 있는 검증 증거 공백만으로는 사용자 리뷰 요청을 작성하지 않는다._ + +- 상태: 없음 +- 사유 유형: 없음 +- 결정 필요: 없음 +- 차단 근거: 없음 +- 실행한 검증/명령: 없음 +- 자동 후속 불가 이유: 없음 +- 재개 조건: 없음 + +## 리뷰어를 위한 체크포인트 + +- 기존 one-shot `register()` API와 tests가 유지되는가? +- persistent session이 registration failure에서는 leak 없이 닫히는가? +- runner가 test-injectable wait/stop 경계를 가져서 unit test가 hang하지 않는가? +- `pubspec.yaml` dependency update가 generated protobuf를 수동 편집하지 않고 해결되는가? + +## 검증 결과 + +_구현 에이전트가 각 중간 검증 및 최종 검증 명령 실행 후 출력을 여기에 붙여 넣는다._ + +> 참고: 변경된 generated protobuf 파일은 `protoc --plugin=protoc-gen-dart=/config/.local/bin/protoc-gen-dart --dart_out= --proto_path=../iop/proto --proto_path=/config/.local/include iop/runtime.proto`로 재생성했다(target = `lib/oto/agent`, `test/support`). 손으로 편집한 generated 파일은 없다. + +### API-1 중간 검증 +```bash +$ cd ../oto && dart pub get +Resolving dependencies... +Downloading packages... + ...(생략: 21개 패키지 newer-version 안내)... +Got dependencies! +21 packages have newer versions incompatible with dependency constraints. +Try `dart pub outdated` for more information. +# exit 0 +``` + +### API-2 / API-3 중간 검증 (대상 테스트) +```bash +$ cd ../oto && dart test test/oto_agent_registration_test.dart +00:02 +5: All tests passed! +# exit 0 +# 통과 테스트: +# - EdgeEndpoint parses host and port from config edge url +# - EdgeRegistrationClient.register one-shot returns session result and closes the session +# - AgentRunner maps config token to register request +# - AgentRunner keeps session open until shutdown then closes +# - AgentRunner reports rejected registration and closes session +``` + +### 최종 검증 +```bash +$ cd ../oto && dart pub get +Got dependencies! +21 packages have newer versions incompatible with dependency constraints. +# pubget_exit=0 + +$ cd ../oto && dart analyze +Analyzing oto... +No issues found! +# analyze_exit=0 + +$ cd ../oto && dart test test/oto_agent_registration_test.dart test/oto_agent_config_test.dart +00:00 +12: All tests passed! +# test_exit=0 +# 12 tests = 5 (oto_agent_registration_test.dart) + 7 (oto_agent_config_test.dart) +``` + +### 변경 파일 (git status) +``` + M lib/oto/agent/agent_runner.dart # API-3 + M lib/oto/agent/edge_registration_client.dart # API-2 + M pubspec.yaml # API-1 + M test/oto_agent_registration_test.dart # API-2, API-3 + M lib/oto/agent/iop/runtime.pb{,enum,json}.dart # API-1 재생성 + M test/support/iop/runtime.pb{,enum,json}.dart # API-1 재생성 + D lib/oto/agent/iop/runtime.pbserver.dart # 재생성 set에서 제외 + D lib/oto/agent/google/protobuf/struct.* # well-known type을 package에서 import + D test/support/iop/runtime.pbserver.dart + D test/support/google/protobuf/struct.* +# pubspec.lock 변경 없음(트래킹 변화 없음). +``` + +--- + +> **[IMPLEMENTING AGENT — BEFORE SAVING] Have you filled in every implementation-owned section: completion table, implementation checklist, changes from plan, design decisions, and verification output?** + +## 코드리뷰 결과 + +### 종합 판정 + +WARN + +### 차원별 평가 + +| 차원 | 평가 | 근거 | +|------|------|------| +| Correctness | Pass | `openSession()`이 register 이후 연결을 유지하고, `register()` one-shot wrapper와 runner `finally` close 경계가 유지된다. | +| Completeness | Pass | 계획된 dependency alignment, persistent session API, runner wait/close hook, targeted tests가 구현되었다. | +| Test coverage | Pass | persistent session 유지/종료, rejected registration close, one-shot compatibility test가 추가되었고 targeted tests 12개가 통과했다. | +| API contract | Pass | 기존 `EdgeRegistrationClient.register()`와 `DefaultAgentRunner.run(config)` call-site 계약이 유지된다. | +| Code quality | Warn | 수동 변경 Dart 파일 2개가 `dart format --output=none --set-exit-if-changed`를 통과하지 못한다. | +| Plan deviation | Pass | protobuf 6.x 대응 generated 재생성/삭제는 dependency 해결을 위한 범위 내 deviation으로 기록되어 있고 수동 generated 편집 정황은 없다. | +| Verification trust | Pass | `dart pub get`, `dart analyze`, targeted `dart test`를 리뷰 중 재실행해 기록과 일치함을 확인했다. | + +### 발견된 문제 + +- Suggested: `../oto/lib/oto/agent/edge_registration_client.dart:147`, `../oto/lib/oto/agent/agent_runner.dart:83` 수동 변경 파일이 Dart formatter 기준을 통과하지 못한다. `cd ../oto && dart format lib/oto/agent/edge_registration_client.dart lib/oto/agent/agent_runner.dart test/oto_agent_registration_test.dart`를 실행한 뒤 `dart format --output=none --set-exit-if-changed ...`, `dart analyze`, targeted tests를 재실행해 기록한다. + +### 리뷰 검증 + +```bash +$ cd ../oto && dart pub get +Got dependencies! +21 packages have newer versions incompatible with dependency constraints. +# exit 0 + +$ cd ../oto && dart analyze +Analyzing oto... +No issues found! +# exit 0 + +$ cd ../oto && dart test test/oto_agent_registration_test.dart test/oto_agent_config_test.dart +00:00 +12: All tests passed! +# exit 0 + +$ cd ../oto && dart format --output=none --set-exit-if-changed lib/oto/agent/edge_registration_client.dart lib/oto/agent/agent_runner.dart test/oto_agent_registration_test.dart +Changed lib/oto/agent/edge_registration_client.dart +Changed lib/oto/agent/agent_runner.dart +Formatted 3 files (2 changed) in 0.04 seconds. +# exit 1 +``` + +참고: `cd ../oto && dart test test/oto_iop_connection_smoke_test.dart`는 추가 확인으로 실행했으나, test가 생성한 iop Edge 프로세스가 20초 내 포트를 열지 못해 `timed out waiting for iop Edge`로 실패했다. OTO client register 호출 전 startup wait 단계의 실패이며, 이 subtask의 필수 검증이나 이번 Suggested 이슈 판정 근거로 사용하지 않는다. + +### 다음 단계 + +WARN/FAIL follow-up: formatter mismatch만 좁게 수정하는 다음 active `PLAN-local-G01.md`와 `CODE_REVIEW-local-G01.md`를 작성한다. diff --git a/agent-task/archive/2026/06/m-oto-registration-online-smoke/02+01_oto_agent_session/code_review_local_G01_1.log b/agent-task/archive/2026/06/m-oto-registration-online-smoke/02+01_oto_agent_session/code_review_local_G01_1.log new file mode 100644 index 0000000..9f9f654 --- /dev/null +++ b/agent-task/archive/2026/06/m-oto-registration-online-smoke/02+01_oto_agent_session/code_review_local_G01_1.log @@ -0,0 +1,207 @@ + + +# 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 by a user-only decision, user-owned external environment prerequisite, or scope conflict, fill `사용자 리뷰 요청` with evidence and stop with active files in place; code-review decides whether to write `USER_REVIEW.md`. Evidence gaps that a follow-up agent can close by rerunning commands or collecting artifacts are normal follow-up issues, not user-review blockers by themselves. +> 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-06-04 +task=m-oto-registration-online-smoke/02+01_oto_agent_session, plan=1, tag=REVIEW_API + +## 이 파일을 읽는 리뷰 에이전트에게 + +> **[REVIEW AGENT ONLY]** 아래 종결 절차는 코드리뷰 에이전트 전용이다. 구현 에이전트는 이 섹션을 실행하지 않는다. + +각 항목의 구현을 실제 소스 파일과 대조하고, `검증 결과` 섹션의 출력이 코드와 일치하는지 확인하세요. +리뷰 완료는 아래 순서까지 끝난 상태를 의미합니다. + +1. 판정을 append한다. +2. `CODE_REVIEW-local-G01.md` -> `code_review_local_G01_N.log`, `PLAN-local-G01.md` -> `plan_local_G01_M.log`로 아카이브한다. +3. PASS이면 `complete.log` 작성 후 `agent-task/archive/YYYY/MM/m-oto-registration-online-smoke/02+01_oto_agent_session/`로 이동한다. WARN/FAIL이면 user-review gate를 확인한 뒤 다음 active plan/review 파일 또는 `USER_REVIEW.md`를 작성한다. +4. PASS이고 task group이 `m-oto-registration-online-smoke`이면 완료 이벤트 메타데이터를 보고한다. roadmap 수정은 런타임 책임이다. +5. 적용 가능한 `코드리뷰 전용 체크리스트` 항목을 최종 `.log` 위치에서 체크한 뒤 보고한다. + +--- + +## 구현 항목별 완료 여부 + +| 항목 | 완료 여부 | +|------|---------| +| [REVIEW_API-1] Dart Formatter Cleanup | [x] | + +## 구현 체크리스트 + +- [x] `cd ../oto && dart format lib/oto/agent/edge_registration_client.dart lib/oto/agent/agent_runner.dart test/oto_agent_registration_test.dart`를 실행해 formatter mismatch를 해소한다. +- [x] `cd ../oto && dart format --output=none --set-exit-if-changed lib/oto/agent/edge_registration_client.dart lib/oto/agent/agent_runner.dart test/oto_agent_registration_test.dart`를 실행해 exit 0을 기록한다. +- [x] `cd ../oto && dart analyze`를 실행한다. +- [x] `cd ../oto && dart test test/oto_agent_registration_test.dart test/oto_agent_config_test.dart`를 실행한다. +- [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_{review_lane}_GNN_N.log`로 아카이브한다. +- [x] active `PLAN-*-G??.md`를 `plan_{build_lane}_GNN_M.log`로 아카이브한다. +- [x] `.gitignore`의 Agent-Ops 관리 block이 `agent-task/**/*.md`와 `agent-task/**/*.log`를 unignore하고 `agent-roadmap/current.md`를 ignore하는지 확인한다. +- [x] PASS이면 `agent-ops/skills/common/code-review/templates/complete-log-template.md` 기준으로 `complete.log`를 작성하고 active `.md` 파일을 남기지 않는다. +- [x] PASS이면 active task 디렉터리 `agent-task/{task_name}/`를 `agent-task/archive/YYYY/MM/{task_name}/`로 이동하고 최종 archive 경로에서 이 체크리스트를 갱신한다. +- [x] PASS이고 task group이 `m-`이면 런타임이 읽을 완료 이벤트 메타데이터를 보고하고, roadmap 수정이나 `update-roadmap` 직접 호출을 하지 않는다. +- [x] PASS split 작업이면 이동 후 빈 active parent `agent-task/{task_group}/`를 제거하거나, 남은 sibling/file이 있어 유지했다고 확인한다. +- [ ] WARN/FAIL이고 user-review gate가 트리거되지 않았으면 다음 active `PLAN-{build_lane}-GNN.md`와 `CODE_REVIEW-{review_lane}-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가 사용자 결정으로 완료/PASS 해소되면 `USER_REVIEW.md`를 해소 상태로 갱신하고 `complete.log`를 작성한 뒤 task directory를 archive로 이동한다. + +## 계획 대비 변경 사항 + +없음. 계획에 명시된 formatter cleanup과 검증 명령만 수행했다. + +## 주요 설계 결정 + +없음. formatter only follow-up이므로 설계 결정이 없다. + +## 사용자 리뷰 요청 + +_기본값은 `없음`이다. 구현 중 사용자 결정, 사용자 소유 외부 환경/secret/서비스 준비, 또는 계획 범위 변경 없이는 안전하게 진행할 수 없으면 아래 항목을 실제 내용으로 교체하고, 구현을 중단한 뒤 active 파일을 그대로 둔 채 리뷰를 요청한다. 후속 에이전트가 명령 재실행이나 산출물 수집으로 해소할 수 있는 검증 증거 공백만으로는 사용자 리뷰 요청을 작성하지 않는다._ + +- 상태: 없음 +- 사유 유형: 없음 +- 결정 필요: 없음 +- 차단 근거: 없음 +- 실행한 검증/명령: 없음 +- 자동 후속 불가 이유: 없음 +- 재개 조건: 없음 + +## 리뷰어를 위한 체크포인트 + +- formatter 적용 외의 동작 변경이 없는가? +- format check가 exit 0으로 통과하는가? +- `dart analyze`와 targeted agent tests가 계속 통과하는가? + +## 검증 결과 + +_구현 에이전트가 각 중간 검증 및 최종 검증 명령 실행 후 출력을 여기에 붙여 넣는다._ + +필수 규칙: +- 검증 명령은 고정된 계약이다. 임의로 대체하지 않는다. +- 대체가 필요하면 `계획 대비 변경 사항`에 이유와 대체 명령을 기록한다. +- `검증 결과`에는 실제 stdout/stderr를 붙여 넣는다. +- 사용자 리뷰 요청으로 명령을 끝까지 실행하지 못했다면 `사용자 리뷰 요청`에 실행한 명령, 실제 출력, 미실행 명령의 사유를 기록한다. + +### REVIEW_API-1 중간 검증 +```bash +$ cd ../oto && dart format lib/oto/agent/edge_registration_client.dart lib/oto/agent/agent_runner.dart test/oto_agent_registration_test.dart +Formatted lib/oto/agent/edge_registration_client.dart +Formatted lib/oto/agent/agent_runner.dart +Formatted 3 files (2 changed) in 0.07 seconds. +# exit 0 + +$ cd ../oto && dart format --output=none --set-exit-if-changed lib/oto/agent/edge_registration_client.dart lib/oto/agent/agent_runner.dart test/oto_agent_registration_test.dart +Formatted 3 files (0 changed) in 0.04 seconds. +# exit 0 +``` + +### 최종 검증 +```bash +$ cd ../oto && dart analyze +Analyzing oto... +No issues found! +# exit 0 + +$ cd ../oto && dart test test/oto_agent_registration_test.dart test/oto_agent_config_test.dart +00:00 +0: loading test/oto_agent_registration_test.dart +00:01 +0: test/oto_agent_registration_test.dart: EdgeEndpoint EdgeEndpoint parses host and port from config edge url +00:01 +1: test/oto_agent_config_test.dart: AgentConfig parses bootstrap config yaml +00:01 +2: test/oto_agent_config_test.dart: AgentConfig parses bootstrap config yaml +00:01 +3: test/oto_agent_config_test.dart: AgentConfig parses bootstrap config yaml +00:01 +4: test/oto_agent_config_test.dart: AgentConfig parses bootstrap config yaml +00:01 +4: test/oto_agent_registration_test.dart: AgentRunner AgentRunner keeps session open until shutdown then closes +00:01 +5: test/oto_agent_registration_test.dart: AgentRunner AgentRunner keeps session open until shutdown then closes +00:01 +6: test/oto_agent_config_test.dart: AgentConfig parses missing alias as null +00:01 +7: test/oto_agent_registration_test.dart: AgentRunner AgentRunner reports rejected registration and closes session +00:01 +8: test/oto_agent_config_test.dart: AgentConfig rejects missing required values +00:01 +9: test/oto_agent_config_test.dart: AgentConfig rejects invalid field types +00:01 +10: test/oto_agent_config_test.dart: AgentConfig rejects invalid field types +00:01 +10: test/oto_agent_config_test.dart: AgentConfig fromFile reads file correctly +00:01 +11: test/oto_agent_config_test.dart: AgentConfig fromFile reads file correctly +00:01 +11: test/oto_agent_config_test.dart: AgentConfig fromFile throws when file not found +00:01 +12: test/oto_agent_config_test.dart: AgentConfig fromFile throws when file not found +00:01 +12: All tests passed! +# exit 0 +``` + +--- + +> **[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. + +## Ownership + +| 섹션 | 소유자 | 설명 | +|------|--------|------| +| 헤더 주석, 개요(date/task/plan/tag), 리뷰 에이전트 지시 | 스텁 생성 시 고정 | 구현 에이전트가 수정하거나 실행하지 않음 | +| 구현 항목별 완료 여부 (항목명) | 스텁 생성 시 고정 | `[ ]` -> `[x]` 체크만 구현 에이전트가 수행 | +| 구현 체크리스트 (항목 텍스트/순서) | follow-up plan에서 복사해 스텁 생성 시 고정 | 구현 에이전트가 `[ ]` -> `[x]` 체크만 수행 | +| 코드리뷰 전용 체크리스트 | Review agent only | 구현 에이전트가 수정하거나 체크하지 않음 | +| 계획 대비 변경 사항, 주요 설계 결정 | 구현 에이전트가 채움 | placeholder 텍스트를 실제 내용으로 교체 | +| 사용자 리뷰 요청 | 구현 에이전트가 채움 | 진행에 사용자 입력이 필요하지 않으면 `상태: 없음` 유지 | +| 리뷰어를 위한 체크포인트 | 스텁 생성 시 고정 | 계획에서 추출한 리뷰 포인트 | +| 검증 결과 (섹션 제목 + 명령) | 스텁 생성 시 고정 | 실행 출력만 구현 에이전트가 채움 | +| 코드리뷰 결과 | 리뷰 에이전트가 append | 스텁에 포함하지 않음 | + +## 코드리뷰 결과 + +### 종합 판정 + +PASS + +### 차원별 평가 + +| 차원 | 평가 | 근거 | +|------|------|------| +| Correctness | Pass | follow-up은 formatter cleanup만 수행했고 persistent session 구현 동작을 변경하지 않았다. | +| Completeness | Pass | plan의 formatter 적용, format check, analyzer, targeted tests, review artifact 작성이 모두 완료되었다. | +| Test coverage | Pass | 기존 targeted agent tests 12개가 리뷰 중 재실행되어 통과했다. | +| API contract | Pass | public API와 call-site 계약 변경이 없다. | +| Code quality | Pass | `dart format --output=none --set-exit-if-changed`와 `git diff --check`가 통과했다. | +| Plan deviation | Pass | 계획 대비 변경 사항 없음. | +| Verification trust | Pass | 구현 기록의 format/analyze/test 결과를 리뷰 중 같은 명령으로 재검증했다. | + +### 발견된 문제 + +없음 + +### 리뷰 검증 + +```bash +$ cd ../oto && dart format --output=none --set-exit-if-changed lib/oto/agent/edge_registration_client.dart lib/oto/agent/agent_runner.dart test/oto_agent_registration_test.dart +Formatted 3 files (0 changed) in 0.04 seconds. +# exit 0 + +$ cd ../oto && dart analyze +Analyzing oto... +No issues found! +# exit 0 + +$ cd ../oto && git diff --check +# empty output, exit 0 + +$ cd ../oto && dart test test/oto_agent_registration_test.dart test/oto_agent_config_test.dart +00:00 +12: All tests passed! +# exit 0 +``` + +### 다음 단계 + +PASS: `complete.log` 작성 후 task directory를 `agent-task/archive/2026/06/m-oto-registration-online-smoke/02+01_oto_agent_session/`로 이동한다. diff --git a/agent-task/archive/2026/06/m-oto-registration-online-smoke/02+01_oto_agent_session/complete.log b/agent-task/archive/2026/06/m-oto-registration-online-smoke/02+01_oto_agent_session/complete.log new file mode 100644 index 0000000..709d534 --- /dev/null +++ b/agent-task/archive/2026/06/m-oto-registration-online-smoke/02+01_oto_agent_session/complete.log @@ -0,0 +1,39 @@ +# Complete - m-oto-registration-online-smoke/02+01_oto_agent_session + +## 완료 일시 + +2026-06-04 + +## 요약 + +OTO agent persistent registration session subtask completed after 2 review loops; final verdict PASS. + +## 루프 이력 + +| Plan | Review | Verdict | 메모 | +|------|--------|---------|------| +| `plan_cloud_G06_0.log` | `code_review_cloud_G06_0.log` | WARN | Persistent session implementation and targeted verification passed, but touched Dart files were not formatter-clean. | +| `plan_local_G01_1.log` | `code_review_local_G01_1.log` | PASS | Formatter cleanup completed; format check, analyzer, whitespace check, and targeted Dart tests passed. | + +## 구현/정리 내용 + +- Aligned OTO `protobuf` dependency with the `proto_socket` path dependency and regenerated protobuf Dart outputs for protobuf 6.x compatibility. +- Added a persistent `EdgeAgentSession` API while preserving one-shot `EdgeRegistrationClient.register()`. +- Updated `DefaultAgentRunner` to keep the registration session alive until shutdown and close it on accepted, rejected, and shutdown paths. +- Added unit coverage for one-shot close behavior, runner persistent-session waiting, and rejected registration close behavior. +- Applied Dart formatter cleanup for the touched manual Dart files. + +## 최종 검증 + +- `cd ../oto && dart format --output=none --set-exit-if-changed lib/oto/agent/edge_registration_client.dart lib/oto/agent/agent_runner.dart test/oto_agent_registration_test.dart` - PASS; `Formatted 3 files (0 changed)`. +- `cd ../oto && dart analyze` - PASS; `No issues found!`. +- `cd ../oto && git diff --check` - PASS; empty output. +- `cd ../oto && dart test test/oto_agent_registration_test.dart test/oto_agent_config_test.dart` - PASS; `00:00 +12: All tests passed!`. + +## 잔여 Nit + +- 없음 + +## 후속 작업 + +- 없음 diff --git a/agent-task/archive/2026/06/m-oto-registration-online-smoke/02+01_oto_agent_session/plan_cloud_G06_0.log b/agent-task/archive/2026/06/m-oto-registration-online-smoke/02+01_oto_agent_session/plan_cloud_G06_0.log new file mode 100644 index 0000000..a1f31db --- /dev/null +++ b/agent-task/archive/2026/06/m-oto-registration-online-smoke/02+01_oto_agent_session/plan_cloud_G06_0.log @@ -0,0 +1,199 @@ + + +# OTO Agent Persistent Registration Session + +## 이 파일을 읽는 구현 에이전트에게 + +`CODE_REVIEW-*-G??.md`의 구현 에이전트 소유 섹션을 채우는 것까지가 구현입니다. 검증 출력은 실제 stdout/stderr를 붙이고 active 파일을 남긴 채 리뷰 요청 상태로 보고하세요. 종결, archive, `complete.log` 작성은 code-review 스킬 전용입니다. + +## 배경 + +OTO `EdgeRegistrationClient.register()`는 registration response를 받은 뒤 client를 즉시 닫는다. iop Milestone은 registration accepted 이후 첫 heartbeat까지 살아 있는 OTO agent를 요구하므로, OTO agent 도메인에 persistent session 경계가 필요하다. 현재 OTO tests는 `protobuf ^3.1.0`과 `proto_socket`의 `protobuf ^6.0.0` 요구가 충돌해 baseline 실행도 막힌다. + +## 사용자 리뷰 요청 흐름 + +구현 중 사용자-only blocker가 있으면 active review stub의 `사용자 리뷰 요청` 섹션에 정확한 결정, 근거, 재개 조건을 남기고 멈춘다. dependency solving이나 test rerun으로 해소 가능한 증거 공백은 사용자 리뷰 요청이 아니다. + +## 분석 결과 + +### 읽은 파일 + +- `agent-roadmap/phase/automation-runtime-bridge/milestones/oto-registration-online-smoke.md` +- `../oto/AGENTS.md` +- `../oto/agent-ops/rules/project/rules.md` +- `../oto/agent-ops/rules/project/domain/agent/rules.md` +- `../oto/agent-ops/rules/project/domain/framework/rules.md` +- `../oto/pubspec.yaml` +- `../oto/lib/oto/agent/edge_registration_client.dart` +- `../oto/lib/oto/agent/agent_runner.dart` +- `../oto/lib/oto/agent/agent_config.dart` +- `../oto/lib/cli/commands/command_agent.dart` +- `../oto/test/oto_agent_registration_test.dart` +- `../oto/test/oto_agent_config_test.dart` +- `../oto/test/oto_iop_connection_smoke_test.dart` +- `../proto-socket/go/base_client.go` +- `../proto-socket/go/tcp_client.go` +- `../proto-socket/go/communicator.go` + +### 테스트 환경 규칙 + +- OTO `agent-test/local/rules.md`는 없다. +- 적용 fallback: OTO agent domain rule의 `dart analyze`와 agent 관련 tests. +- preflight: `command -v dart` 성공, Dart SDK `3.11.3`. +- preflight failure: `cd ../oto && dart test test/oto_agent_registration_test.dart test/oto_agent_config_test.dart`는 `protobuf ^3.1.0`과 path dependency `proto_socket`의 `protobuf ^6.0.0` 충돌로 exit code 65. + +### 테스트 커버리지 공백 + +- OTO persistent session: 없음. `EdgeRegistrationClient.register()`는 [edge_registration_client.dart](/config/workspace/oto/lib/oto/agent/edge_registration_client.dart:118)에서 즉시 close한다. +- Runner daemon behavior: `DefaultAgentRunner.run()`은 registration 성공 로그 후 return한다. +- Existing smoke: `oto_iop_connection_smoke_test.dart`는 accepted response와 runtime config만 확인하고 heartbeat/online evidence는 없다. +- Dependency baseline: tests가 dependency solving 전에 막혀 test coverage가 실행 불가 상태다. + +### 심볼 참조 + +- 변경 후보: `EdgeRegistrationClient.register` override/call sites는 `../oto/test/oto_agent_registration_test.dart`, `../oto/test/oto_iop_connection_smoke_test.dart`, `../oto/lib/oto/agent/agent_runner.dart`에 있다. +- 제거하지 말 것: 기존 one-shot `register()` API는 unit test와 runner injection 호환을 위해 유지하거나 wrapper로 보존한다. + +### 분할 판단 + +- 이 subtask는 `02+01_oto_agent_session`이며 predecessor `01_iop_edge_lifecycle` 완료가 필요하다. +- predecessor check: 아직 `agent-task/m-oto-registration-online-smoke/01_iop_edge_lifecycle/complete.log` 없음. 구현 시작 전 active/archive candidate `01_*` complete.log를 확인해야 한다. +- split 이유: OTO dependency/session 변경은 iop Edge lifecycle foundation과 repo/domain이 다르고 검증 실패 원인이 다르다. + +### 범위 결정 근거 + +- OTO pipeline build/run payload, artifact/log stream은 Milestone 범위 제외이므로 구현하지 않는다. +- scheduler/background service manager 정책은 건드리지 않는다. +- generated protobuf Dart 파일은 직접 수정하지 않는다. +- iop Edge code는 predecessor subtask에서만 수정한다. + +### 빌드 등급 + +- build: `cloud-G06`, review: `cloud-G06`. Dart agent lifecycle과 dependency alignment가 cross-repo로 묶여 있고, tests baseline blocker 해소가 필요하다. + +## 의존 관계 및 구현 순서 + +이 subtask는 디렉터리명 `02+01_oto_agent_session`에 따라 `01_iop_edge_lifecycle`의 `complete.log`가 필요하다. 구현 시작 전 아래 중 정확히 하나를 확인한다. + +- active: `agent-task/m-oto-registration-online-smoke/01_*/complete.log` +- archive: `agent-task/archive/*/*/m-oto-registration-online-smoke/01_*/complete.log` + +## 구현 체크리스트 + +- [x] `../oto/pubspec.yaml`의 protobuf constraint를 path dependency `proto_socket`과 맞춰 tests가 resolve되게 한다. +- [x] 기존 `register()` 호환을 유지하면서 close하지 않는 OTO registration session API를 추가한다. +- [x] `DefaultAgentRunner.run()`이 registration 성공 후 agent connection을 유지하고 종료 신호 또는 cancellation에서 닫도록 조정한다. +- [x] OTO agent unit tests가 persistent session API와 rejected registration behavior를 검증하게 한다. +- [x] `cd ../oto && dart test test/oto_agent_registration_test.dart test/oto_agent_config_test.dart`를 실행한다. +- [x] CODE_REVIEW-*-G??.md의 구현 에이전트 소유 섹션을 실제 구현 내용과 검증 출력으로 채운다. 이 항목이 완료되기 전에는 구현이 완료된 것이 아니다. + +### [API-1] Dependency Alignment + +문제: [pubspec.yaml](/config/workspace/oto/pubspec.yaml:23)은 `proto_socket` path dependency를 쓰고, [pubspec.yaml](/config/workspace/oto/pubspec.yaml:30)은 `protobuf ^3.1.0`을 고정한다. 현재 tests는 `proto_socket`의 `protobuf ^6.0.0` 요구와 충돌한다. + +해결 방법: `protobuf` constraint를 `^6.0.0`으로 올리고 `dart pub get` 후 agent tests를 실행한다. + +수정 파일 및 체크리스트: + +- [x] `../oto/pubspec.yaml` protobuf constraint 업데이트. (`^3.1.0` → `^6.0.0`) +- [x] `../oto/pubspec.lock`이 생성/변경되면 repo 정책에 맞춰 포함 여부 확인. (`dart pub get` 후 git 추적상 변경 없음) +- [x] (추가) constraint 상향으로 protobuf 3.x style generated 코드가 6.x와 충돌 → 원본 proto에서 protoc-gen-dart로 재생성. CODE_REVIEW `계획 대비 변경 사항` 참고. + +테스트 작성: SKIP. dependency alignment는 기존 agent tests 실행 가능 여부로 검증한다. + +중간 검증: + +```bash +cd ../oto && dart pub get +``` + +예상 결과: exit code 0. + +### [API-2] Persistent Edge Session API + +문제: [edge_registration_client.dart](/config/workspace/oto/lib/oto/agent/edge_registration_client.dart:107)는 `register()` 하나만 제공하고, [edge_registration_client.dart](/config/workspace/oto/lib/oto/agent/edge_registration_client.dart:118)는 response 직후 close한다. + +해결 방법: `EdgeAgentSession` 또는 `RegistrationSession` 타입을 추가한다. 새 API는 connect/register 후 client를 유지하고 `close()`를 노출한다. 기존 `register()`는 새 API를 사용하되 finally에서 close하는 one-shot wrapper로 보존한다. + +Before: + +```dart +// ../oto/lib/oto/agent/edge_registration_client.dart:107 +Future register(AgentConfig config) async { + final endpoint = EdgeEndpoint.parse(config.edge.url); + final socket = await ProtobufClient.connect(endpoint.host, endpoint.port); + final client = _OtoIopClient(socket); + try { + final response = + await client.sendRequest( + iop.RegisterRequest()..token = config.agent.enrollmentToken, + timeout: const Duration(seconds: 5), + ); + return RegistrationResult.fromResponse(response); + } finally { + await client.close(); + } +} +``` + +수정 파일 및 체크리스트: + +- [x] `../oto/lib/oto/agent/edge_registration_client.dart` persistent session 타입 추가. (`EdgeAgentSession` + `openSession()`) +- [x] `../oto/lib/oto/agent/edge_registration_client.dart` one-shot `register()` 호환 보존. +- [x] `../oto/test/oto_agent_registration_test.dart` fake client 구조를 새 API와 호환되게 수정. (`FakeEdgeAgentSession` + `openSession()` override) + +테스트 작성: `EdgeRegistrationClient` fake/session close test를 추가하거나 runner test에서 close 호출을 검증한다. + +중간 검증: + +```bash +cd ../oto && dart test test/oto_agent_registration_test.dart +``` + +예상 결과: exit code 0. + +### [API-3] Runner Keeps Agent Alive + +문제: [agent_runner.dart](/config/workspace/oto/lib/oto/agent/agent_runner.dart:40)는 registration success 로그 후 return한다. CLI help는 daemon이라고 말하지만 실제 연결은 유지되지 않는다. + +해결 방법: `DefaultAgentRunner.run()`은 persistent session API를 사용하고, 성공 후 process signal/cancellation 경계까지 대기한다. 테스트 가능성을 위해 runner에 optional wait future 또는 stop signal injection을 둔다. + +수정 파일 및 체크리스트: + +- [x] `../oto/lib/oto/agent/agent_runner.dart` persistent session 사용. (`openSession()` + `finally` close) +- [x] `../oto/lib/oto/agent/agent_runner.dart` test injection 가능한 wait/stop hook 추가. (`waitForShutdown`, 기본값 SIGINT/SIGTERM 대기) +- [x] `../oto/test/oto_agent_registration_test.dart` success path가 session close까지 호출하는지 검증. +- [x] `../oto/lib/cli/commands/command_agent.dart` 변경이 필요하면 기존 `--config` parsing 유지. (call-site 조정 불필요로 미변경) + +테스트 작성: fake client가 persistent session을 반환하고 runner wait future 완료 시 close되는지 확인한다. + +중간 검증: + +```bash +cd ../oto && dart test test/oto_agent_registration_test.dart +``` + +예상 결과: exit code 0. + +## 수정 파일 요약 + +| 파일 | 항목 | +|------|------| +| `../oto/pubspec.yaml` | API-1 | +| `../oto/pubspec.lock` | API-1 | +| `../oto/lib/oto/agent/edge_registration_client.dart` | API-2 | +| `../oto/lib/oto/agent/agent_runner.dart` | API-3 | +| `../oto/lib/cli/commands/command_agent.dart` | API-3, only if runner call-site needs adjustment | +| `../oto/test/oto_agent_registration_test.dart` | API-2, API-3 | + +## 최종 검증 + +```bash +cd ../oto && dart pub get +cd ../oto && dart analyze +cd ../oto && dart test test/oto_agent_registration_test.dart test/oto_agent_config_test.dart +``` + +예상 결과: 모두 exit code 0. Dart dependency solving output은 반드시 실제 출력으로 기록한다. + +모든 코드 변경 완료 후 반드시 `CODE_REVIEW-*-G??.md`의 구현 에이전트 소유 섹션을 채운다. 이 파일 작성이 구현의 마지막 단계다. diff --git a/agent-task/archive/2026/06/m-oto-registration-online-smoke/02+01_oto_agent_session/plan_local_G01_1.log b/agent-task/archive/2026/06/m-oto-registration-online-smoke/02+01_oto_agent_session/plan_local_G01_1.log new file mode 100644 index 0000000..fd58a96 --- /dev/null +++ b/agent-task/archive/2026/06/m-oto-registration-online-smoke/02+01_oto_agent_session/plan_local_G01_1.log @@ -0,0 +1,90 @@ + + +# Follow-up Plan - REVIEW_API + +## 이 파일을 읽는 구현 에이전트에게 + +이 plan은 `code_review_cloud_G06_0.log`의 WARN finding만 처리한다. 코드 동작, protobuf 생성물, runner/session 설계는 바꾸지 말고 formatter mismatch 해소와 검증 출력 기록에 집중한다. + +구현 중 사용자-only blocker가 있으면 active review stub의 `사용자 리뷰 요청` 섹션에 정확한 결정, 근거, 재개 조건을 남기고 멈춘다. 명령 재실행이나 출력 수집으로 해소 가능한 증거 공백은 사용자 리뷰 요청이 아니다. + +## 배경 + +첫 리뷰는 기능 구현과 targeted verification을 통과했지만, 수동 변경 Dart 파일 2개가 formatter 기준을 통과하지 못해 `WARN`으로 종료되었다. + +## 이전 리뷰 요약 + +- Archived plan: `agent-task/m-oto-registration-online-smoke/02+01_oto_agent_session/plan_cloud_G06_0.log` +- Archived review: `agent-task/m-oto-registration-online-smoke/02+01_oto_agent_session/code_review_cloud_G06_0.log` +- Verdict: `WARN` +- Issue: `../oto/lib/oto/agent/edge_registration_client.dart`와 `../oto/lib/oto/agent/agent_runner.dart`가 `dart format --output=none --set-exit-if-changed`를 통과하지 못함. + +## 범위 결정 근거 + +- 이번 follow-up은 format-only 정리다. +- `../oto/lib/oto/agent/**`의 동작, `../oto/pubspec.yaml`, generated protobuf 파일, iop Edge 구현, smoke evidence 수집은 수정하지 않는다. +- `test/oto_iop_connection_smoke_test.dart`는 별도 active subtask `03+01,02_smoke_evidence`의 범위이므로 여기서 고치지 않는다. + +## 구현 체크리스트 + +- [ ] `cd ../oto && dart format lib/oto/agent/edge_registration_client.dart lib/oto/agent/agent_runner.dart test/oto_agent_registration_test.dart`를 실행해 formatter mismatch를 해소한다. +- [ ] `cd ../oto && dart format --output=none --set-exit-if-changed lib/oto/agent/edge_registration_client.dart lib/oto/agent/agent_runner.dart test/oto_agent_registration_test.dart`를 실행해 exit 0을 기록한다. +- [ ] `cd ../oto && dart analyze`를 실행한다. +- [ ] `cd ../oto && dart test test/oto_agent_registration_test.dart test/oto_agent_config_test.dart`를 실행한다. +- [ ] CODE_REVIEW-*-G??.md의 구현 에이전트 소유 섹션을 실제 구현 내용과 검증 출력으로 채운다. 이 항목이 완료되기 전에는 구현이 완료된 것이 아니다. + +### [REVIEW_API-1] Dart Formatter Cleanup + +문제: `code_review_cloud_G06_0.log`에서 `dart format --output=none --set-exit-if-changed`가 `edge_registration_client.dart`와 `agent_runner.dart`를 변경 대상으로 보고했다. + +해결 방법: formatter를 적용하고, 동일 format check가 exit 0이 되는지 확인한다. + +Before: + +```text +Changed lib/oto/agent/edge_registration_client.dart +Changed lib/oto/agent/agent_runner.dart +Formatted 3 files (2 changed) in 0.04 seconds. +# exit 1 +``` + +After: + +```text +Formatted 3 files (0 changed) in ... +# exit 0 +``` + +수정 파일 및 체크리스트: + +- [ ] `../oto/lib/oto/agent/edge_registration_client.dart` formatter 적용. +- [ ] `../oto/lib/oto/agent/agent_runner.dart` formatter 적용. +- [ ] `../oto/test/oto_agent_registration_test.dart` formatter command 대상에 포함하되, 실제 변경 여부를 기록. + +테스트 작성: SKIP. format-only follow-up이며 기존 targeted tests를 재실행한다. + +중간 검증: + +```bash +cd ../oto && dart format --output=none --set-exit-if-changed lib/oto/agent/edge_registration_client.dart lib/oto/agent/agent_runner.dart test/oto_agent_registration_test.dart +``` + +예상 결과: exit code 0. + +## 수정 파일 요약 + +| 파일 | 항목 | +|------|------| +| `../oto/lib/oto/agent/edge_registration_client.dart` | REVIEW_API-1 | +| `../oto/lib/oto/agent/agent_runner.dart` | REVIEW_API-1 | +| `../oto/test/oto_agent_registration_test.dart` | REVIEW_API-1, only if formatter changes it | + +## 최종 검증 + +```bash +cd ../oto && dart format --output=none --set-exit-if-changed lib/oto/agent/edge_registration_client.dart lib/oto/agent/agent_runner.dart test/oto_agent_registration_test.dart +cd ../oto && dart analyze +cd ../oto && dart test test/oto_agent_registration_test.dart test/oto_agent_config_test.dart +``` + +예상 결과: 모두 exit code 0. 모든 코드 변경 완료 후 반드시 `CODE_REVIEW-*-G??.md`의 구현 에이전트 소유 섹션을 채운다. diff --git a/agent-task/archive/2026/06/m-oto-registration-online-smoke/03+01,02_smoke_evidence/code_review_cloud_G07_0.log b/agent-task/archive/2026/06/m-oto-registration-online-smoke/03+01,02_smoke_evidence/code_review_cloud_G07_0.log new file mode 100644 index 0000000..e90b698 --- /dev/null +++ b/agent-task/archive/2026/06/m-oto-registration-online-smoke/03+01,02_smoke_evidence/code_review_cloud_G07_0.log @@ -0,0 +1,188 @@ + + +# Code Review Reference - TEST + +> **[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 user-only decision, user-owned external environment prerequisite, or scope conflict, fill `사용자 리뷰 요청` with evidence and stop with active files in place; code-review decides whether to write `USER_REVIEW.md`. +> Finalization (`코드리뷰 결과`, log rename, `complete.log`, archive moves, `코드리뷰 전용 체크리스트`) is review-agent-only. + +## 개요 + +date=2026-06-03 +task=m-oto-registration-online-smoke/03+01,02_smoke_evidence, plan=0, tag=TEST + +## Roadmap Targets + +- Milestone: `agent-roadmap/phase/automation-runtime-bridge/milestones/oto-registration-online-smoke.md` +- Task ids: + - `heartbeat-online`: 최초 heartbeat 도착 후 online으로 전환된다. 검증: OTO Dart smoke 또는 통합 smoke에서 registration accepted 뒤 heartbeat 도착까지 확인한다. + - `unlock-evidence`: OTO `oto-agent` 등록 흐름 잠금 해제에 사용할 smoke 결과와 실행 명령이 문서 또는 test log에 남는다. +- Completion mode: check-on-pass + +## 이 파일을 읽는 리뷰 에이전트에게 + +> **[REVIEW AGENT ONLY]** 아래 종결 절차는 코드리뷰 에이전트 전용이다. 구현 에이전트는 이 섹션을 실행하지 않는다. + +PASS이면 `complete.log` 작성 후 `agent-task/archive/YYYY/MM/m-oto-registration-online-smoke/03+01,02_smoke_evidence/`로 이동합니다. roadmap 수정은 런타임 책임입니다. + +--- + +## 구현 항목별 완료 여부 + +| 항목 | 완료 여부 | +|------|---------| +| [TEST-1] Make OTO Smoke Repo Path Explicit | [x] | +| [TEST-2] Assert Accepted Then Heartbeat Online | [x] | +| [TEST-3] iop Makefile Target and Evidence Command | [x] | + +## 구현 체크리스트 + +- [x] OTO smoke가 iop repo path를 env로 받을 수 있게 하거나 iop-side wrapper가 현재 repo를 명시하게 한다. +- [x] OTO Dart smoke 또는 iop wrapper가 registration accepted 뒤 heartbeat/online event를 확인한다. +- [x] `make test-oto-registration-online-smoke` 또는 동등한 iop Makefile target을 추가한다. +- [x] smoke output에 unlock handoff용 실행 명령과 accepted/heartbeat/online evidence가 남게 한다. +- [x] `go test -count=1 ./apps/edge/internal/transport ./apps/edge/internal/node ./apps/edge/internal/service ./packages/go/events`와 OTO smoke target을 실행한다. +- [x] CODE_REVIEW-*-G??.md의 구현 에이전트 소유 섹션을 실제 구현 내용과 검증 출력으로 채운다. 이 항목이 완료되기 전에는 구현이 완료된 것이 아니다. + +## 코드리뷰 전용 체크리스트 + +> **[REVIEW AGENT ONLY]** 이 체크리스트는 코드리뷰 에이전트만 사용한다. +> 구현 에이전트는 이 섹션을 수정하거나 체크하지 않는다. + +- [x] `코드리뷰 결과`에 `PASS`, `WARN`, `FAIL` 중 하나의 판정을 append한다. +- [x] active `CODE_REVIEW-*-G??.md`를 `code_review_cloud_G07_N.log`로 아카이브한다. +- [x] active `PLAN-*-G??.md`를 `plan_cloud_G07_M.log`로 아카이브한다. +- [ ] PASS이면 `complete.log`를 작성하고 active `.md` 파일을 남기지 않는다. +- [ ] PASS이면 active task 디렉터리 `agent-task/m-oto-registration-online-smoke/03+01,02_smoke_evidence/`를 archive로 이동한다. +- [ ] PASS split 작업이면 이동 후 parent에 남은 sibling/file이 있어 유지했다고 확인한다. +- [x] WARN follow-up `PLAN-local-G01.md`와 `CODE_REVIEW-local-G01.md`를 작성하고 `complete.log`를 작성하지 않는다. + +## 계획 대비 변경 사항 + +- **online evidence 수집 경로 변경(TEST-2)**: 계획은 "Edge stdout/console log에 node.online marker를 기다린다"였으나, Edge serve는 application(zap) log를 stderr가 아니라 `/logs/edge.log`(미지정 시 기본 경로)에 쓴다(`apps/edge/cmd/edge/main.go:104` `resolveEdgeLogPath`). stderr에는 Fx 부트스트랩 로그만 나온다. 그래서 smoke edge config에 `logging.path`를 temp `edge.log`로 명시하고, test는 process stdout이 아니라 그 로그 파일을 tail해서 `node.online`을 확인한다(`_waitForLogMarker`). process stderr 버퍼는 실패 시 context로만 사용한다. +- **online 대기 시간 60s, 전체 timeout 180s**: `node.online`(`reason=first_heartbeat`)은 등록 직후가 아니라 proto-socket heartbeat interval(~30s)의 첫 주기 heartbeat에서 발생한다(실측 등록 +30s). 계획의 가정보다 늦으므로 online 대기를 60s, test timeout을 45s→180s로 늘렸다. +- **bootstrap.listen free 포트 지정**: Edge serve는 항상 bootstrap artifact server를 띄우고 기본 listen은 고정 `0.0.0.0:18080`이다(`packages/go/config/config.go:270`). 18080이 점유되면 Edge 기동이 실패하므로(OnStart 실패) smoke config에 free `bootstrap.listen` 포트를 할당했다. 계획에 없던 robustness 보완이다. +- **GOCACHE override 변경 + `_waitForPort` 120s**: 기존 test는 매 실행 fresh temp `GOCACHE`로 cold build(실측 ~58s)를 강제해 20s port wait deadline을 넘길 위험이 있었다. 안정적 재현을 위해 stable shared cache(`${systemTemp}/oto-iop-smoke-gocache`)로 바꿔 후속 실행을 warm(빌드 캐시 재사용)하게 하고, 첫 cold build를 견디도록 `_waitForPort` deadline을 20s→120s로 늘렸다. +- **wrapper script 미생성(TEST-3)**: 계획상 선택 사항이던 `scripts/e2e-oto-registration-online-smoke.sh`는 만들지 않았다. inline Makefile target이 더 단순하고 generic `scripts/e2e-smoke.sh`와 섞이지 않으므로 inline로 끝냈다. + +## 주요 설계 결정 + +- **persistent session 사용**: one-shot `register()` 대신 `EdgeRegistrationClient.openSession()`을 써 등록 이후 연결을 유지한다. heartbeat가 Edge에 도달해 online 전환이 일어나려면 연결이 살아 있어야 한다. test finally에서 `session.close()`로 정리한다. +- **`agent_kind: "oto-agent"` 명시**: Edge의 first-heartbeat→online listener는 oto-agent kind에만 붙는다(`apps/edge/internal/transport/server.go:218`). temp config의 node에 `agent_kind`를 넣어 online lifecycle 경로를 활성화했다. 이 값이 없으면 generic-node로 처리되어 online 전환이 일어나지 않는다. +- **IOP_REPO_ROOT env override**: hardcoded `workingDirectory: '../iop'`를 `Platform.environment['IOP_REPO_ROOT'] ?? '../iop'`로 바꿔 same-host 기본은 유지하면서 iop2oto 등 임의 checkout을 대상으로 실행 가능하게 했다. Makefile target은 `IOP_REPO_ROOT=$(CURDIR)`로 현재 repo를 명시한다. +- **online marker로 heartbeat까지 증명**: `node.online` 이벤트의 `reason`이 `first_heartbeat`이므로 online 라인 하나로 "first heartbeat 도착 + online 전환"을 함께 증거로 남긴다. test는 이 라인을 stdout으로 print해 unlock handoff evidence로 노출한다. +- **secret 미커밋**: token은 non-secret test 값(`oto-smoke-token`)이고 temp dir만 사용한다. tracked docs/log에 실제 secret을 남기지 않는다. + +## 사용자 리뷰 요청 + +_기본값은 `없음`이다. 구현 중 사용자 결정, 사용자 소유 외부 환경/secret/서비스 준비, 또는 계획 범위 변경 없이는 안전하게 진행할 수 없으면 아래 항목을 실제 내용으로 교체하고, 구현을 중단한 뒤 active 파일을 그대로 둔 채 리뷰를 요청한다. 후속 에이전트가 명령 재실행이나 산출물 수집으로 해소할 수 있는 검증 증거 공백만으로는 사용자 리뷰 요청을 작성하지 않는다._ + +- 상태: 없음 +- 사유 유형: 없음 +- 결정 필요: 없음 +- 차단 근거: 없음 +- 실행한 검증/명령: 없음 +- 자동 후속 불가 이유: 없음 +- 재개 조건: 없음 + +## 리뷰어를 위한 체크포인트 + +- predecessor complete.log가 `01`과 `02` 모두 있었는가? +- smoke가 same-host 기본 경로를 사용하고 split-host를 기본값으로 요구하지 않는가? +- output에 registration accepted, first heartbeat, online evidence가 모두 남는가? +- token/secret 원문이 tracked docs나 logs에 남지 않는가? + +## 검증 결과 + +_구현 에이전트가 각 중간 검증 및 최종 검증 명령 실행 후 출력을 여기에 붙여 넣는다._ + +### TEST-1 중간 검증 +```bash +$ rg --sort path "../iop|IOP_REPO_ROOT|agent_kind" ../oto/test/oto_iop_connection_smoke_test.dart +/// Defaults to the same-host sibling `../iop` checkout, but `IOP_REPO_ROOT` + final override = Platform.environment['IOP_REPO_ROOT']; + return '../iop'; + agent_kind: "oto-agent" +``` +결과: `IOP_REPO_ROOT`와 `agent_kind`가 보이고, `../iop`는 default fallback일 뿐 hardcoded-only path가 없다. (기대 결과 충족) + +### TEST-2 중간 검증 +```bash +$ cd ../oto && IOP_REPO_ROOT=/config/workspace/iop2oto dart test test/oto_iop_connection_smoke_test.dart +[oto-smoke] iop repo root: /config/workspace/iop2oto +[oto-smoke] registration accepted node=oto-smoke-node alias=oto-smoke +[oto-smoke] online evidence: {"level":"debug","ts":1780562158.140313,"caller":"transport/server.go:314","msg":"node event received","node_id":"oto-smoke-node","type":"node.online","reason":"first_heartbeat"} +00:31 +1: All tests passed! +``` +결과: exit 0. registration accepted, node.online(`reason=first_heartbeat`) evidence 모두 출력됨. + +### TEST-3 중간 검증 +```bash +$ make test-oto-registration-online-smoke +==> OTO registration online smoke +command: cd ../oto && IOP_REPO_ROOT=/config/workspace/iop2oto dart test test/oto_iop_connection_smoke_test.dart +[oto-smoke] iop repo root: /config/workspace/iop2oto +[oto-smoke] registration accepted node=oto-smoke-node alias=oto-smoke +[oto-smoke] online evidence: {"level":"debug",...,"type":"node.online","reason":"first_heartbeat"} +00:32 +1: All tests passed! +make exit: 0 +``` +결과: exit 0. unlock handoff용 실행 command가 target 출력에 echo됨. + +### 최종 검증 +```bash +$ go test -count=1 ./apps/edge/internal/transport ./apps/edge/internal/node ./apps/edge/internal/service ./packages/go/events +ok iop/apps/edge/internal/transport 2.027s +ok iop/apps/edge/internal/node 0.012s +ok iop/apps/edge/internal/service 0.008s +? iop/packages/go/events [no test files] +go test exit: 0 + +$ cd ../oto && dart analyze +Analyzing oto... +No issues found! +dart analyze (project) exit: 0 + +$ make test-oto-registration-online-smoke +... [oto-smoke] registration accepted node=oto-smoke-node alias=oto-smoke +... [oto-smoke] online evidence: {... "type":"node.online","reason":"first_heartbeat"} +00:32 +1: All tests passed! +make exit: 0 +``` +결과: 모두 exit 0. go test는 fresh(`-count=1`)로 캐시 출력 없이 통과(`packages/go/events`는 test 파일 없음). dart analyze 무이슈. OTO online smoke가 command·accepted·first-heartbeat/online evidence를 함께 남긴다. + +## Unlock Handoff 명령 (evidence) + +OTO `oto-agent` 등록 흐름 잠금 해제 evidence로 사용할 재현 명령: + +```bash +make test-oto-registration-online-smoke +# 또는 직접: +cd ../oto && IOP_REPO_ROOT=/config/workspace/iop2oto dart test test/oto_iop_connection_smoke_test.dart +``` + +기대 evidence: +- `[oto-smoke] registration accepted node=oto-smoke-node alias=oto-smoke` +- `node.online` 이벤트(`reason=first_heartbeat`) — registration accepted 뒤 첫 heartbeat 도착으로 online 전환 + +--- + +> **[IMPLEMENTING AGENT — BEFORE SAVING] Have you filled in every implementation-owned section: completion table, implementation checklist, changes from plan, design decisions, and verification output?** + +## 코드리뷰 결과 + +- 종합 판정: WARN +- 차원별 평가: + - correctness: Pass + - completeness: Pass + - test coverage: Pass + - API contract: Pass + - code quality: Warn + - plan deviation: Pass + - verification trust: Pass +- 발견된 문제: + - Suggested: `../oto/test/oto_iop_connection_smoke_test.dart:211`가 Dart formatter 기준을 만족하지 않는다. `dart format --output=none --set-exit-if-changed test/oto_iop_connection_smoke_test.dart lib/oto/agent/edge_registration_client.dart lib/oto/agent/agent_runner.dart test/oto_agent_registration_test.dart`에서 `Changed test/oto_iop_connection_smoke_test.dart`로 실패했다. 해당 파일을 `dart format`으로 정리한 뒤 format check, `dart analyze`, `make test-oto-registration-online-smoke`를 재확인한다. +- 다음 단계: WARN follow-up plan/review를 작성해 formatter cleanup과 재검증만 수행한다. diff --git a/agent-task/archive/2026/06/m-oto-registration-online-smoke/03+01,02_smoke_evidence/code_review_local_G01_1.log b/agent-task/archive/2026/06/m-oto-registration-online-smoke/03+01,02_smoke_evidence/code_review_local_G01_1.log new file mode 100644 index 0000000..c146f3f --- /dev/null +++ b/agent-task/archive/2026/06/m-oto-registration-online-smoke/03+01,02_smoke_evidence/code_review_local_G01_1.log @@ -0,0 +1,149 @@ + + +# Code Review Reference - REVIEW_TEST + +> **[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 user-only decision, user-owned external environment prerequisite, or scope conflict, fill `사용자 리뷰 요청` with evidence and stop with active files in place; code-review decides whether to write `USER_REVIEW.md`. Evidence gaps that a follow-up agent can close by rerunning commands or collecting artifacts are normal follow-up issues, not user-review blockers by themselves. +> 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-06-04 +task=m-oto-registration-online-smoke/03+01,02_smoke_evidence, plan=1, tag=REVIEW_TEST + +## Roadmap Targets + +- Milestone: `agent-roadmap/phase/automation-runtime-bridge/milestones/oto-registration-online-smoke.md` +- Task ids: + - `heartbeat-online`: 최초 heartbeat 도착 후 online으로 전환된다. 검증: OTO Dart smoke 또는 통합 smoke에서 registration accepted 뒤 heartbeat 도착까지 확인한다. + - `unlock-evidence`: OTO `oto-agent` 등록 흐름 잠금 해제에 사용할 smoke 결과와 실행 명령이 문서 또는 test log에 남는다. +- Completion mode: check-on-pass + +## 이 파일을 읽는 리뷰 에이전트에게 + +> **[REVIEW AGENT ONLY]** 아래 종결 절차는 코드리뷰 에이전트 전용이다. 구현 에이전트는 이 섹션을 실행하지 않는다. + +각 항목의 구현을 실제 소스 파일과 대조하고, `검증 결과` 섹션의 출력이 코드와 일치하는지 확인하세요. +리뷰 완료는 아래 순서까지 끝난 상태를 의미합니다. + +1. 판정을 append한다. +2. `CODE_REVIEW-{review_lane}-GNN.md` -> `code_review_{review_lane}_GNN_N.log`, `PLAN-{build_lane}-GNN.md` -> `plan_{build_lane}_GNN_M.log`로 아카이브한다. +3. PASS이면 `complete.log` 작성 후 active task 디렉터리를 `agent-task/archive/YYYY/MM/m-oto-registration-online-smoke/03+01,02_smoke_evidence/`로 이동한다. WARN/FAIL이면 user-review gate를 확인한 뒤 다음 active plan/review 파일 또는 `USER_REVIEW.md`를 작성한다. +4. PASS이고 task group이 `m-`이면 완료 이벤트 메타데이터를 보고한다. roadmap 상태 체크와 `update-roadmap` 호출은 런타임 책임이다. +5. 적용 가능한 `코드리뷰 전용 체크리스트` 항목을 최종 `.log` 위치에서 체크한 뒤 보고한다. + +--- + +## 구현 항목별 완료 여부 + +| 항목 | 완료 여부 | +|------|---------| +| [REVIEW_TEST-1] Format OTO Smoke Test | [x] | + +## 구현 체크리스트 + +- [x] `../oto/test/oto_iop_connection_smoke_test.dart`를 Dart formatter 기준으로 정리한다. +- [x] `cd ../oto && dart format --output=none --set-exit-if-changed test/oto_iop_connection_smoke_test.dart lib/oto/agent/edge_registration_client.dart lib/oto/agent/agent_runner.dart test/oto_agent_registration_test.dart`를 실행해 formatter-clean을 확인한다. +- [x] `cd ../oto && dart analyze`와 `make test-oto-registration-online-smoke`를 실행해 기존 analyzer/smoke evidence가 유지되는지 확인한다. +- [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_{review_lane}_GNN_N.log`로 아카이브한다. +- [x] active `PLAN-*-G??.md`를 `plan_{build_lane}_GNN_M.log`로 아카이브한다. +- [x] `.gitignore`의 Agent-Ops 관리 block이 `agent-task/**/*.md`와 `agent-task/**/*.log`를 unignore하고 `agent-roadmap/current.md`를 ignore하는지 확인한다. +- [x] PASS이면 `agent-ops/skills/common/code-review/templates/complete-log-template.md` 기준으로 `complete.log`를 작성하고 active `.md` 파일을 남기지 않는다. +- [x] PASS이면 active task 디렉터리 `agent-task/m-oto-registration-online-smoke/03+01,02_smoke_evidence/`를 `agent-task/archive/YYYY/MM/m-oto-registration-online-smoke/03+01,02_smoke_evidence/`로 이동하고 최종 archive 경로에서 이 체크리스트를 갱신한다. +- [x] PASS이고 task group이 `m-`이면 런타임이 읽을 완료 이벤트 메타데이터를 보고하고, roadmap 수정이나 `update-roadmap` 직접 호출을 하지 않는다. +- [x] PASS split 작업이면 이동 후 빈 active parent `agent-task/m-oto-registration-online-smoke/`를 제거하거나, 남은 sibling/file이 있어 유지했다고 확인한다. +- [ ] WARN/FAIL이고 user-review gate가 트리거되지 않았으면 다음 active `PLAN-{build_lane}-GNN.md`와 `CODE_REVIEW-{review_lane}-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가 사용자 결정으로 완료/PASS 해소되면 `USER_REVIEW.md`를 해소 상태로 갱신하고 `complete.log`를 작성한 뒤 task directory를 archive로 이동한다. + +## 계획 대비 변경 사항 + +- 계획 대비 변경 사항 없음. `PLAN-local-G01.md`에 명시된 대로 `../oto/test/oto_iop_connection_smoke_test.dart` 파일에 `dart format`을 적용하여 서식을 정리했습니다. + +## 주요 설계 결정 + +- 기능적인 로직 변경 사항은 없으며, Dart SDK 공식 포맷터 표준에 맞춰 긴 삼항 할당 및 들여쓰기 등을 정리했습니다. + +## 사용자 리뷰 요청 + +- 상태: 없음 +- 사유 유형: 없음 +- 결정 필요: 없음 +- 차단 근거: 없음 +- 실행한 검증/명령: 없음 +- 자동 후속 불가 이유: 없음 +- 재개 조건: 없음 + +## 리뷰어를 위한 체크포인트 + +- formatter-clean 명령이 exit code 0인가? +- formatter cleanup 외의 기능 변경이 들어가지 않았는가? +- analyzer와 OTO online smoke evidence가 유지되는가? + +## 검증 결과 + +1. **Dart 포맷 검사 결과** +```bash +cd ../oto && dart format --output=none --set-exit-if-changed test/oto_iop_connection_smoke_test.dart lib/oto/agent/edge_registration_client.dart lib/oto/agent/agent_runner.dart test/oto_agent_registration_test.dart +``` +```text +Formatted 4 files (0 changed) in 0.06 seconds. +``` + +2. **Dart analyze 실행 결과** +```bash +cd ../oto && dart analyze +``` +```text +Analyzing oto... +No issues found! +``` + +3. **E2E Smoke 테스트 실행 결과** +```bash +make test-oto-registration-online-smoke +``` +```text +==> OTO registration online smoke +command: cd ../oto && IOP_REPO_ROOT=/config/workspace/iop2oto dart test test/oto_iop_connection_smoke_test.dart +cd ../oto && IOP_REPO_ROOT=/config/workspace/iop2oto dart test test/oto_iop_connection_smoke_test.dart +00:00 +0: loading test/oto_iop_connection_smoke_test.dart +00:01 +0: loading test/oto_iop_connection_smoke_test.dart +00:01 +0: OTO Dart proto-socket client registers with iop Edge and goes online +[oto-smoke] iop repo root: /config/workspace/iop2oto +[oto-smoke] registration accepted node=oto-smoke-node alias=oto-smoke +[oto-smoke] online evidence: {"level":"debug","ts":1780564855.8409436,"caller":"transport/server.go:314","msg":"node event received","node_id":"oto-smoke-node","type":"node.online","reason":"first_heartbeat"} +00:34 +1: OTO Dart proto-socket client registers with iop Edge and goes online +00:34 +1: All tests passed! +``` + +--- + +> **[IMPLEMENTING AGENT — BEFORE SAVING] Have you filled in every implementation-owned section: completion table, implementation checklist, changes from plan, design decisions, and verification output?** Yes. + +## 코드리뷰 결과 + +- 종합 판정: PASS +- 차원별 평가: + - correctness: Pass + - completeness: Pass + - test coverage: Pass + - API contract: Pass + - code quality: Pass + - plan deviation: Pass + - verification trust: Pass +- 발견된 문제: 없음 +- 다음 단계: PASS 종결 처리로 `complete.log` 작성 후 task directory를 archive로 이동한다. diff --git a/agent-task/archive/2026/06/m-oto-registration-online-smoke/03+01,02_smoke_evidence/complete.log b/agent-task/archive/2026/06/m-oto-registration-online-smoke/03+01,02_smoke_evidence/complete.log new file mode 100644 index 0000000..9c32fc2 --- /dev/null +++ b/agent-task/archive/2026/06/m-oto-registration-online-smoke/03+01,02_smoke_evidence/complete.log @@ -0,0 +1,47 @@ +# Complete - m-oto-registration-online-smoke/03+01,02_smoke_evidence + +## 완료 일시 + +2026-06-04 + +## 요약 + +OTO registration online smoke evidence subtask completed after 2 review loops; final verdict PASS. + +## 루프 이력 + +| Plan | Review | Verdict | 메모 | +|------|--------|---------|------| +| `plan_cloud_G07_0.log` | `code_review_cloud_G07_0.log` | WARN | OTO online smoke implementation and evidence passed, but touched Dart smoke file was not formatter-clean. | +| `plan_local_G01_1.log` | `code_review_local_G01_1.log` | PASS | Formatter cleanup completed; format check, analyzer, whitespace checks, and OTO online smoke passed. | + +## 구현/정리 내용 + +- Added an iop Makefile entrypoint `test-oto-registration-online-smoke` that runs the sibling OTO Dart smoke with `IOP_REPO_ROOT=$(CURDIR)` and prints the unlock handoff command. +- Extended the OTO smoke to keep the registration session alive, wait for `node.online` with `reason=first_heartbeat`, and print accepted plus online evidence. +- Made the OTO smoke repo path explicit through `IOP_REPO_ROOT`, set `agent_kind: "oto-agent"` in the temp Edge config, and used temp ports/log paths for same-host reproducibility. +- Applied Dart formatter cleanup for `test/oto_iop_connection_smoke_test.dart`. + +## 최종 검증 + +- `cd ../oto && dart format --output=none --set-exit-if-changed test/oto_iop_connection_smoke_test.dart lib/oto/agent/edge_registration_client.dart lib/oto/agent/agent_runner.dart test/oto_agent_registration_test.dart` - PASS; `Formatted 4 files (0 changed) in 0.13 seconds.` +- `cd ../oto && dart analyze` - PASS; `No issues found!`. +- `cd ../oto && git diff --check` - PASS; empty output. +- `git diff --check` - PASS; empty output. +- `make test-oto-registration-online-smoke` - PASS; output printed the handoff command, `[oto-smoke] registration accepted node=oto-smoke-node alias=oto-smoke`, and `node.online` evidence with `reason":"first_heartbeat"`. + +## Roadmap Completion + +- Milestone: `agent-roadmap/phase/automation-runtime-bridge/milestones/oto-registration-online-smoke.md` +- Completed task ids: + - `heartbeat-online`: PASS; evidence=`agent-task/archive/2026/06/m-oto-registration-online-smoke/03+01,02_smoke_evidence/plan_cloud_G07_0.log`, `agent-task/archive/2026/06/m-oto-registration-online-smoke/03+01,02_smoke_evidence/code_review_cloud_G07_0.log`, `agent-task/archive/2026/06/m-oto-registration-online-smoke/03+01,02_smoke_evidence/plan_local_G01_1.log`, `agent-task/archive/2026/06/m-oto-registration-online-smoke/03+01,02_smoke_evidence/code_review_local_G01_1.log`; verification=`make test-oto-registration-online-smoke` + - `unlock-evidence`: PASS; evidence=`agent-task/archive/2026/06/m-oto-registration-online-smoke/03+01,02_smoke_evidence/plan_cloud_G07_0.log`, `agent-task/archive/2026/06/m-oto-registration-online-smoke/03+01,02_smoke_evidence/code_review_cloud_G07_0.log`, `agent-task/archive/2026/06/m-oto-registration-online-smoke/03+01,02_smoke_evidence/plan_local_G01_1.log`, `agent-task/archive/2026/06/m-oto-registration-online-smoke/03+01,02_smoke_evidence/code_review_local_G01_1.log`; verification=`make test-oto-registration-online-smoke` +- Not completed task ids: 없음 + +## 잔여 Nit + +- 없음 + +## 후속 작업 + +- 없음 diff --git a/agent-task/archive/2026/06/m-oto-registration-online-smoke/03+01,02_smoke_evidence/plan_cloud_G07_0.log b/agent-task/archive/2026/06/m-oto-registration-online-smoke/03+01,02_smoke_evidence/plan_cloud_G07_0.log new file mode 100644 index 0000000..f7a713b --- /dev/null +++ b/agent-task/archive/2026/06/m-oto-registration-online-smoke/03+01,02_smoke_evidence/plan_cloud_G07_0.log @@ -0,0 +1,184 @@ + + +# OTO Registration Online Smoke Evidence + +## 이 파일을 읽는 구현 에이전트에게 + +이 plan은 실제 smoke/evidence 수집을 끝내는 작업입니다. 구현 후 active `CODE_REVIEW-*-G??.md`의 구현 소유 섹션에 실행 명령과 실제 stdout/stderr를 남기고, code-review가 PASS/complete/archive를 처리하게 두세요. + +## 배경 + +iop Edge lifecycle foundation과 OTO persistent session이 완료되어도 Milestone은 실제 same-host smoke evidence가 남아야 완료될 수 있다. 현재 OTO smoke는 accepted response만 확인하고, iop repo에는 OTO smoke를 재현하는 공식 Makefile/script entrypoint가 없다. 이 subtask는 same-host 기본 smoke를 재현 가능한 명령으로 묶고, heartbeat online과 unlock evidence를 남긴다. + +## 사용자 리뷰 요청 흐름 + +사용자-owned 외부 host, credential, private endpoint가 꼭 필요해지면 review stub의 `사용자 리뷰 요청`에 근거를 남긴다. local same-host smoke에서 해결 가능한 실패는 사용자 리뷰 요청이 아니라 후속 구현/검증 이슈다. + +## Roadmap Targets + +- Milestone: `agent-roadmap/phase/automation-runtime-bridge/milestones/oto-registration-online-smoke.md` +- Task ids: + - `heartbeat-online`: 최초 heartbeat 도착 후 online으로 전환된다. 검증: OTO Dart smoke 또는 통합 smoke에서 registration accepted 뒤 heartbeat 도착까지 확인한다. + - `unlock-evidence`: OTO `oto-agent` 등록 흐름 잠금 해제에 사용할 smoke 결과와 실행 명령이 문서 또는 test log에 남는다. +- Completion mode: check-on-pass + +## 분석 결과 + +### 읽은 파일 + +- `agent-roadmap/phase/automation-runtime-bridge/milestones/oto-registration-online-smoke.md` +- `agent-test/local/rules.md` +- `agent-test/local/edge-smoke.md` +- `agent-test/local/testing-smoke.md` +- `Makefile` +- `../oto/AGENTS.md` +- `../oto/agent-ops/rules/project/rules.md` +- `../oto/agent-ops/rules/project/domain/agent/rules.md` +- `../oto/agent-ops/rules/project/domain/framework/rules.md` +- `../oto/pubspec.yaml` +- `../oto/lib/oto/agent/edge_registration_client.dart` +- `../oto/lib/oto/agent/agent_runner.dart` +- `../oto/test/oto_iop_connection_smoke_test.dart` +- `scripts/e2e-smoke.sh` 일부: 기존 generic node smoke 패턴 확인용. 이 plan에서는 직접 수정하지 않는다. + +### 테스트 환경 규칙 + +- iop test_env: `local`, profiles `edge-smoke`, `testing-smoke`. +- iop required commands: target Go tests, changed user entrypoint full-cycle, optional `make test-e2e`. +- OTO `agent-test/local`은 없음. OTO domain fallback: `dart analyze`, agent tests, `test/oto_iop_connection_smoke_test.dart`. +- preflight: `command -v dart` 성공, but current OTO dependency solving failed before subtask 02. + +### 테스트 커버리지 공백 + +- OTO smoke does not assert heartbeat or Edge online event. +- iop Makefile lacks an OTO-specific smoke target. +- Current OTO smoke hardcodes `workingDirectory: '../iop'`, which is fragile for this workspace where current repo is `iop2oto`. +- No tracked evidence file/log command exists for OTO unlock handoff. + +### 심볼 참조 + +- 변경/추가 후보: Makefile `.PHONY` and test targets; no symbol rename expected. +- OTO smoke references `EdgeRegistrationClient.register()` at `../oto/test/oto_iop_connection_smoke_test.dart:64`. + +### 분할 판단 + +- This subtask is `03+01,02_smoke_evidence`. +- Required predecessors: + - `01_iop_edge_lifecycle`: no active/archive `complete.log` yet. + - `02+01_oto_agent_session`: no active/archive `complete.log` yet. +- The script/evidence work is split because it has cross-process verification and should not start until both code foundations are reviewed. + +### 범위 결정 근거 + +- Do not merge OTO smoke into generic `scripts/e2e-smoke.sh`; that script is generic Edge-Node smoke and already large. +- Do not implement build/deploy run payload, artifact stream, or Control Plane UI. +- Do not commit secret/token values; smoke uses generated temp dirs and non-secret test token. +- Split-host smoke is out of default path; support only documented env overrides if cheap. + +### 빌드 등급 + +- build: `cloud-G07`, review: `cloud-G07`. This is shell/CLI workflow and cross-repo process orchestration with evidence trust risk. + +## 의존 관계 및 구현 순서 + +이 subtask는 디렉터리명 `03+01,02_smoke_evidence`에 따라 아래 predecessor complete logs가 필요하다. + +- `01_iop_edge_lifecycle`: `agent-task/m-oto-registration-online-smoke/01_*/complete.log` 또는 archive equivalent. +- `02+01_oto_agent_session`: `agent-task/m-oto-registration-online-smoke/02+01_*/complete.log` 또는 archive equivalent. + +구현 시작 전 둘 다 정확히 하나씩 확인한다. 없거나 ambiguous하면 구현하지 말고 review stub에 blocker를 기록한다. + +## 구현 체크리스트 + +- [ ] OTO smoke가 iop repo path를 env로 받을 수 있게 하거나 iop-side wrapper가 현재 repo를 명시하게 한다. +- [ ] OTO Dart smoke 또는 iop wrapper가 registration accepted 뒤 heartbeat/online event를 확인한다. +- [ ] `make test-oto-registration-online-smoke` 또는 동등한 iop Makefile target을 추가한다. +- [ ] smoke output에 unlock handoff용 실행 명령과 accepted/heartbeat/online evidence가 남게 한다. +- [ ] `go test -count=1 ./apps/edge/internal/transport ./apps/edge/internal/node ./apps/edge/internal/service ./packages/go/events`와 OTO smoke target을 실행한다. +- [ ] CODE_REVIEW-*-G??.md의 구현 에이전트 소유 섹션을 실제 구현 내용과 검증 출력으로 채운다. 이 항목이 완료되기 전에는 구현이 완료된 것이 아니다. + +### [TEST-1] Make OTO Smoke Repo Path Explicit + +문제: [oto_iop_connection_smoke_test.dart](/config/workspace/oto/test/oto_iop_connection_smoke_test.dart:31)는 `workingDirectory: '../iop'`를 하드코딩한다. `iop2oto` clone에서는 실패한다. + +해결 방법: OTO test에 `IOP_REPO_ROOT` env override를 추가하거나, iop-side wrapper에서 OTO test를 실행할 때 cwd/env를 명시한다. 기존 same-host 기본은 유지한다. + +수정 파일 및 체크리스트: + +- [ ] `../oto/test/oto_iop_connection_smoke_test.dart` env override 추가 또는 wrapper-only 전략 선택. +- [ ] `../oto/test/oto_iop_connection_smoke_test.dart` temp config에 `agent_kind: "oto-agent"` 포함. +- [ ] hardcoded `../iop`가 남지 않는지 `rg --sort path "../iop|IOP_REPO_ROOT" ../oto/test/oto_iop_connection_smoke_test.dart`로 확인. + +테스트 작성: 기존 smoke test를 수정해 path override가 적용되는지 same test에서 사용한다. + +중간 검증: + +```bash +rg --sort path "../iop|IOP_REPO_ROOT|agent_kind" ../oto/test/oto_iop_connection_smoke_test.dart +``` + +예상 결과: `IOP_REPO_ROOT`와 `agent_kind`가 보이고 hardcoded-only path가 없다. + +### [TEST-2] Assert Accepted Then Heartbeat Online + +문제: [oto_iop_connection_smoke_test.dart](/config/workspace/oto/test/oto_iop_connection_smoke_test.dart:67)는 `result.accepted`만 확인한다. online event나 heartbeat evidence가 없다. + +해결 방법: OTO persistent session API로 registration 후 연결을 유지하고, iop Edge log/event output 또는 service-visible state를 확인한다. 가장 작은 경로는 Edge stdout/console log에 `node.online` or lifecycle online marker를 기다리는 것이다. + +수정 파일 및 체크리스트: + +- [ ] `../oto/test/oto_iop_connection_smoke_test.dart` accepted 뒤 online marker wait 추가. +- [ ] timeout failure message에 Edge output을 포함. +- [ ] test finally에서 persistent session과 Edge process를 닫는다. + +테스트 작성: existing smoke test를 heartbeat online까지 확장한다. + +중간 검증: + +```bash +cd ../oto && IOP_REPO_ROOT=/config/workspace/iop2oto dart test test/oto_iop_connection_smoke_test.dart +``` + +예상 결과: exit code 0, output contains accepted and online evidence. + +### [TEST-3] iop Makefile Target and Evidence Command + +문제: [Makefile](/config/workspace/iop2oto/Makefile:77)는 generic `test-e2e`만 있고 OTO online smoke target이 없다. + +해결 방법: `.PHONY`에 `test-oto-registration-online-smoke`를 추가하고 target은 `cd ../oto && IOP_REPO_ROOT=$(CURDIR) dart test test/oto_iop_connection_smoke_test.dart`를 실행한다. 필요하면 `scripts/e2e-oto-registration-online-smoke.sh`를 새로 만들되, generic `scripts/e2e-smoke.sh`에 섞지 않는다. + +수정 파일 및 체크리스트: + +- [ ] `Makefile` `.PHONY`에 target 추가. +- [ ] `Makefile` target 추가 또는 새 `scripts/e2e-oto-registration-online-smoke.sh` 추가. +- [ ] output에 unlock handoff용 command line을 echo한다. + +테스트 작성: Makefile target 자체가 smoke test다. 별도 unit test는 skip. + +중간 검증: + +```bash +make test-oto-registration-online-smoke +``` + +예상 결과: exit code 0. + +## 수정 파일 요약 + +| 파일 | 항목 | +|------|------| +| `../oto/test/oto_iop_connection_smoke_test.dart` | TEST-1, TEST-2 | +| `Makefile` | TEST-3 | +| `scripts/e2e-oto-registration-online-smoke.sh` | TEST-3, only if wrapper script is cleaner than inline Makefile command | + +## 최종 검증 + +```bash +go test -count=1 ./apps/edge/internal/transport ./apps/edge/internal/node ./apps/edge/internal/service ./packages/go/events +cd ../oto && dart analyze +make test-oto-registration-online-smoke +``` + +예상 결과: 모두 exit code 0. Smoke output must include the command used, registration accepted evidence, and first-heartbeat/online evidence. Go test cache output은 허용하지 않는다. + +모든 코드 변경 완료 후 반드시 `CODE_REVIEW-*-G??.md`의 구현 에이전트 소유 섹션을 채운다. 이 파일 작성이 구현의 마지막 단계다. diff --git a/agent-task/archive/2026/06/m-oto-registration-online-smoke/03+01,02_smoke_evidence/plan_local_G01_1.log b/agent-task/archive/2026/06/m-oto-registration-online-smoke/03+01,02_smoke_evidence/plan_local_G01_1.log new file mode 100644 index 0000000..e8a3d88 --- /dev/null +++ b/agent-task/archive/2026/06/m-oto-registration-online-smoke/03+01,02_smoke_evidence/plan_local_G01_1.log @@ -0,0 +1,82 @@ + + +# OTO Registration Online Smoke Formatter Follow-up + +## 이 파일을 읽는 구현 에이전트에게 + +이 plan은 1차 리뷰 WARN에서 발견된 Dart formatter cleanup만 수행한다. 기능 변경, smoke 구조 변경, 추가 리팩터링은 범위 밖이다. 구현 후 active `CODE_REVIEW-*-G??.md`의 구현 소유 섹션에 실제 명령과 stdout/stderr를 남기고, code-review가 archive/complete 처리를 맡게 둔다. + +## 배경 + +1차 `cloud-G07` 리뷰에서 OTO registration online smoke의 기능 검증은 통과했다. 다만 `dart format --output=none --set-exit-if-changed ...`가 `test/oto_iop_connection_smoke_test.dart` formatter 차이를 보고해 WARN으로 후속 루프가 필요하다. + +## 사용자 리뷰 요청 흐름 + +사용자-owned 외부 host, credential, private endpoint가 꼭 필요해지면 review stub의 `사용자 리뷰 요청`에 근거를 남긴다. 이 follow-up은 기본적으로 local formatter cleanup이므로, formatter/test 재실행으로 확인 가능한 실패는 사용자 리뷰 요청이 아니라 후속 구현/검증 이슈다. + +## Roadmap Targets + +- Milestone: `agent-roadmap/phase/automation-runtime-bridge/milestones/oto-registration-online-smoke.md` +- Task ids: + - `heartbeat-online`: 최초 heartbeat 도착 후 online으로 전환된다. 검증: OTO Dart smoke 또는 통합 smoke에서 registration accepted 뒤 heartbeat 도착까지 확인한다. + - `unlock-evidence`: OTO `oto-agent` 등록 흐름 잠금 해제에 사용할 smoke 결과와 실행 명령이 문서 또는 test log에 남는다. +- Completion mode: check-on-pass + +## 범위 결정 근거 + +- 이전 루프에서 `go test`, `dart analyze`, `make test-oto-registration-online-smoke`는 통과했다. +- WARN 사유는 formatter-clean 하나이므로, 이번 루프는 Dart format 적용과 그에 따른 검증 재확인만 수행한다. +- `Makefile`, Edge lifecycle code, OTO persistent session API, smoke assertion 로직은 변경하지 않는다. + +## 구현 체크리스트 + +- [ ] `../oto/test/oto_iop_connection_smoke_test.dart`를 Dart formatter 기준으로 정리한다. +- [ ] `cd ../oto && dart format --output=none --set-exit-if-changed test/oto_iop_connection_smoke_test.dart lib/oto/agent/edge_registration_client.dart lib/oto/agent/agent_runner.dart test/oto_agent_registration_test.dart`를 실행해 formatter-clean을 확인한다. +- [ ] `cd ../oto && dart analyze`와 `make test-oto-registration-online-smoke`를 실행해 기존 analyzer/smoke evidence가 유지되는지 확인한다. +- [ ] CODE_REVIEW-*-G??.md의 구현 에이전트 소유 섹션을 실제 구현 내용과 검증 출력으로 채운다. 이 항목이 완료되기 전에는 구현이 완료된 것이 아니다. + +### [REVIEW_TEST-1] Format OTO Smoke Test + +문제: `../oto/test/oto_iop_connection_smoke_test.dart:211`의 긴 삼항 할당이 Dart formatter 기준을 만족하지 않는다. 1차 리뷰에서 아래 명령이 실패했다. + +```bash +cd ../oto && dart format --output=none --set-exit-if-changed test/oto_iop_connection_smoke_test.dart lib/oto/agent/edge_registration_client.dart lib/oto/agent/agent_runner.dart test/oto_agent_registration_test.dart +``` + +출력 요약: + +```text +Changed test/oto_iop_connection_smoke_test.dart +Formatted 4 files (1 changed) in 0.13 seconds. +``` + +해결 방법: `../oto/test/oto_iop_connection_smoke_test.dart`에 `dart format`을 적용한다. formatter가 바꾸는 줄 외의 기능 변경은 하지 않는다. + +수정 파일 및 체크리스트: + +- [ ] `../oto/test/oto_iop_connection_smoke_test.dart` format 적용. +- [ ] 기능 로직이나 smoke timeout/marker/command는 변경하지 않는다. + +중간 검증: + +```bash +cd ../oto && dart format --output=none --set-exit-if-changed test/oto_iop_connection_smoke_test.dart lib/oto/agent/edge_registration_client.dart lib/oto/agent/agent_runner.dart test/oto_agent_registration_test.dart +``` + +예상 결과: exit code 0, `Formatted 4 files (0 changed)`. + +## 수정 파일 요약 + +| 파일 | 항목 | +|------|------| +| `../oto/test/oto_iop_connection_smoke_test.dart` | REVIEW_TEST-1 | + +## 최종 검증 + +```bash +cd ../oto && dart format --output=none --set-exit-if-changed test/oto_iop_connection_smoke_test.dart lib/oto/agent/edge_registration_client.dart lib/oto/agent/agent_runner.dart test/oto_agent_registration_test.dart +cd ../oto && dart analyze +make test-oto-registration-online-smoke +``` + +예상 결과: 모두 exit code 0. Smoke output은 command, registration accepted, `node.online`/`first_heartbeat` evidence를 유지해야 한다. diff --git a/apps/edge/cmd/edge/edge.yaml b/apps/edge/cmd/edge/edge.yaml new file mode 100644 index 0000000..5882b57 --- /dev/null +++ b/apps/edge/cmd/edge/edge.yaml @@ -0,0 +1,30 @@ +edge: + id: "edge-local" + name: "Local Edge" + +server: + listen: "0.0.0.0:9090" + advertise_host: "" + +bootstrap: + listen: "0.0.0.0:18080" + artifact_base_url: "" + artifact_dir: "artifacts" + +tls: + enabled: false + +logging: + level: "info" + pretty: false + path: "" + +metrics: + port: 9092 + +control_plane: + enabled: false + wire_addr: "" + reconnect_interval_sec: 5 + +nodes: [] diff --git a/apps/edge/cmd/edge/main.go b/apps/edge/cmd/edge/main.go index 7be0a7c..482f9e9 100644 --- a/apps/edge/cmd/edge/main.go +++ b/apps/edge/cmd/edge/main.go @@ -49,7 +49,7 @@ It manages token-based node registration, node registry, node configuration deli The official local development and field test flow is as follows: 1. config init - Write a clean local edge.yaml configuration template. 2. env - Verify resolved local environment, log paths, and listen URLs. - 3. node register - Add a node record to edge.yaml and generate its one-line bootstrap command. + 3. node register - Add a node record to edge.yaml and generate its one-line bootstrap command. (Use 'agent register' for specialized OTO agents.) 4. config check - Validate the edge.yaml configuration structure and constraints. 5. serve (or console) - Start the Edge server (or interactive console) to accept connections. 6. nodes list - List configured nodes and show their connection status. @@ -57,13 +57,14 @@ The official local development and field test flow is as follows: Example: ` iop-edge config init iop-edge env iop-edge node register my-node --adapter cli + iop-edge agent register my-agent --bootstrap-script-url https://example.com/boot.sh --release-base-url https://example.com/release iop-edge config check iop-edge serve iop-edge nodes list iop-edge smoke openai --model gemma4:26b`, } root.PersistentFlags().StringVarP(&cfgFile, "config", "c", "", "config file path (defaults to bundle-local edge.yaml)") - root.AddCommand(serveCmd(), consoleCmd(), versionCmd(), configCmd(), envCmd(), setupCmd(), bootstrapCmd(), nodeCmd(), nodesCmd(), smokeCmd()) + root.AddCommand(serveCmd(), consoleCmd(), versionCmd(), configCmd(), envCmd(), setupCmd(), bootstrapCmd(), nodeCmd(), nodesCmd(), smokeCmd(), agentCmd()) return root } @@ -821,15 +822,19 @@ exec "$BIN_FILE" serve --config "$CONFIG_FILE" } var ( - regAlias string - regToken string - regAdapter string - regOllamaBaseURL string - regOllamaContextSize int - regRuntimeConcurrency int - regWorkspaceRoot string - regTarget string - regArtifactBaseURL string + regAlias string + regToken string + regAdapter string + regOllamaBaseURL string + regOllamaContextSize int + regRuntimeConcurrency int + regWorkspaceRoot string + regTarget string + regArtifactBaseURL string + regEdgeURL string + regOTOBootstrapScriptURL string + regOTOReleaseBaseURL string + regAllowInsecureBootstrapURL bool ) var tokenGenerator = func() string { @@ -880,7 +885,7 @@ func validateEdgeConfig(cfg *config.EdgeConfig) error { name = fmt.Sprintf("index %d", i) } - if !n.Adapters.Ollama.Enabled && !n.Adapters.CLI.Enabled && !n.Adapters.Vllm.Enabled { + if n.AgentKind != config.AgentKindOTOAgent && !n.Adapters.Ollama.Enabled && !n.Adapters.CLI.Enabled && !n.Adapters.Vllm.Enabled { return fmt.Errorf("node %q: at least one adapter must be enabled", name) } if n.Adapters.Ollama.Enabled && n.Adapters.Ollama.BaseURL == "" { @@ -893,6 +898,158 @@ func validateEdgeConfig(cfg *config.EdgeConfig) error { return nil } +func agentCmd() *cobra.Command { + c := &cobra.Command{ + Use: "agent", + Short: "Manage specialized agents", + Long: `Manage agent records stored in edge.yaml. + +Use agent register to create or update the agent definition and print the +one-line bootstrap command for the target host.`, + } + c.AddCommand(agentRegisterCmd()) + return c +} + +func agentRegisterCmd() *cobra.Command { + c := &cobra.Command{ + Use: "register [agent-id]", + Short: "Register or update a specialized agent record", + Long: `Register or update an agent record in edge.yaml and print a one-line bootstrap command. + +The command writes the agent id, alias, token, and agent_kind into edge.yaml. +The printed bootstrap command is the user-facing command to run on the agent host.`, + Example: ` iop-edge agent register my-agent --bootstrap-script-url https://example.com/boot.sh --release-base-url https://example.com/release + iop-edge config check`, + Args: cobra.ExactArgs(1), + RunE: func(cmd *cobra.Command, args []string) error { + agentID := args[0] + + path, err := resolveConfigPath(cmd) + if err != nil { + return err + } + + cfgPtr, err := config.LoadEdge(path) + if err != nil { + return fmt.Errorf("load config: %w", err) + } + cfg := *cfgPtr + + data, err := os.ReadFile(path) + if err != nil { + return fmt.Errorf("read config file: %w", err) + } + + var bootstrapScriptURL string + if cmd.Flags().Changed("bootstrap-script-url") { + bootstrapScriptURL = regOTOBootstrapScriptURL + } else { + bootstrapScriptURL = cfg.Bootstrap.OTOBootstrapScriptURL + } + + if bootstrapScriptURL == "" { + return errors.New("validate agent register: bootstrap-script-url must not be empty (set via flag or config)") + } + + if strings.HasPrefix(bootstrapScriptURL, "http://") { + if !regAllowInsecureBootstrapURL { + return errors.New("validate agent register: bootstrap-script-url must start with https:// unless --allow-insecure-bootstrap-url is set") + } + } else if !strings.HasPrefix(bootstrapScriptURL, "https://") { + return errors.New("validate agent register: bootstrap-script-url must start with https:// (or http:// if allowed)") + } + + var releaseBaseURL string + if cmd.Flags().Changed("release-base-url") { + releaseBaseURL = regOTOReleaseBaseURL + } else { + releaseBaseURL = cfg.Bootstrap.OTOReleaseBaseURL + } + + if releaseBaseURL == "" { + return errors.New("validate agent register: release-base-url must not be empty (set via flag or config)") + } + + if !strings.HasPrefix(releaseBaseURL, "https://") { + return errors.New("validate agent register: release-base-url must start with https://") + } + + alias := agentID + if cmd.Flags().Changed("alias") { + alias = regAlias + } + + token := regToken + if token == "" { + token = tokenGenerator() + } + + edgeURL := regEdgeURL + if edgeURL == "" { + adv := resolveAdvertiseHost(cfg.Server.AdvertiseHost) + edgeURL = addressFor(adv.Host, cfg.Server.Listen) + } + + var nodeDef *config.NodeDefinition + exists := false + for i, n := range cfg.Nodes { + if n.ID == agentID { + nodeDef = &cfg.Nodes[i] + exists = true + break + } + } + + if exists { + nodeDef.Alias = alias + nodeDef.Token = token + nodeDef.AgentKind = config.AgentKindOTOAgent + nodeDef.Adapters = config.AdaptersConf{} + } else { + newDef := config.NodeDefinition{ + ID: agentID, + Alias: alias, + Token: token, + AgentKind: config.AgentKindOTOAgent, + } + cfg.Nodes = append(cfg.Nodes, newDef) + nodeDef = &cfg.Nodes[len(cfg.Nodes)-1] + } + + cfg.Bootstrap.OTOBootstrapScriptURL = bootstrapScriptURL + cfg.Bootstrap.OTOReleaseBaseURL = releaseBaseURL + + if err := validateEdgeConfig(&cfg); err != nil { + return fmt.Errorf("validate agent register: %w", err) + } + + outData, err := patchYAML(data, &cfg) + if err != nil { + return fmt.Errorf("patch config: %w", err) + } + if err := os.WriteFile(path, outData, 0o600); err != nil { + return fmt.Errorf("write config file: %w", err) + } + + // curl -fsSL | bash -s -- --edge-url --agent-id --enrollment-token --release-base-url --agent-alias + fmt.Fprintf(cmd.OutOrStdout(), "curl -fsSL %s | bash -s -- --edge-url %s --agent-id %s --enrollment-token %s --release-base-url %s --agent-alias %s\n", + bootstrapScriptURL, edgeURL, nodeDef.ID, nodeDef.Token, releaseBaseURL, nodeDef.Alias) + + return nil + }, + } + + c.Flags().StringVar(®Alias, "alias", "", "agent alias (defaults to agent-id)") + c.Flags().StringVar(®Token, "token", "", "agent enrollment token (defaults to auto-generated uuid)") + c.Flags().StringVar(®EdgeURL, "edge-url", "", "edge URL for agent registration (defaults to server advertise address)") + c.Flags().StringVar(®OTOBootstrapScriptURL, "bootstrap-script-url", "", "bootstrap script URL for OTO agent") + c.Flags().StringVar(®OTOReleaseBaseURL, "release-base-url", "", "release base URL for OTO agent") + c.Flags().BoolVar(®AllowInsecureBootstrapURL, "allow-insecure-bootstrap-url", false, "allow insecure http:// bootstrap script URL") + + return c +} + func nodeCmd() *cobra.Command { c := &cobra.Command{ Use: "node", @@ -1160,41 +1317,18 @@ func patchYAML(data []byte, cfg *config.EdgeConfig) ([]byte, error) { rootMap.Content = append(rootMap.Content, keyNode, nodesValNode) } - // 2. Patch "bootstrap.artifact_base_url" if bootstrap config has artifact URL + // 2. Patch bootstrap fields if any are present in the config + fieldsToPatch := make(map[string]string) if cfg.Bootstrap.ArtifactBaseURL != "" { - foundBootstrap := false - for i := 0; i < len(rootMap.Content); i += 2 { - if rootMap.Content[i].Value == "bootstrap" { - bootstrapVal := rootMap.Content[i+1] - if bootstrapVal.Kind == yaml.MappingNode { - foundURL := false - for j := 0; j < len(bootstrapVal.Content); j += 2 { - if bootstrapVal.Content[j].Value == "artifact_base_url" { - bootstrapVal.Content[j+1].Value = cfg.Bootstrap.ArtifactBaseURL - bootstrapVal.Content[j+1].Tag = "!!str" - foundURL = true - break - } - } - if !foundURL { - urlKey := &yaml.Node{Kind: yaml.ScalarNode, Tag: "!!str", Value: "artifact_base_url"} - urlVal := &yaml.Node{Kind: yaml.ScalarNode, Tag: "!!str", Value: cfg.Bootstrap.ArtifactBaseURL} - bootstrapVal.Content = append(bootstrapVal.Content, urlKey, urlVal) - } - } - foundBootstrap = true - break - } - } - if !foundBootstrap { - bootstrapKey := &yaml.Node{Kind: yaml.ScalarNode, Tag: "!!str", Value: "bootstrap"} - bootstrapVal := &yaml.Node{Kind: yaml.MappingNode, Tag: "!!map"} - urlKey := &yaml.Node{Kind: yaml.ScalarNode, Tag: "!!str", Value: "artifact_base_url"} - urlVal := &yaml.Node{Kind: yaml.ScalarNode, Tag: "!!str", Value: cfg.Bootstrap.ArtifactBaseURL} - bootstrapVal.Content = append(bootstrapVal.Content, urlKey, urlVal) - rootMap.Content = append(rootMap.Content, bootstrapKey, bootstrapVal) - } + fieldsToPatch["artifact_base_url"] = cfg.Bootstrap.ArtifactBaseURL } + if cfg.Bootstrap.OTOBootstrapScriptURL != "" { + fieldsToPatch["oto_bootstrap_script_url"] = cfg.Bootstrap.OTOBootstrapScriptURL + } + if cfg.Bootstrap.OTOReleaseBaseURL != "" { + fieldsToPatch["oto_release_base_url"] = cfg.Bootstrap.OTOReleaseBaseURL + } + patchBootstrapStringFields(rootMap, fieldsToPatch) out, err := yaml.Marshal(&doc) if err != nil { @@ -1203,6 +1337,55 @@ func patchYAML(data []byte, cfg *config.EdgeConfig) ([]byte, error) { return out, nil } +func patchBootstrapStringFields(rootMap *yaml.Node, fields map[string]string) { + if len(fields) == 0 { + return + } + + var bootstrapVal *yaml.Node + foundBootstrap := false + for i := 0; i < len(rootMap.Content); i += 2 { + if rootMap.Content[i].Value == "bootstrap" { + bootstrapVal = rootMap.Content[i+1] + foundBootstrap = true + break + } + } + + if !foundBootstrap { + bootstrapKey := &yaml.Node{Kind: yaml.ScalarNode, Tag: "!!str", Value: "bootstrap"} + bootstrapVal = &yaml.Node{Kind: yaml.MappingNode, Tag: "!!map"} + rootMap.Content = append(rootMap.Content, bootstrapKey, bootstrapVal) + } + + if bootstrapVal.Kind != yaml.MappingNode { + return + } + + // Iterate in a deterministic order + keys := []string{"artifact_base_url", "oto_bootstrap_script_url", "oto_release_base_url"} + for _, k := range keys { + v, ok := fields[k] + if !ok || v == "" { + continue + } + foundField := false + for j := 0; j < len(bootstrapVal.Content); j += 2 { + if bootstrapVal.Content[j].Value == k { + bootstrapVal.Content[j+1].Value = v + bootstrapVal.Content[j+1].Tag = "!!str" + foundField = true + break + } + } + if !foundField { + keyNode := &yaml.Node{Kind: yaml.ScalarNode, Tag: "!!str", Value: k} + valNode := &yaml.Node{Kind: yaml.ScalarNode, Tag: "!!str", Value: v} + bootstrapVal.Content = append(bootstrapVal.Content, keyNode, valNode) + } + } +} + func nodesCmd() *cobra.Command { c := &cobra.Command{ Use: "nodes", diff --git a/apps/edge/cmd/edge/main_test.go b/apps/edge/cmd/edge/main_test.go index ed82707..2a25b2a 100644 --- a/apps/edge/cmd/edge/main_test.go +++ b/apps/edge/cmd/edge/main_test.go @@ -639,6 +639,35 @@ nodes: } } +func TestConfigCheckAcceptsOTOAgentWithoutAdapters(t *testing.T) { + dir := t.TempDir() + cfgPath := filepath.Join(dir, "edge.yaml") + + yamlStr := `server: + listen: "0.0.0.0:9090" +nodes: + - id: "oto-agent-a" + alias: "oto-a" + token: "oto-token-a" + agent_kind: "oto-agent" +` + if err := os.WriteFile(cfgPath, []byte(yamlStr), 0o600); err != nil { + t.Fatalf("write yaml: %v", err) + } + + root := rootCmd() + var out bytes.Buffer + root.SetOut(&out) + root.SetErr(&out) + root.SetArgs([]string{"config", "check", "--config", cfgPath}) + if err := root.Execute(); err != nil { + t.Fatalf("expected OTO agent config to check without adapters: %v\n%s", err, out.String()) + } + if !strings.Contains(out.String(), "OK "+cfgPath) { + t.Fatalf("expected config check OK for OTO agent config, got %q", out.String()) + } +} + func TestNodeRegisterUpsertsYAMLAndOutputsBootstrap(t *testing.T) { // Mock token generator origGen := tokenGenerator @@ -1110,6 +1139,16 @@ func TestHelpDiscoversEdgeLocalDevFlow(t *testing.T) { args: []string{"node", "register", "--help"}, want: []string{"one-line bootstrap command", "node.yaml", "iop-node serve", "--adapter", "--target"}, }, + { + name: "agent", + args: []string{"agent", "--help"}, + want: []string{"agent register", "edge.yaml"}, + }, + { + name: "agent register", + args: []string{"agent", "register", "--help"}, + want: []string{"one-line bootstrap command", "edge.yaml", "--alias", "--token", "--bootstrap-script-url", "--release-base-url"}, + }, { name: "smoke", args: []string{"smoke", "--help"}, @@ -1304,3 +1343,277 @@ func TestSmokeOpenAICommandFailure(t *testing.T) { t.Errorf("unexpected error message: %v", err3) } } + +func TestPatchYAMLUpdatesOTOBootstrapFields(t *testing.T) { + origYAML := `server: + listen: "0.0.0.0:9090" +bootstrap: + artifact_base_url: "http://example.com/artifacts" +` + cfg := &config.EdgeConfig{ + Bootstrap: config.EdgeBootstrapConf{ + ArtifactBaseURL: "http://example.com/artifacts", + OTOBootstrapScriptURL: "https://example.com/bootstrap.sh", + OTOReleaseBaseURL: "https://example.com/releases", + }, + } + + patched, err := patchYAML([]byte(origYAML), cfg) + if err != nil { + t.Fatalf("patchYAML: %v", err) + } + + var doc struct { + Server struct { + Listen string `yaml:"listen"` + } `yaml:"server"` + Bootstrap struct { + ArtifactBaseURL string `yaml:"artifact_base_url"` + OTOBootstrapScriptURL string `yaml:"oto_bootstrap_script_url"` + OTOReleaseBaseURL string `yaml:"oto_release_base_url"` + } `yaml:"bootstrap"` + } + + if err := yaml.Unmarshal(patched, &doc); err != nil { + t.Fatalf("unmarshal patched: %v", err) + } + + if doc.Server.Listen != "0.0.0.0:9090" { + t.Errorf("server.listen was modified: %q", doc.Server.Listen) + } + if doc.Bootstrap.ArtifactBaseURL != "http://example.com/artifacts" { + t.Errorf("artifact_base_url = %q", doc.Bootstrap.ArtifactBaseURL) + } + if doc.Bootstrap.OTOBootstrapScriptURL != "https://example.com/bootstrap.sh" { + t.Errorf("oto_bootstrap_script_url = %q", doc.Bootstrap.OTOBootstrapScriptURL) + } + if doc.Bootstrap.OTOReleaseBaseURL != "https://example.com/releases" { + t.Errorf("oto_release_base_url = %q", doc.Bootstrap.OTOReleaseBaseURL) + } +} + +func TestAgentRegisterUpsertsOTOAgentRecord(t *testing.T) { + tmpdir := t.TempDir() + cfgPath := filepath.Join(tmpdir, "edge.yaml") + initialYAML := `server: + listen: "127.0.0.1:9090" + advertise_host: "127.0.0.1" +bootstrap: + oto_bootstrap_script_url: "https://example.com/boot.sh" + oto_release_base_url: "https://example.com/releases" +nodes: [] +` + if err := os.WriteFile(cfgPath, []byte(initialYAML), 0o600); err != nil { + t.Fatalf("write initial config: %v", err) + } + + root := rootCmd() + var out bytes.Buffer + root.SetOut(&out) + root.SetErr(&out) + root.SetArgs([]string{"--config", cfgPath, "agent", "register", "oto-agent-1", "--alias", "custom-alias", "--token", "custom-token"}) + + if err := root.Execute(); err != nil { + t.Fatalf("agent register failed: %v", err) + } + + cfg, err := config.LoadEdge(cfgPath) + if err != nil { + t.Fatalf("load config failed: %v", err) + } + + if len(cfg.Nodes) != 1 { + t.Fatalf("expected 1 node record, got %d", len(cfg.Nodes)) + } + + n := cfg.Nodes[0] + if n.ID != "oto-agent-1" { + t.Errorf("expected ID oto-agent-1, got %s", n.ID) + } + if n.Alias != "custom-alias" { + t.Errorf("expected Alias custom-alias, got %s", n.Alias) + } + if n.Token != "custom-token" { + t.Errorf("expected Token custom-token, got %s", n.Token) + } + if n.AgentKind != config.AgentKindOTOAgent { + t.Errorf("expected AgentKind %q, got %s", config.AgentKindOTOAgent, n.AgentKind) + } + if n.Adapters.Ollama.Enabled || n.Adapters.CLI.Enabled { + t.Errorf("expected adapters to be disabled for OTO agent, got Ollama=%v, CLI=%v", n.Adapters.Ollama.Enabled, n.Adapters.CLI.Enabled) + } +} + +func TestAgentRegisterPrintsOneLineOTOCommand(t *testing.T) { + tmpdir := t.TempDir() + cfgPath := filepath.Join(tmpdir, "edge.yaml") + initialYAML := `server: + listen: "127.0.0.1:9090" + advertise_host: "127.0.0.1" +bootstrap: + oto_bootstrap_script_url: "https://example.com/boot.sh" + oto_release_base_url: "https://example.com/releases" +nodes: [] +` + if err := os.WriteFile(cfgPath, []byte(initialYAML), 0o600); err != nil { + t.Fatalf("write initial config: %v", err) + } + + root := rootCmd() + var out bytes.Buffer + root.SetOut(&out) + root.SetErr(&out) + root.SetArgs([]string{"--config", cfgPath, "agent", "register", "oto-agent-1", "--alias", "custom-alias", "--token", "custom-token", "--edge-url", "http://edge.test:9090"}) + + if err := root.Execute(); err != nil { + t.Fatalf("agent register failed: %v", err) + } + + got := out.String() + want := "curl -fsSL https://example.com/boot.sh | bash -s -- --edge-url http://edge.test:9090 --agent-id oto-agent-1 --enrollment-token custom-token --release-base-url https://example.com/releases --agent-alias custom-alias\n" + if got != want { + t.Errorf("unexpected output:\n got: %q\nwant: %q", got, want) + } +} + +func TestAgentRegisterRejectsInsecureBootstrapURLWithoutFlag(t *testing.T) { + tmpdir := t.TempDir() + cfgPath := filepath.Join(tmpdir, "edge.yaml") + initialYAML := `server: + listen: "127.0.0.1:9090" + advertise_host: "127.0.0.1" +bootstrap: + oto_release_base_url: "https://example.com/releases" +nodes: [] +` + if err := os.WriteFile(cfgPath, []byte(initialYAML), 0o600); err != nil { + t.Fatalf("write initial config: %v", err) + } + + root := rootCmd() + var out bytes.Buffer + root.SetOut(&out) + root.SetErr(&out) + root.SetArgs([]string{"--config", cfgPath, "agent", "register", "oto-agent-1", "--bootstrap-script-url", "http://example.com/boot.sh"}) + + err := root.Execute() + if err == nil { + t.Fatal("expected error, got nil") + } + if !strings.Contains(err.Error(), "bootstrap-script-url must start with https:// unless --allow-insecure-bootstrap-url is set") { + t.Errorf("unexpected error: %v", err) + } +} + +func TestAgentRegisterRejectsInsecureReleaseBaseURL(t *testing.T) { + tmpdir := t.TempDir() + cfgPath := filepath.Join(tmpdir, "edge.yaml") + initialYAML := `server: + listen: "127.0.0.1:9090" + advertise_host: "127.0.0.1" +bootstrap: + oto_bootstrap_script_url: "https://example.com/boot.sh" +nodes: [] +` + if err := os.WriteFile(cfgPath, []byte(initialYAML), 0o600); err != nil { + t.Fatalf("write initial config: %v", err) + } + + root := rootCmd() + var out bytes.Buffer + root.SetOut(&out) + root.SetErr(&out) + root.SetArgs([]string{"--config", cfgPath, "agent", "register", "oto-agent-1", "--release-base-url", "http://example.com/releases"}) + + err := root.Execute() + if err == nil { + t.Fatal("expected error, got nil") + } + if !strings.Contains(err.Error(), "release-base-url must start with https://") { + t.Errorf("unexpected error: %v", err) + } +} + +func TestAgentRegisterAllowsInsecureBootstrapURLWithExplicitFlag(t *testing.T) { + tmpdir := t.TempDir() + cfgPath := filepath.Join(tmpdir, "edge.yaml") + initialYAML := `server: + listen: "127.0.0.1:9090" + advertise_host: "127.0.0.1" +bootstrap: + oto_release_base_url: "https://example.com/releases" +nodes: [] +` + if err := os.WriteFile(cfgPath, []byte(initialYAML), 0o600); err != nil { + t.Fatalf("write initial config: %v", err) + } + + root := rootCmd() + var out bytes.Buffer + root.SetOut(&out) + root.SetErr(&out) + root.SetArgs([]string{"--config", cfgPath, "agent", "register", "oto-agent-1", "--bootstrap-script-url", "http://example.com/boot.sh", "--allow-insecure-bootstrap-url"}) + + if err := root.Execute(); err != nil { + t.Fatalf("agent register failed with flag: %v", err) + } + + got := out.String() + if !strings.Contains(got, "http://example.com/boot.sh") { + t.Errorf("output should contain http bootstrap URL, got: %q", got) + } +} + +func TestTempFileCLIWorkflowConfigInitAgentRegisterConfigCheck(t *testing.T) { + tmpdir := t.TempDir() + cfgPath := filepath.Join(tmpdir, "edge.yaml") + t.Chdir(tmpdir) + + // 1. config init + root := rootCmd() + var out bytes.Buffer + root.SetOut(&out) + root.SetErr(&out) + root.SetArgs([]string{"config", "init", "--output", cfgPath}) + if err := root.Execute(); err != nil { + t.Fatalf("config init failed: %v", err) + } + + // 2. agent register + root = rootCmd() + out.Reset() + root.SetOut(&out) + root.SetErr(&out) + root.SetArgs([]string{ + "--config", cfgPath, + "agent", "register", "oto-agent-test", + "--bootstrap-script-url", "https://oto.example.test/bootstrap/oto_agent_bootstrap.sh", + "--release-base-url", "https://oto.example.test/releases", + "--token", "test-token", + "--edge-url", "localhost:9090", + }) + if err := root.Execute(); err != nil { + t.Fatalf("agent register failed: %v", err) + } + + // Assert exact-output format + gotCmd := out.String() + wantCmd := "curl -fsSL https://oto.example.test/bootstrap/oto_agent_bootstrap.sh | bash -s -- --edge-url localhost:9090 --agent-id oto-agent-test --enrollment-token test-token --release-base-url https://oto.example.test/releases --agent-alias oto-agent-test\n" + if gotCmd != wantCmd { + t.Errorf("unexpected printed command:\n got: %q\nwant: %q", gotCmd, wantCmd) + } + + // 3. config check + root = rootCmd() + out.Reset() + root.SetOut(&out) + root.SetErr(&out) + root.SetArgs([]string{"--config", cfgPath, "config", "check"}) + if err := root.Execute(); err != nil { + t.Fatalf("config check failed: %v", err) + } + gotCheck := out.String() + if !strings.Contains(gotCheck, "OK") { + t.Errorf("expected OK, got: %q", gotCheck) + } +} diff --git a/apps/edge/internal/node/registry.go b/apps/edge/internal/node/registry.go index 7d97ead..ed27f68 100644 --- a/apps/edge/internal/node/registry.go +++ b/apps/edge/internal/node/registry.go @@ -12,8 +12,12 @@ import ( "iop/packages/go/config" ) -// LifecycleConnected is the default lifecycle state for a freshly registered entry. -const LifecycleConnected = "connected" +const ( + LifecycleAccepted = "accepted" + LifecycleOnline = "online" + LifecycleFailed = "failed" + LifecycleConnected = "connected" +) // NodeEntry represents one connected node. type NodeEntry struct { @@ -74,6 +78,17 @@ func (r *Registry) Register(entry *NodeEntry) { } } +func (r *Registry) UpdateLifecycle(nodeID string, state string) bool { + r.mu.Lock() + defer r.mu.Unlock() + entry, ok := r.byID[nodeID] + if !ok { + return false + } + entry.LifecycleState = state + return true +} + func (e *NodeEntry) DisplayLabel() string { if e == nil { return "unknown" diff --git a/apps/edge/internal/node/registry_test.go b/apps/edge/internal/node/registry_test.go index dd112fc..d90742a 100644 --- a/apps/edge/internal/node/registry_test.go +++ b/apps/edge/internal/node/registry_test.go @@ -126,3 +126,26 @@ func TestRegistry_UnregisterRemovesAlias(t *testing.T) { t.Error("alias should be removed after unregister") } } + +func TestRegistryUpdateLifecycle(t *testing.T) { + reg := edgenode.NewRegistry() + + if ok := reg.UpdateLifecycle("non-existent", edgenode.LifecycleOnline); ok { + t.Error("expected false for updating non-existent node") + } + + entry := &edgenode.NodeEntry{NodeID: "node-1", Alias: "alias-1"} + reg.Register(entry) + + if ok := reg.UpdateLifecycle("node-1", edgenode.LifecycleOnline); !ok { + t.Error("expected true for updating existing node") + } + + retrieved, ok := reg.Get("node-1") + if !ok { + t.Fatal("failed to get node") + } + if retrieved.LifecycleState != edgenode.LifecycleOnline { + t.Errorf("lifecycle: got %q, want %q", retrieved.LifecycleState, edgenode.LifecycleOnline) + } +} diff --git a/apps/edge/internal/transport/integration_test.go b/apps/edge/internal/transport/integration_test.go index 4d3510c..513d63c 100644 --- a/apps/edge/internal/transport/integration_test.go +++ b/apps/edge/internal/transport/integration_test.go @@ -11,6 +11,7 @@ import ( "google.golang.org/protobuf/proto" toki "git.toki-labs.com/toki/proto-socket/go" + "git.toki-labs.com/toki/proto-socket/go/packets" edgenode "iop/apps/edge/internal/node" "iop/apps/edge/internal/transport" @@ -114,8 +115,8 @@ func TestEdgeServerOTORegistrationKind(t *testing.T) { if entry.AgentKind != config.AgentKindOTOAgent { t.Fatalf("agent kind: got %q want %q", entry.AgentKind, config.AgentKindOTOAgent) } - if entry.LifecycleState != edgenode.LifecycleConnected { - t.Fatalf("lifecycle: got %q want %q", entry.LifecycleState, edgenode.LifecycleConnected) + if entry.LifecycleState != edgenode.LifecycleAccepted { + t.Fatalf("lifecycle: got %q want %q", entry.LifecycleState, edgenode.LifecycleAccepted) } } @@ -331,3 +332,291 @@ func TestEdgeServerIntegration(t *testing.T) { t.Fatal("timeout waiting for node disconnected event") } } + +func TestEdgeServerOTOFirstHeartbeatMarksOnline(t *testing.T) { + ctx, cancel := context.WithTimeout(context.Background(), 5*time.Second) + defer cancel() + + logger := zap.NewNop() + listenAddr := getFreePort(t) + registry := edgenode.NewRegistry() + nodeStore, err := edgenode.LoadFromConfig([]config.NodeDefinition{ + { + ID: "oto-hb-01", + Alias: "oto-hb-node", + Token: "oto-hb-token", + AgentKind: config.AgentKindOTOAgent, + }, + }) + if err != nil { + t.Fatalf("load node store: %v", err) + } + + server, err := transport.NewServer(listenAddr, registry, nodeStore, logger) + if err != nil { + t.Fatalf("new server: %v", err) + } + + nodeEventCh := make(chan *iop.EdgeNodeEvent, 10) + server.SetNodeEventHandler(func(event *iop.EdgeNodeEvent) { + nodeEventCh <- event + }) + + if err := server.Start(ctx); err != nil { + t.Fatalf("start server: %v", err) + } + defer server.Stop() + + client := dialNode(t, ctx, listenAddr) + defer client.Close() + + resp, err := toki.SendRequestTyped[*iop.RegisterRequest, *iop.RegisterResponse]( + &client.Communicator, + &iop.RegisterRequest{Token: "oto-hb-token"}, + 2*time.Second, + ) + if err != nil { + t.Fatalf("register request: %v", err) + } + if !resp.GetAccepted() { + t.Fatalf("expected accepted, got reason %q", resp.GetReason()) + } + + select { + case event := <-nodeEventCh: + if event.GetType() != eventpkg.TypeNodeConnected { + t.Fatalf("expected connected event, got %q", event.GetType()) + } + if event.GetMetadata()[eventpkg.MetadataLifecycleState] != edgenode.LifecycleAccepted { + t.Fatalf("expected lifecycle accepted, got %q", event.GetMetadata()[eventpkg.MetadataLifecycleState]) + } + case <-time.After(2 * time.Second): + t.Fatal("timeout waiting for node connected event") + } + + entry, ok := waitForRegistryEntry(ctx, registry, "oto-hb-01") + if !ok || entry.LifecycleState != edgenode.LifecycleAccepted { + t.Fatalf("expected registry state accepted, ok: %v, state: %q", ok, entry.LifecycleState) + } + + err = client.Send(&packets.HeartBeat{}) + if err != nil { + t.Fatalf("send heartbeat: %v", err) + } + + select { + case event := <-nodeEventCh: + if event.GetType() != eventpkg.TypeNodeOnline { + t.Fatalf("expected online event, got %q", event.GetType()) + } + if event.GetReason() != eventpkg.ReasonFirstHeartbeat { + t.Fatalf("expected reason first_heartbeat, got %q", event.GetReason()) + } + if event.GetMetadata()[eventpkg.MetadataLifecycleState] != edgenode.LifecycleOnline { + t.Fatalf("expected lifecycle online, got %q", event.GetMetadata()[eventpkg.MetadataLifecycleState]) + } + case <-time.After(2 * time.Second): + t.Fatal("timeout waiting for node online event") + } + + if entry.LifecycleState != edgenode.LifecycleOnline { + t.Fatalf("expected registry state online, got %q", entry.LifecycleState) + } +} + +func TestEdgeServerRegistrationFailureReasons(t *testing.T) { + ctx, cancel := context.WithTimeout(context.Background(), 5*time.Second) + defer cancel() + + logger := zap.NewNop() + listenAddr := getFreePort(t) + registry := edgenode.NewRegistry() + nodeStore, err := edgenode.LoadFromConfig([]config.NodeDefinition{ + { + ID: "node-fail-01", + Alias: "fail-node", + Token: "valid-token", + AgentKind: config.AgentKindGenericNode, + }, + }) + if err != nil { + t.Fatalf("load node store: %v", err) + } + + server, err := transport.NewServer(listenAddr, registry, nodeStore, logger) + if err != nil { + t.Fatalf("new server: %v", err) + } + + nodeEventCh := make(chan *iop.EdgeNodeEvent, 10) + server.SetNodeEventHandler(func(event *iop.EdgeNodeEvent) { + nodeEventCh <- event + }) + + if err := server.Start(ctx); err != nil { + t.Fatalf("start server: %v", err) + } + defer server.Stop() + + // 1. Unknown Token Test + client1 := dialNode(t, ctx, listenAddr) + resp1, err := toki.SendRequestTyped[*iop.RegisterRequest, *iop.RegisterResponse]( + &client1.Communicator, + &iop.RegisterRequest{Token: "unknown-token-12345"}, + 2*time.Second, + ) + if err != nil { + t.Fatalf("send register 1: %v", err) + } + if resp1.GetAccepted() { + t.Fatal("expected rejected") + } + + select { + case event := <-nodeEventCh: + if event.GetType() != eventpkg.TypeNodeRegistrationFailed { + t.Fatalf("expected registration failed type, got %q", event.GetType()) + } + if event.GetReason() != eventpkg.ReasonUnknownToken { + t.Fatalf("expected reason unknown_token, got %q", event.GetReason()) + } + if event.GetMetadata()[eventpkg.MetadataFailureReason] != eventpkg.ReasonUnknownToken { + t.Fatalf("expected metadata failure reason, got %q", event.GetMetadata()[eventpkg.MetadataFailureReason]) + } + if event.GetMetadata()[eventpkg.MetadataTokenPrefix] != "unknown-..." { + t.Fatalf("expected token prefix unknown-..., got %q", event.GetMetadata()[eventpkg.MetadataTokenPrefix]) + } + case <-time.After(2 * time.Second): + t.Fatal("timeout waiting for unknown token registration failed event") + } + client1.Close() + + // 2. Duplicate Connection Test + client2 := dialNode(t, ctx, listenAddr) + resp2, err := toki.SendRequestTyped[*iop.RegisterRequest, *iop.RegisterResponse]( + &client2.Communicator, + &iop.RegisterRequest{Token: "valid-token"}, + 2*time.Second, + ) + if err != nil { + t.Fatalf("send register 2: %v", err) + } + if !resp2.GetAccepted() { + t.Fatalf("expected register 2 accepted, got %q", resp2.GetReason()) + } + // Consume connection event + <-nodeEventCh + + client3 := dialNode(t, ctx, listenAddr) + resp3, err := toki.SendRequestTyped[*iop.RegisterRequest, *iop.RegisterResponse]( + &client3.Communicator, + &iop.RegisterRequest{Token: "valid-token"}, + 2*time.Second, + ) + if err != nil { + t.Fatalf("send register 3: %v", err) + } + if resp3.GetAccepted() { + t.Fatal("expected duplicate connection registration rejected") + } + + select { + case event := <-nodeEventCh: + if event.GetType() != eventpkg.TypeNodeRegistrationFailed { + t.Fatalf("expected registration failed type, got %q", event.GetType()) + } + if event.GetReason() != eventpkg.ReasonDuplicateConnection { + t.Fatalf("expected reason duplicate_connection, got %q", event.GetReason()) + } + if event.GetMetadata()[eventpkg.MetadataFailureReason] != eventpkg.ReasonDuplicateConnection { + t.Fatalf("expected failure reason metadata, got %q", event.GetMetadata()[eventpkg.MetadataFailureReason]) + } + if event.GetMetadata()[eventpkg.MetadataAgentKind] != config.AgentKindGenericNode { + t.Fatalf("expected agent kind metadata, got %q", event.GetMetadata()[eventpkg.MetadataAgentKind]) + } + case <-time.After(2 * time.Second): + t.Fatal("timeout waiting for duplicate registration failed event") + } + client2.Close() + client3.Close() +} + +func TestEdgeServerHeartbeatTimeoutDisconnect(t *testing.T) { + ctx, cancel := context.WithTimeout(context.Background(), 5*time.Second) + defer cancel() + + logger := zap.NewNop() + listenAddr := getFreePort(t) + registry := edgenode.NewRegistry() + nodeStore, err := edgenode.LoadFromConfig([]config.NodeDefinition{ + { + ID: "node-timeout-01", + Alias: "timeout-node", + Token: "timeout-token", + AgentKind: config.AgentKindOTOAgent, + }, + }) + if err != nil { + t.Fatalf("load node store: %v", err) + } + + server, err := transport.NewServer(listenAddr, registry, nodeStore, logger) + if err != nil { + t.Fatalf("new server: %v", err) + } + server.HeartbeatInterval = 1 + server.HeartbeatWait = 1 + + nodeEventCh := make(chan *iop.EdgeNodeEvent, 10) + server.SetNodeEventHandler(func(event *iop.EdgeNodeEvent) { + nodeEventCh <- event + }) + + if err := server.Start(ctx); err != nil { + t.Fatalf("start server: %v", err) + } + defer server.Stop() + + client := dialNode(t, ctx, listenAddr) + defer client.Close() + client.RemoveListeners(toki.TypeNameOf(&packets.HeartBeat{})) + + resp, err := toki.SendRequestTyped[*iop.RegisterRequest, *iop.RegisterResponse]( + &client.Communicator, + &iop.RegisterRequest{Token: "timeout-token"}, + 2*time.Second, + ) + if err != nil { + t.Fatalf("register request: %v", err) + } + if !resp.GetAccepted() { + t.Fatalf("expected accepted, got %q", resp.GetReason()) + } + + select { + case event := <-nodeEventCh: + if event.GetType() != eventpkg.TypeNodeConnected { + t.Fatalf("expected connected event, got %q", event.GetType()) + } + case <-time.After(2 * time.Second): + t.Fatal("timeout waiting for connected event") + } + + select { + case event := <-nodeEventCh: + if event.GetType() != eventpkg.TypeNodeDisconnected { + t.Fatalf("expected disconnected event, got %q", event.GetType()) + } + if event.GetReason() != eventpkg.ReasonHeartbeatTimeout { + t.Fatalf("expected reason heartbeat_timeout, got %q", event.GetReason()) + } + if event.GetMetadata()[eventpkg.MetadataFailureReason] != eventpkg.ReasonHeartbeatTimeout { + t.Fatalf("expected failure reason heartbeat_timeout, got %q", event.GetMetadata()[eventpkg.MetadataFailureReason]) + } + if event.GetMetadata()[eventpkg.MetadataLifecycleState] != edgenode.LifecycleFailed { + t.Fatalf("expected lifecycle state failed, got %q", event.GetMetadata()[eventpkg.MetadataLifecycleState]) + } + case <-time.After(3 * time.Second): + t.Fatal("timeout waiting for heartbeat timeout disconnected event") + } +} diff --git a/apps/edge/internal/transport/server.go b/apps/edge/internal/transport/server.go index 1af6cd1..e0ed19b 100644 --- a/apps/edge/internal/transport/server.go +++ b/apps/edge/internal/transport/server.go @@ -8,6 +8,7 @@ import ( "sync/atomic" toki "git.toki-labs.com/toki/proto-socket/go" + "git.toki-labs.com/toki/proto-socket/go/packets" "go.uber.org/zap" "google.golang.org/protobuf/proto" @@ -47,15 +48,17 @@ func edgeParserMap() toki.ParserMap { // Server wraps proto-socket TcpServer and manages node connections. type Server struct { - tcp *toki.TcpServer - listen string - registry *edgenode.Registry - nodeStore *edgenode.NodeStore - logger *zap.Logger - handlerMu sync.RWMutex - onRunEvent func(*iop.RunEvent) - onNodeEvent func(*iop.EdgeNodeEvent) - stopping atomic.Bool + tcp *toki.TcpServer + listen string + registry *edgenode.Registry + nodeStore *edgenode.NodeStore + logger *zap.Logger + handlerMu sync.RWMutex + onRunEvent func(*iop.RunEvent) + onNodeEvent func(*iop.EdgeNodeEvent) + stopping atomic.Bool + HeartbeatInterval int + HeartbeatWait int } func NewServer(listen string, registry *edgenode.Registry, nodeStore *edgenode.NodeStore, logger *zap.Logger) (*Server, error) { @@ -68,9 +71,16 @@ func NewServer(listen string, registry *edgenode.Registry, nodeStore *edgenode.N return nil, err } - s := &Server{listen: listen, registry: registry, nodeStore: nodeStore, logger: logger} + s := &Server{ + listen: listen, + registry: registry, + nodeStore: nodeStore, + logger: logger, + HeartbeatInterval: heartbeatIntervalSec, + HeartbeatWait: heartbeatWaitSec, + } s.tcp = toki.NewTcpServer(host, port, func(conn net.Conn) *toki.TcpClient { - return toki.NewTcpClient(conn, heartbeatIntervalSec, heartbeatWaitSec, edgeParserMap()) + return toki.NewTcpClient(conn, s.HeartbeatInterval, s.HeartbeatWait, edgeParserMap()) }) s.tcp.OnClientConnected = s.onNodeConnected return s, nil @@ -137,6 +147,17 @@ func (s *Server) onNodeConnected(client *toki.TcpClient) { rec, ok := s.nodeStore.FindByToken(req.GetToken()) if !ok { s.logger.Warn("unknown token", zap.String("token_prefix", safePrefix(req.GetToken()))) + s.emitNodeEvent(events.NewEdgeNodeEvent( + events.SourceEdge, + events.TypeNodeRegistrationFailed, + "", + "", + events.ReasonUnknownToken, + map[string]string{ + events.MetadataFailureReason: events.ReasonUnknownToken, + events.MetadataTokenPrefix: safePrefix(req.GetToken()), + }, + )) return &iop.RegisterResponse{Accepted: false, Reason: "unknown token"}, nil } @@ -149,6 +170,17 @@ func (s *Server) onNodeConnected(client *toki.TcpClient) { zap.String("node_id", rec.ID), zap.String("agent_kind", rec.AgentKind), ) + s.emitNodeEvent(events.NewEdgeNodeEvent( + events.SourceEdge, + events.TypeNodeRegistrationFailed, + rec.ID, + rec.Alias, + events.ReasonDuplicateConnection, + map[string]string{ + events.MetadataFailureReason: events.ReasonDuplicateConnection, + events.MetadataAgentKind: rec.AgentKind, + }, + )) return &iop.RegisterResponse{Accepted: false, Reason: reason}, nil } @@ -161,13 +193,18 @@ func (s *Server) onNodeConnected(client *toki.TcpClient) { return &iop.RegisterResponse{Accepted: false, Reason: "internal config error"}, nil } + lifecycle := edgenode.LifecycleConnected + if rec.AgentKind == config.AgentKindOTOAgent { + lifecycle = edgenode.LifecycleAccepted + } entry := &edgenode.NodeEntry{ - NodeID: rec.ID, - Alias: rec.Alias, - AgentKind: rec.AgentKind, - Client: client, - Index: rec.Index, - HasIndex: true, + NodeID: rec.ID, + Alias: rec.Alias, + AgentKind: rec.AgentKind, + LifecycleState: lifecycle, + Client: client, + Index: rec.Index, + HasIndex: true, } toki.AddListenerTyped[*iop.EdgeNodeEvent](&client.Communicator, func(e *iop.EdgeNodeEvent) { if e.NodeId == "" { @@ -178,22 +215,58 @@ func (s *Server) onNodeConnected(client *toki.TcpClient) { } s.emitNodeEvent(e) }) + if rec.AgentKind == config.AgentKindOTOAgent { + var onlineOnce sync.Once + toki.AddListenerTyped[*packets.HeartBeat](&client.Communicator, func(*packets.HeartBeat) { + onlineOnce.Do(func() { + if s.registry.UpdateLifecycle(rec.ID, edgenode.LifecycleOnline) { + s.emitNodeEvent(events.NewEdgeNodeEvent( + events.SourceEdge, + events.TypeNodeOnline, + rec.ID, + rec.Alias, + events.ReasonFirstHeartbeat, + map[string]string{ + events.MetadataAgentKind: rec.AgentKind, + events.MetadataLifecycleState: edgenode.LifecycleOnline, + }, + )) + } + }) + }) + } client.AddDisconnectListener(func(_ *toki.TcpClient) { transportInfo := client.DisconnectInfo() fields := []zap.Field{zap.String("node_id", rec.ID)} fields = append(fields, transportDisconnectFields(transportInfo)...) s.logger.Info("node unregistered", fields...) reason := events.ReasonTransportClosed - if s.stopping.Load() { + if transportInfo.Reason == "heartbeat_timeout" { + reason = events.ReasonHeartbeatTimeout + s.registry.UpdateLifecycle(rec.ID, edgenode.LifecycleFailed) + } else if s.stopping.Load() { reason = events.ReasonEdgeShutdown } + + meta := transportDisconnectMetadata(transportInfo) + if meta == nil { + meta = make(map[string]string) + } + if reason == events.ReasonHeartbeatTimeout { + meta[events.MetadataFailureReason] = events.ReasonHeartbeatTimeout + meta[events.MetadataLifecycleState] = edgenode.LifecycleFailed + } else { + meta[events.MetadataLifecycleState] = edgenode.LifecycleFailed + } + meta[events.MetadataAgentKind] = rec.AgentKind + s.emitNodeEvent(events.NewEdgeNodeEvent( events.SourceEdge, events.TypeNodeDisconnected, rec.ID, rec.Alias, reason, - transportDisconnectMetadata(transportInfo), + meta, )) s.registry.Unregister(rec.ID) }) @@ -208,7 +281,10 @@ func (s *Server) onNodeConnected(client *toki.TcpClient) { rec.ID, rec.Alias, events.ReasonRegistered, - nil, + map[string]string{ + events.MetadataAgentKind: rec.AgentKind, + events.MetadataLifecycleState: lifecycle, + }, )) return &iop.RegisterResponse{ diff --git a/configs/edge.yaml b/configs/edge.yaml index bfc8b14..56488b2 100644 --- a/configs/edge.yaml +++ b/configs/edge.yaml @@ -13,6 +13,8 @@ server: # field channel is provisioned. bootstrap: artifact_base_url: "" + oto_bootstrap_script_url: "" + oto_release_base_url: "" tls: enabled: false diff --git a/packages/go/config/config.go b/packages/go/config/config.go index 2aecab8..2ca5651 100644 --- a/packages/go/config/config.go +++ b/packages/go/config/config.go @@ -84,9 +84,11 @@ type EdgeServerConf struct { } type EdgeBootstrapConf struct { - ArtifactBaseURL string `mapstructure:"artifact_base_url" yaml:"artifact_base_url"` - Listen string `mapstructure:"listen" yaml:"listen"` - ArtifactDir string `mapstructure:"artifact_dir" yaml:"artifact_dir"` + ArtifactBaseURL string `mapstructure:"artifact_base_url" yaml:"artifact_base_url"` + Listen string `mapstructure:"listen" yaml:"listen"` + ArtifactDir string `mapstructure:"artifact_dir" yaml:"artifact_dir"` + OTOBootstrapScriptURL string `mapstructure:"oto_bootstrap_script_url" yaml:"oto_bootstrap_script_url"` + OTOReleaseBaseURL string `mapstructure:"oto_release_base_url" yaml:"oto_release_base_url"` } type EdgeOpenAIConf struct { @@ -267,6 +269,8 @@ func setEdgeDefaults(v *viper.Viper) { v.SetDefault("server.listen", "0.0.0.0:9090") v.SetDefault("bootstrap.listen", "0.0.0.0:18080") v.SetDefault("bootstrap.artifact_dir", "artifacts") + v.SetDefault("bootstrap.oto_bootstrap_script_url", "") + v.SetDefault("bootstrap.oto_release_base_url", "") v.SetDefault("openai.enabled", false) v.SetDefault("openai.listen", "0.0.0.0:8080") v.SetDefault("openai.adapter", "ollama") diff --git a/packages/go/config/config_test.go b/packages/go/config/config_test.go index e08964c..b106c0d 100644 --- a/packages/go/config/config_test.go +++ b/packages/go/config/config_test.go @@ -722,6 +722,48 @@ logging: } } +func TestLoadEdge_OTOBootstrapDefaults(t *testing.T) { + dir := t.TempDir() + f := filepath.Join(dir, "edge.yaml") + if err := os.WriteFile(f, []byte("server:\n listen: \"0.0.0.0:9090\"\n"), 0o600); err != nil { + t.Fatalf("write yaml: %v", err) + } + cfg, err := config.LoadEdge(f) + if err != nil { + t.Fatalf("load: %v", err) + } + if cfg.Bootstrap.OTOBootstrapScriptURL != "" { + t.Errorf("expected empty bootstrap.oto_bootstrap_script_url by default, got %q", cfg.Bootstrap.OTOBootstrapScriptURL) + } + if cfg.Bootstrap.OTOReleaseBaseURL != "" { + t.Errorf("expected empty bootstrap.oto_release_base_url by default, got %q", cfg.Bootstrap.OTOReleaseBaseURL) + } +} + +func TestLoadEdge_OTOBootstrapExplicit(t *testing.T) { + dir := t.TempDir() + f := filepath.Join(dir, "edge.yaml") + yaml := `server: + listen: "0.0.0.0:9090" +bootstrap: + oto_bootstrap_script_url: "https://example.com/bootstrap.sh" + oto_release_base_url: "https://example.com/releases" +` + if err := os.WriteFile(f, []byte(yaml), 0o600); err != nil { + t.Fatalf("write yaml: %v", err) + } + cfg, err := config.LoadEdge(f) + if err != nil { + t.Fatalf("load: %v", err) + } + if cfg.Bootstrap.OTOBootstrapScriptURL != "https://example.com/bootstrap.sh" { + t.Errorf("bootstrap.oto_bootstrap_script_url=%q", cfg.Bootstrap.OTOBootstrapScriptURL) + } + if cfg.Bootstrap.OTOReleaseBaseURL != "https://example.com/releases" { + t.Errorf("bootstrap.oto_release_base_url=%q", cfg.Bootstrap.OTOReleaseBaseURL) + } +} + func TestLoadEdge_A2ADefaults(t *testing.T) { dir := t.TempDir() f := filepath.Join(dir, "edge.yaml") diff --git a/packages/go/events/events.go b/packages/go/events/events.go index 3208367..089bcdf 100644 --- a/packages/go/events/events.go +++ b/packages/go/events/events.go @@ -12,17 +12,27 @@ const ( SourceEdge = "edge" SourceNode = "node" - TypeNodeConnected = "node.connected" - TypeNodeDisconnected = "node.disconnected" - TypeEdgeDisconnected = "edge.disconnected" + TypeNodeConnected = "node.connected" + TypeNodeDisconnected = "node.disconnected" + TypeEdgeDisconnected = "edge.disconnected" + TypeNodeOnline = "node.online" + TypeNodeRegistrationFailed = "node.registration_failed" - ReasonRegistered = "registered" - ReasonTransportClosed = "transport_closed" - ReasonLocalShutdown = "local_shutdown" - ReasonEdgeShutdown = "edge_shutdown" + ReasonRegistered = "registered" + ReasonTransportClosed = "transport_closed" + ReasonLocalShutdown = "local_shutdown" + ReasonEdgeShutdown = "edge_shutdown" + ReasonFirstHeartbeat = "first_heartbeat" + ReasonUnknownToken = "unknown_token" + ReasonDuplicateConnection = "duplicate_connection" + ReasonHeartbeatTimeout = "heartbeat_timeout" MetadataTransportCloseReason = "transport_close_reason" MetadataTransportCloseError = "transport_close_error" + MetadataAgentKind = "agent_kind" + MetadataLifecycleState = "lifecycle_state" + MetadataFailureReason = "failure_reason" + MetadataTokenPrefix = "token_prefix" ) func NewEdgeNodeEvent(source, eventType, nodeID, alias, reason string, metadata map[string]string) *iop.EdgeNodeEvent {