fix(02-02): fix false positive docker connection error

- Add -s (silent) flag to curl to suppress progress output
- Change error handling to only check stdout for valid JSON
- Ignore stderr which contains curl progress info

n8n-workflow.json

@@ -87,7 +87,7 @@
     },
     {
       "parameters": {
-        "command": "curl --unix-socket /var/run/docker.sock 'http://localhost/v1.47/containers/json?all=true'",
+        "command": "curl -s --unix-socket /var/run/docker.sock 'http://localhost/v1.47/containers/json?all=true'",
         "options": {}
       },
       "id": "exec-docker-list",
@@ -98,7 +98,7 @@
     },
     {
       "parameters": {
-        "jsCode": "// Get Docker API response and user input\nconst dockerOutput = $input.item.json.stdout;\nconst userMessage = $('Telegram Trigger').item.json.message.text.toLowerCase().trim();\nconst chatId = $('Telegram Trigger').item.json.message.chat.id;\n\n// Check for Docker connection error\nif (!dockerOutput || $input.item.json.stderr) {\n  return [{\n    json: {\n      chatId: chatId,\n      error: true,\n      text: \"❌ Can't reach Docker — check if n8n has socket access\"\n    }\n  }];\n}\n\n// Parse JSON response\nlet containers;\ntry {\n  containers = JSON.parse(dockerOutput);\n} catch (e) {\n  return [{\n    json: {\n      chatId: chatId,\n      error: true,\n      text: \"❌ Unexpected Docker response — failed to parse JSON\"\n    }\n  }];\n}\n\n// Function to normalize container names\nfunction normalizeName(name) {\n  return name\n    .replace(/^\\//, '')  // Remove leading slash\n    .replace(/^(linuxserver[-_]|binhex[-_])/i, '')  // Remove common prefixes\n    .toLowerCase();\n}\n\n// Extract container name from user query\n// Remove common query words to get the container name\nconst queryWords = ['status', 'show', 'check', 'container', 'docker', 'what', 'is', 'the', 'of', 'for'];\nconst words = userMessage.split(/\\s+/).filter(word => !queryWords.includes(word));\nconst requestedName = words.join(' ').trim();\n\n// If no container name specified, return summary\nif (!requestedName || requestedName === '') {\n  const counts = containers.reduce((acc, c) => {\n    acc[c.State] = (acc[c.State] || 0) + 1;\n    return acc;\n  }, {});\n\n  const parts = [];\n  if (counts.running) parts.push(`${counts.running} running`);\n  if (counts.exited) parts.push(`${counts.exited} stopped`);\n  if (counts.paused) parts.push(`${counts.paused} paused`);\n  if (counts.restarting) parts.push(`${counts.restarting} restarting`);\n\n  const summary = parts.length > 0 ? parts.join(', ') : 'No containers found';\n\n  return [{\n    json: {\n      chatId: chatId,\n      summary: true,\n      containers: containers,\n      text: `📊 Container summary: ${summary}`\n    }\n  }];\n}\n\n// Find matching containers using fuzzy matching\nconst matches = containers.filter(c => {\n  const containerName = normalizeName(c.Names[0]);\n  const normalized = requestedName.toLowerCase();\n  return containerName.includes(normalized) || normalized.includes(containerName);\n});\n\n// Handle no matches\nif (matches.length === 0) {\n  return [{\n    json: {\n      chatId: chatId,\n      error: true,\n      text: `❌ No container found matching \"${requestedName}\".\\n\\nTry \"status\" to see all containers.`\n    }\n  }];\n}\n\n// Handle multiple matches\nif (matches.length > 1) {\n  const names = matches.map(c => c.Names[0].replace(/^\\//, '')).join('\\n• ');\n  return [{\n    json: {\n      chatId: chatId,\n      multipleMatches: true,\n      matches: matches,\n      text: `🔍 Found ${matches.length} matches:\\n\\n• ${names}\\n\\nPlease be more specific.`\n    }\n  }];\n}\n\n// Single match - return container details\nconst container = matches[0];\nreturn [{\n  json: {\n    chatId: chatId,\n    singleMatch: true,\n    container: {\n      id: container.Id,\n      name: container.Names[0].replace(/^\\//, ''),\n      state: container.State,\n      status: container.Status,\n      image: container.Image\n    }\n  }\n}];"
+        "jsCode": "// Get Docker API response and user input\nconst dockerOutput = $input.item.json.stdout;\nconst userMessage = $('Telegram Trigger').item.json.message.text.toLowerCase().trim();\nconst chatId = $('Telegram Trigger').item.json.message.chat.id;\n\n// Parse JSON response - only error if we can't parse valid JSON from stdout\nlet containers;\ntry {\n  if (!dockerOutput || dockerOutput.trim() === '') {\n    throw new Error('Empty response');\n  }\n  containers = JSON.parse(dockerOutput);\n} catch (e) {\n  return [{\n    json: {\n      chatId: chatId,\n      error: true,\n      text: \"❌ Can't reach Docker — check if n8n has socket access\"\n    }\n  }];\n}\n\n// Function to normalize container names\nfunction normalizeName(name) {\n  return name\n    .replace(/^\\//, '')  // Remove leading slash\n    .replace(/^(linuxserver[-_]|binhex[-_])/i, '')  // Remove common prefixes\n    .toLowerCase();\n}\n\n// Extract container name from user query\n// Remove common query words to get the container name\nconst queryWords = ['status', 'show', 'check', 'container', 'docker', 'what', 'is', 'the', 'of', 'for'];\nconst words = userMessage.split(/\\s+/).filter(word => !queryWords.includes(word));\nconst requestedName = words.join(' ').trim();\n\n// If no container name specified, return summary\nif (!requestedName || requestedName === '') {\n  const counts = containers.reduce((acc, c) => {\n    acc[c.State] = (acc[c.State] || 0) + 1;\n    return acc;\n  }, {});\n\n  const parts = [];\n  if (counts.running) parts.push(`${counts.running} running`);\n  if (counts.exited) parts.push(`${counts.exited} stopped`);\n  if (counts.paused) parts.push(`${counts.paused} paused`);\n  if (counts.restarting) parts.push(`${counts.restarting} restarting`);\n\n  const summary = parts.length > 0 ? parts.join(', ') : 'No containers found';\n\n  return [{\n    json: {\n      chatId: chatId,\n      summary: true,\n      containers: containers,\n      text: `📊 Container summary: ${summary}`\n    }\n  }];\n}\n\n// Find matching containers using fuzzy matching\nconst matches = containers.filter(c => {\n  const containerName = normalizeName(c.Names[0]);\n  const normalized = requestedName.toLowerCase();\n  return containerName.includes(normalized) || normalized.includes(containerName);\n});\n\n// Handle no matches\nif (matches.length === 0) {\n  return [{\n    json: {\n      chatId: chatId,\n      error: true,\n      text: `❌ No container found matching \"${requestedName}\".\\n\\nTry \"status\" to see all containers.`\n    }\n  }];\n}\n\n// Handle multiple matches\nif (matches.length > 1) {\n  const names = matches.map(c => c.Names[0].replace(/^\\//, '')).join('\\n• ');\n  return [{\n    json: {\n      chatId: chatId,\n      multipleMatches: true,\n      matches: matches,\n      text: `🔍 Found ${matches.length} matches:\\n\\n• ${names}\\n\\nPlease be more specific.`\n    }\n  }];\n}\n\n// Single match - return container details\nconst container = matches[0];\nreturn [{\n  json: {\n    chatId: chatId,\n    singleMatch: true,\n    container: {\n      id: container.Id,\n      name: container.Names[0].replace(/^\\//, ''),\n      state: container.State,\n      status: container.Status,\n      image: container.Image\n    }\n  }\n}];"
       },
       "id": "code-parse-match",
       "name": "Parse and Match",