Skip to content

Commit e75c8ca

Browse files
CaelmBleiddclaude
andcommitted
Fix ci-ets: pin @types/node@18 for the ArkAnalyzer build
ArkAnalyzer's npm install pulls the floating latest @types/node, whose new d.ts files (e.g. ffi.d.ts) use syntax that ArkAnalyzer's bundled TypeScript cannot parse, so `npm run build` fails with TS1139/TS1005 errors and the whole ci-ets job dies before any Gradle test runs. Pin a compatible @types/node major right after npm install. Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
1 parent dfb9993 commit e75c8ca

1 file changed

Lines changed: 4 additions & 0 deletions

File tree

.github/workflows/ci.yml

Lines changed: 4 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -135,6 +135,10 @@ jobs:
135135
cd $DEST_DIR
136136
137137
npm install
138+
# ArkAnalyzer's bundled TypeScript cannot parse the d.ts files of the
139+
# latest floating @types/node (e.g. ffi.d.ts uses newer syntax and
140+
# breaks `npm run build` with TS1139 etc.); pin a compatible major.
141+
npm install --no-save @types/node@18
138142
npm run build
139143
140144
- name: Run ETS tests

0 commit comments

Comments
 (0)