fix: windows config path for cursor and windsurf

Author: calmini

src/mcpm/clients/managers/continue_extension.py

@@ -43,12 +43,10 @@ def __init__(self, config_path=None):
             self.config_path = config_path
         else:
             # Set config path based on detected platform
-            if os.name == "Darwin":  # macOS
-                self.config_path = os.path.expanduser("~/.continue/config.yaml")
-            elif os.name == "Windows":
+            if self._system == "Windows":
                 self.config_path = os.path.join(os.environ.get("USERPROFILE", ""), ".continue", "config.yaml")
             else:
-                # Linux
+                # MacOS or Linux
                 self.config_path = os.path.expanduser("~/.continue/config.yaml")
 
             # Also check for workspace config

src/mcpm/clients/managers/cursor.py

@@ -31,13 +31,11 @@ def __init__(self, config_path=None):
             self.config_path = config_path
         else:
             # Set config path based on detected platform
-            if self._system == "Darwin":  # macOS
-                self.config_path = os.path.expanduser("~/Library/Application Support/Cursor/User/mcp_config.json")
-            elif self._system == "Windows":
-                self.config_path = os.path.join(os.environ.get("APPDATA", ""), "Cursor", "User", "mcp_config.json")
+            if self._system == "Windows":
+                self.config_path = os.path.join(os.environ.get("USERPROFILE", ""), ".cursor", "mcp.json")
             else:
-                # Linux
-                self.config_path = os.path.expanduser("~/.config/Cursor/User/mcp_config.json")
+                # MacOS or Linux
+                self.config_path = os.path.expanduser("~/.cursor/mcp.json")
 
     def _get_empty_config(self) -> Dict[str, Any]:
         """Get empty config structure for Cursor"""

src/mcpm/clients/managers/goose.py

@@ -41,12 +41,10 @@ def __init__(self, config_path=None):
             self.config_path = config_path
         else:
             # Set config path based on detected platform
-            if os.name == "Darwin":  # macOS
-                self.config_path = os.path.expanduser("~/.config/goose/config.yaml")
-            elif os.name == "Windows":
+            if self._system == "Windows":
                 self.config_path = os.path.join(os.environ.get("USERPROFILE", ""), ".config", "goose", "config.yaml")
             else:
-                # Linux
+                # MacOS or Linux
                 self.config_path = os.path.expanduser("~/.config/goose/config.yaml")
 
             # Also check for workspace config

src/mcpm/clients/managers/windsurf.py

@@ -32,14 +32,12 @@ def __init__(self, config_path=None):
             self.config_path = config_path
         else:
             # Set config path based on detected platform
-            if self._system == "Darwin":  # macOS
-                self.config_path = os.path.expanduser("~/.codeium/windsurf/mcp_config.json")
-            elif self._system == "Windows":
+            if self._system == "Windows":
                 self.config_path = os.path.join(
-                    os.environ.get("LOCALAPPDATA", ""), "Codeium", "windsurf", "mcp_config.json"
+                    os.environ.get("USERPROFILE", ""), ".codeium", "windsurf", "mcp_config.json"
                 )
             else:
-                # Linux
+                # MacOS or Linux
                 self.config_path = os.path.expanduser("~/.codeium/windsurf/mcp_config.json")
 
     def to_client_format(self, server_config: ServerConfig) -> Dict[str, Any]:

src/mcpm/commands/client.py

@@ -83,6 +83,16 @@ def set_client(client_name):
 
     CLIENT is the name of the client to set as active.
     """
+    # block the user to change client with a specific profile activated, or user will fail to deactivate a profile
+    # cause mcpm router is not attached to the client switched
+    active_profile = ClientRegistry.get_active_profile()
+    if active_profile:
+        console.print(
+            f"[bold blue]Note:[/] Profile '{active_profile}' needs to be deactivated before switching clients."
+        )
+        console.print("Please run [bold cyan]mcpm deactivate[/] to release the current profile.")
+        return
+
     # Get the list of supported clients
     supported_clients = ClientRegistry.get_supported_clients()
 

src/mcpm/commands/server_operations/common.py

@@ -54,9 +54,11 @@ def client_get_server(client: str, server: str) -> ServerConfig | None:
 
 def profile_add_server(profile: str, server_config: ServerConfig, force: bool = False) -> bool:
     profile_manager = ProfileConfigManager()
-    if not profile_manager.get_profile(profile):
+    profile_info = profile_manager.get_profile(profile)
+    if profile_info is None:
         console.print(f"[bold red]Error:[/] Profile '{profile}' not found.")
         return False
+
     if profile_manager.get_profile_server(profile, server_config.name) and not force:
         console.print(f"[bold red]Error:[/] Server '{server_config.name}' already exists in {profile}.")
         console.print("Use --force to override.")