@mgsoftwarebv/mg-dashboard-mcp 5.0.1 → 6.0.1

package/dist/index.js

@@ -317,41 +317,19 @@ var TRIGGER_TOOLS = [
     }
   },
   {
-    name: "trigger-run-detail",
-    description: "Get full details of a specific run: status, payload, output, and error stack traces. Use after trigger-runs or trigger-test-task to inspect results.",
+    name: "trigger-run",
+    description: 'Inspect, create, or replay a Trigger.dev run. Pick the mode with `action`:\n- "detail": fetch full run state \u2014 status, payload, output, error stack traces, and logs. Required: runId.\n- "test": kick off a task run and (by default) wait for it to complete, returning the final detail in one shot. Required: taskId. Optional: payload (JSON string), waitSeconds (default 60, max 300, set 0 to fire-and-forget).\n- "replay": re-run a previously failed/completed run with the same payload. Required: runId. Optional: waitSeconds (when >0, also waits for the new run to finish and returns its full detail).\n\nTip: replace any old `trigger-run-detail` / `trigger-test-task` / `trigger-replay-run` calls with this single tool \u2014 output format is identical.',
     inputSchema: {
       type: "object",
       properties: {
-        project: { type: "string", description: "Project slug from trigger-list" },
-        runId: { type: "string", description: "Run ID (e.g. run_xxxxx)" }
-      },
-      required: ["project", "runId"]
-    }
-  },
-  {
-    name: "trigger-test-task",
-    description: "Trigger a task run and wait for it to complete. Returns the final status, output, or error with stack trace. The main tool for testing trigger tasks from Cursor.",
-    inputSchema: {
-      type: "object",
-      properties: {
-        project: { type: "string", description: "Project slug from trigger-list" },
-        taskId: { type: "string", description: 'Task identifier (e.g. "hello-world", "execute-pipeline")' },
-        payload: { type: "string", description: "JSON payload string to pass to the task (optional)" },
-        waitSeconds: { type: "number", description: "Max seconds to wait for completion (default 60, max 300)" }
-      },
-      required: ["project", "taskId"]
-    }
-  },
-  {
-    name: "trigger-replay-run",
-    description: "Replay a previously failed or completed run with the same payload. Returns the new run ID.",
-    inputSchema: {
-      type: "object",
-      properties: {
-        project: { type: "string", description: "Project slug from trigger-list" },
-        runId: { type: "string", description: "Run ID to replay (e.g. run_xxxxx)" }
+        action: { type: "string", enum: ["detail", "test", "replay"], description: "Which run operation to perform." },
+        project: { type: "string", description: 'Project slug from trigger-list (e.g. "mg-dashboard-bHfS")' },
+        runId: { type: "string", description: 'Run ID (e.g. run_xxxxx). Required for action="detail" or action="replay".' },
+        taskId: { type: "string", description: 'Task identifier (e.g. "hello-world"). Required for action="test".' },
+        payload: { type: "string", description: 'JSON payload string passed to the task (action="test" only).' },
+        waitSeconds: { type: "number", description: 'Max seconds to wait for completion. Default 60 (action="test") / 0 (action="replay", fire-and-forget). Max 300. Set 0 to skip the wait.' }
       },
-      required: ["project", "runId"]
+      required: ["action", "project"]
     }
   }
 ];
@@ -359,9 +337,7 @@ var TRIGGER_TOOL_NAMES = new Set(TRIGGER_TOOLS.map((t) => t.name));
 var TRIGGER_TOOL_MODULE_MAP = {
   "trigger-list": "ci_cd",
   "trigger-runs": "ci_cd",
-  "trigger-run-detail": "ci_cd",
-  "trigger-test-task": "ci_cd",
-  "trigger-replay-run": "ci_cd"
+  "trigger-run": "ci_cd"
 };
 async function discoverInstance(projectSlug, conn, proxy, sshExec2) {
   const sql = `SELECT re.\\"apiKey\\" FROM \\"RuntimeEnvironment\\" re JOIN \\"Project\\" p ON re.\\"projectId\\" = p.id WHERE p.slug='${projectSlug}' AND re.slug='prod' LIMIT 1`;
@@ -525,104 +501,69 @@ ${rawJson.substring(0, 500)}` }] };
       return { content: [{ type: "text", text: formatRunsTable(runs) }] };
     }
     // -----------------------------------------------------------------
-    case "trigger-run-detail": {
-      const project = String(args2.project);
-      const runId = String(args2.runId);
-      const instance = await discoverInstance(project, conn, proxy, sshExec2);
-      const [rawJson, logs] = await Promise.all([
-        triggerApi(conn, proxy, sshExec2, instance, "GET", `/api/v3/runs/${encodeURIComponent(runId)}`),
-        fetchRunLogs(runId, conn, proxy, sshExec2)
-      ]);
-      let run;
-      try {
-        run = JSON.parse(rawJson);
-      } catch {
-        return { content: [{ type: "text", text: `Invalid API response:
-${rawJson.substring(0, 500)}` }] };
+    case "trigger-run": {
+      const action = String(args2.action);
+      if (action !== "detail" && action !== "test" && action !== "replay") {
+        return { content: [{ type: "text", text: "Error: action must be one of: detail, test, replay" }] };
       }
-      let text = formatRunDetail(run);
-      if (logs) {
-        text += "\n\n--- Logs ---\n" + logs;
-      }
-      return { content: [{ type: "text", text }] };
-    }
-    // -----------------------------------------------------------------
-    case "trigger-test-task": {
       const project = String(args2.project);
-      const taskId = String(args2.taskId);
-      const waitSeconds = Math.min(Math.max(Number(args2.waitSeconds) || 60, 5), 300);
-      conn.timeout = (waitSeconds + 30) * 1e3;
-      const instance = await discoverInstance(project, conn, proxy, sshExec2);
-      let payload = "{}";
-      if (args2.payload) {
-        try {
-          JSON.parse(String(args2.payload));
-          payload = String(args2.payload);
-        } catch {
-          throw new Error(`Invalid JSON payload: ${String(args2.payload).substring(0, 200)}`);
-        }
+      if (action === "detail") {
+        const runId2 = String(args2.runId ?? "");
+        if (!runId2) return { content: [{ type: "text", text: 'Error: action="detail" requires runId' }] };
+        const instance2 = await discoverInstance(project, conn, proxy, sshExec2);
+        return await fetchAndFormatRun(conn, proxy, sshExec2, instance2, runId2);
       }
-      const triggerBody = JSON.stringify({
-        payload: JSON.parse(payload),
-        options: { tags: ["mcp-test"], test: true }
-      });
-      const triggerJson = await triggerApi(
-        conn,
-        proxy,
-        sshExec2,
-        instance,
-        "POST",
-        `/api/v1/tasks/${encodeURIComponent(taskId)}/trigger`,
-        triggerBody
-      );
-      let triggerResp;
-      try {
-        triggerResp = JSON.parse(triggerJson);
-      } catch {
-        return { content: [{ type: "text", text: `Failed to trigger task. API response:
-${triggerJson.substring(0, 500)}` }] };
-      }
-      const runId = triggerResp.id;
-      if (!runId) {
-        return { content: [{ type: "text", text: `Trigger API did not return a run ID:
-${triggerJson.substring(0, 500)}` }] };
-      }
-      const pollInterval = 3e3;
-      const maxPolls = Math.ceil(waitSeconds * 1e3 / pollInterval);
-      for (let i = 0; i < maxPolls; i++) {
-        await new Promise((r) => setTimeout(r, pollInterval));
-        const pollJson = await triggerApi(
+      if (action === "test") {
+        const taskId = String(args2.taskId ?? "");
+        if (!taskId) return { content: [{ type: "text", text: 'Error: action="test" requires taskId' }] };
+        const waitArg2 = args2.waitSeconds === void 0 ? 60 : Number(args2.waitSeconds);
+        const waitSeconds2 = Math.min(Math.max(Number.isFinite(waitArg2) ? waitArg2 : 60, 0), 300);
+        conn.timeout = (waitSeconds2 + 30) * 1e3;
+        const instance2 = await discoverInstance(project, conn, proxy, sshExec2);
+        let payload = "{}";
+        if (args2.payload) {
+          try {
+            JSON.parse(String(args2.payload));
+            payload = String(args2.payload);
+          } catch {
+            throw new Error(`Invalid JSON payload: ${String(args2.payload).substring(0, 200)}`);
+          }
+        }
+        const triggerBody = JSON.stringify({
+          payload: JSON.parse(payload),
+          options: { tags: ["mcp-test"], test: true }
+        });
+        const triggerJson = await triggerApi(
           conn,
           proxy,
           sshExec2,
-          instance,
-          "GET",
-          `/api/v3/runs/${encodeURIComponent(runId)}`
+          instance2,
+          "POST",
+          `/api/v1/tasks/${encodeURIComponent(taskId)}/trigger`,
+          triggerBody
         );
-        let run;
+        let triggerResp;
         try {
-          run = JSON.parse(pollJson);
+          triggerResp = JSON.parse(triggerJson);
         } catch {
-          continue;
+          return { content: [{ type: "text", text: `Failed to trigger task. API response:
+${triggerJson.substring(0, 500)}` }] };
         }
-        if (TERMINAL_STATUSES.has(run.status)) {
-          let text = formatRunDetail(run);
-          const logs = await fetchRunLogs(runId, conn, proxy, sshExec2);
-          if (logs) text += "\n\n--- Logs ---\n" + logs;
-          return { content: [{ type: "text", text }] };
+        const runId2 = triggerResp.id;
+        if (!runId2) {
+          return { content: [{ type: "text", text: `Trigger API did not return a run ID:
+${triggerJson.substring(0, 500)}` }] };
         }
+        if (waitSeconds2 === 0) {
+          return { content: [{ type: "text", text: `Run triggered. ID: ${runId2} (fire-and-forget; use action="detail" to check).` }] };
+        }
+        return await waitForCompletion(conn, proxy, sshExec2, instance2, runId2, waitSeconds2);
       }
-      return {
-        content: [{
-          type: "text",
-          text: `Run ${runId} did not complete within ${waitSeconds}s (still running). Use trigger-run-detail to check later.`
-        }]
-      };
-    }
-    // -----------------------------------------------------------------
-    case "trigger-replay-run": {
-      const project = String(args2.project);
-      const runId = String(args2.runId);
+      const runId = String(args2.runId ?? "");
+      if (!runId) return { content: [{ type: "text", text: 'Error: action="replay" requires runId' }] };
+      const waitArg = args2.waitSeconds === void 0 ? 0 : Number(args2.waitSeconds);
+      const waitSeconds = Math.min(Math.max(Number.isFinite(waitArg) ? waitArg : 0, 0), 300);
+      if (waitSeconds > 0) conn.timeout = (waitSeconds + 30) * 1e3;
       const instance = await discoverInstance(project, conn, proxy, sshExec2);
       const rawJson = await triggerApi(
         conn,
@@ -639,19 +580,69 @@ ${triggerJson.substring(0, 500)}` }] };
         return { content: [{ type: "text", text: `Replay response:
 ${rawJson.substring(0, 500)}` }] };
       }
-      return {
-        content: [{
-          type: "text",
-          text: result.id ? `Run replayed. New run ID: ${result.id}` : `Replay response:
-${rawJson.substring(0, 500)}`
-        }]
-      };
+      if (!result.id) {
+        return { content: [{ type: "text", text: `Replay response:
+${rawJson.substring(0, 500)}` }] };
+      }
+      if (waitSeconds === 0) {
+        return { content: [{ type: "text", text: `Run replayed. New run ID: ${result.id} (use action="detail" to check status).` }] };
+      }
+      return await waitForCompletion(conn, proxy, sshExec2, instance, result.id, waitSeconds);
     }
     // -----------------------------------------------------------------
     default:
       return { content: [{ type: "text", text: `Unknown trigger tool: ${name}` }] };
   }
 }
+async function fetchAndFormatRun(conn, proxy, sshExec2, instance, runId) {
+  const [rawJson, logs] = await Promise.all([
+    triggerApi(conn, proxy, sshExec2, instance, "GET", `/api/v3/runs/${encodeURIComponent(runId)}`),
+    fetchRunLogs(runId, conn, proxy, sshExec2)
+  ]);
+  let run;
+  try {
+    run = JSON.parse(rawJson);
+  } catch {
+    return { content: [{ type: "text", text: `Invalid API response:
+${rawJson.substring(0, 500)}` }] };
+  }
+  let text = formatRunDetail(run);
+  if (logs) text += "\n\n--- Logs ---\n" + logs;
+  return { content: [{ type: "text", text }] };
+}
+async function waitForCompletion(conn, proxy, sshExec2, instance, runId, waitSeconds) {
+  const pollInterval = 3e3;
+  const maxPolls = Math.ceil(waitSeconds * 1e3 / pollInterval);
+  for (let i = 0; i < maxPolls; i++) {
+    await new Promise((r) => setTimeout(r, pollInterval));
+    const pollJson = await triggerApi(
+      conn,
+      proxy,
+      sshExec2,
+      instance,
+      "GET",
+      `/api/v3/runs/${encodeURIComponent(runId)}`
+    );
+    let run;
+    try {
+      run = JSON.parse(pollJson);
+    } catch {
+      continue;
+    }
+    if (TERMINAL_STATUSES.has(run.status)) {
+      let text = formatRunDetail(run);
+      const logs = await fetchRunLogs(runId, conn, proxy, sshExec2);
+      if (logs) text += "\n\n--- Logs ---\n" + logs;
+      return { content: [{ type: "text", text }] };
+    }
+  }
+  return {
+    content: [{
+      type: "text",
+      text: `Run ${runId} did not complete within ${waitSeconds}s (still running). Use action="detail" with this runId to check later.`
+    }]
+  };
+}
 
 // src/vercel-tools.ts
 var VERCEL_API = "https://api.vercel.com";
@@ -1130,6 +1121,7 @@ async function handleVercelTool(name, args2, deps) {
           sinceExplicit ? Promise.resolve(null) : getDeploymentCreatedMs(token, deploymentId)
         ]);
         const maxWindowMin = 7 * 24 * 60;
+        const autoCapMin = 30;
         let sinceMs;
         let windowNote = "";
         if (sinceExplicit) {
@@ -1138,17 +1130,18 @@ async function handleVercelTool(name, args2, deps) {
           windowNote = `window: last ${capped} min (caller-specified)`;
         } else if (deploymentCreatedMs) {
           const bufferMs = 5 * 6e4;
-          sinceMs = deploymentCreatedMs - bufferMs;
-          const ageMin = Math.max(1, Math.round((Date.now() - sinceMs) / 6e4));
-          if (ageMin > maxWindowMin) {
-            sinceMs = Date.now() - maxWindowMin * 6e4;
-            windowNote = `window: capped to ${maxWindowMin} min (deployment is older than 7 days)`;
-          } else {
+          const sinceDeploymentMs = deploymentCreatedMs - bufferMs;
+          const ageMin = Math.max(1, Math.round((Date.now() - sinceDeploymentMs) / 6e4));
+          if (ageMin <= autoCapMin) {
+            sinceMs = sinceDeploymentMs;
             windowNote = `window: auto ${ageMin} min from deployment createdAt - 5 min buffer`;
+          } else {
+            sinceMs = Date.now() - autoCapMin * 6e4;
+            windowNote = `window: capped to last ${autoCapMin} min (deployment is ${ageMin} min old \u2014 pass sinceMinutes to widen up to ${maxWindowMin})`;
           }
         } else {
-          sinceMs = Date.now() - 60 * 6e4;
-          windowNote = "window: last 60 min (deployment metadata unavailable, used fallback)";
+          sinceMs = Date.now() - autoCapMin * 6e4;
+          windowNote = `window: last ${autoCapMin} min (deployment metadata unavailable, used fallback)`;
         }
         const { logs, error } = await getRuntimeLogs(
           token,
@@ -1161,10 +1154,15 @@ async function handleVercelTool(name, args2, deps) {
           const hint = error.includes("404") || error.includes("400") ? '\n\nThis endpoint requires both project ID and deployment ID and may not be available on every Vercel plan. Use kind="webhooks" or the supabase MCP (vercel_deployment_log table) for archived runtime logs.' : "";
           return { content: [{ type: "text", text: `Error: ${error}${hint}` }] };
         }
-        const text = `${formatRuntimeLogs(logs)}
+        const body = formatRuntimeLogs(logs);
+        const hitDurationLimit = /Exceeded query duration limit/i.test(body);
+        const footer = hitDurationLimit ? `
+
+[${windowNote}]
+[hint] Vercel hit its 5-min query budget for this window. Try a smaller sinceMinutes (e.g. 5-10), lower limit, or use kind="webhooks" / the supabase MCP vercel_deployment_log table for archived logs.` : `
 
 [${windowNote}]`;
-        return { content: [{ type: "text", text }] };
+        return { content: [{ type: "text", text: body + footer }] };
       }
       if (kind === "webhooks") {
         const limit = Math.min(Math.max(Number(args2.limit) || 25, 1), 200);
@@ -1419,9 +1417,7 @@ var TOOL_MODULE_MAP = {
   "env-store": "ci_cd",
   "domain-list": "domains",
   "dns-list": "domains",
-  "dns-create": "domains",
-  "dns-update": "domains",
-  "dns-delete": "domains",
+  "dns-record": "domains",
   ...TRIGGER_TOOL_MODULE_MAP,
   ...VERCEL_TOOL_MODULE_MAP
 };
@@ -3397,7 +3393,10 @@ function formatDbQueryFooter(output, appliedLimit, maxRows, explainMode) {
     return "\n\n[explain] Plan returned, no rows executed.";
   }
   if (!appliedLimit) return "";
-  const rows = output.split("\n").filter((l) => l.trim() && !/^\s*[-+|]/.test(l)).length;
+  const rows = parseRowCountFromOutput(output);
+  if (rows == null) return `
+
+[ok] auto-LIMIT ${maxRows} applied (row count not detected).`;
   if (rows > maxRows) {
     return `
 
@@ -3407,6 +3406,21 @@ function formatDbQueryFooter(output, appliedLimit, maxRows, explainMode) {
 
 [ok] returned ${rows} row(s), under auto-LIMIT ${maxRows}.`;
 }
+function parseRowCountFromOutput(output) {
+  const lines = output.split("\n");
+  for (let i = lines.length - 1; i >= 0; i--) {
+    const l = lines[i]?.trim();
+    if (!l) continue;
+    const m1 = /^\(\s*(\d+)\s+rows?\s*\)$/i.exec(l);
+    if (m1?.[1]) return Number(m1[1]);
+    const m2 = /^(\d+)\s+rows?\s+in\s+set\b/i.exec(l);
+    if (m2?.[1]) return Number(m2[1]);
+    const m3 = /^\(\s*(\d+)\s+rows?\s+affected\s*\)$/i.exec(l);
+    if (m3?.[1]) return Number(m3[1]);
+    if (/[a-zA-Z0-9]/.test(l) && i < lines.length - 3) return null;
+  }
+  return null;
+}
 function assertSafeSql(query) {
   const trimmed = query.trim();
   for (const pattern of BLOCKED_SQL_PATTERNS) {
@@ -3821,7 +3835,7 @@ var TOOLS = [
   // ----- Domains (mijn.host) -----
   {
     name: "domain-list",
-    description: "List all domains from the mijn.host account. Returns domain name, status, renewal date (= expiration), and tags. Requires MIJNHOST_API_KEY.\n\nPass `details: true` to also fetch DNS zone summary per domain in parallel: NS records, MX target, and presence of SPF/DMARC. Useful as a single-call overview instead of N follow-up dns-list calls. Skipped for inactive/expired domains.",
+    description: "List all domains from the mijn.host account. Returns domain name, status, renewal date (= expiration), and tags. Requires MIJNHOST_API_KEY.\n\nPass `details: true` to also fetch DNS zone summary per domain in parallel: MX target(s) and presence of SPF/DMARC TXT records. Useful as a single-call overview instead of N follow-up dns-list calls. Skipped for inactive/expired domains.",
     inputSchema: {
       type: "object",
       properties: {
@@ -3843,51 +3857,22 @@ var TOOLS = [
     }
   },
   {
-    name: "dns-create",
-    description: "Add a new DNS record to a domain. Uses PATCH to add without replacing existing records.\n\nPass `dryRun: true` to preview the proposed change (full before/after diff) without calling the mijn.host PUT \u2014 recommended before touching MX, SPF, DKIM, or DMARC records.",
+    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.',
     inputSchema: {
       type: "object",
       properties: {
+        action: { type: "string", enum: ["create", "update", "delete"], description: "Which mutation to perform." },
         domain: { type: "string", description: "Domain name (e.g. example.com)" },
         type: { type: "string", description: "Record type: A, AAAA, CNAME, MX, TXT, NS, SRV, CAA, or TLSA" },
-        name: { type: "string", description: "Record name (e.g. @ or subdomain)" },
-        value: { type: "string", description: "Record value (e.g. IP address, hostname)" },
-        ttl: { type: "number", description: "TTL in seconds (min 60, default 3600)" },
-        dryRun: { type: "boolean", description: "Preview the change (returns proposed diff) without applying." }
+        name: { type: "string", description: "Record name (e.g. @ or subdomain)." },
+        value: { type: "string", description: "Record value. Required for create + delete. Use newValue for update." },
+        oldValue: { type: "string", description: "Current value of the record (update only) \u2014 used to identify which record to replace." },
+        newValue: { type: "string", description: "New value for the record (update only)." },
+        ttl: { type: "number", description: "TTL in seconds (min 60). Default 3600 for create; defaults to existing TTL for update." },
+        dryRun: { type: "boolean", description: "Preview the change (returns proposed diff) without applying. Recommended for MX/SPF/DKIM/DMARC." }
       },
-      required: ["domain", "type", "name", "value"]
-    }
-  },
-  {
-    name: "dns-update",
-    description: "Update an existing DNS record. Identifies the record by type+name+oldValue, then replaces it with new values via PATCH.\n\nPass `dryRun: true` to preview the change without applying \u2014 strongly recommended for MX/SPF/DKIM/DMARC.",
-    inputSchema: {
-      type: "object",
-      properties: {
-        domain: { type: "string", description: "Domain name (e.g. example.com)" },
-        type: { type: "string", description: "Record type: A, AAAA, CNAME, MX, TXT, NS, SRV, CAA, or TLSA" },
-        name: { type: "string", description: "Record name" },
-        oldValue: { type: "string", description: "Current value of the record to update" },
-        newValue: { type: "string", description: "New value for the record" },
-        ttl: { type: "number", description: "New TTL in seconds (min 60)" },
-        dryRun: { type: "boolean", description: "Preview the change (returns proposed diff) without applying." }
-      },
-      required: ["domain", "type", "name", "oldValue", "newValue"]
-    }
-  },
-  {
-    name: "dns-delete",
-    description: "Delete a DNS record by type, name, and value. Fetches all records, removes the matching one, then replaces the full set.\n\nPass `dryRun: true` to preview the deletion without applying.",
-    inputSchema: {
-      type: "object",
-      properties: {
-        domain: { type: "string", description: "Domain name (e.g. example.com)" },
-        type: { type: "string", description: "Record type to delete" },
-        name: { type: "string", description: "Record name to delete" },
-        value: { type: "string", description: "Record value to delete (must match exactly)" },
-        dryRun: { type: "boolean", description: "Preview the deletion (returns proposed diff) without applying." }
-      },
-      required: ["domain", "type", "name", "value"]
+      required: ["action", "domain", "type", "name"]
     }
   },
   // ----- Trigger.dev -----
@@ -3895,7 +3880,7 @@ var TOOLS = [
   // ----- Vercel -----
   ...VERCEL_TOOLS
 ];
-var MCP_VERSION = "5.0.1";
+var MCP_VERSION = "6.0.1";
 async function handleListTools() {
   if (!authContext) return { tools: TOOLS };
   const accessible = TOOLS.filter((tool) => {
@@ -5147,15 +5132,14 @@ ${lines2.join("\n")}` }] };
               `/domains/${encodeURIComponent(domain)}/dns`
             );
             const recs = r.data.records || [];
-            const ns = recs.filter((x) => x.type === "NS").map((x) => x.value).sort();
             const mx = recs.filter((x) => x.type === "MX").map((x) => x.value).sort();
             const hasSpf = recs.some((x) => x.type === "TXT" && x.value.toLowerCase().includes("v=spf1"));
             const hasDmarc = recs.some(
               (x) => x.type === "TXT" && (x.name.toLowerCase().startsWith("_dmarc") || x.value.toLowerCase().includes("v=dmarc1"))
             );
-            return { ns, mx, hasSpf, hasDmarc };
+            return { mx, hasSpf, hasDmarc };
           } catch (err) {
-            return { ns: [], mx: [], hasSpf: false, hasDmarc: false, error: err instanceof Error ? err.message : String(err) };
+            return { mx: [], hasSpf: false, hasDmarc: false, error: err instanceof Error ? err.message : String(err) };
           }
         }
         const queue = [...activeDomains];
@@ -5174,12 +5158,9 @@ ${lines2.join("\n")}` }] };
           if (!s) return head;
           if (s.error) return `${head}
     dns: error: ${s.error}`;
-          const ns = s.ns.length ? s.ns.join(", ") : "(none)";
           const mx = s.mx.length ? s.mx.join(", ") : "(none)";
-          const mail = `mx=${mx}  spf=${s.hasSpf ? "yes" : "NO"}  dmarc=${s.hasDmarc ? "yes" : "NO"}`;
           return `${head}
-    ns: ${ns}
-    ${mail}`;
+    mx=${mx}  spf=${s.hasSpf ? "yes" : "NO"}  dmarc=${s.hasDmarc ? "yes" : "NO"}`;
         });
         return {
           content: [{
@@ -5211,112 +5192,80 @@ ${header}
 ${sep}
 ${lines.join("\n")}` }] };
       }
-      case "dns-create": {
-        const domain = String(a.domain);
-        const type = String(a.type).toUpperCase();
-        const dnsName = String(a.name);
-        const value = String(a.value);
-        const ttl = Number(a.ttl) || 3600;
-        const dryRun = a.dryRun === true;
-        if (!domain || !type || !dnsName || !value) {
-          throw new Error("domain, type, name, and value are required");
-        }
-        const current = await mijnhostFetch(
-          `/domains/${encodeURIComponent(domain)}/dns`
-        );
-        const newRecord = { type, name: dnsName, value, ttl };
-        const records = [...current.data.records, newRecord];
-        if (dryRun) {
-          const diff = formatDnsDiff(domain, current.data.records, records, {
-            verb: "create",
-            added: [newRecord]
-          });
-          return { content: [{ type: "text", text: diff }] };
-        }
-        await mijnhostFetch(`/domains/${encodeURIComponent(domain)}/dns`, {
-          method: "PUT",
-          body: JSON.stringify({ records })
-        });
-        return { content: [{ type: "text", text: `DNS record created: ${type} ${dnsName} \u2192 ${value} (TTL: ${ttl})` }] };
-      }
-      case "dns-update": {
-        const domain = String(a.domain);
-        const type = String(a.type).toUpperCase();
-        const dnsName = String(a.name);
-        const oldValue = String(a.oldValue);
-        const newValue = String(a.newValue);
-        const ttl = Number(a.ttl) || void 0;
-        const dryRun = a.dryRun === true;
-        if (!domain || !type || !dnsName || !oldValue || !newValue) {
-          throw new Error("domain, type, name, oldValue, and newValue are required");
-        }
-        const current = await mijnhostFetch(
-          `/domains/${encodeURIComponent(domain)}/dns`
-        );
-        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}`);
-        }
-        const updated = [...current.data.records];
-        const existingTtl = updated[idx].ttl;
-        const before = updated[idx];
-        const after = {
-          type,
-          name: dnsName,
-          value: newValue,
-          ttl: ttl ?? existingTtl
-        };
-        updated[idx] = after;
-        if (dryRun) {
-          const diff = formatDnsDiff(domain, current.data.records, updated, {
-            verb: "update",
-            removed: [before],
-            added: [after]
-          });
-          return { content: [{ type: "text", text: diff }] };
+      case "dns-record": {
+        const action = String(a.action);
+        if (action !== "create" && action !== "update" && action !== "delete") {
+          throw new Error("action must be one of: create, update, delete");
         }
-        await mijnhostFetch(`/domains/${encodeURIComponent(domain)}/dns`, {
-          method: "PUT",
-          body: JSON.stringify({ records: updated })
-        });
-        return { content: [{ type: "text", text: `DNS record updated: ${type} ${dnsName} \u2192 ${newValue}${ttl ? ` (TTL: ${ttl})` : ""}` }] };
-      }
-      case "dns-delete": {
         const domain = String(a.domain);
         const type = String(a.type).toUpperCase();
         const dnsName = String(a.name);
-        const value = String(a.value);
         const dryRun = a.dryRun === true;
-        if (!domain || !type || !dnsName || !value) {
-          throw new Error("domain, type, name, and value are required");
+        if (!domain || !type || !dnsName) {
+          throw new Error("domain, type, and name are required");
         }
         const current = await mijnhostFetch(
           `/domains/${encodeURIComponent(domain)}/dns`
         );
-        const before = current.data.records.length;
-        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 (remaining.length === before) {
-          throw new Error(`No matching DNS record found: ${type} ${dnsName} = ${value}`);
+        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}`);
+          }
+          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}`);
+          }
+          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, remaining, {
-            verb: "delete",
-            removed
-          });
+          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: remaining })
+          body: JSON.stringify({ records: nextRecords })
         });
-        return { content: [{ type: "text", text: `DNS record deleted: ${type} ${dnsName} = ${value} (${remaining.length} records remaining)` }] };
+        return { content: [{ type: "text", text: okMessage }] };
       }
       default:
         if (TRIGGER_TOOL_NAMES.has(name)) {