Conversation
|
@ericdallo This might be a bit controversial 😅 , but some models still try to call tools without a prefix first. For a while, I thought about extending this to MCP if the tool has a unique name and there's no risk of collision, but I ultimately decided to stick with native tools only. I haven't noticed any regressions during several hours of testing, so I believe this is a safe change (and also consistent with what we allow in the config). |
|
@zikajk hum, what models you saw that? |
|
@ericdallo Deepseek 4 and local QWEN. |
|
Ok, let's go ahead since this seems safe enough and see how it goes |
|
@ericdallo Tests are fixed, the error isn't related to changes here which is interesting as other pipelines were green. |
Some LLM tool calls occasionally omit the `eca__` prefix for native tools, such as `write_file` instead of `eca__write_file`. This caused native tool dispatch to fail and could lead to repeated retries before the model recovered.` Resolve bare native ECA tool names to their canonical form, keep MCP tool names strict, and log auto-resolved native tool calls for debugging.
13a3538 to
4d07b35
Compare
Some LLM tool calls occasionally omit the
eca__prefix for native tools, such aswrite_fileinstead ofeca__write_file.This caused native tool dispatch to fail and could lead to repeated retries before the model recovered.`
Resolve bare native ECA tool names to their canonical form, keep MCP tool names strict, and log auto-resolved native tool calls for debugging.