remove all instances of logger.py (see issue #1)

Author: lamaalrajih

docs/development.md

@@ -61,8 +61,7 @@ kicad-mcp/
 │       ├── kicad_utils.py          # KiCad-specific functions
 │       ├── python_path.py          # Python path setup for KiCad modules
 │       ├── drc_history.py          # DRC history tracking
-│       ├── env.py                  # Environment variable handling
-│       └── logger.py               # Logging utilities
+│       └── env.py                  # Environment variable handling
 ```
 
 ## Adding New Features
@@ -254,8 +253,7 @@ For debugging, use:
 
 1. The Python debugger (pdb)
 2. Print statements to the console (captured in Claude Desktop logs)
-3. Structured logging via the Logger class
-4. The MCP Inspector tool
+3. The MCP Inspector tool
 
 ## Performance Considerations
 

kicad_mcp/context.py

@@ -3,16 +3,12 @@
 """
 from contextlib import asynccontextmanager
 from dataclasses import dataclass
-from typing import AsyncIterator, Optional, Dict, Any
+from typing import AsyncIterator, Dict, Any
 
 from mcp.server.fastmcp import FastMCP
 
-from kicad_mcp.utils.logger import Logger
 from kicad_mcp.utils.python_path import setup_kicad_python_path
 
-# Create logger for this module
-logger = Logger()
-
 @dataclass
 class KiCadAppContext:
     """Type-safe context for KiCad MCP server."""
@@ -36,12 +32,12 @@ async def kicad_lifespan(server: FastMCP) -> AsyncIterator[KiCadAppContext]:
     Yields:
         KiCadAppContext: A typed context object shared across all handlers
     """
-    logger.info("Starting KiCad MCP server initialization")
+    print("Starting KiCad MCP server initialization")
 
     # Initialize resources on startup
-    logger.info("Setting up KiCad Python modules")
+    print("Setting up KiCad Python modules")
     kicad_modules_available = setup_kicad_python_path()
-    logger.info(f"KiCad Python modules available: {kicad_modules_available}")
+    print(f"KiCad Python modules available: {kicad_modules_available}")
 
     # Create in-memory cache for expensive operations
     cache: Dict[str, Any] = {}
@@ -53,37 +49,37 @@ async def kicad_lifespan(server: FastMCP) -> AsyncIterator[KiCadAppContext]:
         # Import any KiCad modules that should be preloaded
         if kicad_modules_available:
             try:
-                logger.info("Preloading KiCad Python modules")
+                print("Preloading KiCad Python modules")
 
                 # Core PCB module used in multiple tools
                 import pcbnew
-                logger.info(f"Successfully preloaded pcbnew module: {getattr(pcbnew, 'GetBuildVersion', lambda: 'unknown')()}")
+                print(f"Successfully preloaded pcbnew module: {getattr(pcbnew, 'GetBuildVersion', lambda: 'unknown')()}")
                 cache["pcbnew_version"] = getattr(pcbnew, "GetBuildVersion", lambda: "unknown")()
             except ImportError as e:
-                logger.warning(f"Failed to preload some KiCad modules: {str(e)}")
+                print(f"Failed to preload some KiCad modules: {str(e)}")
 
         # Yield the context to the server - server runs during this time
-        logger.info("KiCad MCP server initialization complete")
+        print("KiCad MCP server initialization complete")
         yield KiCadAppContext(
             kicad_modules_available=kicad_modules_available,
             cache=cache
         )
     finally:
         # Clean up resources when server shuts down
-        logger.info("Shutting down KiCad MCP server")
+        print("Shutting down KiCad MCP server")
 
         # Clear the cache
         if cache:
-            logger.debug(f"Clearing cache with {len(cache)} entries")
+            print(f"Clearing cache with {len(cache)} entries")
             cache.clear()
 
         # Clean up any temporary directories
         import shutil
         for temp_dir in created_temp_dirs:
             try:
-                logger.debug(f"Removing temporary directory: {temp_dir}")
+                print(f"Removing temporary directory: {temp_dir}")
                 shutil.rmtree(temp_dir, ignore_errors=True)
             except Exception as e:
-                logger.error(f"Error cleaning up temporary directory {temp_dir}: {str(e)}")
+                print(f"Error cleaning up temporary directory {temp_dir}: {str(e)}")
 
-        logger.info("KiCad MCP server shutdown complete")
+        print("KiCad MCP server shutdown complete")

kicad_mcp/resources/drc_resources.py

@@ -2,19 +2,13 @@
 Design Rule Check (DRC) resources for KiCad PCB files.
 """
 import os
-import json
-import tempfile
-from typing import Dict, Any, List
+
 from mcp.server.fastmcp import FastMCP
 
 from kicad_mcp.utils.file_utils import get_project_files
-from kicad_mcp.utils.logger import Logger
 from kicad_mcp.utils.drc_history import get_drc_history
 from kicad_mcp.tools.drc_impl.cli_drc import run_drc_via_cli
 
-# Create logger for this module
-logger = Logger()
-
 def register_drc_resources(mcp: FastMCP) -> None:
     """Register DRC resources with the MCP server.
     
@@ -32,7 +26,7 @@ def get_drc_history_report(project_path: str) -> str:
         Returns:
             Markdown-formatted DRC history report
         """
-        logger.info(f"Generating DRC history report for project: {project_path}")
+        print(f"Generating DRC history report for project: {project_path}")
 
         if not os.path.exists(project_path):
             return f"Project not found: {project_path}"
@@ -151,7 +145,7 @@ def get_drc_report(project_path: str) -> str:
         Returns:
             Markdown-formatted DRC report
         """
-        logger.info(f"Generating DRC report for project: {project_path}")
+        print(f"Generating DRC report for project: {project_path}")
 
         if not os.path.exists(project_path):
             return f"Project not found: {project_path}"
@@ -162,7 +156,7 @@ def get_drc_report(project_path: str) -> str:
             return "PCB file not found in project"
 
         pcb_file = files["pcb"]
-        logger.info(f"Found PCB file: {pcb_file}")
+        print(f"Found PCB file: {pcb_file}")
 
         # Try to run DRC via command line
         drc_results = run_drc_via_cli(pcb_file)

kicad_mcp/server.py

@@ -28,33 +28,29 @@
 from kicad_mcp.prompts.pattern_prompts import register_pattern_prompts
 
 # Import utils
-from kicad_mcp.utils.logger import Logger
 from kicad_mcp.utils.python_path import setup_kicad_python_path
 
 # Import context management
 from kicad_mcp.context import kicad_lifespan
 
-# Create logger for this module
-logger = Logger()
-
 def create_server() -> FastMCP:
     """Create and configure the KiCad MCP server."""
-    logger.info("Initializing KiCad MCP server")
+    print("Initializing KiCad MCP server")
 
     # Try to set up KiCad Python path
     kicad_modules_available = setup_kicad_python_path()
 
     if kicad_modules_available:
-        logger.info("KiCad Python modules successfully configured")
+        print("KiCad Python modules successfully configured")
     else:
-        logger.warning("KiCad Python modules not available - some features will be disabled")
+        print("KiCad Python modules not available - some features will be disabled")
 
     # Initialize FastMCP server
     mcp = FastMCP("KiCad", lifespan=kicad_lifespan)
-    logger.info("Created FastMCP server instance with lifespan management")
+    print("Created FastMCP server instance with lifespan management")
 
     # Register resources
-    logger.debug("Registering resources...")
+    print("Registering resources...")
     register_project_resources(mcp)
     register_file_resources(mcp)
     register_drc_resources(mcp)
@@ -63,7 +59,7 @@ def create_server() -> FastMCP:
     register_pattern_resources(mcp)
 
     # Register tools
-    logger.debug("Registering tools...")
+    print("Registering tools...")
     register_project_tools(mcp)
     register_analysis_tools(mcp)
     register_export_tools(mcp)
@@ -73,11 +69,11 @@ def create_server() -> FastMCP:
     register_pattern_tools(mcp)
 
     # Register prompts
-    logger.debug("Registering prompts...")
+    print("Registering prompts...")
     register_prompts(mcp)
     register_drc_prompts(mcp)
     register_bom_prompts(mcp)
     register_pattern_prompts(mcp)
 
-    logger.info("Server initialization complete")
+    print("Server initialization complete")
     return mcp