[0200] 修复插件中连续点击中断导致 crash 的问题 (#3362)

Author: GatsbyUSTC

devel/0200.md

@@ -0,0 +1,104 @@
+# [0200] 修复中断执行时主进程意外退出的问题
+
+## 相关文档
+- [dddd.md](dddd.md) - 任务文档模板
+
+## 任务相关的代码文件
+- `src/System/Link/pipe_link.cpp`
+- `src/Plugins/Qt/qt_pipe_link.cpp`
+
+## 如何测试
+
+### 确定性测试（单元测试）
+暂无单元测试。
+
+### 非确定性测试（文档验证）
+1. 启动 Mogan，打开一个 LLM 插件会话
+2. 输入一个需要较长时间执行的命令（如多行代码生成）
+3. 在命令执行期间点击 "Interrupt execution"（停止图标）
+4. 确认：
+   - 插件子进程被终止
+   - 主进程不会意外退出
+   - 可以正常继续使用 Mogan
+5. 重复多次中断操作，确认稳定性
+
+## 如何提交
+
+提交前执行以下最少步骤：
+
+```bash
+xmake b stem
+```
+
+确认编译通过后提交并创建 PR：
+
+```bash
+git add -A
+git commit -m "[0200] 修复中断执行时主进程意外退出的问题"
+git push origin hongwei/200_27/fix_interrupt
+git pr create
+```
+
+## What
+
+修复在插件会话中点击 "Interrupt execution" 时，主进程意外退出的问题。
+
+修改内容：
+1. `qt_pipe_link.cpp`：`interrupt()` 改为调用 `stop()`，不再发送 `SIGINT`
+2. `pipe_link.cpp`：`interrupt()` 改为调用 `stop()`，不再发送 `SIGINT`
+
+## Why
+
+### 根因分析
+
+日志显示第二次中断时 `target_pid=0`：
+
+```
+[DEBUG qt_pipe_link_rep::interrupt] target_pid=42922
+...
+[DEBUG qt_pipe_link_rep::interrupt] target_pid=0
+```
+
+问题出在这个调用链：
+
+1. 第一次 `interrupt()` 调用 `::kill(pid, SIGINT)` 发信号给子进程 `42922`
+2. 子进程收到 `SIGINT` 后退出
+3. 但 Qt 的 `QProcess` 在进程结束后 `pid()` / `processId()` 返回了 `0`
+4. `alive` 标志未被更新（`connection_rep::interrupt()` 只改 `status`，不改 `alive`）
+5. 第二次 `interrupt()` 调用时，`::kill(0, SIGINT)` 把信号发给了**当前进程组的所有进程**
+6. 主进程自己也收到了 `SIGINT`，触发 `clean_exit_on_signal` 退出
+
+### POSIX 语义
+
+`::kill(0, SIGINT)` 的语义是：把信号发送给**发送者所在进程组的所有进程**。当 `target_pid=0` 时，主进程会自杀。
+
+## How
+
+将两个 `interrupt()` 的实现从发送 `SIGINT` 改为调用 `stop()`：
+
+```cpp
+// src/Plugins/Qt/qt_pipe_link.cpp
+void qt_pipe_link_rep::interrupt () {
+  stop ();
+}
+
+// src/System/Link/pipe_link.cpp
+void pipe_link_rep::interrupt () {
+  stop ();
+}
+```
+
+`stop()` 的行为：
+- `qt_pipe_link_rep::stop()`：`PipeLink.killProcess(0)` 终止 Qt 子进程，`alive = false`
+- `pipe_link_rep::stop()`：发送 `SIGTERM`/`SIGKILL` 终止子进程，关闭管道，`alive = false`
+
+这样避免了 `kill(pid=0)` 导致的主进程自杀风险，同时正确更新了 `alive` 状态。
+
+## 影响范围确认
+
+经全项目检索，`interrupt()` 的调用方**仅有** `src/System/Link/connection.cpp` 内部的两处：
+
+1. `connection_rep::interrupt()` 中调用 `ln->interrupt()`
+2. `connection_interrupt()` 中调用 `con->interrupt()`
+
+项目中不存在任何其他直接或间接调用 `tm_link_rep::interrupt()`、`qt_pipe_link_rep::interrupt()` 或 `pipe_link_rep::interrupt()` 的代码。因此本次修改只影响通过 `connection-interrupt` 发起的插件中断流程，不会波及其他功能。

src/Plugins/Qt/qt_pipe_link.cpp

@@ -26,6 +26,7 @@
 #if defined(OS_MINGW) || defined(OS_WIN)
 #else
 #include <sys/wait.h>
+#include <unistd.h>
 #endif
 #include <errno.h>
 
@@ -142,32 +143,7 @@ qt_pipe_link_rep::is_readable (int channel) {
 
 void
 qt_pipe_link_rep::interrupt () {
-  if (!alive) return;
-#if defined(OS_MINGW) || defined(OS_WIN)
-  // Not implemented
-  qt_error << "SIGINT not implemented on Windows\n";
-#else
-#if QT_VERSION < 0x060000
-  Q_PID pid= PipeLink.pid ();
-
-  // REMARK: previously there were here below a call to ::killpg which does not
-  // seems to work on MacOS I (mgubi) replaced it with ::kill which does the
-  // job. But I do not undestand the difference.
-
-  int ret= ::kill (pid, SIGINT);
-  if (ret == -1) {
-    qt_error << "Interrupt not successful, pid: " << pid
-             << " return code: " << errno << "\n";
-  }
-#else
-  uint64_t pid= PipeLink.processId ();
-  int      ret= ::kill (pid, SIGINT);
-  if (ret == -1) {
-    qt_error << "Interrupt not successful, pid: " << pid
-             << " return code: " << errno << "\n";
-  }
-#endif
-#endif
+  stop ();
 }
 
 void

src/System/Link/pipe_link.cpp

@@ -313,10 +313,7 @@ pipe_link_rep::listen (int msecs) {
 
 void
 pipe_link_rep::interrupt () {
-#if !defined(OS_MINGW) && !defined(OS_WIN)
-  if (!alive) return;
-  killpg (pid, SIGINT);
-#endif
+  stop ();
 }
 
 void