improve logging

mcp/src/aileen3_mcp/cli_slides.py

@@ -10,10 +10,12 @@ import argparse
 import sys
 from pathlib import Path
 
+from .logging_utils import configure_logging
 from .media_tools import _build_reference, _extract_slides_flow, _probe_duration, _slides_json_path
 
 
 def main(argv: list[str] | None = None) -> int:
+    configure_logging()
     parser = argparse.ArgumentParser(description="Extract slides from a video using the Gemini pipeline.")
     parser.add_argument("video", type=Path, help="Path to the video file (mp4 recommended).")
     parser.add_argument(

mcp/src/aileen3_mcp/logging_utils.py

@@ -0,0 +1,49 @@
+from __future__ import annotations
+
+import logging
+import os
+from pathlib import Path
+
+# Logging paths
+BASE_CACHE = Path(os.environ.get("AILEEN3_CACHE_DIR", Path.home() / ".cache" / "aileen3"))
+LOG_DIR = BASE_CACHE / "logs"
+LOG_FILE = LOG_DIR / "aileen3-mcp.log"
+PACKAGE_LOGGER_NAME = "aileen3_mcp"
+
+
+def _resolve_level() -> int:
+    level_name = os.environ.get("AILEEN3_LOGLEVEL", "").upper() or "INFO"
+    level = getattr(logging, level_name, None)
+    if isinstance(level, int):
+        return level
+    # Fallback for non-standard levels
+    try:
+        return int(level_name)
+    except Exception:
+        return logging.INFO
+
+
+def configure_logging() -> Path:
+    """Configure package logging without turning on DEBUG for third-party libs."""
+    LOG_DIR.mkdir(parents=True, exist_ok=True)
+    level = _resolve_level()
+
+    pkg_logger = logging.getLogger(PACKAGE_LOGGER_NAME)
+    pkg_logger.setLevel(level)
+    pkg_logger.propagate = False  # keep third-party noise out of our handler
+
+    existing = None
+    for h in pkg_logger.handlers:
+        if isinstance(h, logging.FileHandler) and Path(getattr(h, "baseFilename", "")) == LOG_FILE:
+            existing = h
+            break
+
+    if existing:
+        existing.setLevel(level)
+    else:
+        handler = logging.FileHandler(LOG_FILE, encoding="utf-8")
+        handler.setLevel(level)
+        handler.setFormatter(logging.Formatter("%(asctime)s [%(levelname)s] %(name)s: %(message)s"))
+        pkg_logger.addHandler(handler)
+
+    return LOG_FILE

mcp/src/aileen3_mcp/media_tools.py

@@ -39,27 +39,6 @@ DEBUG_DIR = Path(tempfile.gettempdir()) / "aileen3-debug"
 if DEBUG:
     DEBUG_DIR.mkdir(parents=True, exist_ok=True)
 
-LOG_DIR = BASE_CACHE / "logs"
-LOG_DIR.mkdir(parents=True, exist_ok=True)
-LOG_FILE = LOG_DIR / "aileen3-mcp.log"
-
-
-def _ensure_file_logging():
-    root = logging.getLogger()
-    # Avoid adding duplicate handlers to root
-    for h in root.handlers:
-        if isinstance(h, logging.FileHandler) and Path(getattr(h, "baseFilename", "")) == LOG_FILE:
-            return
-    handler = logging.FileHandler(LOG_FILE, encoding="utf-8")
-    handler.setLevel(logging.DEBUG if DEBUG else logging.INFO)
-    fmt = logging.Formatter("%(asctime)s [%(levelname)s] %(name)s: %(message)s")
-    handler.setFormatter(fmt)
-    root.addHandler(handler)
-
-
-_ensure_file_logging()
-
-
 def _write_debug(reference: str, suffix: str, data: Any) -> None:
     if not DEBUG:
         return

mcp/src/aileen3_mcp/server.py

@@ -1,6 +1,5 @@
 from __future__ import annotations
 
-import logging
 import os
 from dataclasses import asdict, dataclass
 
@@ -10,6 +9,9 @@ import subprocess
 from fastmcp import FastMCP
 
 from aileen3_mcp.media_tools import register_media_tools, _silence_stdio, _YDLLogger
+from aileen3_mcp.logging_utils import configure_logging
+
+import logging
 
 log = logging.getLogger(__name__)
 
@@ -138,7 +140,7 @@ def make_app() -> FastMCP:
 
 
 def main() -> None:
-    logging.basicConfig(level=logging.INFO)
+    configure_logging()
     app = make_app()
     app.run()  # stdio transport by default