context-doctor 0.12.0 → 0.12.2

package/dist/doctor.js

@@ -26,8 +26,10 @@ function checkMcpEntry(appName, configPath) {
         const entry = config.mcpServers?.["context-doctor"];
         if (!entry)
             return { label: appName, status: "fail", detail: `no context-doctor entry in ${configPath} — run: context-doctor install` };
-        // Absolute-path entries must point at a file that still exists.
-        const target = entry.command === "npx" ? null : entry.args?.[0];
+        // Absolute-path entries must point at a file that still exists. Launcher
+        // forms (npx, and Windows' cmd /c npx) resolve at spawn time, not now.
+        const launcher = entry.command === "npx" || entry.command === "cmd";
+        const target = launcher ? null : entry.args?.[0];
         if (target && !existsSync(target)) {
             return { label: appName, status: "fail", detail: `MCP entry points at missing file ${target} — re-run: context-doctor install` };
         }
@@ -41,7 +43,7 @@ function checkMcpEntry(appName, configPath) {
                 detail: `MCP command ${cmd} no longer exists (a Node upgrade moves version-pinned paths) — re-run: context-doctor install`,
             };
         }
-        return { label: appName, status: "ok", detail: `MCP wired (${entry.command === "npx" ? "npx, tracks npm releases" : "local build"})` };
+        return { label: appName, status: "ok", detail: `MCP wired (${launcher ? "npx, tracks npm releases" : "local build"})` };
     }
     catch (e) {
         return { label: appName, status: "fail", detail: `${configPath} is not valid JSON (${e.message})` };

package/dist/install.d.ts

@@ -6,5 +6,19 @@
  * Every config edit is a careful JSON merge with a .backup file written first.
  * `context-doctor uninstall` reverses it.
  */
+/**
+ * How to invoke the published package as an MCP server on a given platform.
+ *
+ * On Windows npx is `npx.cmd` — a batch script, not an executable. MCP clients
+ * spawn their server directly, without a shell, so a bare "npx" fails with
+ * ENOENT and the app simply shows no tools and no error. Hence the cmd /c
+ * wrapper that every working Windows MCP config uses.
+ *
+ * Exported so the platform branch is testable from any host OS.
+ */
+export declare function npxLauncher(platformName: string): {
+    command: string;
+    args: string[];
+};
 export declare function runInstall(): void;
 export declare function runUninstall(): void;

package/dist/install.js

@@ -43,9 +43,25 @@ function serverEntry() {
     const localMcp = join(selfDir, "mcp.js");
     const fromPackage = selfDir.includes(`${sep}node_modules${sep}`) || selfDir.includes("_npx");
     if (!fromPackage && existsSync(localMcp)) {
+        // `node` is node.exe on Windows — directly spawnable, no shell needed.
         return { command: "node", args: [localMcp] };
     }
-    return { command: "npx", args: ["-y", "context-doctor-mcp"] };
+    return npxLauncher(platform());
+}
+/**
+ * How to invoke the published package as an MCP server on a given platform.
+ *
+ * On Windows npx is `npx.cmd` — a batch script, not an executable. MCP clients
+ * spawn their server directly, without a shell, so a bare "npx" fails with
+ * ENOENT and the app simply shows no tools and no error. Hence the cmd /c
+ * wrapper that every working Windows MCP config uses.
+ *
+ * Exported so the platform branch is testable from any host OS.
+ */
+export function npxLauncher(platformName) {
+    return platformName === "win32"
+        ? { command: "cmd", args: ["/c", "npx", "-y", "context-doctor-mcp"] }
+        : { command: "npx", args: ["-y", "context-doctor-mcp"] };
 }
 function readJson(path) {
     if (!existsSync(path))

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.12.0" }, { instructions: SERVER_INSTRUCTIONS });
+    const server = new McpServer({ name: "context-doctor", version: "0.12.2" }, { 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/dist/optimize.js

@@ -127,6 +127,14 @@ export function optimizeConversation(input, options = {}) {
     if (!Array.isArray(messages)) {
         throw new Error("No `messages` array found in input");
     }
+    // Null / non-object entries occur in truncated and hand-edited files; every
+    // strategy below would throw on them. Drop them IN PLACE rather than working
+    // on a copy — prune-history splices this same array, and the returned
+    // conversation is the caller's original object.
+    for (let i = messages.length - 1; i >= 0; i--) {
+        if (!messages[i] || typeof messages[i] !== "object")
+            messages.splice(i, 1);
+    }
     const tokensBefore = messages.reduce((s, m) => s + estimateTokens(textOf(m.content)), 0);
     const applied = [];
     // -- strip-base64: replace inline blobs with a placeholder --------------------

package/dist/parse.js

@@ -62,7 +62,10 @@ function flattenContent(content) {
     }
     return { text, hasBinary, toolName, kind, toolCallText: toolCallText || undefined };
 }
-function normalizeMessage(raw, index) {
+function normalizeMessage(rawInput, index) {
+    // A null or non-object entry appears in truncated and hand-edited files.
+    // Treat it as an empty message rather than throwing a stack at the user.
+    const raw = rawInput && typeof rawInput === "object" ? rawInput : {};
     const role = String(raw.role ?? "user");
     const flat = flattenContent(raw.content);
     let kind = flat.kind ?? (["system", "user", "assistant"].includes(role) ? role : "other");
@@ -73,8 +76,10 @@ function normalizeMessage(raw, index) {
     if (role === "tool") {
         kind = "tool_result";
     }
-    const toolCalls = raw.tool_calls;
-    if (Array.isArray(toolCalls) && toolCalls.length > 0) {
+    const rawToolCalls = raw.tool_calls;
+    // Entries can be null or malformed in hand-edited or truncated exports.
+    const toolCalls = Array.isArray(rawToolCalls) ? rawToolCalls.filter((tc) => tc && typeof tc === "object") : undefined;
+    if (toolCalls && toolCalls.length > 0) {
         kind = "tool_call";
         toolName = toolCalls[0]?.function?.name ?? toolCalls[0]?.name;
         const calls = toolCalls
@@ -118,7 +123,12 @@ export function parseConversation(input) {
         };
     }
     const obj = data;
-    const rawMessages = obj.messages ?? [];
+    const rawField = obj.messages;
+    const rawMessages = Array.isArray(rawField)
+        ? rawField
+        : [];
+    // `messages` present but not an array is a malformed file, not an empty chat.
+    const malformedMessages = rawField != null && !Array.isArray(rawField);
     const messages = [];
     // Anthropic keeps the system prompt outside the messages array.
     if (obj.system != null) {
@@ -127,9 +137,11 @@ export function parseConversation(input) {
     }
     messages.push(...rawMessages.map((m, i) => normalizeMessage(m, i)));
     const isAnthropic = obj.system != null ||
-        rawMessages.some((m) => Array.isArray(m.content) && m.content.some((b) => b?.type === "tool_use" || b?.type === "tool_result"));
-    const parseWarning = messages.length === 0
-        ? "This JSON has no `messages` array (and no `system`) — it does not look like a conversation. Expected {\"messages\":[{\"role\":…,\"content\":…}]}."
-        : undefined;
+        rawMessages.some((m) => Array.isArray(m?.content) && m.content.some((b) => b?.type === "tool_use" || b?.type === "tool_result"));
+    const parseWarning = malformedMessages
+        ? `\`messages\` is a ${typeof rawField}, not an array — this file is malformed.`
+        : messages.length === 0
+            ? "This JSON has no `messages` array (and no `system`) — it does not look like a conversation. Expected {\"messages\":[{\"role\":…,\"content\":…}]}."
+            : undefined;
     return { sourceFormat: isAnthropic ? "anthropic" : "openai", parseWarning, messages };
 }

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "context-doctor",
-  "version": "0.12.0",
+  "version": "0.12.2",
   "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",
@@ -42,7 +42,7 @@
     "build": "tsc && node -e \"const fs=require('fs');['dist/cli.js','dist/mcp.js'].forEach(f=>fs.chmodSync(f,0o755))\"",
     "prepublishOnly": "npm run build",
     "dev": "tsc --watch",
-    "test": "npm run build && node --test dist/test/smoke.test.js dist/test/proxy.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"
+    "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"
   },
   "dependencies": {
     "@modelcontextprotocol/sdk": "^1.0.0",