datacharter 0.3.1__tar.gz → 0.3.2__tar.gz

{datacharter-0.3.1 → datacharter-0.3.2}/CHANGELOG.md

@@ -5,6 +5,15 @@ Format: [Keep a Changelog](https://keepachangelog.com/en/1.1.0/).
 
 ## [Unreleased]
 
+## [0.3.2] - 2026-07-24
+
+### Fixed
+
+- `datacharter init` then `datacharter serve` failed with a `CharterError`: the
+  scaffolded charter uses an empty `sources: {}`, which the loader wrongly
+  rejected as "must be a non-empty mapping". A fresh workspace is now servable —
+  add sources afterward via `charter.yaml` or the in-app source manager.
+
 ## [0.3.1] - 2026-07-24
 
 ### Changed

{datacharter-0.3.1 → datacharter-0.3.2}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: datacharter
-Version: 0.3.1
+Version: 0.3.2
 Summary: Charter your data — contract-governed local data exploration, powered by DuckDB
 Project-URL: Homepage, https://github.com/datacharter/datacharter
 Project-URL: Repository, https://github.com/datacharter/datacharter

{datacharter-0.3.1 → datacharter-0.3.2}/src/datacharter/__init__.py

@@ -1,3 +1,3 @@
 """DataCharter — contract-governed local data exploration."""
 
-__version__ = "0.3.1"
+__version__ = "0.3.2"

{datacharter-0.3.1 → datacharter-0.3.2}/src/datacharter/contracts/loader.py

@@ -52,8 +52,10 @@ def load_charter(workspace: Path | str, filename: str = CHARTER_FILE) -> Charter
             f"{filename}: unsupported version {version!r}; expected {SUPPORTED_VERSION}."
         )
     sources_raw = raw.get("sources")
-    if not isinstance(sources_raw, dict) or not sources_raw:
-        raise CharterError(f"{filename}: 'sources' must be a non-empty mapping of name -> source.")
+    if sources_raw is None:  # a fresh `datacharter init` workspace: add sources later
+        sources_raw = {}
+    if not isinstance(sources_raw, dict):
+        raise CharterError(f"{filename}: 'sources' must be a mapping of name -> source.")
 
     resolver = SecretResolver(workspace)
     warnings: list[str] = []

{datacharter-0.3.1 → datacharter-0.3.2}/tests/contracts/test_loader.py

@@ -16,6 +16,24 @@ def test_minimal_file_source_loads(tmp_path):
     assert charter.warnings == []
 
 
+def test_empty_sources_loads_as_fresh_workspace(tmp_path):
+    # `datacharter init` scaffolds `sources: {}`; a fresh workspace must be servable
+    # (sources are added later, via charter.yaml or the in-app source manager).
+    charter = load_charter(write_charter(tmp_path, "version: 1\nsources: {}\n"))
+    assert charter.sources == []
+
+
+def test_null_sources_loads_as_empty(tmp_path):
+    charter = load_charter(write_charter(tmp_path, "version: 1\nsources:\n"))
+    assert charter.sources == []
+
+
+def test_non_mapping_sources_rejected(tmp_path):
+    ws = write_charter(tmp_path, "version: 1\nsources:\n  - a\n  - b\n")
+    with pytest.raises(CharterError, match=r"'sources' must be a mapping"):
+        load_charter(ws)
+
+
 def test_env_reference_resolution(tmp_path, monkeypatch):
     monkeypatch.setenv("PG_PW", "resolved-pw-value")
     ws = write_charter(

{datacharter-0.3.1 → datacharter-0.3.2}/tests/test_cli_init.py

@@ -44,6 +44,29 @@ def test_demo_workspace_loads_and_queries_end_to_end(tmp_path):
         assert len(result.rows) == 2
 
 
+def test_plain_init_charter_loads_as_empty(tmp_path):
+    # Regression: plain `init` scaffolds `sources: {}`; the scaffolded charter MUST
+    # load. 0.3.1 shipped an init template that `load_charter` then refused.
+    assert main(["init", str(tmp_path)]) == 0
+    charter = load_charter(tmp_path)
+    assert charter.sources == []
+
+
+def test_plain_init_workspace_serves_end_to_end(tmp_path):
+    # The real user path `init` -> `serve`: create_app must boot on a fresh empty
+    # workspace and the sources API returns an empty list (sources added later).
+    from fastapi.testclient import TestClient
+
+    from datacharter.server import create_app
+
+    assert main(["init", str(tmp_path)]) == 0
+    app = create_app(tmp_path)
+    with TestClient(app, base_url="http://127.0.0.1") as c:
+        resp = c.get("/api/sources")
+        assert resp.status_code == 200
+        assert resp.json()["sources"] == []
+
+
 def test_serve_workspace_resolution_prefers_existing_charter(tmp_path):
     from datacharter.cli import _resolve_serve_workspace, main