Upgrade NodeJS dependencies across workspaces#237
Closed
data-douser wants to merge 7 commits intomainfrom
Closed
Conversation
Contributor
Dependency ReviewThe following issues were found:
|
Contributor
There was a problem hiding this comment.
Pull request overview
Updates NodeJS dependencies across the repository’s npm workspaces (root, server, client, VS Code extension) to keep tooling, typings, and test dependencies current.
Changes:
- Bump lint/format tooling versions (eslint, prettier, typescript-eslint) across workspaces.
- Update runtime dependency
dotenvin server/client. - Refresh lockfile entries (including vitest/coverage and updated bundled server dist output).
Show a summary per file
| File | Description |
|---|---|
package.json |
Align root dev tooling versions (eslint/prettier/typescript-eslint). |
package-lock.json |
Regenerate lockfile to reflect updated dependency graph across workspaces. |
server/package.json |
Update server runtime/dev deps (dotenv, @types/node, vitest stack, eslint/prettier). |
server/dist/codeql-development-mcp-server.js |
Updated bundled output reflecting dependency changes (notably env injection log text). |
client/package.json |
Update client deps/tooling (dotenv, eslint, prettier). |
extensions/vscode/package.json |
Update extension dev deps (including @types/vscode, @types/node, vitest stack, eslint/prettier). |
Copilot's findings
Comments suppressed due to low confidence (1)
extensions/vscode/package.json:184
@types/nodeis bumped to ^25.6.0 while the extension declares engines.node >=24.13.0. If the extension/server code is meant to run on Node 24, compiling against Node 25 typings can accidentally introduce usage of Node 25-only APIs that will fail at runtime on Node 24. Consider aligning@types/nodeto the minimum supported Node major (24.x) or raising engines.node if Node 25 is required.
"@types/node": "^25.6.0",
- Files reviewed: 4/7 changed files
- Comments generated: 2
Updates the minimum required "engines.vscode" version from "^1.110.0" to "^1.115.0" in order to align with newer @types/vscode.
Contributor
There was a problem hiding this comment.
Pull request overview
Updates Node.js toolchain requirements and refreshes key Node/TypeScript linting, formatting, and test dependencies across the monorepo workspaces.
Changes:
- Bumped minimum Node.js engine/version references to
v25.6.0across docs,.node-version, and workspacepackage.jsonfiles. - Upgraded shared dev tooling deps (eslint, prettier, typescript-eslint) and test deps (vitest, @vitest/coverage-v8) across workspaces.
- Regenerated
package-lock.jsonto reflect updated dependency graph for all workspaces.
Show a summary per file
| File | Description |
|---|---|
server/README.md |
Updates documented Node.js prerequisite to v25.6.0+. |
server/package.json |
Raises Node engine requirement and bumps dotenv/types/lint/test tool versions. |
README.md |
Updates top-level Node.js environment requirement. |
package.json |
Raises repo-wide Node engine requirement and bumps root dev tooling versions. |
package-lock.json |
Updates lockfile to align with dependency upgrades across workspaces. |
extensions/vscode/README.md |
Updates Node.js prerequisite for the extension docs. |
extensions/vscode/package.json |
Raises VS Code + Node engine requirements and bumps dev tooling/test deps. |
docs/public.md |
Updates public install guide to reflect new Node.js minimum. |
docs/getting-started.md |
Updates getting started guide Node.js prerequisite. |
client/package.json |
Raises Node engine requirement and bumps dotenv/lint tool versions. |
.node-version |
Updates pinned Node version to v25.6.0 for CI/dev tooling. |
Copilot's findings
- Files reviewed: 10/11 changed files
- Comments generated: 2
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com> Signed-off-by: Nathan Randall <70299490+data-douser@users.noreply.github.com>
37 tasks
Contributor
There was a problem hiding this comment.
Pull request overview
Updates Node.js/tooling dependencies across the monorepo and raises the minimum supported Node.js version, along with corresponding documentation updates.
Changes:
- Bump minimum Node.js engine/version references from
24.13.0to25.6.0across packages and docs. - Upgrade lint/format/test toolchain versions (eslint, prettier, typescript-eslint, vitest, typings).
- Refresh lockfile and regenerated server bundled output in
server/dist/.
Show a summary per file
| File | Description |
|---|---|
README.md |
Updates documented minimum Node.js version. |
.node-version |
Pins repo Node.js version to v25.6.0. |
package.json |
Raises workspace engine requirement; bumps dev tooling versions. |
package-lock.json |
Lockfile update reflecting dependency/tooling upgrades. |
client/package.json |
Raises engine requirement; bumps eslint/prettier; bumps dotenv. |
server/package.json |
Raises engine requirement; bumps eslint/prettier/typescript-eslint/vitest/@types; bumps dotenv. |
server/README.md |
Updates documented minimum Node.js version for server quick start. |
server/dist/codeql-development-mcp-server.js |
Regenerated bundled server output after dependency updates. |
extensions/vscode/package.json |
Raises VS Code + Node engine requirements; bumps dev tooling + typings. |
extensions/vscode/README.md |
Updates documented minimum Node.js version for the extension. |
docs/public.md |
Updates public installation guide minimum Node.js version. |
docs/getting-started.md |
Updates getting-started guide minimum Node.js version. |
Copilot's findings
- Files reviewed: 10/13 changed files
- Comments generated: 1
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Dependency updates:
General tooling and formatting:
eslintto^10.2.0andprettierto^3.8.2in all relevantpackage.jsonfiles to maintain code style and linting consistency. [1] [2] [3] [4]typescript-eslintto^8.58.1for improved TypeScript linting support. [1] [2] [3]Type definitions and testing:
@types/nodeand@types/vscodefor better type coverage, and updated@vitest/coverage-v8andvitestfor testing improvements. [1] [2]dotenvto^17.4.1in bothclientandserverto ensure environment variable management is current. [1] [2]These changes are routine maintenance to keep dependencies secure and up-to-date, with no functional code changes.