Skip to content

chore: upgrade @mongodb-js/zstd from 2.0.0 to 2.0.1#1617

Merged
rlinehan merged 1 commit into
mainfrom
rlinehan/codeql-fix
Jun 5, 2026
Merged

chore: upgrade @mongodb-js/zstd from 2.0.0 to 2.0.1#1617
rlinehan merged 1 commit into
mainfrom
rlinehan/codeql-fix

Conversation

@rlinehan

@rlinehan rlinehan commented Jun 5, 2026

Copy link
Copy Markdown
Contributor

The CodeQL github action was failing on other prs and on main with:

npm error code 127
npm error path /home/runner/work/client-sdk-javascript/client-sdk-javascript/packages/client-sdk-nodejs-compression-zstd/node_modules/@mongodb-js/zstd
npm error command failed
npm error command sh -c prebuild-install --runtime napi || npm run clean-install
npm error > @mongodb-js/zstd@2.0.0 clean-install
npm error > npm run install-zstd && npm run compile
npm error
npm error
npm error > @mongodb-js/zstd@2.0.0 install-zstd
npm error > bash etc/install-zstd.sh
npm error prebuild-install warn This package does not support N-API version undefined
npm error prebuild-install warn install No prebuilt binaries found (target=undefined runtime=napi arch=x64 libc= platform=linux)
npm error bash: etc/install-zstd.sh: No such file or directory

claude suggested that 2.0.0 has a broken fallback install that references a missing etc/install-zstd.sh script, causing npm ci to fail in CI environments where no prebuilt binary is available. This seems to have fixed the issue.

claude suggested that 2.0.0 has a broken fallback install that references a missing etc/install-zstd.sh script, causing npm ci to fail in CI environments where no prebuilt binary is available. Let's try it.
@rlinehan rlinehan marked this pull request as ready for review June 5, 2026 18:13
@rlinehan rlinehan merged commit 0f4fc20 into main Jun 5, 2026
11 checks passed
@rlinehan rlinehan deleted the rlinehan/codeql-fix branch June 5, 2026 18:32
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants