@path58/p58-n8n 0.2.7 → 0.2.9

package/CHANGELOG.md

@@ -5,6 +5,20 @@ 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.9] - 2026-03-11
+
+### Fixed
+
+- **`execute_workflow` `not_found` guidance** — When the webhook POST completes but no execution record is found in n8n (rare timing race), the response now includes actionable guidance to retry instead of the opaque `"Execution status: not_found."` message.
+- **`setup_check` DB health** — Now runs a live `SELECT 1` against the database and reports `db_connected` and `db_error` (exact error string) in the diagnostic output. `catalog_ready` now reflects actual DB reachability instead of always returning `true`. Issues list surfaces DB errors (e.g., circuit breaker) with guidance.
+
+## [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.9",
   // 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") {
@@ -37948,7 +37973,7 @@ async function executeViaClone(config3, workflow, inputData, timeoutMs) {
       } catch {
       }
     }
-    const note = executionStatus === "error" && executionError ? `Execution failed: ${executionError}. See node_debug for the failing node details. Fix the issue and call execute_workflow again.` : executionStatus === "success" ? `Workflow executed successfully via temporary clone. Original workflow (${workflow.id}) is now active.` : `Execution status: ${executionStatus}.`;
+    const note = executionStatus === "error" && executionError ? `Execution failed: ${executionError}. See node_debug for the failing node details. Fix the issue and call execute_workflow again.` : executionStatus === "success" ? `Workflow executed successfully via temporary clone. Original workflow (${workflow.id}) is now active.` : executionStatus === "not_found" ? `Webhook POST completed but no execution record was found in n8n. This can happen when n8n drops the execution before recording it. Retry execute_workflow once \u2014 if it fails again, check n8n logs for errors.` : `Execution status: ${executionStatus}.`;
     return {
       executionId,
       result: { status: executionStatus, node_debug },
@@ -44440,6 +44465,7 @@ When values are collected, marks matching factory.gaps entries as resolved for a
 };
 
 // dist/mcp/tools/handlers/setup-check.js
+init_validatorPostgresClient();
 var SETUP_GUIDE_URL2 = "https://github.com/tsvika58/p58-n8n/blob/main/docs/AGENT_INSTALL.md";
 var N8N_PROBE_TIMEOUT_MS = 2e3;
 var N8N_PROBE_PORTS = [5678, 5679];
@@ -44472,7 +44498,16 @@ function buildToolCounts() {
   const tier1 = OFFLINE_TOOLS.size;
   return { tier1, tier2_7: total - tier1 };
 }
-function collectIssues(apiKeySet, n8nConnected, probes) {
+async function checkDbHealth() {
+  try {
+    await validatorQuery("SELECT 1 AS ok", []);
+    return { connected: true };
+  } catch (err) {
+    const msg = err instanceof Error ? err.message : String(err);
+    return { connected: false, error: msg };
+  }
+}
+function collectIssues(apiKeySet, n8nConnected, probes, dbConnected, dbError) {
   const issues = [];
   if (!apiKeySet) {
     issues.push("N8N_API_KEY not set \u2014 Tier 2-7 deploy tools unavailable");
@@ -44490,6 +44525,10 @@ function collectIssues(apiKeySet, n8nConnected, probes) {
   if (apiKeySet && !n8nConnected) {
     issues.push("N8N_API_KEY is set but n8n is unreachable \u2014 check N8N_API_URL and ensure n8n is running");
   }
+  if (!dbConnected) {
+    const detail = dbError ? `: ${dbError}` : "";
+    issues.push(`Database unavailable${detail} \u2014 catalog features degraded, pattern fallback active`);
+  }
   return issues;
 }
 async function buildReport() {
@@ -44502,7 +44541,8 @@ async function buildReport() {
   if (!n8nUrl) {
     probes = await discoverN8nInstances();
   }
-  const issues = collectIssues(apiKeySet, n8nConnected, probes);
+  const dbHealth = await checkDbHealth();
+  const issues = collectIssues(apiKeySet, n8nConnected, probes, dbHealth.connected, dbHealth.error);
   const toolCounts = buildToolCounts();
   const report = {
     server_version: config.SERVER_VERSION,
@@ -44511,7 +44551,9 @@ async function buildReport() {
     n8n_connected: n8nConnected,
     n8n_api_key_set: apiKeySet,
     credential_count: credentialCount,
-    catalog_ready: true,
+    catalog_ready: dbHealth.connected,
+    db_connected: dbHealth.connected,
+    ...dbHealth.error ? { db_error: dbHealth.error } : {},
     tools_available: toolCounts,
     issues,
     setup_guide: SETUP_GUIDE_URL2

package/package.json

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