chore: use typescript 6, remove esmoduleInteropFlag

[vkarpov15]

Summary

Replaces #16255: turns out we also need to remove esModuleInterop option. Removing esModuleInterop shouldn't affect anything since esModuleInterop: false is the default anyway.

Examples

[Copilot]

Pull request overview

Updates the repo’s TypeScript toolchain by bumping the TypeScript compiler version and simplifying the type-test tsconfig by removing an explicit esModuleInterop override.

Changes:

• Bump typescript devDependency from 5.9.3 to 6.0.3.
• Remove esModuleInterop: false from test/types/tsconfig.json (relying on defaults/inherited config).

Reviewed changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated 1 comment.

File	Description
package.json	Updates the TypeScript compiler version used by repo tooling (linting/type tests/bench workflows that use the root install).
test/types/tsconfig.json	Removes an explicit esModuleInterop setting from the type-test TS config.

[AbdelrahmanHafez]

I wonder if we should run our tests on both TS 5 and 6 versions, the same way we run our js tests on a matrix of node and mongo versions. That way we guarantee our code works in both TS versions.

[vkarpov15]

That would be a substantial change from our current policy of "only latest TypeScript is supported", which could be a big source of headache. Are you upgrading to TypeScript 6 or are you planning on staying on TypeScript 5 for the foreseeable future?

[AbdelrahmanHafez]

I'm not currently using mongoose in any project I'm maintaining, I'd probably update TS to the latest version if I were.

only latest TypeScript is supported
Fair enough.

I see you changed this to 9.7 which makes things better.

[AbdelrahmanHafez]

Other than Copilot's comment LGTM

[mrazauskas]

Perhaps it is worth removing allowSyntheticDefaultImports as well?

Same as esModuleInterop, TS6 deprecates setting allowSyntheticDefaultImports: false (reference).

Seems like it was added here together with esModuleInterop: false, because that would result in having allowSyntheticDefaultImports: false (reference). So it feels like both can be removed for TS6.

Suggested change

"allowSyntheticDefaultImports": true,

[hasezoey]

I just saw the Milestone for this, consider moving this change to at least the next minor version.