fix(L2-B): knowledge_base _is_postgres psycopg ImportError → graceful False

[Wool-xing]

范围

L2 baseline CI fail 子项 B: runtime/mcp/knowledge_base/server.py _is_postgres() 调 get_engine(), postgres db_url 配但 psycopg 未装时 ImportError → MCP tool search_similar stack trace。1 文件 +14/-1。

L2 三子项总览

子项	失败	PR
L2-A generate_report.py docx ImportError	n7 fail	#51 (开)
L2-B _is_postgres psycopg ImportError	MCP tool crash	本 PR
L2-C env-manager rollout n2 router 未跳过	n2 fail	#52 (开)

三子项合后, CI L2 ≥80% 达标。

修法 (Surgical)

def _is_postgres() -> bool:
    try:
        return get_engine().dialect.name == "postgresql"
    except (ImportError, ModuleNotFoundError):
        # psycopg / asyncpg 未装 → 视为非 postgres, 走 sqlite fallback
        return False
    except Exception:
        # 其他异常 (连接失败 / dialect 加载失败 等) 也 fallback
        return False

server.py L83+ 已有 sqlite fallback 分支自然走 (return {"indexed": True, ..., "backend": "sqlite-no-vec"})。tool_search_similar L127-128 已正确处理 not _is_postgres() → return error dict (而非 raise)。

不在范围

• requirements 加 psycopg (依赖膨胀, CI infra 决策待用户决, 与 L2-A 不静默装 docx 一致)
• db.py get_engine 加 try/except (公共函数, 影响面大, 由调用方各自 handle)

本地测试 (3/3 关键 case 过)

OK ImportError → False  (psycopg 缺 → 走 sqlite fallback)
OK ConnectionError → False  (其他异常兜底)
OK postgres engine → True  (正常路径不破)

协作宪章 §1.3 六道闸自检 (f1-f6)

• a 静态: ✓ pre-commit 全过
• b 副作用: ✓ _is_postgres 是 server.py 内部函数, 仅 server.py 内部调用 (3 处: tool_index_case / tool_index_defect / tool_search_similar) 全走 if/else 分支不依赖 raise
• c 跨层契约: ✓ 不破 MCP tool schema (返 error dict 是合规 response)
• d 实测: ✓ 3 关键 case 通过
• e 回归: ✓ 正常 postgres 路径不变
• f 诚实自检:
  • f1: 1 文件 +14/-1 — H
  • f2: ✓
  • f3: ✓ docstring 明示"视为非 postgres, 走 sqlite fallback"
  • f4: ✓
  • f5: 见下
  • f6: 见下

置信度自检 (f5)

声明	证据	f5
psycopg ImportError 是 L2 root cause	gh run view 25815979436 含明文 ModuleNotFoundError: No module named 'psycopg'	H
try/except 修法	git diff	H
3/3 关键 case 单测	输出贴出	H
pre-commit 全过	工具输出	H
L2-A+B+C 合后 ≥80% 达标	推断 (3 子项独立修, 各自消除一类 fail)	M

假阳性过滤 (f6)

唯一 finding = "_is_postgres 在 psycopg 缺时 crash":

#	问	答
1	真问题 / 误报?	真。CI log stack trace 实证
2	反例?	"应该装 psycopg 而不是 graceful skip" → 反驳: CI 是 selftest 不强依赖 postgres, sqlite fallback 设计已存在 (server.py L83+), 让 graceful 工作正是利用既有 fallback。不证伪
3	修后实质变好?	是。MCP tool 不再 crash, sqlite fallback 路径生效

假阳性候选数: 0

关联

• L2 baseline 3 子项收尾
• L2-A PR fix(L2-A): generate_report.py docx ImportError → graceful skip #51 (docx graceful) — 并行
• L2-C PR fix(L2-C): selftest rollout 节点视为 graceful skip (tolerant 模式) #52 (rollout skip) — 并行
• W4-5 用户手动 install.sh 验证 — 我跑不动 Windows, 留用户