@codescene/codehealth-mcp 1.0.0 → 1.0.1

package/lib/download.js

@@ -220,6 +220,12 @@ async function renameExtractedBinary(cacheDir, binaryPath) {
  * @returns {Promise<string>} Path to the binary
  */
 export async function ensureBinary(version) {
+  if (typeof version !== "string" || !version) {
+    throw new Error(
+      `ensureBinary requires a non-empty version string, got: ${JSON.stringify(version)}`
+    );
+  }
+
   const binaryPath = getCachedBinaryPath(version);
 
   if (existsSync(binaryPath)) {

package/lib/index.js

@@ -8,20 +8,50 @@
 import { readFileSync, existsSync } from "node:fs";
 import { join, dirname, resolve } from "node:path";
 import { fileURLToPath } from "node:url";
+import { setTimeout as delay } from "node:timers/promises";
 import { ensureBinary } from "./download.js";
 import { runBinary } from "./run.js";
 
 const __dirname = dirname(fileURLToPath(import.meta.url));
 const PACKAGE_ROOT = join(__dirname, "..");
 
+/** Maximum number of retry attempts when package files are temporarily unavailable. */
+const MAX_RETRIES = 5;
+
+/** Delay in milliseconds between retry attempts. */
+const RETRY_DELAY_MS = 1000;
+
+/**
+ * Returns true if the error is transient — caused by package files being
+ * replaced during an npm update (file missing, partially written, or
+ * missing version field).
+ *
+ * @param {Error} err
+ * @returns {boolean}
+ */
+function isTransientStartupError(err) {
+  if (err.code === "ENOENT") return true;
+  if (err instanceof SyntaxError) return true;
+  if (err.message && err.message.includes("missing version")) return true;
+  return false;
+}
+
 /**
  * Reads the package version from package.json.
  *
  * @returns {string}
+ * @throws {Error} If version is missing or not a string.
  */
 function getPackageVersion() {
   const pkgPath = join(PACKAGE_ROOT, "package.json");
   const pkg = JSON.parse(readFileSync(pkgPath, "utf-8"));
+  if (typeof pkg.version !== "string" || !pkg.version) {
+    throw new Error(
+      "Invalid or missing version in package.json — " +
+        "the package may be updating. " +
+        `Got: ${JSON.stringify(pkg.version)}`
+    );
+  }
   return pkg.version;
 }
 
@@ -33,19 +63,40 @@ function getPackageVersion() {
  * 2. Cached binary for the current package version
  * 3. Download from GitHub releases and cache
  *
+ * When package files are temporarily unavailable (e.g. during an npm
+ * update), retries up to MAX_RETRIES times before giving up. This
+ * prevents transient errors when VS Code restarts the MCP server
+ * while the package is being replaced.
+ *
  * All command-line arguments (except the node binary and script path)
  * are forwarded to the cs-mcp binary.
  */
 export async function main() {
   const args = process.argv.slice(2);
 
-  try {
-    const binaryPath = await resolveBinaryPath();
-    runBinary(binaryPath, args);
-  } catch (err) {
-    process.stderr.write(`Error: ${err.message}\n`);
-    process.exit(1);
+  let lastErr;
+  for (let attempt = 0; attempt <= MAX_RETRIES; attempt++) {
+    try {
+      const binaryPath = await resolveBinaryPath();
+      runBinary(binaryPath, args);
+      return;
+    } catch (err) {
+      lastErr = err;
+      if (attempt < MAX_RETRIES && isTransientStartupError(err)) {
+        process.stderr.write(
+          `Package files unavailable (${err.code || err.constructor.name}), ` +
+            `retrying in ${RETRY_DELAY_MS / 1000}s ` +
+            `(${attempt + 1}/${MAX_RETRIES})...\n`
+        );
+        await delay(RETRY_DELAY_MS);
+        continue;
+      }
+      break;
+    }
   }
+
+  process.stderr.write(`Error: ${lastErr.message}\n`);
+  process.exit(1);
 }
 
 /**

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@codescene/codehealth-mcp",
-  "version": "1.0.0",
+  "version": "1.0.1",
   "mcpName": "com.codescene/codescene-mcp-server",
   "description": "CodeScene MCP Server — Code Health analysis for AI coding agents",
   "type": "module",