toki
c46874055a
- Add edge node unit tests and transport package - Add node test client and related artifacts - Update bootstrap, node, and config modules - Add proto generated files - Update Makefile and configuration files
110 lines
3.7 KiB
Text
110 lines
3.7 KiB
Text
<!-- task=edge_node_arch plan=1 tag=REVIEW_REFACTOR -->
|
|
|
|
# Code Review Reference - REVIEW_REFACTOR
|
|
|
|
## 개요
|
|
|
|
date=2026-05-02
|
|
task=edge_node_arch, plan=1, tag=REVIEW_REFACTOR
|
|
|
|
## 이 파일을 읽는 리뷰 에이전트에게
|
|
|
|
각 항목의 구현을 실제 소스 파일과 대조하고, `검증 결과` 섹션의 출력이 코드와 일치하는지 확인하세요.
|
|
리뷰 완료 후 반드시 아래 순서로 아카이브하세요.
|
|
|
|
1. `CODE_REVIEW.md` → `code_review_1.log` (N = 기존 code_review_*.log 수)
|
|
2. `PLAN.md` → `plan_1.log` (M = 기존 plan_*.log 수)
|
|
3. PASS인 경우 `complete.log` 작성 후 종료. WARN/FAIL인 경우 새 `PLAN.md` + `CODE_REVIEW.md` 스텁 작성.
|
|
|
|
---
|
|
|
|
## 구현 항목별 완료 여부
|
|
|
|
| 항목 | 완료 여부 |
|
|
|------|---------|
|
|
| [REVIEW_REFACTOR-1] onNodeConnected disconnect 리스너 순서 수정 | [x] |
|
|
|
|
## 계획 대비 변경 사항
|
|
|
|
없음.
|
|
|
|
## 주요 설계 결정
|
|
|
|
- `CapabilityResponse`를 받아 `NodeEntry`를 만든 직후 disconnect listener를 먼저 등록하고, 그 다음 registry에 등록한다.
|
|
- listener는 기존과 동일하게 해당 nodeID를 registry에서 제거하고 `node unregistered` 로그를 남긴다.
|
|
|
|
## 리뷰어를 위한 체크포인트
|
|
|
|
- `apps/edge/internal/transport/server.go`의 `onNodeConnected` 고루틴에서 `client.AddDisconnectListener(...)` 블록이 `s.registry.Register(entry)` 이전에 위치하는지 확인
|
|
- `go build ./apps/edge/...`가 오류 없이 통과하는지 확인
|
|
- `go build ./...`와 `go test ./...`가 모두 통과하는지 확인
|
|
|
|
## 검증 결과
|
|
|
|
_구현 에이전트가 각 중간 검증 및 최종 검증 명령 실행 후 출력을 여기에 붙여 넣는다._
|
|
|
|
### REVIEW_REFACTOR-1 중간 검증
|
|
```
|
|
$ PATH=/config/go-sdk/go/bin:/config/go-sdk/bin:$PATH go build ./apps/edge/...
|
|
(no output)
|
|
```
|
|
|
|
### 최종 검증
|
|
```
|
|
$ PATH=/config/go-sdk/go/bin:/config/go-sdk/bin:$PATH go build ./...
|
|
(no output)
|
|
|
|
$ PATH=/config/go-sdk/go/bin:/config/go-sdk/bin:$PATH go test ./...
|
|
? iop/apps/control-plane/cmd/iop-control-plane [no test files]
|
|
? iop/apps/edge/cmd/iop-edge [no test files]
|
|
? iop/apps/edge/internal/bootstrap [no test files]
|
|
ok iop/apps/edge/internal/node (cached)
|
|
? iop/apps/edge/internal/transport [no test files]
|
|
? iop/apps/node/cmd/iop-node [no test files]
|
|
? iop/apps/node/internal/adapters [no test files]
|
|
? iop/apps/node/internal/adapters/cli [no test files]
|
|
? iop/apps/node/internal/adapters/mock [no test files]
|
|
? iop/apps/node/internal/adapters/ollama [no test files]
|
|
? iop/apps/node/internal/adapters/vllm [no test files]
|
|
? iop/apps/node/internal/bootstrap [no test files]
|
|
? iop/apps/node/internal/node [no test files]
|
|
? iop/apps/node/internal/router [no test files]
|
|
? iop/apps/node/internal/runtime [no test files]
|
|
? iop/apps/node/internal/store [no test files]
|
|
? iop/apps/node/internal/transport [no test files]
|
|
? iop/apps/worker/cmd/iop-worker [no test files]
|
|
? iop/packages/auth [no test files]
|
|
? iop/packages/config [no test files]
|
|
? iop/packages/jobs [no test files]
|
|
? iop/packages/metadata [no test files]
|
|
? iop/packages/observability [no test files]
|
|
? iop/packages/policy [no test files]
|
|
? iop/packages/version [no test files]
|
|
? iop/proto/gen/iop [no test files]
|
|
```
|
|
|
|
---
|
|
|
|
## 코드리뷰 결과
|
|
|
|
### 종합 판정: PASS
|
|
|
|
### 차원별 평가
|
|
|
|
| 차원 | 판정 |
|
|
|------|------|
|
|
| Correctness | Pass |
|
|
| Completeness | Pass |
|
|
| Test coverage | Pass |
|
|
| API contract | Pass |
|
|
| Code quality | Pass |
|
|
| Plan deviation | Pass |
|
|
| Verification trust | Pass |
|
|
|
|
### 발견된 문제
|
|
|
|
없음. `server.go:101-106` 기준으로 `AddDisconnectListener`가 `registry.Register` 이전에 위치함이 확인됨. race window 제거.
|
|
|
|
### 다음 단계
|
|
|
|
PASS: `complete.log` 작성 후 종료.
|