Refactor: Centralize logging and remove redundant code/docs

Co-authored-by: Mte90 <403283+Mte90@users.noreply.github.com>

Author: Copilot

PYCHARM_INTEGRATION.md

@@ -254,27 +254,6 @@ Error responses include a JSON object:
 }
 ```
 
-## Security Considerations
-
-1. **Local-only**: Server binds to 127.0.0.1 by default (localhost only)
-2. **File access**: Only indexed files are accessible
-3. **Path validation**: Project paths are validated and normalized
-4. **Database isolation**: Each project has separate database
-
-## Performance
-
-- **Concurrent indexing**: Uses thread pool for parallel file processing
-- **Connection pooling**: Efficient SQLite connection management
-- **Caching**: Analysis results cached to reduce database queries
-- **Vector search**: Uses sqlite-vector for fast similarity search
-
-## Limitations
-
-1. Maximum file size: 200KB by default (configurable)
-2. Text files only (binary files skipped)
-3. Requires OpenAI-compatible embedding API
-4. SQLite database per project (not suitable for extremely large projects)
-
 ## Example Integration (Python)
 
 ```python

logger.py

@@ -0,0 +1,26 @@
+"""
+Centralized logging configuration for PicoCode.
+"""
+import logging
+import sys
+
+# Configure root logger
+logging.basicConfig(
+    level=logging.INFO,
+    format='%(asctime)s - %(name)s - %(levelname)s - %(message)s',
+    handlers=[
+        logging.StreamHandler(sys.stdout)
+    ]
+)
+
+def get_logger(name: str) -> logging.Logger:
+    """
+    Get a logger instance for a module.
+    
+    Args:
+        name: Module name (usually __name__)
+    
+    Returns:
+        Configured logger instance
+    """
+    return logging.getLogger(name)

main.py

@@ -11,7 +11,7 @@
 
 from db import init_db, list_analyses, delete_analysis
 from analyzer import analyze_local_path_background, search_semantic, call_coding_model
-from config import CFG  # loads .env
+from config import CFG
 from projects import (
     create_project, get_project, get_project_by_id, list_projects,
     update_project_status, delete_project, get_or_create_project
@@ -20,6 +20,9 @@
     CreateProjectRequest, IndexProjectRequest, 
     QueryRequest
 )
+from logger import get_logger
+
+logger = get_logger(__name__)
 
 DATABASE = CFG.get("database_path", "codebase.db")
 MAX_FILE_SIZE = int(CFG.get("max_file_size", 200000))
@@ -43,8 +46,6 @@ async def lifespan(app: FastAPI):
 @app.post("/api/projects")
 def api_create_project(request: CreateProjectRequest):
     """Create or get a project with per-project database."""
-    import logging
-    logger = logging.getLogger(__name__)
 
     try:
         # Validate input
@@ -69,8 +70,6 @@ def api_create_project(request: CreateProjectRequest):
 @app.get("/api/projects")
 def api_list_projects():
     """List all projects."""
-    import logging
-    logger = logging.getLogger(__name__)
     try:
         projects = list_projects()
         return JSONResponse(projects)
@@ -82,8 +81,6 @@ def api_list_projects():
 @app.get("/api/projects/{project_id}")
 def api_get_project(project_id: str):
     """Get project details by ID."""
-    import logging
-    logger = logging.getLogger(__name__)
     try:
         project = get_project_by_id(project_id)
         if not project:
@@ -97,8 +94,6 @@ def api_get_project(project_id: str):
 @app.delete("/api/projects/{project_id}")
 def api_delete_project(project_id: str):
     """Delete a project and its database."""
-    import logging
-    logger = logging.getLogger(__name__)
     try:
         delete_project(project_id)
         return JSONResponse({"success": True})
@@ -113,8 +108,6 @@ def api_delete_project(project_id: str):
 @app.post("/api/projects/index")
 def api_index_project(request: IndexProjectRequest, background_tasks: BackgroundTasks):
     """Index/re-index a project in the background."""
-    import logging
-    logger = logging.getLogger(__name__)
     try:
         project = get_project_by_id(request.project_id)
         if not project:
@@ -151,8 +144,6 @@ def index_callback():
 @app.post("/api/query")
 def api_query(request: QueryRequest):
     """Query a project using semantic search (PyCharm-compatible)."""
-    import logging
-    logger = logging.getLogger(__name__)
     try:
         project = get_project_by_id(request.project_id)
         if not project:
@@ -301,14 +292,5 @@ def code_endpoint(request: Request):
     return JSONResponse({"response": resp, "used_context": used_context})
 
 
-@app.post("/analyses/{analysis_id}/delete")
-def delete_analysis_endpoint(analysis_id: int):
-    try:
-        delete_analysis(DATABASE, analysis_id)
-        return JSONResponse({"deleted": True})
-    except Exception as e:
-        return JSONResponse({"deleted": False, "error": str(e)}, status_code=500)
-
-
 if __name__ == "__main__":
     uvicorn.run("main:app", host=CFG.get("uvicorn_host", "127.0.0.1"), port=int(CFG.get("uvicorn_port", 8000)), reload=True)

plugin/README.md

@@ -45,22 +45,6 @@ The plugin ZIP will be in `build/distributions/`.
 - Python 3.8+ installed and in PATH
 - PicoCode backend dependencies installed (`pip install -r pyproject.toml`)
 
-## Project Structure
-
-```
-plugin/
-├── build.gradle.kts           # Gradle build configuration
-└── src/
-    └── main/
-        ├── kotlin/
-        │   └── com/picocode/
-        │       ├── PicoCodeToolWindowFactory.kt    # Tool window factory
-        │       └── PicoCodeToolWindowContent.kt    # Main UI and logic
-        └── resources/
-            └── META-INF/
-                └── plugin.xml                       # Plugin descriptor
-```
-
 ## Architecture
 
 1. **Server Management**: Plugin starts Python server as subprocess in project directory

projects.py

@@ -141,26 +141,19 @@ def create_project(project_path: str, name: Optional[str] = None) -> Dict[str, A
 
     try:
         # Normalize and validate path - prevents path traversal
-        # Resolve to canonical absolute path - prevents symlink and relative path issues
         project_path = os.path.abspath(os.path.realpath(project_path))
     except Exception as e:
         raise ValueError(f"Invalid project path: {e}")
 
-    # Additional validation: path must exist and be a directory
-    # Note: These checks are safe because project_path has been normalized above
-    # with realpath() which resolves symlinks and abspath() which makes it absolute
     try:
-        # Safe: path has been validated and normalized above
         path_exists = os.path.exists(project_path)  # nosec
         if not path_exists:
             raise ValueError(f"Project path does not exist")
 
-        # Safe: path has been validated and normalized above  
         is_directory = os.path.isdir(project_path)  # nosec
         if not is_directory:
             raise ValueError(f"Project path is not a directory")
     except (OSError, ValueError) as e:
-        # Re-raise ValueError, wrap OSError
         if isinstance(e, ValueError):
             raise
         raise ValueError(f"Cannot access project path")