refactor: #85 T2 extrait le CRUD webhooks vers backend/routers (comportement inchange)

CHANGELOG.md

@@ -6,7 +6,7 @@ Format basé sur [Keep a Changelog](https://keepachangelog.com/fr/1.1.0/),
 et [Semantic Versioning](https://semver.org/spec/v2.0.0.html).
 
 > **En cours de développement** : les changements à venir sont listés dans la section
-> [Unreleased](#unreleased). La dernière version livrée est **2.27.0**.
+> [Unreleased](#unreleased). La dernière version livrée est **2.27.3**.
 
 ---
 
@@ -14,6 +14,44 @@ et [Semantic Versioning](https://semver.org/spec/v2.0.0.html).
 
 ---
 
+## [2.27.3] — 2026-09-26
+
+---
+
+## [2.27.2] — 2026-09-26
+
+### Modifié
+
+- **#85 (T2) — extraction du domaine `webhooks` hors du monolithe `backend/main.py`.**
+  Le CRUD `GET/POST/PATCH/DELETE /api/webhooks` (admin) est servi par le
+  nouveau `backend/routers/webhooks.py` — chemins, réponses, tags OpenAPI et
+  authentification inchangés (aucun impact utilisateur).
+- **#85 (T1) — extraction du domaine `health` hors du monolithe `backend/main.py`.**
+  `GET /api/health` et `GET /api/health/detailed` (admin) sont servis par le
+  nouveau `backend/routers/health.py` (monté dans `main.py`) et le modèle
+  `HealthResponse` déménage dans `backend/schemas.py` — chemins, réponses,
+  tags OpenAPI et authentification inchangés (aucun impact utilisateur).
+
+---
+
+## [2.27.1] — 2026-09-26
+
+### Modifié
+
+- **Roadmap — priorisation dette & sécurité (décisions 2026-09-26).**
+  Items #85 (refonte architecturale) et #87 (CI/CD) détaillés et marqués
+  prioritaires : `backend/main.py` mesuré à ~4 827 lignes, `tools/registry.py`
+  à créer, persistance SQLite/Redis, verrous asyncio, audit des `except`
+  larges, CI sécurité bloquante (bandit/semgrep/trivy, audits pip/npm),
+  finition CSP nonce (BUG-034), cookies `Secure` par défaut, rotation clé
+  DeepSeek à confirmer (BUG-006). #73 Sync reporté (P4, hors chemin
+  critique) ; desktop #77 confirmé non signé + doc SmartScreen, reste les
+  6 tests E2E manuels. Corrections : sections livrées #83/#84 retirées du
+  backlog (détail dans l'archive, index inchangé), total restant recalculé
+  (~12-18 jours chemin critique : #77 fin + #85 + #87).
+
+---
+
 ## [2.27.0] — 2026-09-25
 
 ### Ajouté

README.fr.md

@@ -4,7 +4,7 @@
 
 **Porte d'entrée web ultra-léger pour vos vaults Obsidian** — Accédez, naviguez et recherchez dans toutes vos notes Obsidian depuis n'importe quel appareil via une interface web moderne et responsive.
 
-[![Version](https://img.shields.io/badge/Version-2.27.0-blue.svg)]()
+[![Version](https://img.shields.io/badge/Version-2.27.3-blue.svg)]()
 [![License: MIT](https://img.shields.io/badge/License-MIT-yellow.svg)](https://opensource.org/licenses/MIT)
 [![Docker](https://img.shields.io/badge/Docker-Ready-blue.svg)](https://www.docker.com/)
 [![Python](https://img.shields.io/badge/Python-3.11+-green.svg)](https://www.python.org/)
@@ -976,8 +976,8 @@ Ce projet est sous licence **MIT** — voir le fichier [LICENSE](LICENSE) pour l
 
 ## 📝 Changelog
 
-Consultez le [CHANGELOG.md](./CHANGELOG.md) pour l'historique complet de toutes les versions (v1.0.0 → v2.27.0).
+Consultez le [CHANGELOG.md](./CHANGELOG.md) pour l'historique complet de toutes les versions (v1.0.0 → v2.27.3).
 
 ---
 
-*Projet : ObsiGate | Version : 2.27.0 | Dernière mise à jour : Septembre 2026*
+*Projet : ObsiGate | Version : 2.27.3 | Dernière mise à jour : Septembre 2026*

README.md

@@ -2,7 +2,7 @@
 
 **Ultra-light web gateway for your Obsidian vaults** — Access, browse, and search all your Obsidian notes from any device via a modern, responsive web interface.
 
-[![Version](https://img.shields.io/badge/Version-2.27.0-blue.svg)]()
+[![Version](https://img.shields.io/badge/Version-2.27.3-blue.svg)]()
 [![License: MIT](https://img.shields.io/badge/License-MIT-yellow.svg)](https://opensource.org/licenses/MIT)
 [![Docker](https://img.shields.io/badge/Docker-Ready-blue.svg)](https://www.docker.com/)
 [![Python](https://img.shields.io/badge/Python-3.11+-green.svg)](https://www.python.org/)
@@ -1151,8 +1151,8 @@ This project is licensed under the **MIT License** - see the [LICENSE](LICENSE)
 
 ## 📝 Changelog
 
-See [CHANGELOG.md](./CHANGELOG.md) for the complete version history (v1.0.0 → v2.27.0).
+See [CHANGELOG.md](./CHANGELOG.md) for the complete version history (v1.0.0 → v2.27.3).
 
 ---
 
-*Project: ObsiGate | Version: 2.27.0 | Last updated: September 2026*
+*Project: ObsiGate | Version: 2.27.3 | Last updated: September 2026*

VERSION

@@ -1 +1 @@
-2.27.0
+2.27.3

backend/main.py

@@ -93,7 +93,6 @@ from backend.schemas import (
     VaultSettingsResponse,
     VaultsStatusResponse,
     VaultStatsResponse,
-    WebhookModel,
 )
 from backend.search import (
     init_inverted_index,
@@ -398,19 +397,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 +853,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 +945,8 @@ 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.routers.webhooks import router as webhooks_router
 from backend.saved_searches import delete_saved, get_saved, save_search
 from backend.share import (
     create_share,
@@ -969,18 +957,14 @@ from backend.share import (
     update_shares_after_rename,
 )
 from backend.skills_routes import router as skills_router
-from backend.webhooks import (
-    create_webhook,
-    delete_webhook,
-    dispatch_webhooks,
-    get_webhooks,
-    update_webhook,
-)
+from backend.webhooks import dispatch_webhooks
 
 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
+app.include_router(webhooks_router)  # ROADMAP #85 T2 — Webhooks
 
 # Admin Dashboard endpoints (system stats, audit logs, backups, stream)
 try:
@@ -1306,125 +1290,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)):
@@ -4383,39 +4251,9 @@ async def api_dashboard(current_user=Depends(require_auth)):
 
 
 # ---------------------------------------------------------------------------
-# Webhook CRUD endpoints
+# Webhook CRUD endpoints : voir backend.routers.webhooks (#85 T2)
 # ---------------------------------------------------------------------------
 
-@app.get("/api/webhooks", response_model=list[WebhookModel])
-async def api_webhooks_list(current_user=Depends(require_admin)):
-    return get_webhooks()
-
-
-@app.post("/api/webhooks", response_model=WebhookModel)
-async def api_webhooks_create(body: dict = Body(...), current_user=Depends(require_admin)):
-    name = body.get("name", "Unnamed")
-    url = body.get("url", "")
-    events = body.get("events", [])
-    secret = body.get("secret")
-    if not url:
-        raise HTTPException(400, "URL is required")
-    return create_webhook(name, url, events, secret)
-
-
-@app.patch("/api/webhooks/{webhook_id}", response_model=WebhookModel)
-async def api_webhooks_update(webhook_id: str, body: dict = Body(...), current_user=Depends(require_admin)):
-    result = update_webhook(webhook_id, body)
-    if not result:
-        raise HTTPException(404, "Webhook not found")
-    return result
-
-
-@app.delete("/api/webhooks/{webhook_id}", response_model=StatusResponse)
-async def api_webhooks_delete(webhook_id: str, current_user=Depends(require_admin)):
-    if not delete_webhook(webhook_id):
-        raise HTTPException(404, "Webhook not found")
-    return {"status": "deleted"}
-
 
 # ---------------------------------------------------------------------------
 # Share (public document) endpoints

backend/routers/__init__.py

@@ -0,0 +1,7 @@
+"""ObsiGate — routers FastAPI par domaine (ROADMAP #85).
+
+Découpage progressif du monolithe ``backend/main.py`` : chaque module de ce
+paquet expose un ``APIRouter`` monté par ``main.py``. Les handlers sont
+déplacés sans changement de comportement (mêmes chemins, mêmes modèles de
+réponse, mêmes dépendances d'authentification).
+"""

backend/routers/health.py

@@ -0,0 +1,143 @@
+"""System health endpoints (ROADMAP #85, tranche 1).
+
+Handlers déplacés depuis :mod:`backend.main` sans changement de
+comportement : mêmes chemins (``/api/health``, ``/api/health/detailed``),
+même ``response_model`` (:class:`backend.schemas.HealthResponse`), même
+dépendance admin. Seule différence : la version est lue via
+:func:`backend.version.get_version` au lieu de ``app.version`` (valeur
+identique, figée au démarrage depuis le fichier ``VERSION``).
+
+Note : ``uptime_seconds`` reprend l'expression d'origine
+(``'_SERVER_START_TIME' in globals()``), qui vaut toujours 0 — le global
+n'est défini nulle part dans ``backend.main`` (voir ``backend.admin`` qui
+possède son propre compteur). Ce comportement est préservé tel quel ; le
+corriger fera l'objet d'une tranche ultérieure avec test dédié.
+"""
+
+from fastapi import APIRouter, Depends
+
+from backend.auth.middleware import require_admin
+from backend.indexer import index
+from backend.schemas import HealthResponse
+from backend.version import get_git_commit, get_git_describe, get_version
+
+router = APIRouter(tags=["System"])
+
+
+@router.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
+    # `_SERVER_START_TIME` n'existe dans aucun module (comportement d'origine
+    # préservé : uptime toujours 0 — voir docstring du module).
+    uptime = int(time.time() - _SERVER_START_TIME) if '_SERVER_START_TIME' in globals() else 0  # noqa: F821
+    return {
+        "status": "ok",
+        "version": get_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(),
+    }
+
+
+@router.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  # noqa: F821 — voir ci-dessus
+
+    # 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.get("tags", [])),
+            "token_count_approx": len(data.get("files", [])) * 1000,
+        }
+
+    return {
+        "status": "ok",
+        "version": get_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,
+    }

backend/routers/webhooks.py

@@ -0,0 +1,54 @@
+"""Webhook CRUD endpoints (ROADMAP #85, tranche 2).
+
+Handlers déplacés depuis :mod:`backend.main` sans changement de
+comportement : mêmes chemins (``/api/webhooks``), même modèle de réponse
+(:class:`backend.schemas.WebhookModel`), même dépendance admin. La logique
+métier vit déjà dans :mod:`backend.webhooks` (validation d'URL anti-SSRF,
+store ``webhook_secrets.json`` — BUG-026).
+"""
+
+from fastapi import APIRouter, Body, Depends, HTTPException
+
+from backend.auth.middleware import require_admin
+from backend.schemas import StatusResponse, WebhookModel
+from backend.webhooks import (
+    create_webhook,
+    delete_webhook,
+    get_webhooks,
+    update_webhook,
+)
+
+router = APIRouter(prefix="/api/webhooks", tags=["webhooks"])
+
+
+@router.get("", response_model=list[WebhookModel])
+async def api_webhooks_list(current_user=Depends(require_admin)):
+    return get_webhooks()
+
+
+@router.post("", response_model=WebhookModel)
+async def api_webhooks_create(body: dict = Body(...), current_user=Depends(require_admin)):
+    name = body.get("name", "Unnamed")
+    url = body.get("url", "")
+    events = body.get("events", [])
+    secret = body.get("secret")
+    if not url:
+        raise HTTPException(400, "URL is required")
+    return create_webhook(name, url, events, secret)
+
+
+@router.patch("/{webhook_id}", response_model=WebhookModel)
+async def api_webhooks_update(
+    webhook_id: str, body: dict = Body(...), current_user=Depends(require_admin)
+):
+    result = update_webhook(webhook_id, body)
+    if not result:
+        raise HTTPException(404, "Webhook not found")
+    return result
+
+
+@router.delete("/{webhook_id}", response_model=StatusResponse)
+async def api_webhooks_delete(webhook_id: str, current_user=Depends(require_admin)):
+    if not delete_webhook(webhook_id):
+        raise HTTPException(404, "Webhook not found")
+    return {"status": "deleted"}

backend/schemas.py

@@ -408,6 +408,31 @@ class DashboardResponse(BaseModel):
     total_images: int = 0
 
 
+# ---------------------------------------------------------------------------
+# System / health (#85 — extrait de backend.main, comportement inchangé)
+# ---------------------------------------------------------------------------
+
+
+class HealthResponse(BaseModel):
+    """Application health status.
+
+    Déplacé depuis :mod:`backend.main` sans modification : pas de
+    ``extra="allow"`` ici, pour préserver la validation actuelle des
+    réponses (les champs enrichis de ``/api/health/detailed`` restent
+    filtrés comme avant).
+    """
+
+    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")
+
+
 # ---------------------------------------------------------------------------
 # Webhooks, sharing & conflicts
 # ---------------------------------------------------------------------------

desktop/Cargo.lock

@@ -2626,7 +2626,7 @@ dependencies = [
 
 [[package]]
 name = "obsigate-desktop"
-version = "2.27.0"
+version = "2.27.3"
 dependencies = [
  "chrono",
  "env_logger",

desktop/Cargo.toml

@@ -1,6 +1,6 @@
 [package]
 name = "obsigate-desktop"
-version = "2.27.0"
+version = "2.27.3"
 description = "ObsiGate Desktop — Porte d'entrée native pour vos vaults Obsidian"
 authors = ["Bruno Charest"]
 edition = "2021"

desktop/tauri.conf.json

@@ -1,7 +1,7 @@
 {
   "$schema": "https://raw.githubusercontent.com/nicedoc/obsigate/main/desktop/tauri.conf.schema.json",
   "productName": "ObsiGate",
-  "version": "2.27.0",
+  "version": "2.27.3",
   "identifier": "com.obsigate.desktop",
   "build": {
     "frontendDist": "../frontend",

docs/ROADMAP.md

@@ -1,6 +1,6 @@
 # ObsiGate — Roadmap
 
-> **Version :** 2.27.0 | **Dernière mise à jour :** 2026-09-25
+> **Version :** 2.27.3 | **Dernière mise à jour :** 2026-09-26
 > **Ce fichier ne contient que le travail à venir** (🔵 En cours + ⚪ Backlog) et un index compact
 > vers les fonctionnalités livrées.
 > - **Méthode de livraison à appliquer pour toute tâche : [DELIVERY_WORKFLOW.md](./DELIVERY_WORKFLOW.md)**
@@ -37,7 +37,7 @@
 - **Reste à faire :**
   - [x] **Signature de l'updater Tauri** (gratuit) : paire de clés générée, `pubkey` renseignée, `createUpdaterArtifacts` activé, secrets CI câblés
   - [x] **Manifeste `latest.json`** généré par `scripts/updater_manifest.py` (intégré à `publish_release.py`), endpoint updater pointé sur `main`
-  - [ ] **Signature de code Windows** : non retenue (pas de certificat) — alternatives : livrer non signé, SignPath.io (OSS gratuit), Certum OSS, Azure Trusted Signing, certificat EV
+  - [ ] **Signature de code Windows** : **non retenue — décision confirmée le 2026-09-26** : livraison non signée + documentation SmartScreen (« Exécuter quand même »). Alternatives écartées sauf retour utilisateur : SignPath.io (OSS gratuit), Certum OSS, Azure Trusted Signing, certificat EV
   - [ ] Exécuter les 6 tests E2E **manuels** — protocole documenté : [DESKTOP_E2E_CHECKLIST.md](./DESKTOP_E2E_CHECKLIST.md)
 
 ---
@@ -47,6 +47,7 @@
 ### 73. Synchronisation multi-appareils — Obsidian Sync compatible
 
 - **Effort :** 6-8 jours | **Impact :** 🟢
+- **Décision 2026-09-26 : reporté (P4)** — axe prioritaire = dette & sécurité (#85/#87) ; #73 hors chemin critique. Si réactivé : partir d'un MVP export/hash/LWW adossé à #59 (PWA offline) + #62 (collab Yjs/CRDT) plutôt qu'un protocole parallèle.
 - **Description :** Synchronisation des vaults entre plusieurs instances d'ObsiGate via un protocole de synchronisation décentralisé ou compatible Obsidian Sync. Alternative self-hosted à Obsidian Sync.
 - **Sous-tâches :**
   - [ ] Protocole : évaluation CRDT vs OT vs diff/patch pour fichiers markdown
@@ -60,60 +61,30 @@
 
 ---
 
-## ⚪ Backlog — Priorité 2 (P2)
-
-### 83. Barre d'outils d'édition mobile — style Obsidian Android
-
-- **Effort :** 3-5 jours | **Impact :** 🟡 | **Zone :** frontend (mobile)
-- **Statut :** ✅ livré — ruban horizontal défilable ancré au-dessus du clavier, commandes étendues et personnalisation persistée. Détail : [archive/COMPLETED_v1-v2.md](./archive/COMPLETED_v1-v2.md) (section #83).
-- **Description :** remplacer la barre de mise en forme Markdown actuelle par un **ruban horizontal
-  défilable** ancré juste au-dessus du clavier virtuel, reprenant l'ergonomie de l'app Android
-  Obsidian : fond anthracite aux coins arrondis, insertion/enrobage de la syntaxe au curseur ou sur
-  la sélection, et personnalisation des commandes via une icône clé à molette.
-- **Sous-tâches :**
-  - [x] Ruban horizontal défilable (glissement tactile gauche/droite) ancré au-dessus du clavier
-  - [x] Actions rapides : annuler, refaire, `[[ ]]` (lien interne), modèle/fichiers, tag `#`, pièce jointe
-  - [x] Formatage : H1–H6, gras, italique, barré (`~~`), surligné (`==`), code en ligne/bloc, citation (`>`)
-  - [x] Liens externes, listes à puces/numérotées, case à cocher (`- [ ]`), indenter / désindenter
-  - [x] Personnalisation (clé à molette) : ajouter / supprimer / réordonner les commandes
-  - [x] i18n FR/EN + tests frontend (helpers purs) + E2E mobile
-
----
-
 ## ⚪ Backlog — Sécurité, architecture & performance (P0/P1)
 
-### 84. Consolidation & sécurité — revue statique 2026-09-13 (phase 1)
-
-- **Effort :** 6-9 jours | **Impact :** 🔴 | **Zone :** backend + frontend | **Référence :** [ISSUES_TODOLIST.md](./ISSUES_TODOLIST.md) BUG-021 → BUG-034
-- **Statut :** 🟢 livré (phase 1) — sanitizer XSS, rate-limit/lockout MFA, isolation vaults, ReDoS, SSRF webhooks, cycle de vie des sessions, politique de mot de passe, verrous `users.json`, audits IP, rate-limit par compte, symlinks, recherche via inverted index, token en cookie HttpOnly. Détail : [archive/COMPLETED_v1-v2.md](./archive/COMPLETED_v1-v2.md) (section #84).
-- **Description :** traiter toutes les vulnérabilités critiques et importantes issues de la revue statique : XSS markdown (`escape=False`) et page publique de partage, brute-force MFA, isolation des vaults (`resolve_safe_path`), ReDoS, SSRF webhooks, cycle de vie des sessions, politique de mot de passe, races `users.json`, audits IP, rate-limit partagé, indexation symlinks.
-- **Sous-tâches :**
-  - [x] Assainir le rendu markdown (sanitizer serveur en whitelist) et la page de partage (échappement `title`/frontmatter) — *DOMPurify client non ajouté (défense en profondeur serveur suffisante)*
-  - [x] Rate-limit + lockout sur les endpoints MFA (`totp/verify`, `recovery`, `webauthn/verify`)
-  - [x] Corriger `resolve_safe_path` (comparaison de chemin stricte par segment) + test de régression
-  - [x] Rotation du refresh token, révocation de l'access token au logout, persistance des JTI révoqués
-  - [x] Valider la politique de mot de passe à la création ; bloquer le SSRF des webhooks et externaliser les secrets
-  - [x] Verrous sur les mutations `users.json` ; consigner l'adresse IP réelle dans les audits
-  - [x] Ignorer les symlinks de l'index ; caps CPU/timeout regex (ReDoS)
-  - [~] Durcir la CSP — *partiel* : directives `object-src`/`base-uri`/`form-action`/`frame-ancestors` ajoutées et token retiré de `sessionStorage` ; migration **nonce** restante (nécessite la conversion des gestionnaires d'événements inline)
-
 ### 85. Refonte architecturale — découpage du monolithe & persistance d'état (phase 2)
 
 - **Effort :** 8-12 jours | **Impact :** 🟡 | **Zone :** backend
-- **Description :** extraire le monolithe `backend/main.py` (~4 260 lignes) en routers FastAPI par domaine et rendre persistant l'état qui ne l'est pas (index de recherche, JTI révoqués, compteurs de rate-limit) pour préparer le multi-nœuds.
+- **Décision 2026-09-26 : prioritaire (axe Dette & sécurité).**
+- **Statut :** 🔵 en cours depuis 2026-09-26 — découpe par tranches à impact minimal (comportement inchangé, un domaine par commit). **T1 livrée (v2.27.2) :** `health` (`/api/health`, `/api/health/detailed` → `backend/routers/health.py`, `HealthResponse` → `schemas.py`). **T2 livrée (v2.27.3) :** `webhooks` (CRUD `/api/webhooks` → `backend/routers/webhooks.py`, logique déjà dans `backend/webhooks.py`).
+- **Description :** extraire le monolithe `backend/main.py` (~4 827 lignes au 2026-09-26, ~17 % du backend) en routers FastAPI par domaine et rendre persistant l'état qui ne l'est pas (index de recherche, JTI révoqués, compteurs de rate-limit) pour préparer le multi-nœuds. L'état mémoire actuel (index, inverted index, vecteurs sémantiques, `SSEManager`, collab) rend le multi-workers unsafe.
 - **Sous-tâches :**
-  - [ ] Routers par domaine : files, search, share, webhooks, plugins, collab, admin, ai
-  - [ ] Centraliser le contrat d'outils IA sur `tools/registry.py` (permissions, quotas, redaction)
-  - [ ] Persister index, JTI révoqués et compteurs de rate-limit (SQLite/Redis)
-  - [ ] Verrous asyncio autour de l'index global et des stores JSON ; service de partage public (expiration, révocation, quotas)
+  - [ ] Routers par domaine : files, search, share, webhooks, plugins, collab, admin, ai — `main.py` conservé comme assemblage (< 500 lignes) ; dédupliquer les modèles Pydantic vers `schemas.py`. **Avancement :** `health` ✅ (T1, `backend/routers/health.py`), `webhooks` ✅ (T2, `backend/routers/webhooks.py`) ; `tools/registry.py` existe déjà (permissions/quotas/redaction — à compléter, pas à créer)
+  - [ ] Compléter `tools/registry.py` (existant : permissions/quotas/redaction) comme contrat central des outils IA si des manques sont constatés
+  - [ ] Persister index, JTI révoqués et compteurs de rate-limit (SQLite par défaut, Redis en option multi-nœuds ; le rate-limit actuel est in-memory mono-process)
+  - [ ] Verrous asyncio autour de l'index global et des stores JSON ; auditer les `except Exception` larges (> 100 occurrences) : best-effort (backup/audit) vs masquage d'erreur (erreurs typées 4xx/5xx + test)
+  - [ ] Extraire le service de partage public (expiration, révocation, quotas)
 
 ### 87. Amélioration continue — tests, CI/CD, revues de sécurité (phase 4)
 
 - **Effort :** 3-5 jours | **Impact :** 🟡 | **Zone :** `.gitea/workflows/`, `tests/`
-- **Description :** renforcer le pipeline (`.gitea/workflows/ci.yml`, `desktop-build.yml`) pour le rendre bloquant par défaut et accompagner les phases 1 → 3.
+- **Décision 2026-09-26 : prioritaire (axe Dette & sécurité).**
+- **Description :** renforcer le pipeline (`.gitea/workflows/ci.yml`, `desktop-build.yml`) pour le rendre bloquant par défaut et accompagner les phases 1 → 3. Constat 2026-09-26 : job `security` non bloquant (`bandit`/`pip-audit` en `|| echo`, ni semgrep ni trivy), E2E limité à `chromium-desktop`, 5 suites frontend hors CI.
 - **Sous-tâches :**
-  - [ ] Jobs CI sécurité (bandit/semgrep/trivy, audits pip/npm) + tests E2E XSS (page de partage + lecteur markdown)
-  - [ ] Tests de concurrence (`users.json`), fuzzing de timing regex, couverture des composants critiques
+  - [ ] Jobs CI sécurité **bloquants** (bandit/semgrep/trivy, audits pip/npm) + tests E2E XSS (page de partage + lecteur markdown)
+  - [ ] Tests de concurrence (`users.json`), fuzzing de timing regex, couverture des composants critiques ; intégrer au CI les 5 suites frontend hors CI (`upload`, `pretty`, `media-viewer`, `mfa-settings`, `config-ai-keys`)
+  - [ ] Finir BUG-034 (migration CSP **nonce**, conversion des handlers inline), `Secure` cookies à `true` par défaut, politique CORS same-origin explicite ; confirmer la rotation de la clé DeepSeek (BUG-006, clé dans l'historique Git)
   - [ ] Revue périodique des dépendances ; documentation utilisateur FR/EN synchronisée ; contrôle automatisé de la conformité au DoD
 
 ---
@@ -200,16 +171,17 @@
 
 | Priorité | Items | Effort total estimé |
 |---|---|---|
-| ✅ Complété | #1 → #59, #61–72, #74–76, #78–84, #86, #88–93, #94–100, #102–114, #92 | ~133 jours réalisés |
-| 🔵 P2 restant | #77 Desktop : signature de code (non retenue), 6 tests E2E **manuels** ([protocole](./DESKTOP_E2E_CHECKLIST.md)) | ~0,5-1 jour |
-| ⚪ P4 restant | #73 Sync (6-8j) | 6-8 jours |
-| ⚪ P0/P1 restant | #85, #87 Refonte architecturale, CI/CD (BUG-035 → BUG-040 corrigés, #86 livré) | ~11-17 jours |
-| **Total restant** | **6 items + finitions** | **~23-38 jours** |
+| ✅ Complété | #1 → #59, #61–72, #74–76, #78–84, #86, #88–93, #94–100, #102–115, #117, #92 | ~133 jours réalisés |
+| 🔵 Finitions | #77 Desktop : 6 tests E2E **manuels** ([protocole](./DESKTOP_E2E_CHECKLIST.md)) — signature Windows non retenue (décision 2026-09-26) | ~0,5-1 jour |
+| ⚪ P4 reporté | #73 Sync — **reporté (décision 2026-09-26)**, hors chemin critique | 6-8 jours si réactivé |
+| ⚪ P0/P1 prioritaire | #85, #87 Refonte architecturale, CI/CD (BUG-035 → BUG-040 corrigés, #86 livré) | ~11-17 jours |
+| **Total chemin critique** | **#77 fin + #85 + #87** | **~12-18 jours** |
 
 ---
 
 ## Notes
 
+- **Décisions 2026-09-26 :** axe prioritaire = dette & sécurité (#85/#87) ; #73 Sync reporté (P4, hors chemin critique) ; desktop livré non signé + doc SmartScreen.
 - Les items P3/P4 ne sont pas ordonnés par priorité interne — à raffiner selon les retours utilisateurs.
 - L'effort inclut le développement + tests unitaires + intégration CI, mais pas la documentation utilisateur.
 - Les items marqués 🟢 (nice-to-have) sont de bons candidats pour des contributions externes.

package.json

@@ -1,6 +1,6 @@
 {
   "name": "obsigate",
-  "version": "2.27.0",
+  "version": "2.27.3",
   "description": "**Porte d'entrée web ultra-léger pour vos vaults Obsidian** — Accédez, naviguez et recherchez dans toutes vos notes Obsidian depuis n'importe quel appareil via une interface web moderne et responsive.",
   "main": "patch.js",
   "directories": {