ui(config): brancher save/load au frontend (Phase 4.3 chantier post-rewrite)

Avant : ``/api/config/save`` + ``/api/config/load`` existaient et étaient
testés côté serveur, mais aucun bouton UI ne les appelait — code zombie
typique post-rewrite signalé par l'audit (chemins UI/API/runner désynchronisés).

Ajouté :
- Boutons "💾 Sauvegarder config" et "📂 Charger config" dans
``_view_benchmark.html`` (à côté du bouton "Lancer le benchmark").
- ``saveConfigToFile()`` : POST ``/api/config/save`` avec l'état UI
courant (corpus_path, competitors, normalization, output_dir, …),
déclenche le download du JSON.
- ``loadConfigFromFile()`` + ``onConfigFileSelected()`` : FileReader
→ POST ``/api/config/load`` (validation + upgrade serveur) →
``_applyConfig()`` restaure l'UI.
- ``renderCompetitors()`` appelé après restauration pour réafficher
la liste composée.

Tests : ``TestConfigSaveLoadUIBindings`` vérifie que (a) le template
expose les bons handlers, (b) le JS définit les 3 fonctions et appelle
les bons endpoints. Aucun test serveur existant cassé (25 passed).

https://claude.ai/code/session_01ArfZ8kcgv7Cyda7VbJVmpn

picarones/interfaces/web/static/web-app.js

@@ -1166,6 +1166,135 @@ async function deleteUploadedCorpus(corpusId) {
   } catch(e) {}
 }
 
+// ─── Config save / load ──────────────────────────────────────────────────────
+// Bindings UI pour /api/config/save et /api/config/load (Phase 4.3 du
+// chantier post-rewrite).  Avant ce wiring, les endpoints existaient
+// côté serveur (avec tests dédiés) mais aucun bouton ne les appelait —
+// code zombie typique post-rewrite.
+
+function _gatherCurrentConfig() {
+  /** Sérialise l'état UI courant en dict compatible
+   * ``/api/config/save``.  Inclut les compétiteurs composés
+   * (_competitors), les options de normalisation et le profil de
+   * langue rapport. */
+  return {
+    label: document.getElementById("report-name").value || "picarones-config",
+    corpus_path: document.getElementById("corpus-path").value,
+    competitors: _competitors,
+    normalization_profile: document.getElementById("norm-profile").value,
+    char_exclude: document.getElementById("char-exclude").value,
+    output_dir: document.getElementById("output-dir").value,
+    report_name: document.getElementById("report-name").value,
+  };
+}
+
+async function saveConfigToFile() {
+  /** POST la config courante à /api/config/save et déclenche le
+   * téléchargement du JSON retourné. */
+  const cfg = _gatherCurrentConfig();
+  try {
+    const r = await fetch("/api/config/save", {
+      method: "POST",
+      headers: {"Content-Type": "application/json"},
+      body: JSON.stringify(cfg),
+    });
+    if (!r.ok) {
+      const detail = await r.text();
+      alert(lang === "fr"
+        ? "Erreur sauvegarde config : " + detail
+        : "Save config error: " + detail);
+      return;
+    }
+    const blob = await r.blob();
+    // Reconstitue le filename depuis le header Content-Disposition.
+    const cd = r.headers.get("Content-Disposition") || "";
+    const m = cd.match(/filename="([^"]+)"/);
+    const filename = m ? m[1] : "picarones-config.json";
+    const url = URL.createObjectURL(blob);
+    const a = document.createElement("a");
+    a.href = url;
+    a.download = filename;
+    document.body.appendChild(a);
+    a.click();
+    document.body.removeChild(a);
+    URL.revokeObjectURL(url);
+  } catch (e) {
+    alert(lang === "fr"
+      ? "Erreur sauvegarde config : " + e.message
+      : "Save config error: " + e.message);
+  }
+}
+
+function loadConfigFromFile() {
+  /** Déclenche le sélecteur de fichier — l'utilisateur choisit un
+   * JSON, ``onConfigFileSelected`` fait le reste. */
+  document.getElementById("config-file-input").click();
+}
+
+async function onConfigFileSelected(event) {
+  /** Lit le fichier JSON, POST à /api/config/load pour validation +
+   * upgrade éventuel, puis restaure l'état UI depuis le dict retourné. */
+  const file = event.target.files[0];
+  if (!file) return;
+  // Reset l'input pour permettre un re-chargement du même fichier.
+  event.target.value = "";
+  try {
+    const text = await file.text();
+    let parsed;
+    try {
+      parsed = JSON.parse(text);
+    } catch (e) {
+      alert(lang === "fr"
+        ? "Fichier JSON invalide : " + e.message
+        : "Invalid JSON file: " + e.message);
+      return;
+    }
+    const r = await fetch("/api/config/load", {
+      method: "POST",
+      headers: {"Content-Type": "application/json"},
+      body: JSON.stringify(parsed),
+    });
+    if (!r.ok) {
+      const detail = await r.text();
+      alert(lang === "fr"
+        ? "Erreur chargement config : " + detail
+        : "Load config error: " + detail);
+      return;
+    }
+    const result = await r.json();
+    _applyConfig(result.config || {});
+  } catch (e) {
+    alert(lang === "fr"
+      ? "Erreur chargement config : " + e.message
+      : "Load config error: " + e.message);
+  }
+}
+
+function _applyConfig(cfg) {
+  /** Restaure l'état UI depuis un dict de config validé serveur.
+   * Champs inconnus = ignorés silencieusement (responsabilité de
+   * ``filter_config`` côté serveur). */
+  if (typeof cfg.corpus_path === "string") {
+    document.getElementById("corpus-path").value = cfg.corpus_path;
+  }
+  if (typeof cfg.normalization_profile === "string") {
+    document.getElementById("norm-profile").value = cfg.normalization_profile;
+  }
+  if (typeof cfg.char_exclude === "string") {
+    document.getElementById("char-exclude").value = cfg.char_exclude;
+  }
+  if (typeof cfg.output_dir === "string") {
+    document.getElementById("output-dir").value = cfg.output_dir;
+  }
+  if (typeof cfg.report_name === "string") {
+    document.getElementById("report-name").value = cfg.report_name;
+  }
+  if (Array.isArray(cfg.competitors)) {
+    _competitors = cfg.competitors;
+    renderCompetitors();
+  }
+}
+
 // ─── Init ────────────────────────────────────────────────────────────────────
 document.addEventListener("DOMContentLoaded", async () => {
   loadStatus();

picarones/interfaces/web/templates/_view_benchmark.html

@@ -174,6 +174,10 @@
       <button class="btn btn-primary" id="start-btn" onclick="startBenchmark()" data-i18n="bench_start">▶ Lancer le benchmark</button>
       <button class="btn btn-secondary" id="cancel-btn" style="display:none;" onclick="cancelBenchmark()" data-i18n="bench_cancel">✕ Annuler</button>
       <span id="bench-status-text" style="font-size:12px; color: var(--text-muted);"></span>
+      <span style="flex:1;"></span>
+      <button class="btn btn-secondary btn-sm" id="config-save-btn" onclick="saveConfigToFile()" title="Télécharger la configuration courante en JSON" data-i18n="bench_config_save">💾 Sauvegarder config</button>
+      <button class="btn btn-secondary btn-sm" id="config-load-btn" onclick="loadConfigFromFile()" title="Charger une configuration depuis un fichier JSON" data-i18n="bench_config_load">📂 Charger config</button>
+      <input type="file" id="config-file-input" accept=".json" style="display:none;" onchange="onConfigFileSelected(event)" />
     </div>
 
     <div id="bench-progress-section" style="display:none;">

tests/web/test_sprint28_ux_save_compare.py

@@ -190,6 +190,52 @@ def client():
     return TestClient(app)
 
 
+class TestConfigSaveLoadUIBindings:
+    """Phase 4.3 du chantier post-rewrite : ``saveConfigToFile`` et
+    ``loadConfigFromFile`` sont désormais branchés au HTML +
+    ``/api/config/save`` et ``/api/config/load`` (avant : endpoints
+    fonctionnels mais aucun bouton UI ne les appelait — code zombie)."""
+
+    def test_template_exposes_save_load_buttons(self) -> None:
+        from pathlib import Path
+
+        tmpl = (
+            Path(__file__).resolve().parents[2]
+            / "picarones/interfaces/web/templates/_view_benchmark.html"
+        )
+        html = tmpl.read_text(encoding="utf-8")
+        assert "saveConfigToFile()" in html, (
+            "Le bouton 'Sauvegarder config' doit appeler "
+            "saveConfigToFile() dans _view_benchmark.html"
+        )
+        assert "loadConfigFromFile()" in html, (
+            "Le bouton 'Charger config' doit appeler "
+            "loadConfigFromFile() dans _view_benchmark.html"
+        )
+        assert "config-file-input" in html, (
+            "Un input file caché ``config-file-input`` doit servir de "
+            "déclencheur pour ``onConfigFileSelected``"
+        )
+
+    def test_js_defines_save_and_load_functions(self) -> None:
+        from pathlib import Path
+
+        js = (
+            Path(__file__).resolve().parents[2]
+            / "picarones/interfaces/web/static/web-app.js"
+        )
+        src = js.read_text(encoding="utf-8")
+        assert "function saveConfigToFile" in src or "async function saveConfigToFile" in src
+        assert "function loadConfigFromFile" in src
+        assert "function onConfigFileSelected" in src or "async function onConfigFileSelected" in src
+        assert '"/api/config/save"' in src, (
+            "saveConfigToFile doit appeler /api/config/save"
+        )
+        assert '"/api/config/load"' in src, (
+            "onConfigFileSelected doit appeler /api/config/load"
+        )
+
+
 class TestConfigSaveLoad:
     def test_save_returns_attachment(self, client):
         r = client.post("/api/config/save", json={