build(deps): bump @typescript-eslint/parser from 6.17.0 to 8.59.0#1300
build(deps): bump @typescript-eslint/parser from 6.17.0 to 8.59.0#1300dependabot[bot] wants to merge 1 commit into
Conversation
Bumps [@typescript-eslint/parser](https://github.com/typescript-eslint/typescript-eslint/tree/HEAD/packages/parser) from 6.17.0 to 8.59.0. - [Release notes](https://github.com/typescript-eslint/typescript-eslint/releases) - [Changelog](https://github.com/typescript-eslint/typescript-eslint/blob/main/packages/parser/CHANGELOG.md) - [Commits](https://github.com/typescript-eslint/typescript-eslint/commits/v8.59.0/packages/parser) --- updated-dependencies: - dependency-name: "@typescript-eslint/parser" dependency-version: 8.59.0 dependency-type: direct:production update-type: version-update:semver-major ... Signed-off-by: dependabot[bot] <support@github.com>
Up to standards ✅🟢 Issues
|
| Metric | Results |
|---|---|
| Complexity | 0 |
| Duplication | 0 |
AI Reviewer: first review requested successfully. AI can make mistakes. Always validate suggestions.
TIP This summary will be updated as you push new changes.
There was a problem hiding this comment.
Pull Request Overview
This PR upgrades @typescript-eslint/parser to v8.59.0, which introduces breaking changes regarding environment requirements and peer dependencies. While the PR is up to standards according to Codacy, there are two significant issues that should be addressed before merging:
- Peer Dependency Mismatch: The new parser requires
eslint^8.57.0, but the manifest still specifies ^8.56.0. - Node.js Engine Conflict: New transitive dependencies (e.g.,
eslint-visitor-keysv5.0.1) require Node versions >=20.19.0, which conflicts with the project's current >=20.0.0 requirement.
Updating these fields in package.json is necessary to prevent installation failures and peer dependency warnings in CI/CD environments.
About this PR
- The updated lockfile includes several packages that have been marked as deprecated, including
eslintv8.57.1 and@humanwhocodes/config-array. While these are transitive or resolved versions, be aware that future upgrades to ESLint v9+ will likely be required to resolve these deprecations.
1 comment outside of the diff
package.json
line 61🟡 MEDIUM RISK
The project's Node.js requirement of>=20.0.0is now broader than its transitive dependencies. Specifically,eslint-visitor-keys(v5.0.1) requires^20.19.0 || ^22.13.0 || >=24. Update theengines.nodefield inpackage.jsonto^20.19.0 || >=22.0.0to align with the requirements of the new dependencies and prevent installation failures on older Node 20 patches.
Test suggestions
- Execute the project's linting process (e.g., 'npm run lint') to ensure no new parsing errors or rule regressions are introduced.
- Verify that 'npm install' or 'npm ci' completes without peer dependency warnings, specifically regarding the ESLint version mismatch.
- Validate the application on a Node.js 20.0.0 environment to ensure compatibility with transitive dependencies requiring higher minor versions (e.g., Node 20.19.0).
Prompt proposal for missing tests
Consider implementing these tests if applicable:
1. Execute the project's linting process (e.g., 'npm run lint') to ensure no new parsing errors or rule regressions are introduced.
2. Verify that 'npm install' or 'npm ci' completes without peer dependency warnings, specifically regarding the ESLint version mismatch.
3. Validate the application on a Node.js 20.0.0 environment to ensure compatibility with transitive dependencies requiring higher minor versions (e.g., Node 20.19.0).
TIP Improve review quality by adding custom instructions
TIP How was this review? Give us feedback
| "dependencies": { | ||
| "@nodelib/fs.walk": "^2.0.0", | ||
| "@typescript-eslint/parser": "^6.17.0", | ||
| "@typescript-eslint/parser": "^8.59.0", |
There was a problem hiding this comment.
🟡 MEDIUM RISK
The @typescript-eslint/parser v8.59.0 requires eslint version ^8.57.0 or higher as a peer dependency, but package.json currently specifies ^8.56.0. Update the eslint dependency to at least ^8.57.0 to satisfy the peer dependency of the new parser and prevent warnings in environments performing strict manifest validation.
Bumps @typescript-eslint/parser from 6.17.0 to 8.59.0.
Release notes
Sourced from @typescript-eslint/parser's releases.
... (truncated)
Changelog
Sourced from @typescript-eslint/parser's changelog.
... (truncated)
Commits
ea9ae4fchore(release): publish 8.59.090c2803chore(release): publish 8.58.2be6b49afix: remove tsbuildinfo cache file from published packages (#12187)5311ed3chore(release): publish 8.58.14933417chore(release): publish 8.58.08cde2d0feat: support TypeScript 6 (#12124)be4d54dchore(release): publish 8.57.2c7c38aachore(release): publish 8.57.12c6aeeechore(release): publish 8.57.0f696dadchore: use pnpm catalog (#12047)Maintainer changes
This version was pushed to npm by GitHub Actions, a new releaser for
@typescript-eslint/parsersince your current version.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 this major versionwill close this PR and stop Dependabot creating any more for this major version (unless you reopen the PR or upgrade to it yourself)@dependabot ignore this minor versionwill close this PR and stop Dependabot creating any more for this minor version (unless you reopen the PR or upgrade to it yourself)@dependabot ignore this dependencywill close this PR and stop Dependabot creating any more for this dependency (unless you reopen the PR or upgrade to it yourself)