cursor-mcp-feedback 2.0.4 → 2.0.6

package/README.md

@@ -138,7 +138,7 @@ On first startup (or `npm install -g`), the MCP server **automatically configure
 
 - **Cursor rule** (`~/.cursor/rules/cursor-mcp-feedback.mdc`) — instructs the agent to call `interactive_feedback`
 - **Cursor hooks** (`~/.cursor/hooks/`) — subagent protection + pending message delivery + event logging
-- **hooks.json entries** — registers `sessionStart`, `subagentStart`, `subagentStop`, `beforeMCPExecution`, `preToolUse`, `afterMCPExecution` hooks
+- **hooks.json entries** — registers `subagentStart`, `subagentStop`, `beforeMCPExecution`, `afterMCPExecution`, `preToolUse` hooks
 
 All auto-installed files are kept in sync on every startup (hash-based diffing, idempotent).
 
@@ -182,11 +182,8 @@ Add to `~/Library/Application Support/Claude/claude_desktop_config.json` (macOS)
 
 | Hook | Event | Purpose |
 |------|-------|---------|
-| `block-cursor-mcp-feedback.js` | `sessionStart` | Create per-session directory & metadata |
-| `block-cursor-mcp-feedback.js` | `sessionEnd` | Mark session inactive, reset subagent counter |
 | `block-cursor-mcp-feedback.js` | `subagentStart` | Increment active subagent counter for parent conversation |
 | `block-cursor-mcp-feedback.js` | `subagentStop` | Decrement active subagent counter |
-| `block-cursor-mcp-feedback.js` | `preToolUse` | Deny feedback tool calls when subagents are active (matcher: `MCP:interactive_feedback\|MCP:submit_feedback`) |
 | `block-cursor-mcp-feedback.js` | `beforeMCPExecution` | Deny cursor-mcp-feedback MCP calls when subagents are active |
 | `block-cursor-mcp-feedback.js` | `afterMCPExecution` | Log feedback_request/response events to events.jsonl |
 | `consume-pending.js` | `preToolUse` | Consume pending messages and inject as agent feedback |

package/dist/main.js

@@ -99,16 +99,13 @@ function installCursorHooks() {
         // Each entry has a unique _source tag for idempotent upsert.
         // Multiple entries per event are supported via distinct _source values.
         const entries = [
-            { event: "sessionStart", hook: { command: `${node} ${blockHook} sessionStart`, _source: "cursor-mcp-feedback" } },
-            { event: "sessionEnd", hook: { command: `${node} ${blockHook} sessionEnd`, _source: "cursor-mcp-feedback" } },
             { event: "subagentStart", hook: { command: `${node} ${blockHook} subagentStart`, _source: "cursor-mcp-feedback" } },
             { event: "subagentStop", hook: { command: `${node} ${blockHook} subagentStop`, _source: "cursor-mcp-feedback" } },
             { event: "beforeMCPExecution", hook: { command: `${node} ${blockHook} beforeMCPExecution`, failClosed: true, _source: "cursor-mcp-feedback" } },
             { event: "afterMCPExecution", hook: { command: `${node} ${blockHook} afterMCPExecution`, _source: "cursor-mcp-feedback" } },
             { event: "preToolUse", hook: { command: `${node} ${pendingHook}`, _source: "cursor-mcp-feedback" } },
-            { event: "preToolUse", hook: { command: `${node} ${blockHook} preToolUse`, matcher: "MCP:interactive_feedback|MCP:submit_feedback", failClosed: true, _source: "cursor-mcp-feedback-block" } },
         ];
-        const allSources = new Set(entries.map((e) => e.hook._source));
+        const allSources = new Set([...entries.map((e) => e.hook._source), "cursor-mcp-feedback-block"]);
         const before = JSON.stringify(config);
         // First pass: remove ALL our entries from every event (clean slate)
         for (const arr of Object.values(config.hooks)) {