fix: removes auto-discovery check for rm (#144)

ChrisJBurns · web-flow · commit f82b9f746c27 · 2025-04-09T06:34:03.000+03:00
Signed-off-by: ChrisJBurns &lt;29541485+ChrisJBurns@users.noreply.github.com&gt;
diff --git a/cmd/thv/rm.go b/cmd/thv/rm.go
@@ -118,7 +118,7 @@ func rmCmdFunc(_ *cobra.Command, args []string) error {
 
 func shouldRemoveClientConfig() bool {
 	c := config.GetConfig()
-	return len(c.Clients.RegisteredClients) > 0 && c.Clients.AutoDiscovery
+	return len(c.Clients.RegisteredClients) > 0 || c.Clients.AutoDiscovery
 }
 
 // updateClientConfigurations updates client configuration files with the MCP server URL

Original file line number	Diff line number	Diff line change
`@@ -118,7 +118,7 @@ func rmCmdFunc(_ *cobra.Command, args []string) error {`
`118`	`118`
`119`	`119`	`func shouldRemoveClientConfig() bool {`
`120`	`120`	`c := config.GetConfig()`
`121`		`- return len(c.Clients.RegisteredClients) > 0 && c.Clients.AutoDiscovery`
	`121`	`+ return len(c.Clients.RegisteredClients) > 0 \|\| c.Clients.AutoDiscovery`
`122`	`122`	`}`
`123`	`123`
`124`	`124`	`// updateClientConfigurations updates client configuration files with the MCP server URL`