Skip to content

feat(MCP Server Trigger Node): Handle multiple tool calls in mcp server trigger #15064

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Conversation

ggozad
Copy link
Contributor

@ggozad ggozad commented May 2, 2025

Summary

Handle case of multiple tool calls in MCP Server trigger.

Related Linear tickets, Github issues, and Community forum posts

Linear: https://linear.app/n8n/issue/AI-899/community-issue-mcp-trigger-isnt-able-to-execute-multiple-tools-at-the

Fixes #14777

Review / Merge checklist

  • PR title and summary are descriptive. (conventions)
  • Docs updated or follow-up ticket created.
  • Tests included.
  • PR Labeled with release/backport (if the PR is an urgent fix that needs to be backported)

Copy link

@cubic-dev-ai cubic-dev-ai bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

mrge found 3 issues across 3 files. View them in mrge.io

@ggozad ggozad changed the title feat(MCP Server Trigger): handle multiple tool calls in mcp server trigger. feat(MCP Server Trigger): handle multiple tool calls in mcp server trigger May 2, 2025
@n8n-assistant n8n-assistant bot added the n8n team Authored by the n8n team label May 2, 2025
@ggozad ggozad requested a review from jeanpaul May 2, 2025 15:05
@ggozad ggozad changed the title feat(MCP Server Trigger): handle multiple tool calls in mcp server trigger feat(MCP Server Trigger Node): Handle multiple tool calls in mcp server trigger May 2, 2025
Copy link

codecov bot commented May 2, 2025

Codecov Report

Attention: Patch coverage is 31.91489% with 32 lines in your changes missing coverage. Please review.

Files with missing lines Patch % Lines
.../nodes-langchain/nodes/mcp/McpTrigger/McpServer.ts 31.91% 30 Missing and 2 partials ⚠️

📢 Thoughts on this report? Let us know!

@ggozad
Copy link
Contributor Author

ggozad commented May 5, 2025

This has been tested with 3 tools with a client based on a month-old MCP python SDK as well as the latest.

OlegIvaniv
OlegIvaniv previously approved these changes May 5, 2025
Copy link
Contributor

github-actions bot commented May 5, 2025

✅ All Cypress E2E specs passed

@ggozad ggozad requested a review from jeanpaul May 5, 2025 12:18
jeanpaul
jeanpaul previously approved these changes May 6, 2025
Copy link
Contributor

@jeanpaul jeanpaul left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good 🚀

Copy link
Contributor

github-actions bot commented May 6, 2025

✅ All Cypress E2E specs passed

@ggozad ggozad dismissed stale reviews from jeanpaul and OlegIvaniv via cf4f4b9 May 7, 2025 14:39
Copy link
Contributor

github-actions bot commented May 7, 2025

✅ All Cypress E2E specs passed

@ggozad ggozad merged commit 59ba162 into master May 7, 2025
53 of 54 checks passed
@ggozad ggozad deleted the ai-899-community-issue-mcp-trigger-isnt-able-to-execute-multiple branch May 7, 2025 19:37
@janober
Copy link
Member

janober commented May 12, 2025

Got released with [email protected]

ozcangungor pushed a commit to ozcangungor/n8n that referenced this pull request May 25, 2025
etobella pushed a commit to etobella/n8n that referenced this pull request May 25, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
n8n team Authored by the n8n team Released
Projects
None yet
Development

Successfully merging this pull request may close these issues.

n8n MCP trigger isn't able to execute multiple tools at the same time
4 participants