scriptoria 0.6.1__tar.gz → 0.6.3__tar.gz

{scriptoria-0.6.1 → scriptoria-0.6.3}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: scriptoria
-Version: 0.6.1
+Version: 0.6.3
 Summary: Deterministic scriptorium-keeper (the `scrip` CLI): staleness, provenance integrity, and fact queries for an agent-compiled knowledge base
 Project-URL: Homepage, https://github.com/coredipper/scriptorium
 Project-URL: Changelog, https://github.com/coredipper/scriptorium/blob/main/CHANGELOG.md

{scriptoria-0.6.1 → scriptoria-0.6.3}/pyproject.toml

@@ -2,7 +2,7 @@
 # Distribution name on PyPI is `scriptoria` (scrip/scriptorium were taken); the
 # CLI command and the import package both remain `scrip`.
 name = "scriptoria"
-version = "0.6.1"
+version = "0.6.3"
 description = "Deterministic scriptorium-keeper (the `scrip` CLI): staleness, provenance integrity, and fact queries for an agent-compiled knowledge base"
 readme = "README.md"
 requires-python = ">=3.10"

{scriptoria-0.6.1 → scriptoria-0.6.3}/src/scrip/__init__.py

@@ -13,7 +13,7 @@ from __future__ import annotations
 
 from pathlib import Path
 
-__version__ = "0.6.1"
+__version__ = "0.6.3"
 
 # --- canonical vault layout ------------------------------------------------
 # ``root`` is the repo/instance root: the directory containing ``vault/``.

{scriptoria-0.6.1 → scriptoria-0.6.3}/src/scrip/frontmatter.py

@@ -7,6 +7,7 @@ whole CLI stay "a few hundred LOC" with only duckdb + pyyaml as real deps.
 
 from __future__ import annotations
 
+import io
 from collections.abc import Iterable
 from pathlib import Path
 
@@ -25,23 +26,11 @@ FENCE = "---"
 def parse(text: str) -> tuple[dict, str]:
     """Return ``(meta, body)``. If there is no frontmatter, ``meta`` is ``{}``
     and ``body`` is the whole text."""
-    lines = text.splitlines(keepends=True)
-    if not lines or lines[0].strip() != FENCE:
+    f = io.StringIO(text, newline="")
+    found, meta = _read_frontmatter(f)
+    if not found:
         return {}, text
-    for i in range(1, len(lines)):
-        if lines[i].strip() == FENCE:
-            fm_text = "".join(lines[1:i])
-            body = "".join(lines[i + 1 :])
-            try:
-                meta = yaml.load(fm_text, Loader=SafeLoader)
-            except yaml.YAMLError as e:
-                raise DataError(f"invalid YAML frontmatter: {e}") from e
-            if meta is None:
-                meta = {}
-            if not isinstance(meta, dict):
-                raise DataError("frontmatter must be a YAML mapping")
-            return meta, body
-    raise DataError("unterminated frontmatter (missing closing '---')")
+    return meta, f.read()
 
 
 def _read_frontmatter(f) -> tuple[bool, dict]:

{scriptoria-0.6.1 → scriptoria-0.6.3}/src/scrip/query.py

@@ -55,26 +55,35 @@ _NAMED = {
 _FILTERABLE = {"claims", "entities", "edges", "reconciliations"}
 
 
+def _create_empty_reconciliations_view(con: duckdb.DuckDBPyConnection) -> None:
+    # Always present (empty stub) so `contradictions` can anti-join it and raw SQL
+    # over its columns works even before any reconciliation exists.
+    con.execute(
+        "CREATE VIEW reconciliations AS SELECT "
+        "NULL::VARCHAR AS reconciliation_id, NULL::VARCHAR AS decision, "
+        "NULL::VARCHAR AS claim_a, NULL::VARCHAR AS claim_b, "
+        "NULL::VARCHAR AS winner, NULL::VARCHAR AS rationale, "
+        "NULL::VARCHAR AS at WHERE FALSE"
+    )
+
+
+def _has_rows(path: Path) -> bool:
+    with path.open(encoding="utf-8") as f:
+        return any(line.strip() for line in f)
+
+
 def _connect(root: Path) -> duckdb.DuckDBPyConnection:
     con = duckdb.connect(":memory:")
     fd = facts_dir(root)
     for view, fname in _VIEWS.items():
         p = fd / fname
-        if p.exists():
+        if p.exists() and (view != "reconciliations" or _has_rows(p)):
             con.execute(
                 f"CREATE VIEW {view} AS "
                 f"SELECT * FROM read_ndjson_auto('{p.as_posix()}')"
             )
         elif view == "reconciliations":
-            # Always present (empty stub) so `contradictions` can anti-join it and
-            # raw SQL over its columns works even before any reconciliation exists.
-            con.execute(
-                "CREATE VIEW reconciliations AS SELECT "
-                "NULL::VARCHAR AS reconciliation_id, NULL::VARCHAR AS decision, "
-                "NULL::VARCHAR AS claim_a, NULL::VARCHAR AS claim_b, "
-                "NULL::VARCHAR AS winner, NULL::VARCHAR AS rationale, "
-                "NULL::VARCHAR AS at WHERE FALSE"
-            )
+            _create_empty_reconciliations_view(con)
     return con
 
 

{scriptoria-0.6.1 → scriptoria-0.6.3}/tests/test_query.py

@@ -74,6 +74,16 @@ def test_contradictions_works_without_reconciliations_file(kb):
     assert len(query.run(kb.root, name="contradictions")[1]) == 1
 
 
+def test_contradictions_works_with_empty_reconciliations_file(kb):
+    # an existing-but-empty file should use the same stub schema as a missing file
+    _contradiction_pair(kb)
+    (kb.root / "vault" / "facts" / "reconciliations.ndjson").write_text(
+        "\n", encoding="utf-8"
+    )
+
+    assert len(query.run(kb.root, name="contradictions")[1]) == 1
+
+
 def test_query_reconciliations_cli_choice(kb):
     # the named query must be a valid CLI `query` choice, not just in query.run
     _contradiction_pair(kb)
@@ -88,6 +98,20 @@ def test_reconciliations_stub_exposes_full_schema(kb):
     assert "decision" in cols
 
 
+def test_empty_reconciliations_file_exposes_full_schema(kb):
+    _contradiction_pair(kb)
+    (kb.root / "vault" / "facts" / "reconciliations.ndjson").write_text(
+        "\n", encoding="utf-8"
+    )
+
+    cols, rows = query.run(
+        kb.root,
+        sql="SELECT decision, winner, reconciliation_id FROM reconciliations",
+    )
+    assert rows == []
+    assert "decision" in cols
+
+
 def test_where_and_limit(kb):
     kb.add_raw("a", "# A\n\nThe sky is blue.\n")
     kb.add_claim("clm_1", "a", "the sky is blue")

{scriptoria-0.6.1 → scriptoria-0.6.3}/uv.lock

@@ -1141,7 +1141,7 @@ wheels = [
 
 [[package]]
 name = "scriptoria"
-version = "0.6.1"
+version = "0.6.2"
 source = { editable = "." }
 dependencies = [
     { name = "duckdb" },