YEP-3037 Allow to use only selected api tools in MCP#19
Conversation
marcos-muino-garcia
requested review from
santicastro and
sergio-rs
and removed request for
a team
![cursor[bot]](https://avatars.githubusercontent.com/in/1210556?s=60&v=4){kind=small}
There was a problem hiding this comment.
This is the final PR Bugbot will review for you during this billing cycle
Your free Bugbot reviews will reset on December 14
Details
Your team is on the Bugbot Free tier. On this plan, Bugbot will review limited PRs each billing cycle for each member of your team.
To receive Bugbot reviews on all of your PRs, visit the Cursor dashboard to activate Pro and start your 14-day free trial.
| for (const tool of [...DEFAULT_API_TOOLS, ...ADDITIONAL_API_TOOLS]) { | ||
| if (this.tools.includes(tool.name)) { | ||
| tools.push(tool); | ||
| } |
There was a problem hiding this comment.
Bug: Duplicate tools added when combining categories with specific names
When a user specifies both a category tag (like yc_api or yc_api_full) and a specific tool name that belongs to that category, the tool gets added to the tools array twice. The category check pushes all tools from DEFAULT_API_TOOLS/ADDITIONAL_API_TOOLS, and then the for-loop unconditionally pushes any tool whose name matches this.tools, without checking if it was already added. This results in duplicate entries in the ListTools response.
2 participants
Note
Enables specifying individual API tools by name in YEPCODE_MCP_TOOLS and refactors tool listing with grouped constants.
DEFAULT_API_TOOLSandADDITIONAL_API_TOOLSconstants to group API tools.ListToolshandler to:tool.nameis listed inYEPCODE_MCP_TOOLS.YEPCODE_MCP_TOOLScan include specific API tool names (e.g.,execute_process_sync,get_execution).Written by Cursor Bugbot for commit 38129cc. This will update automatically on new commits. Configure here.