fix(registry): time_coverage del config vince sull'period editoriale stale

tests/test_registry_builder.py

@@ -305,6 +305,63 @@ def test_clean_catalog_normalizes_semantic_type_from_editorial(tmp_path: Path) -
     assert "semantic_type" not in cols["value"]
 
 
+@pytest.mark.policy
+def test_period_time_coverage_beats_editorial(tmp_path: Path) -> None:
+    """policy: time_coverage del config vince sull'period editoriale stale.
+
+    Regression energia-italia #6: l'espansione della serie (2015→2000) non
+    propagava al registry perché il merge editoriale sovrascriveva sempre
+    il period con quello storico.
+    """
+    layout = _make_repo(tmp_path)
+    existing = {
+        "datasets": [
+            {
+                "slug": "my_dataset",
+                "name": "My Dataset",
+                "description": "",
+                "source_id": "src1",
+                "period": {"start": 2015, "end": 2024},  # stale
+                "columns": [],
+                "location": {"type": "gcs", "path": "gs://dataciviclab-clean/my_dataset/"},
+                "stage": "incubating",
+            }
+        ]
+    }
+
+    catalog, _ = build_clean_catalog(layout, existing=existing)
+    ds = catalog["datasets"][0]
+
+    # MART_YML dichiara time_coverage 2000-2024: deve vincere su 2015-2024
+    assert ds["period"] == {"start": 2000, "end": 2024}
+
+
+@pytest.mark.policy
+def test_period_editorial_fills_when_no_time_coverage(tmp_path: Path) -> None:
+    """policy: senza time_coverage nel config, l'period editoriale resta la fonte."""
+    # with_mart=False → SIMPLE_YML (nessun time_coverage), parquet già nel fixture
+    layout = _make_repo(tmp_path, with_mart=False)
+    existing = {
+        "datasets": [
+            {
+                "slug": "my_dataset",
+                "name": "Simple",
+                "description": "",
+                "source_id": "src1",
+                "period": {"start": 2020, "end": 2026},
+                "columns": [],
+                "location": {"type": "gcs", "path": "gs://dataciviclab-clean/my_dataset/"},
+                "stage": "incubating",
+            }
+        ]
+    }
+
+    catalog, _ = build_clean_catalog(layout, existing=existing)
+    ds = catalog["datasets"][0]
+
+    assert ds["period"] == {"start": 2020, "end": 2026}
+
+
 # ---------------------------------------------------------------------------
 # mart_catalog
 # ---------------------------------------------------------------------------

toolkit/registry/builders.py

@@ -184,9 +184,15 @@ def build_clean_catalog(
         # o un tipo invalido, viene risolto al tipo corretto dal vocabolario.
         old = editorial.get(manifest.slug)
         if old:
-            for field in ("name", "description", "source", "source_id", "stage", "period"):
+            for field in ("name", "description", "source", "source_id", "stage"):
                 if old.get(field):
                     entry[field] = old[field]
+            # period: il time_coverage dichiarato nel config ha priorita'
+            # sull'editoriale; l'editoriale copre solo l'assenza di time_coverage
+            # (senza questa eccezione un cambio di copertura nel config non
+            # propagherebbe mai al registry — regression energia-italia #6)
+            if not manifest.period and old.get("period"):
+                entry["period"] = old["period"]
             old_cols = {c.get("name"): c for c in old.get("columns", [])}
             for col in entry["columns"]:
                 oc = old_cols.get(col["name"])