cursor-feedback 2.1.0 → 2.1.1

package/CHANGELOG.md

@@ -2,6 +2,13 @@
 
 All notable changes to this project will be documented in this file. See [standard-version](https://github.com/conventional-changelog/standard-version) for commit guidelines.
 
+### [2.1.1](https://github.com/jianger666/cursor-feedback-extension/compare/v2.1.0...v2.1.1) (2026-07-02)
+
+
+### Bug Fixes
+
+* **mcp:** 等待反馈期间不再被僵尸判定误杀——Connection closed 断连根因修复 ([39cc9d5](https://github.com/jianger666/cursor-feedback-extension/commit/39cc9d5fe5730023069014c8a749841993117efa))
+
 ## [2.1.0](https://github.com/jianger666/cursor-feedback-extension/compare/v2.0.5...v2.1.0) (2026-07-02)
 
 

package/dist/extension.js

@@ -46,6 +46,12 @@ let feedbackViewProvider = null;
 function normalizePath(p) {
     return (p || '').replace(/\\/g, '/').replace(/\/+$/, '').toLowerCase();
 }
+/** 两个已归一化的路径互为前缀（相等 / 一方是另一方的子目录）即视为同一窗口语境 */
+function pathsRelated(a, b) {
+    if (!a || !b)
+        return false;
+    return a === b || a.startsWith(b + '/') || b.startsWith(a + '/');
+}
 function activate(context) {
     // 注册侧边栏 WebView（端口从 61927 开始自动扫描）
     feedbackViewProvider = new FeedbackViewProvider(context.extensionUri, 61927, context.globalState);
@@ -109,8 +115,9 @@ function getWorkspacePaths() {
     return folders.map(f => f.uri.fsPath);
 }
 /**
- * 检查路径是否匹配当前工作区（精确匹配）
- * - 有工作区的窗口：只接收匹配工作区路径的消息
+ * 检查路径是否匹配当前工作区
+ * - 有工作区的窗口：接收匹配工作区路径的消息（相等或互为子目录——AI 传的
+ *   project_directory 常是工作区的子目录，精确匹配会漏收，面板收不到、心跳也失配）
  * - 没有工作区的窗口：只接收没有指定项目路径的消息
  */
 function isPathInWorkspace(targetPath) {
@@ -128,9 +135,7 @@ function isPathInWorkspace(targetPath) {
         return false;
     }
     for (const wsPath of workspacePaths) {
-        const normalizedWs = normalizePath(wsPath);
-        // 精确匹配：只匹配完全相同的路径
-        if (normalizedTarget === normalizedWs) {
+        if (pathsRelated(normalizedTarget, normalizePath(wsPath))) {
             return true;
         }
     }
@@ -331,10 +336,10 @@ class FeedbackViewProvider {
             // 如果有活跃端口，先尝试只轮询该端口
             if (this._activePort) {
                 const result = await this._checkPortForRequest(this._activePort);
-                // 检查是否仍然是我们的 Server
+                // 检查是否仍然是我们的 Server（owner 可能是本工作区的子目录：AI 传的 project_directory）
                 if (result.connected) {
                     const serverOwner = result.ownerWorkspace ? normalizePath(result.ownerWorkspace) : '';
-                    const isMyServer = !serverOwner || serverOwner === normalizedCurrentWorkspace;
+                    const isMyServer = !serverOwner || pathsRelated(serverOwner, normalizedCurrentWorkspace);
                     if (isMyServer) {
                         // 端口仍然有效，保持使用
                         this._debugInfo.connectedPorts = [this._activePort];
@@ -376,7 +381,7 @@ class FeedbackViewProvider {
                     return false;
                 }
                 const serverOwner = r.ownerWorkspace ? normalizePath(r.ownerWorkspace) : '';
-                return !serverOwner || serverOwner === normalizedCurrentWorkspace;
+                return !serverOwner || pathsRelated(serverOwner, normalizedCurrentWorkspace);
             }).sort((a, b) => b.request.timestamp - a.request.timestamp);
             // 处理最新的请求
             if (myRequests.length > 0) {

package/dist/mcp-server.js

@@ -108,6 +108,10 @@ class McpFeedbackServer {
         // 所属工作区（只在 AI 调用 feedback 时设置）
         // 只有来自同一工作区的轮询才会更新活动时间
         this.ownerWorkspace = null;
+        // owner 身份是否已被真实窗口的轮询验证过。验证后 ownerWorkspace 不再被后续调用改写：
+        // AI 传的 project_directory 是「正在操作的项目」，不一定等于窗口工作区（同窗口操作兄弟/子目录很常见），
+        // 每次改写会让插件心跳失配 → 实例被防线 3 当僵尸误杀（等待反馈中直接 Connection closed）。
+        this.ownerConfirmed = false;
         // Server 启动时间
         this.startTime = Date.now();
         // 最近一次活动时间（任意 HTTP 轮询 / MCP 调用都会刷新）
@@ -122,6 +126,9 @@ class McpFeedbackServer {
         this.everOwnerPolled = false;
         // 看门狗定时器（兜底退出，防止进程残留 / CPU 占满）
         this.watchdogTimer = null;
+        // stop() 防重入：server.close() 会触发 transport.onclose → 回调里又调 stop()，
+        // 无标志位会无限递归直至 "Maximum call stack size exceeded"（日志曾刷出数万条 Stopping server...）
+        this.stopping = false;
         this.port = port;
         this.basePort = port;
         this.server = new index_js_1.Server({
@@ -255,9 +262,18 @@ class McpFeedbackServer {
         // 不清理会让 pendingCount 虚高、全局视角误判「多个窗口在等」
         //（即你看到的两个一模一样的 cursor-feedback-extension）。
         this.cancelStalePending(projectDir);
-        // AI 调用 feedback 时设置 ownerWorkspace（这是唯一正确的时机）
-        this.ownerWorkspace = this.normalizePath(projectDir);
-        debugLog(`Owner workspace set to: ${this.ownerWorkspace}`);
+        // AI 调用 feedback 时设置 ownerWorkspace；但 owner 一旦被真实窗口的轮询验证过（ownerConfirmed）
+        // 就锁定不再改写——project_directory 只代表 AI 正在操作的项目，改写已验证的窗口身份
+        // 会让心跳失配、实例被防线 3 误杀。
+        const normalizedProjectDir = this.normalizePath(projectDir);
+        if (!this.ownerConfirmed) {
+            this.ownerWorkspace = normalizedProjectDir;
+            debugLog(`Owner workspace set to: ${this.ownerWorkspace}`);
+        }
+        else if (normalizedProjectDir !== this.ownerWorkspace) {
+            debugLog(`Owner workspace kept as ${this.ownerWorkspace} (confirmed by window polling); ` +
+                `request project ${normalizedProjectDir} differs and does not rewrite owner`);
+        }
         // 创建反馈请求
         this.currentRequest = {
             id: requestId,
@@ -459,6 +475,16 @@ class McpFeedbackServer {
     normalizePath(p) {
         return (p || '').replace(/\\/g, '/').replace(/\/+$/, '').toLowerCase();
     }
+    /**
+     * 两个已归一化的路径是否属于同一窗口语境：相等，或一方是另一方的子目录。
+     * 用于 owner 心跳匹配——AI 传的 project_directory 常是窗口工作区的子目录（monorepo 子包等），
+     * 精确相等会漏判、心跳失配导致实例被防线 3 误杀。
+     */
+    pathsRelated(a, b) {
+        if (!a || !b)
+            return false;
+        return a === b || a.startsWith(b + '/') || b.startsWith(a + '/');
+    }
     /** 飞书回复命中某 requestId → resolve 该 pending，并回执用户 */
     submitFromFeishu(requestId, text, chatId, images = [], attachedFiles = [], ackMessageId) {
         const pending = this.pendingRequests.get(requestId);
@@ -942,10 +968,13 @@ class McpFeedbackServer {
                             // 「我的窗口」心跳：仅当轮询带的 workspace 匹配本实例归属时刷新。
                             // 别的活跃窗口对全端口的扫描虽刷 lastActivityTime，但 workspace 不匹配、不刷此字段，
                             // 故已关窗口的残留 server 不会被别人续命，可被 watchdog / 僵尸自检识别。
+                            // 匹配放宽为「路径互为前缀」：AI 传的 project_directory 可能是窗口工作区的子目录。
+                            // 命中即确认 owner 身份（ownerConfirmed），此后 owner 不再被后续调用改写。
                             const ws = this.normalizePath(u.searchParams.get('workspace') || '');
-                            if (ws && ws === this.ownerWorkspace) {
+                            if (ws && this.ownerWorkspace && this.pathsRelated(ws, this.ownerWorkspace)) {
                                 this.lastOwnerPollTime = Date.now();
                                 this.everOwnerPolled = true;
+                                this.ownerConfirmed = true;
                             }
                         }
                         catch {
@@ -1352,8 +1381,13 @@ class McpFeedbackServer {
             }
             // 防线 3：本实例曾被自己窗口插件轮询，但久未再收到（只剩别的活跃窗口在扫端口刷 lastActivityTime）
             // → 我的窗口已关、本实例是僵尸 → 退出，避免持续被全局 pending 计数误算成「另一个在等的窗口」。
+            // ⚠️ 与防线 2 同款约束：还有 pending 反馈在等用户时绝不退——曾因缺这一条，AI 在等待反馈期间
+            //    传了与窗口工作区不同的 project_directory（改写 owner 后心跳失配），30 秒后实例被当僵尸杀掉，
+            //    正在等待的 interactive_feedback 直接报 "MCP error -32000: Connection closed"。
             const ownerIdle = Date.now() - this.lastOwnerPollTime;
-            if (this.everOwnerPolled && ownerIdle > IDLE_TIMEOUT) {
+            if (this.everOwnerPolled &&
+                this.pendingRequests.size === 0 &&
+                ownerIdle > IDLE_TIMEOUT) {
                 debugLog(`Owner window idle for ${ownerIdle}ms, stale instance, exiting...`);
                 this.stop();
                 process.exit(0);
@@ -1363,9 +1397,13 @@ class McpFeedbackServer {
         this.watchdogTimer.unref();
     }
     /**
-     * 停止服务器
+     * 停止服务器（幂等：重复调用 / server.close() 触发 onclose 再次进入时直接返回）
      */
     stop() {
+        if (this.stopping) {
+            return;
+        }
+        this.stopping = true;
         debugLog('Stopping server...');
         // 关闭看门狗定时器
         if (this.watchdogTimer) {

package/package.json

@@ -2,7 +2,7 @@
   "name": "cursor-feedback",
   "displayName": "Cursor Feedback",
   "description": "One Cursor conversation, unlimited AI interactions - Save your monthly request quota! Interactive feedback loop for AI chat via MCP",
-  "version": "2.1.0",
+  "version": "2.1.1",
   "icon": "icon.png",
   "author": "jianger666",
   "license": "MIT",