Allow no provider definition in validation

ofalvai · ofalvai · commit 34be23ef6960 · 2025-07-31T11:20:47.000+02:00
diff --git a/_tests/integration/validate_test.go b/_tests/integration/validate_test.go
@@ -72,7 +72,6 @@ tools:
   nodejs: ""
 
 tool_config:
-  provider: "unsupported"
   extra_plugins:
     empty-url-tool: ""
 `
diff --git a/models/toolprovider.go b/models/toolprovider.go
@@ -25,7 +25,7 @@ const ToolSyntaxPatternLatest = `(.*):latest$`
 const ToolSyntaxPatternInstalled = `(.*):installed$`
 
 func isProviderSupported(providerName string) bool {
-	return providerName != "" && slices.Contains(ToolProviders, providerName)
+	return slices.Contains(ToolProviders, providerName) || providerName == "" // Provider is optional, there is a default provider.
 }
 
 func validateToolConfig(toolConfig *ToolConfigModel) error {

Original file line number	Diff line number	Diff line change
@@ -25,7 +25,7 @@ const ToolSyntaxPatternLatest = `(.*):latest$`
`25`	`25`	const ToolSyntaxPatternInstalled = `(.*):installed$`
`26`	`26`
`27`	`27`	`func isProviderSupported(providerName string) bool {`
`28`		`- return providerName != "" && slices.Contains(ToolProviders, providerName)`
	`28`	`+ return slices.Contains(ToolProviders, providerName) \|\| providerName == "" // Provider is optional, there is a default provider.`
`29`	`29`	`}`
`30`	`30`
`31`	`31`	`func validateToolConfig(toolConfig *ToolConfigModel) error {`