pythia-plsql 0.11.1__tar.gz → 0.11.2__tar.gz

{pythia_plsql-0.11.1/scripts/pythia_plsql.egg-info → pythia_plsql-0.11.2}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: pythia-plsql
-Version: 0.11.1
+Version: 0.11.2
 Summary: PL/SQL development for AI agents on Oracle Database - expert data-dictionary queries, impact analysis, and a snapshot-verified write path with honest rollback.
 License: MIT
 Project-URL: Homepage, https://github.com/thaildhe172591/pythia

{pythia_plsql-0.11.1 → pythia_plsql-0.11.2}/pyproject.toml

@@ -4,7 +4,7 @@ build-backend = "setuptools.build_meta"
 
 [project]
 name = "pythia-plsql"
-version = "0.11.1"
+version = "0.11.2"
 description = "PL/SQL development for AI agents on Oracle Database - expert data-dictionary queries, impact analysis, and a snapshot-verified write path with honest rollback."
 readme = "README.md"
 license = { text = "MIT" }

{pythia_plsql-0.11.1 → pythia_plsql-0.11.2}/scripts/pythia.py

@@ -2331,10 +2331,17 @@ def token_mismatch_message(ns, row_set):
     base = ("The confirmation token does not match: the file or the "
             "database object changed since that preview. Preview again:\n"
             f"  {invocation()} apply {ns.file}")
-    if not row_set:
-        return base
     _, meta = find_preview_by_token(ns.project_root,
                                     str(ns.confirm).strip().lower())
+    if (meta or {}).get("applied"):
+        # the token names a preview that already ran. "the file changed" is
+        # the wrong story, and sends the agent hunting a difference that is
+        # not there
+        return (f"Token {str(ns.confirm).strip().lower()} was already applied "
+                "— one approval, one write.\nPreview again, then get the new "
+                f"token approved:\n  {invocation()} apply {ns.file}")
+    if not row_set:
+        return base
     then = (meta or {}).get("row_set") or {}
     if not then or fingerprint_text(then) == fingerprint_text(row_set):
         return base                       # the file moved, not the rows
@@ -3467,10 +3474,15 @@ def build_parser():
 def main(argv=None):
     argv = sys.argv[1:] if argv is None else argv
     forbid_write_flag(argv)
-    for stream in (sys.stdout, sys.stderr):
+    # stdin too, and before anything reads it: the AskUserQuestion hook
+    # payload is UTF-8, and Windows would decode it as cp1252 — an em dash
+    # arrives as mojibake and no approval card ever matches again.
+    # line_buffering keeps stdout in order with stderr when both are pipes,
+    # so a driver error lands after the preview it belongs to, not above it.
+    for stream in (sys.stdout, sys.stderr, sys.stdin):
         try:
-            stream.reconfigure(encoding="utf-8")
-        except AttributeError:
+            stream.reconfigure(encoding="utf-8", line_buffering=True)
+        except (AttributeError, ValueError):
             pass
     ns = build_parser().parse_args(argv)
     # refusals that need no database come first — an unreachable or locked

{pythia_plsql-0.11.1 → pythia_plsql-0.11.2/scripts/pythia_plsql.egg-info}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: pythia-plsql
-Version: 0.11.1
+Version: 0.11.2
 Summary: PL/SQL development for AI agents on Oracle Database - expert data-dictionary queries, impact analysis, and a snapshot-verified write path with honest rollback.
 License: MIT
 Project-URL: Homepage, https://github.com/thaildhe172591/pythia

{pythia_plsql-0.11.1 → pythia_plsql-0.11.2}/tests/test_phase3.py

@@ -1565,5 +1565,49 @@ def main():
     print("OK")
 
 
+
+
+def test_hook_matches_a_card_with_a_non_ascii_dash_through_real_stdin():
+    """The whole hook, end to end, the way Claude Code runs it: UTF-8 bytes on
+    a pipe. Windows decodes stdin as cp1252 unless told otherwise, and the em
+    dash in the structural card would arrive as mojibake — no card would ever
+    match again. Subprocess on purpose: an in-process StringIO is already
+    decoded and cannot fail this way."""
+    import subprocess
+    with tempfile.TemporaryDirectory() as td:
+        stmt = "ALTER PROCEDURE PKG_ORDER COMPILE;"
+        pythia.write_journal_entry(
+            td, "STRUCTURAL", "STATEMENT", "", stmt,
+            {"token": "dc8d27", "connection": "DEV", "schema": "APP",
+             "group": "structural", "applied": False})
+        _, _, body = pythia.approval_card(td, "dc8d27")
+        card = "python -m pythia approve dc8d27\n" + "\n".join(body)
+        assert "—" in card, "the structural card is the non-ASCII one"
+        payload = json.dumps(_hook_payload(card, "Approve"), ensure_ascii=False)
+        p = subprocess.run(
+            [sys.executable,
+             str(pathlib.Path(pythia.__file__).resolve()), "approve", "--hook"],
+            input=payload.encode("utf-8"), cwd=td, capture_output=True)
+        assert p.returncode == 0, p.stderr.decode("utf-8", "replace")
+        g = pythia.read_grant(td, "dc8d27")
+        assert g and g["approver"] == "chat", p.stdout.decode("utf-8", "replace")
+
+
+def test_token_mismatch_names_an_already_applied_preview():
+    """A spent token is not a moved file. Saying 'the file or the object
+    changed' sends the agent looking for a difference that is not there."""
+    import argparse
+    with tempfile.TemporaryDirectory() as td:
+        pythia.write_journal_entry(
+            td, "PACKAGE BODY", "PKG_ORDER", OLD_SRC, NEW_FILE,
+            {"token": "7f3a91", "connection": "DEV", "schema": "APP",
+             "group": "plsql_source", "applied": True})
+        ns = argparse.Namespace(project_root=td, confirm="7f3a91",
+                                file="pkg.sql")
+        msg = pythia.token_mismatch_message(ns, None)
+        assert "already applied" in msg and "changed since that preview" not in msg
+
+
+
 if __name__ == "__main__":
     main()