Address feedback about Windsurf change (#1214)

Based on internal feedback.

* Use `windsurf-jetbrains` for consistency.
* Print the windsurf options in the command help.

Author: dmjb

cmd/thv/app/client.go

@@ -40,7 +40,9 @@ Valid clients are:
   - cursor: Cursor editor
   - roo-code: Roo Code extension for VS Code
   - vscode: Visual Studio Code
-  - vscode-insider: Visual Studio Code Insiders edition`,
+  - vscode-insider: Visual Studio Code Insiders edition
+  - windsurf: Windsurf IDE
+  - windsurf-jetbrains: Windsurf for JetBrains IDEs`,
 	Args: cobra.ExactArgs(1),
 	RunE: clientRegisterCmdFunc,
 }
@@ -55,7 +57,9 @@ Valid clients are:
   - cursor: Cursor editor
   - roo-code: Roo Code extension for VS Code
   - vscode: Visual Studio Code
-  - vscode-insider: Visual Studio Code Insiders edition`,
+  - vscode-insider: Visual Studio Code Insiders edition
+  - windsurf: Windsurf IDE
+  - windsurf-jetbrains: Windsurf for JetBrains IDEs`,
 	Args: cobra.ExactArgs(1),
 	RunE: clientRemoveCmdFunc,
 }
@@ -148,12 +152,12 @@ func clientRegisterCmdFunc(cmd *cobra.Command, args []string) error {
 
 	// Validate the client type
 	switch clientType {
-	case "roo-code", "cline", "cursor", "claude-code", "vscode-insider", "vscode", "windsurf", "windsurf-intellij":
+	case "roo-code", "cline", "cursor", "claude-code", "vscode-insider", "vscode", "windsurf", "windsurf-jetbrains":
 		// Valid client type
 	default:
 		return fmt.Errorf(
 			"invalid client type: %s (valid types: roo-code, cline, cursor, claude-code, vscode, "+
-				"vscode-insider, windsurf, windsurf-intellij)",
+				"vscode-insider, windsurf, windsurf-jetbrains)",
 			clientType)
 	}
 
@@ -179,12 +183,12 @@ func clientRemoveCmdFunc(cmd *cobra.Command, args []string) error {
 
 	// Validate the client type
 	switch clientType {
-	case "roo-code", "cline", "cursor", "claude-code", "vscode-insider", "vscode", "windsurf", "windsurf-intellij":
+	case "roo-code", "cline", "cursor", "claude-code", "vscode-insider", "vscode", "windsurf", "windsurf-jetbrains":
 		// Valid client type
 	default:
 		return fmt.Errorf(
 			"invalid client type: %s (valid types: roo-code, cline, cursor, claude-code, vscode, "+
-				"vscode-insider, windsurf, windsurf-intellij)",
+				"vscode-insider, windsurf, windsurf-jetbrains)",
 			clientType)
 	}
 

docs/cli/thv_client_register.md

@@ -40,8 +40,8 @@ const (
 	ClaudeCode MCPClient = "claude-code"
 	// Windsurf represents the Windsurf IDE.
 	Windsurf MCPClient = "windsurf"
-	// WindsurfIntelliJ represents the Windsurf plugin for IntelliJ.
-	WindsurfIntelliJ MCPClient = "windsurf-intellij"
+	// WindsurfJetBrains represents the Windsurf plugin for JetBrains.
+	WindsurfJetBrains MCPClient = "windsurf-jetbrains"
 )
 
 // Extension is extension of the client config file.
@@ -207,8 +207,8 @@ var supportedClientIntegrations = []mcpClientConfig{
 		MCPServersUrlLabel:            "serverUrl",
 	},
 	{
-		ClientType:           WindsurfIntelliJ,
-		Description:          "Windsurf plugin for IntelliJ",
+		ClientType:           WindsurfJetBrains,
+		Description:          "Windsurf plugin for JetBrains IDEs",
 		SettingsFile:         "mcp_config.json",
 		MCPServersPathPrefix: "/mcpServers",
 		RelPath:              []string{".codeium"},

docs/cli/thv_client_remove.md

@@ -315,9 +315,9 @@ func TestSuccessfulClientConfigOperations(t *testing.T) {
 			case Windsurf:
 				assert.Contains(t, string(content), `"mcpServers":`,
 					"Windsurf config should contain mcpServers key")
-			case WindsurfIntelliJ:
+			case WindsurfJetBrains:
 				assert.Contains(t, string(content), `"mcpServers":`,
-					"WindsurfIntelliJ config should contain mcpServers key")
+					"WindsurfJetBrains config should contain mcpServers key")
 			}
 		}
 	})
@@ -343,7 +343,7 @@ func TestSuccessfulClientConfigOperations(t *testing.T) {
 			case VSCode, VSCodeInsider:
 				assert.Contains(t, string(content), testURL,
 					"VSCode config should contain the server URL")
-			case Cursor, RooCode, ClaudeCode, Cline, Windsurf, WindsurfIntelliJ:
+			case Cursor, RooCode, ClaudeCode, Cline, Windsurf, WindsurfJetBrains:
 				assert.Contains(t, string(content), testURL,
 					"Config should contain the server URL")
 			}