pythia-plsql 0.3.2__tar.gz → 0.3.4__tar.gz

{pythia_plsql-0.3.2/scripts/pythia_plsql.egg-info → pythia_plsql-0.3.4}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: pythia-plsql
-Version: 0.3.2
+Version: 0.3.4
 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.3.2 → pythia_plsql-0.3.4}/pyproject.toml

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

@@ -1659,20 +1659,26 @@ 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: project .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. The copy
-    merges: only the pack's own skills are refreshed; stale copies of the
-    pack under .agents/skills and old plsql-* names are cleaned."""
+    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.
+
+    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."""
     import shutil
-    dest_root = pathlib.Path(base_dir) / ".claude" / "skills"
+    base = pathlib.Path(base_dir)
+    dest_root = base / ".claude" / "skills"
     for pack in sorted(SKILLS_DIR.iterdir()):
         if not (pack / "SKILL.md").is_file():
             continue
-        shutil.copytree(pack, dest_root / pack.name, dirs_exist_ok=True)
+        _remove_link_first(dest_root / pack.name)
+        _remove_link_first(base / ".agents" / "skills" / pack.name)
+        shutil.copytree(pack, dest_root / pack.name)
     clean_legacy_skills(base_dir)
-    for name in pack_names():   # canonical copy lives in .claude now
-        _remove_link_first(pathlib.Path(base_dir) / ".agents" / "skills" / name)
     return [dest_root]
 
 

{pythia_plsql-0.3.2 → pythia_plsql-0.3.4/scripts/pythia_plsql.egg-info}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: pythia-plsql
-Version: 0.3.2
+Version: 0.3.4
 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.3.2 → pythia_plsql-0.3.4}/skills/pythia-explore/SKILL.md

@@ -1,6 +1,6 @@
 ---
 name: pythia-explore
-description: Use when you need to understand anything in an Oracle schema - finding objects, reading PL/SQL source, signatures, table columns, DDL, searching code, or asking who uses what. The database is the only source of truth; repo dumps and exports drift and lie.
+description: Use when you need to understand anything in an Oracle schema - finding objects, reading PL/SQL source, signatures, table columns, DDL, searching code, or asking who uses what. ALSO use when a .sql file, dump, export, migration script or repo copy of an object appears to answer the question already, including when the developer points you at one: those files drift from the database and are wrong in ways that read as correct. The database is the only source of truth.
 ---
 
 # Exploring the Schema
@@ -22,8 +22,17 @@ database:
 | **Indexes** | **116** | **1,016** | **~89% missing** |
 
 Code that "reads fine" against the dump can reference types and packages the
-dump never heard of. Read files only when the database is unreachable, or to
-compare a repo version against the live one.
+dump never heard of.
+
+**When the developer hands you a `.sql` file** — "explain this procedure",
+"here is the export, what does it do" — the file is a claim, not the truth.
+Read it, then run `pythia src <NAME>` and answer from the database. If the
+two differ, say so first: which is newer is exactly what the developer needs
+to know, and it is invisible from the file alone. Never answer about a
+database object from a file alone while the database is reachable.
+
+Read files as the answer only when the database is unreachable — and say
+that is what you are doing.
 
 ## What you need → what you run
 

{pythia_plsql-0.3.2 → pythia_plsql-0.3.4}/tests/test_install.py

@@ -65,6 +65,29 @@ def test_missing_npx_falls_back_to_the_bundled_pack():
         pythia.copy_bundled_skills(td)                       # idempotent
 
 
+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
+    target left dangling links and no pack — it happened on a real machine."""
+    with tempfile.TemporaryDirectory() as td:
+        base = pathlib.Path(td)
+        agents = base / ".agents" / "skills" / "pythia-explore"
+        claude = base / ".claude" / "skills"
+        agents.mkdir(parents=True)
+        (agents / "SKILL.md").write_text("old", encoding="utf-8")
+        claude.mkdir(parents=True)
+        try:
+            (claude / "pythia-explore").symlink_to(agents,
+                                                   target_is_directory=True)
+        except (OSError, NotImplementedError):
+            return          # no symlink privilege on this box; skip
+        pythia.copy_bundled_skills(td)
+        real = claude / "pythia-explore" / "SKILL.md"
+        assert real.is_file(), "pack vanished through the symlink"
+        assert not (claude / "pythia-explore").is_symlink()
+        assert "old" not in real.read_text(encoding="utf-8")
+
+
 def test_global_pack_detection_skips_project_copies():
     with tempfile.TemporaryDirectory() as home:
         assert not pythia.global_pack_present(home)