toki
554e7ef269
- opencode SSE 스트림 파서 어댑터 추가 (opencode_sse.go) - 블랙박스/내부 테스트 파일 추가 - node/edge 애플리케이션의 CLI 어댑터 개선 - 설정 파일 및 README 업데이트 - agent-task opencode_sse_stream 추가
135 lines
6.1 KiB
Text
135 lines
6.1 KiB
Text
<!-- task=opencode_sse_stream plan=1 tag=REVIEW_OPENCODE_SSE -->
|
|
|
|
# Code Review Reference - REVIEW_OPENCODE_SSE
|
|
|
|
## 개요
|
|
|
|
date=2026-05-11
|
|
task=opencode_sse_stream, plan=1, tag=REVIEW_OPENCODE_SSE
|
|
|
|
## 이 파일을 읽는 리뷰 에이전트에게
|
|
|
|
각 항목의 구현을 실제 소스 파일과 대조하고, `검증 결과` 섹션의 출력이 코드와 일치하는지 확인하세요.
|
|
리뷰 완료 후 반드시 아래 순서로 아카이브하세요.
|
|
|
|
1. `CODE_REVIEW-cloud-G07.md` → `code_review_cloud_G07_N.log` (N = 기존 code_review_*.log 수)
|
|
2. `PLAN-cloud-G07.md` → `plan_cloud_G07_M.log` (M = 기존 plan_*.log 수)
|
|
3. PASS인 경우 `complete.log` 작성 후 종료. WARN/FAIL인 경우 새 routed plan + review 스텁 작성.
|
|
|
|
---
|
|
|
|
## 구현 항목별 완료 여부
|
|
|
|
| 항목 | 완료 여부 |
|
|
|------|---------|
|
|
| [REVIEW_OPENCODE_SSE-1] prompt_async model payload를 OpenCode schema에 맞춘다 | [x] |
|
|
| [REVIEW_OPENCODE_SSE-2] session.status 객체 idle을 complete로 처리한다 | [x] |
|
|
|
|
## 계획 대비 변경 사항
|
|
|
|
- 계획의 helper 이름을 그대로 채용했다(`opencodeModelPayload`, `opencodeStatusIdle`). complete event emit 중복 제거는 plan이 허용한 작은 refactor로 한정해 `emitOpencodeComplete` helper를 추가하고 `session.idle`, `session.status` (object/string), scan-EOF 3개 경로에서 공유한다.
|
|
- 1차 구현의 `StreamsTextDeltas` 테스트가 `--model m1`을 사용했는데, model 객체화 후에는 invalid 값이 되어 동일 케이스를 `prov/m1`로 정정하고 body assertion(`model.providerID`, `model.modelID`, `parts[0].text`)을 추가했다. 별도 케이스 추가 대신 기존 테스트를 강화한 것은 plan이 "TestCLIExecuteOpencodeSSE_StreamsTextDeltas에 prompt body assertion을 추가한다"고 명시했기 때문이다.
|
|
- model parse 실패는 prompt_async POST 이전에 검출되므로 fake server는 prompt 요청을 받지 않는다. 그 흐름을 검증하는 `TestCLIExecuteOpencodeSSE_InvalidModelEmitsError`를 추가했다. error event emit은 `executeOpencodeSSE`의 prompt POST 실패 분기에서 처리한다.
|
|
- 검증 명령은 plan에 명시된 그대로 사용했다. 대체 없음.
|
|
|
|
## 주요 설계 결정
|
|
|
|
- `opencodeModelPayload`는 첫 `/`만으로 split한다. 즉 `openrouter/google/gemini-2.5-pro`는 provider=`openrouter`, modelID=`google/gemini-2.5-pro`로 들어간다. OpenRouter 같은 multi-segment model id 호환을 우선했다.
|
|
- `opencodeStatusIdle`은 `string`과 `map[string]any{"type":"idle"}` 둘 다 idle로 인정한다. 옛 OpenCode build와 신 build 양쪽 호환을 위해 두 형태를 모두 처리한다.
|
|
- complete emit은 단일 helper로 통일했지만, cancel 경로(`finalize`)와 error 경로는 별도 emit/return을 유지한다. complete 의미와 cancel/error 의미를 섞지 않기 위함이다.
|
|
|
|
## 리뷰어를 위한 체크포인트
|
|
|
|
- `--model provider/model`이 `prompt_async` body에서 `{"providerID": provider, "modelID": model}` 객체로 전송되는지 확인한다.
|
|
- model string이 잘못된 경우 silent ignore가 아니라 error event와 error return으로 드러나는지 확인한다.
|
|
- `session.status`의 `status`가 `{"type":"idle"}` 객체일 때 complete event가 emit되는지 확인한다.
|
|
- 새 tests가 fake server에서 request body와 status object event를 실제로 검증하는지 확인한다.
|
|
|
|
## 검증 결과
|
|
|
|
_구현 에이전트가 각 중간 검증 및 최종 검증 명령 실행 후 출력을 여기에 붙여 넣는다._
|
|
|
|
필수 규칙:
|
|
- 검증 명령은 고정된 계약이다. 임의로 대체하지 않는다.
|
|
- 대체가 필요하면 `계획 대비 변경 사항`에 이유와 대체 명령을 기록한다.
|
|
- `검증 결과`에는 실제 stdout/stderr를 붙여 넣는다.
|
|
|
|
### REVIEW_OPENCODE_SSE-1 중간 검증
|
|
```
|
|
$ go test -count=1 ./apps/node/internal/adapters/cli
|
|
ok iop/apps/node/internal/adapters/cli 87.559s
|
|
```
|
|
|
|
### REVIEW_OPENCODE_SSE-2 중간 검증
|
|
```
|
|
$ go test -count=1 ./apps/node/internal/adapters/cli
|
|
ok iop/apps/node/internal/adapters/cli 87.559s
|
|
```
|
|
|
|
### 최종 검증
|
|
```
|
|
$ go test -count=1 ./apps/node/internal/adapters/cli ./apps/node/internal/adapters ./apps/edge/internal/node ./apps/edge/internal/transport ./packages/config
|
|
ok iop/apps/node/internal/adapters/cli 87.535s
|
|
ok iop/apps/node/internal/adapters 0.004s
|
|
ok iop/apps/edge/internal/node 0.004s
|
|
ok iop/apps/edge/internal/transport 0.006s
|
|
ok iop/packages/config 0.007s
|
|
```
|
|
|
|
```
|
|
$ go test -count=1 ./...
|
|
? iop/apps/control-plane/cmd/control-plane [no test files]
|
|
ok iop/apps/edge/cmd/edge 0.005s
|
|
? iop/apps/edge/internal/bootstrap [no test files]
|
|
ok iop/apps/edge/internal/node 0.007s
|
|
ok iop/apps/edge/internal/transport 0.007s
|
|
? iop/apps/node/cmd/node [no test files]
|
|
ok iop/apps/node/internal/adapters 0.005s
|
|
ok iop/apps/node/internal/adapters/cli 87.567s
|
|
? iop/apps/node/internal/adapters/cli/internal/testutil [no test files]
|
|
ok iop/apps/node/internal/adapters/cli/status 0.003s
|
|
? 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]
|
|
ok iop/apps/node/internal/bootstrap 0.165s
|
|
ok iop/apps/node/internal/node 0.009s
|
|
ok iop/apps/node/internal/router 0.004s
|
|
? iop/apps/node/internal/runtime [no test files]
|
|
ok iop/apps/node/internal/store 0.023s
|
|
ok iop/apps/node/internal/transport 0.006s
|
|
? iop/apps/worker/cmd/worker [no test files]
|
|
? iop/packages/auth [no test files]
|
|
ok iop/packages/config 0.009s
|
|
? 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 |
|
|
|
|
### 발견된 문제
|
|
|
|
없음
|
|
|
|
### 다음 단계
|
|
|
|
PASS: active plan/review를 archive하고 `complete.log`를 작성해 종료한다.
|