bug: don't wipe out all tools on set (#2327)

ibuildthecloud · web-flow · commit 66a35219b97c · 2025-03-28T16:24:39.000-07:00
diff --git a/pkg/api/handlers/assistants.go b/pkg/api/handlers/assistants.go
@@ -468,7 +468,7 @@ func (a *AssistantHandler) SetTools(req api.Context) error {
 	}
 
 	for _, tool := range tools.Items {
-		if tool.Enabled && !tool.Builtin && strings.HasPrefix(tool.Name, system.ToolPrefix) {
+		if tool.Enabled && !tool.Builtin && !strings.HasPrefix(tool.ID, system.ToolPrefix) {
 			toolList = append(toolList, tool.ID)
 		}
 	}

Original file line number	Diff line number	Diff line change
`@@ -468,7 +468,7 @@ func (a *AssistantHandler) SetTools(req api.Context) error {`
`468`	`468`	`}`
`469`	`469`
`470`	`470`	`for _, tool := range tools.Items {`
`471`		`- if tool.Enabled && !tool.Builtin && strings.HasPrefix(tool.Name, system.ToolPrefix) {`
	`471`	`+ if tool.Enabled && !tool.Builtin && !strings.HasPrefix(tool.ID, system.ToolPrefix) {`
`472`	`472`	`toolList = append(toolList, tool.ID)`
`473`	`473`	`}`
`474`	`474`	`}`