@tpsdev-ai/flair-mcp 0.7.0 → 0.8.1

package/dist/index.js

@@ -27,8 +27,16 @@ function classifyError(err, flairUrl) {
         const { status, body } = err;
         if (status === 400)
             return `validation_error: ${body}`;
-        if (status === 401 || status === 403)
-            return `auth_error: ${body}`;
+        if (status === 401 || status === 403) {
+            // Auth failure on a previously-working session usually means the daemon
+            // restarted (config reload, Harper alter_user, port change). Tell the
+            // operator how to recover instead of just surfacing the raw 401 body.
+            return `auth_error: ${body}\n` +
+                `(Hint: this often follows a Flair daemon restart. Try:\n` +
+                `  1. Restart your MCP host (Claude Code, Cursor, etc) to spawn a fresh flair-mcp.\n` +
+                `  2. Check daemon: 'flair status' or 'curl ${flairUrl}/Health'.\n` +
+                `  3. Verify your agent key still matches the registered Agent record.)`;
+        }
         if (status === 413)
             return `payload_too_large: ${body}`;
         if (status === 429)
@@ -42,7 +50,10 @@ function classifyError(err, flairUrl) {
             return "timeout — the server took too long. This often happens with large content that requires embedding. Try shorter content or retry.";
         }
         if (err instanceof TypeError && err.message.includes("fetch")) {
-            return `connection_error (retriable): could not reach Flair at ${flairUrl}. Is it running?`;
+            return `connection_error (retriable): could not reach Flair at ${flairUrl}. Is it running?\n` +
+                `(Diagnostics:\n` +
+                `  - 'curl ${flairUrl}/Health' — if this responds 200 or 401, daemon is up + this is an auth issue not a connection one.\n` +
+                `  - 'launchctl list | grep flair' (macOS) or 'systemctl status flair' (Linux).)`;
         }
         return `unexpected_error: ${err.message}`;
     }
@@ -51,6 +62,47 @@ function classifyError(err, flairUrl) {
 function errorResult(err, flairUrl) {
     return { content: [{ type: "text", text: classifyError(err, flairUrl) }], isError: true };
 }
+// ─── Parent-exit watcher ────────────────────────────────────────────────────
+//
+// flair-mcp runs as a child of an MCP host (Claude Code, Cursor, etc) over
+// stdio. When the host exits cleanly it should close stdin/stdout — but in
+// practice we've seen flair-mcp processes orphaned for weeks (PID 1 as
+// parent), holding stale tokens and consuming RAM.
+//
+// Poll process.ppid every 5s. If it drops to 1 (init), the parent died and
+// we got reparented — exit cleanly. Cheap, cross-platform, no native deps.
+// Clamp the poll interval to a safe range. `process.env.FOO ?? 5000` is NOT
+// safe on its own: `??` only falls through on null/undefined, so an empty-string
+// override (`FLAIR_MCP_PARENT_POLL_MS=`) yields `Number("") === 0` and creates
+// a tight CPU-busy loop. Validate explicitly. (Sherlock review on #315.)
+const PARENT_POLL_INTERVAL_MS = (() => {
+    const raw = process.env.FLAIR_MCP_PARENT_POLL_MS;
+    const parsed = raw != null ? Number(raw) : NaN;
+    const FLOOR_MS = 100;
+    const CEILING_MS = 30_000;
+    return Number.isFinite(parsed) && parsed >= FLOOR_MS && parsed <= CEILING_MS
+        ? parsed
+        : 5000;
+})();
+const initialPpid = process.ppid;
+setInterval(() => {
+    // ppid === 1 means init/launchd has adopted us — original parent died.
+    if (process.ppid === 1 && initialPpid !== 1) {
+        console.error("flair-mcp: parent process died (re-parented to init); exiting cleanly.");
+        process.exit(0);
+    }
+}, PARENT_POLL_INTERVAL_MS).unref();
+// Also handle stdin EOF — MCP host closing the pipe means session ended.
+// (StdioServerTransport handles this internally for the MCP protocol, but
+// belt-and-suspenders: if stdin closes we exit, full stop.)
+process.stdin.on("close", () => {
+    console.error("flair-mcp: stdin closed; exiting cleanly.");
+    process.exit(0);
+});
+process.stdin.on("end", () => {
+    console.error("flair-mcp: stdin EOF; exiting cleanly.");
+    process.exit(0);
+});
 // ─── Client setup ────────────────────────────────────────────────────────────
 const agentId = process.env.FLAIR_AGENT_ID;
 if (!agentId) {
@@ -109,11 +161,9 @@ server.tool("memory_store", "Save information to persistent memory. Use for less
             dedup: true,
             dedupThreshold: 0.95,
         });
-        // Check if dedup returned an existing memory (different ID than what we generated)
-        const generatedPrefix = `${agentId}-`;
-        const wasDeduped = result.id && !result.id.startsWith(generatedPrefix);
-        if (wasDeduped) {
-            return { content: [{ type: "text", text: `Similar memory already exists (id: ${result.id}): ${result.content?.slice(0, 200)}` }] };
+        // Signal when dedup returned an existing memory instead of writing.
+        if (result.deduped) {
+            return { content: [{ type: "text", text: `Similar memory already exists (id: ${result.id}): ${result.content?.slice(0, 200)}\n(no new entry written)` }] };
         }
         const preview = content.length > 120 ? content.slice(0, 120) + "..." : content;
         const tagStr = tags && tags.length > 0 ? tags.join(", ") : "none";

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@tpsdev-ai/flair-mcp",
-  "version": "0.7.0",
+  "version": "0.8.1",
   "description": "MCP server for Flair — persistent memory for Claude Code, Cursor, and any MCP client.",
   "type": "module",
   "main": "dist/index.js",
@@ -14,7 +14,8 @@
   ],
   "scripts": {
     "build": "tsc --noCheck",
-    "prepublishOnly": "npm run build"
+    "prepublishOnly": "npm run build",
+    "postinstall": "node -e \"try{const{chmodSync,statSync}=require('fs');const p='dist/index.js';if(statSync(p).isFile()){chmodSync(p,0o755);console.error('@tpsdev-ai/flair-mcp: chmod +x ' + p + ' OK')}}catch(e){if(e.code!=='ENOENT')console.error('postinstall warn:',e.message)}\""
   },
   "publishConfig": {
     "access": "public"
@@ -24,13 +25,13 @@
   },
   "dependencies": {
     "@modelcontextprotocol/sdk": "1.27.1",
-    "@tpsdev-ai/flair-client": "0.7.0",
+    "@tpsdev-ai/flair-client": "0.8.1",
     "zod": "4.3.6"
   },
   "license": "Apache-2.0",
   "repository": {
     "type": "git",
-    "url": "https://github.com/tpsdev-ai/flair.git",
+    "url": "git+https://github.com/tpsdev-ai/flair.git",
     "directory": "packages/flair-mcp"
   },
   "homepage": "https://tps.dev/#flair",