slyplan-mcp 1.7.0 → 1.7.2

package/dist/cli.js

@@ -11,7 +11,7 @@ const SUPABASE_URL = 'https://omfzpkwtuzucwwxmyuqt.supabase.co';
 const SUPABASE_ANON_KEY = 'eyJhbGciOiJIUzI1NiIsInR5cCI6IkpXVCJ9.eyJpc3MiOiJzdXBhYmFzZSIsInJlZiI6Im9tZnpwa3d0dXp1Y3d3eG15dXF0Iiwicm9sZSI6ImFub24iLCJpYXQiOjE3NzA5MjMwNDIsImV4cCI6MjA4NjQ5OTA0Mn0.KXGoUez7M45RtFM9qR7mjzGX6UhlaRE-gggAJxSkIHY';
 const SLYPLAN_ORIGIN = 'https://slyplan.com';
 // Smart transcript-aware hook: checks if set_project + add_to_work_mode have been called.
-// Silent when OK, reminds only when needed. Also shows update notification.
+// Optimized: uses string search instead of parsing every JSON line.
 const PRE_HOOK_FILE_CONTENT = `#!/usr/bin/env node
 const fs = require('fs');
 const path = require('path');
@@ -25,63 +25,41 @@ process.stdin.on('end', () => {
     const data = JSON.parse(input);
 
     if (!data.transcript_path) {
-      const output = JSON.stringify({
-        suppressOutput: true,
+      process.stdout.write(JSON.stringify({
         hookSpecificOutput: {
           hookEventName: "PreToolCall",
           additionalContext: "BLOCKED: Call list_projects + set_project before doing any work. This is not optional."
         }
-      });
-      process.stdout.write(output);
+      }));
       process.exit(0);
     }
 
     const transcriptPath = data.transcript_path.replace(/^~/, process.env.HOME || process.env.USERPROFILE || '');
     if (!fs.existsSync(transcriptPath)) { process.exit(0); }
 
+    // Fast: read file as string and use indexOf instead of parsing every JSON line
     const content = fs.readFileSync(transcriptPath, 'utf8');
-    const lines = content.split('\\n').filter(Boolean);
 
-    let hasSetProject = false;
-    let hasWorkModeNode = false;
-
-    for (const line of lines) {
-      try {
-        const entry = JSON.parse(line);
-        if (entry.type === 'assistant' && entry.message && Array.isArray(entry.message.content)) {
-          for (const block of entry.message.content) {
-            if (block.type !== 'tool_use') continue;
-            const name = block.name || '';
-            if (name.includes('set_project')) hasSetProject = true;
-            // Track add/remove \\u2014 last action wins
-            if (name.includes('add_to_work_mode')) hasWorkModeNode = true;
-            if (name.includes('remove_from_work_mode')) hasWorkModeNode = false;
-          }
-        }
-      } catch {}
-    }
-
-    if (!hasSetProject) {
-      const output = JSON.stringify({
-        suppressOutput: true,
+    if (!content.includes('set_project')) {
+      process.stdout.write(JSON.stringify({
         hookSpecificOutput: {
           hookEventName: "PreToolCall",
           additionalContext: "BLOCKED: Call list_projects + set_project before doing any work. This is not optional."
         }
-      });
-      process.stdout.write(output);
+      }));
       process.exit(0);
     }
 
-    if (!hasWorkModeNode) {
-      const output = JSON.stringify({
-        suppressOutput: true,
+    // For work mode, we need to check the LAST occurrence wins (add vs remove)
+    const lastAdd = content.lastIndexOf('add_to_work_mode');
+    const lastRemove = content.lastIndexOf('remove_from_work_mode');
+    if (lastAdd < 0 || (lastRemove > lastAdd)) {
+      process.stdout.write(JSON.stringify({
         hookSpecificOutput: {
           hookEventName: "PreToolCall",
           additionalContext: "BLOCKED: No node in work mode. Call search + add_to_work_mode before doing any work. This is not optional."
         }
-      });
-      process.stdout.write(output);
+      }));
       process.exit(0);
     }
 
@@ -91,14 +69,12 @@ process.stdin.on('end', () => {
       if (fs.existsSync(cacheFile)) {
         const cache = JSON.parse(fs.readFileSync(cacheFile, 'utf8'));
         if (cache.update_available) {
-          const output = JSON.stringify({
-            suppressOutput: true,
+          process.stdout.write(JSON.stringify({
             hookSpecificOutput: {
               hookEventName: "PreToolCall",
-              additionalContext: "\\u26A0\\uFE0F SlyPlan update available: " + cache.installed + " \\u2192 " + cache.latest + ". Run /sly:update to update."
+              additionalContext: "SlyPlan update available: " + cache.installed + " -> " + cache.latest + ". Run /sly:update to update."
             }
-          });
-          process.stdout.write(output);
+          }));
           process.exit(0);
         }
       }
@@ -111,11 +87,12 @@ process.stdin.on('end', () => {
 });
 `;
 // Smart transcript-aware hook: counts file changes SINCE last SlyPlan sync.
-// Silent when synced. Fires after 3+ unsynced file changes or a git commit.
+// Optimized: reads only the tail of the transcript to find recent activity.
 const POST_HOOK_FILE_CONTENT = `#!/usr/bin/env node
 const fs = require('fs');
 
 const BATCH_THRESHOLD = 3;
+const TAIL_BYTES = 32768; // Read last 32KB — enough for recent tool calls
 
 let input = '';
 process.stdin.setEncoding('utf8');
@@ -129,80 +106,74 @@ process.stdin.on('end', () => {
     const transcriptPath = data.transcript_path.replace(/^~/, process.env.HOME || process.env.USERPROFILE || '');
     if (!fs.existsSync(transcriptPath)) { process.exit(0); }
 
-    const content = fs.readFileSync(transcriptPath, 'utf8');
-    const lines = content.split('\\n').filter(Boolean);
+    // Read only the tail of the file for performance
+    const stat = fs.statSync(transcriptPath);
+    const fd = fs.openSync(transcriptPath, 'r');
+    const readStart = Math.max(0, stat.size - TAIL_BYTES);
+    const buf = Buffer.alloc(Math.min(stat.size, TAIL_BYTES));
+    fs.readSync(fd, buf, 0, buf.length, readStart);
+    fs.closeSync(fd);
+    const tail = buf.toString('utf8');
 
-    const FILE_TOOLS = ['Write', 'Edit', 'NotebookEdit'];
-    const SLYPLAN_SYNC_TOOLS = ['update_node', 'add_node', 'add_to_work_mode'];
+    // If we're reading a partial file, skip the first (possibly incomplete) line
+    const lines = tail.split('\\n').filter(Boolean);
+    if (readStart > 0 && lines.length > 0) lines.shift();
 
-    let lastSlyplanSyncIdx = -1;
-    let fileChangesSinceSync = 0;
-    let hasGitCommitSinceSync = false;
+    let lastSyncIdx = -1;
+    let fileChanges = 0;
+    let hasCommit = false;
 
     for (let i = 0; i < lines.length; i++) {
+      // Fast string checks before expensive JSON.parse
+      const line = lines[i];
+      if (!line.includes('tool_use')) continue;
+
       try {
-        const entry = JSON.parse(lines[i]);
-        if (entry.type === 'assistant' && entry.message && Array.isArray(entry.message.content)) {
-          for (const block of entry.message.content) {
-            if (block.type !== 'tool_use') continue;
-            const name = block.name || '';
-
-            // Track SlyPlan syncs — resets the counter
-            for (const st of SLYPLAN_SYNC_TOOLS) {
-              if (name.includes(st)) {
-                lastSlyplanSyncIdx = i;
-                fileChangesSinceSync = 0;
-                hasGitCommitSinceSync = false;
-              }
-            }
+        const entry = JSON.parse(line);
+        if (entry.type !== 'assistant' || !entry.message || !Array.isArray(entry.message.content)) continue;
+        for (const block of entry.message.content) {
+          if (block.type !== 'tool_use') continue;
+          const name = block.name || '';
+
+          if (name.includes('update_node') || name.includes('add_node') || name.includes('add_to_work_mode')) {
+            lastSyncIdx = i;
+            fileChanges = 0;
+            hasCommit = false;
+          }
 
-            // Count file changes since last sync
-            if (FILE_TOOLS.includes(name) && i > lastSlyplanSyncIdx) {
-              fileChangesSinceSync++;
-            }
+          if ((name === 'Write' || name === 'Edit' || name === 'NotebookEdit') && i > lastSyncIdx) {
+            fileChanges++;
+          }
 
-            // Detect git commit since last sync
-            if (name === 'Bash' && block.input && typeof block.input.command === 'string') {
-              const cmd = block.input.command;
-              if (cmd.includes('git commit') && i > lastSlyplanSyncIdx) {
-                hasGitCommitSinceSync = true;
-              }
-            }
+          if (name === 'Bash' && line.includes('git commit') && i > lastSyncIdx) {
+            hasCommit = true;
           }
         }
       } catch {}
     }
 
-    // No changes since last sync — stay silent
-    if (fileChangesSinceSync === 0 && !hasGitCommitSinceSync) { process.exit(0); }
+    if (fileChanges === 0 && !hasCommit) { process.exit(0); }
 
-    // Fire on git commit
-    if (hasGitCommitSinceSync) {
-      const output = JSON.stringify({
-        suppressOutput: true,
+    if (hasCommit) {
+      process.stdout.write(JSON.stringify({
         hookSpecificOutput: {
           hookEventName: "PostToolUse",
-          additionalContext: "SYNC NOW: Git commit detected \\u2014 update SlyPlan node with progress before continuing."
+          additionalContext: "SYNC NOW: Git commit detected. Call update_node with current progress."
         }
-      });
-      process.stdout.write(output);
+      }));
       process.exit(0);
     }
 
-    // Fire when batch threshold reached
-    if (fileChangesSinceSync >= BATCH_THRESHOLD) {
-      const output = JSON.stringify({
-        suppressOutput: true,
+    if (fileChanges >= BATCH_THRESHOLD) {
+      process.stdout.write(JSON.stringify({
         hookSpecificOutput: {
           hookEventName: "PostToolUse",
-          additionalContext: "SYNC NOW: " + fileChangesSinceSync + " file change(s) since last sync. Call update_node with current progress."
+          additionalContext: "SYNC NOW: " + fileChanges + " file change(s) since last sync. Call update_node with current progress."
         }
-      });
-      process.stdout.write(output);
+      }));
       process.exit(0);
     }
 
-    // Under threshold — stay silent
     process.exit(0);
   } catch (e) {
     process.exit(0);

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "slyplan-mcp",
-  "version": "1.7.0",
+  "version": "1.7.2",
   "description": "MCP server for Slyplan — visual project management via Claude",
   "type": "module",
   "bin": {