Tools cannot be updated (suggested fix included)

[samdoeswork]

Describe the bug
The mcp.tool() method checks _registeredTools and errors if the tool has already been created. Unfortunately there is no method exposed to delete a tool. This means that a use case we have (a tool can change) cannot be supported.

To Reproduce
You could register a tool and then try to update it by re-registering, but also just look at the code above.

Expected behavior
This is half-way between bug and feature request - but I would like it to simply overwrite the tool.
Or to simply have a deleteToolIfExists() method.

Logs
If applicable, add logs to help explain your problem.

Additional context
Add any other context about the problem here.

[samdoeswork]

Here's four ways to fix this (hence not doing a PR - you may have a preference & it's super trivial):

1. Just delete this line as I think it does more harm than good (why not let us change our tools?)

if (this._registeredTools[name]) {
throw new Error(Tool ${name} is already registered);
}

2. Same as above but with an option to decide whether this should be allowed

3. Add a removeTool() method

4. Remove the private flag on the class, so we can remove older definitions ourselves

[hamishll]

@samdoeswork did you find a workaround for this in the mean time? I've just run into this and it's a pretty big feature gap to not be able to remove or update tools!

[flintzPax8]

Looks like this is resolved in this merged PR.
#247

Eagerly awaiting it's release!