From ee0afbe1938de20b0f327947f202c0d750c61319 Mon Sep 17 00:00:00 2001
From: dj-oyu <68707227+dj-oyu@users.noreply.github.com>
Date: Tue, 24 Feb 2026 14:25:49 +0900
Subject: [PATCH] docs: add 8th review feedback (M-1) to CLAUDE.md
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit

M-1: Phase 3-2 "効果範囲の限定" example incorrectly lists GetReviewContext
as a split optimization target — it only does sb.WriteString(content) with
no split helpers. Same for GetInterviewContext. Actual targets are
GetPlanContext and FormatPlanDisplay (already correct in the action section).
No impact on implementation correctness.

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
---
 CLAUDE.md | 28 ++++++++++++++++++++++++++++
 1 file changed, 28 insertions(+)

diff --git a/CLAUDE.md b/CLAUDE.md
index 33fa15456..ce0c22393 100644
--- a/CLAUDE.md
+++ b/CLAUDE.md
@@ -678,3 +678,31 @@ Phase 0 ──→ Phase 1 ──→ Phase 2 ──→ Phase 3 ──→ Phase 4
 - Phase 4: **ディスク書き込み削減** (microSD 寿命保護)
 - Phase 5: 必要に応じて個別判断
 
+---
+
+## FEEDBACK — 第8回レビュー (M)
+
+> レビュー日: 2026-02-24。アクション指示は正確。Phase 3-2 の説明注記に軽微な誤りあり。
+
+### M-1. Phase 3-2「効果範囲の限定」の例示が不正確 (軽微)
+
+**指摘内容**: Phase 3-2 の説明注記に「この統合が効くのは読み取り専用メソッド (`GetPlanContext`, `GetReviewContext`, `FormatPlanDisplay` 等) のみ」とあるが、`GetReviewContext()` は `strings.Split` を呼ぶヘルパーを使わないため Phase 3-2 の恩恵がない。
+
+**`GetReviewContext()` L545-555 の実態**:
+```go
+func (ms *MemoryStore) GetReviewContext() string {
+    content := ms.ReadLongTerm()
+    var sb strings.Builder
+    sb.WriteString(content)   // content をそのまま書き込むだけ — Split なし
+    ...
+}
+```
+
+同様に `GetInterviewContext()` (L497-541) も `sb.WriteString(content)` するだけで split ヘルパーを呼ばない。
+
+Phase 3-2 で実際に恩恵を受けるのは:
+- `GetPlanContext()` — `extractPhaseContent` / `extractCommandsSection` / `extractContextSection` を直接呼ぶ (対応セクションに正しく記載済み ✓)
+- `FormatPlanDisplay()` — `getPlanPhasesFrom` / `extractCommandsSection` / `extractContextSection` を直接呼ぶ (同上 ✓)
+
+**実装への影響**: なし (対応セクションの指示は正確)。「効果範囲の限定」注記の例示を `GetReviewContext` → `GetPlanContext`/`FormatPlanDisplay` に差し替えれば正確になる。
+