klypix-mcp 1.13.0 → 1.13.1

package/bin/klypix-install.mjs

@@ -58,13 +58,16 @@ const flatten = (code) => code
     .replace(/from '\.\.\/src\/klypix-core\.mjs'/g, "from './klypix-core.mjs'")
     .replace(/from '\.\.\/src\/klypix-format\.mjs'/g, "from './klypix-format.mjs'")
     .replace(/\.\.\/src\/klypix-(core|format)\.mjs/g, './klypix-$1.mjs')
+    // brain-doctor + agent-rules (the server's lazy `import('../src/brain-doctor.mjs')`
+    // for the brain_doctor tool) → flat sibling refs in the runtime layout.
+    .replace(/\.\.\/src\/(brain-doctor|agent-rules)\.mjs/g, './$1.mjs')
     .replace(/const PKG_VERSION = \(\(\) => \{[\s\S]*?\}\)\(\);/, `const PKG_VERSION = '${VERSION}'; // baked at install (flat layout has no package.json)`);
 
 try {
     fs.mkdirSync(BRAIN_DIR, { recursive: true });
     // 1) flat engine + hook scripts (their imports are already './…' → verbatim)
     let n = 0;
-    for (const f of ['global-brain-hook.mjs', 'brain-semantic.mjs', 'brain-note.mjs', 'brain-git-hook.mjs', 'klypix-format.mjs', 'klypix-core.mjs']) {
+    for (const f of ['global-brain-hook.mjs', 'brain-semantic.mjs', 'brain-note.mjs', 'brain-git-hook.mjs', 'klypix-format.mjs', 'klypix-core.mjs', 'agent-rules.mjs', 'brain-doctor.mjs']) {
         const s = path.join(SRC, f); if (exists(s)) { fs.writeFileSync(path.join(BRAIN_DIR, f), fs.readFileSync(s, 'utf8')); n++; }
     }
     // 2) the two servers, flattened to the *-server.mjs names the runtime/config expect

package/bin/klypix-mcp.mjs

@@ -199,6 +199,30 @@ server.registerTool('brain_note', {
   return toContent(await opBrainNote({ vault: VAULT, canvas, text, area, marker: marker || '', closes, via }));
 });
 
+server.registerTool('brain_doctor', {
+  title: 'Brain doctor — is this brain current, wired, and in sync?',
+  description: 'Read-only self-check of the installed klypix brain, as ONE verdict: VERSION (the deployed brain-core version + optional npm-latest currency), HOOKS (are all 4 Claude Code hooks wired — liveness vs readiness), TOOLS (the discoverable MCP verb manifest), PEERS (other live sessions on this project\'s brain right now), and HARNESS (per-file projection drift: ok/stale/hand-edited/missing). Use to answer "is my brain current + correctly installed + in sync, and who else is live?" without file-spelunking. Never writes. The MCP-callable twin of `npx klypix-mcp doctor`.',
+  inputSchema: {
+    project: z.string().optional().describe('Project dir to audit harness + peers for. Defaults to the server\'s working directory.'),
+    check_npm: z.boolean().optional().describe('Also fetch npm latest to flag a stale brain (default false — this one does a network `npm view`).'),
+  },
+}, async ({ project, check_npm }) => {
+  try {
+    // Lazy import so a flat runtime missing brain-doctor.mjs can't crash server STARTUP —
+    // the tool degrades gracefully (errors only when called) instead of taking the server down.
+    const { inspect, render } = await import('../src/brain-doctor.mjs');
+    let npmLatest = null;
+    if (check_npm) {
+      try { const { execSync } = await import('child_process'); npmLatest = execSync('npm view klypix-mcp version', { encoding: 'utf8', stdio: ['ignore', 'pipe', 'ignore'], timeout: 8000 }).trim(); }
+      catch { npmLatest = '(offline)'; }
+    }
+    const report = inspect({ projectDir: project ? path.resolve(project) : process.cwd(), npmLatest });
+    return { content: [{ type: 'text', text: render(report, { color: false }) }] };
+  } catch (e) {
+    return { content: [{ type: 'text', text: `brain_doctor unavailable: ${e?.message || e}` }], isError: true };
+  }
+});
+
 const transport = new StdioServerTransport();
 await server.connect(transport);
 log(`ready · vault=${VAULT}`);

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "klypix-mcp",
-  "version": "1.13.0",
+  "version": "1.13.1",
   "description": "An open, local-first, agent-neutral canvas file your AI reads and writes over MCP — works with Claude, Cursor, Cline, any model.",
   "type": "module",
   "license": "MIT",

package/src/agent-rules.mjs

@@ -152,13 +152,22 @@ function mergeMcpJson(file, wrapKey, withType) {
   return { action: before === undefined ? 'created' : 'updated' };
 }
 
-// Check-only classifier for an MCP json: is the klypix-canvas server wired?
+// Check-only classifier for an MCP json: is the klypix-canvas server wired AND does
+// its entry still actually launch klypix-mcp? Presence alone isn't enough — a hand-edit
+// that mangles the command/args (so the server never starts) reads as "wired" but is
+// broken. We DON'T strict-hash the entry: a customized `--vault <path>` is legitimate,
+// so only the invocation (npx/node … klypix-mcp …) must survive; if it doesn't, that's
+// drift the user needs to re-`link`.
 function classifyMcp(file, wrapKey) {
   if (!exists(file)) return { status: 'missing' };
-  try {
-    const cfg = JSON.parse(fs.readFileSync(file, 'utf8') || '{}');
-    return cfg?.[wrapKey]?.['klypix-canvas'] ? { status: 'ok' } : { status: 'missing' };
-  } catch { return { status: 'hand-edited', why: 'invalid JSON' }; }
+  let cfg;
+  try { cfg = JSON.parse(fs.readFileSync(file, 'utf8') || '{}'); }
+  catch { return { status: 'hand-edited', why: 'invalid JSON' }; }
+  const entry = cfg?.[wrapKey]?.['klypix-canvas'];
+  if (!entry) return { status: 'missing' };
+  const args = Array.isArray(entry.args) ? entry.args.map(String) : [];
+  const launches = (entry.command === 'npx' || entry.command === 'node') && args.some(a => a.includes('klypix-mcp'));
+  return launches ? { status: 'ok' } : { status: 'hand-edited', why: 'entry no longer launches klypix-mcp' };
 }
 
 // The projection map — the single source of truth shared by WRITE (linkProject) and