style: rewrite comments that addressed the reader or narrated old behavior

- A handful of comments were written as instructions to a reader or as
  a changelog ("previously returned 500", "kept for API compatibility");
  they now state the current rule and its reason only
- graph.js pointed at a static/js/include-graph.js that does not exist;
  the include graph lives in static/addons/c-include/addon.js
- ref-map/addon.js had a blank line after every line for a 57-line
  stretch, an editing accident

Author: Taka-S-dev

api/handlers.go

@@ -127,7 +127,7 @@ func (h *Handler) Register(mux *http.ServeMux) {
 	mux.HandleFunc("/api/structure/build", h.handleStructureBuild)
 	mux.HandleFunc("/api/structure/children", h.handleStructureChildren)
 	mux.HandleFunc("/api/structure/edge-symbols", h.handleStructureEdgeSymbols)
-	// [GNU Global] 以下の4行を削除し、definition/hover/callersの分岐を除去で取り外し可能
+	// [GNU Global] この5行と definition/hover/callers の分岐を消せば取り外せる
 	mux.HandleFunc("/api/gtags/status", h.handleGtagsStatus)
 	mux.HandleFunc("/api/gtags/index", h.handleGtagsIndex)
 	mux.HandleFunc("/api/gtags/update", h.handleGtagsUpdate)
@@ -137,7 +137,7 @@ func (h *Handler) Register(mux *http.ServeMux) {
 	mux.HandleFunc("/api/ctags/index", h.handleCtagsIndex)
 	mux.HandleFunc("/api/ctags/file-symbols", h.handleCtagsFileSymbols)
 	mux.HandleFunc("/api/ctags/macros", h.handleCtagsMacros)
-	// [C言語アドオン] 以下の3行を削除するとインクルードグラフAPIが無効になります
+	// [C言語アドオン] この3行を消すとインクルードグラフ API が無効になる
 	mux.HandleFunc("/api/include-graph", h.handleIncludeGraph)
 	mux.HandleFunc("/api/include-file", h.handleIncludeFile)
 	mux.HandleFunc("/api/include-by", h.handleIncludeBy)

api/insertions.go

@@ -376,7 +376,8 @@ func (h *Handler) deleteInsertionByID(w http.ResponseWriter, r *http.Request, id
 			return
 		}
 		// ErrUnsupportedEncoding→415 / ErrMismatch・errRecordedLineModified→409
-		// など、POST/PUT と同じマッピングを通す（以前は一律500にしていた）。
+		// など、POST/PUT と同じマッピングを通す。一律 500 だと呼び出し側が
+		// 「行が変わっていた」と「壊れた」を区別できない
 		patchErrStatus(w, err)
 		return
 	}

lsp/completion.go

@@ -14,8 +14,7 @@ import (
 // 補完はエディタの未保存バッファを見ないと成立しない（"s->" と打った瞬間、
 // ディスクにはまだ無い）。textDocumentSync を full で受け、開いている文書の
 // 内容を持つ。他のハンドラ（hover / definition）も開いていればこちらを優先する。
-// didChange を無視していた「保存済み状態だけを見る」方針からの変更点で、
-// 索引（定義位置など）が保存済み基準なのは変わらない。
+// 索引（定義位置など）は保存済みのファイルが基準のまま。
 
 func (s *server) handleDidOpen(raw json.RawMessage) {
 	var p struct {

mcp/README.md

@@ -295,7 +295,7 @@ main.go の line 1 を「Claude 経由テスト」のラベルでルートノー
 - **callees の self 除外**: `word` を渡しておけば同名は自動で除外される (`excluded_self` カウントが結果に含まれる)。
 - **`fetch failed` 単独で出てきたら**: ブリッジは `ECONNREFUSED` / `ENOTFOUND` / `ETIMEDOUT` を検出して原因コードと対処を表示する。それでも出ない場合は `GREPNAVI_URL` の port を疑う (デフォルト 8080)。
 - **GUI と AI の memo 同時編集**: `/api/graph/memos` が**全置換 PUT** な性質上、AI が memo を書き、ほぼ同時刻に GUI でも別 memo を編集していると一方が消える race window がある。ブリッジは「現状 fetch → merge → PUT」で minimize、加えて server が `memos.updated` SSE を発火 → browser が即時 reload するので**最終状態は通常収束**する。ただし「AI が memo X を書く」と「GUI で memo Y を typing 中」が ms 単位で重なった場合は GUI のローカル変更が prefer される。
-- **version mismatch**: grepnavi 本体や bridge を更新したら**両方とも再起動**する必要がある。bridge は古い `/api/callees` 形式 (`[]string`) も互換吸収するが、その場合 `call_line` が 0 で返る。更新時は `go build && 再起動` + `npm run build` + Claude Code 再起動の **3 ステップ**を忘れずに。
+- **version mismatch**: grepnavi 本体や bridge を更新したら**両方とも再起動**する必要がある。bridge は古い `/api/callees` 形式 (`[]string`) も互換吸収するが、その場合 `call_line` が 0 で返る。更新時は `go build && 再起動` + `npm run build` + Claude Code 再起動の **3 ステップ**が要る。
 
 ## 既知の制限
 

search/ctags.go

@@ -385,7 +385,7 @@ func ctagsParseSymbols(tagsPath string) (SymbolsByKind, error) {
 		rest := line[tab1:]
 		// ファイル欄（2列目）で C/C++ 以外と除外パスを落とす。tags が古い設定で
 		// 作られていると生成物（doxygen の html/js など）の名前が混ざり、補完で
-		// cryptlib_8c.html のようなゴミが出る
+		// cryptlib_8c.html のような候補が出る
 		if tab2 := bytes.IndexByte(rest[1:], '\t'); tab2 >= 0 {
 			file := string(rest[1 : 1+tab2])
 			if !ctagsIsCFile(file) || IsExcluded(filepath.Join(tagsDir, strings.TrimPrefix(file, "./"))) {

search/definition.go

@@ -312,7 +312,7 @@ func findDefinitionsInFiles(ctx context.Context, word string, files []string) ([
 //
 // 実行戦略:
 //   - 全パターン（define / struct / enum / typedef / func）を1つの正規表現に統合して
-//     rg を1回だけ呼ぶ（従来の7プロセス → 1プロセス）。
+//     rg を1回だけ呼ぶ（パターンごとに起動すると 7 プロセスになる）。
 //   - 各ヒットの kind はマッチ行のテキストから事後判定する。
 func FindDefinitionsN(ctx context.Context, word, dir, glob string, maxPerQuery int) ([]DefHit, error) {
 	if word == "" || dir == "" {

search/hover.go

@@ -170,9 +170,8 @@ func FindHover(ctx context.Context, word, dir, glob, root string, includeChain .
 			commentLine = findContainingBlockStart(lines, h.Line)
 		case "func":
 			body = extractBraceBlock(lines, h.Line, 20)
-			// { がない場合は宣言（プロトタイプ）
-			// NOTE: 宣言/定義の判定ロジックは search/classify.go の ClassifyKind と対になっています。
-			// 片方を変更した場合はもう片方も確認してください。
+			// { がない場合は宣言（プロトタイプ）。
+			// 判定は search/classify.go の ClassifyKind と対。片方を変えたらもう片方も直す
 			if !strings.Contains(body, "{") {
 				body = h.Text
 				isDecl = true
@@ -194,9 +193,9 @@ func FindHover(ctx context.Context, word, dir, glob, root string, includeChain .
 		result = append(result, HoverHit{File: h.File, Line: h.Line, Kind: h.Kind, Body: body, Decl: isDecl, Value: value, Healed: h.Healed})
 	}
 
-	// func 結果を実装（decl:false）優先・上限2件でフィルタ
-	// 宣言（decl:true）は実装が見つからない場合のみ最大2件補完
-	// ※ 以前の funcCount >= 2 制限は宣言2件で実装がスキップされるバグがあったため廃止
+	// func 結果を実装（decl:false）優先・上限2件でフィルタ。
+	// 宣言（decl:true）は実装が見つからない場合のみ最大2件補完。宣言の件数で
+	// 先に打ち切ると、宣言が2件あるだけで実装が出なくなる
 	var funcDefs, funcDecls []HoverHit
 	var nonFunc []HoverHit
 	for _, h := range result {

static/addons/ref-map/addon.js

@@ -98,61 +98,33 @@ function closeRefMap() {
 }
 
 // ===== 移動の履歴 =====
-
 // パンくずは祖先しか辿れない。`▾` の兄弟移動やファイルツリーからの直行が入って、
-
 // 「さっき見ていた場所」がパンくずの上に無いことが普通になったので、来た道を
-
 // 別に持つ。エディタ側の履歴 (editor.js の navHistory) と同じ規約で、履歴を
-
 // 辿っている間は積まない。
-
 let _rmHist = [];
-
 let _rmHistIdx = -1;
 
-
-
 function rmHistPush(focus) {
-
   if (_rmHistIdx >= 0 && _rmHist[_rmHistIdx] === focus) return;
-
   _rmHist = _rmHist.slice(0, _rmHistIdx + 1);
-
   _rmHist.push(focus);
-
   _rmHistIdx = _rmHist.length - 1;
-
 }
 
-
-
 function rmHistGo(delta) {
-
   const i = _rmHistIdx + delta;
-
   if (i < 0 || i >= _rmHist.length) return false;
-
   _rmHistIdx = i;
-
   rmLoad(_rmHist[i], { fromHistory: true });
-
   return true;
-
 }
 
-
-
 function rmUpdateNavButtons() {
-
   const b = document.getElementById('rm-back');
-
   const f = document.getElementById('rm-fwd');
-
   if (b) b.disabled = _rmHistIdx <= 0;
-
   if (f) f.disabled = _rmHistIdx >= _rmHist.length - 1;
-
 }
 
 async function rmLoad(focus, opts) {

static/js/graph.js

@@ -1434,7 +1434,7 @@ const nodeBadgeEditor = createNodeFieldModal({
   successToast: () => 'バッジ保存',
 });
 
-// F2 / 右クリック「名前を変更」のエントリポイント。lbl/m は API 互換のため残置。
+// F2 / 右クリック「名前を変更」のエントリポイント。lbl/m は呼び出し側の形を揃えるためだけに受ける。
 function startNodeLabelEdit(lbl, node, m) {
   nodeLabelEditor.open(node);
 }
@@ -2676,7 +2676,7 @@ function attachIndentDrag(handle, row, nodeId, depth) {
   }
 }
 
-// インクルード依存グラフ機能は static/js/include-graph.js に分離されています。
+// インクルード依存グラフは static/addons/c-include/addon.js にある。
 
 // ===== CODE SNAPSHOT =====
 // 実装は editor.js の exportSelectionSnapshot() を参照