Skip to content

dependencies: bump eslint and friends to latest#1042

Open
nirinchev wants to merge 1 commit intomainfrom
ni/eslint
Open

dependencies: bump eslint and friends to latest#1042
nirinchev wants to merge 1 commit intomainfrom
ni/eslint

Conversation

@nirinchev
Copy link
Copy Markdown
Collaborator

Proposed changes

Bumps eslint to 10.x and addresses the issues flagged by the new rules.

@nirinchev nirinchev requested a review from a team as a code owner April 10, 2026 23:27
@nirinchev nirinchev requested review from Copilot and jeroenvervaeke and removed request for a team April 10, 2026 23:27
@nirinchev nirinchev added the no-title-validation Add this label to disable the title check for this PR. label Apr 10, 2026
Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

Updates the repo’s linting toolchain to ESLint 10 and applies the associated code tweaks needed to satisfy the newer rules and typings.

Changes:

  • Bump ESLint and related lint dependencies (plus lockfile refresh).
  • Add Error cause chaining in several throw sites to preserve original error context.
  • Adjust TypeScript build libs to include ES2022 typings needed by the new patterns.

Reviewed changes

Copilot reviewed 11 out of 12 changed files in this pull request and generated no comments.

Show a summary per file
File Description
tsconfig.build.json Adds ES2022/DOM libs to support newer standard library typings (e.g., Error options).
package.json Bumps eslint, @eslint/js, @typescript-eslint/*, @vitest/eslint-plugin versions.
pnpm-lock.yaml Lockfile updates reflecting the dependency bumps and peer resolutions.
eslint-rules/enforce-zod-v4.js Updates custom rule to use the new ESLint context filename access.
tests/integration/transports/configOverrides.test.ts Updates test error rethrows to include { cause } for better diagnostics.
src/tools/mongodb/update/updateMany.ts Adjusts message initialization to satisfy stricter lint/TS checks.
src/setup/aiTool.ts Adds { cause } when rethrowing config write failures.
src/server.ts Adds { cause } for connection-string / apiBaseUrl validation failures.
src/helpers/connectionOptions.ts Adds { cause } for connection-string validation errors.
src/common/atlas/performanceAdvisorUtils.ts Adds { cause } when rethrowing Atlas Performance Advisor request failures.
src/common/atlas/cluster.ts Adds { cause } when rethrowing cluster processId retrieval failures.
scripts/generate/generateToolDocumentation.ts Adds { cause } when tool instantiation fails during doc generation.
Files not reviewed (1)
  • pnpm-lock.yaml: Language not supported

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

no-title-validation Add this label to disable the title check for this PR. type: chore

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants