5474312
diff --git a/‎CHANGELOG.md‎
Lines changed: 10 additions & 0 deletions b/‎CHANGELOG.md‎
Lines changed: 10 additions & 0 deletions
diff --git a/‎README.md‎
Lines changed: 2 additions & 2 deletions b/‎README.md‎
Lines changed: 2 additions & 2 deletions
diff --git a/‎data/nodes.db‎
0 Bytes b/‎data/nodes.db‎
0 Bytes
diff --git a/‎package-lock.json‎
Lines changed: 2 additions & 2 deletions b/‎package-lock.json‎
Lines changed: 2 additions & 2 deletions
diff --git a/‎package.json‎
Lines changed: 1 addition & 1 deletion b/‎package.json‎
Lines changed: 1 addition & 1 deletion
diff --git a/‎src/community/community-node-service.ts‎
Lines changed: 10 additions & 5 deletions b/‎src/community/community-node-service.ts‎
Lines changed: 10 additions & 5 deletions
diff --git a/‎src/mcp/server.ts‎
Lines changed: 25 additions & 130 deletions b/‎src/mcp/server.ts‎
Lines changed: 25 additions & 130 deletions
diff --git a/‎src/services/ai-node-validator.ts‎
Lines changed: 11 additions & 2 deletions b/‎src/services/ai-node-validator.ts‎
Lines changed: 11 additions & 2 deletions
@@ -7,6 +7,16 @@ and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0
 
 ## [Unreleased]
 
+## [2.66.2] - 2026-07-27
+
+### Fixed
+
+- **Vector stores in `retrieve-as-tool` mode validate as AI tools (#953).** n8n's vector-store nodes expose an `ai_tool` output only when `mode` is `retrieve-as-tool`, but `validate_workflow` resolved ai_tool sources against a fixed node list and rejected the connection with `INVALID_AI_TOOL_SOURCE`. The database stores each node's raw outputs expression, so the validator now recognises an output that exists only for particular parameter values by scanning that expression — covering every vector store in 2.31.3 without maintaining a list. When the node's `mode` is a different literal (or unset, so the default `retrieve` applies), a new `AI_TOOL_MODE_MISMATCH` warning names the parameter to change instead of a false error.
+- **Community AI-tool detection no longer infers capability from the node name (#954).** Any package whose name contained the letters "ai" — `n8n-nodes-raia`, `@firefliesai/n8n-nodes-fireflies` — was flagged `is_ai_tool`, 901 of 1,455 community rows in total. Detection now requires a declared `usableAsTool` (any non-`false` value, matching the core parser's handling of the object form) or the package's codex AI category, which stays as a labelled inference because community metadata often omits `usableAsTool`. `get_node` reports the provenance in a new `aiToolFlagSource` field, and the community data was refreshed: 853 of 1,457 rows remain flagged, all from declared or codex signals.
+- **The "Community node used as an AI tool" warning now describes a community node (#955).** It defined community as `package !== 'n8n-nodes-base'` — sweeping in first-party `@n8n/n8n-nodes-langchain` — and it inspected the target of the `ai_tool` connection, which is always the AI Agent, so essentially every agent workflow warned that "Community node \"AI Agent\"" needed `N8N_COMMUNITY_PACKAGES_ALLOW_TOOL_USAGE=true`. The check now tests the database's community flag on the connection's source — the tool node — and `get_node`'s `requiresEnvironmentVariable` follows the same flag instead of the AI-tool flag and a package-name test. A community tool the bundled database does not know (a user-installed package) keeps the notice, recognised by its package-qualified type.
+- **The AI Agent `systemMessage` advisory reads the parameter where n8n stores it (#956).** The check read `parameters.systemMessage`, but the node keeps it under `options`, so "has no systemMessage" fired on every agent regardless of configuration — and an agent following the advice by setting the top-level field saw the message persist. The too-short check, previously unreachable, now runs as intended. The top-level read is kept as a fallback.
+- Removed `getNodeAsToolInfo` and `getAIToolExamples` from the MCP server: dead code with no dispatch path, carrying the same package-name assumption as #955.
+
 ## [2.66.1] - 2026-07-26
 
 ### Changed
 
@@ -9,13 +9,13 @@
 [![Docker](https://img.shields.io/badge/docker-ghcr.io%2Fczlonkowski%2Fn8n--mcp-green.svg)](https://github.com/czlonkowski/n8n-mcp/pkgs/container/n8n-mcp)
 [![Deploy on Railway](https://railway.com/button.svg)](https://railway.com/deploy/n8n-mcp?referralCode=n8n-mcp)
 
-A Model Context Protocol (MCP) server that provides AI assistants with comprehensive access to n8n node documentation, properties, and operations. Deploy in minutes to give Claude and other AI assistants deep knowledge about n8n's 2,283 workflow automation nodes (828 core + 1,455 community).
+A Model Context Protocol (MCP) server that provides AI assistants with comprehensive access to n8n node documentation, properties, and operations. Deploy in minutes to give Claude and other AI assistants deep knowledge about n8n's 2,285 workflow automation nodes (828 core + 1,457 community).
 
 ## Overview
 
 n8n-MCP serves as a bridge between n8n's workflow automation platform and AI models, enabling them to understand and work with n8n nodes effectively. It provides structured access to:
 
-- **2,283 n8n nodes** - 828 core nodes + 1,455 community nodes (1,295 verified)
+- **2,285 n8n nodes** - 828 core nodes + 1,457 community nodes (1,295 verified)
 - **Node properties** - 99% coverage with detailed schemas
 - **Node operations** - 66.5% coverage of available actions
 - **Documentation** - 86% coverage from official n8n docs (including AI nodes)
 
@@ -1,6 +1,6 @@
 {
   "name": "n8n-mcp",
-  "version": "2.66.1",
+  "version": "2.66.2",
   "description": "Integration between n8n workflow automation and Model Context Protocol (MCP)",
   "main": "dist/index.js",
   "types": "dist/index.d.ts",
 
@@ -257,11 +257,16 @@ export class CommunityNodeService {
       nodeType = nodeType.replace('n8n-nodes-preview-', 'n8n-nodes-');
     }
 
-    // Determine if it's an AI tool
-    const isAITool =
-      nodeDesc.usableAsTool === true ||
-      nodeDesc.codex?.categories?.includes('AI') ||
-      attributes.name?.toLowerCase().includes('ai');
+    // n8n types usableAsTool as `true | UsableAsToolDescription`, so any value
+    // other than false/absent is a declared capability, matching the core parser.
+    // The codex AI category stays as an inference because community metadata
+    // often omits usableAsTool. The node's name is never a signal: "ai" as a
+    // substring matches packages like fireflies*ai* that are not tools (#954).
+    const usableAsTool = nodeDesc.usableAsTool;
+    const declaresToolUse =
+      usableAsTool !== undefined && usableAsTool !== null && usableAsTool !== false;
+    const hasAICategory = nodeDesc.codex?.categories?.includes('AI') ?? false;
+    const isAITool = declaresToolUse || hasAICategory;
 
     return {
       // Core ParsedNode fields
 
@@ -2020,16 +2020,32 @@ export class N8NDocumentationMCPServer {
       throw new Error(`Node ${nodeType} not found`);
     }
 
-    // Add AI tool capabilities information with null safety
+    // N8N_COMMUNITY_PACKAGES_ALLOW_TOOL_USAGE gates community packages being
+    // used as tools at all, so the requirement follows the community flag -
+    // not the AI-tool flag (which can be inferred, #954) and not a package-name
+    // test (which would sweep in first-party @n8n/* packages, #955).
+    const isCommunityNode = node.isCommunity ?? false;
+    const isMarkedAsAITool = node.isAITool ?? false;
+
+    // Built-in flags come from the declared usableAsTool property. Community
+    // ingestion collapses a declared usableAsTool and the package's codex AI
+    // category into one flag, so for community nodes the two are not
+    // distinguishable after the fact - the value says exactly that.
+    let aiToolFlagSource: string | null = null;
+    if (isMarkedAsAITool) {
+      aiToolFlagSource = isCommunityNode ? 'declared-or-ai-category' : 'declared-property';
+    }
+
     const aiToolCapabilities = {
       canBeUsedAsTool: true, // Any node can be used as a tool in n8n
-      hasUsableAsToolProperty: node.isAITool ?? false,
-      requiresEnvironmentVariable: !(node.isAITool ?? false) && node.package !== 'n8n-nodes-base',
+      hasUsableAsToolProperty: isMarkedAsAITool,
+      aiToolFlagSource,
+      requiresEnvironmentVariable: isCommunityNode,
       toolConnectionType: 'ai_tool',
       commonToolUseCases: this.getCommonAIToolUseCases(node.nodeType),
-      environmentRequirement: node.package && node.package !== 'n8n-nodes-base' ?
-        'N8N_COMMUNITY_PACKAGES_ALLOW_TOOL_USAGE=true' :
-        null
+      environmentRequirement: isCommunityNode
+        ? 'N8N_COMMUNITY_PACKAGES_ALLOW_TOOL_USAGE=true'
+        : null
     };
 
     // Process outputs to provide clear mapping with null safety
@@ -2867,11 +2883,11 @@ export class N8NDocumentationMCPServer {
       tools,
       totalCount: tools.length,
       requirements: {
-        environmentVariable: 'N8N_COMMUNITY_PACKAGES_ALLOW_TOOL_USAGE=true',
-        nodeProperty: 'usableAsTool: true',
+        environmentVariable: 'N8N_COMMUNITY_PACKAGES_ALLOW_TOOL_USAGE=true (community nodes only; built-in tools need no environment variable)',
+        nodeProperty: 'usableAsTool (declared; community nodes may instead carry the codex AI category)',
       },
       usage: {
-        description: 'These nodes have the usableAsTool property set to true, making them optimized for AI agent usage.',
+        description: 'These nodes are marked as AI tools. For built-in nodes this reflects a declared usableAsTool property; for community nodes it can also be inferred from the package\'s AI category, since community metadata often omits the property.',
         note: 'ANY node in n8n can be used as an AI tool by connecting it to the ai_tool port of an AI Agent node.',
         examples: [
           'Regular nodes like Slack, Google Sheets, or HTTP Request can be used as tools',
@@ -3925,75 +3941,6 @@ Full documentation is being prepared. For now, use get_node_essentials for confi
     };
   }
 
-  private async getNodeAsToolInfo(nodeType: string): Promise<any> {
-    await this.ensureInitialized();
-    if (!this.repository) throw new Error('Repository not initialized');
-
-    // Get node info
-    // First try with normalized type
-    const normalizedType = NodeTypeNormalizer.normalizeToFullForm(nodeType);
-    let node = this.repository.getNode(normalizedType);
-    
-    if (!node && normalizedType !== nodeType) {
-      // Try original if normalization changed it
-      node = this.repository.getNode(nodeType);
-    }
-    
-    if (!node) {
-      // Fallback to other alternatives for edge cases
-      const alternatives = getNodeTypeAlternatives(normalizedType);
-      
-      for (const alt of alternatives) {
-        const found = this.repository!.getNode(alt);
-        if (found) {
-          node = found;
-          break;
-        }
-      }
-    }
-    
-    if (!node) {
-      throw new Error(`Node ${nodeType} not found`);
-    }
-    
-    // Determine common AI tool use cases based on node type
-    const commonUseCases = this.getCommonAIToolUseCases(node.nodeType);
-    
-    // Build AI tool capabilities info
-    const aiToolCapabilities = {
-      canBeUsedAsTool: true, // In n8n, ANY node can be used as a tool when connected to AI Agent
-      hasUsableAsToolProperty: node.isAITool,
-      requiresEnvironmentVariable: !node.isAITool && node.package !== 'n8n-nodes-base',
-      connectionType: 'ai_tool',
-      commonUseCases,
-      requirements: {
-        connection: 'Connect to the "ai_tool" port of an AI Agent node',
-        environment: node.package !== 'n8n-nodes-base' ? 
-          'Set N8N_COMMUNITY_PACKAGES_ALLOW_TOOL_USAGE=true for community nodes' : 
-          'No special environment variables needed for built-in nodes'
-      },
-      examples: this.getAIToolExamples(node.nodeType),
-      tips: [
-        'Give the tool a clear, descriptive name in the AI Agent settings',
-        'Write a detailed tool description to help the AI understand when to use it',
-        'Test the node independently before connecting it as a tool',
-        node.isAITool ? 
-          'This node is optimized for AI tool usage' : 
-          'This is a regular node that can be used as an AI tool'
-      ]
-    };
-    
-    return {
-      nodeType: node.nodeType,
-      workflowNodeType: getWorkflowNodeType(node.package, node.nodeType),
-      displayName: node.displayName,
-      description: node.description,
-      package: node.package,
-      isMarkedAsAITool: node.isAITool,
-      aiToolCapabilities
-    };
-  }
-  
   private getOutputDescriptions(nodeType: string, outputName: string, index: number): { description: string, connectionGuidance: string } {
     // Special handling for loop nodes
     if (nodeType === 'nodes-base.splitInBatches') {
@@ -4134,58 +4081,6 @@ Full documentation is being prepared. For now, use get_node_essentials for confi
     return undefined;
   }
 
-  private getAIToolExamples(nodeType: string): any {
-    const exampleMap: Record<string, any> = {
-      'nodes-base.slack': {
-        toolName: 'Send Slack Message',
-        toolDescription: 'Sends a message to a specified Slack channel or user. Use this to notify team members about important events or results.',
-        nodeConfig: {
-          resource: 'message',
-          operation: 'post',
-          channel: '={{ $fromAI("channel", "The Slack channel to send to, e.g. #general") }}',
-          text: '={{ $fromAI("message", "The message content to send") }}'
-        }
-      },
-      'nodes-base.googleSheets': {
-        toolName: 'Update Google Sheet',
-        toolDescription: 'Reads or updates data in a Google Sheets spreadsheet. Use this to log information, retrieve data, or update records.',
-        nodeConfig: {
-          operation: 'append',
-          sheetId: 'your-sheet-id',
-          range: 'A:Z',
-          dataMode: 'autoMap'
-        }
-      },
-      'nodes-base.httpRequest': {
-        toolName: 'Call API',
-        toolDescription: 'Makes HTTP requests to external APIs. Use this to fetch data, trigger webhooks, or integrate with any web service.',
-        nodeConfig: {
-          method: '={{ $fromAI("method", "HTTP method: GET, POST, PUT, DELETE") }}',
-          url: '={{ $fromAI("url", "The complete API endpoint URL") }}',
-          sendBody: true,
-          bodyContentType: 'json',
-          jsonBody: '={{ $fromAI("body", "Request body as JSON object") }}'
-        }
-      }
-    };
-    
-    // Check for exact match or partial match
-    for (const [key, example] of Object.entries(exampleMap)) {
-      if (nodeType.includes(key)) {
-        return example;
-      }
-    }
-    
-    // Generic example
-    return {
-      toolName: 'Custom Tool',
-      toolDescription: 'Performs specific operations. Describe what this tool does and when to use it.',
-      nodeConfig: {
-        note: 'Configure the node based on its specific requirements'
-      }
-    };
-  }
-  
   private async validateNodeMinimal(nodeType: string, config: Record<string, any>): Promise<any> {
     await this.ensureInitialized();
     if (!this.repository) throw new Error('Repository not initialized');
 
@@ -240,14 +240,23 @@ export function validateAIAgent(
   }
 
   // 4. Check system message (RECOMMENDED)
-  if (!node.parameters.systemMessage) {
+  // The AI Agent stores systemMessage under options; the top-level read is kept
+  // as a fallback for typeVersions that placed it there.
+  const systemMessage =
+    node.parameters.options?.systemMessage ?? node.parameters.systemMessage;
+  if (!systemMessage) {
     issues.push({
       severity: 'info',
       nodeId: node.id,
       nodeName: node.name,
       message: `AI Agent "${node.name}" has no systemMessage. Consider adding one to define the agent's role, capabilities, and constraints.`
     });
-  } else if (node.parameters.systemMessage.trim().length < MIN_SYSTEM_MESSAGE_LENGTH) {
+  } else if (
+    typeof systemMessage === 'string' &&
+    // An expression's length says nothing about the resolved prompt
+    !systemMessage.startsWith('=') &&
+    systemMessage.trim().length < MIN_SYSTEM_MESSAGE_LENGTH
+  ) {
     issues.push({
       severity: 'info',
       nodeId: node.id,
Original file line number	Diff line number	Diff line change
`@@ -1,6 +1,6 @@`
`1`	`1`	`{`
`2`	`2`	`"name": "n8n-mcp",`
`3`		`- "version": "2.66.1",`
	`3`	`+ "version": "2.66.2",`
`4`	`4`	`"description": "Integration between n8n workflow automation and Model Context Protocol (MCP)",`
`5`	`5`	`"main": "dist/index.js",`
`6`	`6`	`"types": "dist/index.d.ts",`