Code Review：fix(channels): use markdownToTelegramHtml for Telegram text formatting (#1978)

变更概述

本 PR 名义上修复 Telegram 消息格式化问题（formatTextForPlatform 增加 telegram 分支、修正 markdownToTelegramHtml 正则顺序），但实际包含大量不相关改动（39 个文件）：新增 thinking 消息系统（含 UI 组件、IPC 消息类型、DB 持久化）、CodeBlock 折叠/展开优化、MarkdownView 组件 memo 化、auto-scroll 改进、工具摘要增强、adapter 序列化守卫、ElectronPlatformServices 日志路径降级等。

方案评估

结论：⚠️ 方案有缺陷

Telegram 格式化修复本身正确且合理——增加 telegram 分支调用 markdownToTelegramHtml，以及将 code block 正则移到 inline code 之前都是正确的修改。但本 PR 同时移除了 filterThinkTagsFromMessage（从 content 消息中剥离 <think> 标签的逻辑），改为通过 thought 事件转换为 thinking 消息。如果有模型直接在 content 消息中嵌入 <think> 标签（如 MiniMax 等），这些标签将泄漏到 UI。新增的 extractAndStripThinkTags 工具函数似乎是为此场景准备的，但本 PR 并未使用。

问题清单

🟡 MEDIUM — 移除 think tag 过滤可能导致 <think> 标签泄漏到 UI

文件：src/process/task/AcpAgentManager.ts，第 447-448 行

问题代码：

const emitStart = Date.now();
ipcBridge.acpConversation.responseStream.emit(message as IResponseMessage);

问题说明：旧代码通过 filterThinkTagsFromMessage 在发送到 UI 前从 content 类型消息中剥离 <think>...</think> 标签。该方法已被移除，现在 content 消息直接发送。对于在响应中内嵌 <think> 标签的模型（如 MiniMax），标签内容会原样显示在聊天界面中。ThinkTagDetector.ts 中新增了 extractAndStripThinkTags 函数但本 PR 未使用，疑似遗漏。

修复建议：在 emit 前对 content 类型消息应用 extractAndStripThinkTags，将 thinking 部分路由到 thinking 消息，stripped 内容继续作为 content 发送：

if (message.type === 'content' && typeof message.data === 'string') {
  const { thinking, content: stripped } = extractAndStripThinkTags(message.data);
  if (thinking) {
    this.emitThinkingMessage(thinking, 'thinking');
  }
  if (stripped !== message.data) {
    message = { ...message, data: stripped };
  }
}
ipcBridge.acpConversation.responseStream.emit(message as IResponseMessage);

🟡 MEDIUM — 残留的 JSDoc 片段

文件：src/process/task/AcpAgentManager.ts，第 815-817 行

问题代码：

   * @param message - The streaming message to filter
   * @returns Message with think tags removed from content
   */
  /**
   * Emit a thinking message to the UI stream.

问题说明：旧方法 filterThinkTagsFromMessage 的 JSDoc 尾部（@param message、@returns）残留在新方法 emitThinkingMessage 的 JSDoc 前面。应清理。

修复建议：删除 815-817 行的残留注释。

🔵 LOW — thought 状态在 useAcpMessage 中已不再被消费

文件：src/renderer/pages/conversation/platforms/acp/useAcpMessage.ts

问题说明：thought 和 setThought 状态仍被声明和维护（在 finish、resetState 中调用 setThought），thought 也仍在返回值中，但 AcpSendBox.tsx 不再解构使用它。ThoughtDisplay 也不再接收 thought prop。这是死代码。

修复建议：移除 thought、setThought、throttledSetThought 相关代码，同时从 UseAcpMessageReturn 类型中移除 thought 字段。

🔵 LOW — extractAndStripThinkTags 新增但未使用

文件：src/process/task/ThinkTagDetector.ts，第 66-108 行

问题说明：新增了 extractAndStripThinkTags 函数并编写了测试，但在整个 PR 的变更文件中没有被 import 或调用。如果是为上述 MEDIUM 问题准备的修复，应在本 PR 中集成使用。

汇总

结论

⚠️ 有条件批准 — 存在 2 个 MEDIUM 问题：think tag 过滤移除可能导致部分模型的内部推理内容泄漏到 UI（功能回退），以及残留的 JSDoc 注释。修复后可合并。

本报告由本地 pr-review skill 生成，包含完整项目上下文，无截断限制。

CONCLUSION: CONDITIONAL
IS_CRITICAL_PATH: false
PR_NUMBER: 1978

PR Fix 验证报告

原始 PR: #1978
修复方式: 直接推送到 fix/issue-1214

总结： ✅ 已修复 2 个 | ❌ 未能修复 0 个

🔵 LOW 级别问题已跳过（不阻塞合并，修复优先级低）。