Lint: Fix import order, remove unused imports, and clean up per ruff recommendations

Author: silkspace

run_graphistry_mcp.py

@@ -82,8 +82,8 @@ def find_available_port(start_port=8080, max_attempts=20):
 
 # Check if MCP server port is available (commonly uses port 8080)
 if not is_port_available(8080):
-    logger.warning(f"⚠️ Port 8080 is already in use by another process")
-    print(f"⚠️ Warning: Port 8080 is already in use by another process.", file=sys.stderr)
+    logger.warning("⚠️ Port 8080 is already in use by another process")
+    print("⚠️ Warning: Port 8080 is already in use by another process.", file=sys.stderr)
 
     # Find an alternative port
     alt_port = find_available_port(8081)
@@ -94,7 +94,7 @@ def find_available_port(start_port=8080, max_attempts=20):
         # Set environment variable with alternative port suggestion
         os.environ["GRAPHISTRY_SUGGESTED_PORT"] = str(alt_port)
     else:
-        print(f"❌ No alternative ports available in range 8081-8100.", file=sys.stderr)
+        print("❌ No alternative ports available in range 8081-8100.", file=sys.stderr)
         logger.warning("No alternative ports available in range 8081-8100")
 
     try:
@@ -106,7 +106,7 @@ def find_available_port(start_port=8080, max_attempts=20):
                 if result.stdout:
                     print(f"Process using port 8080:\n{result.stdout}", file=sys.stderr)
                     logger.info(f"Process using port 8080:\n{result.stdout}")
-                    print(f"To free the port, you can terminate the process using: kill <PID>", file=sys.stderr)
+                    print("To free the port, you can terminate the process using: kill <PID>", file=sys.stderr)
             except Exception as e:
                 logger.error(f"Unable to check what's using port 8080: {e}")
     except Exception as e:
@@ -255,7 +255,7 @@ def install_dependency(package):
             env_username = os.environ.get("GRAPHISTRY_USERNAME")
             env_password = os.environ.get("GRAPHISTRY_PASSWORD")
 
-            print(f"Environment variables:")
+            print("Environment variables:")
             print(f"GRAPHISTRY_USERNAME found: {'Yes' if env_username else 'No'}")
             print(f"GRAPHISTRY_PASSWORD found: {'Yes' if env_password else 'No'}")
 
@@ -268,7 +268,7 @@ def install_dependency(package):
                 env_values = dotenv_values(dotenv_path)
                 dotenv_username = env_values.get('GRAPHISTRY_USERNAME')
                 dotenv_password = env_values.get('GRAPHISTRY_PASSWORD')
-                print(f"Credentials from .env file:")
+                print("Credentials from .env file:")
                 print(f"GRAPHISTRY_USERNAME found: {'Yes' if dotenv_username else 'No'}")
                 print(f"GRAPHISTRY_PASSWORD found: {'Yes' if dotenv_password else 'No'}")
 

src/graphistry_mcp_server/server.py

@@ -9,10 +9,8 @@
 GPU-accelerated visualization capabilities.
 """
 
-import asyncio
-import json
 import logging
-from typing import Any, Dict, List, Optional, Tuple, Union, cast
+from typing import Any, Dict, Optional
 import os
 
 from dotenv import load_dotenv

tests/test_server.py

@@ -3,7 +3,7 @@
 """
 import json
 import os
-from typing import Dict, Any, List
+from typing import Dict, Any
 import pytest
 import graphistry
 from dotenv import load_dotenv
@@ -21,7 +21,7 @@
 print(f"\nInitializing Graphistry with username: {username}")
 graphistry.register(api=3, protocol="https", server="hub.graphistry.com", username=username, password=password)
 
-def parse_mcp_response(response) -> Dict[str, Any]:
+def parse_mcp_response(response: Any) -> Dict[str, Any]:
     """Parse MCP response into a dictionary."""
     if isinstance(response, list) and len(response) > 0:
         content = response[0]