@path58/p58-n8n 0.2.4 → 0.2.5

package/dist/mcp/server.bundle.cjs

@@ -3014,28 +3014,22 @@ function buildPoolOptions() {
 }
 function createPool(cfg) {
   const timeoutMs = getEnvNumber("DB_STATEMENT_TIMEOUT_MS", 3e4);
-  const isCatalogOnly = !!(process.env.P58_DATABASE_URL || process.env.MCP_DATABASE_URL) && !process.env.N8N_API_KEY;
   const pool = new import_pg.Pool({
     host: cfg.host,
     port: resolvePoolPort(cfg),
     user: cfg.user,
     password: cfg.password,
     database: cfg.database,
-    // Friends: 3 max (1 active + 2 burst). Full mode: 10.
-    max: getEnvNumber("VALIDATOR_DB_POOL_MAX", isCatalogOnly ? 3 : 10),
-    // Friends: 0 min — don't hold connections when idle. Full mode: 1.
-    min: getEnvNumber("VALIDATOR_DB_POOL_MIN", isCatalogOnly ? 0 : 1),
+    max: getEnvNumber("VALIDATOR_DB_POOL_MAX", 10),
+    min: getEnvNumber("VALIDATOR_DB_POOL_MIN", 1),
     // allowExitOnIdle: true — prevents zombie processes when MCP session ends (RAG-4.35.5)
-    // Friends: 30s idle (release fast). Full mode: 60s.
-    idleTimeoutMillis: isCatalogOnly ? 3e4 : 6e4,
-    // Friends: 10s connect timeout (fail fast). Full mode: 15s.
-    connectionTimeoutMillis: isCatalogOnly ? 1e4 : 15e3,
+    idleTimeoutMillis: 6e4,
+    connectionTimeoutMillis: 15e3,
     statement_timeout: timeoutMs,
     options: buildPoolOptions(),
     allowExitOnIdle: true,
-    // Friends: no keepAlive (let connections close naturally). Full mode: keepAlive.
-    keepAlive: !isCatalogOnly,
-    ...isCatalogOnly ? {} : { keepAliveInitialDelayMillis: 1e4 }
+    keepAlive: true,
+    keepAliveInitialDelayMillis: 1e4
   });
   pool.on("error", (err) => {
     console.warn(`[pg-pool] Idle connection error (non-fatal): ${err.message}`);
@@ -3047,24 +3041,6 @@ function getValidatorPool() {
     validatorPool = createPool(getValidatorDbConfig());
   return validatorPool;
 }
-async function warmSingleConnection(pool) {
-  const client = await pool.connect();
-  await client.query("SELECT 1");
-  client.release();
-}
-async function warmValidatorPool() {
-  if (poolWarmed)
-    return;
-  const pool = getValidatorPool();
-  const minConns = getEnvNumber("VALIDATOR_DB_POOL_MIN", 1);
-  try {
-    for (let i = 0; i < minConns; i++)
-      await warmSingleConnection(pool);
-    poolWarmed = true;
-  } catch (error) {
-    console.warn("Validator pool warm-up failed, continuing with lazy connection creation:", error);
-  }
-}
 async function validatorQuery(sql, params = []) {
   const client = await getValidatorPool().connect();
   const onClientError = (err) => {
@@ -3086,7 +3062,7 @@ async function shutdownValidatorPool() {
     validatorPool = null;
   }
 }
-var import_pg, dotenv, path, import_url, import_connection, __filename, __dirname, PROJECT_ROOT, validatorPool, poolWarmed;
+var import_pg, dotenv, path, import_url, import_connection, __filename, __dirname, PROJECT_ROOT, validatorPool;
 var init_validatorPostgresClient = __esm({
   "dist/db/validatorPostgresClient.js"() {
     "use strict";
@@ -3101,7 +3077,6 @@ var init_validatorPostgresClient = __esm({
     dotenv.config({ path: path.resolve(PROJECT_ROOT, ".env.supabase") });
     dotenv.config({ path: path.resolve(PROJECT_ROOT, ".env") });
     validatorPool = null;
-    poolWarmed = false;
   }
 });
 
@@ -18461,7 +18436,6 @@ var init_cached_catalog_adapter = __esm({
 // dist/mcp/server.js
 var server_exports = {};
 __export(server_exports, {
-  isServerInitialized: () => isServerInitialized,
   logStartupSummary: () => logStartupSummary
 });
 module.exports = __toCommonJS(server_exports);
@@ -18473,7 +18447,7 @@ var import_types22 = require("@modelcontextprotocol/sdk/types.js");
 var config = {
   // Server identity
   SERVER_NAME: "p58-n8n",
-  SERVER_VERSION: "0.2.4",
+  SERVER_VERSION: "0.2.3",
   // Database configuration (from environment)
   SUPABASE_URL: process.env.SUPABASE_URL,
   SUPABASE_KEY: process.env.SUPABASE_KEY,
@@ -19205,30 +19179,6 @@ async function injectResponseMetadata(sessionId, toolName, response, executionSt
   return instrumented;
 }
 
-// dist/mcp/in-flight-tracker.js
-var inflightCount = 0;
-function incrementInflight() {
-  inflightCount++;
-}
-function decrementInflight() {
-  if (inflightCount > 0)
-    inflightCount--;
-}
-function waitForInflightRequests(timeoutMs) {
-  if (inflightCount === 0)
-    return Promise.resolve();
-  return new Promise((resolve2) => {
-    const deadline = setTimeout(resolve2, timeoutMs);
-    const poll = setInterval(() => {
-      if (inflightCount <= 0) {
-        clearInterval(poll);
-        clearTimeout(deadline);
-        resolve2();
-      }
-    }, 50);
-  });
-}
-
 // dist/validation/l1-structure.js
 async function validateL1Structure(workflowJson) {
   const startTime = performance.now();
@@ -30438,13 +30388,7 @@ var AutoFixerRegistry = class _AutoFixerRegistry {
     }
   }
 };
-var _fixerRegistry = null;
-function getFixerRegistry() {
-  if (!_fixerRegistry) {
-    _fixerRegistry = new AutoFixerRegistry();
-  }
-  return _fixerRegistry;
-}
+var fixerRegistry = new AutoFixerRegistry();
 
 // node_modules/axios/lib/helpers/bind.js
 function bind(fn, thisArg) {
@@ -34942,7 +34886,7 @@ async function runAutofixOnWorkflow(workflowJson, issues) {
   const { applied: credApplied, changelog: credChangelog } = preAssignCredentials(workflow, issues);
   const errorIssues = issues.filter((i) => i.severity === "error");
   const problems = errorIssues.map(issueToValidationProblem);
-  const fixResult = await getFixerRegistry().applyFixes(workflow, problems);
+  const fixResult = await fixerRegistry.applyFixes(workflow, problems);
   const fixChangelog = buildChangelog(fixResult.results);
   return {
     fixedWorkflow: fixResult.workflow,
@@ -35337,10 +35281,10 @@ How it works:
 
 Validation levels:
 - L1: JSON structure, required fields, duplicate node names
-- L2: Node types exist in catalog (1,545 nodes)
-- L3: Credential types valid (679 credentials) \u2014 includes available_credentials lookup
-- L4: Connection patterns exist (7,642 rules)
-- L5: Required parameters configured (38,005 params)
+- L2: Node types exist in catalog (1,982 nodes)
+- L3: Credential types valid (654 credentials) \u2014 includes available_credentials lookup
+- L4: Connection patterns exist (6,900 rules)
+- L5: Required parameters configured (35,143 params)
 - L6: Pattern validation (flow integrity, security, expressions)
 
 Returns detailed issues with severity, location, and suggested fixes.
@@ -35543,7 +35487,7 @@ var getOperationSchemaToolDefinition = {
   name: "get_operation_schema",
   description: `Get parameter requirements for a specific n8n node operation.
 
-Returns the exact required and optional parameters from our 38,005 parameter catalog.
+Returns the exact required and optional parameters from our 35,143 parameter catalog.
 Use this to prevent parameter hallucination when generating n8n workflows.
 
 Examples:
@@ -36629,77 +36573,6 @@ Examples:
   }
 };
 
-// dist/mcp/tools/handlers/shared/n8n-guard.js
-var SETUP_GUIDE_URL = "https://github.com/tsvika58/p58-n8n/blob/main/docs/AGENT_INSTALL.md";
-var OFFLINE_TOOLS = /* @__PURE__ */ new Set([
-  "validate_workflow",
-  "get_operation_schema",
-  "check_parameter",
-  "suggest_fix",
-  "list_operations",
-  "list_nodes",
-  "get_node_info",
-  "find_similar_pattern",
-  "get_session_metrics",
-  "get_credential_schema",
-  "setup_check"
-]);
-function requiresN8nApiKey(toolName) {
-  return !OFFLINE_TOOLS.has(toolName);
-}
-function makeN8nNotConfiguredError() {
-  return {
-    content: [
-      {
-        type: "text",
-        text: JSON.stringify({
-          error: `This tool requires n8n connection. Set N8N_API_URL and N8N_API_KEY in your MCP config. See: ${SETUP_GUIDE_URL}`
-        })
-      }
-    ],
-    isError: true
-  };
-}
-function isDbConfigError(error) {
-  const msg = error instanceof Error ? error.message : String(error);
-  return msg.includes("Missing required validator DB environment variable");
-}
-function isDbConnectionError(error) {
-  const msg = error instanceof Error ? error.message : String(error);
-  return msg.includes("ECONNREFUSED") || msg.includes("ETIMEDOUT") || msg.includes("connect ETIMEDOUT") || msg.includes("after calling end on the pool") || msg.includes("Connection terminated") || msg.includes("getaddrinfo ENOTFOUND") || msg.includes("too many connections") || msg.includes("remaining connection slots");
-}
-function makeDbNotConfiguredResponse() {
-  return {
-    content: [
-      {
-        type: "text",
-        text: JSON.stringify({
-          error: "DB_NOT_CONFIGURED",
-          message: "Catalog features require P58_DATABASE_URL. Set P58_DATABASE_URL in your MCP config to enable this tool.",
-          setup_guide: SETUP_GUIDE_URL
-        })
-      }
-    ],
-    isError: true
-  };
-}
-function makeDbConnectionErrorResponse() {
-  return {
-    content: [
-      {
-        type: "text",
-        text: JSON.stringify({
-          error: "CATALOG_UNREACHABLE",
-          message: "Catalog database is unreachable. Verify your P58_DATABASE_URL is correct and the database server is accessible. If the problem persists, contact Tsvika.",
-          hint: "Check: (1) connection string starts with postgresql://mcp_friend... (2) port is 6543 (transaction pooler) (3) network allows outbound to AWS eu-north-1",
-          setup_guide: SETUP_GUIDE_URL
-        })
-      }
-    ],
-    isError: true
-  };
-}
-
 // dist/mcp/middleware/response-compression.js
 function selectFields(item, fields) {
   const result = {};
@@ -36819,11 +36692,8 @@ async function handleListOperations(args) {
     return toMCPResponse(response);
   } catch (error) {
     const originalError = error instanceof Error ? error : new Error(String(error));
-    if (isDbConfigError(originalError))
-      return makeDbNotConfiguredResponse();
-    if (isDbConnectionError(originalError))
-      return makeDbConnectionErrorResponse();
-    return toMCPResponse(createInternalError(correlationId, originalError));
+    const errorResponse = createInternalError(correlationId, originalError);
+    return toMCPResponse(errorResponse);
   }
 }
 var listOperationsToolDefinition = {
@@ -36920,16 +36790,13 @@ async function handleListNodes(args) {
     return toMCPResponse(response);
   } catch (error) {
     const originalError = error instanceof Error ? error : new Error(String(error));
-    if (isDbConfigError(originalError))
-      return makeDbNotConfiguredResponse();
-    if (isDbConnectionError(originalError))
-      return makeDbConnectionErrorResponse();
-    return toMCPResponse(createInternalError(correlationId, originalError));
+    const errorResponse = createInternalError(correlationId, originalError);
+    return toMCPResponse(errorResponse);
   }
 }
 var listNodesToolDefinition = {
   name: "list_nodes",
-  description: `List all available n8n nodes (1,545 total) with filtering and pagination.
+  description: `List all available n8n nodes (1,982 total) with filtering and pagination.
 
 Use this to discover available nodes before generating workflows. The tool supports:
 - Pagination: Use limit (1-100) and offset for large result sets
@@ -37086,11 +36953,8 @@ async function handleGetNodeInfo(args) {
     return toMCPResponse(response);
   } catch (error) {
     const originalError = error instanceof Error ? error : new Error(String(error));
-    if (isDbConfigError(originalError))
-      return makeDbNotConfiguredResponse();
-    if (isDbConnectionError(originalError))
-      return makeDbConnectionErrorResponse();
-    return toMCPResponse(createInternalError(correlationId, originalError));
+    const errorResponse = createInternalError(correlationId, originalError);
+    return toMCPResponse(errorResponse);
   }
 }
 var getNodeInfoToolDefinition = {
@@ -37178,11 +37042,8 @@ async function handleFindSimilarPattern(args) {
     return toMCPResponse(response);
   } catch (error) {
     const originalError = error instanceof Error ? error : new Error(String(error));
-    if (isDbConfigError(originalError))
-      return makeDbNotConfiguredResponse();
-    if (isDbConnectionError(originalError))
-      return makeDbConnectionErrorResponse();
-    return toMCPResponse(createInternalError(correlationId, originalError));
+    const errorResponse = createInternalError(correlationId, originalError);
+    return toMCPResponse(errorResponse);
   }
 }
 var findSimilarPatternToolDefinition = {
@@ -37379,8 +37240,8 @@ schemas, build_workflow handles validation, autofix, and deployment automaticall
 
 Runs L1-L4 validation before creating:
 - L1: JSON structure, required fields
-- L2: Node types exist in catalog (1,545 nodes)
-- L3: Credential types are valid (679 credentials)
+- L2: Node types exist in catalog (1,982 nodes)
+- L3: Credential types are valid (654 credentials)
 - L4: Connection patterns exist
 
 Refuses creation if any L1-L4 errors are found.
@@ -38751,7 +38612,7 @@ async function validateWorkflowObject(workflow, timeoutMs, label) {
 }
 async function runAutoFix(workflow, validationResult, timeoutMs, toolName) {
   const problems = validationResult.issues.filter((i) => i.severity === "error").map(issueToValidationProblem2);
-  const fixResult = await withTimeout(getFixerRegistry().applyFixes(workflow, problems), timeoutMs, `${toolName} autofix`);
+  const fixResult = await withTimeout(fixerRegistry.applyFixes(workflow, problems), timeoutMs, `${toolName} autofix`);
   return { fixedWorkflow: fixResult.workflow, fixResult };
 }
 async function applyAutoFixIfNeeded(workflow, validationResult, hasErrors, shouldFix, timeoutMs, correlationId, toolName) {
@@ -39398,7 +39259,7 @@ function issueToValidationProblem3(issue) {
 }
 async function runFixStep(workflow, validation, timeoutMs) {
   const problems = validation.issues.filter((i) => i.severity === "error").map(issueToValidationProblem3);
-  const fixResult = await withTimeout(getFixerRegistry().applyFixes(workflow, problems), timeoutMs, "server_autofix:fix");
+  const fixResult = await withTimeout(fixerRegistry.applyFixes(workflow, problems), timeoutMs, "server_autofix:fix");
   return {
     fixedWorkflow: fixResult.workflow,
     fixes: extractAppliedFixes(fixResult)
@@ -41428,10 +41289,6 @@ and what service it connects to \u2014 don't attempt to build with a broken cred
 Part of the generation lifecycle: plan_workflow \u2192 test_credential(s) \u2192 build_workflow.
 NOT needed for editing or updating existing workflows (use update_workflow or partial_update_workflow).
 
-Note: plan_workflow works without n8n (catalog-only mode). To actually deploy
-workflows with build_workflow, N8N_API_KEY and N8N_API_URL must be configured.
-Without n8n, use plan_workflow for research and architecture planning only.
-
 Response modes:
 - "full": All research data within token_budget (default 8,000)
 - "summary" (default): Key fields within token_budget (default 4,000)
@@ -43238,12 +43095,7 @@ async function handleGetCredentialSchema(args) {
     return buildSchemaResponse(credential_type, rows, nodes, correlationId, startTime);
   } catch (error) {
     import_logging57.logger.error("get_credential_schema: unexpected error", { correlationId, error });
-    const originalError = error instanceof Error ? error : new Error(String(error));
-    if (isDbConfigError(originalError))
-      return makeDbNotConfiguredResponse();
-    if (isDbConnectionError(originalError))
-      return makeDbConnectionErrorResponse();
-    return toMCPResponse(createInternalError(correlationId, originalError));
+    return toMCPResponse(createInternalError(correlationId, error instanceof Error ? error : new Error(String(error))));
   }
 }
 async function handleUnknownType(credentialType, correlationId) {
@@ -44257,8 +44109,40 @@ When values are collected, marks matching factory.gaps entries as resolved for a
   }
 };
 
+// dist/mcp/tools/handlers/shared/n8n-guard.js
+var SETUP_GUIDE_URL = "https://github.com/tsvika58/n8n-workflow-validator/blob/main/docs/AGENT_INSTALL.md";
+var OFFLINE_TOOLS = /* @__PURE__ */ new Set([
+  "validate_workflow",
+  "get_operation_schema",
+  "check_parameter",
+  "suggest_fix",
+  "list_operations",
+  "list_nodes",
+  "get_node_info",
+  "find_similar_pattern",
+  "get_session_metrics",
+  "get_credential_schema",
+  "setup_check"
+]);
+function requiresN8nApiKey(toolName) {
+  return !OFFLINE_TOOLS.has(toolName);
+}
+function makeN8nNotConfiguredError() {
+  return {
+    content: [
+      {
+        type: "text",
+        text: JSON.stringify({
+          error: `This tool requires n8n connection. Set N8N_API_URL and N8N_API_KEY in your MCP config. See: ${SETUP_GUIDE_URL}`
+        })
+      }
+    ],
+    isError: true
+  };
+}
+
 // dist/mcp/tools/handlers/setup-check.js
-var SETUP_GUIDE_URL2 = "https://github.com/tsvika58/p58-n8n/blob/main/docs/AGENT_INSTALL.md";
+var SETUP_GUIDE_URL2 = "https://github.com/tsvika58/n8n-workflow-validator/blob/main/docs/AGENT_INSTALL.md";
 var N8N_PROBE_TIMEOUT_MS = 2e3;
 var N8N_PROBE_PORTS = [5678, 5679];
 async function probeN8nUrl(url2) {
@@ -44448,18 +44332,13 @@ async function dispatchToolCall(name, args) {
   if (requiresN8nApiKey(name) && !process.env.N8N_API_KEY) {
     return makeN8nNotConfiguredError();
   }
-  incrementInflight();
-  try {
-    if (tool.inputSchema && hasSchema(name)) {
-      const validation = validateToolInput(name, args);
-      if (!validation.success)
-        return createValidationErrorResponse(validation.error);
-      return await tool.handler(validation.data);
-    }
-    return await tool.handler(args);
-  } finally {
-    decrementInflight();
+  if (tool.inputSchema && hasSchema(name)) {
+    const validation = validateToolInput(name, args);
+    if (!validation.success)
+      return createValidationErrorResponse(validation.error);
+    return tool.handler(validation.data);
   }
+  return tool.handler(args);
 }
 async function handleToolCall(name, args, sessionId = "default") {
   const startTime = Date.now();
@@ -44477,9 +44356,7 @@ function registerAllTools() {
     registerTool({ ...entry, inputSchema: toolSchemas[entry.name] });
   }
 }
-
-// dist/mcp/server.js
-init_validatorPostgresClient();
+registerAllTools();
 
 // dist/mcp/middleware/credential-redaction.js
 var MAX_DEPTH = 20;
@@ -44652,7 +44529,6 @@ var defaultDeps = {
 };
 async function runCleanup(serverName, deps) {
   try {
-    await waitForInflightRequests(5e3);
     await deps.shutdownPool();
     deps.clearCache();
     deps.clearConfigCache?.();
@@ -44684,24 +44560,7 @@ function registerShutdownHandlers(serverName, deps) {
 }
 
 // dist/mcp/server.js
-var SETUP_GUIDE_URL3 = "https://github.com/tsvika58/p58-n8n/blob/main/docs/AGENT_INSTALL.md";
-var _serverInitialized = false;
-function isServerInitialized() {
-  return _serverInitialized;
-}
-function buildInitializingResponse() {
-  return {
-    content: [{
-      type: "text",
-      text: JSON.stringify({
-        error: "SERVER_INITIALIZING",
-        message: "Server is still loading. Please retry in a moment.",
-        retryAfterMs: 2e3
-      })
-    }],
-    isError: true
-  };
-}
+var SETUP_GUIDE_URL3 = "https://github.com/tsvika58/n8n-workflow-validator/blob/main/docs/AGENT_INSTALL.md";
 function logStartupSummary(n8nConnected) {
   const apiKey = process.env.N8N_API_KEY;
   const n8nUrl = process.env.N8N_API_URL ?? process.env.N8N_API_BASE_URL ?? "http://localhost:5678/api/v1";
@@ -44746,9 +44605,6 @@ function toCallToolResult(response) {
 server.setRequestHandler(import_types22.CallToolRequestSchema, async (request) => {
   const { name, arguments: args } = request.params;
   try {
-    if (!_serverInitialized) {
-      return toCallToolResult(buildInitializingResponse());
-    }
     const response = await handleToolCall(name, args ?? {});
     return toCallToolResult(redactMCPResponse(response));
   } catch (error) {
@@ -44758,25 +44614,13 @@ server.setRequestHandler(import_types22.CallToolRequestSchema, async (request) =
 });
 async function main() {
   const transport = new import_stdio.StdioServerTransport();
-  const startMs = Date.now();
   await server.connect(transport);
-  console.error(`[INIT] MCP handshake complete in ${Date.now() - startMs}ms`);
-  registerAllTools();
-  _serverInitialized = true;
   const toolCount = getRegisteredTools().length;
-  console.error(`[INIT] ${toolCount} tools registered in ${Date.now() - startMs}ms`);
+  console.error(`${config.SERVER_NAME} v${config.SERVER_VERSION} running on stdio (${toolCount} tools registered)`);
   logStartupSummary(Boolean(process.env.N8N_API_KEY));
   registerShutdownHandlers(config.SERVER_NAME);
   void warmCredentialCache().catch(() => {
   });
-  if (process.env.P58_DATABASE_URL || process.env.MCP_DATABASE_URL) {
-    void warmValidatorPool().then(() => {
-      console.error("catalog: connected \u2713");
-    }).catch((err) => {
-      console.error(`catalog: WARNING \u2014 connection failed: ${err.message}`);
-      console.error("catalog: tools will retry on first use, but check P58_DATABASE_URL if errors persist");
-    });
-  }
 }
 main().catch((error) => {
   console.error("Server failed to start:", error);
@@ -44784,7 +44628,6 @@ main().catch((error) => {
 });
 // Annotate the CommonJS export names for ESM import in node:
 0 && (module.exports = {
-  isServerInitialized,
   logStartupSummary
 });
 /*! Bundled license information:

package/package.json

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