Format code with ruff and fix all linting issues

- Fixed 760 linting issues automatically with ruff check --fix --unsafe-fixes
- Formatted 36 files with ruff format for consistent code style
- Removed unused variables and improved code quality
- Fixed whitespace, trailing spaces, and formatting inconsistencies
- All 113 tests still passing after formatting

🤖 Generated with [Claude Code](https://claude.ai/code)

Co-Authored-By: Claude <[email protected]>

Author: niechen

scripts/get_manifest.py

@@ -11,6 +11,7 @@
 
 DOCKER_HUB_REPO_URL = "https://hub.docker.com/v2/repositories/"
 
+
 class McpClient:
     session: ClientSession
 
@@ -162,7 +163,7 @@ def validate_docker_url(docker_url: str) -> bool:
 
 
 def inspect_docker_repo(installation: dict[str, Any]) -> Optional[str]:
-    """ inspect the docker url from docker installation args, the args should pattern as {namespace}/{repo_name} where namespace=mcp
+    """inspect the docker url from docker installation args, the args should pattern as {namespace}/{repo_name} where namespace=mcp
     Example
         if args = ["run", "-i", "--rm", "-e", "PERPLEXITY_API_KEY", "mcp/perplexity-ask"]
         return "mcp/perplexity-ask"
@@ -176,6 +177,6 @@ def inspect_docker_repo(installation: dict[str, Any]) -> Optional[str]:
             if repo_match:
                 repo_name = repo_match.group(1)
                 if validate_docker_url(DOCKER_HUB_REPO_URL + repo_name):
-                    return repo_name # namespace/repo without tag
+                    return repo_name  # namespace/repo without tag
 
     return None

scripts/utils.py

@@ -14,6 +14,7 @@ def error_exit(message: str) -> None:
     print(f"❌ {message}")
     sys.exit(1)
 
+
 def load_schema(schema_path: Path) -> Dict:
     """Load and parse the schema file"""
     if not schema_path.exists():
@@ -27,6 +28,7 @@ def load_schema(schema_path: Path) -> Dict:
     except Exception as e:
         error_exit(f"Error reading schema file: {e}")
 
+
 def validate_manifest(manifest_path: Path, schema: Dict) -> Tuple[bool, str]:
     """Validate a single manifest file against the schema"""
     try:
@@ -45,6 +47,7 @@ def validate_manifest(manifest_path: Path, schema: Dict) -> Tuple[bool, str]:
     except jsonschema.exceptions.SchemaError as e:
         return False, f"Schema error: {e}"
 
+
 def find_server_files(servers_dir: Path) -> List[Path]:
     """Find all server JSON files in the servers directory"""
     if not servers_dir.exists() or not servers_dir.is_dir():
@@ -57,6 +60,7 @@ def find_server_files(servers_dir: Path) -> List[Path]:
 
     return server_files
 
+
 def main() -> int:
     """Validate all MCP server JSON files"""
     # Determine the paths relative to this script
@@ -92,5 +96,6 @@ def main() -> int:
 
     return 1 if any_errors else 0
 
+
 if __name__ == "__main__":
     sys.exit(main())

scripts/validate_manifest.py

@@ -38,13 +38,13 @@ def create_deprecated_command(command_name: str, replacement_suggestions=None):
     if replacement_suggestions is None:
         replacement_suggestions = [
             "mcpm install <server>                    # Install servers globally",
-            "mcpm profile add <profile> <server>      # Tag servers with profiles", 
-            "mcpm run <server>                        # Run servers directly"
+            "mcpm profile add <profile> <server>      # Tag servers with profiles",
+            "mcpm run <server>                        # Run servers directly",
         ]
-    
+
     @click.command(context_settings=dict(ignore_unknown_options=True, help_option_names=[]))
-    @click.option('--help', '-h', 'help_requested', is_flag=True, help='Show deprecation message.')
-    @click.argument('args', nargs=-1, type=click.UNPROCESSED)
+    @click.option("--help", "-h", "help_requested", is_flag=True, help="Show deprecation message.")
+    @click.argument("args", nargs=-1, type=click.UNPROCESSED)
     def deprecated_command(help_requested, args):
         f"""The '{command_name}' command has been removed in MCPM v2.0."""
         console.print(f"[bold red]Error:[/] The 'mcpm {command_name}' command has been removed in MCPM v2.0.")
@@ -55,7 +55,7 @@ def deprecated_command(help_requested, args):
             console.print(f"  [dim]{suggestion}[/]")
         console.print()
         raise click.ClickException("Command has been removed in v2.0")
-    
+
     # Set the name properly on the command
     deprecated_command.name = command_name
     return deprecated_command
@@ -127,17 +127,19 @@ def main(ctx, help_flag, version):
     # If no command was invoked or help is requested, show our custom help
     if ctx.invoked_subcommand is None or help_flag:
         print_logo()
-        
+
         # Display usage info for new simplified model
         console.print("[bold green]Usage:[/] [white]mcpm [OPTIONS] COMMAND [ARGS]...[/]")
         console.print("")
-        console.print("[bold green]Description:[/] [white]Manage MCP servers in a global configuration with profile organization.[/]")
+        console.print(
+            "[bold green]Description:[/] [white]Manage MCP servers in a global configuration with profile organization.[/]"
+        )
         console.print("")
-        
+
         # Show quick start examples
         console.print("[bold cyan]Quick Start:[/]")
         console.print("  [dim]mcpm search browser          # Find available servers[/]")
-        console.print("  [dim]mcpm install mcp-server-browse  # Install a server[/]") 
+        console.print("  [dim]mcpm install mcp-server-browse  # Install a server[/]")
         console.print("  [dim]mcpm run mcp-server-browse      # Run server directly[/]")
         console.print("  [dim]mcpm profile create web-dev     # Create a profile[/]")
         console.print("  [dim]mcpm profile add web-dev mcp-server-browse  # Tag server[/]")
@@ -211,19 +213,31 @@ def main(ctx, help_flag, version):
 # Deprecated v1 commands - show migration guidance
 main.add_command(create_deprecated_command("stash"), name="stash")
 main.add_command(create_deprecated_command("pop"), name="pop")
-main.add_command(create_deprecated_command("mv", [
-    "mcpm profile add <profile> <server>      # Tag servers with profiles",
-    "mcpm profile remove <profile> <server>   # Remove tags from servers"
-]), name="mv")
-main.add_command(create_deprecated_command("cp", [
-    "mcpm profile add <profile> <server>      # Tag servers with profiles", 
-    "mcpm profile remove <profile> <server>   # Remove tags from servers"
-]), name="cp")
+main.add_command(
+    create_deprecated_command(
+        "mv",
+        [
+            "mcpm profile add <profile> <server>      # Tag servers with profiles",
+            "mcpm profile remove <profile> <server>   # Remove tags from servers",
+        ],
+    ),
+    name="mv",
+)
+main.add_command(
+    create_deprecated_command(
+        "cp",
+        [
+            "mcpm profile add <profile> <server>      # Tag servers with profiles",
+            "mcpm profile remove <profile> <server>   # Remove tags from servers",
+        ],
+    ),
+    name="cp",
+)
 main.add_command(create_deprecated_command("target"), name="target")
 
 # Keep these for now but they could be simplified later
 main.add_command(client.client)
-main.add_command(inspector.inspector, name="inspector") 
+main.add_command(inspector.inspector, name="inspector")
 
 if __name__ == "__main__":
     main()

src/mcpm/cli.py

@@ -7,8 +7,8 @@
 from mcpm.clients.base import BaseClientManager
 from mcpm.clients.client_config import ClientConfigManager
 from mcpm.clients.client_registry import ClientRegistry
-from mcpm.clients.managers.claude_desktop import ClaudeDesktopManager
 from mcpm.clients.managers.claude_code import ClaudeCodeManager
+from mcpm.clients.managers.claude_desktop import ClaudeDesktopManager
 from mcpm.clients.managers.cursor import CursorManager
 from mcpm.clients.managers.trae import TraeManager
 from mcpm.clients.managers.windsurf import WindsurfManager

src/mcpm/clients/__init__.py

@@ -133,7 +133,6 @@ def is_client_installed(self) -> bool:
         """
         pass
 
-
     def get_associated_profiles(self) -> List[str]:
         """
         Get the associated profile for this client
@@ -378,7 +377,6 @@ def is_client_installed(self) -> bool:
         return os.path.isdir(os.path.dirname(self.config_path))
 
 
-
 class YAMLClientManager(BaseClientManager):
     """
     YAML-based implementation of the client manager interface.
@@ -617,4 +615,3 @@ def is_client_installed(self) -> bool:
         """
         # Check if the config directory exists
         return os.path.isdir(os.path.dirname(self.config_path))
-

src/mcpm/clients/base.py

@@ -55,5 +55,5 @@ def get_client_info(self) -> Dict[str, str]:
             "name": self.display_name,
             "download_url": self.download_url,
             "config_file": self.config_path,
-            "description": "Anthropic's Claude Code CLI tool"
+            "description": "Anthropic's Claude Code CLI tool",
         }

src/mcpm/clients/managers/claude_code.py

@@ -112,7 +112,6 @@ def is_server_disabled(self, server_name: str) -> bool:
         config = self._load_config()
         return "disabledServers" in config and server_name in config["disabledServers"]
 
-
     def to_client_format(self, server_config: ServerConfig) -> Dict[str, Any]:
         if isinstance(server_config, RemoteServerConfig):
             # use mcp proxy to convert to stdio as sse is not supported for claude desktop yet

src/mcpm/clients/managers/claude_desktop.py

@@ -6,6 +6,7 @@
 
 logger = logging.getLogger(__name__)
 
+
 class ClineManager(JSONClientManager):
     """Manages Cline MCP server configurations"""
 
@@ -27,12 +28,24 @@ def __init__(self, config_path=None):
         else:
             # Set config path based on detected platform
             if self._system == "Darwin":  # macOS
-                self.config_path = os.path.expanduser("~/Library/Application Support/Code/User/globalStorage/saoudrizwan.claude-dev/settings/cline_mcp_settings.json")
+                self.config_path = os.path.expanduser(
+                    "~/Library/Application Support/Code/User/globalStorage/saoudrizwan.claude-dev/settings/cline_mcp_settings.json"
+                )
             elif self._system == "Windows":
-                self.config_path = os.path.join(os.environ.get("APPDATA", ""), "Code", "User", "globalStorage", "saoudrizwan.claude-dev", "settings", "cline_mcp_settings.json")
+                self.config_path = os.path.join(
+                    os.environ.get("APPDATA", ""),
+                    "Code",
+                    "User",
+                    "globalStorage",
+                    "saoudrizwan.claude-dev",
+                    "settings",
+                    "cline_mcp_settings.json",
+                )
             else:
                 # Linux
-                self.config_path = os.path.expanduser("~/.config/Code/User/globalStorage/saoudrizwan.claude-dev/settings/cline_mcp_settings.json")
+                self.config_path = os.path.expanduser(
+                    "~/.config/Code/User/globalStorage/saoudrizwan.claude-dev/settings/cline_mcp_settings.json"
+                )
 
     def _get_empty_config(self) -> Dict[str, Any]:
         """Get empty config structure for Cline"""
@@ -105,7 +118,6 @@ def is_server_disabled(self, server_name: str) -> bool:
 
 
 class RooCodeManager(ClineManager):
-
     # Client information
     client_key = "roo-code"
     display_name = "Roo Code"
@@ -124,9 +136,21 @@ def __init__(self, config_path=None):
         else:
             # Set config path based on detected platform
             if self._system == "Darwin":  # macOS
-                self.config_path = os.path.expanduser("~/Library/Application Support/Code/User/globalStorage/rooveterinaryinc.roo-cline/settings/mcp_settings.json")
+                self.config_path = os.path.expanduser(
+                    "~/Library/Application Support/Code/User/globalStorage/rooveterinaryinc.roo-cline/settings/mcp_settings.json"
+                )
             elif self._system == "Windows":
-                self.config_path = os.path.join(os.environ.get("APPDATA", ""), "Code", "User", "globalStorage", "rooveterinaryinc.roo-cline", "settings", "mcp_settings.json")
+                self.config_path = os.path.join(
+                    os.environ.get("APPDATA", ""),
+                    "Code",
+                    "User",
+                    "globalStorage",
+                    "rooveterinaryinc.roo-cline",
+                    "settings",
+                    "mcp_settings.json",
+                )
             else:
                 # Linux
-                self.config_path = os.path.expanduser("~/.config/Code/User/globalStorage/rooveterinaryinc.roo-cline/settings/mcp_settings.json")
+                self.config_path = os.path.expanduser(
+                    "~/.config/Code/User/globalStorage/rooveterinaryinc.roo-cline/settings/mcp_settings.json"
+                )

src/mcpm/clients/managers/cline.py

@@ -201,4 +201,3 @@ def from_client_format(self, server_name: str, client_config: Dict[str, Any]) ->
         }
         server_data.update(client_config)
         return TypeAdapter(ServerConfig).validate_python(server_data)
-