@yawlabs/caddy-mcp 1.2.4 → 1.2.6

package/README.md

@@ -80,7 +80,7 @@ That's it. Now ask your AI assistant:
 |---|---|---|
 | `CADDY_ADMIN_URL` | `http://localhost:2019` | Caddy admin API URL. Set to `http://caddy:2019` inside Docker, or an https URL for remote admin. |
 | `CADDY_API_TOKEN` | (none) | Optional Bearer token for authenticated admin endpoints. Only needed if you've configured Caddy with auth. |
-| `CADDY_MAX_RETRIES` | `2` | Number of retries on transient failures (5xx, network errors). 4xx and 412 never retry. POSTs to `/config/*` and `/id/*` also skip retry (non-idempotent appends/creates -- retrying could duplicate routes or 409 a half-applied create). POSTs to `/load`, `/adapt`, `/stop` still retry. Hard-capped at 5. Set to `0` to disable. |
+| `CADDY_MAX_RETRIES` | `2` | Number of retries on transient failures (5xx, network errors). 4xx and 412 never retry. POSTs to `/config/*` and `/id/*` also skip retry (non-idempotent appends/creates -- retrying could duplicate routes or 409 a half-applied create). POSTs to `/load`, `/adapt`, `/stop` still retry. Hard-capped at 5; values above the cap log a one-time stderr notice so the clamp is visible. Set to `0` to disable. |
 | `CADDY_LOAD_TIMEOUT` | `60000` | Timeout in ms for the `/load` endpoint; raise for ACME-heavy bring-ups where provisioning many certificates can exceed the default. Non-numeric, `<= 0`, or fractional values below 1ms fall back to the default. |
 
 **Alternate MCP clients:**
@@ -234,7 +234,7 @@ npm install
 npm run lint       # Biome check
 npm run lint:fix   # Auto-fix
 npm run build      # tsup bundle
-npm test           # Vitest (182 unit tests; +8 live-Caddy integration tests gated by CADDY_MCP_INTEGRATION=1)
+npm test           # Vitest (188 unit tests; +8 live-Caddy integration tests gated by CADDY_MCP_INTEGRATION=1)
 npm run typecheck  # tsc --noEmit
 ```
 

package/dist/index.js

@@ -24,12 +24,18 @@ function setEtag(path, etag) {
 function getBaseUrl() {
   return (process.env.CADDY_ADMIN_URL || DEFAULT_URL).replace(/\/+$/, "");
 }
+var warnedRetryClamp = false;
 function getMaxRetries() {
   const raw = process.env.CADDY_MAX_RETRIES;
   if (raw === void 0) return 2;
   const n = Number(raw);
   if (!Number.isFinite(n) || n < 0) return 2;
-  return Math.min(Math.floor(n), RETRY_HARD_CAP);
+  const floored = Math.floor(n);
+  if (floored > RETRY_HARD_CAP && !warnedRetryClamp) {
+    warnedRetryClamp = true;
+    console.error(`caddy-mcp: CADDY_MAX_RETRIES=${raw} exceeds hard cap; using ${RETRY_HARD_CAP}.`);
+  }
+  return Math.min(floored, RETRY_HARD_CAP);
 }
 function getHeaders(contentType) {
   const headers = {};
@@ -257,11 +263,12 @@ function formatResult(res) {
 }
 
 // src/tools/operational.ts
+var HTTPS_PORT_RE = /:443(?:\D|$)/;
 function describeServer(raw) {
   const listen = Array.isArray(raw.listen) ? raw.listen : [];
   const routes = Array.isArray(raw.routes) ? raw.routes : [];
   const hasExplicitTls = !!raw.tls_connection_policies;
-  const listensHttps = listen.some((l) => typeof l === "string" && l.includes(":443"));
+  const listensHttps = listen.some((l) => typeof l === "string" && HTTPS_PORT_RE.test(l));
   const tls = hasExplicitTls ? "enabled" : listensHttps ? "auto (HTTPS)" : "off (HTTP only)";
   const listenStr = listen.length > 0 ? listen.map(String).join(", ") : "default";
   return `${routes.length} route(s), listen: ${listenStr}, TLS: ${tls}`;

package/dist/server.js

@@ -22,12 +22,18 @@ function setEtag(path, etag) {
 function getBaseUrl() {
   return (process.env.CADDY_ADMIN_URL || DEFAULT_URL).replace(/\/+$/, "");
 }
+var warnedRetryClamp = false;
 function getMaxRetries() {
   const raw = process.env.CADDY_MAX_RETRIES;
   if (raw === void 0) return 2;
   const n = Number(raw);
   if (!Number.isFinite(n) || n < 0) return 2;
-  return Math.min(Math.floor(n), RETRY_HARD_CAP);
+  const floored = Math.floor(n);
+  if (floored > RETRY_HARD_CAP && !warnedRetryClamp) {
+    warnedRetryClamp = true;
+    console.error(`caddy-mcp: CADDY_MAX_RETRIES=${raw} exceeds hard cap; using ${RETRY_HARD_CAP}.`);
+  }
+  return Math.min(floored, RETRY_HARD_CAP);
 }
 function getHeaders(contentType) {
   const headers = {};
@@ -255,11 +261,12 @@ function formatResult(res) {
 }
 
 // src/tools/operational.ts
+var HTTPS_PORT_RE = /:443(?:\D|$)/;
 function describeServer(raw) {
   const listen = Array.isArray(raw.listen) ? raw.listen : [];
   const routes = Array.isArray(raw.routes) ? raw.routes : [];
   const hasExplicitTls = !!raw.tls_connection_policies;
-  const listensHttps = listen.some((l) => typeof l === "string" && l.includes(":443"));
+  const listensHttps = listen.some((l) => typeof l === "string" && HTTPS_PORT_RE.test(l));
   const tls = hasExplicitTls ? "enabled" : listensHttps ? "auto (HTTPS)" : "off (HTTP only)";
   const listenStr = listen.length > 0 ? listen.map(String).join(", ") : "default";
   return `${routes.length} route(s), listen: ${listenStr}, TLS: ${tls}`;

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@yawlabs/caddy-mcp",
-  "version": "1.2.4",
+  "version": "1.2.6",
   "mcpName": "io.github.YawLabs/caddy-mcp",
   "description": "MCP server for managing Caddy web servers via the admin API",
   "license": "MIT",