@mgsoftwarebv/mg-dashboard-mcp 6.5.0 → 6.6.0

package/dist/index.js

@@ -460,7 +460,7 @@ var connectionConfig = {
     }
   },
   connection: {
-    application_name: `mg-dashboard-${process.env.VERCEL ? "vercel" : process.env.TRIGGER_RUN_ID ? "trigger" : "local"}`
+    application_name: `mg-dashboard-${process.env.TRIGGER_RUN_ID ? "trigger" : "local"}`
   }
 };
 function getDatabaseUrl() {
@@ -3924,7 +3924,7 @@ async function mijnhostFetch(path, options = {}) {
       "API-Key": key,
       "Accept": "application/json",
       "Content-Type": "application/json",
-      "User-Agent": "mg-dashboard-mcp/2.2.0",
+      "User-Agent": "mg-dashboard-mcp/6.6.0",
       ...options.headers || {}
     }
   });
@@ -3935,6 +3935,36 @@ async function mijnhostFetch(path, options = {}) {
   }
   return body;
 }
+var DNS_MIN_INTERVAL_MS = 750;
+var DNS_RETRY_BACKOFF_MS = 1500;
+var dnsZoneQueue = /* @__PURE__ */ new Map();
+var dnsLastPutAt = /* @__PURE__ */ new Map();
+function sleep(ms) {
+  return new Promise((resolve) => setTimeout(resolve, ms));
+}
+async function withDnsZoneLock(domain, fn) {
+  const key = domain.toLowerCase();
+  const previous = dnsZoneQueue.get(key) ?? Promise.resolve();
+  const run = previous.then(async () => {
+    const last = dnsLastPutAt.get(key) ?? 0;
+    const wait = DNS_MIN_INTERVAL_MS - (Date.now() - last);
+    if (wait > 0) await sleep(wait);
+    return fn();
+  });
+  const queued = run.finally(() => {
+    if (dnsZoneQueue.get(key) === queued) dnsZoneQueue.delete(key);
+  });
+  dnsZoneQueue.set(key, queued.catch(() => {
+  }));
+  return run;
+}
+function markDnsPut(domain) {
+  dnsLastPutAt.set(domain.toLowerCase(), Date.now());
+}
+function isTransientMijnhostError(err) {
+  const msg = err instanceof Error ? err.message : String(err);
+  return /Internal Server Error|API error: 5\d\d|fetch failed|ETIMEDOUT|ECONNRESET/i.test(msg);
+}
 var TOOLS = [
   {
     name: "list-servers",
@@ -4298,7 +4328,7 @@ var TOOLS = [
   },
   {
     name: "dns-record",
-    description: 'Mutate a single DNS record on a mijn.host domain. Pick the mutation with `action`:\n- "create": add a new record. Required: type, name, value. Optional: ttl (default 3600).\n- "update": replace an existing record. Required: type, name, oldValue, newValue. Optional: ttl.\n- "delete": remove a record. Required: type, name, value.\n\nAlways pass `dryRun: true` first when touching MX / SPF / DKIM / DMARC \u2014 returns a full before/after diff with a mail-auth warning and applies nothing. Re-run without dryRun once the diff looks correct.\n\nUse `dns-list` to inspect the current zone first if you need to identify the right `oldValue`. Requires MIJNHOST_API_KEY.',
+    description: 'Mutate a single DNS record on a mijn.host domain. Pick the mutation with `action`:\n- "create": add a new record. Required: type, name, value. Optional: ttl (default 3600).\n- "update": replace an existing record. Required: type, name, oldValue, newValue. Optional: ttl.\n- "delete": remove a record. Required: type, name, value.\n\nAlways pass `dryRun: true` first when touching MX / SPF / DKIM / DMARC \u2014 returns a full before/after diff with a mail-auth warning and applies nothing. Re-run without dryRun once the diff looks correct.\n\nUse `dns-list` to inspect the current zone first if you need to identify the right `oldValue`. Requires MIJNHOST_API_KEY.\n\nConcurrency: calls are automatically serialized per domain (mijn.host\'s API does GET-then-PUT-the-whole-zone, so parallel calls would race and silently drop records). A minimum 750 ms gap is enforced between PUTs to the same zone, transient 5xx errors are retried once, and a stale-snapshot guard refuses to PUT a zone that suddenly shrunk. You can still fire calls in parallel \u2014 they just queue safely behind each other per domain.',
     inputSchema: {
       type: "object",
       properties: {
@@ -4320,7 +4350,7 @@ var TOOLS = [
   // ----- Repo reference -----
   ...REPO_TOOLS
 ];
-var MCP_VERSION = "6.3.0";
+var MCP_VERSION = "6.6.0";
 async function handleListTools() {
   if (!authContext) return { tools: TOOLS };
   const accessible = TOOLS.filter((tool) => {
@@ -5985,74 +6015,102 @@ ${lines.join("\n")}` }] };
         if (!domain || !type || !dnsName) {
           throw new Error("domain, type, and name are required");
         }
-        const current = await mijnhostFetch(
-          `/domains/${encodeURIComponent(domain)}/dns`
-        );
-        let nextRecords;
-        let diffArgs;
-        let okMessage;
-        if (action === "create") {
-          const value = typeof a.value === "string" ? a.value : "";
-          if (!value) throw new Error('action="create" requires value');
-          const ttl = Number(a.ttl) || 3600;
-          const newRecord = { type, name: dnsName, value, ttl };
-          nextRecords = [...current.data.records, newRecord];
-          diffArgs = { verb: "create", added: [newRecord] };
-          okMessage = `DNS record created: ${type} ${dnsName} \u2192 ${value} (TTL: ${ttl})`;
-        } else if (action === "update") {
-          const oldValue = typeof a.oldValue === "string" ? a.oldValue : "";
-          const newValue = typeof a.newValue === "string" ? a.newValue : "";
-          if (!oldValue || !newValue) throw new Error('action="update" requires oldValue and newValue');
-          const ttlArg = a.ttl !== void 0 ? Number(a.ttl) : void 0;
-          const idx = current.data.records.findIndex(
-            (r) => r.type === type && r.name === dnsName && r.value === oldValue
+        return withDnsZoneLock(domain, async () => {
+          const fetchZone = () => mijnhostFetch(
+            `/domains/${encodeURIComponent(domain)}/dns`
           );
-          if (idx === -1) {
-            throw new Error(
-              `No matching DNS record found: ${type} ${dnsName} = ${oldValue}
+          let current = await fetchZone();
+          if (current.data.records.length === 0) {
+            await sleep(DNS_RETRY_BACKOFF_MS);
+            current = await fetchZone();
+          }
+          let nextRecords;
+          let diffArgs;
+          let okMessage;
+          if (action === "create") {
+            const value = typeof a.value === "string" ? a.value : "";
+            if (!value) throw new Error('action="create" requires value');
+            const ttl = Number(a.ttl) || 3600;
+            const newRecord = { type, name: dnsName, value, ttl };
+            nextRecords = [...current.data.records, newRecord];
+            diffArgs = { verb: "create", added: [newRecord] };
+            okMessage = `DNS record created: ${type} ${dnsName} \u2192 ${value} (TTL: ${ttl})`;
+          } else if (action === "update") {
+            const oldValue = typeof a.oldValue === "string" ? a.oldValue : "";
+            const newValue = typeof a.newValue === "string" ? a.newValue : "";
+            if (!oldValue || !newValue) throw new Error('action="update" requires oldValue and newValue');
+            const ttlArg = a.ttl !== void 0 ? Number(a.ttl) : void 0;
+            const idx = current.data.records.findIndex(
+              (r) => r.type === type && r.name === dnsName && r.value === oldValue
+            );
+            if (idx === -1) {
+              throw new Error(
+                `No matching DNS record found: ${type} ${dnsName} = ${oldValue}
 ` + describeDnsCandidates(current.data.records, type, dnsName, oldValue)
+              );
+            }
+            const updated = [...current.data.records];
+            const before = updated[idx];
+            const after = {
+              type,
+              name: dnsName,
+              value: newValue,
+              ttl: ttlArg ?? before.ttl
+            };
+            updated[idx] = after;
+            nextRecords = updated;
+            diffArgs = { verb: "update", removed: [before], added: [after] };
+            okMessage = `DNS record updated: ${type} ${dnsName} \u2192 ${newValue}${ttlArg ? ` (TTL: ${ttlArg})` : ""}`;
+          } else {
+            const value = typeof a.value === "string" ? a.value : "";
+            if (!value) throw new Error('action="delete" requires value');
+            const removed = current.data.records.filter(
+              (r) => r.type === type && r.name === dnsName && r.value === value
+            );
+            const remaining = current.data.records.filter(
+              (r) => !(r.type === type && r.name === dnsName && r.value === value)
             );
+            if (removed.length === 0) {
+              throw new Error(
+                `No matching DNS record found: ${type} ${dnsName} = ${value}
+` + describeDnsCandidates(current.data.records, type, dnsName, value)
+              );
+            }
+            nextRecords = remaining;
+            diffArgs = { verb: "delete", removed };
+            okMessage = `DNS record deleted: ${type} ${dnsName} = ${value} (${remaining.length} records remaining)`;
           }
-          const updated = [...current.data.records];
-          const before = updated[idx];
-          const after = {
-            type,
-            name: dnsName,
-            value: newValue,
-            ttl: ttlArg ?? before.ttl
-          };
-          updated[idx] = after;
-          nextRecords = updated;
-          diffArgs = { verb: "update", removed: [before], added: [after] };
-          okMessage = `DNS record updated: ${type} ${dnsName} \u2192 ${newValue}${ttlArg ? ` (TTL: ${ttlArg})` : ""}`;
-        } else {
-          const value = typeof a.value === "string" ? a.value : "";
-          if (!value) throw new Error('action="delete" requires value');
-          const removed = current.data.records.filter(
-            (r) => r.type === type && r.name === dnsName && r.value === value
-          );
-          const remaining = current.data.records.filter(
-            (r) => !(r.type === type && r.name === dnsName && r.value === value)
-          );
-          if (removed.length === 0) {
+          if (dryRun) {
+            const diff = formatDnsDiff(domain, current.data.records, nextRecords, diffArgs);
+            return { content: [{ type: "text", text: diff }] };
+          }
+          if (nextRecords.length < 3 && current.data.records.length >= 5 && action !== "delete") {
             throw new Error(
-              `No matching DNS record found: ${type} ${dnsName} = ${value}
-` + describeDnsCandidates(current.data.records, type, dnsName, value)
+              `Aborting PUT: would shrink ${domain} from ${current.data.records.length} to ${nextRecords.length} records. This usually indicates a stale zone snapshot from mijn.host. Run dns-list and retry.`
             );
           }
-          nextRecords = remaining;
-          diffArgs = { verb: "delete", removed };
-          okMessage = `DNS record deleted: ${type} ${dnsName} = ${value} (${remaining.length} records remaining)`;
-        }
-        if (dryRun) {
-          const diff = formatDnsDiff(domain, current.data.records, nextRecords, diffArgs);
-          return { content: [{ type: "text", text: diff }] };
-        }
-        await mijnhostFetch(`/domains/${encodeURIComponent(domain)}/dns`, {
-          method: "PUT",
-          body: JSON.stringify({ records: nextRecords })
+          try {
+            await mijnhostFetch(`/domains/${encodeURIComponent(domain)}/dns`, {
+              method: "PUT",
+              body: JSON.stringify({ records: nextRecords })
+            });
+          } catch (err) {
+            if (!isTransientMijnhostError(err)) throw err;
+            await sleep(DNS_RETRY_BACKOFF_MS);
+            const refetched = await fetchZone();
+            const stillNeedsApply = action === "create" ? !refetched.data.records.some((r) => r.type === type && r.name === dnsName && r.value === (typeof a.value === "string" ? a.value : "")) : true;
+            if (!stillNeedsApply) {
+              markDnsPut(domain);
+              return { content: [{ type: "text", text: `${okMessage} (already applied; recovered from transient error)` }] };
+            }
+            await mijnhostFetch(`/domains/${encodeURIComponent(domain)}/dns`, {
+              method: "PUT",
+              body: JSON.stringify({ records: nextRecords })
+            });
+          }
+          markDnsPut(domain);
+          return { content: [{ type: "text", text: okMessage }] };
         });
-        return { content: [{ type: "text", text: okMessage }] };
       }
       default:
         if (TRIGGER_TOOL_NAMES.has(name)) {