Code Review：fix(gemini): clear activeMsgIdRef on stop to prevent message filtering bug (#1354)

变更概述

此 PR 仅修改 GeminiSendBox.tsx 中的 resetState 回调，在停止按钮触发时额外清空 activeMsgIdRef.current（置为 null）。涉及模块为 Gemini 对话的消息流状态管理逻辑。

方案评估

结论：✅ 方案合理

activeMsgIdRef 被用于过滤"非当前请求"产生的事件（防止旧请求的 thought 消息干扰新请求）。在 Agent/CLI 多轮对话场景中，setActiveMsgId 仅在用户主动发消息时调用，后端自动续接的轮次不会更新该 ref。若停止后 ref 仍保留旧 msg_id，下一轮后端自动续接的 thought 事件（携带新 msg_id）会被误过滤，导致思考进度等 UI 无响应。清空为 null 后，过滤条件 activeMsgIdRef.current && ... 短路为 false，所有新事件畅通无阻直到下一次 setActiveMsgId 调用。方案简洁正确，与现有架构一致。

问题清单

🔵 LOW — resetState 未同步更新 waitingResponseRef.current

文件：src/renderer/pages/conversation/gemini/GeminiSendBox.tsx，第 385–395 行

问题代码：

const resetState = useCallback(() => {
  setWaitingResponse(false);        // ← 异步 state 更新
  setStreamRunning(false);
  streamRunningRef.current = false; // ← ref 同步更新 ✓
  setHasActiveTools(false);
  hasActiveToolsRef.current = false; // ← ref 同步更新 ✓
  setThought({ subject: '', description: '' });
  hasContentInTurnRef.current = false;
  activeMsgIdRef.current = null;    // ← 本 PR 新增，ref 同步更新 ✓
}, []);

问题说明：文件其他位置（如 finish 事件处理、初始化）均采用 setXxx(false) + xxxRef.current = false 双写模式，以确保 ref 在下一个事件到来前就是最新值。但 resetState 中 waitingResponseRef.current 没有同步置 false，仅靠 React 异步 setState，若在渲染周期之间有新事件到来，waitingResponseRef.current 读到的仍是旧值 true。本 PR 中新增的 activeMsgIdRef.current = null 处理方式反而是正确的，因此该不一致更为明显。

注意：这是本 PR 暴露出的已有缺陷，不是 PR 引入的新问题，不阻塞合并。

修复建议：

const resetState = useCallback(() => {
  setWaitingResponse(false);
  waitingResponseRef.current = false;   // add this
  setStreamRunning(false);
  streamRunningRef.current = false;
  setHasActiveTools(false);
  hasActiveToolsRef.current = false;
  setThought({ subject: '', description: '' });
  hasContentInTurnRef.current = false;
  activeMsgIdRef.current = null;
}, []);

🔵 LOW — 缺少对应的回归测试

文件：无测试文件变更

问题说明：PR 修复了一个具体的竞态 bug（停止后 activeMsgIdRef 残留导致 agent 多轮 thought 被过滤），但没有添加对应的单元测试或集成测试。若未来 resetState 被误修改，该 bug 会无声复现。

修复建议：在 tests/unit/ 或 tests/integration/ 下为 useGeminiMessage hook 添加测试，验证调用 resetState() 后 activeMsgIdRef.current 为 null，以及后续事件不会被过滤。

汇总

结论

⚠️ 有条件批准 — 核心修复逻辑正确，方案合理，变更极小且聚焦。两个 LOW 问题均不阻塞合并，建议后续在 resetState 中补齐 waitingResponseRef.current = false 并补充测试用例。

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