context-doctor 0.13.3 → 0.13.4

package/README.md

@@ -34,6 +34,8 @@ Findings (4)
 npx context-doctor install
 ```
 
+`install` configures every app it detects and does not stop at the first problem: a corrupt Claude Desktop config still gets you Claude Code and Cursor. It does not pretend either. Any target that failed is named with a ✗ line, the summary reads "Done with N problem(s)" instead of "Done.", and the **exit code is 1**, so dotfiles and onboarding scripts can react. A broken config file is never overwritten; fix it and re-run.
+
 That single command is also all it takes to **set up context-doctor on anyone else's machine**. Prefer a global install, or want the unreleased `main`? Both work (Node 20+):
 
 ```bash

package/dist/cli.js

@@ -359,7 +359,10 @@ function main() {
         return;
     }
     if (args.command === "install") {
-        runInstall();
+        // Partial success is still installed, but not silent: any failed target
+        // makes the exit code non-zero so automation can react.
+        if (runInstall().failures.length > 0)
+            process.exitCode = 1;
         return;
     }
     if (args.command === "uninstall") {

package/dist/install.d.ts

@@ -20,5 +20,20 @@ export declare function npxLauncher(platformName: string): {
     command: string;
     args: string[];
 };
-export declare function runInstall(): void;
+/** Outcome of an install run, so the CLI can set a truthful exit code. */
+export interface InstallResult {
+    /** Detected targets that could not be configured, with the reason. */
+    failures: string[];
+}
+/**
+ * Install into every detected app.
+ *
+ * A failure in one app must not stop the others: someone with a corrupt
+ * Claude Desktop config still wants Claude Code and Cursor wired. But it
+ * must not be reported as success either — automation (dotfiles, CI,
+ * onboarding scripts) reads the exit code, and a "Done." with exit 0 over a
+ * failed target is a lie that surfaces later as "the tools never showed up".
+ * So: keep going, summarize, and return the failures for a non-zero exit.
+ */
+export declare function runInstall(): InstallResult;
 export declare function runUninstall(): void;

package/dist/install.js

@@ -214,6 +214,16 @@ function installSkill() {
     copyFileSync(skillSource, join(skillDest, "SKILL.md"));
     return join(skillDest, "SKILL.md");
 }
+/**
+ * Install into every detected app.
+ *
+ * A failure in one app must not stop the others: someone with a corrupt
+ * Claude Desktop config still wants Claude Code and Cursor wired. But it
+ * must not be reported as success either — automation (dotfiles, CI,
+ * onboarding scripts) reads the exit code, and a "Done." with exit 0 over a
+ * failed target is a lie that surfaces later as "the tools never showed up".
+ * So: keep going, summarize, and return the failures for a non-zero exit.
+ */
 export function runInstall() {
     const entry = serverEntry();
     const found = targets().filter((t) => t.detect());
@@ -221,8 +231,9 @@ export function runInstall() {
         console.log("No supported AI apps detected (Claude Desktop, Claude Code, Cursor).");
         console.log("Manual setup — add to your app's MCP config:");
         console.log(JSON.stringify({ mcpServers: { "context-doctor": entry } }, null, 2));
-        return;
+        return { failures: [] };
     }
+    const failures = [];
     for (const target of found) {
         try {
             const config = readJson(target.configPath);
@@ -234,22 +245,44 @@ export function runInstall() {
         }
         catch (e) {
             console.error(`✗ ${target.name}: ${e.message}`);
+            failures.push(target.name);
         }
     }
-    const skillPath = installSkill();
-    if (skillPath)
-        console.log(`✓ Agent Skill installed for Claude Code (${skillPath})`);
-    const hookPath = installHook();
-    if (hookPath) {
-        console.log(`✓ Claude Code every-prompt hook installed (${hookPath}) — heavy sessions get automatic hygiene guidance`);
-        // npx resolves the package on every single prompt; a global install makes
-        // the hook a plain exec instead, which is both faster and update-proof.
-        if (hookUsesNpx()) {
-            console.log("  note: the hook falls back to npx. For a faster, permanent hook: npm i -g context-doctor && context-doctor install");
+    try {
+        const skillPath = installSkill();
+        if (skillPath)
+            console.log(`✓ Agent Skill installed for Claude Code (${skillPath})`);
+    }
+    catch (e) {
+        console.error(`✗ Agent Skill: ${e.message}`);
+        failures.push("Agent Skill");
+    }
+    try {
+        const hookPath = installHook();
+        if (hookPath) {
+            console.log(`✓ Claude Code every-prompt hook installed (${hookPath}) — heavy sessions get automatic hygiene guidance`);
+            // npx resolves the package on every single prompt; a global install makes
+            // the hook a plain exec instead, which is both faster and update-proof.
+            if (hookUsesNpx()) {
+                console.log("  note: the hook falls back to npx. For a faster, permanent hook: npm i -g context-doctor && context-doctor install");
+            }
         }
     }
-    console.log("\nDone. Restart the apps to pick up the new tools, then try:");
-    console.log('  "What\'s eating my context?" — or paste a conversation and ask for a profile.');
+    catch (e) {
+        // An unreadable settings.json used to escape as a stack trace and abort
+        // the run; it is a failed target like any other.
+        console.error(`✗ Claude Code every-prompt hook: ${e.message}`);
+        failures.push("Claude Code hook");
+    }
+    if (failures.length > 0) {
+        console.log(`\nDone with ${failures.length} problem(s): ${failures.join(", ")}. See the ✗ lines above.`);
+        console.log("Everything else was installed. Exit code is 1 so scripts can tell; fix the file(s) and re-run install.");
+    }
+    else {
+        console.log("\nDone. Restart the apps to pick up the new tools, then try:");
+        console.log('  "What\'s eating my context?" — or paste a conversation and ask for a profile.');
+    }
+    return { failures };
 }
 export function runUninstall() {
     for (const target of targets().filter((t) => t.detect())) {

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.3" }, { instructions: SERVER_INSTRUCTIONS });
+    const server = new McpServer({ name: "context-doctor", version: "0.13.4" }, { 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.3",
+  "version": "0.13.4",
   "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",