slyplan-mcp 1.2.2 → 1.3.0

package/dist/cli.js

@@ -5,78 +5,211 @@ import * as readline from 'readline';
 import { createClient } from '@supabase/supabase-js';
 const SUPABASE_URL = 'https://omfzpkwtuzucwwxmyuqt.supabase.co';
 const SUPABASE_ANON_KEY = 'eyJhbGciOiJIUzI1NiIsInR5cCI6IkpXVCJ9.eyJpc3MiOiJzdXBhYmFzZSIsInJlZiI6Im9tZnpwa3d0dXp1Y3d3eG15dXF0Iiwicm9sZSI6ImFub24iLCJpYXQiOjE3NzA5MjMwNDIsImV4cCI6MjA4NjQ5OTA0Mn0.KXGoUez7M45RtFM9qR7mjzGX6UhlaRE-gggAJxSkIHY';
-const PRE_DIRECTIVE = "ACTION REQUIRED: If you haven't set a SlyPlan project this session, call list_projects + set_project NOW. Then call search to find the node for your current task and add_to_work_mode. If no node exists, create one with add_node first. Do this BEFORE continuing with other work.";
-const POST_DIRECTIVE = "ACTION REQUIRED: You just made changes. Call search to find the SlyPlan node for this task, then update_node with current progress. Skip only for trivial edits (typos, config tweaks). Never mark done without asking the user first.";
-const POST_HOOK_FILE_CONTENT = `#!/usr/bin/env node
-const output = JSON.stringify({
-  suppressOutput: true,
-  hookSpecificOutput: {
-    hookEventName: "PostToolUse",
-    additionalContext: POST_DIRECTIVE_PLACEHOLDER
+// Smart transcript-aware hook: checks if set_project + add_to_work_mode have been called.
+// Silent when OK, reminds only when needed.
+const PRE_HOOK_FILE_CONTENT = `#!/usr/bin/env node
+const fs = require('fs');
+
+let input = '';
+process.stdin.setEncoding('utf8');
+process.stdin.on('data', (chunk) => { input += chunk; });
+process.stdin.on('end', () => {
+  try {
+    const data = JSON.parse(input);
+
+    if (!data.transcript_path) {
+      const output = JSON.stringify({
+        suppressOutput: true,
+        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); }
+
+    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;
+            if (name.includes('add_to_work_mode')) hasWorkModeNode = true;
+          }
+        }
+      } catch {}
+    }
+
+    if (!hasSetProject) {
+      const output = JSON.stringify({
+        suppressOutput: true,
+        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,
+        hookSpecificOutput: {
+          hookEventName: "PreToolCall",
+          additionalContext: "SYNC NOW: You have a project set but no node in work mode. Call search + add_to_work_mode before continuing."
+        }
+      });
+      process.stdout.write(output);
+      process.exit(0);
+    }
+
+    process.exit(0);
+  } catch (e) {
+    process.exit(0);
   }
 });
-process.stdout.write(output);
-process.exit(0);
-`.replace('POST_DIRECTIVE_PLACEHOLDER', JSON.stringify(POST_DIRECTIVE));
-const PRE_HOOK_FILE_CONTENT = `#!/usr/bin/env node
-const output = JSON.stringify({
-  suppressOutput: true,
-  hookSpecificOutput: {
-    hookEventName: "PreToolCall",
-    additionalContext: PRE_DIRECTIVE_PLACEHOLDER
+`;
+// Smart transcript-aware hook: counts file changes since last SlyPlan sync.
+// Silent when synced, reminds with count when unsynced.
+const POST_HOOK_FILE_CONTENT = `#!/usr/bin/env node
+const fs = require('fs');
+
+let input = '';
+process.stdin.setEncoding('utf8');
+process.stdin.on('data', (chunk) => { input += chunk; });
+process.stdin.on('end', () => {
+  try {
+    const data = JSON.parse(input);
+
+    if (!data.transcript_path) { process.exit(0); }
+
+    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);
+
+    const FILE_TOOLS = ['Write', 'Edit', 'NotebookEdit'];
+    const SLYPLAN_SYNC_TOOLS = ['update_node', 'add_node', 'add_to_work_mode'];
+
+    let lastFileChangeIdx = -1;
+    let lastSlyplanSyncIdx = -1;
+    let fileChangeCount = 0;
+
+    for (let i = 0; i < lines.length; i++) {
+      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 || '';
+
+            if (FILE_TOOLS.includes(name)) {
+              lastFileChangeIdx = i;
+              fileChangeCount++;
+            }
+            if (name === 'Bash' && block.input && typeof block.input.command === 'string') {
+              const cmd = block.input.command;
+              if (cmd.includes('git commit') || cmd.includes('mkdir')) {
+                lastFileChangeIdx = i;
+                fileChangeCount++;
+              }
+            }
+            for (const st of SLYPLAN_SYNC_TOOLS) {
+              if (name.includes(st)) lastSlyplanSyncIdx = i;
+            }
+          }
+        }
+      } catch {}
+    }
+
+    if (fileChangeCount === 0) { process.exit(0); }
+    if (lastSlyplanSyncIdx > lastFileChangeIdx) { process.exit(0); }
+
+    const output = JSON.stringify({
+      suppressOutput: true,
+      hookSpecificOutput: {
+        hookEventName: "PostToolUse",
+        additionalContext: "SYNC NOW: " + fileChangeCount + " file change(s) since last SlyPlan sync. Call update_node with current progress before continuing."
+      }
+    });
+    process.stdout.write(output);
+    process.exit(0);
+  } catch (e) {
+    process.exit(0);
   }
 });
-process.stdout.write(output);
-process.exit(0);
-`.replace('PRE_DIRECTIVE_PLACEHOLDER', JSON.stringify(PRE_DIRECTIVE));
+`;
 const CLAUDEMD_SECTION = `
-## SlyPlan Sync — OBLIGATORISKE REGLER
+## Hook Compliance — HARD REQUIREMENTS
+
+Hook messages containing **"BLOCKED"** or **"SYNC NOW"** are HARD REQUIREMENTS, not suggestions.
 
-DU SKAL holde SlyPlan oppdatert. Dette er ikke valgfritt. Følg disse reglene:
+- **BLOCKED:** You MUST stop what you're doing and follow the instruction BEFORE doing anything else. No exceptions — regardless of whether you're mid-workflow, running a GSD command, or in a multi-step task.
+- **SYNC NOW:** You MUST call \`update_node\` with updated progress BEFORE continuing with the next tool call. Max one file change between each sync.
+- Hooks are transcript-aware — they only fire when sync is actually missing. If you see a message, it means you have NOT done the work.
+- Ignoring hooks because you're "following a recipe" or "in the middle of something" is NEVER a valid reason.
 
-### 1. Sett prosjekt FØRST
-- **Ved start av hver sesjon:** Kall \`list_projects\` + \`set_project\` FØR du gjør noe annet.
-- Match prosjekt basert på cwd-mappenavn eller brukerens kontekst.
-- Hvis tvetydig, spør brukeren. Cache prosjekt-ID resten av sesjonen.
+## SlyPlan Sync — MANDATORY RULES
 
-### 2. Finn eller opprett node FØR du begynner arbeid
-- Kall \`search\` for å finne noden som matcher oppgaven din.
-- Hvis ingen node finnes: opprett med \`add_node\` umiddelbart.
-- Kall \`add_to_work_mode\` på noden FØR du starter med Edit/Write/Bash.
-- **Du skal ALDRI starte arbeid uten en aktiv work mode node.**
+You MUST keep SlyPlan updated. This is not optional. Follow these rules:
 
-### 3. Oppdater progress UNDERVEIS
-- Etter hver meningsfull endring: kall \`update_node\` med oppdatert \`progress\` (0-100).
-- Oppdater \`description\` med hva som faktisk ble gjort.
-- Hopp over for trivielle endringer (typos, config-tweaks).
+### 1. Set project FIRST
+- **At the start of every session:** Call \`list_projects\` + \`set_project\` BEFORE doing anything else.
+- Match the project based on the cwd folder name or user context.
+- If ambiguous, ask the user. Cache the project ID for the rest of the session.
 
-### 4. ALDRI merk noe som done uten å spørre
-- Bruk \`AskUserQuestion\` med valg: "Yes, mark as done" / "Not yet" / "Skip".
-- **Ingen unntak.** Auto-marking done er forbudt.
+### 2. Find or create a node BEFORE starting work
+- Call \`search\` to find the node matching your task.
+- If no node exists: create one with \`add_node\` immediately.
+- Call \`add_to_work_mode\` on the node BEFORE starting with Edit/Write/Bash.
+- **You must NEVER start work without an active work mode node.**
 
-### 5. Rydd opp work mode når du er ferdig
-- Kall \`remove_from_work_mode\` når du er ferdig med en node eller bytter til annen oppgave.
+### 3. Update progress AS YOU GO
+- After each meaningful change: call \`update_node\` with updated \`progress\` (0-100).
+- Update \`description\` with what was actually done.
+- Skip for trivial changes (typos, config tweaks).
 
-### 6. Alt arbeid skal reflekteres
-- Gjelder ALL type arbeid. Hvis kode endret seg, SKAL SlyPlan reflektere det.
-- Små fix = \`plan\`-node under eksisterende phase. Stor feature = ny \`phase\` med \`plan\`-children.
+### 4. NEVER mark anything as done without asking
+- Use \`AskUserQuestion\` with choices: "Yes, mark as done" / "Not yet" / "Skip".
+- **No exceptions.** Auto-marking done is forbidden.
 
-### Node-hierarki
+### 5. Clean up work mode when done
+- Call \`remove_from_work_mode\` when you're done with a node or switching to another task.
+
+### 6. All work must be reflected
+- Applies to ALL types of work. If code changed, SlyPlan MUST reflect it.
+- Small fixes = \`plan\` node under existing phase. Large features = new \`phase\` with \`plan\` children.
+
+### Node hierarchy
 \`project\` > \`category\` > \`phase\` > \`plan\`
 
-### Ytelsesregler
-- ALDRI bruk \`get_tree\` (returnerer ~18k tokens). Bruk \`search\` i stedet.
-
-### MCP-verktøy
-| Verktøy | Bruk |
-|---------|------|
-| \`list_projects\` | Finn prosjekter (sesjonstart) |
-| \`set_project\` | Velg aktivt prosjekt |
-| \`search\` | Finn noder etter navn/innhold |
-| \`add_node\` | Opprett ny node (project/category/phase/plan) |
-| \`update_node\` | Oppdater status, progress, description |
-| \`add_to_work_mode\` | Marker node som aktivt arbeid |
-| \`remove_from_work_mode\` | Fjern fra aktivt arbeid |
+### Performance rules
+- NEVER use \`get_tree\` (returns ~18k tokens). Use \`search\` instead.
+
+### MCP tools
+| Tool | Usage |
+|------|-------|
+| \`list_projects\` | Find projects (session start) |
+| \`set_project\` | Select active project |
+| \`search\` | Find nodes by name/content |
+| \`add_node\` | Create new node (project/category/phase/plan) |
+| \`update_node\` | Update status, progress, description |
+| \`add_to_work_mode\` | Mark node as active work |
+| \`remove_from_work_mode\` | Remove from active work |
 `;
 // --- Helpers ---
 function log(msg) {
@@ -321,7 +454,7 @@ async function runSetup() {
     if (appendAnswer.toLowerCase() !== 'n') {
         if (fs.existsSync(claudeMdPath)) {
             const content = fs.readFileSync(claudeMdPath, 'utf8');
-            if (content.includes('## SlyPlan Sync')) {
+            if (content.includes('## SlyPlan Sync') || content.includes('## Hook Compliance')) {
                 log('  [=] CLAUDE.md already has SlyPlan Sync section — skipped');
             }
             else {

package/package.json

@@ -1,10 +1,10 @@
 {
   "name": "slyplan-mcp",
-  "version": "1.2.2",
+  "version": "1.3.0",
   "description": "MCP server for Slyplan — visual project management via Claude",
   "type": "module",
   "bin": {
-    "slyplan-mcp": "./dist/index.js"
+    "slyplan-mcp": "dist/index.js"
   },
   "files": [
     "dist"