chore: bump mongodb and bson dependencies#829
Conversation
There was a problem hiding this comment.
Pull request overview
This PR updates the monorepo’s MongoDB Node.js driver dependencies (and associated lockfile resolutions) and adjusts the root where helper script to use the monorepo-tools implementation.
Changes:
- Bumped
mongodbfrom^7.2.0to^7.5.0across multiple packages. - Updated
package-lock.jsonto reflectmongodb@7.5.0and newly resolvedbsonversions. - Changed the root
wherescript to point at monorepo-tools’whereimplementation.
Reviewed changes
Copilot reviewed 6 out of 7 changed files in this pull request and generated 1 comment.
Show a summary per file
| File | Description |
|---|---|
| packages/mql-typescript/package.json | Bumps mongodb to ^7.5.0 for this package. |
| packages/mongodb-ts-autocomplete/package.json | Bumps mongodb (devDependency) to ^7.5.0 and reorders deps. |
| packages/mongodb-runner/package.json | Bumps runtime mongodb dependency to ^7.5.0. |
| packages/mongodb-constants/package.json | Reorders bson devDependency entry (no functional change). |
| packages/devtools-connect/package.json | Bumps mongodb devDependency to ^7.5.0 and reorders entries. |
| package.json | Updates the root where script target. |
| package-lock.json | Updates lockfile to mongodb@7.5.0 and related dependency graph changes. |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
| "test-ci": "lerna run test-ci --concurrency 1", | ||
| "test": "lerna run test --concurrency 1 --stream", | ||
| "where": "node ./scripts/src/where.js", | ||
| "where": "node packages/monorepo-tools/dist/where.js", |
| os: [ubuntu-latest, windows-latest, macos-latest] | ||
| # Tests are sharded across hosts by package (see compute-test-shard.mjs). | ||
| # When changing the shard count, update every list entry below. | ||
| shard: [1, 2, 3, 4] |
There was a problem hiding this comment.
nit, not a blocker. Should we use the wording group instead of shard like we do for Compass e2e tests? https://github.com/mongodb-js/compass/blob/80ef906493ef0891197fdb4c9af324c1d98d2e43/.evergreen/buildvariants-and-tasks.in.yml#L162
I'm worried shard could cause confusion. Maybe there's a better word?
There was a problem hiding this comment.
Terminology alignment totally worth doing, group is good, fixed!
There was a problem hiding this comment.
Is there a reason to not use typescript here?
There was a problem hiding this comment.
Node.js 22 is currently the test node here so it would need a flag to run the ts, I could do the flag, or I could upgrade the node, I chose neither 🤣
Description
Bump
mongodbandbsonacross packages and fix where script to do soOpen Questions
Checklist