chore(deps): update dependency ava to v8#1123
Conversation
|
@claude[agent] fix tests error after ava upgrade |
- Change ava extensions config from object to array format for v8 - Update CI Node.js version matrix from ['20', '22', '24'] to ['22', '24'] - Ava v8 requires Node.js >= 22.20 due to iterator helper methods Agent-Logs-Url: https://github.com/napi-rs/node-rs/sessions/8e1e8303-351c-476e-9f4d-3d39ff8d992e
Fixed! The test errors were due to two issues with the ava v8 upgrade:
Changes in commit 6950c09:
|
Edited/Blocked NotificationRenovate will not automatically rebase this PR, because it does not recognize the last commit author and assumes somebody else may have edited the PR. You can manually request rebase by checking the rebase/retry box above. |
|
@claude[agent] fix the bench CI fail |
|
@claude[agent] fix the bench CI fail |
- Remove sse4_crc32 and @types/sse4_crc32 from crc32 package dependencies - Update crc32 benchmark to exclude sse4_crc32 comparisons - sse4_crc32 v7.0.0 lacks prebuilt binaries for Node.js 24 (ABI v137) - Bench CI now passes on Node.js 24 Agent-Logs-Url: https://github.com/napi-rs/node-rs/sessions/1b2c25e3-3d62-43a9-bc72-9f521bb287a4
Fixed in 99eb245. Removed the |
|
@claude[agent] fix the bench CI fail |
The xxhash npm package v0.3.0 does not have prebuilt binaries for Node.js 24 (ABI version 137), causing benchmark failures with "Cannot find module '../build/Release/hash'" errors. Removed xxhash from devDependencies and updated benchmark/xxhash.ts to run benchmarks without the xxhash C++ comparison. The benchmark still includes comparisons with xxhash-addon, xxhashjs, and webpack's wasm implementation. This follows the same pattern used for sse4_crc32 and nodejieba dependencies. 🤖 Generated with [Claude Code](https://claude.com/claude-code) Co-Authored-By: Claude Sonnet 4.5 <noreply@anthropic.com>
The bench CI has been fixed in commits 9ba1b3a and 8d6e65f. Both bench jobs now pass successfully:
The benchmarks still run and compare against other implementations (xxhash-addon, xxhashjs, webpack's wasm). |
|
@claude[agent] armv7-unknown-linux doesn't have a node 24 docker image, omit it from test matrix |
The armv7-unknown-linux-gnueabihf platform doesn't have Node.js 24 docker images available. Restrict testing to Node.js 22 only for this platform. Agent-Logs-Url: https://github.com/napi-rs/node-rs/sessions/cee7816f-b688-4ae5-af92-6f51d48c387c Co-authored-by: Brooooooklyn <3468483+Brooooooklyn@users.noreply.github.com>
Fixed in commit cdedc1c. Excluded Node.js 24 from the armv7-unknown-linux-gnueabihf test matrix since there's no Node.js 24 docker image available for this platform. The armv7 tests now only run on Node.js 22. |
This PR contains the following updates:
^7.0.0→^8.0.0Release Notes
avajs/ava (ava)
v8.0.0Compare Source
Breaking Changes
AVA now expects Node.js 22.20, 24.12 or newer.
Internally AVA is now fully ESM. This is possible now that Node.js supports loading ES modules using
require()calls and simplifies AVA's types and internals.If you use AVA from a CommonJS project you'll have to update your imports:
We expect an increasing number of projects to be ESM only. As per the above, CommonJS is still supported, but we don't expect
cjsextensions to be used. The default file extensions are nowjsandmjs. Specifyextensions: ['cjs', 'js', 'mjs']for AVA to run test files with thecjsextension.All test files (and those loaded through AVA's
requireconfig) are now loaded viaimport(). Use customization hooks for transpilation. The object form of theextensionsconfiguration is no longer supported.If you use AVA with @ava/typescript you must upgrade that package to v7.
New Features
There's two new test modifiers courtesy of @sindresorhus:
test.skipIf()to skip a test based on a runtime condition.test.runIf()is the inverse: the test only runs when the condition is true.These work with other modifiers like
.serialand.failing:Other Changes
*.tsbuildinfofilesthrowsAsync/notThrowsAsyncare not awaited have been improved by @sindresorhus in #3436New Contributors
Full Changelog: avajs/ava@v7.0.0...v8.0.0
Configuration
📅 Schedule: (UTC)
🚦 Automerge: Disabled by config. Please merge this manually once you are satisfied.
♻ Rebasing: Whenever PR is behind base branch, or you tick the rebase/retry checkbox.
🔕 Ignore: Close this PR and you won't be reminded about this update again.
This PR was generated by Mend Renovate. View the repository job log.