Problem/Motivation
The latest version of Tool has Tool Operations that are required.
Currently we hard code it, but it should be set per tool. So when you enable a version of a tool you need to pick this.
Proposed resolution
Add a setting per tool, where you can choose the tool operation for the tool you have enabled. ![]()
| Comment | File | Size | Author |
|---|---|---|---|
| #3 | Screenshot 2025-11-25 at 11.19.21 PM.png | 86.9 KB | harivansh |
| #2 | Screenshot 2025-11-25 at 11.11.01 PM.png | 255.92 KB | harivansh |
Issue fork mcp_client-3559869
Show commands
Start within a Git clone of the project using the version control instructions.
Or, if you do not have SSH keys set up on git.drupalcode.org:
Comments
Comment #2
harivansh commented@marcus_johansson

I think we should add another column in the tool maintenance section
Something like this
We can have another property $tool_operations, in the confirm form, which stores the mapping b/w tool and operation

Comment #3
harivansh commentedComment #4
harivansh commentedComment #5
robertoperuzzoThat's a good idea!
In case we may use this simple method #3559613-4: Remove dependency on AI/AI Agents module to discover the operation's type dynamically, pre-populating the select box.
Comment #6
robertoperuzzoComment #8
robertoperuzzoComment #9
harivansh commentedI think we should consolidate all tool configuration into a single tools property on the entity. This would eliminate the separate enabled_tools, locked_tools, and tool_operations arrays.
To make this more maintainable and type-safe, we should also create dedicated DTOs/Value Objects for this:
ToolCollection - represents the collection of all tools
Tool - represents a single tool with all its configuration
InputSchema - represents the tool's input schema
LockDefinition - represents the locked tool definition
This approach would give us a single source of truth, cleaner code, and better type safety throughout the codebase.
What your though on this.
Comment #10
robertoperuzzoThat's awesome! I opened a new issue for the refactoring #3561170: Consolidate Tool Configuration Into a Single tools Property. Could you take a look at it? Maybe I missed something ...
I propose merging and closing this issue to keep the issue's context clean and separate from the refactoring.
Comment #12
robertoperuzzo