@mgsoftwarebv/mg-dashboard-mcp 5.0.1 → 6.0.0

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)}` }] };
-      }
-      let text = formatRunDetail(run);
-      if (logs) {
-        text += "\n\n--- Logs ---\n" + logs;
+    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" }] };
       }
-      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";
@@ -1419,9 +1410,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
 };
@@ -3843,51 +3832,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.",
-    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 (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." }
-      },
-      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.",
+    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" },
-        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." }
+        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", "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 +3855,7 @@ var TOOLS = [
   // ----- Vercel -----
   ...VERCEL_TOOLS
 ];
-var MCP_VERSION = "5.0.1";
+var MCP_VERSION = "6.0.0";
 async function handleListTools() {
   if (!authContext) return { tools: TOOLS };
   const accessible = TOOLS.filter((tool) => {
@@ -5211,112 +5171,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)) {