Code Review

Bug 1 — Claude SDK 双重错误消息

server/index.js 中 claude-command handler 的 .catch() 会发送 claude-error 给客户端。但 server/claude-sdk.js 的 queryClaudeSDK 内部已有 catch 块发送 claude-error 后 re-throw。客户端会收到两条错误消息。

修复建议：去掉 index.js 中 .catch() 里的 writer.send（只保留 console.error），或让 queryClaudeSDK 不 re-throw。

Bug 2 — sanitizePersistedGeminiContent 行为改变

server/gemini-cli.js 约第788行，sanitizePersistedGeminiContent 调用 stripInternalContextPrefix(content) 。旧的本地函数对空字符串返回 ''，新的共享函数（server/utils/sessionFormatting.js）默认 returnDefaultOnEmpty=true，对纯 context 前缀返回 'New Session'。

这意味着 Gemini 的聊天内容如果只有 [Context: ...] 前缀，会在响应中显示 "New Session"。

修复：调用处改为 stripInternalContextPrefix(content, false)

Bug 3 — 系统消息检测在 strip 之前执行

server/projects.js 的 parseJsonlSessions 中，isSystemMessage 检查（textContent.startsWith('<command-name>') 等）在 stripInternalContextPrefix 之前执行。如果消息以 [Context: ...] <command-name>... 开头，startsWith 返回 false，系统消息不会被过滤，会泄漏到 session name 中。

修复：先 strip 再检查 isSystemMessage，或在 isSystemMessage 检查中也处理 context 前缀的情况。

中等问题

• TOCTOU 竞争：isXSessionActive 检查和 session 注册（addSession）之间有多个 await，两条快速到达的消息可以同时通过检查
• stripInternalContextPrefix 重复 4 次：服务端共享 util + 客户端 3 份（messageTransforms.ts、MainContentTitle.tsx、sidebar/utils/utils.ts），且客户端版本缺少 session-mode 语义回退
• useChatComposerState 的 isExplicitNewSessionStart 只检查 routedSessionId（比旧的 isExplicitNewSession 更宽泛），当有 currentSessionId 但无路由 session 时可能过度清理 sessionStorage

Thanks @Zhang-Henry for your comprehensive review! I already fixed them with a new commit.