@didim365/agent-cli-core 0.1.0

package/dist/docs/00_project/ai_adapter/working_history/Phase3_M3.1.3_ClaudeStreamError/352/263/240/353/217/204/355/231/224_20260209.md

@@ -0,0 +1,198 @@
+# M3.1.3 작업 결과서 — Claude 스트림 에러 처리 고도화
+
+- **작업일**: 2026-02-09
+- **브랜치**: `DID/v0.1`
+- **상태**: ✅ 완료
+
+## 작업 목표
+
+M3.1.1에서 구현한 ClaudeAdapter 스트림의 에러 처리를 고도화하고, Converter의
+stop_reason 매핑과 스트림 cache token 지원을 확장한다:
+
+1. Stream 에러 시 throw 대신 LlmErrorEvent yield
+2. Anthropic SDK 에러 분류 (classifyError 메서드)
+3. 새로운 stop_reason 매핑 (pause_turn, refusal)
+4. Stream Finished 이벤트에 cache token 포함
+
+## 사전 분석
+
+### todolist 대비 현황
+
+| todolist ID | 항목                    | M3.1.1 구현 여부                          |
+| ----------- | ----------------------- | ----------------------------------------- |
+| 3.1.3.1     | Stream error → LlmEvent | ⚠️ throw 패턴 (handleError returns never) |
+| 3.1.3.2     | Error 타입 분류         | ❌ 미구현 (classifyError 없음)            |
+| 3.1.3.3     | stop_reason 추가 매핑   | ⚠️ 4개만 매핑 (pause_turn/refusal 미포함) |
+| 3.1.3.4     | Stream cache tokens     | ❌ Finished event에 cache 미포함          |
+
+**결론**: 핵심 스트림 인프라는 M3.1.1 완료. M3.1.3은 에러 안전성 + 데이터 완전성
+고도화.
+
+### SDK 타입 검증 결과
+
+| 항목                  | SDK 확인 내용                                                              |
+| --------------------- | -------------------------------------------------------------------------- |
+| StopReason            | `end_turn, max_tokens, stop_sequence, tool_use, pause_turn, refusal`       |
+| APIError.status       | HTTP status code (401, 429, 500+, 529 등)                                  |
+| RawMessageStreamEvent | error 타입 없음 — transport 계층에서 처리                                  |
+| message_delta usage   | `output_tokens, cache_read_input_tokens, cache_creation_input_tokens` 포함 |
+
+### 아키텍처 결정
+
+**Stream 에러 yield vs throw**:
+
+- **이전**: `handleError(error)` 호출 → `BaseAdapter.handleError` returns
+  `never` (항상 throw)
+- **문제**: AsyncGenerator에서 throw 시 `for await...of` 루프가 예외 전파 →
+  스트림 소비자가 try/catch 필수
+- **해결**: `classifyError(error)` → `createErrorEvent()` yield → 스트림
+  소비자가 이벤트로 에러 수신
+- **근거**: `StreamAssembler`는 `LlmErrorEvent`를 expect. yield 패턴이 스트림
+  프로토콜에 부합.
+
+## 작업 순서 및 결과
+
+| 순서 | Sub-task | 작업 내용                                   | 테스트 결과       |
+| ---- | -------- | ------------------------------------------- | ----------------- |
+| 1    | RED      | converter.test.ts (+3) adapter.test.ts (+8) | 10 FAIL / 82 PASS |
+| 2    | GREEN    | converter.ts 2항목 + adapter.ts 2항목       | 92/92 PASS        |
+
+## 변경 파일 상세
+
+### 수정 파일
+
+#### `providers/claude/converter.ts`
+
+**1. mapStopReason 확장**:
+
+- `pause_turn` → `end_turn` (정상 종료와 동일 취급)
+- `refusal` → `content_filter` (안전 필터와 동일 취급)
+
+**2. Stream Finished event cache tokens**:
+
+- `message_delta` 이벤트 처리 시 `usage` 객체에서 cache 토큰 추출
+- `cache_read_input_tokens` → `cachedTokens`
+- `cache_creation_input_tokens` → `cacheCreationTokens`
+
+#### `providers/claude/adapter.ts`
+
+**3. classifyError 메서드 추가**:
+
+- HTTP status code 기반 분류:
+  - 401/403 → `AUTHENTICATION` (non-retryable)
+  - 429 → `RATE_LIMIT` (retryable)
+  - 529 → `MODEL_OVERLOADED` (retryable)
+  - 500+ → `SERVER_ERROR` (retryable)
+- Message 휴리스틱 (status 없을 때):
+  - `/timeout/i` → `TIMEOUT` (retryable)
+  - 기타 → `NETWORK` (retryable)
+- DI 유지: Anthropic SDK 타입 직접 import 하지 않음
+
+**4. generateContentStream yield 패턴 변경**:
+
+- 이전: `catch (error) { handleErr(error); }` → 항상 throw
+- 이후: `catch (error) { yield createErrorEvent(classify(error), ...); }` →
+  이벤트 yield
+
+#### `providers/claude/converter.test.ts` — 62 → 65 tests (+3)
+
+| 카테고리            | 테스트 수             | 검증 내용                                         |
+| ------------------- | --------------------- | ------------------------------------------------- |
+| mapStopReason 확장  | 2 (기존 it.each 확장) | pause_turn→end_turn, refusal→content_filter       |
+| stream cache tokens | 1                     | Finished event에 cachedTokens/cacheCreationTokens |
+| (기존 테스트)       | 62                    | M3.1.2 포함 테스트 유지                           |
+
+#### `providers/claude/adapter.test.ts` — 19 → 27 tests (+8)
+
+| 카테고리              | 테스트 수 | 검증 내용                                      |
+| --------------------- | --------- | ---------------------------------------------- |
+| stream error yield    | 2 (수정)  | creation/iteration error → LlmErrorEvent yield |
+| classifyError 401     | 1         | AuthenticationError, non-retryable             |
+| classifyError 429     | 1         | RateLimitError, retryable                      |
+| classifyError 500+    | 1         | ServerError, retryable                         |
+| classifyError 529     | 1         | ModelOverloaded, retryable                     |
+| classifyError timeout | 1         | TimeoutError (message heuristic)               |
+| classifyError network | 1         | NetworkError (no status, no keyword)           |
+| (기존 테스트)         | 19        | M3.1.1 테스트 유지                             |
+
+## Quality Gate
+
+| 항목          | 결과                       |
+| ------------- | -------------------------- |
+| TypeCheck     | ✅ PASS                    |
+| ESLint        | ✅ PASS                    |
+| Claude 테스트 | ✅ 4 files / 104 passed    |
+| Provider 회귀 | ✅ 29 files / 547 passed   |
+| Core 전체     | ✅ 265 files / 4981 passed |
+
+**변화**: 기준선 4974 → +7 tests (converter 3 + adapter 8 - 기존 수정 4 = 7
+순증)
+
+## 설계 결정
+
+### classifyError — status code 기반 분류
+
+- **결정**: HTTP status code 우선, 없으면 message 휴리스틱
+- **근거**: Anthropic SDK의 `APIError` 계층은 `status` 필드를 가짐. SDK 타입을
+  직접 import하지 않고 duck typing으로 접근하여 DI 패턴 유지.
+
+### pause_turn → end_turn
+
+- **결정**: `pause_turn`을 `end_turn`과 동일하게 매핑
+- **근거**: `pause_turn`은 Anthropic의 computer use 등에서 일시 중단을 의미.
+  현재 CLI에서는 정상 종료와 동일 취급이 적절.
+
+### refusal → content_filter
+
+- **결정**: `refusal`을 `content_filter`로 매핑
+- **근거**: `refusal`은 모델이 요청을 거부한 경우로, 콘텐츠 필터와 의미적 유사.
+  `LlmStopReason`에 별도 `refusal` 값 추가 대신 기존 값 재활용.
+
+### Stream yield vs throw
+
+- **결정**: catch 블록에서 `createErrorEvent()` yield
+- **근거**: `BaseAdapter.handleError()` returns `never` (항상 throw)는 동기
+  API(`generateContent`)에 적합하지만, 스트림에서는 소비자가 `for await...of`로
+  이벤트를 수신하므로 yield가 프로토콜에 부합. `StreamAssembler`가
+  `LlmErrorEvent`를 처리하도록 설계되어 있음.
+
+## 리뷰 반영
+
+### 리뷰 이슈 및 수정 결과
+
+| #   | 심각도 | 이슈                                                              | 수정 내용                                                                 |
+| --- | ------ | ----------------------------------------------------------------- | ------------------------------------------------------------------------- |
+| 1   | 중간   | classifyError에 400/422→INVALID_REQUEST, 404→MODEL_NOT_FOUND 누락 | 400/422→INVALID_REQUEST (non-retryable), 404→ModelNotFoundError 분기 추가 |
+| 2   | 중간   | generateContent가 handleError(UNKNOWN) 사용 — 스트림과 비대칭     | `this.handleError(error)` → `throw this.classifyError(error)` 로 변경     |
+| 3   | 낮음   | classifyError가 LlmError 직접 생성 — subclass 미활용              | Auth/RateLimit/ModelNotFound/Network/TimeoutError subclass 사용           |
+| 4   | 낮음   | stream creation error 테스트에서 error.type 검증 누락             | `expect(errorEvent.error.type).toBe(LlmErrorType.TIMEOUT)` 추가           |
+| 5   | 중간   | classifyError가 이미 분류된 LlmError를 보존하지 않고 재분류       | `if (error instanceof LlmError) { return error; }` 선행 가드 추가         |
+| 6   | 낮음   | LlmError passthrough 회귀 테스트 부재                             | stream(ValidationError)/non-stream(RateLimitError) passthrough 테스트 2건 |
+
+**이슈 #5 상세**: `classifyError`에 `LlmError` 인스턴스가 전달될 때, `status`
+속성이 없으면 message 휴리스틱으로 폴백하여 `NetworkError`/`TimeoutError`로
+재분류됨. `isRetryable` 판단이 왜곡되어 retry 로직에 영향.
+
+### 리뷰 후 Quality Gate
+
+| 항목          | 결과                       |
+| ------------- | -------------------------- |
+| TypeCheck     | ✅ PASS                    |
+| ESLint        | ✅ PASS                    |
+| Claude 테스트 | ✅ 4 files / 114 passed    |
+| Provider 회귀 | ✅ 29 files / 557 passed   |
+| Core 전체     | ✅ 265 files / 4991 passed |
+
+**변화**: 리뷰 전 104 tests → 114 tests (+10), Core 4981 → 4991 (+10)
+
+## 향후 작업
+
+- 향후: stream retry 로직 (retryable error 시 자동 재시도)
+
+## 커밋
+
+- `5c5601c7d` feat(providers): M3.1.3 — Claude 스트림 에러 처리 고도화
+- `f47d5dc03` fix(providers): M3.1.3 리뷰 반영 — classifyError 400/404 분기,
+  subclass 활용, generateContent 에러 분류
+- `7c371dd77` fix(providers): M3.1.3 리뷰 반영 — classifyError LlmError
+  passthrough 보존

package/dist/docs/00_project/ai_adapter/working_history/Phase3_M3.1.4_ClaudeErrorMapping_20260209.md

@@ -0,0 +1,98 @@
+# M3.1.4 작업 결과서 — Claude 에러 매핑
+
+- **작업일**: 2026-02-09
+- **브랜치**: `DID/v0.1`
+- **상태**: ✅ 완료 (M3.1.3에서 선행 구현됨)
+
+## 작업 목표
+
+Anthropic SDK 에러를 `LlmError` 계층으로 분류/매핑하는 유틸리티 구현:
+
+1. Anthropic SDK 에러 분석
+2. HTTP status code 기반 에러 매핑 (429, 401/403, 529 등)
+3. 에러 변환 유틸 함수
+
+## 사전 분석
+
+### todolist 대비 현황
+
+| todolist ID | 항목                    | 구현 여부                                        |
+| ----------- | ----------------------- | ------------------------------------------------ |
+| 3.1.4.1     | Anthropic SDK 에러 분석 | ✅ M3.1.3 — classifyError duck typing            |
+| 3.1.4.2     | Rate limit 에러 매핑    | ✅ M3.1.3 — 429→RateLimitError                   |
+| 3.1.4.3     | Auth 에러 매핑          | ✅ M3.1.3 — 401/403→AuthenticationError          |
+| 3.1.4.4     | Overloaded 에러 매핑    | ✅ M3.1.3 — 529→MODEL_OVERLOADED                 |
+| 3.1.4.5     | 에러 변환 유틸 함수     | ✅ M3.1.3 — classifyError + LlmError passthrough |
+
+**결론**: M3.1.3 (스트림 에러 처리 고도화)에서 에러 매핑 전체 범위가 구현
+완료됨. M3.1.4는 별도 코드 변경 없이 todolist 상태 업데이트만 수행.
+
+### 설계서 §3.3.6 대비 구현 범위
+
+| 설계서 요구 사항         | 구현 상태                     | 비고                   |
+| ------------------------ | ----------------------------- | ---------------------- |
+| 429 → RATE_LIMIT         | ✅ RateLimitError             | retryable              |
+| 401/403 → AUTHENTICATION | ✅ AuthenticationError        | non-retryable          |
+| 400 → INVALID_REQUEST    | ✅ LlmError(INVALID_REQUEST)  | 리뷰 #1 반영으로 추가  |
+| 529 → MODEL_OVERLOADED   | ✅ LlmError(MODEL_OVERLOADED) | retryable              |
+| default → UNKNOWN        | ⬆️ NETWORK/TIMEOUT으로 개선   | 설계서보다 세밀한 분류 |
+
+### 설계서 대비 추가 구현
+
+| 추가 항목                     | 구현 위치          | 근거                           |
+| ----------------------------- | ------------------ | ------------------------------ |
+| 400/422 → INVALID_REQUEST     | adapter.ts:202-206 | Gemini 어댑터 패리티 (리뷰 #1) |
+| 404 → ModelNotFoundError      | adapter.ts:211-212 | Gemini 어댑터 패리티 (리뷰 #1) |
+| 500+ → SERVER_ERROR           | adapter.ts:223-227 | 범용 서버 에러 포착            |
+| Timeout 휴리스틱              | adapter.ts:232-233 | status 없는 timeout 에러 분류  |
+| Network 폴백                  | adapter.ts:236     | 최종 폴백 (UNKNOWN 대체)       |
+| LlmError passthrough          | adapter.ts:186-188 | 재분류 방지 (리뷰 #5)          |
+| Stream error yield            | adapter.ts:146-151 | throw 대신 LlmErrorEvent yield |
+| generateContent classifyError | adapter.ts:112     | 스트림과 대칭 (리뷰 #2)        |
+
+## 테스트 현황
+
+adapter.test.ts 에러 관련 테스트: **12건** (전체 37건 중)
+
+| 카테고리              | 테스트 수 | 검증 내용                                             |
+| --------------------- | --------- | ----------------------------------------------------- |
+| status code 분류      | 7         | 400, 401, 404, 429, 500+, 529 + INVALID_REQUEST       |
+| message 휴리스틱      | 2         | timeout keyword, network fallback                     |
+| subclass instanceof   | 5         | Auth, RateLimit, ModelNotFound, Network, Timeout      |
+| LlmError passthrough  | 2         | stream (ValidationError), non-stream (RateLimitError) |
+| stream error yield    | 2         | creation error, iteration error → LlmErrorEvent       |
+| generateContent error | 1         | 429 → RateLimitError throw                            |
+
+## 변경 파일
+
+### 코드 변경: 없음
+
+M3.1.4 범위는 M3.1.3에서 전수 구현됨. 별도 코드 수정 불필요.
+
+### 문서 변경
+
+- `phase3_provider_extension_todolist.md`: M3.1.0~M3.1.4 전체 ⬜→✅ 상태
+  업데이트
+
+## Quality Gate
+
+기존 M3.1.3 Quality Gate 결과 유지:
+
+| 항목          | 결과                     |
+| ------------- | ------------------------ |
+| TypeCheck     | ✅ PASS                  |
+| ESLint        | ✅ PASS                  |
+| Claude 테스트 | ✅ 4 files / 114 passed  |
+| Provider 회귀 | ✅ 29 files / 557 passed |
+
+## M3.1 마일스톤 완료 요약
+
+| Sub-milestone | 작업                       | 상태 | 커밋                                  |
+| ------------- | -------------------------- | ---- | ------------------------------------- |
+| M3.1.0        | SDK 설치 + 부트스트랩 등록 | ✅   | M3.1.1 커밋에 포함                    |
+| M3.1.1        | ClaudeAdapter 구현         | ✅   | 별도 작업 결과서 참조                 |
+| M3.1.2        | Claude 메시지 변환기       | ✅   | 별도 작업 결과서 참조                 |
+| M3.1.3        | Claude 스트림/에러 고도화  | ✅   | `5c5601c7d`, `f47d5dc03`, `7c371dd77` |
+| M3.1.4        | Claude 에러 매핑           | ✅   | M3.1.3에서 선행 구현 — 코드 변경 없음 |
+
+**M3.1 전체 완료**. 다음 마일스톤: M3.2 (OpenAI 어댑터/변환기 구현).

package/dist/docs/00_project/ai_adapter/working_history/Phase3_M3.2.A_OpenAIAdapterCore_20260210.md

@@ -0,0 +1,264 @@
+# M3.2.A 작업 결과서 — OpenAI 어댑터 Core (부트스트랩 + 변환기 + 어댑터)
+
+- **작업일**: 2026-02-10
+- **브랜치**: `DID/v0.1`
+- **상태**: ✅ 완료
+
+## 작업 목표
+
+OpenAI SDK를 래핑하는 어댑터/변환기의 Core 부분(비스트림)을 TDD로 구현:
+
+1. SDK 설치 및 부트스트랩 (`bootstrapOpenAiProvider`)
+2. 요청/응답 변환기 (`OpenAiConverter`)
+3. 어댑터 비스트림 동작 (`OpenAiAdapter.generateContent`)
+4. 에러 분류 (`classifyError`)
+5. 네임스페이스 export
+
+## 사전 분석
+
+### SDK 버전 차이
+
+| 항목         | 설계서               | 실제                                                      |
+| ------------ | -------------------- | --------------------------------------------------------- |
+| SDK 버전     | `^4.70.0`            | `^6.18.0`                                                 |
+| max_tokens   | `max_tokens`         | `max_completion_tokens` (deprecated `max_tokens`)         |
+| tool_choice  | `auto\|none\|{name}` | `auto\|none\|required\|{type:'function',function:{name}}` |
+| stream usage | N/A                  | `stream_options: { include_usage: true }`                 |
+
+### 설계서 대비 구현 범위
+
+| 설계서 요구 사항      | 구현 상태 | 비고                                                              |
+| --------------------- | --------- | ----------------------------------------------------------------- |
+| OpenAI SDK 연동       | ✅        | `openai@^6.18.0`                                                  |
+| BaseAdapter 상속      | ✅        | generateContent, generateContentStream(스켈레톤)                  |
+| 메시지 변환 (4 roles) | ✅        | system/user/assistant/tool                                        |
+| 이미지 변환           | ✅        | base64→data URI, URL 직접 전달                                    |
+| 도구 변환             | ✅        | function wrapping                                                 |
+| JSON mode             | ✅        | responseFormat→response_format                                    |
+| 에러 매핑             | ✅        | Claude 패턴과 동일 classifyError                                  |
+| countTokens           | ✅        | `supportsTokenCount: false` → BaseAdapter UnsupportedFeatureError |
+| 스트림 변환           | ⚠️        | 스켈레톤만 (M3.2.B 범위)                                          |
+
+### Claude 어댑터 대비 차이점
+
+| 항목             | Claude                                          | OpenAI                                    |
+| ---------------- | ----------------------------------------------- | ----------------------------------------- |
+| System 메시지    | 별도 `system` 파라미터 추출                     | messages 배열에 `role: 'system'`으로 포함 |
+| Tool result 역할 | `user` role + tool_result 블록 (연속 role 병합) | `tool` role (직접 지원, 병합 불필요)      |
+| 이미지 지원      | base64만 (URL 미지원)                           | base64→data URI + URL 직접 전달           |
+| max_tokens       | 필수 파라미터 (default 8192)                    | 선택 파라미터 (`max_completion_tokens`)   |
+| Thought/Thinking | 지원 (`supportsThought: true`)                  | 미지원 (`supportsThought: false`)         |
+| Token counting   | SDK `countTokens()` 있음                        | SDK에 없음 (`supportsTokenCount: false`)  |
+| response_format  | 미지원                                          | `json_object` 매핑                        |
+| 529 overloaded   | 있음 (MODEL_OVERLOADED)                         | 없음                                      |
+
+## 작업 순서 및 결과
+
+| 순서 | 작업                             | 테스트 수 | 결과    |
+| ---- | -------------------------------- | --------- | ------- |
+| 1    | RED: bootstrap.test.ts           | 7         | 7 FAIL  |
+| 2    | RED: converter.test.ts           | 40        | 40 FAIL |
+| 3    | RED: adapter.test.ts             | 27        | 27 FAIL |
+| 4    | GREEN: converter.ts              | 40        | 40 PASS |
+| 5    | GREEN: adapter.ts + bootstrap.ts | 34        | 34 PASS |
+| 6    | Fix: TypeCheck + ESLint          | 74        | 74 PASS |
+
+## 변경 파일 상세
+
+### 신규 파일
+
+| 파일                                 | 역할                             | 라인 수 |
+| ------------------------------------ | -------------------------------- | ------- |
+| `providers/openai/converter.ts`      | 요청/응답/스트림 변환기          | ~290    |
+| `providers/openai/adapter.ts`        | OpenAiAdapter (BaseAdapter 상속) | ~170    |
+| `providers/openai/bootstrap.ts`      | 팩토리 등록 + has() 가드         | ~40     |
+| `providers/openai/index.ts`          | 퍼블릭 export                    | ~15     |
+| `providers/openai/converter.test.ts` | 변환기 TDD 테스트                | ~330    |
+| `providers/openai/adapter.test.ts`   | 어댑터 TDD 테스트                | ~260    |
+| `providers/openai/bootstrap.test.ts` | 부트스트랩 TDD 테스트            | ~90     |
+
+### 수정 파일
+
+| 파일                         | 변경 내용                                     |
+| ---------------------------- | --------------------------------------------- |
+| `providers/index.ts`         | `import * as OpenAi` 네임스페이스 export 추가 |
+| `packages/core/package.json` | `"openai": "^6.18.0"` 의존성 추가             |
+
+## 테스트 현황
+
+### bootstrap.test.ts — 7 tests
+
+| 테스트            | 검증 내용                        |
+| ----------------- | -------------------------------- |
+| register factory  | `has('openai')` false→true       |
+| has() guard       | 중복 호출 안전성                 |
+| createAdapter     | `providerName === 'openai'`      |
+| singleton default | 인수 없이 호출 시 singleton 사용 |
+| apiKey passing    | SDK에 apiKey 전달                |
+| baseURL passing   | SDK에 baseURL 전달               |
+| undefined apiKey  | 누락 시 안전 처리                |
+
+### converter.test.ts — 40 tests
+
+| 카테고리           | 테스트 수 | 검증 내용                                                                                                                                                                |
+| ------------------ | --------- | ------------------------------------------------------------------------------------------------------------------------------------------------------------------------ |
+| toOpenAiRequest    | 7         | model, max_completion_tokens, temperature/topP/stop, tools, tool_choice, response_format, systemInstruction                                                              |
+| toOpenAiMessages   | 12        | user/assistant/system/tool 4 role, tool_call, tool_result JSON serialize, base64→data URI, URL image, mixed content, thought skip, empty skip, assistant text+tool_calls |
+| toOpenAiTools      | 2         | function wrapping, optional required                                                                                                                                     |
+| toOpenAiToolChoice | 4         | auto/none/required/specific name                                                                                                                                         |
+| fromOpenAiResponse | 6         | text, tool_calls, text+tool_calls, invalid JSON args, fallback model, cached_tokens                                                                                      |
+| mapFinishReason    | 8         | stop/length/tool_calls/content_filter/function_call/null/undefined/unknown                                                                                               |
+| createStreamState  | 1         | fresh state creation                                                                                                                                                     |
+
+### adapter.test.ts — 27 tests
+
+| 카테고리             | 테스트 수 | 검증 내용                                                                             |
+| -------------------- | --------- | ------------------------------------------------------------------------------------- |
+| capabilities         | 9         | 9개 capability 플래그 검증                                                            |
+| generateContent      | 6         | SDK 호출, 응답 변환, validation (model/messages), classifyError, LlmError passthrough |
+| countTokens          | 1         | UnsupportedFeatureError throw                                                         |
+| error classification | 10        | 8 status codes, timeout heuristic, network fallback                                   |
+| mapToProviderConfig  | 1         | construction 성공 확인                                                                |
+
+## Quality Gate
+
+| 항목          | 결과                     |
+| ------------- | ------------------------ |
+| TypeCheck     | ✅ PASS                  |
+| ESLint        | ✅ PASS                  |
+| OpenAI 테스트 | ✅ 3 files / 74 passed   |
+| Provider 회귀 | ✅ 32 files / 631 passed |
+
+**변화**: 기준선 29 files / 557 → 32 files / 631 (+3 files, +74 tests)
+
+## 설계 결정
+
+### max_completion_tokens vs max_tokens
+
+- **결정**: `max_completion_tokens` 사용
+- **근거**: OpenAI SDK v6.x에서 `max_tokens` 는 deprecated. 최신 SDK 규약 준수.
+
+### supportsTokenCount: false
+
+- **결정**: countTokens 미지원, BaseAdapter 기본 UnsupportedFeatureError 위임
+- **근거**: OpenAI SDK에 내장 countTokens 없음. tiktoken 별도 의존성 추가는
+  YAGNI.
+
+### supportsThought: false
+
+- **결정**: extended thinking 미지원
+- **근거**: 표준 Chat Completions API에 thinking 블록 없음. o-series reasoning은
+  별도 API.
+
+### OpenAI 네이티브 role 활용
+
+- **결정**: Claude와 달리 role 병합 로직 불필요
+- **근거**: OpenAI API가 system/user/assistant/tool 4개 role을 네이티브로 지원.
+  tool_result는 `role: 'tool'`로 직접 매핑.
+
+### classifyError Claude 패턴 재사용
+
+- **결정**: Claude adapter의 classifyError와 동일 구조 사용
+- **근거**: HTTP status code 기반 분류 로직은 프로바이더 독립적. DI 유지 (duck
+  typing).
+
+## 리뷰 반영 사항
+
+### Issue #1 (높음): `bootstrapOpenAiProvider()` 호출 누락
+
+- **문제**: `contentGenerator.ts`의 multi-provider 경로에서
+  `bootstrapGeminiProvider()`와 `bootstrapClaudeProvider()`만 호출되고
+  `bootstrapOpenAiProvider()`가 누락 → OpenAI 선택 시 런타임 "Provider not
+  registered" 오류 발생
+- **수정**:
+  - `contentGenerator.ts`: `import { bootstrapOpenAiProvider }` 추가 +
+    `bootstrapOpenAiProvider()` 호출 추가 (line 286)
+- **검증**: multiProvider 테스트 Scenario 5b 추가로 auto-bootstrap 경로 확인
+
+### Issue #2 (중간): `supportsStreaming: true` + stub 스트림
+
+- **문제**: `OPENAI_CAPABILITIES.supportsStreaming: true`로 선언되었으나
+  `convertStreamEvent`가 빈 배열 `[]`을 반환하는 stub → 스트림 요청 시 이벤트
+  없이 종료
+- **수정**:
+  - `adapter.ts`: `supportsStreaming: false`로 변경, M3.2.B 완료 시 `true`로
+    전환하는 TODO 주석 추가
+  - `adapter.test.ts`: capability 테스트를 `false` 기대값으로 업데이트
+- **근거**: M3.2.B에서 `convertStreamEvent` 구현 전까지 스트림 기능을
+  비활성화하는 것이 안전. 호출자가 `supportsStreaming` 플래그를 확인하여 스트림
+  사용 여부를 결정하므로 false면 비스트림으로 자동 폴백.
+
+### Issue #3 (낮음): auto-bootstrap 통합 테스트 미비
+
+- **문제**: `contentGenerator.multiProvider.test.ts` Scenario 5에서
+  `registry.register('openai', ...)` 수동 등록만 수행 → 실제
+  `bootstrapOpenAiProvider()` 경로 미검증
+- **수정**:
+  - `contentGenerator.multiProvider.test.ts`: `vi.mock('openai', ...)` 모듈 mock
+    추가 + Scenario 5b 테스트 추가 (수동 등록 없이 auto-bootstrap만으로 OpenAI
+    어댑터 생성 확인)
+- **검증**: 14 tests passed (기존 13 + 1)
+
+### 2차 리뷰 Issue #1 (높음): CLI 런타임 경로 legacy Gemini 호출 문제
+
+- **문제**: OpenAI 선택 시 CLI 런타임에서 `GeminiChat.sendMessageStream()` →
+  `getContentGenerator().generateContentStream(legacy Gemini params)` 경로를
+  타서 `Provider "openai" does not support legacy Gemini API` 에러 발생.
+- **분석 결과**: 이것은 **Phase 3 핸드오프 문서에 기록된 알려진 제한사항**임.
+  - "制限 2: Agent 실행 경로 Gemini 고정" — `GeminiChat`, `LocalExecutor`가
+    legacy Gemini 메서드만 호출
+  - 해결을 위해서는 provider-independent `ChatSession` 인터페이스 생성 +
+    `GeminiClient`/`Turn`/`LocalExecutor` DI 리팩토링 필요
+  - 이는 **CLI 통합 마일스톤(M3.4+)** 범위
+- **결정**: M3.2.A 범위에서 수정하지 않음. 작업결과서에 알려진 제한사항으로
+  명시적 기록. Phase 3 후속 마일스톤에서 해결 예정.
+
+### 2차 리뷰 Issue #2 (낮음): `generateContentStream` 스텁 방어 가드 추가
+
+- **문제**: `supportsStreaming: false`이지만 `generateContentStream` 메서드
+  자체는 호출 가능 → 내부 `convertStreamEvent` 스텁이 `[]` 반환 → 빈 스트림 무한
+  소비 위험
+- **수정**:
+  - `adapter.ts`: `generateContentStream` 진입부에 capability 가드 추가
+    (`!this.capabilities.supportsStreaming` → `UnsupportedFeatureError` throw)
+  - `adapter.test.ts`: 가드 테스트 2건 추가 (UnsupportedFeatureError +
+    provider명 포함 확인)
+- **근거**: `countTokens`와 동일한 패턴. capability flag가 false인 기능은 메서드
+  레벨에서도 즉시 에러를 반환하여 빈 스트림보다 명확한 피드백 제공.
+
+### Quality Gate (2차 리뷰 반영 후)
+
+| 항목          | 결과                     |
+| ------------- | ------------------------ |
+| TypeCheck     | ✅ PASS                  |
+| ESLint        | ✅ PASS                  |
+| OpenAI 테스트 | ✅ 3 files / 76 passed   |
+| MultiProvider | ✅ 1 file / 14 passed    |
+| Provider 회귀 | ✅ 33 files / 647 passed |
+
+**변화**: 33 files / 645 → 33 files / 647 (+2 tests: generateContentStream 가드)
+
+## 알려진 제한사항
+
+### CLI 런타임 경로 Gemini 고정 (Phase 3 핸드오프 문서 §制限 2)
+
+현재 CLI의 메인 채팅 루프(`GeminiClient` → `GeminiChat` → `Turn`)는 legacy
+Gemini `generateContentStream(GenerateContentParameters)` 만 호출합니다.
+non-Gemini provider wrapper의 provider-independent 메서드
+(`llmGenerateContentStream`)는 아직 어디서도 호출되지 않습니다.
+
+**영향**: `ENABLE_MULTI_PROVIDER=true` + `LLM_PROVIDER=openai` 설정 시:
+
+- ✅ Content generator 생성, adapter 부트스트랩 정상
+- ❌ 실제 채팅 시 `Provider "openai" does not support legacy Gemini API` 에러
+
+**해결 계획**: CLI 통합 마일스톤(M3.4+)에서 provider-independent `ChatSession`
+인터페이스 도입 및 `GeminiClient`/`LocalExecutor` DI 리팩토링.
+
+## 향후 작업
+
+- **M3.2.B**: 스트림 변환기 구현 (`convertStreamEvent`), 스트림 에러 처리 고도화
+  - text_delta, tool_call delta, finish_reason, usage 추출
+  - `stream_options: { include_usage: true }` 활용
+  - `supportsStreaming` → `true` 전환 + capability 가드 제거
+- **M3.4+**: CLI 통합 — ChatSession 인터페이스 + DI로 non-Gemini 런타임 지원