ci: test new napi build workflow

Author: JounQin

.github/actions/pnpm/action.yml

@@ -2,6 +2,11 @@ name: Pnpm
 
 description: Install pnpm
 
+inputs:
+  architecture:
+    description: The architecture to use
+    required: false
+
 runs:
   using: composite
   steps:
@@ -10,8 +15,13 @@ runs:
     - uses: actions/setup-node@v4
       with:
         node-version-file: .node-version
+        architecture: ${{ inputs.architecture }}
         cache: pnpm
 
+    - if: ${{ inputs.architecture == 'x86' }}
+      shell: bash
+      run: pnpm config set supportedArchitectures.cpu "ia32"
+
     - name: pnpm
       shell: bash
       run: |

.github/workflows/release-napi.yml

@@ -7,6 +7,7 @@ on:
       - main
     paths:
       - npm/package.json # Please only commit this file, so we don't need to wait for test CI to pass.
+  pull_request: null
 
 env:
   DEBUG: "napi:*"
@@ -41,7 +42,7 @@ jobs:
 
   build:
     needs: check
-    if: needs.check.outputs.version_changed == 'true'
+    # if: needs.check.outputs.version_changed == 'true'
     env:
       version: ${{ needs.check.outputs.version }}
     outputs:
@@ -52,41 +53,67 @@ jobs:
         include:
           - os: windows-latest
             target: x86_64-pc-windows-msvc
+            build: pnpm build
           - os: windows-latest
             target: aarch64-pc-windows-msvc
+            build: pnpm build
           - os: windows-latest
             target: i686-pc-windows-msvc
+            architecture: x86
+            build: pnpm build
           - os: ubuntu-latest
             target: x86_64-unknown-linux-gnu
+            build: pnpm build --use-napi-cross
           - os: ubuntu-latest
             target: x86_64-unknown-linux-musl
+            build: pnpm build -x
           - os: ubuntu-latest
             target: aarch64-unknown-linux-gnu
+            build: pnpm build --use-napi-cross
           - os: ubuntu-latest
             target: aarch64-unknown-linux-musl
+            build: pnpm build -x
           - os: ubuntu-latest
             target: armv7-unknown-linux-gnueabihf
+            build: pnpm build --use-napi-cross
           - os: ubuntu-latest
             target: armv7-unknown-linux-musleabihf
+            build: pnpm build -x
           - os: ubuntu-latest
             target: riscv64gc-unknown-linux-gnu
+            setup: |
+              sudo apt-get update
+              sudo apt-get install gcc-riscv64-linux-gnu -y
+              echo "CXX=riscv64-linux-gnu-gcc" >> $GITHUB_ENV
+            build: pnpm build --use-napi-cross
           - os: ubuntu-latest
             target: powerpc64le-unknown-linux-gnu
+            build: pnpm build --use-napi-cross
           - os: ubuntu-latest
             target: s390x-unknown-linux-gnu
+            setup: |
+              sudo apt-get update
+              sudo apt-get install gcc-s390x-linux-gnu -y
+              echo "CXX=s390x-linux-gnu-gcc" >> $GITHUB_ENV
+            build: pnpm build
           - os: macos-latest
             target: x86_64-apple-darwin
+            build: pnpm build
           - os: macos-latest
             target: aarch64-apple-darwin
+            build: pnpm build
           - os: ubuntu-latest
             target: wasm32-wasip1-threads
+            build: pnpm build
 
     name: Package ${{ matrix.target }}
     runs-on: ${{ matrix.os }}
     steps:
       - uses: taiki-e/checkout-action@v1
 
       - uses: ./.github/actions/pnpm
+        with:
+          architecture: ${{ matrix.architecture }}
 
       - run: rustup target add ${{ matrix.target }}
 
@@ -95,21 +122,24 @@ jobs:
         with:
           version: 0.13.0
 
-      - name: Build with zig cross
+      - name: Install cargo-zigbuild
+        uses: taiki-e/install-action@v2
         if: ${{ contains(matrix.target, 'musl') }}
-        run: pnpm build -x --target ${{ matrix.target }}
-
-      - name: Build with napi cross
-        if: ${{ contains(matrix.target, 'gnu') }}
         env:
-          CC: clang # for mimalloc
-        run: pnpm build --use-napi-cross --target ${{ matrix.target }}
+          GITHUB_TOKEN: ${{ github.token }}
+        with:
+          tool: cargo-zigbuild
+
+      - name: Setup toolchain
+        if: ${{ matrix.setup }}
+        run: ${{ matrix.setup }}
+        shell: bash
 
       - name: Build
-        if: ${{ !contains(matrix.target, 'gnu') && !contains(matrix.target, 'musl') }}
+        run: ${{ matrix.build }} --target ${{ matrix.target }}
+        shell: bash
         env:
-          CC: clang # for mimalloc
-        run: pnpm build --target ${{ matrix.target }}
+          CC: ${{ matrix.target != 'riscv64gc-unknown-linux-gnu' && matrix.target != 's390x-unknown-linux-gnu' && 'clang' || '' }}
 
       - name: Upload artifacts
         uses: actions/upload-artifact@v4
@@ -118,10 +148,11 @@ jobs:
           path: |
             napi/*.node
             napi/*.wasm
+          if-no-files-found: error
 
   build-freebsd:
     needs: check
-    if: needs.check.outputs.version_changed == 'true'
+    # if: needs.check.outputs.version_changed == 'true'
     name: Build FreeBSD
     runs-on: ubuntu-latest
     steps:
@@ -171,8 +202,10 @@ jobs:
     permissions:
       id-token: write # for `npm publish --provenance`
     needs:
+      - check
       - build
       - build-freebsd
+    if: needs.check.outputs.version_changed == 'true'
     env:
       COREPACK_INTEGRITY_KEYS: 0
     steps:

package.json

@@ -10,8 +10,8 @@
     "url": "https://github.com/sponsors/Boshen"
   },
   "scripts": {
-    "build": "napi build --platform --release --package-json-path npm/package.json --manifest-path napi/Cargo.toml",
-    "build:debug": "napi build --platform --package-json-path npm/package.json --manifest-path napi/Cargo.toml",
+    "build": "napi build --platform --release --strip --package-json-path npm/package.json --manifest-path napi/Cargo.toml",
+    "build:debug": "napi build --platform --strip --package-json-path npm/package.json --manifest-path napi/Cargo.toml",
     "prepublishOnly": "napi pre-publish -t npm",
     "test": "vitest run -r ./napi"
   },