Skip to content

Conversation

@kmruiz
Copy link
Contributor

@kmruiz kmruiz commented Nov 24, 2025

The idea is to adapt mongosh to use the latest driver, at least, in Node.js 20. The integration with Node.js 24 will happen in another PR.

Copilot AI review requested due to automatic review settings November 24, 2025 13:56
@kmruiz kmruiz requested a review from a team as a code owner November 24, 2025 13:56
@kmruiz kmruiz added the wip Work in Progress label Nov 24, 2025
Copy link
Collaborator

@addaleax addaleax left a comment

Choose a reason for hiding this comment

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

You'll probably want a bunch of changes from #2560 too and/or base this PR on that one – there's a bunch of stuff that's already been addressed there

Copilot finished reviewing on behalf of kmruiz November 24, 2025 13:58

This comment was marked as outdated.

@kmruiz
Copy link
Contributor Author

kmruiz commented Nov 24, 2025

You'll probably want a bunch of changes from #2560 too and/or base this PR on that one – there's a bunch of stuff that's already been addressed there

Touche! I'll rebase my changes on that branch and start from there.

@addaleax addaleax changed the title chore: Upgrade mongodb driver to 7.0.0 MONGOSH-2978 feat(deps): upgrade mongodb driver to 7.0.0 MONGOSH-2978 Nov 24, 2025
"optionalDependencies": {
"kerberos": "2.1.0",
"mongodb-client-encryption": "^6.5.0"
"kerberos": "^7.0.0",
Copy link
Collaborator

Choose a reason for hiding this comment

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

Keep in mind that MONGOSH-1881 on its own is already a sizable ticket

Copy link
Contributor Author

Choose a reason for hiding this comment

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

I'm trying for now to see where it breaks to know how to split this work in PRs that don't depend on each other (if possible). Likely for encryption and kerberos we will need other PRs.

"kerberos": "^7.0.0",
"mongodb-client-encryption": "^7.0.0",
"bson": "^7.0.0",
"mongodb": "^7.0.0"
Copy link
Collaborator

Choose a reason for hiding this comment

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

^ These additions are intended to be temporary as a way of making sure the package-lock file doesn't start hoisting other versions of these dependencies, they'll need to be removed before merging

@kmruiz kmruiz changed the title feat(deps): upgrade mongodb driver to 7.0.0 MONGOSH-2978 feat(deps)!: upgrade mongodb driver to 7.0.0 MONGOSH-2978 Nov 25, 2025
@addaleax addaleax mentioned this pull request Nov 27, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

wip Work in Progress

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants