@yawlabs/caddy-mcp 2.3.1 → 2.3.2

package/bin/caddy-mcp.mjs

@@ -180,7 +180,7 @@ function sandboxFlags() {
     }
   }
 
-  const env = ["CADDY_ADMIN_URL","CADDY_API_TOKEN","CADDY_LOAD_TIMEOUT","CADDY_MAX_RETRIES","CADDY_TIMEOUT"];
+  const env = ["CADDY_ADMIN_URL", "CADDY_API_TOKEN", "CADDY_LOAD_TIMEOUT", "CADDY_MAX_RETRIES", "CADDY_TIMEOUT"];
 
   const flags = ["--permission", netFlag, `--allow-env=${env.join(",")}`];
   return flags;
@@ -276,7 +276,8 @@ if (mode === "node") {
       const { writeSync } = await import("node:fs");
       writeSync(
         2,
-        "caddy-mcp: CADDY_MCP_RUNTIME=oam but no runnable oam binary was found.\n" + shimNote +
+        "caddy-mcp: CADDY_MCP_RUNTIME=oam but no runnable oam binary was found.\n" +
+          shimNote +
           "Install from https://oamjs.org, set OAM_BIN=/path/to/oam, or use CADDY_MCP_RUNTIME=node.\n",
       );
       process.exit(1);
@@ -354,7 +355,6 @@ if (mode === "node") {
     }
 
     if (child) {
-
       // If oam cannot be executed at all (deleted between the stat and the spawn,
       // wrong arch, permission), fall back rather than failing the whole server.
       // `spawned` prevents falling back AFTER the child started, which would
@@ -397,7 +397,12 @@ if (mode === "node") {
       // graceful shutdown the console's own Ctrl-C just started, skipping the
       // child's process.on("exit") cleanup. The console has already notified the
       // child, so on Windows the timer below is the only kill we issue.
-      const ESCALATE_AFTER_MS = 2000;
+      // 5s, not 2s. This is a HARD kill of a server that may be mid-shutdown --
+      // flushing a large config, finishing a TLS provision -- and the cost of
+      // waiting too long (a wedged child lingers a few extra seconds) is far
+      // smaller than the cost of cutting a legitimate shutdown short. The window
+      // only ever elapses when the child has NOT exited on its own.
+      const ESCALATE_AFTER_MS = 5000;
       let escalation = null;
       for (const sig of ["SIGINT", "SIGTERM"]) {
         process.on(sig, () => {

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@yawlabs/caddy-mcp",
-  "version": "2.3.1",
+  "version": "2.3.2",
   "mcpName": "io.github.YawLabs/caddy-mcp",
   "description": "MCP server for managing Caddy web servers via the admin API",
   "license": "MIT",
@@ -28,8 +28,8 @@
     "build": "tsup && tsc -p tsconfig.build.json",
     "dev": "tsup --watch",
     "test": "vitest run",
-    "lint": "biome check src/",
-    "lint:fix": "biome check --write src/",
+    "lint": "biome check src/ bin/ scripts/",
+    "lint:fix": "biome check --write src/ bin/ scripts/",
     "typecheck": "node scripts/typecheck.mjs",
     "typecheck:tsc": "tsc --noEmit",
     "test:ci": "npm run build && npm test",