Closed (fixed)
Project:
MCP Client
Version:
1.0.x-dev
Component:
Code
Priority:
Normal
Category:
Feature request
Assigned:
Unassigned
Reporter:
Created:
25 Nov 2025 at 15:57 UTC
Updated:
17 Dec 2025 at 08:19 UTC
Jump to comment: Most recent, Most recent file
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