From 6067f85c039f67b30ba9c73d2bf923103287a8ae Mon Sep 17 00:00:00 2001
From: JianFeeeee <JianFeeeee@users.noreply.gitcode.com>
Date: Sun, 25 Oct 2026 20:15:43 +0800
Subject: [PATCH] =?UTF-8?q?fix(memory):=20graph=5Fcentroid=20=E5=BB=BA?=
 =?UTF-8?q?=E8=A1=A8=E5=BD=92=E4=BD=8D=E5=88=B0=20schemas=20=E2=80=94?=
 =?UTF-8?q?=E2=80=94=20=E6=96=B0=E5=BA=93=E4=B8=8D=E5=86=8D=E6=AF=8F?=
 =?UTF-8?q?=E8=BD=AE=E5=9B=9E=E9=80=80?=
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit

问题（WSL2 真机诊断报告问题 5）：新库上每轮召回都打
  `[graph] recall blocks: reading centroid failed, falling back to raw vectors:
   graph: load centroid: no such table: graph_centroid`
417KB 日志里占相当比例，并回退到全量 raw vectors 扫描（生产库 3178 块）。

根因不是"迁移脚本漏了"，而是**建表根本没在初始化里**：
graph_centroid 只定义在 centroid_store.go，且只由 SaveCentroid 内部的
ensureCentroidSchema 创建。新建的库在存过第一个中心向量之前**从不**
调 SaveCentroid ⇒ 建表语句一次没跑过；而召回走的是 LoadCentroid。

修法：把 ddlCentroid 加进 graph.go 的 schemas 列表（复用同一个常量，
避免两处 DDL 漂移）。

判据（centroid_store_test.go，2 条，均经变异验证）：
· TestGraphCentroidTableExistsOnNewDB —— 新建库后表必须已存在
· TestLoadCentroidOnNewDBDoesNotFail —— 空库 LoadCentroid 返回
  (nil,false,nil) 而不是 error（贴真实故障路径）
变异：删掉 schemas 里那行 ddlCentroid ⇒ 两条全红，报错文案与真机一致。
---
 internal/memory/centroid_store_test.go | 51 ++++++++++++++++++++++++++
 internal/memory/graph.go               | 10 +++++
 2 files changed, 61 insertions(+)

diff --git a/internal/memory/centroid_store_test.go b/internal/memory/centroid_store_test.go
index 1ec5316d..6a02a282 100644
--- a/internal/memory/centroid_store_test.go
+++ b/internal/memory/centroid_store_test.go
@@ -381,3 +381,54 @@ func putRawBlock(t *testing.T, g *GraphDB, fp, id string, v []float64) {
 		t.Fatal(err)
 	}
 }
+
+// ── 2026-10-25：graph_centroid 必须随库创建，不能等第一次 SaveCentroid ──
+//
+// 缺陷现场（WSL2 真机诊断报告问题 5）：新建库上每轮召回都打
+//
+//	`[graph] recall blocks: reading centroid failed, falling back to raw vectors:
+//	 graph: load centroid: no such table: graph_centroid`
+//
+// 417KB 日志里占相当比例，并回退到全量 raw vectors 扫描。
+//
+// 根因：buildSchemas 的 schemas 列表里漏了 graph_centroid，它只在
+// SaveCentroid 内部被 ensureCentroidSchema 创建——而新库在存过第一个
+// 中心向量之前从不调 SaveCentroid。
+//
+// 判据刻意**不先调 SaveCentroid**：只要一存就会建表，那样就盖住了缺陷
+// （变异验证：把 graph.go 里的 ddlCentroid 那行删掉，本判据必须变红）。
+func TestGraphCentroidTableExistsOnNewDB(t *testing.T) {
+	g := newTestGraph(t)
+	defer g.Close()
+
+	var name string
+	err := g.db.QueryRow(
+		`SELECT name FROM sqlite_master WHERE type='table' AND name='graph_centroid'`).Scan(&name)
+	if err != nil {
+		t.Fatalf("新建库应已有 graph_centroid 表，实际查询失败: %v", err)
+	}
+	if name != "graph_centroid" {
+		t.Fatalf("表名不符: %q", name)
+	}
+}
+
+// 判据二：新建库上直接 LoadCentroid 不得报错。
+//
+// 这条贴着**真实故障路径**（召回时调 LoadCentroid），比只查 sqlite_master
+// 更接近用户看到的现象：查表不存在会以 error 形式冒出来。
+func TestLoadCentroidOnNewDBDoesNotFail(t *testing.T) {
+	g := newTestGraph(t)
+	defer g.Close()
+
+	// 空库查询：应当"查不到"（nil, false）而不是"表不存在"（error）。
+	c, anomalous, err := g.LoadCentroid("any-fingerprint", 512)
+	if err != nil {
+		t.Fatalf("空库 LoadCentroid 不应报错（表必须已存在），实际: %v", err)
+	}
+	if c != nil {
+		t.Fatalf("空库不应返回中心向量，实际: %+v", c)
+	}
+	if anomalous {
+		t.Fatal("空库不应标记 anomalous")
+	}
+}
diff --git a/internal/memory/graph.go b/internal/memory/graph.go
index ce55b50b..c9231e05 100644
--- a/internal/memory/graph.go
+++ b/internal/memory/graph.go
@@ -408,6 +408,16 @@ func (g *GraphDB) initSchema() error {
 		`CREATE INDEX IF NOT EXISTS idx_relation_type ON relations(relation_type)`,
 		`CREATE INDEX IF NOT EXISTS idx_relation_status ON relations(status)`,
 		`CREATE INDEX IF NOT EXISTS idx_relation_session ON relations(session_id)`,
+		// ★ graph_centroid 必须在这里建（2026-10-25 修缺陷）。
+		//
+		// 此前它只由 SaveCentroid 内部的 ensureCentroidSchema 创建，
+		// 而新建的库在存过第一个中心向量之前**从不**调 SaveCentroid。
+		// 后果：LoadCentroid 每次召回都 `no such table: graph_centroid`，
+		// 于是回退到全量 raw vectors 扫描——日志刷屏，且大库上明显变慢
+		// （实测生产库 3178 块）。
+		//
+		// 复用 centroid_store.go 的同一个常量，避免两处 DDL 漂移。
+		ddlCentroid,
 	}
 
 	for _, s := range schemas {