@path58/p58-n8n 0.2.8 → 0.2.10

package/CHANGELOG.md

@@ -5,6 +5,19 @@ 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.10] - 2026-03-11
+
+### Fixed
+
+- **Webhook URL construction bug** — `extractWebhookPath` incorrectly prefixed `{workflowId}/{encodedNodeName}/` to the webhook path, producing URLs like `/webhook/4aDOC5YlrPOTUTgm/webhook/langchain-test-029` instead of `/webhook/langchain-test-029`. This caused 404 errors on every webhook-triggered `test_workflow` and `execute_workflow` call. n8n production webhooks listen at `/webhook/{path}` with no prefix.
+
+## [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

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.8",
+  SERVER_VERSION: "0.2.10",
   // Database configuration (from environment)
   SUPABASE_URL: process.env.SUPABASE_URL,
   SUPABASE_KEY: process.env.SUPABASE_KEY,
@@ -37464,16 +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 || typeof trigger.webhookId === "string" && trigger.webhookId.length > 0 && trigger.webhookId || null;
-  if (!shortPath)
-    return null;
-  const wfId = workflow.id;
-  const nodeName = trigger.name ?? "Webhook";
-  if (wfId) {
-    const encodedNodeName = encodeURIComponent(String(nodeName).toLowerCase());
-    return `${wfId}/${encodedNodeName}/${shortPath}`;
-  }
-  return shortPath;
+  return 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;
 }
 function buildN8nHostUrl() {
   const apiBase = process.env.N8N_API_BASE_URL ?? process.env.N8N_API_URL ?? "http://localhost:5678/api/v1";
@@ -37973,7 +37964,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 },
@@ -44465,6 +44456,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];
@@ -44497,7 +44489,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");
@@ -44515,6 +44516,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() {
@@ -44527,7 +44532,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,
@@ -44536,7 +44542,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.8",
+  "version": "0.2.10",
   "description": "The smartest and fastest n8n MCP server — validate, fix, and discover workflows inside your LLM",
   "keywords": [
     "mcp",