Skip to content

fix(deps): bump eslint to ^10.0.0 — unblocks @eslint/js 10.x (CI run #443) - #76

Draft
Sbussiso wants to merge 2 commits into
dependabot/npm_and_yarn/frontend/eslint/js-10.0.1from
ci-fix/28824401910
Draft

fix(deps): bump eslint to ^10.0.0 — unblocks @eslint/js 10.x (CI run #443)#76
Sbussiso wants to merge 2 commits into
dependabot/npm_and_yarn/frontend/eslint/js-10.0.1from
ci-fix/28824401910

Conversation

@Sbussiso

@Sbussiso Sbussiso commented Jul 6, 2026

Copy link
Copy Markdown
Contributor

What

Fixes the CI failure on Dependabot PR #75 (build(deps-dev): bump @eslint/js from 9.39.4 to 10.0.1) — GitHub Actions run #28824401910, job Frontend audit + build, step Install dependencies.

Root cause

Dependabot bumped @eslint/js to 10.0.1 but left eslint pinned at ^9.39.4. @eslint/js@10.0.1 declares peerOptional eslint: "^10.0.0", which conflicts with eslint@9.39.4, so npm ci fails with:

npm error ERESOLVE could not resolve
npm error While resolving: @eslint/js@10.0.1
npm error Found: eslint@9.39.4
npm error Could not resolve dependency:
npm error   peerOptional eslint "^10.0.0" from @eslint/js@10.0.1

This is a Dependabot-known limitation: it bumps one package at a time and doesn't account for the new peer dependency @eslint/js@10 added on its sibling eslint (see eslint/eslint#20467, which added eslint to @eslint/js's peerDependencies at the 10.0.0 release).

Fix

Align eslint to ^10.0.0 (resolves to 10.6.0) so the peer constraint is satisfied. This supersedes / absorbs Dependabot PR #36 (build(deps-dev): bump eslint from 9.39.4 to 10.6.0) — both bumps end at the same eslint@10.6.0.

Verification (run in sandbox against this branch)

All four Frontend audit + build gates exercised locally:

Gate Result
npm ci ✅ exit 0, 0 vulnerabilities
npx eslint . ✅ exit 0 — warnings only (no errors)
npm test (vitest) ✅ 85/85 passed (10 files)
npm run build ✅ success (343ms)

Lint produces no new errors under eslint 10. The warnings are the same react-hooks/exhaustive-deps and react-refresh/only-export-components advisories already present under eslint 9, plus the react-hooks/set-state-in-effect advisory the v7 plugin already emits — all already demoted to warn in eslint.config.js. No source changes were required; the flat config is compatible with both eslint 9 and 10.

Notes for review

dependabot Bot and others added 2 commits July 6, 2026 21:26
Bumps [@eslint/js](https://github.com/eslint/eslint/tree/HEAD/packages/js) from 9.39.4 to 10.0.1.
- [Release notes](https://github.com/eslint/eslint/releases)
- [Commits](https://github.com/eslint/eslint/commits/v10.0.1/packages/js)

---
updated-dependencies:
- dependency-name: "@eslint/js"
  dependency-version: 10.0.1
  dependency-type: direct:development
  update-type: version-update:semver-major
...

Signed-off-by: dependabot[bot] <support@github.com>
Dependabot bumped @eslint/js to 10.0.1 (PR #75) but left eslint pinned
at ^9.39.4. @eslint/js@10.0.1 declares peerOptional eslint ^10.0.0, so
`npm ci` fails with ERESOLVE:

  While resolving: @eslint/js@10.0.1
  Found: eslint@9.39.4
  Could not resolve dependency:
    peerOptional eslint "^10.0.0" from @eslint/js@10.0.1

This unblocks the Dependabot PR by aligning eslint to ^10.0.0
(resolves to 10.6.0). Verified locally against the exact CI gates:
  - npm ci ................ exit 0, 0 vulnerabilities
  - npx eslint . .......... exit 0 (warnings only, no errors)
  - npm test (vitest) ..... 85/85 passed
  - npm run build ......... success (343ms)

Note: eslint 10 is a major bump. The lint run against the new engine
produces only warnings (no errors) — the same react-hooks advisory
warnings already present under eslint 9, plus a new
`react-hooks/set-state-in-effect` advisory from the v7 plugin. No
source changes were needed; eslint.config.js already targets the v9+
flat config and is compatible with v10.

Relates to: PR #75, run 28824401910
@dependabot
dependabot Bot force-pushed the dependabot/npm_and_yarn/frontend/eslint/js-10.0.1 branch from ef86842 to 6d5840f Compare August 3, 2026 21:27
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.

2 participants