refactor: #85 T1 extrait le domaine health vers backend/routers (comportement inchange)

backend/main.py

@@ -398,19 +398,6 @@ class ReloadResponse(BaseModel):
     vaults: dict[str, Any] = Field(description="Per-vault file counts after reload")
 
 
-class HealthResponse(BaseModel):
-    """Application health status."""
-    status: str = Field(description="Health status ('ok' or 'error')")
-    version: str = Field(description="Application version (x.y.z — latest release tag)")
-    vaults: int = Field(description="Number of configured vaults")
-    total_files: int = Field(description="Total indexed files across all vaults")
-    total_tokens: int = Field(description="Total indexed tokens (approx.) across all vaults", default=0)
-    last_full_index_ts: str = Field(description="ISO timestamp of last full index rebuild", default="")
-    uptime_seconds: int = Field(description="Server uptime in seconds", default=0)
-    git_describe: str = Field(default="", description="Full git describe string (commits beyond tag), empty if no git")
-    git_commit: str = Field(default="", description="Short HEAD commit hash, empty if no git")
-
-
 class DirectoryCreateRequest(BaseModel):
     """Request to create a new directory."""
     path: str = Field(description="Relative path of the new directory")
@@ -867,7 +854,7 @@ async def lifespan(app: FastAPI):
     _search_executor = None
 
 
-from backend.version import get_git_commit, get_git_describe, get_version
+from backend.version import get_version
 
 app = FastAPI(
     title="ObsiGate API",
@@ -959,6 +946,7 @@ except Exception:  # pragma: no cover - WeasyPrint/GTK missing
 from backend.ai_routes import router as ai_router
 from backend.bookslm_routes import router as bookslm_router
 from backend.export import ExportError, export_epub, export_html, export_md_bundle
+from backend.routers.health import router as health_router
 from backend.saved_searches import delete_saved, get_saved, save_search
 from backend.share import (
     create_share,
@@ -981,6 +969,7 @@ app.include_router(auth_router)
 app.include_router(ai_router)
 app.include_router(bookslm_router)
 app.include_router(skills_router)
+app.include_router(health_router)  # ROADMAP #85 T1 — System / health
 
 # Admin Dashboard endpoints (system stats, audit logs, backups, stream)
 try:
@@ -1306,125 +1295,9 @@ def _render_markdown(raw_md: str, vault_name: str, current_file_path: Path | Non
 
 
 # ---------------------------------------------------------------------------
-# API Endpoints
+# API Endpoints — System / health : voir backend.routers.health (#85 T1)
 # ---------------------------------------------------------------------------
 
-@app.get("/api/health", response_model=HealthResponse)
-async def api_health():
-    """Health check endpoint for Docker and monitoring.
-
-    Returns:
-        Application status, version, vault count and total file count.
-    """
-    total_files = sum(len(v["files"]) for v in index.values())
-    total_tokens = sum(len(v.get("files", [])) * 1000 for v in index.values())  # rough approx
-    import time
-
-    from backend.indexer import _last_full_index_ts
-    uptime = int(time.time() - _SERVER_START_TIME) if '_SERVER_START_TIME' in globals() else 0
-    return {
-        "status": "ok",
-        "version": app.version,
-        "vaults": len(index),
-        "total_files": total_files,
-        "total_tokens": total_tokens,
-        "last_full_index_ts": _last_full_index_ts,
-        "uptime_seconds": uptime,
-        "git_describe": get_git_describe(),
-        "git_commit": get_git_commit(),
-    }
-
-
-@app.get("/api/health/detailed", response_model=HealthResponse)
-async def api_health_detailed(current_user=Depends(require_admin)):
-    """Detailed health check — admin only.
-
-    Returns enriched metrics including memory, disk, SSE connections, and backup stats.
-    """
-
-    import psutil
-
-    from backend.admin import _count_active_sessions, _get_disk_stats
-    from backend.indexer import _last_full_index_ts, index
-    
-    total_files = sum(len(v["files"]) for v in index.values())
-    total_tokens = sum(len(v.get("files", [])) * 1000 for v in index.values())
-    import time
-    uptime = int(time.time() - _SERVER_START_TIME) if '_SERVER_START_TIME' in globals() else 0
-    
-    # Memory
-    vm = psutil.virtual_memory()
-    mem_used_mb = round(vm.used / (1024 ** 2), 1)
-    mem_total_mb = round(vm.total / (1024 ** 2), 1)
-    mem_pct = round(vm.percent, 1)
-    
-    # CPU
-    cpu_pct = psutil.cpu_percent(interval=None)
-    
-    # Disk
-    disk_used_gb, disk_total_gb = _get_disk_stats()
-    disk_free_gb = round(disk_total_gb - disk_used_gb, 2)
-    disk_pct = round((disk_used_gb / disk_total_gb * 100) if disk_total_gb > 0 else 0, 1)
-    
-    # SSE connections (approximation)
-    active_sessions = _count_active_sessions()
-    
-    # Backups
-    from backend.admin import _scan_backups
-    backup_rows = _scan_backups()
-    total_backups = len(backup_rows)
-    total_backup_size_mb = round(sum(r["size"] for r in backup_rows) / (1024 ** 2), 2)
-    oldest_backup_age_days = 0.0
-    if backup_rows:
-        now_ts = int(time.time())
-        oldest_ts = min(r["timestamp"] for r in backup_rows)
-        oldest_backup_age_days = round((now_ts - oldest_ts) / 86400, 2)
-    
-    # Index details
-    index_detail = {}
-    for name, data in index.items():
-        index_detail[name] = {
-            "file_count": len(data["files"]),
-            "tag_count": len(data["tags"]),
-            "token_count_approx": len(data.get("files", [])) * 1000,
-        }
-    
-    return {
-        "status": "ok",
-        "version": app.version,
-        "vaults": len(index),
-        "total_files": total_files,
-        "total_tokens": total_tokens,
-        "last_full_index_ts": _last_full_index_ts,
-        "uptime_seconds": uptime,
-        "git_describe": get_git_describe(),
-        "git_commit": get_git_commit(),
-        # Enriched fields
-        "memory": {
-            "used_mb": mem_used_mb,
-            "total_mb": mem_total_mb,
-            "percent": mem_pct,
-        },
-        "cpu": {
-            "percent": cpu_pct,
-        },
-        "disk": {
-            "used_gb": disk_used_gb,
-            "total_gb": disk_total_gb,
-            "free_gb": disk_free_gb,
-            "percent": disk_pct,
-        },
-        "connections": {
-            "active_sse": active_sessions,
-        },
-        "backups": {
-            "total_count": total_backups,
-            "total_size_mb": total_backup_size_mb,
-            "oldest_age_days": oldest_backup_age_days,
-        },
-        "index": index_detail,
-    }
-
 
 @app.get("/api/vaults", response_model=list[VaultInfo])
 async def api_vaults(current_user=Depends(require_auth)):