pyhabitat 1.0.19__tar.gz → 1.0.21__tar.gz

{pyhabitat-1.0.19 → pyhabitat-1.0.21}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: pyhabitat
-Version: 1.0.19
+Version: 1.0.21
 Summary: A lightweight library for detecting system environment, GUI, and build properties.
 Author-email: George Clayton Bennett <george.bennett@memphistn.gov>
 License-Expression: MIT

{pyhabitat-1.0.19 → pyhabitat-1.0.21}/pyhabitat/__init__.py

@@ -1,5 +1,5 @@
 # pyhabitat/__init__.py
-
+from .utils import get_version
 from .environment import (
     matplotlib_is_available_for_gui_plotting,
     matplotlib_is_available_for_headless_image_export,
@@ -58,3 +58,5 @@ __all__ = [
     'interp_path',
     'main',
 ]
+
+__version__ = get_version()

{pyhabitat-1.0.19 → pyhabitat-1.0.21}/pyhabitat/cli.py

@@ -1,18 +1,28 @@
 import argparse
 from pathlib import Path
 from pyhabitat.environment import main
+from pyhabitat.utils import get_version
 
 def run_cli():
     """Parse CLI arguments and run the pyhabitat environment report."""
+    current_version = get_version()
     parser = argparse.ArgumentParser(
         description="PyHabitat: Python environment and build introspection"
     )
+    # Add the version argument
+    parser.add_argument(
+        '-v', '--version', 
+        action='version', 
+        version=f'%(prog)s {current_version}'
+    )
+    # Add the path argument
     parser.add_argument(
         "--path",
         type=str,
         default=None,
         help="Path to a script or binary to inspect (defaults to sys.argv[0])",
     )
+    # Add the debug argument
     parser.add_argument(
         "--debug",
         action="store_true",
@@ -20,3 +30,4 @@ def run_cli():
     )
     args = parser.parse_args()
     main(path=Path(args.path) if args.path else None, debug=args.debug)
+

{pyhabitat-1.0.19 → pyhabitat-1.0.21}/pyhabitat/environment.py

@@ -75,7 +75,8 @@ def matplotlib_is_available_for_gui_plotting(termux_has_gui=False):
         # Force the common GUI backend. At this point, we know tkinter is *available*.
         # # 'TkAgg' is often the most reliable cross-platform test.
         # 'TkAgg' != 'Agg'. The Agg backend is for non-gui image export. 
-        matplotlib.use('TkAgg', force=True)
+        if matplotlib.get_backend().lower() != 'tkagg':
+            matplotlib.use('TkAgg', force=True)
         import matplotlib.pyplot as plt
         # A simple test call to ensure the backend initializes
         # This final test catches any edge cases where tkinter is present but 
@@ -226,18 +227,21 @@ def on_wsl():
     if "WSL_DISTRO_NAME" in os.environ or "WSL_INTEROP" in os.environ:
         return True
 
-    # --- Check kernel info for 'microsoft' string ---
+    # --- Check kernel info for 'microsoft' or 'wsl' string (Fallback) ---
     # False negative risk: 
     # Custom kernels, future Windows versions, or minimal WSL distros may omit 'microsoft' in strings.
     # False negative likelihood: Very low to moderate.
-
     try:
         with open("/proc/version") as f:
+            version_info = f.read().lower() 
             if "microsoft" in version_info or "wsl" in version_info:
                 return True
-    except FileNotFoundError:
+    except (IOError, OSError):
+        # This block would catch the PermissionError!
+        # It would simply 'pass' and move on.
         pass
-        
+
+
     # Check for WSL-specific mounts (fallback)
     """
     /proc/sys/kernel/osrelease
@@ -245,17 +249,17 @@ def on_wsl():
     Very reliable for detecting WSL1 and WSL2 unless someone compiled a custom kernel and removed the microsoft string.
     
     False negative risk: 
-    If /proc/version or /proc/sys/kernel/osrelease cannot be read due to permissions, a containerized WSL distro, or some sandboxed environment.
+    If /proc/sys/kernel/osrelease cannot be read due to permissions, a containerized WSL distro, or some sandboxed environment.
     # False negative likelihood: Very low.
     """
-    if Path("/proc/sys/kernel/osrelease").exists():
-        try:
-            with open("/proc/sys/kernel/osrelease") as f:
-                osrelease = f.read().lower()
-                if "microsoft" in osrelease:
-                    return True
-        except Exception:
-            pass
+    try:
+        with open("/proc/sys/kernel/osrelease") as f:
+            osrelease = f.read().lower()
+            if "microsoft" in osrelease:
+                return True
+    except (IOError, OSError):
+    # This block would catch the PermissionError, an FileNotFound
+        pass
     return False
 
 def on_pydroid():
@@ -427,75 +431,52 @@ def is_macos_executable(exec_path: Path | str | None = None, debug: bool = False
 def is_pipx(exec_path: Path | str | None = None, debug: bool = False, suppress_debug: bool = True) -> bool:
     """Checks if the executable is running from a pipx managed environment."""
     exec_path, is_valid = _check_executable_path(exec_path, debug and not suppress_debug, check_pipx=False)
+    # check_pipx arg should be false when calling from inside of is_pipx() to avoid recursion error
+    # For safety, _check_executable_path() guards against this.
     if not is_valid:
         return False
         
     try:
         interpreter_path = Path(sys.executable).resolve()
         pipx_bin_path, pipx_venv_base_path = _get_pipx_paths()
+
         # Normalize paths for comparison
         norm_exec_path = str(exec_path).lower()
         norm_interp_path = str(interpreter_path).lower()
+        pipx_venv_base_str = str(pipx_venv_base_path).lower()
 
         if debug:
             logging.debug(f"EXEC_PATH: {exec_path}")
             logging.debug(f"INTERP_PATH: {interpreter_path}")
             logging.debug(f"PIPX_BIN_PATH: {pipx_bin_path}")
             logging.debug(f"PIPX_VENV_BASE: {pipx_venv_base_path}")
-            is_in_pipx_venv_base = norm_interp_path.startswith(str(pipx_venv_base_path).lower())
+            is_in_pipx_venv_base = norm_interp_path.startswith(pipx_venv_base_str)
             logging.debug(f"Interpreter path resides somewhere within the pipx venv base hierarchy: {is_in_pipx_venv_base}")
             logging.debug(
                 f"This determines whether the current interpreter is managed by pipx: {is_in_pipx_venv_base}"
             )
         if "pipx/venvs" in norm_exec_path or "pipx/venvs" in norm_interp_path:
             if debug:
-                logging.debug("True (Signature Check)")
+                logging.debug("is_pipx() is True // Signature Check")
             return True
 
-        if norm_interp_path.startswith(str(pipx_venv_base_path).lower()):
+        if norm_interp_path.startswith(pipx_venv_base_str):
             if debug:
-                logging.debug("True (Interpreter Base Check)")
+                logging.debug("is_pipx() is True // Interpreter Base Check")
             return True
 
-        if norm_exec_path.startswith(str(pipx_venv_base_path).lower()):
+        if norm_exec_path.startswith(pipx_venv_base_str):
             if debug:
-                logging.debug("True (Executable Base Check)")
+                logging.debug("is_pipx() is True // Executable Base Check")
             return True
 
-        return False
-    except Exception:
         if debug:
-            logging.debug("False (Exception during pipx check)")
+            logging.debug("is_pipx() is False")
         return False
-        if debug:
-            logging.debug(f"EXEC_PATH: {exec_path}")
-            logging.debug(f"INTERP_PATH: {interpreter_path}")
-            logging.debug(f"PIPX_BIN_PATH: {pipx_bin_path}")
-            logging.debug(f"PIPX_VENV_BASE: {pipx_venv_base_path}")
-            logging.debug(f"Check B result: {norm_interp_path.startswith(str(pipx_venv_base_path).lower())}")
-
-        if "pipx/venvs" in norm_exec_path or "pipx/venvs" in norm_interp_path:
-            if debug:
-                logging.debug("True (Signature Check)")
-            return True
-
-        if norm_interp_path.startswith(str(pipx_venv_base_path).lower()):
-            if debug:
-                logging.debug("True (Interpreter Base Check)")
-            return True
-
-        if norm_exec_path.startswith(str(pipx_venv_base_path).lower()):
-            if debug:
-                logging.debug("True (Executable Base Check)")
-            return True
 
-        if debug:
-            logging.debug("False")
-        return False
     except Exception:
         if debug:
             logging.debug("False (Exception during pipx check)")
-        return False
     
 def is_python_script(path: Path | str | None = None, debug: bool = False, suppress_debug: bool =False) -> bool:
     """
@@ -574,8 +555,8 @@ def edit_textfile(path: Path | str | None = None) -> None:
 #def open_text_file_for_editing(path): # defunct function name as of 1.0.16
     """
     Opens a file with the environment's default application (Windows, Linux, macOS)
-    or a guaranteed console editor (nano) in constrained environments (Termux, iSH)
-    after ensuring line-ending compatibility.
+    or a guaranteed console editor (nano) in constrained environments (Termux, iSH).
+    Ensures line-ending compatibility where possible.
 
     This function is known to fail on PyDroid3, where on_linus() is True but xdg-open 
     is not available.
@@ -590,28 +571,28 @@ def edit_textfile(path: Path | str | None = None) -> None:
             os.startfile(path)
         elif on_termux():
     	    # Install dependencies if missing (Termux pkg returns non-zero if already installed, so no check=True)
-    	    subprocess.run(['pkg','install', 'dos2unix', 'nano'], stdout=subprocess.DEVNULL, stderr=subprocess.DEVNULL)
-    	    _run_dos2unix(path)
-    	    subprocess.run(['nano', path])
+            subprocess.run(['pkg','install', 'dos2unix', 'nano'], stdout=subprocess.DEVNULL, stderr=subprocess.DEVNULL)
+            _run_dos2unix(path)
+            subprocess.run(['nano', str(path)])
         elif on_ish_alpine():
             # Install dependencies if missing (apk returns 0 if already installed, so check=True is safe)
-    	    subprocess.run(['apk','add', 'dos2unix'], stdout=subprocess.DEVNULL, stderr=subprocess.DEVNULL, check=True)
-    	    subprocess.run(['apk','add', 'nano'], stdout=subprocess.DEVNULL, stderr=subprocess.DEVNULL, check=True)
-    	    _run_dos2unix(path)
-    	    subprocess.run(['nano', path])
+            subprocess.run(['apk','add', 'dos2unix'], stdout=subprocess.DEVNULL, stderr=subprocess.DEVNULL, check=True)
+            subprocess.run(['apk','add', 'nano'], stdout=subprocess.DEVNULL, stderr=subprocess.DEVNULL, check=True)
+            _run_dos2unix(path)
+            subprocess.run(['nano', str(path)])
     	# --- Standard Unix-like Systems (Conversion + Default App) ---
         elif on_linux():
-    	    _run_dos2unix(path) # Safety conversion for user-defined console apps
-    	    subprocess.run(['xdg-open', path])
+            _run_dos2unix(path) # Safety conversion for user-defined console apps
+            subprocess.run(['xdg-open', str(path)])
         elif on_apple():
-    	    _run_dos2unix(path) # Safety conversion for user-defined console apps
-    	    subprocess.run(['open', path])
+            _run_dos2unix(path) # Safety conversion for user-defined console apps
+            subprocess.run(['open', str(path)])
         else:
-    	    print("Unsupported operating system.")
+            print("Unsupported operating system.")
     except Exception as e:
-	    print("The file could not be opened for editing in the current environment. Known failure points: Pydroid3")
-
-    """Why Not Use check=True on Termux:
+        print("The file could not be opened for editing in the current environment: {e}")
+    """
+    Why Not Use check=True on Termux:
     The pkg utility in Termux is a wrapper around Debian's apt. When you run pkg install <package>, if the package is already installed, the utility often returns an exit code of 100 (or another non-zero value) to indicate that no changes were made because the package was already present.
     """
 
@@ -633,8 +614,10 @@ def _run_dos2unix(path: Path | str | None = None):
         # Catch other subprocess errors (e.g. permission issues)
         pass
     
-def read_magic_bytes(path: str, length: int = 4, debug: bool = False) -> bytes:
-    """Return the first few bytes of a file for type detection."""
+def read_magic_bytes(path: str, length: int = 4, debug: bool = False) -> bytes | None:
+    """Return the first few bytes of a file for type detection.
+    Returns None if the file cannot be read or does not exist.
+    """
     try:
         with open(path, "rb") as f:
             magic = f.read(length)
@@ -644,7 +627,9 @@ def read_magic_bytes(path: str, length: int = 4, debug: bool = False) -> bytes:
     except Exception as e:
         if debug:
             logging.debug(f"False (Error during file check: {e})")
-        return False
+        #return False # not typesafe
+        #return b'' # could be misunderstood as what was found
+        return None # no way to conflate that this was a legitimate error
     
 def _get_pipx_paths():
     """
@@ -685,8 +670,19 @@ def _check_if_zip(path: Path | str | None) -> bool:
         # Handle cases where the path might be invalid, or other unexpected errors
         return False
 
-def _check_executable_path(exec_path: Path | str | None, debug: bool = False, check_pipx: bool = True) -> tuple[Path | None, bool]:
-    """Helper function to resolve executable path and perform common checks."""
+def _check_executable_path(exec_path: Path | str | None, 
+                           debug: bool = False, 
+                           check_pipx: bool = True
+) -> tuple[Path | None, bool]: #compensate with __future__, may cause type checker issues
+    """
+    Helper function to resolve an executable path and perform common checks.
+
+    Returns:
+        tuple[Path | None, bool]: (Resolved path, is_valid)
+        - Path: The resolved Path object, or None if invalid
+        - bool: Whether the path should be considered valid for subsequent checks
+    """
+    # 1. Determine path
     if exec_path is None:
         exec_path = Path(sys.argv[0]).resolve() if sys.argv[0] and sys.argv[0] != '-c' else None
     else:
@@ -695,22 +691,31 @@ def _check_executable_path(exec_path: Path | str | None, debug: bool = False, ch
     if debug:
         logging.debug(f"Checking executable path: {exec_path}")
 
+    # 2. Handle missing path
     if exec_path is None:
         if debug:
-            logging.debug("False (No valid path)")
+            logging.debug("_check_executable_path() returns (None, False) // exec_path is None")
         return None, False
-
-    if check_pipx and is_pipx(exec_path, debug):
+    
+    # 3. Ensure path actually exists and is a file
+    if not exec_path.is_file(): 
         if debug:
-            logging.debug("False (is_pipx is True)")
+            logging.debug("_check_executable_path() returns (exec_path, False) // exec_path is not a file")
         return exec_path, False
 
-    if not exec_path.is_file():
-        if debug:
-            logging.debug("False (Not a file)")
-        return exec_path, False
+    # 4. Avoid recursive pipx check loops
+    # This guard ensures we don’t recursively call _check_executable_path()
+    # via is_pipx() -> _check_executable_path() -> is_pipx() -> ...
+    if check_pipx:
+        caller = sys._getframe(1).f_code.co_name
+        if caller != "is_pipx":
+            if is_pipx(exec_path, debug):
+                if debug:
+                    logging.debug("_check_executable_path() returns (exec_path, False) // is_pipx(exec_path) is True")
+                return exec_path, False
 
-    return exec_path, True        
+    return exec_path, True       
+ 
 
 # --- Main Function for report and CLI compatibility ---
 
@@ -795,7 +800,7 @@ def main(path=None, debug=False):
             # Supress redundant prints explicity using suppress_debug=True, 
             # so that only unique information gets printed for each check, 
             # even when more than one use the same functions which include debugging logs.
-            p#rint(f"_check_executable_path(script_path, debug=True)")
+            #print(f"_check_executable_path(script_path, debug=True)")
             _check_executable_path(script_path, debug=debug)
             #print(f"read_magic_bites(script_path, debug=True)")
             read_magic_bytes(script_path, debug=debug)
@@ -818,4 +823,7 @@ def main(path=None, debug=False):
     print("=================================")
     print("=== PyHabitat Report Complete ===")
     print("=================================")
-    print("")
+    print("")
+
+if __name__ == "__main__": 
+    main(debug=True)

pyhabitat-1.0.21/pyhabitat/utils.py

@@ -0,0 +1,10 @@
+from importlib.metadata import version, PackageNotFoundError
+def get_version() -> str:
+    """Retrieves the installed package version."""
+    try:
+        # The package name 'pyhabitat' must exactly match the name in your pyproject.toml
+        return version('pyhabitat')
+    except PackageNotFoundError:
+        # This occurs if the script is run directly from the source directory
+        # without being installed in editable mode, or if the package name is wrong.
+        return "Not Installed (Local Development or Incorrect Name)"

{pyhabitat-1.0.19 → pyhabitat-1.0.21}/pyhabitat.egg-info/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: pyhabitat
-Version: 1.0.19
+Version: 1.0.21
 Summary: A lightweight library for detecting system environment, GUI, and build properties.
 Author-email: George Clayton Bennett <george.bennett@memphistn.gov>
 License-Expression: MIT

{pyhabitat-1.0.19 → pyhabitat-1.0.21}/pyhabitat.egg-info/SOURCES.txt

@@ -6,6 +6,7 @@ pyhabitat/__main__.py
 pyhabitat/__main__stable.py
 pyhabitat/cli.py
 pyhabitat/environment.py
+pyhabitat/utils.py
 pyhabitat.egg-info/PKG-INFO
 pyhabitat.egg-info/SOURCES.txt
 pyhabitat.egg-info/dependency_links.txt

{pyhabitat-1.0.19 → pyhabitat-1.0.21}/pyproject.toml

@@ -6,7 +6,7 @@ build-backend = "setuptools.build_meta"
 
 [project]
 name = "pyhabitat"
-version = "1.0.19"
+version = "1.0.21"
 #dynamic = ["version"] # 
 authors = [
   { name="George Clayton Bennett", email="george.bennett@memphistn.gov" },