pypilot-cli 1.0.5__tar.gz → 1.0.6__tar.gz

{pypilot_cli-1.0.5 → pypilot_cli-1.0.6}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.5
 Name: pypilot-cli
-Version: 1.0.5
+Version: 1.0.6
 Summary: Deterministic Python environment doctor, dependency solver, and hardware compatibility manager.
 Project-URL: Homepage, https://github.com/Abdullah-Masood-05/pypilot
 Project-URL: Repository, https://github.com/Abdullah-Masood-05/pypilot

{pypilot_cli-1.0.5 → pypilot_cli-1.0.6}/pyproject.toml

@@ -4,7 +4,7 @@ build-backend = "hatchling.build"
 
 [project]
 name = "pypilot-cli"
-version = "1.0.5"
+version = "1.0.6"
 description = "Deterministic Python environment doctor, dependency solver, and hardware compatibility manager."
 readme = "README.md"
 requires-python = ">=3.8"

{pypilot_cli-1.0.5 → pypilot_cli-1.0.6}/src/pypilot/__init__.py

@@ -10,7 +10,7 @@ from typing import List, Optional
 
 from .binary import ensure_binary, get_cache_dir
 
-__version__ = "1.0.5"
+__version__ = "1.0.6"
 
 
 def run(args: Optional[List[str]] = None, capture_output: bool = False) -> subprocess.CompletedProcess[str]:

{pypilot_cli-1.0.5 → pypilot_cli-1.0.6}/src/pypilot/binary.py

@@ -17,7 +17,7 @@ import zipfile
 from pathlib import Path
 
 GITHUB_REPO = "Abdullah-Masood-05/pypilot"
-DEFAULT_TAG = "v1.0.4"
+DEFAULT_TAG = "v1.0.5"
 # Oldest helper accepted from ~/.cargo/bin or PATH.
 MIN_VERSION = (1, 0, 2)
 # The release this package ships with. A cached download older than this is
@@ -87,7 +87,10 @@ def probe_binary(path: Path | str) -> tuple[int, int, int] | None:
             [str(path), "--version"],
             capture_output=True,
             text=True,
-            timeout=5,
+            # macOS scans a freshly downloaded binary on its first launch, which
+            # can take several seconds. Timing out here made a good engine look
+            # broken and triggered a needless re-download.
+            timeout=30,
             check=False,
         )
         if res.returncode == 0:
@@ -143,14 +146,37 @@ def download_binary(tag: str = DEFAULT_TAG) -> Path:
             else:
                 raise FileNotFoundError(f"{bin_name} not found in downloaded archive {archive_name}")
 
-        shutil.copy2(extracted_bin, dest_binary)
-
-    if sys.platform != "win32":
-        dest_binary.chmod(dest_binary.stat().st_mode | stat.S_IXUSR | stat.S_IXGRP | stat.S_IXOTH)
+        install_binary(extracted_bin, dest_binary)
 
     return dest_binary
 
 
+def install_binary(src: Path, dest: Path) -> None:
+    """Put `src` at `dest` as a new file, never by rewriting `dest` in place.
+
+    macOS caches what it knows about an executable per file. Overwriting a
+    binary that was just run leaves the old cache entry pointing at new bytes,
+    and the next launch fails with "Exec format error". Copying to a temporary
+    name and renaming over `dest` gives it a fresh file instead.
+
+    Windows has no such cache, but its antivirus briefly holds a freshly
+    written .exe open, which makes an immediate rename fail with "Access is
+    denied". There, a plain copy is both sufficient and reliable.
+    """
+    if sys.platform == "win32":
+        shutil.copy2(src, dest)
+        return
+
+    tmp = dest.with_name(f".{dest.name}.{os.getpid()}.tmp")
+    try:
+        shutil.copy2(src, tmp)
+        tmp.chmod(tmp.stat().st_mode | stat.S_IXUSR | stat.S_IXGRP | stat.S_IXOTH)
+        os.replace(tmp, dest)
+    finally:
+        if tmp.exists():
+            tmp.unlink()
+
+
 def ensure_binary() -> Path:
     """
     Locates an existing binary on PATH or in cache, or downloads it from GitHub.

{pypilot_cli-1.0.5 → pypilot_cli-1.0.6}/tests/test_cli.py

@@ -87,3 +87,26 @@ def test_extract_archive_refuses_members_outside_the_destination(tmp_path):
     with pytest.raises(tarfile.TarError):
         extract_archive(archive, out)
     assert not (tmp_path / "escaped").exists()
+
+
+def test_install_binary_replaces_instead_of_rewriting_in_place(tmp_path):
+    import os
+    import sys
+
+    from pypilot.binary import install_binary
+
+    dest = tmp_path / "pypilot"
+    dest.write_bytes(b"old engine")
+    old_inode = os.stat(dest).st_ino
+
+    src = tmp_path / "new"
+    src.write_bytes(b"new engine")
+    install_binary(src, dest)
+
+    assert dest.read_bytes() == b"new engine"
+    # A rename gives `dest` a new file; rewriting in place would keep the inode.
+    if sys.platform != "win32":
+        assert os.stat(dest).st_ino != old_inode
+        assert os.access(dest, os.X_OK)
+    # No temporary file is left behind.
+    assert sorted(p.name for p in tmp_path.iterdir()) == ["new", "pypilot"]

{pypilot_cli-1.0.5 → pypilot_cli-1.0.6}/uv.lock

@@ -1157,7 +1157,7 @@ wheels = [
 
 [[package]]
 name = "pypilot-cli"
-version = "1.0.5"
+version = "1.0.6"
 source = { editable = "." }
 
 [package.dev-dependencies]