iop/agent-task/m-multi-edge-operations/01_native_contract/CODE_REVIEW-cloud-G06.md

1.5 KiB

Code Review - Native Fleet Contract

Review Scope

  • proto/iop/control.proto
  • proto generated files
  • apps/control-plane/internal/wire/edge.go
  • apps/edge/internal/controlplane/connector.go
  • ownership guard tests

Roadmap Targets

  • agent-roadmap/phase/control-plane-portal-ops/milestones/multi-edge-operations.md [native-fleet]
  • agent-roadmap/phase/control-plane-portal-ops/milestones/multi-edge-operations.md [edge-ownership]

Reviewer Checklist

  • Control Plane이 Edge 설정, Node registry, runtime/automation state 원본을 소유하는 필드나 저장소가 생기지 않았다.
  • OTO/build-deploy는 capability/status/command summary로만 표현된다.
  • command 요청/응답/event relay 메시지가 request/result 추적에 충분하다.
  • parser map이 양쪽에서 대칭적으로 갱신되었다.
  • proto 생성물이 누락되지 않았다.
  • 실패/unknown enum 처리 테스트가 있다.

Verification Result

  • make proto
  • make proto-dart
  • go test ./apps/control-plane/internal/wire ./apps/edge/internal/controlplane
  • 필요한 경우 cd apps/client && flutter test

User Review

검토 기본값: 이 리뷰는 사용자가 명시적으로 보류하지 않으면 구현자가 자체 검증 후 완료 처리한다.

Notes

  • 리뷰 중 발견한 계약 변경은 후속 plan의 구현 전제에 영향을 주므로 milestone 작업 맥락에 함께 반영한다.