Uh oh!
There was an error while loading. Please reload this page.
chore(deps): bump the js-npm group in /crates/bashkit-js with 2 updates - #2322
Closed
dependabot[bot] wants to merge 1 commit into
Closed
chore(deps): bump the js-npm group in /crates/bashkit-js with 2 updates#2322dependabot[bot] wants to merge 1 commit into
dependabot[bot] wants to merge 1 commit into
Conversation
Bumps the js-npm group in /crates/bashkit-js with 2 updates: [ava](https://github.com/avajs/ava) and [oxlint](https://github.com/oxc-project/oxc/tree/HEAD/npm/oxlint). Updates `ava` from 6.4.1 to 8.0.1 - [Release notes](https://github.com/avajs/ava/releases) - [Commits](avajs/ava@v6.4.1...v8.0.1) Updates `oxlint` from 1.78.0 to 1.79.0 - [Release notes](https://github.com/oxc-project/oxc/releases) - [Changelog](https://github.com/oxc-project/oxc/blob/main/npm/oxlint/CHANGELOG.md) - [Commits](https://github.com/oxc-project/oxc/commits/oxlint_v1.79.0/npm/oxlint) --- updated-dependencies: - dependency-name: ava dependency-version: 8.0.1 dependency-type: direct:development update-type: version-update:semver-major dependency-group: js-npm - dependency-name: oxlint dependency-version: 1.79.0 dependency-type: direct:development update-type: version-update:semver-minor dependency-group: js-npm ... Signed-off-by: dependabot[bot] <support@github.com>
Deploying with |
| Status | Name | Latest Commit | Preview URL | Updated (UTC) |
|---|---|---|---|---|
| ✅ Deployment successful! View logs | bashkit | 3e8cf4c | Commit Preview URL Branch Preview URL | Aug 21 2026, 10:22 PM |
2 tasks
chaliy
commented
Aug 22, 2026
Contributor
Superseded by #2339, which carries the same
Closing in favour of #2339. Generated by Claude Code |
ContributorAuthor
This pull request was built based on a group rule. Closing it will not ignore any of these versions in future pull requests. To ignore these dependencies, configure ignore rules in dependabot.yml |
dependabotBot
deleted the
dependabot/npm_and_yarn/crates/bashkit-js/js-npm-9040fccdb7
branch
August 22, 2026 00:43
chaliy added a commit
that referenced
this pull request
Aug 22, 2026
) ## What changed Takes the grouped `js-npm` Dependabot update (`ava` 6.4.1 → 8.0.1, `oxlint` 1.78.0 → 1.79.0) and adds the fixups the bot could not make on its own. - `ava.extensions` moves from the object form to the array form — ava 8 removed the object form. - The two ava-hosted CI steps (`pnpm test`, `pnpm run test:interop`) now run on Node 22+ only. ava 8 requires Node 22.20+. - The random-fs interop spec becomes `interop.spec.mts` so it loads as ESM. No runtime or published-package change: `ava` and `oxlint` are both devDependencies. ## Why Dependabot opened #2322 for this group, but the `ava` bump is a breaking major and the PR fails CI as-is: ``` ✘ The extensions option must be an array ELIFECYCLE Test failed. ``` Two further breakages surface only after that one is fixed: 1. ava 8 requires Node 22.20+, so the `node 20` matrix job cannot run it at all. 2. ava 8 loads every test file through `import()`. `test-fixtures/random-fs/` has its own `package.json` with no `"type": "module"`, so `interop.spec.ts` resolved as CommonJS and ava's now ESM-only worker failed to initialise: ``` AssertionError [ERR_ASSERTION]: The expression evaluated to a falsy value: assert.ok(refs.runnerChain) at .../ava@8.0.1/node_modules/ava/lib/worker/main.js:11:21 at Module._compile (node:internal/modules/cjs/loader:1705:14) ``` Renaming to `.mts` makes the file ESM regardless of the nearest `package.json`, and leaves the napi-generated CommonJS `index.js` alone — the spec already reaches it through `createRequire`. **Node 20 stays in the JS matrix.** The package still declares `engines: node >= 18`, and node 20 continues to run the runtime-compat tests and all six examples, which exercise the shipped binding. Only the dev-time test runner moves forward. ## Before / After Before (Dependabot #2322, node 20/22/24/latest all red): ``` > @everruns/bashkit@0.16.0 test > ava ✘ The extensions option must be an array ELIFECYCLE Test failed. See above for more details. ``` After, on Node 22.22.2: ``` $ pnpm test 569 tests passed $ pnpm run test:interop √ downstream random filesystem external mounts into bash 1 test passed $ pnpm exec tsc --noEmit -p tsconfig.check.json (clean) $ pnpm run lint (3 pre-existing warnings, unchanged) ``` `pnpm install --frozen-lockfile` applies the `supertap@3.0.1` patch unchanged — ava 8 still resolves that exact version. ## Risk - Low - Node 20 no longer runs the ava suites. Runtime-compat tests and the examples still cover node 20, so a node-20 regression in the shipped binding is still caught; a node-20-only regression in behaviour covered *solely* by the ava suite would not be. - `tsconfig.check.json` now globs `test-fixtures/**/*.spec.mts`; verified with `tsc --listFiles` that the renamed spec is still type-checked. ## Checklist - [x] Tests added or updated - [x] Backward compatibility considered Closes#2322. --- _Generated by [Claude Code](https://claude.ai/code/session_01GbhUV1eSeVM2DcwzPfZooS)_
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for freeto join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Bumps the js-npm group in /crates/bashkit-js with 2 updates: ava and oxlint.
Updates
avafrom 6.4.1 to 8.0.1Release notes
Sourced from ava's releases.
... (truncated)
Commits
bbfd946v8.0.1332fcd7Pre-release updates (#3451)fcaaf43Add Node.js 26 support; drop 25 (#3450)a913804Test using AVA 8 (#3440)fe31286v8.0.07bdbabbImprove error whenthrowsAsync/notThrowsAsyncis not awaited (#3436)a304990Full ESM internals; bias away from CJS; require@ava/typescript@7d012c64Update release process documentation (#3435)2cea71eCI tweaks (#3438)27e9c84Remove Node.js 20 support; update dependencies prior to release (#3432)Maintainer changes
This version was pushed to npm by GitHub Actions, a new releaser for ava since your current version.
Updates
oxlintfrom 1.78.0 to 1.79.0Changelog
Sourced from oxlint's changelog.
Commits
0db127crelease(apps): oxlint v1.79.0 && oxfmt v0.64.0 (#25866)228e8e0fix(linter): resolve inactive React compiler rules (#25830)aa49d86fix(linter): allow spread rule options in config types (#25675)8922381refactor(linter): remove inactive react config rule (#25740)8c4552dfeat(linter)!: split react/react-compiler into per-category rules (#25500)36f8451fix(linter/eslint/no-eval): align indirect default with ESLint (#25656)beb724dfix(linter/eslint/no-unused-vars): report bare underscore parameters (#25663)4004c10fix(linter/eslint/no-irregular-whitespace): check comments by default (#25660)285820efix(linter/no-large-snapshots): precompile and document allowed snapshot matc...4df5835fix(linter): allow capitalized built-in calls (#25516)Dependabot will resolve any conflicts with this PR as long as you don't alter it yourself. You can also trigger a rebase manually by commenting
@dependabot rebase.Dependabot commands and options
You can trigger Dependabot actions by commenting on this PR:
@dependabot rebasewill rebase this PR@dependabot recreatewill recreate this PR, overwriting any edits that have been made to it@dependabot show <dependency name> ignore conditionswill show all of the ignore conditions of the specified dependency@dependabot ignore <dependency name> major versionwill close this group update PR and stop Dependabot creating any more for the specific dependency's major version (unless you unignore this specific dependency's major version or upgrade to it yourself)@dependabot ignore <dependency name> minor versionwill close this group update PR and stop Dependabot creating any more for the specific dependency's minor version (unless you unignore this specific dependency's minor version or upgrade to it yourself)@dependabot ignore <dependency name>will close this group update PR and stop Dependabot creating any more for the specific dependency (unless you unignore this specific dependency or upgrade to it yourself)@dependabot unignore <dependency name>will remove all of the ignore conditions of the specified dependency@dependabot unignore <dependency name> <ignore condition>will remove the ignore condition of the specified dependency and ignore conditions