Update release.yml to properly select between bash and alpine chroot (#187)

Author: esimkowitz

.github/workflows/release.yml

@@ -35,12 +35,13 @@ jobs:
           - host: ubuntu-latest
             target: x86_64-unknown-linux-musl
             npm_platform: linux-x64-musl
-            docker: node:22-alpine
+            alpine: true
+            alpine_arch: x86_64
           - host: ubuntu-latest
             target: aarch64-unknown-linux-musl
             npm_platform: linux-arm64-musl
-            docker: node:22-alpine
-            docker_arch: linux/arm64
+            alpine: true
+            alpine_arch: aarch64
     steps:
       # Speed up apt-get operations by disabling unnecessary man-db auto-update
       - name: Disable man-db auto-update
@@ -60,59 +61,34 @@ jobs:
           cache: false
       - name: Setup Task runner
         uses: go-task/setup-task@v1
-      - name: Install Linux dependencies
-        if: runner.os == 'Linux' && !matrix.settings.docker
+      - name: Setup Alpine Linux
+        if: matrix.settings.alpine
+        uses: jirutka/setup-alpine@v1
+        with:
+          arch: ${{ matrix.settings.alpine_arch }}
+          packages: >
+            build-base
+            cups-dev
+            nodejs
+            npm
+            go-task
+      - name: Install Linux dependencies (glibc)
+        if: runner.os == 'Linux' && !matrix.settings.alpine
         run: |
           sudo apt-get update
           sudo apt-get install -y libcups2-dev pkg-config clang
+      - name: Prepare Rust toolchain (musl)
+        if: matrix.settings.alpine
+        shell: alpine.sh {0}
+        run: rustup target add ${{ matrix.settings.target }}
       - name: Install dependencies
-        if: ${{ !matrix.settings.docker }}
+        shell: ${{ matrix.settings.alpine && 'alpine.sh {0}' || 'bash' }}
         run: npm ci
-      - name: Build (native)
-        if: ${{ !matrix.settings.docker }}
-        shell: bash
+      - name: Build N-API module
+        shell: ${{ matrix.settings.alpine && 'alpine.sh {0}' || 'bash' }}
         run: |
-          # Our build script auto-detects the platform and builds for the current target
           echo "Building for target: ${{ matrix.settings.target }}"
           task build:napi
-      - name: Setup QEMU for cross-compilation
-        if: matrix.settings.docker && matrix.settings.docker_arch
-        uses: docker/setup-qemu-action@v3
-        with:
-          platforms: ${{ matrix.settings.docker_arch }}
-      - name: Build (Docker/musl)
-        if: matrix.settings.docker
-        uses: addnab/docker-run-action@v3
-        with:
-          image: ${{ matrix.settings.docker }}
-          options: --user 0:0 --platform ${{ matrix.settings.docker_arch || 'linux/amd64' }} -v ${{ github.workspace }}/.cargo-cache/git/db:/usr/local/cargo/git/db -v ${{ github.workspace }}/.cargo/registry/cache:/usr/local/cargo/registry/cache -v ${{ github.workspace }}/.cargo/registry/index:/usr/local/cargo/registry/index -v ${{ github.workspace }}:/build -w /build
-          run: |
-            set -e
-
-            # Install build dependencies
-            apk add --no-cache \
-              cups-dev \
-              pkgconf \
-              clang \
-              lld \
-              musl-dev \
-              rust \
-              cargo \
-              git
-
-            # Add Rust target
-            rustup target add ${{ matrix.settings.target }}
-
-            # Enable corepack and install Node.js dependencies
-            corepack enable
-            npm ci
-
-            # Build the N-API module
-            npx napi build --platform --release --target ${{ matrix.settings.target }} --esm --output-dir npm/${{ matrix.settings.npm_platform }}
-
-            # Run post-build scripts
-            node scripts/remove-env-check.js --dir=npm/${{ matrix.settings.npm_platform }}
-            node scripts/fix-platform-packages.js
       - name: Post-build cleanup
         shell: bash
         run: |
@@ -150,22 +126,32 @@ jobs:
     strategy:
       fail-fast: false
       matrix:
-        # We can't run the cross-runtime tests on Windows ARM runners because Deno and Bun do not have ARM64 Windows builds
         settings:
           - host: macos-13
             target: x86_64-apple-darwin
           - host: macos-14
             target: aarch64-apple-darwin
           - host: windows-latest
             target: x86_64-pc-windows-msvc
+          - host: windows-11-arm
+            target: aarch64-pc-windows-msvc
+            test_runtimes: rust node
           - host: ubuntu-latest
             target: x86_64-unknown-linux-gnu
           - host: ubuntu-24.04-arm
             target: aarch64-unknown-linux-gnu
+          - host: ubuntu-latest
+            target: x86_64-unknown-linux-musl
+            alpine: true
+            alpine_arch: x86_64
+          - host: ubuntu-latest
+            target: aarch64-unknown-linux-musl
+            alpine: true
+            alpine_arch: aarch64
     steps:
       # Speed up apt-get operations by disabling unnecessary man-db auto-update
-      - name: Disable man-db auto-update and install deps
-        if: runner.os == 'Linux'
+      - name: Disable man-db auto-update and install deps (glibc)
+        if: runner.os == 'Linux' && !matrix.settings.alpine
         run: |
           echo "man-db man-db/auto-update boolean false" | sudo debconf-set-selections
           sudo rm -f /var/lib/man-db/auto-update
@@ -191,12 +177,19 @@ jobs:
         uses: actions-rust-lang/setup-rust-toolchain@v1
         with:
           cache: false
-      # Install Task runner
       - name: Setup Task
         uses: arduino/setup-task@v2
         with:
           version: 3.x
           repo-token: ${{ secrets.GITHUB_TOKEN }}
+      - name: Setup Alpine Linux
+        if: matrix.settings.alpine
+        uses: jirutka/setup-alpine@v1
+        with:
+          arch: ${{ matrix.settings.alpine_arch }}
+          packages: >
+            cups-dev
+            go-task
       - name: Download all N-API binary artifacts
         uses: actions/download-artifact@v5
         with:
@@ -215,9 +208,18 @@ jobs:
           done
           ls -la npm/ 2>/dev/null || dir npm
       - name: Install Node.js dependencies
+        shell: ${{ matrix.settings.alpine && 'alpine.sh {0}' || 'bash' }}
         run: npm ci
       - name: Run comprehensive tests (includes compilation)
-        run: task test
+        shell: ${{ matrix.settings.alpine && 'alpine.sh {0}' || 'bash' }}
+        run: |
+          if [ -n "${{ matrix.settings.test_runtimes }}" ]; then
+            echo "Running selective runtime tests: ${{ matrix.settings.test_runtimes }}"
+            node scripts/test-runtimes.js ${{ matrix.settings.test_runtimes }}
+          else
+            echo "Running all runtime tests"
+            task test
+          fi
 
   publish:
     name: Publish to npm