docbrain-mcp 1.5.10 → 1.5.11

package/install.js

@@ -40,14 +40,31 @@ function download(url) {
           reject(new Error(`Download failed: HTTP ${res.statusCode}`));
           return;
         }
-        const file = fs.createWriteStream(dest);
+        // Download to a scratch path and rename into place. rename() is atomic
+        // within a filesystem, so a download that is interrupted — or two that
+        // race, now that run.js can trigger one on first use — can never leave a
+        // half-written file at `dest`. A truncated binary is the worst outcome
+        // available here: it exists, so nothing re-downloads it, and it fails at
+        // exec with an error that says nothing about the real cause.
+        const tmp = `${dest}.tmp-${process.pid}`;
+        const file = fs.createWriteStream(tmp);
         res.pipe(file);
         file.on("finish", () => {
-          file.close();
-          fs.chmodSync(dest, 0o755);
-          resolve();
+          file.close(() => {
+            try {
+              fs.chmodSync(tmp, 0o755);
+              fs.renameSync(tmp, dest);
+              resolve();
+            } catch (err) {
+              try { fs.unlinkSync(tmp); } catch {}
+              reject(err);
+            }
+          });
+        });
+        file.on("error", (err) => {
+          try { fs.unlinkSync(tmp); } catch {}
+          reject(err);
         });
-        file.on("error", reject);
       })
       .on("error", reject);
   });

package/package.json

@@ -1,13 +1,13 @@
 {
   "name": "docbrain-mcp",
-  "version": "1.5.10",
-  "description": "DocBrain MCP server — query your documentation from Claude Code, Cursor, and any MCP-compatible editor",
+  "version": "1.5.11",
+  "description": "DocBrain MCP server \u2014 query your documentation from Claude Code, Cursor, and any MCP-compatible editor",
   "homepage": "https://github.com/docbrain-ai/docbrain",
   "repository": {
     "type": "git",
     "url": "https://github.com/docbrain-ai/docbrain.git"
   },
-  "license": "BSL-1.1",
+  "license": "MIT",
   "bin": {
     "docbrain-mcp": "run.js"
   },

package/run.js

@@ -1,11 +1,47 @@
 #!/usr/bin/env node
 "use strict";
 
-const { spawn } = require("child_process");
+const { spawn, execFileSync } = require("child_process");
+const fs = require("fs");
 const path = require("path");
 
 const binary = path.join(__dirname, "bin", "docbrain-mcp");
 
+// npm 12 turned lifecycle scripts off by default, so the postinstall hook that
+// fetches this binary may never have run — and `npx -y docbrain-mcp@latest`,
+// which is how the MCP setup instructions tell people to launch this, goes
+// through exactly that path. Fetching on first use covers it.
+//
+// Deliberately synchronous, before the spawn below. This runs once, at startup,
+// before any MCP traffic exists to delay; doing it asynchronously would mean
+// wiring the signal forwarding and exit plumbing around a pending download for
+// no benefit.
+//
+// Note that stderr is used for the progress line, never stdout: stdout is the
+// MCP stdio transport, and a stray byte there corrupts the protocol stream.
+function ensureBinary() {
+  if (fs.existsSync(binary)) return;
+
+  console.error("DocBrain MCP binary not found — downloading it now (one-time).");
+  try {
+    execFileSync(process.execPath, [path.join(__dirname, "install.js")], {
+      stdio: ["ignore", "ignore", "inherit"],
+    });
+  } catch {
+    console.error("");
+    console.error(`Could not place the binary at: ${binary}`);
+    console.error(
+      "If that path is not writable, run this once as the user that installed the package,"
+    );
+    console.error(
+      "or download the binary manually from https://github.com/docbrain-ai/docbrain/releases"
+    );
+    process.exit(1);
+  }
+}
+
+ensureBinary();
+
 const child = spawn(binary, process.argv.slice(2), {
   stdio: "inherit",
   env: process.env,
@@ -25,7 +61,10 @@ child.on("exit", (code, signal) => {
 });
 
 child.on("error", (err) => {
+  // ensureBinary already guaranteed the file exists, so this is no longer a
+  // missing-binary case — it is a permissions or exec-format problem, and
+  // suggesting a reinstall would send the user down the wrong path.
   console.error(`Failed to run DocBrain MCP server: ${err.message}`);
-  console.error("Try reinstalling: npx -y docbrain-mcp@latest");
+  console.error(`Binary: ${binary}`);
   process.exit(1);
 });