feat: cleaned

api/server.py

@@ -37,9 +37,8 @@ import uvicorn
 from fastapi import FastAPI, File, Form, HTTPException, UploadFile
 from fastapi.middleware.cors import CORSMiddleware
 from fastapi.middleware.gzip import GZipMiddleware
-from pydantic import BaseModel, Field
+from pydantic import BaseModel, Field, field_validator
 
-from banner import print_banner
 from core import ConversionError, ConversionResult, DocumentConverter, SUPPORTED_EXTENSIONS
 from extraction.generic_json_extractor import extract
 from extraction.label_mapper import validate_mappings
@@ -100,7 +99,6 @@ def _start_ping_scheduler() -> None:
 @asynccontextmanager
 async def lifespan(app: FastAPI):
     """Application lifespan handler — runs startup and shutdown logic."""
-    print_banner()
     logger.info(
         "MarkItDown API starting | version=2.1.0 | host=0.0.0.0:7860 | started_at=%s",
         datetime.now(timezone.utc).strftime("%Y-%m-%d %H:%M:%S UTC"),

app/core/config/settings.py

@@ -1,28 +0,0 @@
-"""Application settings via Pydantic Settings.
-
-Configuration is driven by environment variables with sensible defaults
-for local development.  In production, set these via the container's
-environment (Docker Compose, K8s ConfigMap/Secret, etc.).
-"""
-
-from __future__ import annotations
-
-import os
-
-
-class Settings:
-    """Simple settings container — no external dependencies required."""
-
-    SERVICE_NAME: str = "reconciliation-file-processing-service"
-    API_VERSION: str = "2.1.0"
-    ENVIRONMENT: str = os.getenv("ENVIRONMENT", "development")
-
-
-_settings: Settings | None = None
-
-
-def get_settings() -> Settings:
-    global _settings
-    if _settings is None:
-        _settings = Settings()
-    return _settings

banner.py

@@ -1,21 +1,23 @@
+import os
+
 import pyfiglet
 from rich.console import Console
 from rich.rule import Rule
 from rich.text import Text
 
-from app.core.config.settings import get_settings
-
 _console = Console()
 
+SERVICE_NAME = os.getenv("SERVICE_NAME", "reconciliation-file-processing-service")
+API_VERSION  = os.getenv("API_VERSION", "2.1.0")
+ENVIRONMENT  = os.getenv("ENVIRONMENT", "development")
 
-def print_banner() -> None:
-    settings = get_settings()
 
+def print_banner() -> None:
     art = pyfiglet.figlet_format("ValidOps", font="slant")
 
     _console.print(Text(art, style="bold cyan"))
-    _console.print(f"  [bold white]{'Service:':<14}[/bold white]  [cyan]{settings.SERVICE_NAME}[/cyan]")
-    _console.print(f"  [bold white]{'Version:':<14}[/bold white]  [cyan]v{settings.API_VERSION}[/cyan]")
-    _console.print(f"  [bold white]{'Environment:':<14}[/bold white]  [cyan]{settings.ENVIRONMENT}[/cyan]")
+    _console.print(f"  [bold white]{'Service:':<14}[/bold white]  [cyan]{SERVICE_NAME}[/cyan]")
+    _console.print(f"  [bold white]{'Version:':<14}[/bold white]  [cyan]v{API_VERSION}[/cyan]")
+    _console.print(f"  [bold white]{'Environment:':<14}[/bold white]  [cyan]{ENVIRONMENT}[/cyan]")
     _console.print(Rule(style="dim cyan"))
     _console.print()

main.py

@@ -14,11 +14,13 @@ import os
 
 import uvicorn
 
+from banner import print_banner
 from logger import get_logger
 
 logger = get_logger(__name__)
 
 if __name__ == "__main__":
+    print_banner()
     workers = int(os.getenv("WORKERS", os.cpu_count() or 1))
     port = int(os.getenv("PORT", "7860"))
 

start.sh

@@ -66,6 +66,11 @@ else
     log_warn "Import smoke test failed. Check $LOG_FILE for details. Continuing startup."
 fi
 
+# ---------------------------------------------------------------------------
+# Print startup banner
+# ---------------------------------------------------------------------------
+python -c "from banner import print_banner; print_banner()" 2>/dev/null || true
+
 # ---------------------------------------------------------------------------
 # Start uvicorn
 # ---------------------------------------------------------------------------