Skip to content

build(deps): bump typescript-deps group + fix moduleResolution for TS6#154

Open
clemos wants to merge 2 commits into
mainfrom
fix/typescript-ts6-module-resolution
Open

build(deps): bump typescript-deps group + fix moduleResolution for TS6#154
clemos wants to merge 2 commits into
mainfrom
fix/typescript-ts6-module-resolution

Conversation

@clemos

@clemos clemos commented Jun 8, 2026

Copy link
Copy Markdown
Contributor

Supersedes Dependabot PR #153.

Includes the same 5 package bumps from PR #153 (typescript ^6.0.3, tsx ^4.22.4, prettier ^3.8.3, dotenv ^17.4.2, @types/node ^25.9.2) plus a required fix for TypeScript 6 compatibility.

Why PR #153 failed CI

TypeScript 6.0 deprecated moduleResolution: "node" (alias node10) with error TS5107. The fix is to update typescript/tsconfig.json to use "moduleResolution": "bundler", which is the correct modern option for "module": "ES2022" output.

Changes

Made with Cursor

Summary by CodeRabbit

Chores

  • Updated core dependencies and development tools to their latest stable versions to enhance compatibility and ensure long-term support. Includes updates to TypeScript, Node.js type definitions, code formatting tools, and build utilities.
  • Updated module resolution configuration to align with modern build standards and improve overall compatibility across different development environments and build scenarios.

dependabot Bot and others added 2 commits June 8, 2026 10:06
… updates

Bumps the typescript-deps group with 5 updates in the /typescript directory:

| Package | From | To |
| --- | --- | --- |
| [dotenv](https://github.com/motdotla/dotenv) | `16.6.1` | `17.4.2` |
| [@types/node](https://github.com/DefinitelyTyped/DefinitelyTyped/tree/HEAD/types/node) | `20.19.37` | `25.9.2` |
| [prettier](https://github.com/prettier/prettier) | `3.8.1` | `3.8.3` |
| [tsx](https://github.com/privatenumber/tsx) | `4.21.0` | `4.22.4` |
| [typescript](https://github.com/microsoft/TypeScript) | `5.9.3` | `6.0.3` |



Updates `dotenv` from 16.6.1 to 17.4.2
- [Changelog](https://github.com/motdotla/dotenv/blob/master/CHANGELOG.md)
- [Commits](motdotla/dotenv@v16.6.1...v17.4.2)

Updates `@types/node` from 20.19.37 to 25.9.2
- [Release notes](https://github.com/DefinitelyTyped/DefinitelyTyped/releases)
- [Commits](https://github.com/DefinitelyTyped/DefinitelyTyped/commits/HEAD/types/node)

Updates `prettier` from 3.8.1 to 3.8.3
- [Release notes](https://github.com/prettier/prettier/releases)
- [Changelog](https://github.com/prettier/prettier/blob/main/CHANGELOG.md)
- [Commits](prettier/prettier@3.8.1...3.8.3)

Updates `tsx` from 4.21.0 to 4.22.4
- [Release notes](https://github.com/privatenumber/tsx/releases)
- [Changelog](https://github.com/privatenumber/tsx/blob/master/release.config.cjs)
- [Commits](privatenumber/tsx@v4.21.0...v4.22.4)

Updates `typescript` from 5.9.3 to 6.0.3
- [Release notes](https://github.com/microsoft/TypeScript/releases)
- [Commits](microsoft/TypeScript@v5.9.3...v6.0.3)

---
updated-dependencies:
- dependency-name: dotenv
  dependency-version: 17.4.2
  dependency-type: direct:production
  update-type: version-update:semver-major
  dependency-group: typescript-deps
- dependency-name: "@types/node"
  dependency-version: 25.9.2
  dependency-type: direct:development
  update-type: version-update:semver-major
  dependency-group: typescript-deps
- dependency-name: prettier
  dependency-version: 3.8.3
  dependency-type: direct:development
  update-type: version-update:semver-patch
  dependency-group: typescript-deps
- dependency-name: tsx
  dependency-version: 4.22.4
  dependency-type: direct:development
  update-type: version-update:semver-minor
  dependency-group: typescript-deps
- dependency-name: typescript
  dependency-version: 6.0.3
  dependency-type: direct:development
  update-type: version-update:semver-major
  dependency-group: typescript-deps
...

Signed-off-by: dependabot[bot] <support@github.com>
moduleResolution 'node' (node10) is deprecated in TypeScript 6.0 and
errors with TS5107. Switch to 'bundler' which is the correct option for
ES2022 module output.

Co-authored-by: Cursor <cursoragent@cursor.com>
@coderabbitai

coderabbitai Bot commented Jun 8, 2026

Copy link
Copy Markdown
Contributor

Review Change Stack

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: defaults

Review profile: CHILL

Plan: Pro

Run ID: 001b1dd3-b20c-439a-a6da-3affb4210b1e

📥 Commits

Reviewing files that changed from the base of the PR and between 5b72681 and 8752ef0.

⛔ Files ignored due to path filters (1)
  • typescript/package-lock.json is excluded by !**/package-lock.json
📒 Files selected for processing (2)
  • typescript/package.json
  • typescript/tsconfig.json

📝 Walkthrough

Walkthrough

The PR updates the TypeScript build toolchain by bumping five dependencies to newer versions—dotenv, @types/node, prettier, tsx, and typescript—while adjusting the TypeScript compiler's module resolution strategy from node to bundler.

Changes

TypeScript Toolchain and Compiler Configuration Updates

Layer / File(s) Summary
Dependency versions
typescript/package.json
dotenv is bumped to ^17.4.2, and four dev dependencies are updated: @types/node to ^25.9.2, prettier to ^3.8.3, tsx to ^4.22.4, and typescript to ^6.0.3.
TypeScript compiler configuration
typescript/tsconfig.json
compilerOptions.moduleResolution is changed from node to bundler, altering how TypeScript resolves module specifiers.

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~8 minutes

Possibly related PRs

  • gladiaio/gladia-samples#48: Both PRs touch the TypeScript build setup, with this PR updating dependency versions and tsconfig module resolution while the related PR adjusts TypeScript config and scripts for JS regeneration.

Suggested reviewers

  • nmorel

Poem

🐰 Dependencies dance in versions new,
From bundler-resolved modules breaking through,
TypeScript's path finds cleaner ways,
As toolchain shines on build-time days! 📦✨

🚥 Pre-merge checks | ✅ 5
✅ Passed checks (5 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed The title directly and clearly summarizes the two main changes: dependency updates and a moduleResolution configuration fix for TypeScript 6 compatibility.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
Linked Issues check ✅ Passed Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check ✅ Passed Check skipped because no linked issues were found for this pull request.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.

✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch fix/typescript-ts6-module-resolution

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

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.

1 participant