chore: supprimer unregister_job mort + factoriser _numbers_in_payload partagé

Audit F4 — deux nettoyages mineurs.

(a) ``picarones/web/state.py:255`` exposait ``unregister_job(job_id)`` qui
mute ``JOBS`` sous ``JOBS_LOCK`` (idempotent). La fonction n'avait
aucun consommateur dans le code (ni dans les tests). Conservée elle
n'aurait fait que tromper un futur lecteur sur la discipline d'accès
au registre — la suppression effective des jobs vit dans
``cleanup_old_jobs``. Suppression : code + entrée ``__all__`` + mention
dans la docstring de ``JOBS``.

(b) ``_numbers_in_payload`` (garde-fou anti-hallucination du moteur
narratif) était dupliqué dans deux fichiers de tests :

- ``tests/measurements/test_sprint19_narrative_engine.py`` (version
complète : walk récursif + parse strings via regex + variantes
décimales 1/2/3/4) ;
- ``tests/measurements/test_sprint36_ensemble_narrative.py`` (version
simplifiée, walk top-level seulement).

Création de ``tests/measurements/_helpers.py`` avec ``numbers_in_payload``
(version complète, strict superset de la simplifiée). Les deux fichiers
sprint réimportent désormais sous l'alias ``_numbers_in_payload`` pour
préserver l'API consommée par
``test_sprint23_anti_hallucination.py:166`` qui fait
``from tests.measurements.test_sprint19_narrative_engine import _numbers_in_payload``.

Aucune régression : sprint36 utilisait déjà la version la plus permissive
n'importe comment (la version sprint19 est strictement plus large), donc
les assertions ``in traceable | whitelist`` passent sans modification.

371 passed (sprint19 + sprint23 + sprint36 + tests/web/). ``ruff check``
clean.

https://claude.ai/code/session_01Hsd7kL8yeCbXn1mA7GQK9L

picarones/web/state.py

@@ -223,7 +223,7 @@ JOBS: dict[str, BenchmarkJob] = {}
 
 **Discipline d'accès** : tous les ``read`` et ``write`` doivent passer
 par les helpers ``register_job``, ``get_job_in_memory``,
-``unregister_job`` qui prennent ``JOBS_LOCK``. Lire ou muter ce dict
+``cleanup_old_jobs`` qui prennent ``JOBS_LOCK``. Lire ou muter ce dict
 sans verrou expose à un ``RuntimeError: dictionary changed size
 during iteration`` sous charge concurrente (le GIL protège l'atomicité
 d'une opération mais pas la cohérence d'une boucle).
@@ -252,12 +252,6 @@ def get_job_in_memory(job_id: str) -> Optional[BenchmarkJob]:
         return JOBS.get(job_id)
 
 
-def unregister_job(job_id: str) -> None:
-    """Retire un job du registre mémoire (thread-safe ; idempotent)."""
-    with JOBS_LOCK:
-        JOBS.pop(job_id, None)
-
-
 def cleanup_old_jobs() -> None:
     """Supprime les jobs terminés les plus anciens si on dépasse ``JOBS_MAX``."""
     with JOBS_LOCK:
@@ -289,6 +283,5 @@ __all__ = [
     "JOBS_LOCK",
     "register_job",
     "get_job_in_memory",
-    "unregister_job",
     "cleanup_old_jobs",
 ]

tests/measurements/_helpers.py

@@ -0,0 +1,58 @@
+"""Helpers partagés par les tests ``tests/measurements/``.
+
+Centralise des utilitaires de test réutilisés par plusieurs fichiers
+sprint, en particulier le garde-fou anti-hallucination du moteur
+narratif.
+"""
+
+from __future__ import annotations
+
+import re
+from typing import Any
+
+
+def numbers_in_payload(payload: Any) -> set[str]:
+    """Collecte toutes les représentations numériques d'un payload de Fact.
+
+    Inclut les variantes usuelles produites par ``str.format`` :
+    ``5``, ``5.0``, ``5.00``, ``5.000``, ``5.0000``, etc., pour tolérer
+    les patterns ``{x}`` et ``{x:.2f}`` dans les templates narratifs.
+
+    Le walk est récursif (dict / list / tuple) et parse également les
+    chaînes via ``\\d+(?:\\.\\d+)?`` pour couvrir les payloads où une
+    valeur numérique a été pré-formatée en string.
+    """
+    out: set[str] = set()
+
+    def _add_variants(v: Any) -> None:
+        try:
+            f = float(v)
+        except (TypeError, ValueError):
+            return
+        out.add(str(v))
+        out.add(str(f))
+        if f == int(f):
+            out.add(str(int(f)))
+        for dec in (1, 2, 3, 4):
+            out.add(f"{f:.{dec}f}")
+
+    def _walk(x: Any) -> None:
+        if isinstance(x, dict):
+            for v in x.values():
+                _walk(v)
+        elif isinstance(x, (list, tuple)):
+            for v in x:
+                _walk(v)
+        elif isinstance(x, bool):
+            return
+        elif isinstance(x, (int, float)):
+            _add_variants(x)
+        elif isinstance(x, str):
+            for n in re.findall(r"\d+(?:\.\d+)?", x):
+                _add_variants(n)
+
+    _walk(payload)
+    return out
+
+
+__all__ = ["numbers_in_payload"]

tests/measurements/test_sprint19_narrative_engine.py

@@ -453,44 +453,10 @@ class TestBuildSynthesisE2E:
 # Garde-fou anti-hallucination : traçabilité des nombres
 # ---------------------------------------------------------------------------
 
-def _numbers_in_payload(payload: dict) -> set[str]:
-    """Collecte tous les nombres d'un payload de Fact sous formes multiples.
-
-    Inclut les représentations usuelles produites par ``str.format`` :
-    ``5``, ``5.0``, ``5.00``, ``5.000``, etc., pour tolérer les formats
-    ``{x}`` et ``{x:.2f}`` dans les templates.
-    """
-    out: set[str] = set()
-
-    def _add_variants(v):
-        try:
-            f = float(v)
-        except (TypeError, ValueError):
-            return
-        out.add(str(v))
-        out.add(str(f))
-        if f == int(f):
-            out.add(str(int(f)))
-        for dec in (1, 2, 3, 4):
-            out.add(f"{f:.{dec}f}")
-
-    def _walk(x):
-        if isinstance(x, dict):
-            for v in x.values():
-                _walk(v)
-        elif isinstance(x, (list, tuple)):
-            for v in x:
-                _walk(v)
-        elif isinstance(x, bool):
-            return
-        elif isinstance(x, (int, float)):
-            _add_variants(x)
-        elif isinstance(x, str):
-            for n in re.findall(r"\d+(?:\.\d+)?", x):
-                _add_variants(n)
-
-    _walk(payload)
-    return out
+# ``_numbers_in_payload`` vit dans ``tests/measurements/_helpers.py`` ;
+# on le ré-expose sous son ancien nom privé pour compatibilité avec les
+# tests qui l'importent depuis ce module (ex. test_sprint23).
+from tests.measurements._helpers import numbers_in_payload as _numbers_in_payload  # noqa: E402
 
 
 # Sprint 23 : whitelist vidée. Tout nombre rendu dans la synthèse doit

tests/measurements/test_sprint36_ensemble_narrative.py

@@ -264,22 +264,7 @@ class TestSynthesisIntegration:
 # ──────────────────────────────────────────────────────────────────────────
 
 
-def _numbers_in_payload(payload: dict) -> set[str]:
-    """Extrait toutes les représentations numériques du payload."""
-    found: set[str] = set()
-    for v in payload.values():
-        if isinstance(v, (int, float)):
-            s = f"{v}"
-            found.add(s)
-            # ajoute la version sans .0 pour les floats entiers
-            if isinstance(v, float) and v.is_integer():
-                found.add(str(int(v)))
-            # ajoute la version "70.0" si valeur "70.0"
-            try:
-                found.add(f"{float(v)}")
-            except (TypeError, ValueError):
-                pass
-    return found
+from tests.measurements._helpers import numbers_in_payload as _numbers_in_payload  # noqa: E402
 
 
 class TestTraceability: