refactor(measurements): promouvoir modules philologiques/académiques/governance depuis extras/

Les modules suivants vivaient dans ``picarones/extras/`` (cercle 3,
"plugins") alors qu'ils sont enregistrés dans le ``metric_registry``
et calculés automatiquement par le runner — ils sont en fait des
métriques officielles (cercle 2).

Modules philologiques (``extras/historical/`` → ``measurements/``) :
abbreviations, early_modern_typography, lexical_modernization,
modern_archives, mufi, philological_runner, roman_numerals,
unicode_blocks.

Modules académiques (``extras/academic/`` → ``measurements/``) :
image_predictive, taxonomy_cooccurrence, taxonomy_intra_doc.

Module governance (``extras/governance/`` → ``measurements/``) :
module_policy.

Les sous-packages ``extras/{historical,academic,governance}/`` sont
supprimés (vides après promotion). Tests de migration phaseA/B
(qui validaient ces locations transitoires) supprimés — leur rôle
est désormais documenté dans ``docs/architecture.md``.

https://claude.ai/code/session_01Hsd7kL8yeCbXn1mA7GQK9L

picarones/extras/academic/__init__.py

@@ -1,18 +0,0 @@
-"""Modules techniques sans cas d'usage prod direct.
-
-Ces 3 modules calculent des distributions intéressantes pour la
-recherche académique mais ne participent pas à la décision
-*« peut-on déployer ce moteur en prod ? »*.
-
-Modules
--------
-- :mod:`taxonomy_intra_doc`   — heatmap classe×position intra-document.
-- :mod:`taxonomy_cooccurrence` — matrice Jaccard inter-classes au niveau document.
-- :mod:`image_predictive`     — score de complexité paléographique (poids éditoriaux).
-
-Rétrocompat
------------
-Les imports historiques ``from picarones.core.taxonomy_intra_doc import
-...`` continuent à fonctionner via des fichiers-shims laissés à
-l'ancien emplacement.
-"""

picarones/extras/governance/__init__.py

@@ -1,8 +0,0 @@
-"""Gouvernance préventive pour modules contribués externes.
-
-Aujourd'hui Picarones n'a pas encore de modules tiers contribués par
-des utilisateurs externes. Le module ``module_policy`` ici est livré
-en avance pour préparer la phase d'ouverture (lointaine).
-
-Sera réintégré au Cercle 2 si/quand 5+ modules tiers sont publiés.
-"""

picarones/extras/historical/__init__.py

@@ -1,30 +0,0 @@
-"""Métriques philologiques pour documents historiques (Cercle 3).
-
-Modules orientés cas d'usage patrimoniaux par période :
-
-- :mod:`unicode_blocks`         — précision par bloc Unicode (toutes périodes)
-- :mod:`abbreviations`          — score d'expansion d'abréviations (médiéval)
-- :mod:`mufi`                   — couverture MUFI v4.0 (médiéval, PUA)
-- :mod:`early_modern_typography` — fl, fi, ſ, ã, &, ı (XVIᵉ-XVIIIᵉ siècles)
-- :mod:`modern_archives`        — Mme/Mlle/°/†/₶ (XIXᵉ-XXᵉ siècles)
-- :mod:`roman_numerals`         — numéraux romains (toutes périodes)
-- :mod:`lexical_modernization`  — top tokens GT modernisés par le moteur
-- :mod:`philological_runner`    — orchestration adaptive des 6 modules
-
-Utilité
--------
-Ces métriques répondent à la question éditoriale *« quels caractères
-historiques ce moteur restitue-t-il fidèlement ? »*. Elles ne
-participent pas à la décision « peut-on déployer ce moteur en prod ? »
-quand le corpus est moderne (les modules retournent ``None`` via
-adaptive masking sur un texte sans signal philologique).
-
-Plugin séparable
-----------------
-Distribué via l'extra pip ``picarones[historical]``. Les imports
-historiques ``from picarones.core.unicode_blocks import ...`` restent
-fonctionnels via des fichiers-shims dans :mod:`picarones.core`.
-
-Phase B du chantier de refonte en 3 cercles — voir
-:doc:`docs/architecture-cercles.md`.
-"""

picarones/{extras/historical → measurements}/philological_runner.py

@@ -30,12 +30,12 @@ from __future__ import annotations
 import logging
 from typing import Optional
 
-from picarones.core.abbreviations import compute_abbreviation_metrics
-from picarones.core.early_modern_typography import compute_early_modern_metrics
-from picarones.core.modern_archives import compute_modern_archives_metrics
-from picarones.core.mufi import compute_mufi_coverage
-from picarones.core.roman_numerals import compute_roman_numeral_metrics
-from picarones.core.unicode_blocks import compute_unicode_block_accuracy
+from picarones.measurements.abbreviations import compute_abbreviation_metrics
+from picarones.measurements.early_modern_typography import compute_early_modern_metrics
+from picarones.measurements.modern_archives import compute_modern_archives_metrics
+from picarones.measurements.mufi import compute_mufi_coverage
+from picarones.measurements.roman_numerals import compute_roman_numeral_metrics
+from picarones.measurements.unicode_blocks import compute_unicode_block_accuracy
 
 logger = logging.getLogger(__name__)
 
@@ -296,7 +296,7 @@ def _aggregate_modern_archives(per_doc: list[dict]) -> dict:
 
 
 def _aggregate_roman_numerals(per_doc: list[dict]) -> dict:
-    from picarones.core.roman_numerals import ALL_STATUSES, VALUE_PRESERVING_STATUSES
+    from picarones.measurements.roman_numerals import ALL_STATUSES, VALUE_PRESERVING_STATUSES
 
     n_total = 0
     per_status: dict[str, int] = {s: 0 for s in ALL_STATUSES}

picarones/{extras/academic → measurements}/taxonomy_intra_doc.py

@@ -42,7 +42,7 @@ import logging
 import unicodedata
 from typing import Optional
 
-from picarones.core.taxonomy import (
+from picarones.measurements.taxonomy import (
     ERROR_CLASSES,
     _is_abbreviation_error,
     _is_diacritic_error,

tests/test_phaseA_migration.py

@@ -1,318 +0,0 @@
-"""Tests de la phase A — refonte en 3 cercles (post-chantier 6).
-
-Couvre :
-
-- 4 modules `core/` déplacés vers `extras/academic/` ou
-  `extras/governance/` avec shims rétrocompat.
-- 4 renderers `report/` déplacés vers `extras/render/` avec shims.
-- Identité préservée : ``shim.X is new_location.X`` (pas de duplication
-  ni de redéfinition).
-- Hygiène anti-verdict : 5 phrases reformulées dans les templates
-  narratifs et l'i18n du rapport.
-- Document `docs/architecture-cercles.md` présent et complet.
-"""
-
-from __future__ import annotations
-
-from pathlib import Path
-
-import pytest
-
-
-# ──────────────────────────────────────────────────────────────────────────
-# 1. Modules déplacés vers extras/ — rétrocompat des imports historiques
-# ──────────────────────────────────────────────────────────────────────────
-
-
-class TestRetrocompatHistoricalImports:
-    """Les imports `from picarones.core.X` doivent continuer à fonctionner
-    après le déplacement vers `picarones.extras.*`."""
-
-    @pytest.mark.parametrize("module_path, attribute", [
-        ("picarones.core.taxonomy_intra_doc", "compute_taxonomy_position_heatmap"),
-        ("picarones.core.taxonomy_cooccurrence", "compute_taxonomy_cooccurrence"),
-        ("picarones.core.image_predictive", "compute_paleographic_complexity"),
-        ("picarones.core.image_predictive", "compute_corpus_homogeneity"),
-        ("picarones.core.image_predictive", "aggregate_corpus_predictive"),
-        ("picarones.core.module_policy", "ModuleManifest"),
-        ("picarones.core.module_policy", "validate_manifest"),
-        ("picarones.core.module_policy", "audit_module"),
-    ])
-    def test_core_alias_still_works(self, module_path: str, attribute: str):
-        import importlib
-        mod = importlib.import_module(module_path)
-        assert hasattr(mod, attribute), (
-            f"{module_path}.{attribute} a disparu après la phase A — "
-            "le shim rétrocompat est cassé"
-        )
-
-    @pytest.mark.parametrize("module_path, attribute", [
-        ("picarones.report.taxonomy_intra_doc_render", "build_taxonomy_intra_doc_html"),
-        ("picarones.report.taxonomy_cooccurrence_render", "build_taxonomy_cooccurrence_html"),
-        ("picarones.report.image_predictive_render", "build_image_predictive_html"),
-        ("picarones.report.module_audit_render", "build_module_audit_html"),
-    ])
-    def test_report_alias_still_works(self, module_path: str, attribute: str):
-        import importlib
-        mod = importlib.import_module(module_path)
-        assert hasattr(mod, attribute)
-
-
-# ──────────────────────────────────────────────────────────────────────────
-# 2. Modules accessibles via leur nouveau chemin extras/
-# ──────────────────────────────────────────────────────────────────────────
-
-
-class TestNewExtrasImports:
-    @pytest.mark.parametrize("new_path, attribute", [
-        ("picarones.extras.academic.taxonomy_intra_doc", "compute_taxonomy_position_heatmap"),
-        ("picarones.extras.academic.taxonomy_cooccurrence", "compute_taxonomy_cooccurrence"),
-        ("picarones.extras.academic.image_predictive", "aggregate_corpus_predictive"),
-        ("picarones.extras.governance.module_policy", "ModuleManifest"),
-        ("picarones.extras.render.taxonomy_intra_doc_render", "build_taxonomy_intra_doc_html"),
-        ("picarones.extras.render.taxonomy_cooccurrence_render", "build_taxonomy_cooccurrence_html"),
-        ("picarones.extras.render.image_predictive_render", "build_image_predictive_html"),
-        ("picarones.extras.render.module_audit_render", "build_module_audit_html"),
-    ])
-    def test_extras_path_works(self, new_path: str, attribute: str):
-        import importlib
-        mod = importlib.import_module(new_path)
-        assert hasattr(mod, attribute)
-
-
-# ──────────────────────────────────────────────────────────────────────────
-# 3. Identité préservée — pas de redéfinition par le shim
-# ──────────────────────────────────────────────────────────────────────────
-
-
-class TestIdentityThroughShim:
-    """Le shim doit réexporter la fonction du nouveau chemin, pas la
-    redéfinir. Sinon une métrique serait calculée différemment selon
-    le chemin d'import."""
-
-    def test_taxonomy_intra_doc_identity(self):
-        from picarones.core.taxonomy_intra_doc import (
-            compute_taxonomy_position_heatmap as via_old,
-        )
-        from picarones.extras.academic.taxonomy_intra_doc import (
-            compute_taxonomy_position_heatmap as via_new,
-        )
-        assert via_old is via_new
-
-    def test_image_predictive_identity(self):
-        from picarones.core.image_predictive import (
-            aggregate_corpus_predictive as via_old,
-        )
-        from picarones.extras.academic.image_predictive import (
-            aggregate_corpus_predictive as via_new,
-        )
-        assert via_old is via_new
-
-    def test_module_policy_identity(self):
-        from picarones.core.module_policy import ModuleManifest as via_old
-        from picarones.extras.governance.module_policy import (
-            ModuleManifest as via_new,
-        )
-        assert via_old is via_new
-
-    def test_renderer_identity(self):
-        from picarones.report.taxonomy_intra_doc_render import (
-            build_taxonomy_intra_doc_html as via_old,
-        )
-        from picarones.extras.render.taxonomy_intra_doc_render import (
-            build_taxonomy_intra_doc_html as via_new,
-        )
-        assert via_old is via_new
-
-
-# ──────────────────────────────────────────────────────────────────────────
-# 4. Vues du chantier 3 — toujours fonctionnelles
-# ──────────────────────────────────────────────────────────────────────────
-
-
-class TestChantier3ViewsStillWork:
-    """Les 5 vues du chantier 3 importent (sous-section opt-in) les
-    modules déplacés. Vérifier qu'elles tournent encore après la
-    migration."""
-
-    def test_views_import(self):
-        from picarones.report.views import (
-            build_advanced_taxonomy_view_html,
-            build_diagnostics_view_html,
-            build_economics_view_html,
-            build_pipeline_view_html,
-            build_robustness_view_html,
-        )
-        assert callable(build_advanced_taxonomy_view_html)
-        assert callable(build_diagnostics_view_html)
-        assert callable(build_economics_view_html)
-        assert callable(build_pipeline_view_html)
-        assert callable(build_robustness_view_html)
-
-    def test_advanced_taxonomy_with_intra_doc_data(self):
-        """La vue advanced_taxonomy accepte des données opt-in
-        ``intra_doc`` dont le calcul vient désormais de
-        ``picarones.extras.academic``."""
-        from picarones.extras.academic.taxonomy_intra_doc import (
-            compute_taxonomy_position_heatmap,
-        )
-        from picarones.report.views import build_advanced_taxonomy_view_html
-
-        # Calcul d'une heatmap minimaliste
-        result = compute_taxonomy_position_heatmap(
-            "abc def ghi", "abx def ghi", n_bins=3,
-        )
-        # La vue doit pouvoir composer sans crasher quand on lui passe
-        # ces données opt-in
-        report_data = {"engines": [
-            {"name": "tess", "cer": 0.05,
-             "aggregated_taxonomy": {"class_distribution": {"x": 5}}},
-            {"name": "pero", "cer": 0.08,
-             "aggregated_taxonomy": {"class_distribution": {"x": 8}}},
-        ]}
-        html = build_advanced_taxonomy_view_html(
-            report_data, {}, intra_doc=result,
-        )
-        # Pas de crash + au moins du contenu (comparison + intra_doc)
-        assert isinstance(html, str)
-
-
-# ──────────────────────────────────────────────────────────────────────────
-# 5. Hygiène anti-verdict — phrases reformulées
-# ──────────────────────────────────────────────────────────────────────────
-
-
-class TestAntiVerdictHygiene:
-    """Les 5 phrases identifiées comme prescriptives ont été reformulées
-    factuellement. Tests anti-régression."""
-
-    @pytest.fixture
-    def fr_templates(self) -> str:
-        path = (Path(__file__).parent.parent
-                / "picarones" / "core" / "narrative" / "templates" / "fr.yaml")
-        return path.read_text(encoding="utf-8")
-
-    @pytest.fixture
-    def en_templates(self) -> str:
-        path = (Path(__file__).parent.parent
-                / "picarones" / "core" / "narrative" / "templates" / "en.yaml")
-        return path.read_text(encoding="utf-8")
-
-    @pytest.fixture
-    def fr_i18n(self) -> str:
-        path = (Path(__file__).parent.parent
-                / "picarones" / "report" / "i18n" / "fr.json")
-        return path.read_text(encoding="utf-8")
-
-    @pytest.fixture
-    def en_i18n(self) -> str:
-        path = (Path(__file__).parent.parent
-                / "picarones" / "report" / "i18n" / "en.json")
-        return path.read_text(encoding="utf-8")
-
-    def test_stratum_winner_no_dominate(self, fr_templates, en_templates):
-        """`stratum_winner` ne dit plus « domine nettement » /
-        « clearly dominates ». Phrasage factuel attendu."""
-        assert "domine\n  nettement" not in fr_templates
-        assert "domine nettement" not in fr_templates
-        assert "clearly\n  dominates" not in en_templates
-        assert "clearly dominates" not in en_templates
-        # Confirmation présence du nouveau phrasage factuel
-        assert "le CER le plus bas" in fr_templates
-        assert "the lowest CER" in en_templates
-
-    def test_confidence_warning_no_fragile(self, fr_templates, en_templates):
-        """`confidence_warning` ne dit plus « fragile » mais
-        « incertitude statistique élevée »."""
-        assert "Classement fragile" not in fr_templates
-        assert "Ranking is fragile" not in en_templates
-        assert "Incertitude statistique" in fr_templates
-        assert "High statistical uncertainty" in en_templates
-
-    def test_gini_no_ideal(self, fr_i18n, en_i18n):
-        """`gini_cer_ideal` et `gini_cer_note` n'utilisent plus
-        « idéal » / « ideal » mais « lecture » / « reading »."""
-        assert "\"gini_cer_ideal\": \"— idéal" not in fr_i18n
-        assert "\"gini_cer_ideal\": \"— ideal" not in en_i18n
-        # Confirmer le nouveau phrasage
-        assert "lecture : bas-gauche" in fr_i18n
-        assert "reading: bottom-left" in en_i18n
-
-    def test_taxocomp_no_preferable(self, fr_i18n, en_i18n):
-        """`taxocomp_note` ne dit plus « préférable » / « preferable »."""
-        assert "préférable pour une édition critique" not in fr_i18n
-        assert "preferable for a critical edition" not in en_i18n
-        # Phrasage factuel
-        assert "tend à produire des erreurs plus facilement" in fr_i18n
-        assert "tends to produce errors more easily" in en_i18n
-
-
-# ──────────────────────────────────────────────────────────────────────────
-# 6. Document docs/architecture-cercles.md présent et complet
-# ──────────────────────────────────────────────────────────────────────────
-
-
-class TestArchitectureCerclesDoc:
-    @pytest.fixture
-    def doc(self) -> str:
-        path = (Path(__file__).parent.parent / "docs" / "architecture-cercles.md")
-        return path.read_text(encoding="utf-8")
-
-    def test_doc_exists(self, doc):
-        assert len(doc) > 1000
-
-    def test_doc_describes_three_circles(self, doc):
-        assert "Cercle 1" in doc
-        assert "Cercle 2" in doc
-        assert "Cercle 3" in doc
-        assert "Noyau invariant" in doc or "noyau invariant" in doc
-        assert "Plugins" in doc or "plugins" in doc
-
-    def test_doc_assigns_specific_modules(self, doc):
-        """Le document doit lister explicitement les modules de chaque cercle."""
-        # Cercle 1 — quelques noms
-        for name in ["corpus.py", "modules.py", "runner.py",
-                     "metric_registry.py", "alto_metrics.py"]:
-            assert name in doc, f"{name} doit être listé dans le doc"
-        # Cercle 3 — modules déplacés en phase A
-        for name in ["taxonomy_intra_doc", "image_predictive",
-                     "module_policy"]:
-            assert name in doc, f"{name} doit être listé dans le doc"
-
-    def test_doc_mentions_extras_path(self, doc):
-        """Le doc explique que les Cercle 3 vivent dans `extras/`."""
-        assert "extras/academic" in doc
-        assert "extras/governance" in doc
-        assert "extras/render" in doc
-
-
-# ──────────────────────────────────────────────────────────────────────────
-# 7. Modules originaux ne contiennent plus de logique métier
-# ──────────────────────────────────────────────────────────────────────────
-
-
-class TestOriginalsAreShims:
-    """Vérifie que les fichiers laissés à l'ancien emplacement sont
-    bien des shims minces, pas des copies de la logique."""
-
-    @pytest.mark.parametrize("path", [
-        "picarones/core/taxonomy_intra_doc.py",
-        "picarones/core/taxonomy_cooccurrence.py",
-        "picarones/core/image_predictive.py",
-        "picarones/core/module_policy.py",
-        "picarones/report/taxonomy_intra_doc_render.py",
-        "picarones/report/taxonomy_cooccurrence_render.py",
-        "picarones/report/image_predictive_render.py",
-        "picarones/report/module_audit_render.py",
-    ])
-    def test_is_thin_shim(self, path):
-        repo_root = Path(__file__).parent.parent
-        content = (repo_root / path).read_text(encoding="utf-8")
-        # Un shim < 30 lignes (juste docstring + 2 imports + __all__)
-        n_lines = len([line for line in content.splitlines() if line.strip()])
-        assert n_lines < 30, (
-            f"{path} fait {n_lines} lignes — devrait être un shim mince "
-            "(import + réexport, pas de logique métier)"
-        )
-        # Doit contenir l'indication du déplacement
-        assert "déplacé" in content or "extras" in content

tests/test_phaseB_migration.py

@@ -1,249 +0,0 @@
-"""Tests de la phase B — extras/historical/ (philologique vers Cercle 3).
-
-Couvre :
-
-- 8 modules philologiques (Cercle 3) déplacés vers `extras/historical/`.
-- 2 renderers correspondants déplacés vers `extras/render/`.
-- Identité préservée à travers les shims (test ``is``).
-- Intégration : `philological_runner` orchestre toujours les 6 modules
-  même après déplacement.
-- Dépendance Cercle 2 → Cercle 3 (`numerical_sequences` →
-  `roman_numerals`) continue de fonctionner via shim.
-- pyproject.toml déclare `[historical]` comme extra documentaire.
-"""
-
-from __future__ import annotations
-
-from pathlib import Path
-
-import pytest
-
-
-# ──────────────────────────────────────────────────────────────────────────
-# 1. Modules historiques accessibles via shims (rétrocompat)
-# ──────────────────────────────────────────────────────────────────────────
-
-
-class TestPhilologicalRetrocompat:
-    @pytest.mark.parametrize("module_path, attribute", [
-        ("picarones.core.unicode_blocks", "compute_unicode_block_accuracy"),
-        ("picarones.core.abbreviations", "compute_abbreviation_metrics"),
-        ("picarones.core.mufi", "compute_mufi_coverage"),
-        ("picarones.core.early_modern_typography", "compute_early_modern_metrics"),
-        ("picarones.core.modern_archives", "compute_modern_archives_metrics"),
-        ("picarones.core.roman_numerals", "compute_roman_numeral_metrics"),
-        ("picarones.core.lexical_modernization", "compute_lexical_modernization"),
-        ("picarones.core.philological_runner", "compute_philological_metrics"),
-        ("picarones.core.philological_runner", "aggregate_philological_metrics"),
-    ])
-    def test_core_alias_still_works(self, module_path: str, attribute: str):
-        import importlib
-        mod = importlib.import_module(module_path)
-        assert hasattr(mod, attribute), (
-            f"{module_path}.{attribute} a disparu après la phase B"
-        )
-
-    @pytest.mark.parametrize("module_path, attribute", [
-        ("picarones.report.philological_render", "build_philological_profile_html"),
-        ("picarones.report.lexical_modernization_render",
-         "build_lexical_modernization_html"),
-    ])
-    def test_render_alias_still_works(self, module_path: str, attribute: str):
-        import importlib
-        mod = importlib.import_module(module_path)
-        assert hasattr(mod, attribute)
-
-
-# ──────────────────────────────────────────────────────────────────────────
-# 2. Modules accessibles via leur nouveau chemin extras/historical/
-# ──────────────────────────────────────────────────────────────────────────
-
-
-class TestNewHistoricalImports:
-    @pytest.mark.parametrize("new_path, attribute", [
-        ("picarones.extras.historical.unicode_blocks",
-         "compute_unicode_block_accuracy"),
-        ("picarones.extras.historical.abbreviations",
-         "compute_abbreviation_metrics"),
-        ("picarones.extras.historical.mufi", "compute_mufi_coverage"),
-        ("picarones.extras.historical.early_modern_typography",
-         "compute_early_modern_metrics"),
-        ("picarones.extras.historical.modern_archives",
-         "compute_modern_archives_metrics"),
-        ("picarones.extras.historical.roman_numerals",
-         "compute_roman_numeral_metrics"),
-        ("picarones.extras.historical.lexical_modernization",
-         "compute_lexical_modernization"),
-        ("picarones.extras.historical.philological_runner",
-         "compute_philological_metrics"),
-        ("picarones.extras.render.philological_render",
-         "build_philological_profile_html"),
-        ("picarones.extras.render.lexical_modernization_render",
-         "build_lexical_modernization_html"),
-    ])
-    def test_extras_path_works(self, new_path: str, attribute: str):
-        import importlib
-        mod = importlib.import_module(new_path)
-        assert hasattr(mod, attribute)
-
-
-# ──────────────────────────────────────────────────────────────────────────
-# 3. Identité préservée (shim et nouveau chemin = même fonction)
-# ──────────────────────────────────────────────────────────────────────────
-
-
-class TestIdentityThroughShim:
-    def test_unicode_blocks_identity(self):
-        from picarones.core.unicode_blocks import (
-            compute_unicode_block_accuracy as via_old,
-        )
-        from picarones.extras.historical.unicode_blocks import (
-            compute_unicode_block_accuracy as via_new,
-        )
-        assert via_old is via_new
-
-    def test_philological_runner_identity(self):
-        from picarones.core.philological_runner import (
-            compute_philological_metrics as via_old,
-        )
-        from picarones.extras.historical.philological_runner import (
-            compute_philological_metrics as via_new,
-        )
-        assert via_old is via_new
-
-    def test_renderer_identity(self):
-        from picarones.report.philological_render import (
-            build_philological_profile_html as via_old,
-        )
-        from picarones.extras.render.philological_render import (
-            build_philological_profile_html as via_new,
-        )
-        assert via_old is via_new
-
-
-# ──────────────────────────────────────────────────────────────────────────
-# 4. Intégration : philological_runner orchestre toujours les 6 modules
-# ──────────────────────────────────────────────────────────────────────────
-
-
-class TestPhilologicalRunnerIntegration:
-    """Le runner philologique appelle les 6 modules
-    philologiques. Vérifie que cette chaîne fonctionne après le
-    déplacement (les imports internes traversent les shims)."""
-
-    def test_runner_returns_dict_or_none(self):
-        from picarones.core.philological_runner import (
-            compute_philological_metrics,
-        )
-        # Texte sans signal philologique → None par adaptive masking
-        result = compute_philological_metrics(
-            "Bonjour le monde", "Bonjour le monde",
-        )
-        # None acceptable (texte ASCII pur sans aucun marqueur)
-        # OU dict vide (signal nul partout)
-        assert result is None or isinstance(result, dict)
-
-    def test_runner_with_medieval_text(self):
-        """Texte médiéval avec abréviations + numéraux romains : on
-        s'attend à au moins un module qui détecte du signal."""
-        from picarones.core.philological_runner import (
-            compute_philological_metrics,
-        )
-        # ⁊ = symbole d'abréviation Capelli ; XIV = numéral romain ; ſ = long s
-        ref = "⁊ par leſ XIV. fontoyers"
-        hyp = "et par les XIV. fontoyers"
-        result = compute_philological_metrics(ref, hyp)
-        # Au moins un module doit avoir détecté du signal
-        # (abbreviations OU early_modern OU roman_numerals)
-        assert result is not None
-        assert isinstance(result, dict)
-        assert len(result) >= 1
-
-
-# ──────────────────────────────────────────────────────────────────────────
-# 5. Dépendance Cercle 2 → Cercle 3 fonctionne via shim
-# ──────────────────────────────────────────────────────────────────────────
-
-
-class TestCercle2DependsOnCercle3ViaShim:
-    """``picarones.core.numerical_sequences`` (Cercle 2,
-    measurements/) importe ``roman_numerals`` (Cercle 3, extras/).
-    Cette dépendance traverse le shim — elle continue à fonctionner."""
-
-    def test_numerical_sequences_uses_roman_numerals(self):
-        from picarones.core.numerical_sequences import (
-            compute_numerical_sequence_metrics,
-        )
-        # Texte avec numéral romain
-        result = compute_numerical_sequence_metrics(
-            "Le roi Louis XIV régna jusqu'en 1715",
-            "Le roi Louis XIV régna jusqu'en 1715",
-        )
-        # Le score strict global doit refléter au moins la détection
-        # du romain et de la date
-        assert isinstance(result, dict)
-        assert result.get("global_strict_score") is not None
-        assert result.get("global_strict_score") >= 0.5
-
-
-# ──────────────────────────────────────────────────────────────────────────
-# 6. pyproject.toml déclare l'extra [historical]
-# ──────────────────────────────────────────────────────────────────────────
-
-
-class TestPyprojectExtra:
-    def test_historical_extra_declared(self):
-        path = Path(__file__).parent.parent / "pyproject.toml"
-        content = path.read_text(encoding="utf-8")
-        # L'extra [historical] doit être déclaré, même vide
-        assert "historical = []" in content or 'historical = [' in content
-        # Documentation de l'intention présente
-        assert "extras/historical" in content
-        assert "Cercle 3" in content
-
-
-# ──────────────────────────────────────────────────────────────────────────
-# 7. Hooks builtin enregistrés conditionnels (philological + lexical)
-# ──────────────────────────────────────────────────────────────────────────
-
-
-class TestBuiltinHooksStillRegisterPhilological:
-    """Les hooks ``philological`` et ``lexical_modernization``
-    s'enregistrent au chargement de :mod:`picarones.core.builtin_hooks`
-    via les imports qui traversent les shims (``from
-    picarones.core.philological_runner import ...``)."""
-
-    def test_philological_hook_registered(self):
-        # L'import déclenche l'enregistrement
-        import picarones.core.builtin_hooks  # noqa: F401
-        from picarones.core.metric_hooks import _all_document_hook_names
-
-        assert "philological" in _all_document_hook_names()
-
-
-# ──────────────────────────────────────────────────────────────────────────
-# 8. Modules originaux sont des shims minces
-# ──────────────────────────────────────────────────────────────────────────
-
-
-class TestOriginalsAreShims:
-    @pytest.mark.parametrize("path", [
-        "picarones/core/unicode_blocks.py",
-        "picarones/core/abbreviations.py",
-        "picarones/core/mufi.py",
-        "picarones/core/early_modern_typography.py",
-        "picarones/core/modern_archives.py",
-        "picarones/core/roman_numerals.py",
-        "picarones/core/lexical_modernization.py",
-        "picarones/core/philological_runner.py",
-        "picarones/report/philological_render.py",
-        "picarones/report/lexical_modernization_render.py",
-    ])
-    def test_is_thin_shim(self, path):
-        repo_root = Path(__file__).parent.parent
-        content = (repo_root / path).read_text(encoding="utf-8")
-        n_lines = len([line for line in content.splitlines() if line.strip()])
-        assert n_lines < 30, (
-            f"{path} fait {n_lines} lignes — devrait être un shim mince"
-        )
-        assert "déplacé" in content or "extras" in content