driverclient 0.2.28__tar.gz → 0.2.30__tar.gz

{driverclient-0.2.28 → driverclient-0.2.30}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: driverclient
-Version: 0.2.28
+Version: 0.2.30
 Summary: Driver Server client with an embeddable connector for config injection at runtime.
 Project-URL: Homepage, https://example.com/driverclient
 Author-email: Raja Sanaullah <sanaullah@99technologies.com>

{driverclient-0.2.28 → driverclient-0.2.30}/pyproject.toml

@@ -4,7 +4,7 @@ build-backend = "hatchling.build"
 
 [project]
 name = "driverclient"
-version = "0.2.28"
+version = "0.2.30"
 description = "Driver Server client with an embeddable connector for config injection at runtime."
 readme = "README.md"
 requires-python = ">=3.10"

{driverclient-0.2.28 → driverclient-0.2.30}/src/driverclient/__init__.py

@@ -26,7 +26,7 @@ from driverclient.main import run as _run
 
 __all__ = ["ClientEvent", "DriverClient"]
 
-__version__ = "0.2.28"
+__version__ = "0.2.30"
 
 
 class DriverClient:

{driverclient-0.2.28 → driverclient-0.2.30}/src/driverclient/ops/capture.py

@@ -404,22 +404,12 @@ def wu_full() -> CaptureResult:
 
 # ══ Missing-HWID classification ═══════════════════════════════════════════════
 
-_INBOX_INADEQUATE_CLASSES = {"display adapters"}
-
 
 def classify_missing(not_found: list[str], scan_result: ScanResult) -> dict[str, list[str]]:
     """Bucket repo-missing HWIDs by what the machine says about their device:
     inbox (Windows ships the driver — not applicable), oem_local (OEM driver
     installed locally — capturable), driverless (genuinely missing — WU
-    candidate), unknown (HWID not in scan).
-
-    Display adapters are excluded from the inbox bucket: Microsoft Basic
-    Display Adapter is Windows' inbox fallback for a GPU whose real driver
-    was never bound, and it leaves the device non-accelerated — unlike other
-    inbox drivers (HID, mass storage, ...) it is not adequate coverage. Such
-    a device is treated as driverless so it still reaches WU / gets captured
-    once a real driver shows up, instead of being silently written off.
-    """
+    candidate), unknown (HWID not in scan)."""
     hwid_to_device: dict[str, Device] = {}
     for scanned in scan_result.devices:
         for h in scanned.hwids:
@@ -437,10 +427,8 @@ def classify_missing(not_found: list[str], scan_result: ScanResult) -> dict[str,
             buckets["unknown"].append(hwid)
         elif not dev.has_driver:
             buckets["driverless"].append(hwid)
-        elif dev.is_inbox and dev.class_name.lower() not in _INBOX_INADEQUATE_CLASSES:
-            buckets["inbox"].append(hwid)
         elif dev.is_inbox:
-            buckets["driverless"].append(hwid)
+            buckets["inbox"].append(hwid)
         else:
             buckets["oem_local"].append(hwid)
     return buckets
@@ -1214,4 +1202,4 @@ def _upload_worker(upload_q: queue.Queue, counts: dict, lock: threading.Lock, cf
 # Extension selection was removed: capture now packs every file
 # pnputil /export-driver produces (see _do_export), so the dump_extensions /
 # dump_extensions_overrides config keys are no longer consulted — filtering by
-# extension is what caused incomplete packages (broken link D).
+# extension is what caused incomplete packages (broken link D).

{driverclient-0.2.28 → driverclient-0.2.30}/src/driverclient/ops/download.py

@@ -12,6 +12,7 @@ Factored out here so the two modes share one download implementation:
   • download_drivers() — fan the above out across a driver list in parallel
 """
 
+import shutil
 from concurrent.futures import ThreadPoolExecutor, as_completed
 from pathlib import Path
 
@@ -20,7 +21,21 @@ from driverclient.core.http import http_download_retry
 
 def download_driver(driver: dict, base: Path, repo_url: str, cfg: dict) -> tuple[Path, bool]:
     """Download all files for one driver into base/<binding_id>/.
-    Returns (dest_dir, success). success is False if ANY file failed."""
+    Returns (dest_dir, success). success is False if ANY file failed.
+
+    On failure, dest is removed before returning. The live install path stages
+    every driver under one shared `base` and installs it with a single batched
+    `pnputil /add-driver base\\*.inf /subdirs` — that command globs the
+    filesystem directly, so a driver this function reports as failed (and the
+    caller correctly excludes from the install list) would otherwise still
+    leave its already-downloaded files (commonly just the small .inf, if a
+    large companion .sys/.dll timed out) sitting under base/<binding_id>/ for
+    pnputil to find anyway. pnputil then tries to add that incomplete package —
+    "cannot find the file specified" for its missing payload — which large
+    graphics drivers hit most often (biggest files, likeliest to time out
+    partway), and can poison the whole batched pass for the OTHER, fully
+    downloaded drivers riding in the same pnputil call.
+    """
     bid = driver.get("binding_id") or driver.get("package_id", "")
     pkg_id = driver["package_id"]
     dest = base / bid
@@ -36,6 +51,9 @@ def download_driver(driver: dict, base: Path, repo_url: str, cfg: dict) -> tuple
         if not http_download_retry(url, path, f["sha256"], timeout=cfg["timeout_long"]):
             ok = False
 
+    if not ok:
+        shutil.rmtree(dest, ignore_errors=True)
+
     return dest, ok