@didim365/agent-cli-core 0.1.0 → 0.1.2

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

@@ -1,198 +0,0 @@
-# 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

@@ -1,98 +0,0 @@
-# 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

@@ -1,264 +0,0 @@
-# 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 런타임 지원