Skip to content

Commit c1351ce

Browse files
committed
Block publishing until all 5 native addon prebuilds are verified
The verify-prebuilds job was just printing ls output and had no teeth. It now checks that all expected .node files are present and fails if any are missing. Publish is gated behind this check, so a broken or incomplete matrix build can never reach npm.
1 parent 886a9f1 commit c1351ce

1 file changed

Lines changed: 12 additions & 3 deletions

File tree

.github/workflows/release.yaml

Lines changed: 12 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -57,13 +57,22 @@ jobs:
5757
pattern: native-addon-*
5858
path: prebuilds
5959
merge-multiple: true
60-
- name: list prebuilds
61-
run: ls -lh prebuilds/
60+
- name: verify all platforms are present
61+
run: |
62+
expected="fuzzer-darwin-arm64.node fuzzer-darwin-x64.node fuzzer-linux-arm64.node fuzzer-linux-x64.node fuzzer-win32-x64.node"
63+
for f in $expected; do
64+
if [ ! -f "prebuilds/$f" ]; then
65+
echo "MISSING: $f"
66+
exit 1
67+
fi
68+
done
69+
echo "All 5 prebuilds present:"
70+
ls -lh prebuilds/
6271
6372
publish:
6473
name: publish
6574
if: startsWith(github.ref, 'refs/tags/')
66-
needs: [prebuilds]
75+
needs: [verify-prebuilds]
6776
runs-on: ubuntu-latest
6877
permissions:
6978
contents: write # for creating releases

0 commit comments

Comments
 (0)