Skip to content

Commit 73c75ae

Browse files
romtsnclaude
andcommitted
fix(ci): Scope napi-postinstall to release branch only
Only the release path uses --omit=optional which skips native bindings. The regular npm ci path installs them via the lockfile. Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
1 parent 068e571 commit 73c75ae

File tree

1 file changed

+4
-3
lines changed

1 file changed

+4
-3
lines changed

.github/workflows/test_node.yml

Lines changed: 4 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -96,9 +96,10 @@ jobs:
9696
if: ${{ inputs.triggered-by-release }}
9797
run: npm install --omit=optional --ignore-scripts
9898

99-
# --ignore-scripts skips napi-postinstall which installs native resolver
100-
# bindings required by Jest 30's @unrs/resolver
101-
- name: Install native bindings
99+
# --omit=optional skips native resolver bindings required by Jest 30.
100+
# napi-postinstall installs the correct platform-specific binary.
101+
- name: Install native bindings (for pushes to release branch)
102+
if: ${{ inputs.triggered-by-release }}
102103
run: npx napi-postinstall unrs-resolver
103104

104105
- name: Download build artifact

0 commit comments

Comments
 (0)