pythia-plsql 0.12.0__tar.gz → 0.13.0__tar.gz

{pythia_plsql-0.12.0/scripts/pythia_plsql.egg-info → pythia_plsql-0.13.0}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: pythia-plsql
-Version: 0.12.0
+Version: 0.13.0
 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
@@ -128,7 +128,7 @@ Or the same thing piecewise:
 
 ```bash
 pip install pythia-plsql   # the CLI (thin driver — no Oracle Instant Client needed)
-python -m pythia install   # skills into your agent + .pythia/ scaffold + Claude Code hooks
+python -m pythia install   # skills + .pythia/ scaffold + Claude Code hooks + Codex AGENTS.md
 pythia check               # fill in connections.json first, then verify
 ```
 
@@ -212,6 +212,10 @@ stops it prompting for the read-only commands and asks it to pause on
 writes — optional, and yours to install
 ([why pythia does not](GUIDE.md#11-optional-claude-code-permission-settings)).
 
+On Codex, the same command writes the harness into `AGENTS.md` (loaded every
+session) and a session-start hook into `.codex/hooks.json`; approve previews in
+the terminal with `pythia approve <token>`, since Codex has no chat-mint.
+
 Per-group write policy, `.pythia/policy.json` (defaults shown):
 
 | Group | Default | Is rollback real? |

{pythia_plsql-0.12.0 → pythia_plsql-0.13.0}/README.md

@@ -114,7 +114,7 @@ Or the same thing piecewise:
 
 ```bash
 pip install pythia-plsql   # the CLI (thin driver — no Oracle Instant Client needed)
-python -m pythia install   # skills into your agent + .pythia/ scaffold + Claude Code hooks
+python -m pythia install   # skills + .pythia/ scaffold + Claude Code hooks + Codex AGENTS.md
 pythia check               # fill in connections.json first, then verify
 ```
 
@@ -198,6 +198,10 @@ stops it prompting for the read-only commands and asks it to pause on
 writes — optional, and yours to install
 ([why pythia does not](GUIDE.md#11-optional-claude-code-permission-settings)).
 
+On Codex, the same command writes the harness into `AGENTS.md` (loaded every
+session) and a session-start hook into `.codex/hooks.json`; approve previews in
+the terminal with `pythia approve <token>`, since Codex has no chat-mint.
+
 Per-group write policy, `.pythia/policy.json` (defaults shown):
 
 | Group | Default | Is rollback real? |

{pythia_plsql-0.12.0 → pythia_plsql-0.13.0}/pyproject.toml

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

@@ -2818,6 +2818,51 @@ DO     writes go through `pythia apply` only - snapshot, token, verify. The
 """
 
 
+AGENTS_BEGIN = "<!-- pythia:begin (managed by `pythia install`; edit outside this block) -->"
+AGENTS_END = "<!-- pythia:end -->"
+
+
+def agents_md_block():
+    """The pythia section for AGENTS.md — Codex's always-loaded instruction
+    file. Body is BRIEF_GUIDE verbatim (the same constant `guide --brief`
+    prints and the Claude SessionStart hook runs, so the two cannot drift),
+    plus the one line Codex needs that Claude gets from its hook: where the
+    approval door is, since Codex has no chat-mint. Marker-wrapped so a re-run
+    replaces it in place and the developer's own text is untouched."""
+    return (f"{AGENTS_BEGIN}\n"
+            "## pythia — developing PL/SQL on Oracle\n\n"
+            f"{BRIEF_GUIDE.rstrip()}\n\n"
+            "On Codex there is no chat approval hook: the developer approves a "
+            "preview in this terminal with `pythia approve <token>`, then the "
+            "agent runs `pythia apply <file> --confirm <token>`.\n"
+            f"{AGENTS_END}")
+
+
+def merge_agents_md(path):
+    """Write the pythia block into an AGENTS.md. The file is the developer's,
+    shared with every AGENTS.md-reading agent, so only the delimited block is
+    ours: present -> replaced in place, absent -> appended below their text,
+    everything else byte-identical. Returns (path, action) where action is
+    'created' | 'updated' | 'unchanged'."""
+    path = pathlib.Path(path)
+    block = agents_md_block()
+    old = path.read_text(encoding="utf-8") if path.is_file() else None
+    if old is None:
+        new, action = block + "\n", "created"
+    elif AGENTS_BEGIN in old and AGENTS_END in old:
+        pre = old[:old.index(AGENTS_BEGIN)]
+        post = old[old.index(AGENTS_END) + len(AGENTS_END):]
+        new = pre + block + post
+        action = "unchanged" if new == old else "updated"
+    else:
+        tail = "" if old.endswith("\n\n") else "\n" if old.endswith("\n") else "\n\n"
+        new, action = old + tail + block + "\n", "updated"
+    if new != old:
+        path.parent.mkdir(parents=True, exist_ok=True)
+        path.write_text(new, encoding="utf-8")
+    return path, action
+
+
 OPERATING_GUIDE = """\
 THE OPERATING MODEL — Learn, Ask, Do (Hoc - Hoi - Lam)
 
@@ -3236,29 +3281,36 @@ def global_pack_present(home=None):
 
 
 def copy_bundled_skills(base_dir):
-    """No-Node fallback: copy the wheel-bundled pack into
-    <base>/.claude/skills/ ONLY — the directory Claude Code reliably reads
-    in both scopes (field evidence: a project's .agents/skills is invisible
-    to some Claude Code versions, and a second copy doubles every menu
-    entry). base_dir is the project root, or the home directory for -g.
+    """No-Node fallback: copy the wheel-bundled pack into BOTH conventional
+    roots — <base>/.claude/skills/ (Claude Code) and <base>/.agents/skills/
+    (Codex, and ~/.agents/skills on -g). base_dir is the project root, or the
+    home directory for -g. Node users get the same reach from `npx skills
+    add`; this is the machine that has only Python.
+
+    Trade-off, stated plainly: this reverses an earlier one-root-only choice.
+    A Claude Code version that reads both roots may list a pack entry twice —
+    accepted, because a Codex user without Node otherwise gets no skills at
+    all, and Codex reads only .agents/skills.
 
     Both destinations are cleared link-first before the copy. An earlier
     `npx skills add` leaves .claude/skills/<name> as a symlink into
-    .agents/skills/<name>; copying into that symlink would write through to
-    its target, which the .agents cleanup then deletes — leaving a dangling
-    link and no pack at all. Only the pack's own names are touched; other
-    skills in those directories are left alone."""
+    .agents/skills/<name>; removing both links first means each copytree
+    writes a fresh real directory, never through a link into a target the
+    other copy would then overwrite. Only the pack's own names are touched;
+    other skills in those directories are left alone."""
     import shutil
     base = pathlib.Path(base_dir)
-    dest_root = base / ".claude" / "skills"
+    claude_root = base / ".claude" / "skills"      # Claude Code reads this
+    agents_root = base / ".agents" / "skills"      # Codex reads this
     for pack in sorted(SKILLS_DIR.iterdir()):
         if not (pack / "SKILL.md").is_file():
             continue
-        _remove_link_first(dest_root / pack.name)
-        _remove_link_first(base / ".agents" / "skills" / pack.name)
-        shutil.copytree(pack, dest_root / pack.name)
+        _remove_link_first(claude_root / pack.name)
+        _remove_link_first(agents_root / pack.name)
+        shutil.copytree(pack, claude_root / pack.name)
+        shutil.copytree(pack, agents_root / pack.name)
     clean_legacy_skills(base_dir)
-    return [dest_root]
+    return [claude_root, agents_root]
 
 
 def _remove_link_first(path):
@@ -3302,6 +3354,43 @@ CLAUDE_DENY = ["Bash(pythia approve --hook*)",
 # is recognised by what it runs, not by how it invokes it
 HOOK_MARKS = {"SessionStart": "guide --brief", "PostToolUse": "approve --hook"}
 
+# Codex loads hooks from <repo>/.codex/hooks.json once the layer is trusted
+# (/hooks). Only the session-start guide is wired — Codex has no answer-reading
+# PostToolUse and no question tool, so the chat approve-mint cannot port; the
+# developer approves in the terminal. Same shape and merge discipline as the
+# Claude hooks.
+CODEX_HOOKS = {
+    "SessionStart": [{"hooks": [
+        {"type": "command", "command": "python -m pythia guide --brief"}]}],
+}
+CODEX_HOOK_MARKS = {"SessionStart": "guide --brief"}
+
+
+def merge_codex_hooks(path, events=None):
+    """Merge pythia's Codex hooks into a hooks.json ({"hooks": {...}}). Every
+    other key survives, a hook already present in any spelling is recognised by
+    what it runs, and a file that will not parse is left byte-identical.
+    Returns (path, added|None)."""
+    path = pathlib.Path(path)
+    try:
+        s = json.loads(path.read_text(encoding="utf-8")) if path.is_file() else {}
+        hooks = s.setdefault("hooks", {})
+        added = []
+        for event in events or CODEX_HOOKS:
+            present = hooks.setdefault(event, [])
+            cmds = " ".join(h.get("command", "")
+                            for e in present for h in e.get("hooks", []))
+            if CODEX_HOOK_MARKS[event] not in cmds:
+                present.extend(json.loads(json.dumps(CODEX_HOOKS[event])))
+                added.append(f"hooks.{event}")
+    except (ValueError, TypeError, AttributeError):
+        return path, None
+    if added:
+        path.parent.mkdir(parents=True, exist_ok=True)
+        path.write_text(json.dumps(s, indent=2, ensure_ascii=False) + "\n",
+                        encoding="utf-8")
+    return path, added
+
 
 def install_claude_hooks(base_dir, events=None):
     """Merge the two hooks and the deny rule into <base>/.claude/settings.json
@@ -3357,6 +3446,33 @@ def report_claude_hooks(base_dir, ns, events=None):
         print(f"\nClaude Code hooks already in {path}.")
 
 
+def report_codex_agents_md(project_root):
+    """Project scope: write the harness into <project>/AGENTS.md, the file
+    Codex loads every session. Not done on -g — a global AGENTS.md is noise in
+    every non-Oracle session, as the Claude guide is per-project."""
+    path, action = merge_agents_md(pathlib.Path(project_root) / "AGENTS.md")
+    verb = {"created": "Wrote the pythia harness into",
+            "updated": "Updated the pythia block in",
+            "unchanged": "pythia harness already current in"}[action]
+    print(f"\n{verb} {path} (Codex and other AGENTS.md agents load it each session).")
+
+
+def report_codex_hooks(project_root, ns):
+    """Project scope: the Codex session-start guide hook. --no-hooks skips it,
+    as with the Claude hooks. AGENTS.md already carries the guide, so this is
+    reinforcement for hook-driven Codex users."""
+    if getattr(ns, "no_hooks", False):
+        return
+    path, added = merge_codex_hooks(pathlib.Path(project_root) / ".codex" / "hooks.json")
+    if added is None:
+        print(f"\n! {path} is not valid JSON, so it was left untouched.")
+    elif added:
+        print(f"\nWired the Codex session-start hook into {path}.")
+        print("In Codex, run /hooks once to trust this project's .codex layer.")
+    else:
+        print(f"\nCodex session-start hook already in {path}.")
+
+
 def cmd_install(conn, schema, ns):
     import shutil
     en = getattr(ns, "color", False)
@@ -3398,6 +3514,8 @@ def cmd_install(conn, schema, ns):
         else:
             clean_legacy_skills(ns.project_root)
     report_claude_hooks(ns.project_root, ns)
+    report_codex_agents_md(ns.project_root)
+    report_codex_hooks(ns.project_root, ns)
     print(f"\nNext: fill in {path}")
     print(f"Then: {invocation()} check")
     scripts_dir = installed_scripts_dir()

{pythia_plsql-0.12.0 → pythia_plsql-0.13.0/scripts/pythia_plsql.egg-info}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: pythia-plsql
-Version: 0.12.0
+Version: 0.13.0
 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
@@ -128,7 +128,7 @@ Or the same thing piecewise:
 
 ```bash
 pip install pythia-plsql   # the CLI (thin driver — no Oracle Instant Client needed)
-python -m pythia install   # skills into your agent + .pythia/ scaffold + Claude Code hooks
+python -m pythia install   # skills + .pythia/ scaffold + Claude Code hooks + Codex AGENTS.md
 pythia check               # fill in connections.json first, then verify
 ```
 
@@ -212,6 +212,10 @@ stops it prompting for the read-only commands and asks it to pause on
 writes — optional, and yours to install
 ([why pythia does not](GUIDE.md#11-optional-claude-code-permission-settings)).
 
+On Codex, the same command writes the harness into `AGENTS.md` (loaded every
+session) and a session-start hook into `.codex/hooks.json`; approve previews in
+the terminal with `pythia approve <token>`, since Codex has no chat-mint.
+
 Per-group write policy, `.pythia/policy.json` (defaults shown):
 
 | Group | Default | Is rollback real? |

{pythia_plsql-0.12.0 → pythia_plsql-0.13.0}/skills/pythia-apply/SKILL.md

@@ -60,7 +60,7 @@ confirms a number you already knew.
    in the option descriptions; the hook does not read those. Header `pythia`;
    options exactly `Approve` / `Reject`, no "(Recommended)"; up to four tokens
    per call. Anything but `Approve` is not approval: stop and ask what should
-   change. File changed → step 1. No hook? Relay `pythia approve <token>`.
+   change. File changed → step 1. No hook (Codex, or none)? Relay `pythia approve <token>` in the terminal — Codex has no chat-mint.
 4. **Apply** by running the exact `then the agent:` line pythia printed. Two
    refusals are normal here, and neither is a malfunction:
    - *"no developer approval is on file"* — the answer was not `Approve`, or

{pythia_plsql-0.12.0 → pythia_plsql-0.13.0}/tests/test_install.py

@@ -57,16 +57,27 @@ def test_missing_npx_falls_back_to_the_bundled_pack():
         (agents / "pythia-apply").mkdir()      # stale copy from 0.2.0-0.2.3
         (claude / "plsql-review").mkdir(parents=True)
         targets = pythia.copy_bundled_skills(td)
-        assert targets == [claude]
+        assert targets == [claude, agents]
         assert (claude / "pythia-apply" / "SKILL.md").is_file()
         assert (claude / "pythia-review" / "reference" / "antipatterns.md").is_file()
-        assert not (agents / "pythia-apply").exists()       # single copy
+        assert (agents / "pythia-apply" / "SKILL.md").is_file()   # Codex reads this
         assert (agents / "my-team-skill" / "SKILL.md").read_text(
             encoding="utf-8") == "x"                         # merge, not wipe
         assert not (claude / "plsql-review").exists()        # legacy cleaned
         pythia.copy_bundled_skills(td)                       # idempotent
 
 
+def test_fallback_serves_both_claude_and_codex():
+    """Claude reads .claude/skills; Codex reads .agents/skills. The no-Node
+    fallback must populate both, or a Codex user without Node gets nothing."""
+    with tempfile.TemporaryDirectory() as td:
+        base = pathlib.Path(td)
+        targets = pythia.copy_bundled_skills(td)
+        assert targets == [base / ".claude" / "skills", base / ".agents" / "skills"]
+        for root in targets:
+            assert (root / "pythia-apply" / "SKILL.md").is_file()
+
+
 def test_copy_survives_a_symlinked_destination():
     """`npx skills add` leaves .claude/skills/<name> as a symlink into
     .agents/skills/<name>. Copying through that link and then cleaning the
@@ -350,6 +361,97 @@ def test_global_install_wires_the_approve_hook_only():
         assert s["hooks"]["PostToolUse"] == pythia.CLAUDE_HOOKS["PostToolUse"]
 
 
+def test_cmd_install_serves_codex_in_the_project_scope():
+    """Project install writes the harness into AGENTS.md always, and the
+    session-start hook unless --no-hooks. (The .agents/skills copy is gated by
+    global_pack_present() — a machine with a global pack skips it — so that is
+    proven directly in test_fallback_serves_both_claude_and_codex, not here.)"""
+    import argparse
+    import contextlib
+    import io
+    old = os.environ.get("PATH")
+    os.environ["PATH"] = ""                     # no npx: bundled fallback
+    try:
+        for no_hooks in (False, True):
+            with tempfile.TemporaryDirectory() as td:
+                ns = argparse.Namespace(project_root=td, glob=False, source=None,
+                                        color=False, json=False, no_hooks=no_hooks)
+                with contextlib.redirect_stdout(io.StringIO()):
+                    pythia.cmd_install(None, None, ns)
+                base = pathlib.Path(td)
+                agents_md = base / "AGENTS.md"
+                assert agents_md.is_file()                        # always
+                assert pythia.BRIEF_GUIDE.rstrip() in agents_md.read_text(encoding="utf-8")
+                hooks = base / ".codex" / "hooks.json"
+                assert hooks.is_file() != no_hooks                # skipped iff --no-hooks
+    finally:
+        if old is not None:
+            os.environ["PATH"] = old
+
+
+# --- Codex: AGENTS.md harness (TP1) ------------------------------------------
+
+def test_agents_md_block_is_the_guide_verbatim():
+    block = pythia.agents_md_block()
+    assert block.startswith(pythia.AGENTS_BEGIN)
+    assert block.rstrip().endswith(pythia.AGENTS_END)
+    assert pythia.BRIEF_GUIDE.rstrip() in block          # one source, no drift
+    assert "pythia approve <token>" in block             # the Codex approval door
+
+
+def test_merge_agents_md_creates_updates_and_is_idempotent():
+    with tempfile.TemporaryDirectory() as td:
+        path = pathlib.Path(td) / "AGENTS.md"
+        p, action = pythia.merge_agents_md(path)
+        assert p == path and action == "created"
+        assert pythia.BRIEF_GUIDE.rstrip() in path.read_text(encoding="utf-8")
+        assert pythia.merge_agents_md(path)[1] == "unchanged"
+
+
+def test_merge_agents_md_preserves_the_developers_text():
+    with tempfile.TemporaryDirectory() as td:
+        path = pathlib.Path(td) / "AGENTS.md"
+        path.write_text("# House rules\n\nUse tabs.\n", encoding="utf-8")
+        pythia.merge_agents_md(path)
+        body = path.read_text(encoding="utf-8")
+        assert body.startswith("# House rules\n\nUse tabs.\n")   # kept, on top
+        assert pythia.AGENTS_BEGIN in body and pythia.BRIEF_GUIDE.rstrip() in body
+        pythia.merge_agents_md(path)                             # a stale block
+        assert path.read_text(encoding="utf-8").count(pythia.AGENTS_BEGIN) == 1
+
+
+# --- Codex: session-start hook (TP3) -----------------------------------------
+
+def test_merge_codex_hooks_creates_and_is_idempotent():
+    with tempfile.TemporaryDirectory() as td:
+        path = pathlib.Path(td) / ".codex" / "hooks.json"
+        p, added = pythia.merge_codex_hooks(path)
+        assert p == path and added == ["hooks.SessionStart"]
+        s = json.loads(path.read_text(encoding="utf-8"))
+        assert s["hooks"]["SessionStart"] == pythia.CODEX_HOOKS["SessionStart"]
+        assert pythia.merge_codex_hooks(path) == (path, [])
+
+
+def test_merge_codex_hooks_recognises_an_existing_spelling():
+    with tempfile.TemporaryDirectory() as td:
+        path = pathlib.Path(td) / ".codex" / "hooks.json"
+        path.parent.mkdir(parents=True)
+        path.write_text(json.dumps({"hooks": {"SessionStart": [
+            {"hooks": [{"type": "command",
+                        "command": "python3 -m pythia guide --brief"}]}]}}),
+            encoding="utf-8")
+        assert pythia.merge_codex_hooks(path) == (path, [])   # not doubled
+
+
+def test_merge_codex_hooks_refuses_a_file_it_cannot_parse():
+    with tempfile.TemporaryDirectory() as td:
+        path = pathlib.Path(td) / ".codex" / "hooks.json"
+        path.parent.mkdir(parents=True)
+        path.write_text("{not json", encoding="utf-8")
+        assert pythia.merge_codex_hooks(path) == (path, None)
+        assert path.read_text(encoding="utf-8") == "{not json"
+
+
 # Native paths: on POSIX ':' is the PATH separator and would cut a
 # Windows path in half, so these are built from os.sep.
 SCRIPTS = os.path.join(os.sep + 'opt', 'a', 'Scripts')