openom-cli 0.1.2__tar.gz → 0.1.3__tar.gz

{openom_cli-0.1.2 → openom_cli-0.1.3}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.5
 Name: openom-cli
-Version: 0.1.2
+Version: 0.1.3
 Summary: openOM CLI - the `om` command over openom-core. Zero inference.
 Project-URL: Homepage, https://openom.app
 Project-URL: Documentation, https://openom.app/docs/

{openom_cli-0.1.2 → openom_cli-0.1.3}/pyproject.toml

@@ -4,7 +4,7 @@ build-backend = "hatchling.build"
 
 [project]
 name = "openom-cli"
-version = "0.1.2"
+version = "0.1.3"
 description = "openOM CLI - the `om` command over openom-core. Zero inference."
 readme = "README.md"
 requires-python = ">=3.11"

{openom_cli-0.1.2 → openom_cli-0.1.3}/src/openom_cli/main.py

@@ -197,8 +197,17 @@ def _load_json(path: Path) -> dict[str, Any]:
     # parse_hardened enforces the same §J read invariants the embed/MCP paths do - reject duplicate
     # keys and over-deep nesting - degrading a pathological payload to a structured OM-IO error
     # (caught by _guard -> exit 3) instead of a RecursionError traceback.
-    text = sys.stdin.read() if str(path) == "-" else path.read_text(encoding="utf-8")
-    return cast("dict[str, Any]", _parse_hardened(text))
+    raw = sys.stdin.buffer.read() if str(path) == "-" else path.read_bytes()
+    if raw[:5].startswith(b"%PDF"):
+        # The natural first mistake: pointing `validate` (or another JSON verb) at an OM PDF. Give a
+        # plain hint toward the right verb instead of a cryptic UnicodeDecodeError on binary bytes.
+        typer.echo(
+            f"error: {path} looks like a PDF, not a JSON payload. To read the data embedded in an "
+            f"OM PDF use `om read {path}`; `validate` checks a deal.json (see `om init`).",
+            err=True,
+        )
+        raise typer.Exit(3)
+    return cast("dict[str, Any]", _parse_hardened(raw.decode("utf-8")))
 
 
 def _emit(obj: Any) -> None:

{openom_cli-0.1.2 → openom_cli-0.1.3}/tests/test_cli.py

@@ -486,6 +486,15 @@ def test_validate_deeply_nested_json_degrades_cleanly(tmp_path: Path) -> None:
     assert "OM-IO-STRUCTURE" in r.output
 
 
+def test_validate_on_a_pdf_gives_a_useful_hint(tmp_path: Path) -> None:
+    # The natural first mistake - validate a PDF - must point at `om read`, not die on a raw
+    # UnicodeDecodeError decoding binary bytes.
+    pdf = _base_pdf(tmp_path / "deal.pdf")
+    r = runner.invoke(app, ["validate", str(pdf)])
+    assert r.exit_code == 3
+    assert "looks like a PDF" in r.output and "om read" in r.output
+
+
 def test_check_payload_json(tmp_path: Path) -> None:
     # Consistency tier only (no schema): the valid sample is internally consistent -> exit 0.
     r = runner.invoke(app, ["check", str(SPEC / "samples" / "valid-stnl.json")])