Avoid mutable default for toolsets

Author: kingrubic

p4mcp/server.py

@@ -15,9 +15,9 @@
 class P4MCPServer:
     """Perforce MCP Server with improved structure"""
 
-    def __init__(self, session_id: str = None, readonly: bool = True, toolsets: list = [], search_transform: str = None, ssl_verify=None, log_dir: str = None, max_results=None, max_scan_rows=None):
+    def __init__(self, session_id: str = None, readonly: bool = True, toolsets: list = None, search_transform: str = None, ssl_verify=None, log_dir: str = None, max_results=None, max_scan_rows=None):
         self.readonly = readonly
-        self.toolsets = toolsets
+        self.toolsets = toolsets or []
         self.session_id = session_id
         self.search_transform = search_transform
 
@@ -177,4 +177,4 @@ def _apply_search_transforms(self):
 
     def run(self, transport="stdio", **kwargs):
         """Run the MCP server"""
-        self.mcp.run(transport=transport, **kwargs)
+        self.mcp.run(transport=transport, **kwargs)