Add Windows support with Scoop provider and POSIX compatibility shims

.github/workflows/tests.yml

@@ -83,6 +83,11 @@ jobs:
     runs-on: ${{ matrix.target.os }}
     timeout-minutes: 20
     if: ${{ needs.discover-standard-tests.outputs.test-files != '[]' }}
+    # Use git-bash on Windows runners so the (mostly POSIX) setup scripts
+    # below keep working without duplicating them in PowerShell.
+    defaults:
+      run:
+        shell: bash
     strategy:
       fail-fast: false
       max-parallel: 20
@@ -94,6 +99,8 @@ jobs:
           python_version: '3.14'
         - os: macOS-latest
           python_version: '3.13'
+        - os: windows-latest
+          python_version: '3.13'
         test: ${{ fromJson(needs.discover-standard-tests.outputs.test-files) }}
 
     steps:
@@ -103,7 +110,7 @@ jobs:
       uses: actions/setup-python@v6
       with:
         python-version: ${{ matrix.target.python_version }}
-   
+
     - name: Install uv
       uses: astral-sh/setup-uv@v8.0.0
       with:
@@ -121,6 +128,9 @@ jobs:
         node-version: '22'
 
     - name: Setup Yarn (classic + Berry)
+      # Uses ``ln -sf`` / Unix prefix dirs — not applicable to the Windows
+      # runner and not needed for the Windows test matrix anyway.
+      if: runner.os != 'Windows'
       run: |
         npm install -g yarn@1.22.22
         if [ "$(uname -s)" = "Darwin" ]; then
@@ -146,7 +156,18 @@ jobs:
         yarn-berry --version
         yarn --version | grep -q '^1\.' || { echo "ERROR: yarn is not 1.x"; exit 1; }
 
+    - name: Setup Yarn classic (Windows)
+      # Minimal Yarn setup for Windows: classic yarn is enough for most
+      # tests that ``require_tool("yarn")``; Berry's Unix prefix scheme
+      # isn't portable to Windows and the Berry-specific tests are
+      # already gated behind ``require_tool("yarn-berry")``.
+      if: runner.os == 'Windows'
+      run: npm install -g yarn@1.22.22
+
     - name: Setup Bun
+      # Scoop/choco are preinstalled on windows runners but setup-bun
+      # doesn't support Windows at all, so skip there.
+      if: runner.os != 'Windows'
       uses: oven-sh/setup-bun@v2
       with:
         bun-version: latest
@@ -162,31 +183,46 @@ jobs:
         go-version: '1.25'
 
     - name: Install Nix
+      # Nix has no Windows build.
+      if: runner.os != 'Windows'
       uses: DeterminateSystems/nix-installer-action@v22
 
     - name: Setup venv and install pip dependencies
       run: |
         export PNPM_HOME="${RUNNER_TEMP}/pnpm"
         uv venv --python "${{ matrix.target.python_version }}"
-        uv sync --all-extras
+        # ``ansible`` has no Windows build, and ``pyinfra`` pulls it in
+        # transitively, so on Windows we install the base extras only.
+        if [ "${{ runner.os }}" = "Windows" ]; then
+          uv sync
+        else
+          uv sync --all-extras
+        fi
         uv pip install pip
         mkdir -p "$PNPM_HOME"
         echo "$PNPM_HOME" >> "$GITHUB_PATH"
-        echo "/home/linuxbrew/.linuxbrew/bin" >> "$GITHUB_PATH"
-        if [ -d /nix/var/nix/profiles/default/bin ]; then echo "/nix/var/nix/profiles/default/bin" >> "$GITHUB_PATH"; fi
+        if [ "${{ runner.os }}" != "Windows" ]; then
+          echo "/home/linuxbrew/.linuxbrew/bin" >> "$GITHUB_PATH"
+          if [ -d /nix/var/nix/profiles/default/bin ]; then echo "/nix/var/nix/profiles/default/bin" >> "$GITHUB_PATH"; fi
+        fi
 
     - name: Environment diagnostic
       run: |
         echo "=== OS ==="; uname -a
         # Activate the uv-managed venv so ``.venv/bin`` (where
         # ``uv sync --all-extras`` installs pyinfra / ansible / pip /
-        # etc.) is on PATH, matching what ``uv run pytest`` sees.
+        # etc.) is on PATH, matching what ``uv run pytest`` sees. On
+        # Windows the executable scripts go into ``.venv/Scripts`` instead.
         if [ -d .venv/bin ]; then
           export PATH="$PWD/.venv/bin:$PATH"
           echo "=== venv === $PWD/.venv"
         fi
+        if [ -d .venv/Scripts ]; then
+          export PATH="$PWD/.venv/Scripts:$PATH"
+          echo "=== venv === $PWD/.venv (Windows layout)"
+        fi
         echo "=== PATH ==="; echo "$PATH" | tr ':' '\n'
-        for bin in python pip uv node npm pnpm yarn bun deno go gem cargo rustc brew apt-get dpkg docker nix nix-env ansible ansible-playbook pyinfra sh bash; do
+        for bin in python pip uv node npm pnpm yarn bun deno go gem cargo rustc brew apt-get dpkg docker nix nix-env ansible ansible-playbook pyinfra scoop sh bash; do
           path=$(command -v "$bin" 2>/dev/null || true)
           if [ -z "$path" ]; then
             echo "=== $bin === (not installed)"

abxpkg/__init__.py

@@ -67,6 +67,8 @@
 from .binprovider_puppeteer import PuppeteerProvider
 from .binprovider_playwright import PlaywrightProvider
 from .binprovider_bash import BashProvider
+from .binprovider_scoop import ScoopProvider
+from .windows_compat import IS_WINDOWS, UNIX_ONLY_PROVIDER_NAMES
 
 ALL_PROVIDERS = [
     EnvProvider,
@@ -90,6 +92,7 @@
     PyinfraProvider,
     ChromeWebstoreProvider,
     BashProvider,
+    ScoopProvider,
 ]
 
 
@@ -105,12 +108,18 @@ def _provider_class(provider: type[BinProvider] | BinProvider) -> type[BinProvid
 ]  # PipProvider, AptProvider, BrewProvider, etc.
 
 
-# Default provider names: names of providers that are enabled by default based on the current OS
+# Default provider names: names of providers that are enabled by default based on the current OS.
+# On Windows we also drop everything in ``UNIX_ONLY_PROVIDER_NAMES`` (apt,
+# brew, nix, bash, ansible, pyinfra, docker) since none of them have a
+# working Windows backend, and we drop ``scoop`` on non-Windows hosts
+# since it's Windows-only.
 DEFAULT_PROVIDER_NAMES = [
     provider_name
     for provider_name in ALL_PROVIDER_NAMES
     if not (OPERATING_SYSTEM == "darwin" and provider_name == "apt")
     and provider_name not in ("ansible", "pyinfra")
+    and not (IS_WINDOWS and provider_name in UNIX_ONLY_PROVIDER_NAMES)
+    and not (not IS_WINDOWS and provider_name == "scoop")
 ]
 
 # Lazy provider singletons: maps provider name -> class
@@ -204,6 +213,7 @@ def __getattr__(name: str):
     "PuppeteerProvider",
     "PlaywrightProvider",
     "BashProvider",
+    "ScoopProvider",
     # Note: provider singleton names (apt, pip, brew, etc.) are intentionally
     # excluded from __all__ so that `from abxpkg import *` does not eagerly
     # instantiate every provider. Use explicit imports instead:

abxpkg/base_types.py

@@ -74,9 +74,9 @@ def validate_bin_dir(path: Path) -> Path:
 
 
 def validate_PATH(PATH: str | list[str]) -> str:
-    paths = PATH.split(":") if isinstance(PATH, str) else list(PATH)
+    paths = PATH.split(os.pathsep) if isinstance(PATH, str) else list(PATH)
     assert all(Path(bin_dir) for bin_dir in paths)
-    return ":".join(paths).strip(":")
+    return os.pathsep.join(paths).strip(os.pathsep)
 
 
 PATHStr = Annotated[str, BeforeValidator(validate_PATH)]
@@ -218,7 +218,7 @@ def bin_abspath(
         # print(bin_path_or_name, PATH.split(':'), binpath, 'GOPINGNGN')
         if not binpath:
             # some bins dont show up with shutil.which (e.g. django-admin.py)
-            for path in PATH.split(":"):
+            for path in PATH.split(os.pathsep):
                 bin_dir = Path(path)
                 # print('BIN_DIR', bin_dir, bin_dir.is_dir())
                 if not (os.path.isdir(bin_dir) and os.access(bin_dir, os.R_OK)):
@@ -258,7 +258,7 @@ def bin_abspaths(
         abspaths.append(Path(bin_path_or_name).expanduser().absolute())
     else:
         # not a path yet, get path using shutil.which
-        for path in PATH.split(":"):
+        for path in PATH.split(os.pathsep):
             binpath = shutil.which(bin_path_or_name, mode=os.X_OK, path=path)
             if binpath and str(Path(binpath).parent) in PATH:
                 abspaths.append(binpath)

abxpkg/binary.py

@@ -1,5 +1,6 @@
 __package__ = "abxpkg"
 
+import os
 from typing import Any
 from typing import Self
 
@@ -189,7 +190,7 @@ def loaded_abspaths(self) -> dict[BinProviderName, list[HostBinPath]]:
     @property
     def loaded_bin_dirs(self) -> dict[BinProviderName, PATHStr]:
         return {
-            provider_name: ":".join(
+            provider_name: os.pathsep.join(
                 [str(bin_abspath.parent) for bin_abspath in bin_abspaths],
             )
             for provider_name, bin_abspaths in self.loaded_abspaths.items()