pythia-plsql 0.4.7__tar.gz → 0.4.8__tar.gz

{pythia_plsql-0.4.7/scripts/pythia_plsql.egg-info → pythia_plsql-0.4.8}/PKG-INFO

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

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

@@ -209,12 +209,43 @@ def offer_path_fix(scripts_dir, interactive, forced=False):
         print(f"Could not update your PATH: {e}")
         return False
     print(NL + f"  {'Added' if changed else 'Already in'} your user PATH: {scripts_dir}")
+    warn = path_length_warning(os.environ.get("PATH", ""))
+    if warn:
+        print("  " + warn)
     print("  Open a NEW terminal for it to take effect — an already-running")
     print("  one keeps the environment it started with, and so do its tabs.")
     print(f"  Until then: {pathlib.Path(sys.executable).stem} -m pythia <command>")
     return True
 
 
+def path_length_warning(combined):
+    """Windows tooling still truncates PATH around 2047 characters, and a
+    freshly appended entry sits last -- so it is the first thing lost, and it
+    is lost silently. The usual cause is a PATH that had the system entries
+    copied into the user's."""
+    n = len(combined or "")
+    if n < 1900:
+        return None
+    return (f"Your PATH is {n} characters; Windows truncates near 2047, and the "
+            "entry just added is last in line.\n"
+            "  The usual cause is duplicate entries — the system PATH copied "
+            "into the user PATH by\n"
+            "  a `$env:PATH` one-liner. Compare the two scopes and remove what "
+            "appears in both.")
+
+
+def stored_path_has(directory):
+    """Is the directory in the PATH as *stored*, rather than the one this
+    process inherited? The two differ for every terminal opened before the
+    last change, which is the common confusion."""
+    if not directory or os.name != "nt":
+        return False
+    try:
+        return path_contains(_read_user_path(), directory)
+    except OSError:
+        return False
+
+
 def path_contains(path_value, directory):
     """Is this directory already listed in a PATH string? Compared the way
     the platform does: case-insensitively on Windows, ignoring trailing
@@ -280,7 +311,7 @@ def installed_scripts_dir():
     return candidates[-1] if candidates else None
 
 
-def entry_point_hint(found, scripts_dir):
+def entry_point_hint(found, scripts_dir, in_stored_path=False):
     """What to say when `pythia` will not resolve as a command. pip installs
     the executable into a scripts directory that is often not on PATH — the
     default for `pip install --user` on Windows — and the shell's
@@ -288,6 +319,21 @@ def entry_point_hint(found, scripts_dir):
     step immediately after the install the docs tell you to run."""
     if found:
         return None
+    if in_stored_path:
+        # The directory is in the stored PATH but not in this process: the
+        # terminal started before the change. Sending someone to re-run the
+        # install here wastes their time -- it already worked.
+        return (NL.join([
+            "`pythia` is already on your PATH — this terminal just started "
+            "before that was true.",
+            "  Open a NEW terminal WINDOW. A new tab is not enough: tabs "
+            "inherit the",
+            "  environment of the window that spawned them, so if you use "
+            "Windows Terminal,",
+            "  VS Code or Cursor, restart the application itself.",
+            "  In this one, keep using: "
+            + pathlib.Path(sys.executable).stem + " -m pythia <command>",
+        ]))
     out = ["`pythia` is installed but not on your PATH."]
     if scripts_dir:
         out.append("  It lives in: " + str(scripts_dir))
@@ -2055,11 +2101,13 @@ def cmd_install(conn, schema, ns):
             clean_legacy_skills(ns.project_root)
     print(f"\nNext: fill in {path}")
     print(f"Then: {invocation()} check")
-    hint = entry_point_hint(shutil.which("pythia"), installed_scripts_dir())
+    scripts_dir = installed_scripts_dir()
+    hint = entry_point_hint(shutil.which("pythia"), scripts_dir,
+                            in_stored_path=stored_path_has(scripts_dir))
     if hint:
         print()
         print(hint)
-        offer_path_fix(installed_scripts_dir(), interactive,
+        offer_path_fix(scripts_dir, interactive,
                        getattr(ns, "add_to_path", False))
 
 

{pythia_plsql-0.4.7 → pythia_plsql-0.4.8/scripts/pythia_plsql.egg-info}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: pythia-plsql
-Version: 0.4.7
+Version: 0.4.8
 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.7 → pythia_plsql-0.4.8}/tests/test_install.py

@@ -295,6 +295,36 @@ def test_add_to_user_path_handles_an_empty_and_untidy_value():
     assert written == [USERBIN + os.pathsep + SCRIPTS]   # no doubled separator
 
 
+def test_hint_tells_a_stale_shell_apart_from_an_unconfigured_one():
+    """The directory being in the stored PATH while the running process
+    cannot see it means one thing only: this terminal started earlier. Saying
+    "not on your PATH" there sends people to re-run an install that already
+    worked -- which is exactly what happened in the field."""
+    stale = pythia.entry_point_hint(found=None, scripts_dir=SCRIPTS,
+                                    in_stored_path=True)
+    assert "already" in stale.lower()
+    assert "new" in stale.lower() and "tab" in stale.lower()   # window, not tab
+    assert "--add-to-path" not in stale        # nothing to redo
+    unconfigured = pythia.entry_point_hint(found=None, scripts_dir=SCRIPTS,
+                                           in_stored_path=False)
+    assert SCRIPTS in unconfigured and "-m pythia" in unconfigured
+    # how you fix it is platform-specific: pythia can edit the registry on
+    # Windows, while a POSIX shell profile is the developer's to edit
+    assert ("--add-to-path" if os.name == "nt" else "profile") in unconfigured
+    # and once the command resolves, still silence
+    assert pythia.entry_point_hint(found="/x/pythia", scripts_dir=SCRIPTS,
+                                   in_stored_path=True) is None
+
+
+def test_long_path_is_flagged_before_it_silently_truncates():
+    """Windows tooling still truncates PATH near 2047 characters, and the
+    entry we just appended is the last one -- the first to be lost."""
+    assert pythia.path_length_warning("x" * 1000) is None
+    w = pythia.path_length_warning("x" * 2040)
+    assert w is not None and "2047" in w
+    assert "duplicate" in w.lower()      # names the usual cause
+
+
 def main():
     failed = 0
     for name, fn in sorted(globals().items()):