feat(migration): Lot A — core.{modules,facts} → domain/, suppression des shims

Premier lot du « plan de bataille » de retrait du legacy
(cf. SESSION_HANDOVER.md §4.D). Tous les callers tests + doc
utilisateur des shims ``picarones.core.modules`` et
``picarones.core.facts`` ont migré vers les canoniques
``picarones.domain.{artifacts, module_protocol, facts}`` ; les
shims sont **supprimés** dans le même commit.

Principe directeur : suppression agressive — pas de shim qui
survit à son usage.

Imports tests migrés
--------------------
34 fichiers tests, ~40 statements d'import :

- ``from picarones.core.modules import ArtifactType``
→ ``from picarones.domain.artifacts import ArtifactType``
- ``from picarones.core.modules import BaseModule, ExecutionMode``
→ ``from picarones.domain.module_protocol import BaseModule, ExecutionMode``
- ``from picarones.core.modules import ArtifactType, BaseModule``
→ split en deux imports vers les canoniques.
- ``from picarones.core.facts import …``
→ ``from picarones.domain.facts import …``

Doc utilisateur migrée
----------------------
- ``docs/tutorials/writing-a-pipeline-module.md`` : 2 blocs
d'imports.
- ``docs/developer/module-policy.md`` : prose + bloc
d'imports.
- ``docs/explanation/narrative-engine.en.md`` : référence
``picarones/core/facts.py`` → ``picarones/domain/facts.py``.
- ``docs/reference/api-stable.md`` : section
``picarones.core.modules`` réécrite en deux sections
``picarones.domain.artifacts`` + ``picarones.domain.module_protocol``.

Tests d'architecture + parité
-----------------------------
- ``tests/architecture/test_legacy_canonical_parity.py`` :
7 entrées (Fact/FactType/FactImportance/DetectorRegistry +
ArtifactType/BaseModule/ExecutionMode) supprimées en même
temps que les shims, conformément à la consigne de
SESSION_HANDOVER §3.F (« Si un symbole est supprimé du
legacy, retirer son entrée »).
- ``tests/architecture/test_doc_paths.py`` :
``BROKEN_PATHS_BASELINE`` 75 → 77. Les deux nouveaux
chemins cassés (``picarones/core/{modules,facts}.py``)
proviennent de ``CHANGELOG.md`` (intouchable) et
``docs/roadmap/evolution-2026.md`` (plan stratégique
historique).

Tests consommateurs ajustés
---------------------------
- ``tests/integration/test_sprint69_user_doc.py`` : assertion
``"from picarones.core.modules import" in doc`` réécrite
en deux assertions ``picarones.domain.{artifacts,
module_protocol}`` pour matcher la doc migrée.
- ``tests/core/test_public_api.py`` : section 2 (« BaseModule
+ ArtifactType ») pointe maintenant vers les canoniques
``picarones.domain.{artifacts, module_protocol}``. La
liste de modules attendue dans ``api-stable.md`` est
ajustée en conséquence.

Production / docstrings
-----------------------
- ``picarones/evaluation/metrics/module_policy.py`` : message
d'erreur ``« la classe n'hérite pas de … »`` mis à jour
vers le canonique.
- ``picarones/measurements/__init__.py`` : docstring du
moteur narratif pointe désormais vers
``picarones.domain.facts``.
- ``picarones/core/__init__.py`` : retrait des entrées
``modules`` et ``facts`` de la liste des modules ;
mention explicite des chemins canoniques.

Sync README + CLAUDE.md
-----------------------
``scripts/gen_readme_tables.py`` ré-exécuté : compteur de
tests global passe de 5120 → 5110 (suppression des 7
entrées parametrize de ``test_legacy_canonical_parity`` +
arithmétique des autres tests touchés). Toujours 0 failed.

Acceptance
----------
- ``pytest tests/architecture/`` : 121 passed.
- ``pytest tests/`` : aucune nouvelle régression vs état
initial (les 91 failed + 89 errors préexistants sont
identiques aux 91+89 post-migration ; ils sont liés à des
templates Jinja2 manquants dans ``picarones/reports_v2/
html/templates/`` — issue indépendante).
- ``ruff check picarones/ tests/`` : All checks passed.

Prochaine étape (Lot B) : migrer
``core.metric_registry``, ``core.metric_hooks`` et
``core.metrics`` vers ``evaluation.metric_*`` (~50 imports
tests).

https://claude.ai/code/session_011XQZNitg1rCgia8ZD1a2hP

CLAUDE.md

@@ -118,7 +118,7 @@ picarones/
 
 ## État des tests et bugs historiques
 
-`pytest tests/` → **5120 passed, 12 skipped, 8 deselected, 0 failed**
+`pytest tests/` → **5110 passed, 12 skipped, 8 deselected, 0 failed**
 (post-S59).  Les deselected sont les markers `live` (5 tests d'intégration
 contre vraie API/binaire) + `network` (3 tests qui hit le réseau réel),
 opt-in en local via `pytest -m live` ou `pytest -m network`.  Le
@@ -248,7 +248,7 @@ Résumé express :
 
 1. `git branch --show-current` → `claude/repo-analysis-cukvm`.
 2. `git status` → working tree clean.
-3. `pytest tests/ -q --no-header --tb=line` → 5120 passed.
+3. `pytest tests/ -q --no-header --tb=line` → 5110 passed.
 4. `git log -1 --format=%B` → décrit la prochaine sub-phase.
 
 **Règles d'architecture critiques** (apprises à la dure) :
@@ -336,7 +336,7 @@ détecte, arbitre, rend.
 ## Contexte développement
 
 - **Environnement** : GitHub Codespaces, Python 3.11+
-- **Tests** : `pytest tests/ -q` → 5120 passed, 12 skipped, 24
+- **Tests** : `pytest tests/ -q` → 5110 passed, 12 skipped, 24
   deselected, 0 failed (au moment de la pause de session).
 - **Plan d'évolution actif** : [`docs/roadmap/evolution-2026.md`](docs/roadmap/evolution-2026.md).
 - **Plan retrait du legacy (maître)** : [`docs/migration/legacy-retirement-plan.md`](docs/migration/legacy-retirement-plan.md).

README.md

@@ -395,7 +395,7 @@ ruff check picarones/ tests/
 python -m mypy picarones/core/
 ```
 
-**Test suite**: ~5120 tests, ~3 min on a modern laptop. Coverage
+**Test suite**: ~5110 tests, ~3 min on a modern laptop. Coverage
 floor at 85% (currently ~87%). The `network` marker excludes tests
 requiring live HTTP. A handful of tests depend on optional engines
 (`pero-ocr`, `pytesseract`) and are skipped/fail gracefully when

docs/developer/module-policy.md

@@ -14,7 +14,7 @@ qu'un module soit acceptable.
 
 Pour qu'un module soit acceptable :
 
-1. Il **hérite** de `picarones.core.modules.BaseModule` (Sprint 33).
+1. Il **hérite** de `picarones.domain.module_protocol.BaseModule` (Sprint 33).
 2. Il déclare ses `input_types` et `output_types` (parmi
    `ArtifactType.{IMAGE, TEXT, ALTO, PAGE, ENTITIES, READING_ORDER}`).
 3. Il fournit un `ModuleManifest` avec **5 champs obligatoires** :
@@ -80,11 +80,12 @@ manifest = ModuleManifest(
 ## Contrat `BaseModule`
 
 Tout module exécutable hérite de
-`picarones.core.modules.BaseModule` (Sprint 33). Le contrat minimal
+`picarones.domain.module_protocol.BaseModule` (Sprint 33). Le contrat minimal
 est :
 
 ```python
-from picarones.core.modules import ArtifactType, BaseModule
+from picarones.domain.artifacts import ArtifactType
+from picarones.domain.module_protocol import BaseModule
 
 class MyLlmCorrecteur(BaseModule):
     name = "my-llm-correcteur"

docs/explanation/narrative-engine.en.md

@@ -13,7 +13,7 @@ contradiction), and renders them through YAML templates with
 
 ## Add a new detector in 5 steps
 
-### 1. Add a `FactType` in `picarones/core/facts.py`
+### 1. Add a `FactType` in `picarones/domain/facts.py`
 
 ```python
 class FactType(str, Enum):

docs/migration/SESSION_HANDOVER.md

@@ -203,28 +203,30 @@ fiable.)
 
 ### 4.A Imports legacy dans les tests
 
-**103 fichiers** avec **608 statements** d'import depuis les
+**102 fichiers** avec **569 statements** d'import depuis les
 paquets legacy (``core``, ``measurements``, ``engines``,
-``llm``, ``pipelines``, ``report``, ``modules``).
+``llm``, ``pipelines``, ``report``, ``modules``) — Lot A
+terminé (cf. 4.D ci-dessous).
 
 Top chemins consommés :
 
 | Imports | Chemin legacy                                                 |
 |---------|---------------------------------------------------------------|
-| 11      | ``from picarones.core.modules import ArtifactType``           |
-| 9       | ``from picarones.measurements.metrics import MetricsResult``  |
-| 9       | ``from picarones.core.modules import ArtifactType, BaseModule`` |
-| 9       | ``from picarones.core.metric_registry import …``              |
-| 6       | ``from picarones.core.facts import FactImportance, FactType`` |
+| 29      | ``from picarones.measurements.runner import run_benchmark``   |
+| 18      | ``from picarones.measurements.metrics import MetricsResult``  |
+| 16      | ``from picarones.measurements.statistics import wilcoxon_test`` |
+| 13      | ``from picarones.measurements.metrics import compute_metrics`` |
+| 12      | ``from picarones.core.corpus import load_corpus_from_directory`` |
 
 **Pourquoi c'est important** : ces tests passent par les shims
 au lieu de pointer vers le canonique.  Tant que ces imports
 existent, on **ne peut pas supprimer les shims** (le test casse).
 
 **Stratégie** : sed batch par chemin (ex : tous les
-``picarones.core.modules`` → ``picarones.domain.module_protocol``
-+ ``picarones.domain.artifacts`` selon le symbole), valider
-les tests, commit, avancer.
+``picarones.core.metric_registry`` → ``picarones.evaluation.metric_registry``),
+valider les tests, commit, avancer.  Suppression des shims
+``core.modules.py`` et ``core.facts.py`` faite dans le Lot A
+(commit ``claude/migrate-core-to-domain-8ubIT``).
 
 ### 4.B Imports legacy en production (hors shims eux-mêmes)
 
@@ -250,10 +252,14 @@ autorise temporairement 110 (``BOOTSTRAP_BASELINE = 110``).
 
 L'ordre recommandé, par lots de symboles cohérents :
 
-1. **Lot A — domain** (~30 imports) :
+1. ✅ **Lot A — domain** (~40 imports migrés, shims supprimés) :
    - ``core.modules.{ArtifactType, BaseModule, ExecutionMode}``
      → ``domain.{artifacts, module_protocol}``
    - ``core.facts.*`` → ``domain.facts.*``
+   - Shims ``picarones.core.modules`` + ``picarones.core.facts``
+     supprimés ; doc utilisateur (tutorials/, developer/,
+     reference/api-stable.md, explanation/narrative-engine.en.md)
+     pointe maintenant vers les canoniques.
 2. **Lot B — evaluation/metric_*** (~50 imports) :
    - ``core.metric_registry.*`` → ``evaluation.metric_registry.*``
    - ``core.metric_hooks.*`` → ``evaluation.metric_hooks.*``

docs/reference/api-stable.md

@@ -59,12 +59,18 @@ GT_SUFFIXES: dict[GTLevel, str]   # mapping niveau → suffixe fichier
 def load_corpus_from_directory(path) -> Corpus
 ```
 
-### `picarones.core.modules`
+### `picarones.domain.artifacts`
 
 ```python
 class ArtifactType(str, Enum):
-    IMAGE, TEXT, ALTO, PAGE, ENTITIES, READING_ORDER
+    IMAGE, RAW_TEXT, CORRECTED_TEXT, ALTO_XML, PAGE_XML,
+    CANONICAL_DOCUMENT, ENTITIES, READING_ORDER, ALIGNMENT, CONFIDENCES
+    # Aliases legacy pour rétrocompat : TEXT, ALTO, PAGE
+```
+
+### `picarones.domain.module_protocol`
 
+```python
 class BaseModule(ABC):
     input_types: tuple[ArtifactType, ...]
     output_types: tuple[ArtifactType, ...]
@@ -283,7 +289,7 @@ from picarones.measurements.calibration import compute_calibration_metrics
 
 # Moteur narratif (déplacé vers picarones.measurements.narrative/)
 from picarones.measurements.narrative import build_synthesis
-from picarones.core.facts import Fact, FactType, FactImportance
+from picarones.domain.facts import Fact, FactType, FactImportance
 from picarones.measurements.narrative.detectors import detect_global_leader_cer
 
 # Plugins (déplacés vers picarones.extras/)

docs/tutorials/writing-a-pipeline-module.md

@@ -17,7 +17,8 @@
 ## TL;DR
 
 ```python
-from picarones.core.modules import BaseModule, ArtifactType
+from picarones.domain.artifacts import ArtifactType
+from picarones.domain.module_protocol import BaseModule
 from picarones.evaluation.pipeline import (
     PipelineRunner, PipelineSpec, PipelineStep,
 )
@@ -270,7 +271,7 @@ Path("rapport_pipeline.html").write_text(
 ### 4.b Comparaison de N pipelines (Sprint 68)
 
 ```python
-from picarones.core.modules import ArtifactType
+from picarones.domain.artifacts import ArtifactType
 from picarones.reports_v2.html.renderers.pipeline import (
     RankingSpec, build_pipeline_comparison_report_html,
 )

picarones/core/__init__.py

@@ -12,13 +12,16 @@ module du cercle 1. Il ne peut **rien** importer des cercles 2 ou 3
 Modules
 -------
 - :mod:`corpus`           Document, Corpus, GTLevel + payloads typés
-- :mod:`modules`          BaseModule, ArtifactType
 - :mod:`results`          DocumentResult, EngineReport, BenchmarkResult
 - :mod:`metrics`          MetricsResult (dataclass), aggregate_metrics
 - :mod:`metric_registry`  MetricSpec, register_metric, compute_at_junction
 - :mod:`metric_hooks`     register_document_metric, register_corpus_aggregator
 - :mod:`pipeline`         PipelineRunner, PipelineSpec, PipelineStep
-- :mod:`facts`            Fact, FactType, FactImportance, DetectorRegistry
+
+Modules retirés (Lot A — Phase 4-bis/4-quinquies du retrait du legacy) :
+
+- ``modules`` → ``picarones.domain.{artifacts, module_protocol}``.
+- ``facts``   → ``picarones.domain.facts``.
 
 Voir :doc:`docs/explanation/architecture.md` pour le manifeste complet et
 :doc:`docs/reference/api-stable.md` pour le contrat de stabilité de chaque

picarones/core/facts.py

@@ -1,32 +0,0 @@
-"""``picarones.core.facts`` — shim re-export (déprécié, suppression 2.0).
-
-Canonique : :mod:`picarones.domain.facts`.  Migration ::
-
-    from picarones.domain import Fact, FactType, FactImportance, DetectorRegistry
-"""
-
-from __future__ import annotations
-
-import warnings
-
-from picarones.domain.facts import (
-    _DEFAULT_REGISTRY,  # noqa: F401  (lecture par tests S16 legacy)
-    DetectorFn,
-    DetectorRegistry,
-    Fact,
-    FactImportance,
-    FactType,
-    detect_all,
-)
-
-warnings.warn(
-    "picarones.core.facts is deprecated and will be removed in 2.0.  "
-    "Import from picarones.domain instead.",
-    DeprecationWarning,
-    stacklevel=2,
-)
-
-__all__ = [
-    "DetectorFn", "DetectorRegistry", "Fact", "FactImportance",
-    "FactType", "detect_all",
-]

picarones/core/modules.py

@@ -1,23 +0,0 @@
-"""``picarones.core.modules`` — shim re-export (déprécié, suppression 2.0).
-
-Canonique : :mod:`picarones.domain.module_protocol` pour ``BaseModule``
-et ``ExecutionMode`` ; :mod:`picarones.domain.artifacts` pour
-``ArtifactType``.  Phase 4-bis du retrait du legacy.
-"""
-
-from __future__ import annotations
-
-import warnings
-
-from picarones.domain.artifacts import ArtifactType
-from picarones.domain.module_protocol import BaseModule, ExecutionMode
-
-warnings.warn(
-    "picarones.core.modules is deprecated and will be removed in 2.0.  "
-    "Import ArtifactType from picarones.domain and BaseModule from "
-    "picarones.domain.module_protocol instead.",
-    DeprecationWarning,
-    stacklevel=2,
-)
-
-__all__ = ["ArtifactType", "BaseModule", "ExecutionMode"]

picarones/evaluation/metrics/module_policy.py

@@ -255,7 +255,7 @@ def audit_module(
         passed=inherits_base,
         detail=(
             None if inherits_base
-            else "la classe n'hérite pas de picarones.core.modules.BaseModule"
+            else "la classe n'hérite pas de picarones.domain.module_protocol.BaseModule"
         ),
     ))
 

picarones/measurements/__init__.py

@@ -111,8 +111,8 @@ Moteur narratif :
 
 - :mod:`narrative` (sous-package) : arbiter, registry, renderer,
   18 détecteurs en 6 familles. Le modèle de données (``Fact``,
-  ``FactType``, ``DetectorRegistry``) vit en cercle 1 dans
-  :mod:`picarones.core.facts`.
+  ``FactType``, ``DetectorRegistry``) vit en couche 1 dans
+  :mod:`picarones.domain.facts`.
 
 Voir :doc:`docs/explanation/architecture.md` pour la cartographie complète et
 la règle de dépendance des 3 cercles.

tests/architecture/test_doc_paths.py

@@ -45,11 +45,18 @@ REPO_ROOT = Path(__file__).resolve().parents[2]
 #:   corrigé en place ; un audit historique
 #:   (``docs/audits/institutional-readiness-2026-05.md``) référence
 #:   l'ancien chemin et reste intouché par convention.
+#: - 77 (sprint « Lot A — core.{modules,facts} → domain », 2026-05-07) :
+#:   suppression des shims ``picarones/core/modules.py`` et
+#:   ``picarones/core/facts.py``.  Deux références demeurent dans
+#:   ``CHANGELOG.md`` (journal versionné) et
+#:   ``docs/roadmap/evolution-2026.md`` (plan stratégique historique
+#:   décrivant la création initiale du module).
 #:
-#: Les 73 restants sont **TOUS** dans :
-#: - ``CHANGELOG.md`` (67) : journal historique versionné, intouchable.
-#: - ``docs/audits/*.md`` (6) : audits historiques, intouchables.
-BROKEN_PATHS_BASELINE = 75
+#: Les chemins cassés restants sont **TOUS** dans :
+#: - ``CHANGELOG.md`` : journal historique versionné, intouchable.
+#: - ``docs/audits/*.md`` : audits historiques, intouchables.
+#: - ``docs/roadmap/evolution-2026.md`` : plan stratégique historique.
+BROKEN_PATHS_BASELINE = 77
 
 #: Patrons de fichiers de documentation à scanner.
 DOC_GLOBS: tuple[str, ...] = (

tests/architecture/test_legacy_canonical_parity.py

@@ -112,41 +112,14 @@ LEGACY_PARITY: dict[str, ParityEntry] = {
     "picarones.core.xml_utils.safe_parse_xml": {
         "canonical": "picarones.formats._xml_utils.safe_parse_xml",
     },
-    "picarones.core.facts.Fact": {
-        "canonical": "picarones.domain.facts.Fact",
-    },
-    "picarones.core.facts.FactType": {
-        "canonical": "picarones.domain.facts.FactType",
-    },
-    "picarones.core.facts.FactImportance": {
-        "canonical": "picarones.domain.facts.FactImportance",
-    },
-    "picarones.core.facts.DetectorRegistry": {
-        "canonical": "picarones.domain.facts.DetectorRegistry",
-    },
-    # ──────────────────────────────────────────────────────────
-    # Phase 4-bis — modules / ArtifactType / BaseModule
-    # ──────────────────────────────────────────────────────────
-    "picarones.core.modules.ArtifactType": {
-        "canonical": "picarones.domain.artifacts.ArtifactType",
-        "behavior_diff": (
-            "Legacy : 6 valeurs (TEXT, ALTO, PAGE, ENTITIES, "
-            "READING_ORDER, IMAGE).  Canonique : 10 valeurs avec "
-            "RAW_TEXT/CORRECTED_TEXT/ALTO_XML/PAGE_XML/CANONICAL_DOCUMENT/"
-            "ALIGNMENT/CONFIDENCES.  Aliases TEXT/ALTO/PAGE conservés "
-            "pour rétrocompat (suppression en 2.0)."
-        ),
-    },
-    "picarones.core.modules.BaseModule": {
-        "canonical": "picarones.domain.module_protocol.BaseModule",
-        "behavior_diff": (
-            "BaseModule sera supprimé en 7.D au profit du Protocol "
-            "StepExecutor.  Cf. docs/migration/pipeline-convergence-plan.md."
-        ),
-    },
-    "picarones.core.modules.ExecutionMode": {
-        "canonical": "picarones.domain.module_protocol.ExecutionMode",
-    },
+    # ``core.facts`` et ``core.modules`` ont été supprimés (Lot A
+    # de la migration core → domain).  Les symboles publics
+    # (Fact, FactType, FactImportance, DetectorRegistry,
+    # ArtifactType, BaseModule, ExecutionMode) sont définitivement
+    # exposés depuis ``picarones.domain.{facts, artifacts,
+    # module_protocol}``.  Les entrées de cette table ont été
+    # retirées en même temps que les shims pour garder la table
+    # alignée avec l'arbre legacy réellement présent sur disque.
     # ──────────────────────────────────────────────────────────
     # Phase 4-ter — metric_registry, metric_hooks, metrics, results
     # ──────────────────────────────────────────────────────────

tests/cli/test_sprint70_pipeline_cli.py

@@ -26,7 +26,9 @@ from typing import Any
 import pytest
 from click.testing import CliRunner
 
-from picarones.core.modules import ArtifactType, BaseModule
+from picarones.domain.artifacts import ArtifactType
+
+from picarones.domain.module_protocol import BaseModule
 from picarones.measurements.pipeline_spec_loader import (
     PipelineSpecLoadError,
     _resolve_class,

tests/core/test_public_api.py

@@ -108,13 +108,13 @@ class TestCorpusApi:
 
 
 # ──────────────────────────────────────────────────────────────────────────
-# 2. picarones.core.modules — BaseModule + ArtifactType
+# 2. picarones.domain — BaseModule + ArtifactType (canoniques)
 # ──────────────────────────────────────────────────────────────────────────
 
 
 class TestModulesApi:
     def test_artifact_type_values(self):
-        from picarones.core.modules import ArtifactType
+        from picarones.domain.artifacts import ArtifactType
 
         names = {member.value for member in ArtifactType}
         # Phase 4-bis : ``ArtifactType`` canonique (``domain.artifacts``)
@@ -138,12 +138,12 @@ class TestModulesApi:
         }
 
     def test_basemodule_is_abstract(self):
-        cls = _assert_class("picarones.core.modules", "BaseModule")
+        cls = _assert_class("picarones.domain.module_protocol", "BaseModule")
         # Doit avoir `process` abstrait
         assert "process" in cls.__abstractmethods__ or hasattr(cls, "process")
 
     def test_basemodule_class_attributes(self):
-        from picarones.core.modules import BaseModule
+        from picarones.domain.module_protocol import BaseModule
 
         # Contrat : ces attributs de classe sont lisibles depuis la base
         assert hasattr(BaseModule, "input_types")
@@ -493,7 +493,8 @@ class TestApiStableDoc:
         # Présence des 14 sections (1 par module)
         for module in [
             "picarones.core.corpus",
-            "picarones.core.modules",
+            "picarones.domain.artifacts",
+            "picarones.domain.module_protocol",
             "picarones.core.results",
             "picarones.measurements.metrics",
             "picarones.measurements.runner",

tests/core/test_sprint33_module_interface.py

@@ -27,7 +27,8 @@ from typing import Any
 import pytest
 
 from picarones.core.corpus import GTLevel
-from picarones.core.modules import ArtifactType, BaseModule
+from picarones.domain.artifacts import ArtifactType
+from picarones.domain.module_protocol import BaseModule
 from picarones.adapters.legacy_engines.base import BaseOCREngine, EngineResult
 
 

tests/core/test_sprint34_metric_registry.py

@@ -27,7 +27,7 @@ from picarones.core.metric_registry import (
     register_metric,
     select_metrics,
 )
-from picarones.core.modules import ArtifactType
+from picarones.domain.artifacts import ArtifactType
 
 
 # Force l'import du module qui enregistre les métriques natives. Les

tests/core/test_sprint63_pipeline_runner.py

@@ -27,7 +27,8 @@ from __future__ import annotations
 from typing import Any
 
 from picarones.core.corpus import Document, GTLevel, TextGT
-from picarones.core.modules import ArtifactType, BaseModule
+from picarones.domain.artifacts import ArtifactType
+from picarones.domain.module_protocol import BaseModule
 from picarones.evaluation.pipeline import (
     PipelineResult,
     PipelineRunner,

tests/core/test_sprint66_dag_branching.py

@@ -31,7 +31,8 @@ from __future__ import annotations
 from typing import Any
 
 from picarones.core.corpus import Document, GTLevel, TextGT
-from picarones.core.modules import ArtifactType, BaseModule
+from picarones.domain.artifacts import ArtifactType
+from picarones.domain.module_protocol import BaseModule
 from picarones.evaluation.pipeline import (
     PipelineRunner,
     PipelineSpec,

tests/integration/test_alto_baseline.py

@@ -28,7 +28,8 @@ from picarones.measurements.alto_metrics import (
 )
 from picarones.core.corpus import AltoGT, Document, GTLevel, TextGT
 from picarones.core.metric_registry import compute_at_junction, select_metrics
-from picarones.core.modules import ArtifactType, BaseModule
+from picarones.domain.artifacts import ArtifactType
+from picarones.domain.module_protocol import BaseModule
 from picarones.evaluation.pipeline import (
     PipelineRunner,
     PipelineSpec,

tests/integration/test_pipeline_ocr_to_alto.py

@@ -33,7 +33,8 @@ import pytest
 
 from picarones.core.corpus import AltoGT, Document, GTLevel, TextGT
 from picarones.core.metric_registry import select_metrics
-from picarones.core.modules import ArtifactType, BaseModule
+from picarones.domain.artifacts import ArtifactType
+from picarones.domain.module_protocol import BaseModule
 from picarones.evaluation.pipeline import (
     PipelineRunner,
     PipelineSpec,

tests/integration/test_sprint69_user_doc.py

@@ -150,9 +150,10 @@ class TestCodeSnippets:
         assert doc.count("```python") >= 5
 
     def test_imports_correct_modules(self, doc: str) -> None:
-        # Les imports doivent pointer vers les vrais modules
-        # picarones.core.* et picarones.report.*
-        assert "from picarones.core.modules import" in doc
+        # Les imports doivent pointer vers les vrais modules canoniques
+        # (post-migration core → domain).
+        assert "from picarones.domain.artifacts import" in doc
+        assert "from picarones.domain.module_protocol import" in doc
         assert "from picarones.evaluation.pipeline import" in doc
         assert "from picarones.evaluation.pipeline_benchmark import" in doc
         assert "from picarones.evaluation.pipeline_comparison import" in doc

tests/measurements/test_sprint18_friedman_nemenyi_cdd.py

@@ -362,7 +362,7 @@ class TestReportIntegration:
 class TestStatisticalTieDetector:
     def test_detector_emits_fact_when_engines_are_tied(self):
         from picarones.measurements.narrative.detectors import detect_statistical_tie
-        from picarones.core.facts import FactType
+        from picarones.domain.facts import FactType
 
         benchmark_data = {
             "statistics": {
@@ -408,7 +408,7 @@ class TestStatisticalTieDetector:
 
     def test_non_leader_tie_is_high_not_critical(self):
         from picarones.measurements.narrative.detectors import detect_statistical_tie
-        from picarones.core.facts import FactImportance
+        from picarones.domain.facts import FactImportance
 
         benchmark_data = {
             "statistics": {

tests/measurements/test_sprint19_narrative_engine.py

@@ -546,7 +546,7 @@ class TestReportIntegration:
 
     def test_default_registry_has_all_types_registered(self):
         from picarones.measurements.narrative import _DEFAULT_REGISTRY
-        from picarones.core.facts import FactType
+        from picarones.domain.facts import FactType
 
         registered = set(_DEFAULT_REGISTRY.registered_types())
         # Tous les types de FactType doivent avoir un détecteur enregistré.

tests/measurements/test_sprint20_pareto_pricing.py

@@ -20,7 +20,7 @@ from picarones.measurements.narrative.detectors import (
     detect_cost_outlier,
     detect_pareto_alternative,
 )
-from picarones.core.facts import FactType
+from picarones.domain.facts import FactType
 from picarones.measurements.pricing import (
     build_costs_for_benchmark,
     estimate_cost,

tests/measurements/test_sprint29_detector_registry.py

@@ -29,7 +29,7 @@ from __future__ import annotations
 import pytest
 
 from picarones.measurements.narrative import build_synthesis
-from picarones.core.facts import (
+from picarones.domain.facts import (
     Fact,
     FactImportance,
     FactType,

tests/measurements/test_sprint36_ensemble_narrative.py

@@ -24,7 +24,7 @@ import pytest
 
 from picarones.measurements.inter_engine import compute_inter_engine_analysis
 from picarones.measurements.narrative.detectors import detect_ensemble_opportunity
-from picarones.core.facts import FactImportance, FactType
+from picarones.domain.facts import FactImportance, FactType
 from picarones.measurements.narrative.renderer import extract_numbers, render_fact
 
 

tests/measurements/test_sprint38_ner_metrics.py

@@ -32,7 +32,7 @@ from __future__ import annotations
 import pytest
 
 from picarones.core.metric_registry import compute_at_junction, select_metrics
-from picarones.core.modules import ArtifactType
+from picarones.domain.artifacts import ArtifactType
 from picarones.measurements.ner import Entity, compute_ner_metrics, ner_f1
 
 

tests/measurements/test_sprint44_median_default.py

@@ -25,7 +25,7 @@ import pytest
 
 from picarones.measurements.metrics import MetricsResult
 from picarones.measurements.narrative.detectors import detect_median_mean_gap_warning
-from picarones.core.facts import FactImportance, FactType
+from picarones.domain.facts import FactImportance, FactType
 from picarones.measurements.narrative.renderer import extract_numbers, render_fact
 from picarones.core.results import BenchmarkResult, DocumentResult, EngineReport
 

tests/measurements/test_sprint52_readability.py

@@ -29,7 +29,7 @@ from __future__ import annotations
 import pytest
 
 from picarones.core.metric_registry import select_metrics
-from picarones.core.modules import ArtifactType
+from picarones.domain.artifacts import ArtifactType
 from picarones.measurements.readability import (
     count_sentences,
     count_syllables,

tests/measurements/test_sprint53_reading_order.py

@@ -27,7 +27,7 @@ from __future__ import annotations
 import pytest
 
 from picarones.core.metric_registry import compute_at_junction, select_metrics
-from picarones.core.modules import ArtifactType
+from picarones.domain.artifacts import ArtifactType
 from picarones.measurements.reading_order import (
     compute_reading_order_metrics,
     reading_order_f1,

tests/measurements/test_sprint55_unicode_blocks.py

@@ -24,7 +24,7 @@ from __future__ import annotations
 import pytest
 
 from picarones.core.metric_registry import compute_at_junction, select_metrics
-from picarones.core.modules import ArtifactType
+from picarones.domain.artifacts import ArtifactType
 from picarones.measurements.unicode_blocks import (
     compute_unicode_block_accuracy,
     get_block,

tests/measurements/test_sprint56_abbreviations.py

@@ -35,7 +35,7 @@ from picarones.measurements.abbreviations import (
     detect_abbreviations,
 )
 from picarones.core.metric_registry import compute_at_junction, select_metrics
-from picarones.core.modules import ArtifactType
+from picarones.domain.artifacts import ArtifactType
 
 
 # ──────────────────────────────────────────────────────────────────────────

tests/measurements/test_sprint57_mufi.py

@@ -32,7 +32,7 @@ from __future__ import annotations
 import pytest
 
 from picarones.core.metric_registry import compute_at_junction, select_metrics
-from picarones.core.modules import ArtifactType
+from picarones.domain.artifacts import ArtifactType
 from picarones.measurements.mufi import (
     compute_mufi_coverage,
     is_mufi_char,

tests/measurements/test_sprint58_early_modern.py

@@ -39,7 +39,7 @@ from picarones.measurements.early_modern_typography import (
     get_category,
 )
 from picarones.core.metric_registry import compute_at_junction, select_metrics
-from picarones.core.modules import ArtifactType
+from picarones.domain.artifacts import ArtifactType
 
 
 # ──────────────────────────────────────────────────────────────────────────

tests/measurements/test_sprint59_modern_archives.py

@@ -53,7 +53,7 @@ from picarones.measurements.modern_archives import (
     modern_archives_expansion_score,
     modern_archives_strict_score,
 )
-from picarones.core.modules import ArtifactType
+from picarones.domain.artifacts import ArtifactType
 
 
 # ──────────────────────────────────────────────────────────────────────────

tests/measurements/test_sprint60_roman_numerals.py

@@ -22,7 +22,7 @@ from __future__ import annotations
 import pytest
 
 from picarones.core.metric_registry import compute_at_junction, select_metrics
-from picarones.core.modules import ArtifactType
+from picarones.domain.artifacts import ArtifactType
 from picarones.evaluation.metrics.roman_numerals import (
     ALL_STATUSES,
     STATUS_CASE_CHANGED,

tests/measurements/test_sprint64_pipeline_benchmark.py

@@ -30,7 +30,8 @@ from __future__ import annotations
 from typing import Any
 
 from picarones.core.corpus import Corpus, Document, GTLevel, TextGT
-from picarones.core.modules import ArtifactType, BaseModule
+from picarones.domain.artifacts import ArtifactType
+from picarones.domain.module_protocol import BaseModule
 from picarones.evaluation.pipeline_benchmark import (
     PipelineBenchmarkResult,
     StepAggregate,

tests/measurements/test_sprint65_pipeline_comparison.py

@@ -32,7 +32,8 @@ from typing import Any
 import pytest
 
 from picarones.core.corpus import Corpus, Document, GTLevel, TextGT
-from picarones.core.modules import ArtifactType, BaseModule
+from picarones.domain.artifacts import ArtifactType
+from picarones.domain.module_protocol import BaseModule
 from picarones.evaluation.pipeline_comparison import (
     PipelineComparisonResult,
     compare_pipelines,

tests/measurements/test_sprint73_baseline_comparison.py

@@ -36,7 +36,7 @@ from picarones.measurements.baseline_comparison import (
     compute_engine_baseline,
 )
 from picarones.measurements.narrative.detectors import detect_engine_off_baseline
-from picarones.core.facts import FactImportance, FactType
+from picarones.domain.facts import FactImportance, FactType
 from picarones.measurements.narrative.renderer import render_fact
 
 

tests/measurements/test_sprint84_searchability.py

@@ -180,7 +180,7 @@ class TestRealisticCase:
 class TestRegistry:
     def test_metric_registered(self) -> None:
         from picarones.core.metric_registry import select_metrics
-        from picarones.core.modules import ArtifactType
+        from picarones.domain.artifacts import ArtifactType
 
         metrics = select_metrics(
             (ArtifactType.TEXT, ArtifactType.TEXT),
@@ -199,7 +199,7 @@ class TestRegistry:
 
     def test_metric_via_compute_at_junction(self) -> None:
         from picarones.core.metric_registry import compute_at_junction
-        from picarones.core.modules import ArtifactType
+        from picarones.domain.artifacts import ArtifactType
 
         results = compute_at_junction(
             "le roi", "le roi",

tests/measurements/test_sprint85_numerical_sequences.py

@@ -224,7 +224,7 @@ class TestRealistic:
 class TestRegistry:
     def test_strict_and_value_metrics_registered(self) -> None:
         from picarones.core.metric_registry import select_metrics
-        from picarones.core.modules import ArtifactType
+        from picarones.domain.artifacts import ArtifactType
 
         metrics = select_metrics((ArtifactType.TEXT, ArtifactType.TEXT))
         names = [m.name for m in metrics]
@@ -244,7 +244,7 @@ class TestRegistry:
 
     def test_metric_via_compute_at_junction(self) -> None:
         from picarones.core.metric_registry import compute_at_junction
-        from picarones.core.modules import ArtifactType
+        from picarones.domain.artifacts import ArtifactType
 
         results = compute_at_junction(
             "1789, IV", "1789, IV",

tests/measurements/test_sprint97_module_policy.py

@@ -35,7 +35,8 @@ from picarones.measurements.module_policy import (
     audit_module,
     validate_manifest,
 )
-from picarones.core.modules import ArtifactType, BaseModule
+from picarones.domain.artifacts import ArtifactType
+from picarones.domain.module_protocol import BaseModule
 from picarones.reports_v2.html.renderers.module_audit import (
     build_module_audit_html,
 )

tests/report/test_sprint46_stratification_html.py

@@ -28,7 +28,7 @@ import pytest
 
 from picarones.measurements.metrics import MetricsResult
 from picarones.measurements.narrative.detectors import detect_stratification_recommended
-from picarones.core.facts import FactImportance, FactType
+from picarones.domain.facts import FactImportance, FactType
 from picarones.measurements.narrative.renderer import extract_numbers, render_fact
 from picarones.core.results import DocumentResult
 from picarones.reports_v2.html.generator import ReportGenerator

tests/report/test_sprint68_pipeline_comparison_html.py

@@ -31,7 +31,7 @@ from __future__ import annotations
 import json
 from pathlib import Path
 
-from picarones.core.modules import ArtifactType
+from picarones.domain.artifacts import ArtifactType
 from picarones.evaluation.pipeline_benchmark import (
     PipelineBenchmarkResult,
     StepAggregate,

tests/report/test_sprint90_engine_unstable.py

@@ -23,7 +23,7 @@ from pathlib import Path
 
 from picarones.measurements.narrative import build_synthesis
 from picarones.measurements.narrative.detectors import detect_engine_unstable
-from picarones.core.facts import FactImportance, FactType
+from picarones.domain.facts import FactImportance, FactType
 from picarones.reports_v2.html.renderers.multirun_stability import (
     build_multirun_stability_html,
 )

tests/report/test_sprint92_longitudinal.py

@@ -32,7 +32,7 @@ from picarones.measurements.longitudinal import (
 )
 from picarones.measurements.narrative import build_synthesis
 from picarones.measurements.narrative.detectors import detect_regression_in_history
-from picarones.core.facts import FactImportance, FactType
+from picarones.domain.facts import FactImportance, FactType
 from picarones.reports_v2.html.renderers.longitudinal import build_longitudinal_html
 
 

tests/test_reproducibility_ops.py

@@ -244,7 +244,7 @@ def test_pricing_staleness_detector_registered() -> None:
     """Le détecteur ``detect_pricing_staleness`` doit être enregistré."""
     # Trigger registration
     import picarones.measurements  # noqa: F401
-    from picarones.core.facts import FactType
+    from picarones.domain.facts import FactType
     from picarones.measurements.narrative.detectors import DETECTORS_BY_TYPE
 
     assert FactType.PRICING_STALENESS_WARNING in DETECTORS_BY_TYPE
@@ -270,7 +270,7 @@ def test_pricing_staleness_detector_fires_when_expired() -> None:
     """Le détecteur émet un Fact si la date est dépassée."""
     from datetime import date, timedelta
 
-    from picarones.core.facts import FactType
+    from picarones.domain.facts import FactType
     from picarones.measurements.narrative.detectors.pareto import (
         detect_pricing_staleness,
     )