@kadoa/mcp 0.3.5-rc.4 → 0.3.5-rc.6

package/dist/index.js

@@ -49167,6 +49167,9 @@ var init_client2 = __esm(() => {
 });
 
 // src/tools.ts
+function strictSchema(shape) {
+  return exports_external.object(shape).strict();
+}
 function coerceArray(wrapPlainString = false) {
   return (val) => {
     if (typeof val === "string") {
@@ -49258,6 +49261,24 @@ function classifyError(error48) {
   return "Unknown error";
 }
 function registerTools(server, ctx) {
+  async function elicit(message, schema, required2) {
+    try {
+      const result = await server.server.elicitInput({
+        message,
+        requestedSchema: {
+          type: "object",
+          properties: schema,
+          ...required2 ? { required: required2 } : {}
+        }
+      });
+      if (result.action === "accept" && result.content) {
+        return result.content;
+      }
+      return null;
+    } catch {
+      return null;
+    }
+  }
   function withErrorHandling(name, handler) {
     return async (...args) => {
       try {
@@ -49298,8 +49319,8 @@ function registerTools(server, ctx) {
 ` + `• REAL_TIME workflows are fundamentally different from regular extraction workflows. They route to a separate Observer service, run continuously, and cannot be triggered manually.
 ` + `• A regular workflow CANNOT be converted to real-time after creation (and vice versa). The interval=REAL_TIME flag MUST be set at creation time.
 ` + `• If the user wants to monitor a page for changes, set interval='REAL_TIME' here. Do NOT create a regular workflow and then try to change its interval to REAL_TIME via update_workflow — this will be rejected.
-` + "• When unclear whether the user wants monitoring (real-time) or data extraction (regular), ask them to clarify before creating the workflow.",
-    inputSchema: {
+` + "• When the user says 'monitor', 'track', 'watch', or 'alert me' — ASK if they want real-time continuous monitoring (interval='REAL_TIME') or scheduled extraction. Do NOT assume one or the other.",
+    inputSchema: strictSchema({
       prompt: exports_external.string().describe('Natural language description of what to extract (e.g., "Extract product prices and names")'),
       url: exports_external.string().optional().describe("Single URL — prefer using 'urls' instead. If both are provided, 'urls' takes precedence."),
       urls: exports_external.preprocess(coerceArray(true), exports_external.array(exports_external.string()).min(1)).optional().describe("Starting URLs for the workflow (array of strings). Also accepts a single URL string."),
@@ -49336,16 +49357,17 @@ function registerTools(server, ctx) {
         }).optional().describe("Send notifications to a Slack channel"),
         websocket: exports_external.boolean().optional().describe("Enable WebSocket notifications for programmatic real-time consumption")
       }).optional().describe("Notification channels to alert when data changes. At least one channel is required for REAL_TIME workflows.")
-    },
+    }),
     annotations: { readOnlyHint: false, destructiveHint: false, idempotentHint: false }
   }, withErrorHandling("create_workflow", async (args) => {
     const urls = args.urls ?? (args.url ? [args.url] : []);
     if (urls.length === 0) {
       return errorResult("At least one URL is required. Provide 'urls' (array) or 'url' (string).");
     }
+    const interval = args.interval;
     const n = args.notifications;
     const hasNotifications = n && (n.email || n.webhook || n.slack || n.websocket);
-    if (args.interval === "REAL_TIME" && !hasNotifications) {
+    if (interval === "REAL_TIME" && !hasNotifications) {
       return errorResult("Real-time workflows require at least one notification channel so you get alerted when data changes. " + "Add a notifications object with one or more channels: email, webhook, slack, or websocket.");
     }
     const extraction = args.schema ? (builder2) => {
@@ -49355,14 +49377,14 @@ function registerTools(server, ctx) {
       }
       return entity;
     } : undefined;
-    const isRealTime = args.interval === "REAL_TIME";
+    const isRealTime = interval === "REAL_TIME";
     let builder = ctx.client.extract({
       urls,
       name: args.name || "Untitled Workflow",
       ...isRealTime ? {} : { navigationMode: "agentic-navigation" },
       userPrompt: args.prompt,
       extraction,
-      interval: args.interval
+      interval
     });
     if (hasNotifications) {
       const channels = {};
@@ -49401,7 +49423,7 @@ function registerTools(server, ctx) {
     if (n?.websocket)
       enabledChannels.push("websocket");
     let message = "Workflow created successfully.";
-    if (args.interval === "REAL_TIME") {
+    if (interval === "REAL_TIME") {
       message += " This real-time workflow monitors continuously — no manual runs needed.";
     } else {
       message += " The AI agent will start extracting data automatically.";
@@ -49516,16 +49538,40 @@ function registerTools(server, ctx) {
     });
   }));
   server.registerTool("delete_workflow", {
-    description: "Delete a workflow permanently",
+    description: "Delete a workflow permanently. This is irreversible. " + "You MUST first call this tool without confirmed=true to preview what will be deleted, " + "then show the user the workflow name and ask for confirmation, " + "then call again with confirmed=true only after the user explicitly agrees.",
     inputSchema: {
-      workflowId: exports_external.string().describe("The workflow ID to delete")
+      workflowId: exports_external.string().describe("The workflow ID to delete"),
+      confirmed: exports_external.boolean().optional().describe("Set to true only after the user has explicitly confirmed deletion. Omit or set to false for the initial preview call.")
     },
     annotations: { readOnlyHint: false, destructiveHint: true, idempotentHint: true }
   }, withErrorHandling("delete_workflow", async (args) => {
+    let workflowName = args.workflowId;
+    let workflowStatus = "unknown";
+    try {
+      const workflow = await ctx.client.workflow.get(args.workflowId);
+      workflowName = workflow.name || args.workflowId;
+      workflowStatus = workflow.displayState ?? workflow.state ?? "unknown";
+    } catch {}
+    if (!args.confirmed) {
+      const confirmation = await elicit(`Are you sure you want to permanently delete "${workflowName}" (${workflowStatus})? This cannot be undone.`, { confirm: { type: "boolean", description: "Confirm deletion" } }, ["confirm"]);
+      if (confirmation !== null) {
+        if (!confirmation.confirm) {
+          return jsonResult({ success: false, message: "Deletion cancelled by user." });
+        }
+      } else {
+        return jsonResult({
+          pending: true,
+          workflowId: args.workflowId,
+          workflowName,
+          status: workflowStatus,
+          message: `⚠️ You are about to permanently delete "${workflowName}" (${workflowStatus}). This cannot be undone. Ask the user to confirm, then call delete_workflow again with confirmed=true.`
+        });
+      }
+    }
     await ctx.client.workflow.delete(args.workflowId);
     return jsonResult({
       success: true,
-      message: "Workflow deleted successfully"
+      message: `Workflow "${workflowName}" deleted successfully`
     });
   }));
   server.registerTool("approve_workflow", {
@@ -49546,7 +49592,7 @@ function registerTools(server, ctx) {
     description: `Update a workflow's configuration. All fields are optional — only provided fields will be updated. Use this to change the name, URLs, extraction schema, entity, prompt, schedule, or other metadata.
 
 ` + "IMPORTANT: You cannot change a workflow's interval to or from REAL_TIME. Real-time workflows are architecturally different (Observer service vs Scraper) and must be created as real-time from the start using create_workflow with interval='REAL_TIME'. To switch a workflow between real-time and regular, delete it and create a new one with the correct interval.",
-    inputSchema: {
+    inputSchema: strictSchema({
       workflowId: exports_external.string().describe("The workflow ID to update"),
       name: exports_external.string().optional().describe("New name for the workflow"),
       urls: exports_external.preprocess(coerceArray(true), exports_external.array(exports_external.string()).min(1)).optional().describe("New target URLs for the workflow (array of strings). Also accepts a single URL string."),
@@ -49576,7 +49622,7 @@ function registerTools(server, ctx) {
       ]).optional().describe("How often the workflow should run. CUSTOM requires schedules. Note: REAL_TIME is NOT allowed here — real-time workflows must be created via create_workflow."),
       schedules: exports_external.preprocess(coerceArray(true), exports_external.array(exports_external.string())).optional().describe("Cron expressions for CUSTOM update interval"),
       limit: exports_external.number().optional().describe("Maximum number of records to extract per run")
-    },
+    }),
     annotations: { readOnlyHint: false, destructiveHint: false, idempotentHint: true }
   }, withErrorHandling("update_workflow", async (args) => {
     const { workflowId, ...updates } = args;
@@ -49649,12 +49695,27 @@ function registerTools(server, ctx) {
     });
   }));
   server.registerTool("delete_notification_channel", {
-    description: "Delete a notification channel by ID",
+    description: "Delete a notification channel by ID. " + "Call first without confirmed=true to preview, then with confirmed=true after user agrees.",
     inputSchema: {
-      channelId: exports_external.string().describe("The channel ID to delete")
+      channelId: exports_external.string().describe("The channel ID to delete"),
+      confirmed: exports_external.boolean().optional().describe("Set to true only after the user has explicitly confirmed deletion.")
     },
     annotations: { readOnlyHint: false, destructiveHint: true, idempotentHint: true }
   }, withErrorHandling("delete_notification_channel", async (args) => {
+    if (!args.confirmed) {
+      const confirmation = await elicit(`Are you sure you want to delete notification channel "${args.channelId}"?`, { confirm: { type: "boolean", description: "Confirm deletion" } }, ["confirm"]);
+      if (confirmation !== null) {
+        if (!confirmation.confirm) {
+          return jsonResult({ success: false, message: "Deletion cancelled by user." });
+        }
+      } else {
+        return jsonResult({
+          pending: true,
+          channelId: args.channelId,
+          message: `⚠️ About to delete notification channel "${args.channelId}". Ask the user to confirm, then call again with confirmed=true.`
+        });
+      }
+    }
     await ctx.client.notification.channels.deleteChannel(args.channelId);
     return jsonResult({
       success: true,
@@ -49765,12 +49826,27 @@ function registerTools(server, ctx) {
     });
   }));
   server.registerTool("delete_notification_setting", {
-    description: "Delete a notification setting by ID. This removes the event-to-channel mapping but does not delete the channels themselves.",
+    description: "Delete a notification setting by ID. This removes the event-to-channel mapping but does not delete the channels themselves. " + "Call first without confirmed=true to preview, then with confirmed=true after user agrees.",
     inputSchema: {
-      settingsId: exports_external.string().describe("The notification settings ID to delete")
+      settingsId: exports_external.string().describe("The notification settings ID to delete"),
+      confirmed: exports_external.boolean().optional().describe("Set to true only after the user has explicitly confirmed deletion.")
     },
     annotations: { readOnlyHint: false, destructiveHint: true, idempotentHint: true }
   }, withErrorHandling("delete_notification_setting", async (args) => {
+    if (!args.confirmed) {
+      const confirmation = await elicit(`Are you sure you want to delete notification setting "${args.settingsId}"?`, { confirm: { type: "boolean", description: "Confirm deletion" } }, ["confirm"]);
+      if (confirmation !== null) {
+        if (!confirmation.confirm) {
+          return jsonResult({ success: false, message: "Deletion cancelled by user." });
+        }
+      } else {
+        return jsonResult({
+          pending: true,
+          settingsId: args.settingsId,
+          message: `⚠️ About to delete notification setting "${args.settingsId}". Ask the user to confirm, then call again with confirmed=true.`
+        });
+      }
+    }
     await ctx.client.notification.settings.deleteSettings(args.settingsId);
     return jsonResult({
       success: true,

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@kadoa/mcp",
-  "version": "0.3.5-rc.4",
+  "version": "0.3.5-rc.6",
   "description": "Kadoa MCP Server — manage workflows from Claude Desktop, Cursor, and other MCP clients",
   "type": "module",
   "main": "dist/index.js",