cursor-mcp-feedback 2.0.4 → 2.0.6

package/hooks/block-cursor-mcp-feedback.js

@@ -1,12 +1,11 @@
 #!/usr/bin/env node
-// Cursor hook: session lifecycle + subagent protection.
+// Cursor hook: subagent protection + feedback event logging.
 //
 // Events (argv[2]):
-//   sessionStart         — create per-session directory & meta
-//   sessionEnd           — mark session inactive
-//   subagentStart        — record subagent to negative list
-//   subagentStop         — remove subagent
-//   beforeMCPExecution   — deny cursor-mcp-feedback calls from subagents
+//   subagentStart        — increment active subagent counter
+//   subagentStop         — decrement active subagent counter
+//   beforeMCPExecution   — deny cursor-mcp-feedback calls when subagents active
+//   afterMCPExecution    — log feedback events to events.jsonl
 
 'use strict';
 
@@ -20,42 +19,11 @@ process.stdin.on('end', () => {
   let input;
   try { input = JSON.parse(raw); } catch { input = {}; }
 
-  if (event === 'sessionStart') {
-    su.cleanupStaleSessions();
-    const convId = input.session_id || input.conversation_id;
-    if (convId) {
-      const workspace = (input.workspace_roots || [])[0] || '';
-      su.createSession(convId, {
-        workspace,
-        mode: input.composer_mode || 'agent',
-        model: input.model || '',
-        is_background: input.is_background_agent || false,
-        transcript_path: input.transcript_path || null,
-      });
-    }
-    su.writeHookOutput({});
-    return;
-  }
-
-  if (event === 'sessionEnd') {
-    const convId = input.session_id || input.conversation_id;
-    if (convId) {
-      su.endSession(convId, input.reason || 'unknown');
-      su.resetActiveSubagents(convId);
-    }
-    su.writeHookOutput({});
-    return;
-  }
-
   if (event === 'subagentStart') {
     const subagentId = input.subagent_id;
     const parentConvId = input.parent_conversation_id || input.conversation_id;
-    if (subagentId) {
-      su.recordSubagent(subagentId, parentConvId);
-    }
-    if (parentConvId) {
-      su.incrementActiveSubagents(parentConvId);
-    }
+    if (subagentId) su.recordSubagent(subagentId, parentConvId);
+    if (parentConvId) su.incrementActiveSubagents(parentConvId);
     su.writeHookOutput({});
     return;
   }
@@ -64,41 +32,11 @@ process.stdin.on('end', () => {
     const subagentId = input.subagent_id || input.conversation_id;
     const parentConvId = input.parent_conversation_id || input.conversation_id;
     if (subagentId) su.removeSubagent(subagentId);
-    if (parentConvId) {
-      su.decrementActiveSubagents(parentConvId);
-    }
+    if (parentConvId) su.decrementActiveSubagents(parentConvId);
     su.writeHookOutput({});
     return;
   }
 
-  if (event === 'preToolUse') {
-    const toolName = (input.tool_name || '').toLowerCase();
-    const isFeedbackTool =
-      toolName.includes('interactive_feedback') ||
-      toolName.includes('submit_feedback');
-
-    if (!isFeedbackTool) {
-      su.writeHookOutput({});
-      return;
-    }
-
-    const convId = input.conversation_id;
-    if (convId && su.hasActiveSubagents(convId)) {
-      su.writeHookOutput({
-        permission: 'deny',
-        user_message: '[Hook] Blocked feedback tool call — subagent(s) active',
-        agent_message:
-          'DENIED by preToolUse hook. ' +
-          'cursor-mcp-feedback tools are for the MAIN agent only. ' +
-          'Active subagents detected for this conversation. ' +
-          'Return your results as text in your final response instead.',
-      });
-    } else {
-      su.writeHookOutput({});
-    }
-    return;
-  }
-
   if (event === 'beforeMCPExecution') {
     const command = (input.command || '').toLowerCase();
     const isFeedbackApp =
@@ -114,7 +52,7 @@ process.stdin.on('end', () => {
     if (convId && su.hasActiveSubagents(convId)) {
       su.writeHookOutput({
         permission: 'deny',
-        user_message: '[Hook] Blocked cursor-mcp-feedback call — subagent(s) active',
+        user_message: '[Hook] Blocked cursor-mcp-feedback — subagent(s) active',
         agent_message:
           'DENIED by beforeMCPExecution hook. ' +
           'cursor-mcp-feedback tools are for the MAIN agent only. ' +
@@ -137,6 +75,11 @@ process.stdin.on('end', () => {
 
     const isFeedback = toolName === 'interactive_feedback' || toolName === 'submit_feedback';
     if (convId && isFeedback) {
+      su.ensureSession(convId, {
+        workspace: (input.workspace_roots || [])[0] || '',
+        model: input.model || '',
+      });
+
       const eventsFile = require('path').join(su.sessionDir(convId), 'events.jsonl');
       let toolInput = {};
       try { toolInput = typeof input.tool_input === 'string' ? JSON.parse(input.tool_input) : (input.tool_input || {}); } catch {}

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "cursor-mcp-feedback",
-  "version": "2.0.4",
+  "version": "2.0.6",
   "description": "MCP App for interactive feedback — renders a chat UI directly inside MCP hosts (Claude, Cursor, etc.)",
   "type": "module",
   "main": "dist/main.js",