@path58/p58-n8n 0.2.6 → 0.2.8

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.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
+
+- **LangChain connection type pattern fallback** — `ConnectionTypeResolver` now includes a static prefix-based resolver that correctly identifies LangChain AI sub-node types (`ai_languageModel`, `ai_tool`, `ai_memory`, etc.) even when the database is unreachable (e.g., Supabase circuit breaker open). Previously, any DB connectivity failure caused all connections to silently default to `"main"`, breaking LangChain workflows.
+- Error message for DB load failures now surfaces the actual error string for easier diagnostics
+
 ## [0.2.6] - 2026-03-11
 
 ### Added
@@ -150,6 +164,7 @@ and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0
 - npm package published as `@path58/p58-n8n`
 - ESM module support with shebang for direct `npx` execution
 
+[0.2.7]: https://github.com/tsvika58/p58-n8n/releases/tag/v0.2.7
 [0.2.6]: https://github.com/tsvika58/p58-n8n/releases/tag/v0.2.6
 [0.2.5]: https://github.com/tsvika58/p58-n8n/releases/tag/v0.2.5
 [0.2.4]: https://github.com/tsvika58/p58-n8n/releases/tag/v0.2.4

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.6",
+  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") {
@@ -41711,6 +41736,27 @@ var AI_TYPE_TO_CONNECTION_TYPE = {
   reranker: "ai_reranker"
 };
 var DEFAULT_CONNECTION_TYPE = "main";
+var LANGCHAIN_PREFIX_TO_CONNECTION = [
+  ["@n8n/n8n-nodes-langchain.lmchat", "ai_languageModel"],
+  ["@n8n/n8n-nodes-langchain.lm", "ai_languageModel"],
+  ["@n8n/n8n-nodes-langchain.memory", "ai_memory"],
+  ["@n8n/n8n-nodes-langchain.tool", "ai_tool"],
+  ["@n8n/n8n-nodes-langchain.embeddings", "ai_embedding"],
+  ["@n8n/n8n-nodes-langchain.outputparser", "ai_outputParser"],
+  ["@n8n/n8n-nodes-langchain.retriever", "ai_retriever"],
+  ["@n8n/n8n-nodes-langchain.document", "ai_document"],
+  ["@n8n/n8n-nodes-langchain.textsplitter", "ai_textSplitter"],
+  ["@n8n/n8n-nodes-langchain.vectorstore", "ai_vectorStore"],
+  ["@n8n/n8n-nodes-langchain.reranker", "ai_reranker"]
+];
+function resolveByPattern(nodeType) {
+  const lower = nodeType.toLowerCase();
+  for (const [prefix, connType] of LANGCHAIN_PREFIX_TO_CONNECTION) {
+    if (lower.startsWith(prefix))
+      return connType;
+  }
+  return void 0;
+}
 async function loadConnectionRules2() {
   const { rows } = await (0, import_retry7.retryDbQuery)(() => validatorQuery(`SELECT LOWER(source_node_type) AS source_node_type,
               LOWER(target_node_type) AS target_node_type,
@@ -41770,7 +41816,8 @@ var ConnectionTypeResolver = class _ConnectionTypeResolver {
       });
       return new _ConnectionTypeResolver(buildRuleMap(ruleRows), buildAITypeMap(aiNodeRows));
     } catch (err) {
-      import_logging53.logger.warn('connectionTypeResolver: DB load failed, defaulting all connections to "main"', { error: err });
+      const msg = err instanceof Error ? err.message : String(err);
+      import_logging53.logger.warn("connectionTypeResolver: DB load failed \u2014 pattern fallback active for LangChain nodes", { error: msg });
       return new _ConnectionTypeResolver(/* @__PURE__ */ new Map(), /* @__PURE__ */ new Map());
     }
   }
@@ -41794,6 +41841,9 @@ var ConnectionTypeResolver = class _ConnectionTypeResolver {
       const aiType = this.aiTypeMap.get(sourceNodeType);
       if (aiType)
         return aiType;
+      const patternType = resolveByPattern(sourceNodeType);
+      if (patternType)
+        return patternType;
     }
     return DEFAULT_CONNECTION_TYPE;
   }

package/package.json

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