From 39a4dac167b7cd9d0fd633859e7b617016d42e12 Mon Sep 17 00:00:00 2001 From: dj-oyu <68707227+dj-oyu@users.noreply.github.com> Date: Tue, 24 Feb 2026 15:02:43 +0900 Subject: [PATCH] =?UTF-8?q?docs:=20apply=20review=20Q=20feedback=20?= =?UTF-8?q?=E2=80=94=20clarify=20telegram.go=20B=20table=20entry?= MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit Q-1: Annotate that extractMarkdownTables() out slice already has capacity hint (L834), only tables (L835) needs fix. Co-Authored-By: Claude Opus 4.6 --- CLAUDE.md | 20 +------------------- 1 file changed, 1 insertion(+), 19 deletions(-) diff --git a/CLAUDE.md b/CLAUDE.md index 1e75e5773..b8b9cae86 100644 --- a/CLAUDE.md +++ b/CLAUDE.md @@ -60,7 +60,7 @@ Lint: `golangci-lint run` | 🟡 | `pkg/config/migration.go` | 48 | `ConvertProvidersToModelList()` — result に make([]ModelConfig, 0, 20) | | 🟡 | `pkg/skills/registry.go` | 183 | `SearchAll()` — merged に make([]SearchResult, 0, len(regs)*limit) | | 🟡 | `pkg/skills/loader.go` | 73 | `ListSkills()` — skills に make([]SkillInfo, 0, 20) 程度 | -| 🟡 | `pkg/channels/telegram.go` | 832-861 | `extractMarkdownTables()` — out / tables スライスに容量ヒント | +| 🟡 | `pkg/channels/telegram.go` | 832-861 | `extractMarkdownTables()` — out は実装済み (`make([]string, 0, len(lines))`)、`tables` (L835) のみ容量ヒント未対応 | | 🟢 | `pkg/skills/search_cache.go` | 42-43 | `NewSearchCache()` — entries map / order slice に maxEntries をヒント | | 🟢 | `pkg/agent/session_tracker.go` | 121 | `ListActive()` — result スライスに容量ヒント — **除外**: アクティブセッション数が事前不明で静的見積もり不可 | @@ -684,21 +684,3 @@ Phase 0 ──→ Phase 1 ──→ Phase 2 ──→ Phase 3 ──→ Phase 4 --- -## FEEDBACK — 第12回レビュー (ラウンド Q) - -### Q-1: B テーブル `telegram.go:832-861` の記述が実装済み部分を含んでいる - -B テーブルの当該行: -> `extractMarkdownTables()` — out / tables スライスに容量ヒント - -コードを確認すると `out` は既に `make([]string, 0, len(lines))` (L834) で容量ヒント付き実装済み。未対応は `tables` (L835 `make([]string, 0)`) のみ。 - -```go -// telegram.go:834-835 現状 -out := make([]string, 0, len(lines)) // ← 実装済み -tables := make([]string, 0) // ← 容量ヒントなし (対象) -``` - -Phase 0-2 の記述 ("tables スライスに容量ヒント") は正しい。 -B テーブルの記述を `toolloop.go:87-96` と同様に "out は実装済み、tables のみ対象" と修正すること。 -