Skip to content

Commit 11780db

Browse files
committed
ci: use npm ci for reproducible installs across all jobs
All five CI jobs ran 'npm install', which resolves caret-ranges against the live npm registry and ignores package-lock.json. This caused dependency drift between local runs and CI — most recently suspected as a contributor to the SaaS 8.7 hard-fail in Worker-Failure-Retries.spec.ts (passes locally with the locked grpc-js, fails in CI with whatever the resolver picked). Switching to 'npm ci' makes CI installs deterministic against package-lock.json. If we still see the 8.7 failure after this lands, we can rule dependency drift out and look elsewhere (Node 22 vs local Node, GHA runner network/TLS, etc.).
1 parent 9b8dc3d commit 11780db

1 file changed

Lines changed: 5 additions & 5 deletions

File tree

.github/workflows/ci.yml

Lines changed: 5 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -28,7 +28,7 @@ jobs:
2828
node-version: "22"
2929

3030
- name: Install dependencies
31-
run: npm install
31+
run: npm ci
3232

3333
- name: Build and Smoke Test
3434
run: npm run test:smoketest
@@ -51,7 +51,7 @@ jobs:
5151
node-version: "22"
5252

5353
- name: Install dependencies
54-
run: npm install
54+
run: npm ci
5555

5656
# Workaround for https://github.com/actions/runner-images/issues/2821
5757
- name: Remove mono blocking 8084 port
@@ -100,7 +100,7 @@ jobs:
100100
node-version: "22"
101101

102102
- name: Install dependencies
103-
run: npm install
103+
run: npm ci
104104

105105
# Workaround for https://github.com/actions/runner-images/issues/2821
106106
- name: Remove mono blocking 8084 port
@@ -153,7 +153,7 @@ jobs:
153153
node-version: "22"
154154

155155
- name: Install dependencies
156-
run: npm install
156+
run: npm ci
157157

158158
- name: Run Integration Tests
159159
run: |
@@ -190,7 +190,7 @@ jobs:
190190
node-version: "22"
191191

192192
- name: Install dependencies
193-
run: npm install
193+
run: npm ci
194194

195195
- name: Run Integration Tests
196196
run: |

0 commit comments

Comments
 (0)