@path58/p58-n8n 0.2.7 → 0.2.8

package/CHANGELOG.md

@@ -5,6 +5,13 @@ All notable changes to this project will be documented in this file.
 The format is based on [Keep a Changelog](https://keepachangelog.com/en/1.1.0/),
 and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0.html).
 
+## [0.2.8] - 2026-03-11
+
+### Fixed
+
+- **Webhook path extraction for `build_workflow` workflows** — `extractWebhookPath` now correctly reads the top-level `node.webhookId` set by `addWebhookIdIfNeeded()` in `build_workflow`. Previously it only checked `parameters.webhookId` and `parameters.path`, causing the path to always be null for `build_workflow`-created webhook workflows, leading to wrong webhook URLs.
+- **Webhook 404 after activation (timing)** — `test_workflow` and `execute_workflow` now wait 3 seconds (up from 1.5s) after activating a clone workflow before POSTing. If a 404 is still received, they retry up to 3 times with backoff (1s, 2s). This handles n8n 2.x's async webhook registration.
+
 ## [0.2.7] - 2026-03-11
 
 ### Fixed

package/dist/mcp/server.bundle.cjs

@@ -18473,7 +18473,7 @@ var import_types22 = require("@modelcontextprotocol/sdk/types.js");
 var config = {
   // Server identity
   SERVER_NAME: "p58-n8n",
-  SERVER_VERSION: "0.2.7",
+  SERVER_VERSION: "0.2.8",
   // Database configuration (from environment)
   SUPABASE_URL: process.env.SUPABASE_URL,
   SUPABASE_KEY: process.env.SUPABASE_KEY,
@@ -37464,7 +37464,7 @@ function extractWebhookPath(workflow) {
   if (!trigger)
     return null;
   const params = trigger.parameters ?? {};
-  const shortPath = typeof params.path === "string" && params.path.length > 0 && params.path || typeof params.webhookId === "string" && params.webhookId.length > 0 && params.webhookId || null;
+  const shortPath = typeof params.path === "string" && params.path.length > 0 && params.path || typeof params.webhookId === "string" && params.webhookId.length > 0 && params.webhookId || typeof trigger.webhookId === "string" && trigger.webhookId.length > 0 && trigger.webhookId || null;
   if (!shortPath)
     return null;
   const wfId = workflow.id;
@@ -37580,15 +37580,28 @@ async function testViaClone(workflow, triggerType, testPayload, timeoutMs, apiCo
         result_summary: "Test clone could not be activated \u2014 check credentials in n8n Settings \u2192 Credentials."
       };
     }
-    await new Promise((r) => setTimeout(r, 1500));
-    try {
-      await postWebhook(buildN8nHostUrl(), webhookPath, testPayload, Math.min(timeoutMs, 1e4));
-    } catch (webhookErr) {
-      const errMsg = webhookErr instanceof Error ? webhookErr.message : String(webhookErr);
-      if (!errMsg.includes("timeout"))
-        throw webhookErr;
+    await new Promise((r) => setTimeout(r, 3e3));
+    let webhookPosted = false;
+    for (let attempt = 0; attempt < 3 && !webhookPosted; attempt++) {
+      try {
+        if (attempt > 0)
+          await new Promise((r) => setTimeout(r, 1e3 * attempt));
+        await postWebhook(buildN8nHostUrl(), webhookPath, testPayload, Math.min(timeoutMs, 15e3));
+        webhookPosted = true;
+      } catch (webhookErr) {
+        const errMsg = webhookErr instanceof Error ? webhookErr.message : String(webhookErr);
+        const is404 = errMsg.includes("404") || errMsg.includes("not found");
+        const isTimeout = errMsg.includes("timeout");
+        if (isTimeout) {
+          webhookPosted = true;
+          break;
+        }
+        if (!is404 || attempt === 2)
+          throw webhookErr;
+        import_logging31.logger.debug("test_workflow: webhook 404 on attempt, retrying", { attempt, webhookPath });
+      }
     }
-    await new Promise((r) => setTimeout(r, 1500));
+    await new Promise((r) => setTimeout(r, 1e3));
     const deadline = Date.now() + Math.min(timeoutMs, 3e4);
     let latestExecId = null;
     let nodeDebug;
@@ -37930,15 +37943,27 @@ async function executeViaClone(config3, workflow, inputData, timeoutMs) {
         note: "Execution clone could not be activated. This typically means credentials referenced by the workflow are not configured in n8n. Check n8n Settings \u2192 Credentials."
       };
     }
-    await new Promise((r) => setTimeout(r, 1500));
+    await new Promise((r) => setTimeout(r, 3e3));
     const payload = inputData ?? buildDefaultWebhookPayload();
-    try {
-      await postWebhook(buildN8nHostUrl(), webhookPath, payload, Math.min(timeoutMs, 1e4));
-    } catch (webhookErr) {
-      const errMsg = webhookErr instanceof Error ? webhookErr.message : String(webhookErr);
-      if (!errMsg.includes("timeout"))
-        throw webhookErr;
-      import_logging32.logger.debug("execute_workflow: webhook POST timed out, polling for execution", { cloneId });
+    let webhookPosted = false;
+    for (let attempt = 0; attempt < 3 && !webhookPosted; attempt++) {
+      try {
+        if (attempt > 0)
+          await new Promise((r) => setTimeout(r, 1e3 * attempt));
+        await postWebhook(buildN8nHostUrl(), webhookPath, payload, Math.min(timeoutMs, 15e3));
+        webhookPosted = true;
+      } catch (webhookErr) {
+        const errMsg = webhookErr instanceof Error ? webhookErr.message : String(webhookErr);
+        const is404 = errMsg.includes("404") || errMsg.includes("not found");
+        const isTimeout = errMsg.includes("timeout");
+        if (isTimeout) {
+          webhookPosted = true;
+          break;
+        }
+        if (!is404 || attempt === 2)
+          throw webhookErr;
+        import_logging32.logger.debug("execute_workflow: webhook 404 on attempt, retrying", { attempt, cloneId });
+      }
     }
     const { executionId, node_debug, executionStatus, executionError } = await fetchLatestExecutionDebug(config3, cloneId, timeoutMs);
     if (executionStatus === "success") {

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@path58/p58-n8n",
-  "version": "0.2.7",
+  "version": "0.2.8",
   "description": "The smartest and fastest n8n MCP server — validate, fix, and discover workflows inside your LLM",
   "keywords": [
     "mcp",