context-mode 1.0.96 → 1.0.98

package/hooks/core/mcp-ready.mjs

@@ -1,30 +1,73 @@
 /**
  * MCP readiness sentinel — checks if MCP server has started.
- * Server writes sentinel (containing its PID) after connect(),
- * hooks check before denying tools that redirect to MCP alternatives.
  *
- * Sentinel path: ${tmpdir()}/context-mode-mcp-ready-${process.ppid}
- * Both hooks and MCP server share the same ppid (Claude Code process).
+ * Server writes sentinel (containing its PID) after connect().
+ * Hooks scan for any live sentinel to detect MCP readiness.
+ *
+ * Fix for #347: Claude Code spawns hooks via `bash -c "node ..."` on Linux/WSL2.
+ * The intermediate shell makes process.ppid point to a transient bash PID, not
+ * Claude Code. Directory-scan + PID liveness probe works regardless of spawn topology.
+ *
+ * Sentinel path: <tmpRoot>/context-mode-mcp-ready-<MCP_PID>
+ * Scan: glob all context-mode-mcp-ready-* files, probe each PID.
  */
-import { readFileSync } from "node:fs";
+import { readFileSync, readdirSync, unlinkSync } from "node:fs";
 import { tmpdir } from "node:os";
-import { resolve } from "node:path";
+import { join } from "node:path";
+
+const SENTINEL_PREFIX = "context-mode-mcp-ready-";
+
+/** Resolve the temp root — hardcoded /tmp on Unix to avoid TMPDIR mismatch. */
+export function sentinelDir() {
+  return process.platform === "win32" ? tmpdir() : "/tmp";
+}
+
+/**
+ * Build sentinel path for a given PID.
+ * Used by server.ts to write its own sentinel.
+ */
+export function sentinelPathForPid(pid) {
+  return join(sentinelDir(), `${SENTINEL_PREFIX}${pid}`);
+}
 
-/** Compute the sentinel file path, scoped to session via parent PID. */
+/**
+ * @deprecated Use sentinelPathForPid(process.pid) from server.ts.
+ * Kept for backward compat during migration — tests that still
+ * write sentinels with process.ppid will work for one release cycle.
+ */
 export function sentinelPath() {
-  return resolve(tmpdir(), `context-mode-mcp-ready-${process.ppid}`);
+  return join(sentinelDir(), `${SENTINEL_PREFIX}${process.ppid}`);
 }
 
 /**
- * Check if MCP server is alive by reading sentinel PID and probing it.
- * Handles stale sentinels from crashed servers (SIGKILL, OOM) — if the
- * PID in the sentinel is dead, returns false so hooks allow fallback.
+ * Check if any MCP server is alive by scanning sentinel files.
+ *
+ * Scans sentinelDir() for context-mode-mcp-ready-* files, reads the PID
+ * from each, and probes with kill(pid, 0). Cleans up stale sentinels
+ * from crashed servers.
+ *
+ * Handles:
+ * - PPID mismatch (WSL2 shell wrappers) — no ppid dependency
+ * - Stale sentinels (SIGKILL, OOM) — PID liveness check
+ * - TMPDIR mismatch — hardcoded /tmp on Unix
  */
 export function isMCPReady() {
   try {
-    const pid = parseInt(readFileSync(sentinelPath(), "utf8"), 10);
-    process.kill(pid, 0); // throws if process doesn't exist
-    return true;
+    const dir = sentinelDir();
+    const files = readdirSync(dir).filter(f => f.startsWith(SENTINEL_PREFIX));
+    for (const f of files) {
+      const fullPath = join(dir, f);
+      try {
+        const pid = parseInt(readFileSync(fullPath, "utf8"), 10);
+        if (isNaN(pid)) continue;
+        process.kill(pid, 0); // throws if process doesn't exist
+        return true;
+      } catch {
+        // Dead PID or unreadable — clean up stale sentinel
+        try { unlinkSync(fullPath); } catch {}
+      }
+    }
+    return false;
   } catch {
     return false;
   }

package/openclaw.plugin.json

@@ -3,7 +3,7 @@
   "name": "Context Mode",
   "kind": "tool",
   "description": "OpenClaw plugin that saves 98% of your context window. Sandboxed code execution in 11 languages, FTS5 knowledge base with BM25 ranking, and intent-driven search.",
-  "version": "1.0.96",
+  "version": "1.0.98",
   "sandbox": {
     "mode": "permissive",
     "filesystem_access": "full",

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "context-mode",
-  "version": "1.0.96",
+  "version": "1.0.98",
   "type": "module",
   "description": "MCP plugin that saves 98% of your context window. Works with Claude Code, Gemini CLI, VS Code Copilot, OpenCode, and Codex CLI. Sandboxed code execution, FTS5 knowledge base, and intent-driven search.",
   "author": "Mert Koseoğlu",

package/scripts/postinstall.mjs

@@ -8,9 +8,9 @@
  *    Creates a directory junction so npm's %~dp0\node_modules\... resolves.
  */
 
-import { existsSync, mkdirSync, readFileSync, writeFileSync, symlinkSync } from "node:fs";
+import { existsSync, mkdirSync, readFileSync, writeFileSync, symlinkSync, lstatSync, unlinkSync } from "node:fs";
 import { execSync } from "node:child_process";
-import { dirname, resolve, join } from "node:path";
+import { dirname, resolve, join, sep } from "node:path";
 import { fileURLToPath } from "node:url";
 import { homedir } from "node:os";
 
@@ -32,34 +32,16 @@ try {
   const ipPath = resolve(homedir(), ".claude", "plugins", "installed_plugins.json");
   if (existsSync(ipPath)) {
     const ip = JSON.parse(readFileSync(ipPath, "utf-8"));
+    const cacheRoot = resolve(homedir(), ".claude", "plugins", "cache");
     for (const [key, entries] of Object.entries(ip.plugins || {})) {
-      if (!key.toLowerCase().includes("context-mode")) continue;
-      for (const entry of entries) {
-        const rp = entry.installPath;
-        if (!rp || existsSync(rp)) continue;
-        // Registry points to a missing dir — symlink it to us
-        const rpParent = dirname(rp);
-        if (!existsSync(rpParent)) mkdirSync(rpParent, { recursive: true });
-        try {
-          symlinkSync(pkgRoot, rp, process.platform === "win32" ? "junction" : undefined);
-        } catch { /* may fail if path is locked or permissions */ }
-      }
-    }
-  }
-} catch { /* best effort — don't block install */ }
-
-// ── 0. Self-heal Layer 3: Backward symlink for stale registry (anthropics/claude-code#46915) ──
-// When this install completes, installed_plugins.json may still point to an old
-// non-existent path. Create a symlink from that old path → our new directory.
-try {
-  const ipPath = resolve(homedir(), ".claude", "plugins", "installed_plugins.json");
-  if (existsSync(ipPath)) {
-    const ip = JSON.parse(readFileSync(ipPath, "utf-8"));
-    for (const [key, entries] of Object.entries(ip.plugins || {})) {
-      if (!key.toLowerCase().includes("context-mode")) continue;
+      if (key !== "context-mode@context-mode") continue;
       for (const entry of entries) {
         const rp = entry.installPath;
         if (!rp || existsSync(rp)) continue;
+        // Path traversal guard
+        if (!resolve(rp).startsWith(cacheRoot + sep)) continue;
+        // Remove dangling symlink
+        try { if (lstatSync(rp).isSymbolicLink()) unlinkSync(rp); } catch {}
         const rpParent = dirname(rp);
         if (!existsSync(rpParent)) mkdirSync(rpParent, { recursive: true });
         try {