@didim365/agent-cli-core 0.1.0

package/dist/docs/ai_adapter/working_history/Phase2_M2.6_/353/235/274/354/232/260/355/214/205/353/240/210/354/235/264/354/226/264/355/203/200/354/236/205/353/217/205/353/246/275/355/231/224_20260208.md

@@ -0,0 +1,228 @@
+# Phase 2: M2.6 라우팅 레이어 타입 독립화 작업 결과서
+
+> 📅 **작업일**: 2026-02-08 📚 **Phase**: Phase 2 - M2.6 🎯 **목표**: 라우팅
+> 레이어의 `@google/genai` 의존성 제거 — RoutingContext와 전략 구현체의
+> provider-independent 타입 전환 🔗 **이전 작업**:
+> [Phase2*M2.5*유틸리티레이어리팩토링\_20260208.md](./Phase2_M2.5_유틸리티레이어리팩토링_20260208.md)
+> 🔗 **관련 문서**:
+> [phase2_core_refactoring_todolist.md](../todolist/phase2_core_refactoring_todolist.md) -
+> M2.6
+
+---
+
+## 📋 작업 요약
+
+### 접근 전략
+
+**인라인 타입 전환**: M2.5의 "병행 추가 + @deprecated" 패턴과 달리, M2.6에서는
+RoutingContext 인터페이스를 직접 변경하는 방식을 채택. 이유: todolist 검증
+기준이 "routingStrategy.ts에서 @google/genai import 제거"를 명시적으로 요구.
+
+**변환 브릿지 패턴**: 호출 지점(client.ts, local-executor.ts)에서 Gemini SDK
+타입을 LlmMessage/LlmContent로 변환하는 유틸리티 함수를 새로 생성. 라우팅 레이어
+내부는 완전히 provider-independent하게 유지.
+
+**작업 순서**: 유틸리티 함수 TDD → RoutingContext 타입 전환(2.6.1) → 전략 구현체
+마이그레이션(2.6.2) → 호출 지점 + 테스트 전환(2.6.3)
+
+### 핵심 발견 사항 (사전작업)
+
+- `baseLlmClient.generateJson()`이 이미
+  `LlmGenerateJsonOptions`(`messages: LlmMessage[]` 형식)를 지원
+- `Type.OBJECT`/`Type.STRING`/`Type.INTEGER` 등 `@google/genai`의 Type enum 값은
+  단순 문자열 리터럴(`'OBJECT'`, `'STRING'`, `'INTEGER'`)로 대체 가능
+- 라우팅 전략 중 `compositeStrategy`, `defaultStrategy`, `overrideStrategy`,
+  `fallbackStrategy`는 `@google/genai`를 직접 import하지 않아 구현·테스트 모두
+  코드 변경 불필요 (`{} as RoutingContext` 캐스팅으로 타입 변경에 영향 없음)
+
+---
+
+## ✅ 변경 사항
+
+### 신규 파일
+
+| 파일                                     | 설명                                           |
+| ---------------------------------------- | ---------------------------------------------- |
+| `src/utils/geminiTypeConversion.ts`      | Gemini SDK → LlmMessage/LlmContent 변환 브릿지 |
+| `src/utils/geminiTypeConversion.test.ts` | 변환 함수 테스트 (18개)                        |
+
+### 2.6.1 RoutingContext 타입 전환 ✅
+
+| 항목                                               | 상태 | 비고                                                    |
+| -------------------------------------------------- | :--: | ------------------------------------------------------- |
+| `routingStrategy.ts` — `@google/genai` import 제거 |  ✅  | `Content`, `PartListUnion` → `LlmMessage`, `LlmContent` |
+| `history: Content[]` → `history: LlmMessage[]`     |  ✅  | 직접 타입 전환                                          |
+| `request: PartListUnion` → `request: LlmContent[]` |  ✅  | 직접 타입 전환                                          |
+
+### 2.6.2 라우팅 전략 구현체 마이그레이션 ✅
+
+| 항목                                                    | 상태 | 비고                                                                              |
+| ------------------------------------------------------- | :--: | --------------------------------------------------------------------------------- |
+| `classifierStrategy.ts` — `@google/genai` 제거          |  ✅  | `createUserContent`, `Type` import 제거                                           |
+| `classifierStrategy.ts` — `messages` 형식 전환          |  ✅  | `contents` → `messages` (LlmGenerateJsonOptions)                                  |
+| `classifierStrategy.ts` — 필터 로직 전환                |  ✅  | `isFunctionCall`/`isFunctionResponse` → `isToolCallMessage`/`isToolResultMessage` |
+| `numericalClassifierStrategy.ts` — `@google/genai` 제거 |  ✅  | `Type` import 제거, `extractText` 활용                                            |
+| `numericalClassifierStrategy.ts` — `messages` 형식 전환 |  ✅  | `contents` → `messages` (LlmGenerateJsonOptions)                                  |
+| `compositeStrategy.ts` — 코드 변경 불필요               |  ✅  | RoutingContext 타입 자동 전파                                                     |
+| `defaultStrategy.ts` — 코드 변경 불필요                 |  ✅  | RoutingContext 타입 자동 전파                                                     |
+| `fallbackStrategy.ts` — 코드 변경 불필요                |  ✅  | RoutingContext 타입 자동 전파                                                     |
+| `overrideStrategy.ts` — 코드 변경 불필요                |  ✅  | RoutingContext 타입 자동 전파                                                     |
+
+### 2.6.3 호출 지점 마이그레이션 ✅
+
+| 항목                                         | 상태 | 비고                                                                    |
+| -------------------------------------------- | :--: | ----------------------------------------------------------------------- |
+| `client.ts` — 변환 브릿지 적용               |  ✅  | `convertContentsToLlmMessages()`, `convertPartListUnionToLlmContents()` |
+| `local-executor.ts` — 변환 브릿지 적용       |  ✅  | 동일 변환 함수 적용                                                     |
+| `modelRouterService.test.ts` — mock 업데이트 |  ✅  | `request` 형식을 LlmContent[] 포맷으로 전환                             |
+
+### Refactor ✅
+
+| 항목                                        | 상태 | 비고                                                                                      |
+| ------------------------------------------- | :--: | ----------------------------------------------------------------------------------------- |
+| `messageInspectors.ts` — `@deprecated` 추가 |  ✅  | `isFunctionCall` → `isToolCallMessage`, `isFunctionResponse` → `isToolResultMessage` 안내 |
+| `llmUtils.ts` — 메시지 레벨 inspector 추가  |  ✅  | `isToolCallMessage()`, `isToolResultMessage()` (M2.5의 content-level guard를 보완)        |
+
+---
+
+## 🧪 테스트 현황
+
+### 신규 테스트
+
+| 파일                           | 테스트 수 | 내용                                                                                              |
+| ------------------------------ | :-------: | ------------------------------------------------------------------------------------------------- |
+| `geminiTypeConversion.test.ts` |    18     | `convertContentToLlmMessage`, `convertContentsToLlmMessages`, `convertPartListUnionToLlmContents` |
+| `llmUtils.test.ts` (추가분)    |     8     | `isToolCallMessage`, `isToolResultMessage`                                                        |
+
+### 수정된 테스트 파일
+
+| 파일                                  | 변경 내용                                                                                                    |
+| ------------------------------------- | ------------------------------------------------------------------------------------------------------------ |
+| `classifierStrategy.test.ts`          | `Content[]` → `LlmMessage[]`, `GenerateJsonOptions` → `LlmGenerateJsonOptions`, `contents` → `messages` 접근 |
+| `numericalClassifierStrategy.test.ts` | `Content` import → `LlmMessage`, mock 데이터 LlmMessage/LlmContent 형식 전환                                 |
+| `modelRouterService.test.ts`          | `request` mock을 LlmContent[] 형식으로 전환                                                                  |
+
+### 전체 테스트 결과
+
+```
+ Test Files  260 passed (260)
+      Tests  4829 passed | 24 skipped (4853)
+ TypeCheck   All workspaces passed
+ Lint        0 errors, 0 warnings
+```
+
+---
+
+## 📁 변경 파일 목록
+
+### 구현 파일 (7개)
+
+| 파일                                                    | 변경 유형                                                  |
+| ------------------------------------------------------- | ---------------------------------------------------------- |
+| `src/routing/routingStrategy.ts`                        | Modified — 타입 전환                                       |
+| `src/routing/strategies/classifierStrategy.ts`          | Modified — `@google/genai` 제거, `messages` 형식 전환      |
+| `src/routing/strategies/numericalClassifierStrategy.ts` | Modified — `@google/genai` 제거, `messages` 형식 전환      |
+| `src/core/client.ts`                                    | Modified — 변환 브릿지 적용                                |
+| `src/agents/local-executor.ts`                          | Modified — 변환 브릿지 적용                                |
+| `src/utils/geminiTypeConversion.ts`                     | **New** — Gemini SDK → LlmMessage/LlmContent 변환          |
+| `src/utils/llmUtils.ts`                                 | Modified — `isToolCallMessage`, `isToolResultMessage` 추가 |
+
+### 리팩토링 파일 (1개)
+
+| 파일                             | 변경 유형                          |
+| -------------------------------- | ---------------------------------- |
+| `src/utils/messageInspectors.ts` | Modified — `@deprecated` 주석 추가 |
+
+### 테스트 파일 (5개)
+
+| 파일                                                         | 변경 유형                                    |
+| ------------------------------------------------------------ | -------------------------------------------- |
+| `src/utils/geminiTypeConversion.test.ts`                     | **New** — 변환 함수 테스트                   |
+| `src/utils/llmUtils.test.ts`                                 | Modified — 메시지 레벨 inspector 테스트 추가 |
+| `src/routing/strategies/classifierStrategy.test.ts`          | Modified — LlmMessage 형식 전환              |
+| `src/routing/strategies/numericalClassifierStrategy.test.ts` | Modified — LlmMessage 형식 전환              |
+| `src/routing/modelRouterService.test.ts`                     | Modified — mock 데이터 전환                  |
+
+---
+
+## 🔑 검증 기준 충족 확인
+
+| 검증 기준                                            | 결과 |
+| ---------------------------------------------------- | :--: |
+| `routingStrategy.ts`에서 `@google/genai` import 제거 |  ✅  |
+| 모든 라우팅 전략 구현체 타입 전환 완료               |  ✅  |
+| 기존 라우팅 테스트 100% 통과 (52개)                  |  ✅  |
+| 전체 테스트 스위트 통과 (4829개)                     |  ✅  |
+| TypeScript 타입체크 통과                             |  ✅  |
+| ESLint 통과                                          |  ✅  |
+
+---
+
+## 📝 설계 결정 사항
+
+### 인라인 전환 vs 병행 추가
+
+M2.5에서는 기존 함수를 유지하고 `@deprecated`를 붙이는 병행 추가 패턴을
+사용했지만, M2.6에서는 `RoutingContext` 인터페이스를 직접 전환. 이유:
+
+1. todolist 검증 기준이 명시적으로 `@google/genai` import 제거를 요구
+2. `RoutingContext`는 내부 인터페이스로 외부 API 아님
+3. 호출 지점이 2곳(client.ts, local-executor.ts)으로 제한적
+
+### 변환 브릿지 분리
+
+`geminiTypeConversion.ts`를 독립 유틸리티로 생성 (GeminiConverter 클래스 확장 안
+함). 이유:
+
+1. 호출 지점(client.ts, local-executor.ts)은 core 모듈 → provider-specific
+   클래스 의존 회피
+2. 변환은 일시적 브릿지 → 향후 호출 지점 자체가 LlmMessage 기반으로 전환되면
+   제거 예정
+
+### Type enum 문자열 리터럴 대체
+
+`@google/genai`의 `Type.OBJECT` = `'OBJECT'` 등이 단순 문자열임을 확인 후
+리터럴로 대체. JSON Schema 형식의 RESPONSE_SCHEMA가 Gemini API에 직접 전달되므로
+값은 동일.
+
+---
+
+## 🔧 리뷰 수정 사항
+
+### 이슈 1 (High): `fileData` 변환 누락
+
+- **위치**: `geminiTypeConversion.ts:59`
+- **문제**: `inlineData`만 처리하고 `fileData`(URI 기반 파일 참조)는 `null` 반환
+- **수정**: `fileData` → `LlmImageContent(source: { type: 'url' })` 분기 추가
+- **회귀 테스트**:
+  `should convert fileData parts to image content with url source`
+
+### 이슈 2 (High): 빈 content 배열의 `every()` 오탐
+
+- **위치**: `llmUtils.ts:80,91`
+- **문제**: `[].every(fn)` = `true` (vacuous truth) → 빈 content 메시지가 tool
+  메시지로 오분류
+- **수정**: `message.content.length > 0` 가드 추가
+- **회귀 테스트**:
+  `should return false for assistant/user message with empty content` (2건)
+
+### 이슈 3 (Medium): `thought: false` 텍스트의 thought 오분류
+
+- **위치**: `geminiTypeConversion.ts:27`
+- **문제**: `thought !== undefined` 조건이 `thought: false`도 통과시켜 일반
+  텍스트를 thought로 변환
+- **수정**: truthiness 체크 (`thoughtPart.thought`)로 변경 — SDK의
+  `semantic.ts`(`part.thought`) 패턴과 일치
+- **회귀 테스트**:
+  `should treat thought:false text as regular text, not thought`,
+  `should treat thought:true as thought content`
+
+---
+
+## 🔗 후속 작업
+
+- `messageInspectors.ts`의 `isFunctionCall`/`isFunctionResponse`는
+  `loopDetectionService.ts`, `geminiChat.ts`, `editCorrector.ts`,
+  `nextSpeakerChecker.ts`에서 아직 사용 중 → M2.7 이후 마이그레이션 대상
+- `geminiTypeConversion.ts` 변환 브릿지는 호출 지점(client.ts,
+  local-executor.ts)이 LlmMessage 기반으로 전환되면 제거 예정

package/dist/docs/ai_adapter/working_history/Phase3_ETC_/352/270/260/353/263/270/354/273/250/355/205/215/354/212/244/355/212/270/355/214/214/354/235/274/353/252/205AGENTS/354/240/204/355/231/230_20260211.md

@@ -0,0 +1,367 @@
+# Phase3 ETC 기본 컨텍스트 파일명 AGENTS 전환 작업 결과서
+
+- 작업일: 2026-02-11
+- 범위: 기본 컨텍스트 파일명을 `GEMINI.md`에서 `AGENTS.md`로 전환
+- 상태: 완료
+
+---
+
+## 1) 작업 배경
+
+- 현재 컨텍스트 파일 기본값이 `GEMINI.md`로 고정되어 있어 운영 관점에서
+  `AGENTS.md` 중심 정책과 불일치.
+- 요청사항:
+  - 기본 파일명 자체를 `AGENTS.md`로 변경
+  - `/init`, `/memory`, extension 기본 탐색 경로, UI 안내 문구를 일관화
+
+---
+
+## 2) 코드 변경 요약
+
+### 2.1 기본값 전환
+
+- `packages/core/src/tools/memoryTool.ts`
+  - `DEFAULT_CONTEXT_FILENAME`:
+    - `GEMINI.md` → `AGENTS.md`
+
+### 2.2 `/init` 경로 전환
+
+- `packages/core/src/commands/init.ts`
+  - 기본 파일명을 상수(`DEFAULT_CONTEXT_FILENAME`) 기반으로 사용
+  - 안내 문구도 동적 파일명 반영
+- `packages/cli/src/ui/commands/initCommand.ts`
+  - 생성 파일 경로를 `AGENTS.md` 기본값으로 사용
+- `packages/a2a-server/src/commands/init.ts`
+  - 생성 파일 경로/설명 문구를 `AGENTS.md` 기본값으로 사용
+
+### 2.3 extension 기본 context 파일 전환
+
+- `packages/cli/src/config/extension-manager.ts`
+  - extension config에 `contextFileName` 미지정 시 기본값:
+    - `['GEMINI.md']` → `[DEFAULT_CONTEXT_FILENAME, 'GEMINI.md']` (fallback 체인)
+  - 첫 번째로 발견된 파일만 사용하여 중복 로드 방지
+- `packages/a2a-server/src/config/extension.ts`
+  - 동일하게 fallback 체인 + 첫 발견 파일 사용 로직 적용
+
+### 2.4 `/memory` 및 UI 노출 문구 정리
+
+- `packages/core/src/commands/memory.ts`
+  - `/memory list` 응답에서 하드코딩 `GEMINI.md` 제거
+  - 다중 파일명 지원: `getAllGeminiMdFilenames()` 기준 출력
+  - 단일/다중 파일 구분 표시
+- `packages/cli/src/ui/components/Tips.tsx`
+- `packages/cli/src/ui/constants/tips.ts`
+- `packages/cli/src/ui/AppContainer.tsx`
+- `packages/cli/src/ui/hooks/useShowMemoryCommand.ts`
+- `packages/cli/src/ui/commands/directoryCommand.tsx`
+- `packages/cli/src/ui/commands/memoryCommand.ts`
+- `packages/a2a-server/src/commands/memory.ts`
+- `packages/cli/src/config/settingsSchema.ts`
+  - 사용자 노출 문구를 `AGENTS.md` 기준 또는 일반화된 context 파일 표현으로 정리
+
+### 2.5 주석/설명 텍스트 정리
+
+- `packages/core/src/utils/memoryDiscovery.ts`
+- `packages/core/src/utils/memoryImportProcessor.ts`
+- `packages/core/src/tools/read-many-files.ts`
+- `packages/core/src/config/config.ts`
+  - 문서/주석의 `GEMINI.md` 고정 표현을 context 파일 기준으로 정리
+
+---
+
+## 3) 테스트/검증
+
+### 3.1 Core
+
+```bash
+npm test --workspace @google/gemini-cli-core -- \
+  src/commands/init.test.ts \
+  src/commands/memory.test.ts \
+  src/tools/memoryTool.test.ts \
+  src/config/config.test.ts
+```
+
+- 결과: PASS (`171 passed`)
+
+### 3.2 CLI
+
+```bash
+npm test --workspace @google/gemini-cli -- \
+  src/ui/commands/initCommand.test.ts \
+  src/ui/commands/memoryCommand.test.ts \
+  src/ui/components/Tips.test.tsx \
+  src/ui/commands/directoryCommand.test.tsx \
+  src/ui/hooks/useShowMemoryCommand.test.ts
+```
+
+- 결과: PASS (`37 passed`)
+
+```bash
+npm test --workspace @google/gemini-cli -- \
+  src/ui/components/AppHeader.test.tsx \
+  src/ui/components/AlternateBufferQuittingDisplay.test.tsx -u
+```
+
+- 결과: PASS (`17 passed`)
+- 비고: `Tips` 문구 변경으로 snapshot 12건 업데이트
+
+### 3.3 A2A Server
+
+```bash
+npm test --workspace @google/gemini-cli-a2a-server -- \
+  src/commands/init.test.ts \
+  src/commands/memory.test.ts
+```
+
+- 결과: PASS (`12 passed`)
+- 비고: `extension.test.ts` 파일은 존재하지 않음 (§6.3 참조)
+
+### 3.4 Typecheck
+
+```bash
+npm run typecheck --workspace @google/gemini-cli-core
+npm run typecheck --workspace @google/gemini-cli
+npm run typecheck --workspace @google/gemini-cli-a2a-server
+```
+
+- 결과: 전부 PASS
+
+---
+
+## 4) 변경 파일 (본 작업 범위)
+
+- `packages/core/src/tools/memoryTool.ts`
+- `packages/core/src/commands/init.ts`
+- `packages/core/src/commands/memory.ts`
+- `packages/core/src/commands/memory.test.ts`
+- `packages/core/src/tools/memoryTool.test.ts`
+- `packages/core/src/utils/memoryDiscovery.ts`
+- `packages/core/src/utils/memoryImportProcessor.ts`
+- `packages/core/src/tools/read-many-files.ts`
+- `packages/core/src/config/config.ts`
+- `packages/cli/src/ui/commands/initCommand.ts`
+- `packages/cli/src/config/extension-manager.ts`
+- `packages/cli/src/ui/commands/memoryCommand.ts`
+- `packages/cli/src/ui/components/Tips.tsx`
+- `packages/cli/src/ui/components/Tips.test.tsx`
+- `packages/cli/src/ui/constants/tips.ts`
+- `packages/cli/src/ui/AppContainer.tsx`
+- `packages/cli/src/ui/hooks/useShowMemoryCommand.ts`
+- `packages/cli/src/ui/commands/directoryCommand.tsx`
+- `packages/cli/src/config/settingsSchema.ts`
+- `packages/cli/src/ui/components/__snapshots__/AppHeader.test.tsx.snap`
+- `packages/cli/src/ui/components/__snapshots__/AlternateBufferQuittingDisplay.test.tsx.snap`
+- `packages/a2a-server/src/config/extension.ts`
+- `packages/a2a-server/src/commands/init.ts`
+- `packages/a2a-server/src/commands/memory.ts`
+
+---
+
+## 5) 호환성 메모
+
+- 기본값은 `AGENTS.md`로 전환되었지만, 설정으로 다중 파일명을 유지 가능:
+  - `context.fileName: ["AGENTS.md", "GEMINI.md"]`
+- 즉, 기존 `GEMINI.md` 기반 프로젝트도 설정으로 병행 운영 가능.
+
+---
+
+## 6) 리뷰 반영 (2026-02-11)
+
+### 6.1 중간: `/memory list` 다중 파일명 오표기
+
+**문제**: `getCurrentGeminiMdFilename()`은 배열일 때 첫 번째 파일명만 반환
+(`memoryTool.ts:81-86`). `listMemoryFiles()`가 단일 파일명으로 전체 결과를
+라벨링하여, `context.fileName=["AGENTS.md","GEMINI.md"]` 설정 시 "There are N
+AGENTS.md file(s)..."로 고정 표기됨.
+
+**영향**: 다중 컨텍스트 파일 설정 시 사용자가 실제 탐색 범위를 오인할 수 있음.
+
+**수정** (`memory.ts:82-100`):
+- `getAllGeminiMdFilenames()`를 사용하여 모든 설정 파일명 획득
+- 다중 파일 시: `"context file(s) (AGENTS.md, GEMINI.md)"`
+- 단일 파일 시: `"AGENTS.md file(s)"` (기존 동작 유지)
+- else 블록(파일 없음)도 동일 패턴 적용
+
+**부수 수정**:
+- `getCurrentGeminiMdFilename` import 제거 (미사용)
+
+### 6.2 중간: Extension 레거시 호환성 리스크
+
+**문제**: `getContextFileNames()` 기본값이 `[DEFAULT_CONTEXT_FILENAME]`
+(= `['AGENTS.md']`) 단일값으로 변경됨 (`extension-manager.ts:920`,
+`extension.ts:131`). `contextFileName` 미지정 + `GEMINI.md`만 가진 기존 확장은
+컨텍스트 로드 실패 (조용한 실패 → context 미적용).
+
+**영향**: 레거시 확장 호환성 리스크. 기존 확장이 예고 없이 컨텍스트를 로드하지
+못할 수 있음.
+
+**수정**:
+- 기본값을 `[DEFAULT_CONTEXT_FILENAME, 'GEMINI.md']`로 변경하여 fallback 체인
+  지원
+- **첫 번째로 발견된 파일만 사용**: `extension-manager.ts:610-621`,
+  `extension.ts:107-117`에서 for 루프 + break 패턴 구현
+- 우선순위: AGENTS.md → GEMINI.md (둘 다 있으면 AGENTS.md만 로드)
+- 중복 로드 방지: 두 파일이 모두 있어도 첫 발견 파일만 사용하여 프롬프트 중복/충돌
+  회피
+
+### 6.3 낮음: 작업결과서 검증 명령 부정확
+
+**문제**: 결과서 §3.3 (line 112)에 존재하지 않는 테스트 파일 포함:
+`src/config/extension.test.ts` (실제: a2a-server에 해당 파일 없음).
+
+**수정**: 검증 명령 정정:
+```bash
+# 수정 전 (부정확)
+npm test --workspace @google/gemini-cli-a2a-server -- \
+  src/commands/init.test.ts \
+  src/commands/memory.test.ts \
+  src/config/extension.test.ts
+
+# 수정 후 (정확)
+npm test --workspace @google/gemini-cli-a2a-server -- \
+  src/commands/init.test.ts \
+  src/commands/memory.test.ts
+```
+
+**비고**: a2a-server는 `config.test.ts`, `settings.test.ts`만 존재. extension 관련
+테스트 파일 없음.
+
+### 6.4 리뷰 반영 후 검증 (1차)
+
+| 범위 | 결과 |
+|------|------|
+| memory.test.ts (12 tests) | ✅ All passed |
+| Lint | ✅ Clean |
+| Typecheck | ✅ Clean |
+
+**기능 검증**:
+- 단일 파일 설정 (`AGENTS.md`): "There are N AGENTS.md file(s)..." ✅
+- 다중 파일 설정 (`["AGENTS.md", "GEMINI.md"]`): "There are N context file(s)
+  (AGENTS.md, GEMINI.md)..." ✅
+- 파일 없음: "No AGENTS.md files in use." (단일) / "No context files (AGENTS.md,
+  GEMINI.md) in use." (다중) ✅
+- Extension 레거시 호환: `contextFileName` 미지정 시 AGENTS.md → GEMINI.md
+  fallback 동작 ✅
+
+---
+
+## 7) 추가 리뷰 반영 (2026-02-11)
+
+### 7.1 중간: Extension fallback 중복 로드 방지
+
+**문제 (1차 리뷰 반영 후 발견)**: `extension-manager.ts:610-614`,
+`extension.ts:107-109`에서 `getContextFileNames()`가 반환한 **모든** 파일명을
+매핑하고 존재하는 것 전부를 로드함. 결과서 §6.2에는 "첫 번째로 발견된 파일만
+사용"이라 설명했지만, 실제 코드는 `.filter(fs.existsSync)`로 **존재하는 모든
+파일**을 로드.
+
+**영향**: extension 디렉터리에 AGENTS.md와 GEMINI.md가 함께 있으면 컨텍스트가
+중복 로드되어 의도치 않은 프롬프트 중복/충돌 가능성.
+
+**수정**:
+- `extension-manager.ts:610-621`: for 루프 + break로 첫 발견 파일만 사용
+- `extension.ts:107-117`: 동일 패턴 적용
+- 주석 추가: "Use the first existing context file from the configured list. This
+  prevents duplicate context loading when multiple files exist."
+
+### 7.2 낮음: `/memory list` "파일 없음" 메시지 `files` 중복
+
+**문제**: `memory.ts:97-99`에서 `"context files (AGENTS.md, GEMINI.md)"`
+(line 97) + `"files in use."` (line 99) → `"No context files (...) files in
+use."` — `files` 중복.
+
+**수정**:
+- 단일 파일: `No ${allFilenames[0]} files in use.`
+- 다중 파일: `No context files (AGENTS.md, GEMINI.md) in use.`
+
+### 7.3 낮음: 작업결과서 본문 일관성
+
+**문제**: §2.3/2.4에 예전 설명(`[DEFAULT_CONTEXT_FILENAME]` 단일,
+`getCurrentGeminiMdFilename`) 잔존. §6.2에는 fallback 체인/다중 처리로 기술되어
+상충.
+
+**수정**: §2.3/2.4를 fallback 체인 + 첫 발견 파일 사용 로직으로 정정.
+
+### 7.4 추가 리뷰 반영 후 검증 (2차)
+
+| 범위 | 결과 |
+|------|------|
+| memory.test.ts (12 tests) | ✅ All passed |
+| Lint | ✅ Clean |
+| Typecheck | ✅ Clean |
+
+**기능 검증**:
+- Extension 첫 발견 파일 사용: AGENTS.md + GEMINI.md 공존 시 AGENTS.md만 로드 ✅
+- `/memory list` "파일 없음" 문구: `files` 중복 제거 ✅
+- 작업결과서 본문 일관성: §2.3/2.4와 §6.2/§7 내용 정합성 확보 ✅
+
+---
+
+## 8) 회귀 테스트 추가 (2026-02-12)
+
+### 8.1 배경
+
+리뷰를 통해 식별된 주요 기능 변경에 대한 회귀 테스트 부재:
+- **낮음**: Extension fallback(AGENTS→GEMINI, 첫 발견 1개만 로드) 동작 미검증
+- **낮음**: `/memory list`의 다중 파일명 + 파일 없음 문구 분기 미검증
+
+**리스크**: 향후 리팩터링 시 중복 로드 방지 로직 또는 다중 파일 표시 로직이
+깨져도 조기 탐지 어려움.
+
+### 8.2 추가된 테스트
+
+#### 8.2.1 Extension Fallback 회귀 테스트
+
+**파일**: `packages/cli/src/config/extension.test.ts`
+
+**추가된 테스트** (3건):
+
+1. **`should load only AGENTS.md when both AGENTS.md and GEMINI.md exist (first-found fallback)`**
+   - 시나리오: `contextFileName` 미지정 + AGENTS.md + GEMINI.md 공존
+   - 검증: `contextFiles`에 AGENTS.md만 포함 (길이 1)
+   - 목적: 첫 발견 파일만 로드하는 로직 검증 (중복 방지)
+
+2. **`should load GEMINI.md when only GEMINI.md exists (fallback works)`**
+   - 시나리오: `contextFileName` 미지정 + GEMINI.md만 존재 (레거시 확장)
+   - 검증: `contextFiles`에 GEMINI.md 포함
+   - 목적: Fallback chain 동작 검증 (레거시 호환)
+
+3. **`should load AGENTS.md when only AGENTS.md exists (primary path)`**
+   - 시나리오: `contextFileName` 미지정 + AGENTS.md만 존재
+   - 검증: `contextFiles`에 AGENTS.md 포함
+   - 목적: Primary path 우선 로드 검증
+
+**구현 상세**:
+- `DEFAULT_CONTEXT_FILENAME` import 추가 (`@google/gemini-cli-core`)
+- `createExtension()` + 수동 `fs.writeFileSync()`로 파일 조합 제어
+- `contextFiles` 배열 길이 및 경로 정확성 검증
+
+#### 8.2.2 Multi-File Display 회귀 테스트
+
+**파일**: `packages/core/src/commands/memory.test.ts`
+
+**추가된 테스트** (2건, §7에서 추가됨):
+
+1. **`should display multiple filenames when configured`**
+   - 시나리오: `setGeminiMdFilename(['AGENTS.md', 'GEMINI.md'])` + 파일 2개
+   - 검증: `"There are 2 context file(s) (AGENTS.md, GEMINI.md) in use:"`
+   - 목적: 다중 파일 라벨 표시 검증
+
+2. **`should display multi-filename label when no files exist`**
+   - 시나리오: `setGeminiMdFilename(['AGENTS.md', 'GEMINI.md'])` + 파일 0개
+   - 검증: `"No context files (AGENTS.md, GEMINI.md) in use."`
+   - 목적: 다중 파일 "파일 없음" 문구 검증
+
+### 8.3 검증 결과
+
+| 범위 | 테스트 수 | 결과 |
+|------|----------|------|
+| extension.test.ts | 79 (76+3) | ✅ All passed |
+| memory.test.ts | 14 (12+2) | ✅ All passed |
+| Lint (CLI) | - | ✅ Clean |
+| Typecheck (CLI) | - | ✅ Clean |
+
+**회귀 방지 효과**:
+- Extension 중복 로드 방지 로직 변경 시 조기 탐지 가능
+- Multi-file display 문구 변경 시 예상치 못한 문구 오류 방지
+- 리팩터링 안전성 확보 (fallback chain + 첫 발견 패턴 보존)