Merge pull request #14 from dj-oyu/fix/tool-not-found-message
fix: include available tool names in tool-not-found error
This commit is contained in:
commit
ebc5fbfb47
1 changed files with 4 additions and 1 deletions
|
|
@ -82,11 +82,14 @@ func (r *ToolRegistry) ExecuteWithContext(
|
||||||
|
|
||||||
tool, ok := r.Get(name)
|
tool, ok := r.Get(name)
|
||||||
if !ok {
|
if !ok {
|
||||||
|
available := strings.Join(r.List(), ", ")
|
||||||
logger.ErrorCF("tool", "Tool not found",
|
logger.ErrorCF("tool", "Tool not found",
|
||||||
map[string]any{
|
map[string]any{
|
||||||
"tool": name,
|
"tool": name,
|
||||||
})
|
})
|
||||||
return ErrorResult(fmt.Sprintf("tool %q not found", name)).WithError(fmt.Errorf("tool not found"))
|
return ErrorResult(fmt.Sprintf(
|
||||||
|
"tool %q not found. Available tools: %s", name, available,
|
||||||
|
)).WithError(fmt.Errorf("tool not found"))
|
||||||
}
|
}
|
||||||
|
|
||||||
// If tool implements ContextualTool, set context
|
// If tool implements ContextualTool, set context
|
||||||
|
|
|
||||||
Loading…
Add table
Reference in a new issue