context-doctor 0.13.2 → 0.13.3

package/README.md

@@ -383,6 +383,8 @@ npm publish              # prompts for the npm 2FA code
 git push --follow-tags
 ```
 
+**What the npm download number measures.** `install` writes `npx -y context-doctor-mcp` into MCP configs, and npx re-fetches the tarball whenever a new version exists. So every release is downloaded once by every active install within about a day, and the daily count is almost entirely those refreshes: on this package, release days run ~170 downloads and non-release days ~27. Read it as "size of the active installed base × number of releases", not as new users — a quiet week with no releases will look like a decline while nothing has changed. Two corollaries: the release-day figure is a live count of machines running context-doctor, and a broken release reaches all of them automatically, which is why `prepublishOnly` runs the full test suite. npm's stats also lag by several days and occasionally record a day as zero; a zero on a release day is a gap in their pipeline, not in usage.
+
 Known gotcha: if `npm publish` fails with **`404 Not Found - PUT …/context-doctor`** on a package that clearly exists, the real cause is an **expired npm login token** — npm reports unauthenticated publishes as a 404, not a 401. Check with `npm whoami`; if that errors, run `npm login` and publish again.
 
 Also keep the MCP server version in `src/mcp.ts` in sync with `package.json`, and remember `dist/` is committed — run `npm run build` before committing so the CI dist-sync check passes.

package/dist/doctor.js

@@ -50,6 +50,23 @@ function checkMcpEntry(appName, configPath) {
         return { label: appName, status: "fail", detail: `${configPath} is not valid JSON (${e.message})` };
     }
 }
+/**
+ * For a hook command, the path that must exist for it to run — or null when it
+ * resolves through PATH (`node`, `npx`) and there is nothing to check here.
+ *
+ * Forms written by install: `node "<cli.js>" hook`, `"<binary>" hook`,
+ * `npx -y context-doctor hook`.
+ */
+function hookBinaryMissing(command) {
+    const quoted = [...command.matchAll(/"([^"]+)"/g)].map((m) => m[1]);
+    const first = command.trim().split(/\s+/)[0]?.replace(/^"|"$/g, "") ?? "";
+    const candidates = quoted.length > 0 ? quoted : /[\\/]/.test(first) ? [first] : [];
+    for (const path of candidates) {
+        if (!existsSync(path))
+            return path;
+    }
+    return null;
+}
 /** Spawn our own MCP server and run the initialize handshake over stdio. */
 function checkMcpHandshake() {
     const label = "MCP server handshake";
@@ -96,10 +113,20 @@ export async function runDoctor() {
     if (existsSync(settingsPath)) {
         try {
             const settings = JSON.parse(readFileSync(settingsPath, "utf8"));
-            const registered = JSON.stringify(settings.hooks?.UserPromptSubmit ?? []).includes("context-doctor");
-            checks.push(registered
-                ? { label: "Every-prompt hook", status: "ok", detail: "registered in ~/.claude/settings.json" }
-                : { label: "Every-prompt hook", status: "fail", detail: "not registered — run: context-doctor install" });
+            const entries = settings.hooks?.UserPromptSubmit ?? [];
+            const ours = entries.map((e) => e.hooks?.[0]?.command ?? "").find((c) => /context-doctor|cli\.js"?\s+hook/.test(c));
+            if (!ours) {
+                checks.push({ label: "Every-prompt hook", status: "fail", detail: "not registered — run: context-doctor install" });
+            }
+            else {
+                // "Registered" is not "working": a hook whose binary has been deleted
+                // (an npx cache sweep, a Node upgrade) fails silently on every prompt,
+                // and this check used to report it as fine.
+                const missing = hookBinaryMissing(ours);
+                checks.push(missing
+                    ? { label: "Every-prompt hook", status: "fail", detail: `registered, but ${missing} no longer exists — re-run: context-doctor install` }
+                    : { label: "Every-prompt hook", status: "ok", detail: "registered in ~/.claude/settings.json; command resolves" });
+            }
         }
         catch (e) {
             checks.push({ label: "Every-prompt hook", status: "fail", detail: `settings.json unreadable (${e.message})` });

package/dist/install.js

@@ -89,6 +89,13 @@ function binOnPath(name) {
     for (const dir of (process.env.PATH ?? "").split(delimiter)) {
         if (!dir)
             continue;
+        // npx prepends its OWN cache's .bin to PATH while it runs the command. So
+        // during `npx -y context-doctor install`, the first "global binary" on
+        // PATH is inside _npx — the garbage-collected directory this lookup exists
+        // to avoid. That hole put a cache path in the hook of every user who
+        // followed the README's headline command.
+        if (isEphemeralPath(dir))
+            continue;
         for (const ext of exts) {
             const candidate = join(dir, name + ext);
             if (existsSync(candidate))
@@ -97,6 +104,10 @@ function binOnPath(name) {
     }
     return null;
 }
+/** Paths npm may delete at any time: the npx cache and the npm cache itself. */
+function isEphemeralPath(path) {
+    return /[\\/]_npx[\\/]/.test(path) || /[\\/]\.npm[\\/]/.test(path) || /[\\/]npm-cache[\\/]/i.test(path);
+}
 /**
  * Shell command used for the Claude Code every-prompt hook.
  *
@@ -113,8 +124,7 @@ function binOnPath(name) {
 function hookCommand() {
     const selfDir = dirname(fileURLToPath(import.meta.url));
     const localCli = join(selfDir, "cli.js");
-    const ephemeral = selfDir.includes("_npx");
-    if (!ephemeral && existsSync(localCli))
+    if (!isEphemeralPath(selfDir + sep) && existsSync(localCli))
         return `node "${localCli}" hook`;
     const global = binOnPath("context-doctor");
     if (global)

package/dist/mcp.js

@@ -37,7 +37,7 @@ const STRATEGY_IDS = ["dedupe", "trim-tool-results", "trim-tool-calls", "strip-b
  * recommended pattern.
  */
 function createServer() {
-    const server = new McpServer({ name: "context-doctor", version: "0.13.2" }, { instructions: SERVER_INSTRUCTIONS });
+    const server = new McpServer({ name: "context-doctor", version: "0.13.3" }, { instructions: SERVER_INSTRUCTIONS });
     server.tool("profile_context", "Profile an LLM conversation or prompt: token breakdown by category, largest messages, and actionable findings about wasted context (duplicates, oversized tool results, base64 blobs, cache-unfriendly ordering). Accepts OpenAI/Anthropic conversation JSON or raw text. Call this immediately whenever the user asks about token usage, context size, LLM cost, or latency — and proactively offer it once a conversation grows long or accumulates large pasted content.", {
         conversation: z.string().describe("Conversation JSON (OpenAI or Anthropic format, or bare message array) or raw prompt text"),
         model: z.string().optional().describe("Target model name for context-window math, e.g. claude-sonnet-5 or gpt-4o"),

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "context-doctor",
-  "version": "0.13.2",
+  "version": "0.13.3",
   "description": "Profile and optimize LLM context windows. See what's eating your tokens and fix it — works with Claude, GPT, Gemini, and any MCP-capable AI app.",
   "keywords": [
     "llm",
@@ -40,7 +40,7 @@
   },
   "scripts": {
     "build": "tsc && node -e \"const fs=require('fs');['dist/cli.js','dist/mcp.js'].forEach(f=>fs.chmodSync(f,0o755))\"",
-    "prepublishOnly": "npm run build",
+    "prepublishOnly": "npm test",
     "dev": "tsc --watch",
     "test": "npm run build && node --test dist/test/smoke.test.js dist/test/proxy.test.js dist/test/proxy-abort.test.js dist/test/hook.test.js dist/test/mcp-http.test.js dist/test/doctor.test.js dist/test/watch.test.js dist/test/chatgpt-export.test.js dist/test/config.test.js dist/test/dashboard.test.js dist/test/cursor.test.js dist/test/cache.test.js dist/test/session.test.js dist/test/accuracy.test.js dist/test/cache-stability.test.js dist/test/ledger.test.js"
   },