pythia-plsql 0.4.0__tar.gz → 0.4.2__tar.gz

{pythia_plsql-0.4.0/scripts/pythia_plsql.egg-info → pythia_plsql-0.4.2}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: pythia-plsql
-Version: 0.4.0
+Version: 0.4.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.4.0 → pythia_plsql-0.4.2}/pyproject.toml

@@ -4,7 +4,7 @@ build-backend = "setuptools.build_meta"
 
 [project]
 name = "pythia-plsql"
-version = "0.4.0"
+version = "0.4.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.4.0 → pythia_plsql-0.4.2}/scripts/pythia.py

@@ -1337,6 +1337,11 @@ def run_apply(conn, schema, ns, file_text, origin=None):
                 print(f"  {paint_diff_line(ln, en)}")
         print(paint(f"\n  Snapshot saved: {journal_root(ns.project_root) / entry}",
                     "dim", en))
+        if not confirmed:
+            print(paint("  Rollback file for the current database version — "
+                        "use it if this is\n  run by hand instead: "
+                        f"{journal_root(ns.project_root) / entry / 'restore.sql'}",
+                        "dim", en))
         if not confirmed:
             print(f"\n  To apply:\n    "
                   + paint(f"{invocation()} apply {ns.file} --confirm {token}",
@@ -1536,18 +1541,29 @@ def cmd_journal(conn, schema, ns):
     if ns.action == "prune":
         import shutil
         removed = 0
-        for eid in list_journal_entries(root):
-            meta = read_journal_entry(root, eid)["meta"]
+        # A preview wrote nothing to the database, but its restore.sql holds
+        # the version that was live at the time — the only rollback there is
+        # for a change the developer then ran by hand. So drop a preview only
+        # when its rollback is byte-identical to one a newer entry already
+        # keeps: redundant captures go, unique ones never do.
+        kept = {}
+        for eid in list_journal_entries(root):        # newest first
+            e = read_journal_entry(root, eid)
+            meta, obj = e["meta"], str(e["meta"].get("object", ""))
             if meta.get("applied") or meta.get("snapshot"):
-                continue      # applied = a real write; snapshot = the only
-                              # undo a hand-edit outside pythia will ever have
-            shutil.rmtree(journal_root(root) / eid)
-            removed += 1
+                kept.setdefault(obj, set()).add(e["restore"])
+                continue
+            if e["restore"] in kept.get(obj, set()):
+                shutil.rmtree(journal_root(root) / eid)
+                removed += 1
+            else:
+                kept.setdefault(obj, set()).add(e["restore"])
         if ns.json:
             print(json.dumps({"pruned": removed}))
         else:
-            print(f"-- pruned {removed} preview-only entries; applied "
-                  "entries and snapshots are all kept")
+            print(f"-- pruned {removed} redundant previews; applied entries, "
+                  "snapshots, and any preview holding a rollback nothing "
+                  "else has are all kept")
         return
     if not ns.id:
         sys.exit(f"Usage: {invocation()} journal "

{pythia_plsql-0.4.0 → pythia_plsql-0.4.2/scripts/pythia_plsql.egg-info}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: pythia-plsql
-Version: 0.4.0
+Version: 0.4.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.4.0 → pythia_plsql-0.4.2}/skills/pythia-apply/SKILL.md

@@ -31,6 +31,15 @@ Three moves are FORBIDDEN for agents, and the CLI enforces the first two
   your turn; the token is only used after the developer's approval
   arrives as a NEW message.
 
+## When the change will NOT go through apply
+
+Sometimes the developer runs the file themselves — a DBA executes it, a
+release process owns it, or policy denies the group. The preview still ran,
+so a rollback file for the version currently live already exists at
+`.pythia/journal/<entry>/restore.sql`. Never let a change leave your hands
+for manual execution without naming that path in the same message as the
+.sql file. `pythia history <OBJECT>` lists every captured version.
+
 ## The Workflow
 
 **Before step 1:** if this conversation has not yet seen a standalone impact
@@ -54,7 +63,8 @@ first time anyone sees it, a step was skipped.
    If the fresh preview's before-side no longer matches what you last saw,
    say so explicitly: someone else may have changed the object on this
    shared database, and the developer must know that before approving.
-5. **Read the exit code — it is the verdict:**
+5. **Read the exit code — it is the verdict. Never through a pipe:**
+   `apply … | tail` returns *tail's* code, always 0. Unpiped, or `${PIPESTATUS[0]}`.
 
    | Exit | Meaning | What you must do |
    |------|---------|-----------------|
@@ -120,6 +130,7 @@ the stakes are highest.
 | "Token is stale, I'll just take the new one" | The content changed. The developer must see the new preview. |
 | "apply refused it; run-sql will take it" | The refusal is the product working. Relay it. |
 | "I'll restore quietly to clean up my mistake" | Restores are writes. Same gate, same visibility. |
+| "`$?` said 0 after I piped to tail" | That was tail's 0. Read pythia's own words, or its unpiped code. |
 | "The dev said 'stop asking' once" | That covered that task, not forever. Re-confirm scope on the next one. |
 
 ## When NOT to use this skill

{pythia_plsql-0.4.0 → pythia_plsql-0.4.2}/tests/test_phase3.py

@@ -540,25 +540,42 @@ def test_privilege_warning_speaks_proxy():
         FakeConn(direct), "APP", "APP")
 
 
-def test_journal_prune_keeps_applied_entries():
+def test_prune_drops_only_redundant_previews():
+    """A preview wrote nothing, but its restore.sql is the live version at
+    that moment — the only rollback for a change run by hand. Prune may drop
+    it only when a newer entry already keeps the identical rollback."""
     import argparse
+    import contextlib
     import datetime
+    import io
     with tempfile.TemporaryDirectory() as td:
-        base = datetime.datetime(2026, 8, 26, 12, 0, 0)
-        pythia.write_journal_entry(td, "PROCEDURE", "P_PREVIEW", "a", "b",
+        base = datetime.datetime(2026, 8, 27, 9, 0, 0)
+        # two previews of the same object, same live version -> one is spare
+        pythia.write_journal_entry(td, "PROCEDURE", "P_A", "old", "new1",
                                    {"applied": False}, now=base)
-        kept = pythia.write_journal_entry(
-            td, "PROCEDURE", "P_APPLIED", "a", "b", {"applied": True},
+        keep_dup = pythia.write_journal_entry(
+            td, "PROCEDURE", "P_A", "old", "new2", {"applied": False},
             now=base.replace(minute=1))
+        # a preview of another object, unique rollback -> must survive
+        lone = pythia.write_journal_entry(
+            td, "PROCEDURE", "P_B", "only_copy", "x", {"applied": False},
+            now=base.replace(minute=2))
+        applied = pythia.write_journal_entry(
+            td, "PROCEDURE", "P_C", "a", "b", {"applied": True},
+            now=base.replace(minute=3))
+        snap = pythia.write_journal_entry(
+            td, "PROCEDURE", "P_D", "a", "a",
+            {"snapshot": True, "applied": False}, now=base.replace(minute=4))
         ns = argparse.Namespace(action="prune", project_root=td, json=False,
                                 id=None)
-        import contextlib
-        import io
         buf = io.StringIO()
         with contextlib.redirect_stdout(buf):
             pythia.cmd_journal(None, None, ns)
-        assert "pruned 1" in buf.getvalue()
-        assert pythia.list_journal_entries(td) == [kept]
+        left = pythia.list_journal_entries(td)
+        assert "pruned 1" in buf.getvalue(), buf.getvalue()
+        assert applied in left and snap in left      # never touched
+        assert lone in left                          # unique rollback survives
+        assert keep_dup in left                      # newest of the pair kept
 
 
 def test_agent_user_alter_form_and_warning_prediction():
@@ -742,26 +759,6 @@ def test_auto_snapshot_reports_drift_and_keeps_the_old_rollback():
         assert len(pythia.list_journal_entries(td)) == 2
 
 
-def test_prune_never_eats_a_snapshot():
-    import argparse
-    import contextlib
-    import io
-    import datetime
-    with tempfile.TemporaryDirectory() as td:
-        base = datetime.datetime(2026, 8, 27, 9, 0, 0)
-        pythia.write_journal_entry(td, "PROCEDURE", "P_A", "a", "b",
-                                   {"applied": False}, now=base)
-        snap = pythia.write_journal_entry(
-            td, "PROCEDURE", "P_B", "a", "a", {"snapshot": True,
-                                               "applied": False},
-            now=base.replace(minute=1))
-        ns = argparse.Namespace(action="prune", project_root=td, json=False,
-                                id=None)
-        with contextlib.redirect_stdout(io.StringIO()):
-            pythia.cmd_journal(None, None, ns)
-        assert pythia.list_journal_entries(td) == [snap]
-
-
 def main():
     failed = 0
     for name, fn in sorted(globals().items()):