@didim365/agent-cli-core 0.1.0

package/dist/docs/ai_adapter/working_history/Phase1_M1.2_Adapter/354/235/270/355/224/204/353/235/274_20260203.md

@@ -0,0 +1,378 @@
+# Phase 1 M1.2: Adapter 인프라 구축 작업 기록
+
+**작업일**: 2026-02-03 **작업자**: AI Assistant **관련 문서**:
+
+- [00_master_implementation_plan.md](../todolist/00_master_implementation_plan.md) -
+  M1.2
+- [phase1_foundation_todolist.md](../todolist/phase1_foundation_todolist.md) -
+  M1.2
+
+---
+
+## 📋 개요
+
+**목표**: 프로바이더 독립적인 어댑터 패턴의 핵심 인프라 구축 (BaseAdapter,
+Registry, Factory 등)
+
+**진행 방식**: 5단계로 나누어 순차적으로 진행하며, 각 단계 완료 시 피드백 반영.
+
+| 단계  | 작업 항목                 | 상태    | 비고                         |
+| ----- | ------------------------- | ------- | ---------------------------- |
+| 1.2.1 | **BaseAdapter 구현**      | ✅ 완료 | 추상 클래스, 공통 인터페이스 |
+| 1.2.2 | **ProviderRegistry 구현** | ✅ 완료 | 싱글톤 레지스트리 (20 tests) |
+| 1.2.3 | **ProviderFactory 구현**  | ✅ 완료 | 동적 생성 팩토리 (16 tests)  |
+| 1.2.4 | **StreamAssembler 구현**  | ✅ 완료 | 스트림 조립기 (22 tests)     |
+| 1.2.5 | **ConfigAdapter 구현**    | ✅ 완료 | 설정 변환기 (16 tests)       |
+| 1.2.6 | **ModelSpec 구현**        | ✅ 완료 | 모델 메타데이터 (11 tests)   |
+| 1.2.7 | **ContentResolver 구현**  | ✅ 완료 | 콘텐츠 해석기 (18 tests)     |
+
+---
+
+## 🏗️ 1.2.1 BaseAdapter 구현
+
+### 📝 계획
+
+- **목표**: 모든 프로바이더 어댑터의 기반이 되는 추상 클래스 `BaseAdapter` 정의
+- **파일**:
+  - `packages/core/src/providers/baseAdapter.ts` (신규)
+  - `packages/core/src/providers/baseAdapter.test.ts` (신규)
+
+### 🔴 Red Phase (테스트 작성)
+
+- [x] `LlmAdapter` 인터페이스 정의 테스트
+- [x] `BaseAdapter` 추상 메서드 (`generate`, `generateStream`,
+      `mapToProviderConfig`) 검증 테스트
+- [x] `validateConfig` 공통 메서드 테스트
+- **결과**: 테스트 실패 확인 (`Failed to load url ./baseAdapter.js`)
+
+**작성된 테스트 (TestAdapter)**:
+
+```typescript
+class TestAdapter extends BaseAdapter {
+  // ... abstract methods implemented as mocks ...
+}
+
+describe('BaseAdapter', () => {
+  it('should require mapToProviderConfig implementation', () => {
+    const adapter = new TestAdapter(mockConfig);
+    const mappedConfig = adapter.mapToProviderConfig(mockConfig);
+    expect(mappedConfig).toEqual({ ... });
+  });
+});
+```
+
+### 🟢 Green Phase (구현)
+
+- [x] `LlmAdapter` 인터페이스
+- [x] `BaseAdapter` 추상 클래스
+- [x] 공통 유틸리티 메서드 구현
+- **결과**: 테스트 통과 (4 passed)
+
+**구현된 BaseAdapter**:
+
+```typescript
+export abstract class BaseAdapter implements LlmAdapter {
+  protected config: LlmGenerateConfig;
+
+  constructor(config: LlmGenerateConfig) {
+    this.config = config;
+  }
+
+  getCapabilities(): LlmProviderCapabilities {
+    return {
+      supportsStreaming: true,
+      supportsTools: false,
+      supportsImages: false,
+    };
+  }
+
+  // Abstract methods to be implemented by providers
+  abstract mapToProviderConfig(
+    config: LlmGenerateConfig,
+  ): Record<string, unknown>;
+  abstract generate(request: LlmGenerateRequest): Promise<LlmGenerateResponse>;
+  abstract generateStream(request: LlmGenerateRequest): LlmStream;
+}
+```
+
+### 🔄 Refactor Phase
+
+- [x] 타입 안전성 강화
+- [x] 에러 처리 표준화 (`LlmError` 활용)
+
+**변경 내역 (LlmError)**:
+
+```typescript
+validateConfig(): void {
+  if (!this.config) {
+    throw new LlmError(LlmErrorType.VALIDATION, 'Configuration is required');
+  }
+}
+```
+
+- **결과**: 테스트 통과 (4 passed)
+
+**구현된 BaseAdapter Capabilities (Updated)**:
+
+```typescript
+getCapabilities(): LlmProviderCapabilities {
+  return {
+    supportsStreaming: true,
+    supportsToolCalls: false,
+    supportsImageInput: false,
+    supportsImageGeneration: false,
+    supportsEmbedding: false,
+    supportsTokenCount: false,
+    supportsSystemMessage: true,
+    supportsThought: false,
+    maxContextLength: 8192,
+    maxOutputTokens: 4096,
+  };
+}
+```
+
+### � Refactor & Fix Phase (피드백 반영)
+
+- [x] 타입 안전성 강화
+- [x] 에러 처리 표준화 (`LlmError` 활용)
+- [x] **[Round 1 Issue Fix]**
+  - `LlmGenerateConfig` 타입 정의 추가
+  - `LlmProviderCapabilities` 정합화
+  - `index.ts` export 추가
+- [x] **[Round 2 Issue Fix]**
+  - 스트림 타입 통일: `LlmStream` (types.ts) -> `LlmEventStream` (events.ts)로
+    교체
+  - 문서-코드 불일치 해소
+  - 테스트 코드 개선 (미사용 변수 제거, `LlmEvent` 사용)
+
+### 🚀 Critical Refactor (3차 리뷰 반영)
+
+**주요 변경 사항 (Design Alignment)**:
+
+- **인터페이스 변경**: `LlmAdapter` → `ContentGenerator` (설계 문서 §3.2.1 준수)
+- **메서드 서명 수정**:
+  - `generateContent(request, userPromptId, options)`: 필수 인자(`userPromptId`)
+    및 옵션(`signal`, `timeout`) 추가
+  - `generateContentStream(...)`: 동일하게 서명 일치
+- **설정 분리**:
+  - `AdapterConfig`: 인스턴스 초기화용 (API Key 등)
+  - `LlmGenerateConfig`: 생성 요청 기본값용
+- **누락 기능 추가**:
+  - `countTokens(request)`: 추상 메서드 추가
+  - `validateRequest(request)`: 공통 검증 로직 구현
+  - `embedContent(request)`: 기본 구현 (Unsupported Error) 추가
+- **테스트 커버리지 확대**:
+  - `generateContentStream` 및 `countTokens` 테스트 케이스 추가 (총 6개 테스트
+    통과)
+
+### 🔧 4차 리뷰 반영 (report.md 이슈 해결)
+
+**검증 및 수정 완료된 이슈 목록**:
+
+|  우선순위  | 이슈                                        | 조치 내용                                            |
+| :--------: | ------------------------------------------- | ---------------------------------------------------- |
+|  **High**  | `LlmErrorType.UNSUPPORTED` 컴파일 오류      | `UNSUPPORTED_FEATURE`로 수정 (baseAdapter.ts)        |
+| **Medium** | `embedContent` 인터페이스 미노출            | `ContentGenerator`에 optional 메서드 추가 (types.ts) |
+| **Medium** | `LlmStream`/`LlmStreamEvent` 타입 혼선      | `@deprecated` 주석 추가, `LlmEventStream` 사용 권장  |
+| **Medium** | `capabilities` 기본값 문서 불일치           | 코드(abstract property)가 정확함, 문서 해석 오류     |
+|  **Low**   | `validateRequest()` 호출 경로 부재          | JSDoc에 서브클래스 호출 가이드 추가                  |
+|  **Low**   | `LlmTokenCount` 중복 정의                   | 중복 정의 제거 (Line 404-411)                        |
+|  **Low**   | `GenerateOptions`/`LlmGenerateOptions` 중복 | `LlmGenerateOptions`에 `@deprecated` 추가            |
+
+### 🔧 5차 리뷰 반영 (추가 이슈 해결)
+
+|  우선순위  | 이슈                              | 조치 내용                                                     |
+| :--------: | --------------------------------- | ------------------------------------------------------------- |
+| **Medium** | `ContentGenerator` 이중 정의 혼선 | JSDoc에 `core/contentGenerator.ts`와의 구분 설명 추가         |
+|  **Low**   | `embedContent` 에러 타입 불일치   | `UnsupportedFeatureError` 클래스 사용으로 통일                |
+|  **Low**   | `countTokens` 기본 동작 미정의    | 기본 구현 추가 (capability 체크 후 `UnsupportedFeatureError`) |
+|  **Low**   | `embedContent` 타입 미정의        | TODO 주석 추가 (M1.3 예정)                                    |
+
+---
+
+## 🏗️ 1.2.2 ProviderRegistry 구현
+
+### 📝 계획
+
+- **목표**: 프로바이더 어댑터 팩토리를 관리하는 싱글톤 레지스트리 구현
+- **파일**:
+  - `packages/core/src/providers/registry.ts` (신규)
+  - `packages/core/src/providers/registry.test.ts` (신규)
+
+### 🔴 Red Phase (테스트 작성)
+
+- [x] 싱글톤 패턴 테스트
+- [x] `register()` 메서드 테스트 (중복 등록 에러, force 옵션)
+- [x] `get()`, `getOrThrow()` 메서드 테스트
+- [x] `has()`, `list()` 메서드 테스트
+- [x] `unregister()`, `clear()` 메서드 테스트
+- [x] `createAdapter()` 메서드 테스트
+- **결과**: 테스트 실패 확인 (`Cannot find module './registry.js'`)
+
+### 🟢 Green Phase (구현)
+
+- [x] `ProviderRegistry` 싱글톤 클래스
+- [x] `AdapterFactory` 타입 정의
+- [x] 모든 CRUD 메서드 구현
+- **결과**: 테스트 통과 (20 passed)
+
+**구현된 ProviderRegistry**:
+
+```typescript
+export class ProviderRegistry {
+  private static instance: ProviderRegistry | null = null;
+  private readonly providers = new Map<string, AdapterFactory>();
+
+  static getInstance(): ProviderRegistry { ... }
+  register(name, factory, options?): void { ... }
+  get(name): AdapterFactory | undefined { ... }
+  getOrThrow(name): AdapterFactory { ... }
+  has(name): boolean { ... }
+  list(): string[] { ... }
+  unregister(name): boolean { ... }
+  clear(): void { ... }
+  createAdapter(name, config): BaseAdapter { ... }
+}
+```
+
+### 🔄 Refactor Phase
+
+- [x] ESLint 통과
+- [x] `index.ts`에 export 추가
+- [x] 타입 에러 수정 (`LlmEventType` enum 사용, `override` 키워드)
+- **결과**: 테스트 통과 (19 passed), ESLint 통과
+
+### 🔧 6차 리뷰 반영 (report.md 이슈 해결)
+
+**검증 및 수정 완료 이슈 목록**:
+
+|  우선순위  | 이슈                                      | 조치 내용                                                       | 상태 |
+| :--------: | ----------------------------------------- | --------------------------------------------------------------- | :--: |
+| **Medium** | ProviderRegistry 오류 타입 의미 혼선      | `PROVIDER_NOT_FOUND` 신규 에러타입 추가, `MODEL_NOT_FOUND` 대체 |  ✅  |
+| **Medium** | ContentGenerator 인터페이스 이중 정의     | 이전 M1.2.1에서 JSDoc에 구분 설명 추가됨                        |  ✅  |
+|  **Low**   | BaseAdapter 검증 로직 호출 보장 부재      | JSDoc 강화됨 + `validateRequest` 테스트 3개 추가                |  ✅  |
+|  **Low**   | ProviderRegistry 이름 정규화 부재         | `normalizeName()` 메서드 추가, 모든 조회에 `toLowerCase()` 적용 |  ✅  |
+|  **Low**   | LlmStream 타입 잔존                       | 이전 M1.2.1에서 `@deprecated` 처리됨                            |  ✅  |
+|  **Low**   | BaseAdapter 유틸리티 테스트 커버리지 부족 | `validateRequest` 3개 + `handleError` 2개 테스트 추가           |  ✅  |
+
+**테스트 결과**: 31 passed (baseAdapter 11 + registry 20)
+
+---
+
+## 🏗️ 1.2.3 ProviderFactory 구현
+
+### 📝 계획
+
+- **목표**: 프로바이더 어댑터 인스턴스 생성 팩토리
+- **파일**:
+  - `packages/core/src/providers/factory.ts` (신규)
+  - `packages/core/src/providers/factory.test.ts` (신규)
+
+### 🔴 Red Phase (테스트 작성)
+
+- [x] 생성자 테스트 (registry 주입, 기본값)
+- [x] `create()` 메서드 테스트 (등록/미등록, 대소문자 정규화)
+- [x] `createWithValidation()` 메서드 테스트 (apiKey 검증)
+- [x] `canCreate()` 메서드 테스트
+- [x] `getAvailableProviders()` 메서드 테스트
+- [x] 에러 타입 테스트 (`PROVIDER_NOT_FOUND`)
+- **결과**: 테스트 실패 확인 (`Cannot find module './factory.js'`)
+
+### 🟢 Green Phase (구현)
+
+- [x] `ProviderFactory` 클래스
+- [x] `create()` - 어댑터 인스턴스 생성
+- [x] `createWithValidation()` - 설정 검증 후 생성
+- [x] `canCreate()` - 생성 가능 여부 확인
+- [x] `getAvailableProviders()` - 등록된 프로바이더 목록
+- **결과**: 테스트 통과 (14 passed)
+
+**구현된 ProviderFactory**:
+
+```typescript
+export class ProviderFactory {
+  constructor(registry?: ProviderRegistry) { ... }
+  create(providerName: string, config: AdapterConfig): BaseAdapter { ... }
+  createWithValidation(providerName: string, config: AdapterConfig): BaseAdapter { ... }
+  canCreate(providerName: string): boolean { ... }
+  getAvailableProviders(): string[] { ... }
+}
+```
+
+### 🔄 Refactor Phase
+
+- [x] ESLint 통과
+- [x] `index.ts`에 export 추가
+- **결과**: 전체 테스트 통과 (45 passed)
+
+### 🔧 M1.2.3 리뷰 반영
+
+**검증 및 수정 완료 이슈 목록**:
+
+|  우선순위  | 이슈                                             | 조치 내용                                         | 상태 |
+| :--------: | ------------------------------------------------ | ------------------------------------------------- | :--: |
+| **Medium** | apiKey만 검증 - OAuth/ADC 미지원                 | `options.requireApiKey` 플래그 추가, 기본값=false |  ✅  |
+|  **Low**   | export 누락                                      | ❌ 거짓 - `index.ts` Line 23에 이미 존재          |  ⛔  |
+|  **Low**   | 검증 우선순위 (VALIDATION vs PROVIDER_NOT_FOUND) | 프로바이더 확인 먼저 후 config 검증으로 순서 변경 |  ✅  |
+
+**테스트 결과**: 16 passed (factory.test.ts)
+
+---
+
+## 🏗️ 1.2.4 StreamAssembler 구현
+
+### 📝 계획
+
+- **목표**: 스트림 이벤트를 완성된 메시지로 조립
+- **파일**:
+  - `packages/core/src/providers/streamAssembler.ts` (신규)
+  - `packages/core/src/providers/streamAssembler.test.ts` (신규)
+
+### 🔴 Red Phase (테스트 작성)
+
+- [x] 생성자 및 초기 상태 테스트
+- [x] `processEvent()` 기본 동작 테스트
+- [x] 텍스트 델타 합성 (연결, 빈값, 공백)
+- [x] Thought 델타 합성
+- [x] 도구 호출 수집
+- [x] Usage 정보 누적
+- [x] 완료 상태 감지
+- [x] `getAssembledMessage()` 통합 테스트
+- [x] `reset()` 상태 초기화
+- [x] 에러 처리
+- **결과**: 22 tests 작성
+
+### 🟢 Green Phase (구현)
+
+```typescript
+export class StreamAssembler {
+  processEvent(event: LlmEvent): this { ... }
+  getText(): string { ... }
+  getThought(): string { ... }
+  getToolCalls(): LlmToolCallContent[] { ... }
+  getUsage(): LlmTokenUsage | undefined { ... }
+  isComplete(): boolean { ... }
+  getAssembledMessage(): AssembledMessage { ... }
+  reset(): void { ... }
+}
+```
+
+- **결과**: 22 tests 통과
+
+### 🔄 Refactor Phase
+
+- [x] 불필요한 타입 assertion 제거
+- [x] ESLint 통과
+- [x] `index.ts`에 export 추가
+
+### 🔧 M1.2.4 리뷰 반영
+
+|  우선순위  | 이슈                                 | 조치 내용                                                       | 상태 |
+| :--------: | ------------------------------------ | --------------------------------------------------------------- | :--: |
+| **Medium** | LlmTokenUsage 필드명 불일치          | `inputTokens`→`promptTokens`, `outputTokens`→`completionTokens` |  ✅  |
+|  **Low**   | Error 이벤트 시 complete=false       | `handleError`에서 `complete=true` 설정 + 테스트 추가            |  ✅  |
+|  **Low**   | ToolCallResponse/Confirmation 미처리 | 설계 의도 - JSDoc에 이유 문서화                                 |  ✅  |
+
+**테스트 결과**: 22 passed (streamAssembler.test.ts)
+
+---

package/dist/docs/ai_adapter/working_history/Phase1_M1.3_Provider/354/204/240/355/203/235_20260206.md

@@ -0,0 +1,269 @@
+# Phase 1 M1.3 Provider 선택/Config 설계 작업 결과
+
+> 📅 **작업일**: 2026-02-06 🎯 **목표**: Provider 선택 경로와 Config 타입 설계
+
+---
+
+## 📊 진행 현황
+
+| 구성요소                 |  상태   | 테스트 | 비고                                |
+| ------------------------ | :-----: | :----: | ----------------------------------- |
+| 1.3.1 ProviderType       | ✅ 완료 |   19   | enum + 매핑 함수                    |
+| 1.3.2 ProviderConfig     | ✅ 완료 |   16   | 5개 프로바이더 Config + 타입 팩토리 |
+| 1.3.3 ProviderSelector   | ✅ 완료 |   17   | 선택 우선순위 로직 + LlmError       |
+| 1.3.4 Configuration 통합 | ✅ 완료 |   26   | getProviderFromConfig + 검증        |
+
+**총 테스트**: 78개 (M1.3 신규)
+
+---
+
+## 🏗️ 1.3.1 ProviderType 정의
+
+### 구현 파일
+
+- [providerTypes.ts](file:///Users/junghojang/Developments/didimProject/gemini-cli/packages/core/src/providers/providerTypes.ts)
+- [providerTypes.test.ts](file:///Users/junghojang/Developments/didimProject/gemini-cli/packages/core/src/providers/providerTypes.test.ts)
+
+### 주요 내용
+
+```typescript
+enum ProviderType {
+  Gemini = 'gemini',
+  Claude = 'claude',
+  OpenAI = 'openai',
+  OpenAICompatible = 'openai-compatible',
+  Didim = 'didim',
+}
+
+enum AuthType {
+  LOGIN_WITH_GOOGLE = 'oauth-personal',
+  USE_GEMINI = 'gemini-api-key',
+  USE_VERTEX_AI = 'vertex-ai',
+  LEGACY_CLOUD_SHELL = 'cloud-shell',
+  COMPUTE_ADC = 'compute-default-credentials',
+}
+```
+
+### 핵심 함수
+
+- `getAuthTypesForProvider()`: 프로바이더별 지원 인증 타입 반환
+- `getProviderForAuthType()`: AuthType에서 Provider 역추적
+- `isGeminiAuthType()`: Gemini 전용 인증인지 확인
+
+---
+
+## 🏗️ 1.3.2 ProviderConfig 타입
+
+### 구현 파일
+
+- [providerConfig.ts](file:///Users/junghojang/Developments/didimProject/gemini-cli/packages/core/src/providers/providerConfig.ts)
+- [providerConfig.test.ts](file:///Users/junghojang/Developments/didimProject/gemini-cli/packages/core/src/providers/providerConfig.test.ts)
+
+### 프로바이더별 Config
+
+| Interface                | 필수 필드 | 선택 필드                         |
+| ------------------------ | --------- | --------------------------------- |
+| `GeminiProviderConfig`   | -         | `authType`, `project`, `location` |
+| `ClaudeProviderConfig`   | `apiKey`  | `anthropicVersion`                |
+| `OpenAIProviderConfig`   | `apiKey`  | `organization`                    |
+| `OpenAICompatibleConfig` | `baseUrl` | `apiKey`                          |
+| `DidimProviderConfig`    | `apiKey`  | `endpoint`                        |
+
+### Type Guards
+
+- `isGeminiConfig()`, `isClaudeConfig()`, `isOpenAIConfig()`,
+  `isOpenAICompatibleConfig()`, `isDidimConfig()`
+
+---
+
+## 🏗️ 1.3.3 ProviderSelector
+
+### 구현 파일
+
+- [providerSelector.ts](file:///Users/junghojang/Developments/didimProject/gemini-cli/packages/core/src/providers/providerSelector.ts)
+- [providerSelector.test.ts](file:///Users/junghojang/Developments/didimProject/gemini-cli/packages/core/src/providers/providerSelector.test.ts)
+
+### 선택 우선순위
+
+```
+1. LLM_PROVIDER 환경변수
+2. authType 설정값 (Gemini 전용)
+3. GEMINI_API_KEY / GOOGLE_API_KEY (폴백)
+```
+
+### 프로바이더별 환경변수
+
+| Provider          | 필수 환경변수                               |
+| ----------------- | ------------------------------------------- |
+| Gemini            | `GEMINI_API_KEY` 또는 `GOOGLE_API_KEY`      |
+| Claude            | `ANTHROPIC_API_KEY`                         |
+| OpenAI            | `OPENAI_API_KEY`                            |
+| Didim             | `DIDIM_API_KEY`                             |
+| OpenAI-Compatible | `LLM_BASE_URL` (필수), `LLM_API_KEY` (선택) |
+
+### 핵심 함수
+
+- `selectProvider()`: 우선순위 기반 프로바이더 선택
+- `getRequiredEnvVars()`: 필수 환경변수 목록 반환
+- `validateProviderEnv()`: 환경변수 검증
+
+---
+
+## ✅ 테스트 결과
+
+```
+ ✓ src/providers/providerTypes.test.ts (19 tests)
+ ✓ src/providers/providerConfig.test.ts (16 tests)
+ ✓ src/providers/providerSelector.test.ts (17 tests)
+ ✓ src/providers/providerConfigIntegration.test.ts (26 tests)
+
+ Test Files  4 passed (4)
+      Tests  78 passed (78)
+```
+
+### 전체 providers 테스트: **212 passed**
+
+---
+
+## 🔧 기타 변경사항
+
+### 이름 충돌 해결
+
+- `configAdapter.ts`의 `ProviderConfig` → `GenerationConfig`로 변경
+- 생성 파라미터 설정과 프로바이더 구성 설정 구분
+
+### Export 추가 (index.ts)
+
+- `providerTypes.js`
+- `providerConfig.js`
+- `providerSelector.js`
+- `providerConfigIntegration.js`
+
+---
+
+## 🏗️ 1.3.4 Configuration 통합
+
+### 구현 파일
+
+- [providerConfigIntegration.ts](file:///Users/junghojang/Developments/didimProject/gemini-cli/packages/core/src/providers/providerConfigIntegration.ts)
+- [providerConfigIntegration.test.ts](file:///Users/junghojang/Developments/didimProject/gemini-cli/packages/core/src/providers/providerConfigIntegration.test.ts)
+
+### 핵심 함수
+
+| 함수                       | 설명                                               |
+| -------------------------- | -------------------------------------------------- |
+| `getProviderFromConfig()`  | 명시적 옵션 또는 환경변수에서 프로바이더 구성 해결 |
+| `validateProviderConfig()` | 프로바이더별 필수 필드 (apiKey, baseUrl) 검증      |
+| `resolveProviderEnvVars()` | 프로바이더별 환경변수 해결                         |
+
+### 선택 우선순위
+
+```
+1. 명시적 provider 옵션
+2. 명시적 authType 옵션 (Gemini 암시)
+3. LLM_PROVIDER 환경변수
+4. Gemini 기본값
+```
+
+---
+
+## ✅ 검증 기준 체크리스트
+
+- [x] 기존 인증 흐름(ADC/OAuth/Vertex) 유지
+- [x] 신규 provider 선택 경로 동작
+- [x] 우선순위 충돌 시 동작 예측 가능
+- [x] 프로바이더별 환경변수 검증 동작
+
+---
+
+## � M1.3.4 리뷰 피드백 반영 (2026-02-06)
+
+### Issue 1: validateProviderConfig()가 환경변수 기반 구성과 충돌 ✅
+
+**문제**: 옵션만 검증하여 환경변수로 제공된 값이 누락됨
+
+**수정**:
+
+- `validateResolvedConfig()` 함수 추가 - 해결된 구성 검증용
+- `validateProviderConfig()`에 `@deprecated` 주석 추가
+- 테스트 추가: `validateResolvedConfig` 테스트 그룹 (6개)
+
+### Issue 2: timeout 필드가 유실됨 ✅
+
+**문제**: `ResolvedProviderConfig`에 `timeout` 필드 없음
+
+**수정**:
+
+- `ResolvedProviderConfig` 인터페이스에 `timeout?: number` 추가
+- `getProviderFromConfig()`에서 `timeout` 전달/보존
+- 테스트 추가: `should preserve timeout in resolved config`
+
+### 테스트 갭 해결 ✅
+
+| 추가된 테스트                                           | 설명                       |
+| ------------------------------------------------------- | -------------------------- |
+| `should throw for Didim without API key`                | Didim 실패 케이스          |
+| `should resolve baseUrl from env for OpenAI-compatible` | OpenAI-compatible 환경변수 |
+| `validateResolvedConfig` 그룹                           | 6개 테스트 추가            |
+
+**테스트 증가**: 17개 → 26개 (+9개)
+
+---
+
+## �🚀 다음 작업
+
+- **M1.4 유틸리티/테스트 마이그레이션 계획**: 유틸리티 레이어 분석 및
+  마이그레이션 계획 수립
+
+---
+
+## 📝 리뷰 피드백 반영 (2026-02-06)
+
+### Issue 1: ProviderSelector가 OpenAICompatible의 baseUrl 미반환 ✅
+
+**문제**: `ProviderSelection` 인터페이스에 `baseUrl` 필드 없음
+
+**수정**:
+
+- `ProviderSelection` 인터페이스에 `baseUrl?: string` 추가
+- `selectProvider()` 함수에서 `OpenAICompatible` 선택 시 `LLM_BASE_URL` 반환
+- 테스트 추가: `should include baseUrl for OpenAI-compatible`
+
+### Issue 2: 일반 Error 대신 LlmError 사용 ✅
+
+**문제**: `parseProviderEnv()`, `validateProviderEnv()`에서 일반 `Error` 사용
+
+**수정**:
+
+- `LlmError`, `LlmErrorType` import 추가
+- `throw new Error()` → `throw new LlmError(LlmErrorType.INVALID_REQUEST, ...)`
+  변경
+- 테스트 추가: `should throw LlmError for configuration issues`
+
+### Issue 3: AuthType 중복 정의로 인한 혼선 ✅
+
+**문제**: `providerTypes.ts`와 `core/contentGenerator.ts`에 동일 enum 존재
+
+**수정**:
+
+- JSDoc 주석 개선: 순환 의존성 방지를 위한 로컬 복사본임을 명시
+- `@see core/contentGenerator.ts AuthType` 참조 추가
+
+### Issue 4: createProviderConfig 타입 추론 모호성 ✅
+
+**문제**: 제네릭 호출 시 필수 필드(apiKey, baseUrl) 강제 불가
+
+**수정**:
+
+- 프로바이더별 타입 안전 팩토리 함수 추가:
+  - `createGeminiConfig()`
+  - `createClaudeConfig()` - apiKey 필수
+  - `createOpenAIConfig()` - apiKey 필수
+  - `createOpenAICompatibleConfig()` - baseUrl 필수
+  - `createDidimConfig()` - apiKey 필수
+- 테스트 추가: `Typed factory functions` 테스트 그룹
+
+### 문서-코드 차이 수정 ✅
+
+- `isOpenAICompatibleConfig` 타입 가드가 코드에 존재함을 문서에 반영 (위 Type
+  Guards 섹션 확인)