Update npm dependencies (major)#59
Conversation
|
|
|
Summary
Dependency Update: Major Version BumpsThis PR updates three major dependencies that will need validation:
Good news: The codebase is already written in ESM (using What needs verification: Build artifacts, test execution, and runtime behavior of the bundled credential setup/guard scripts. What reviewers should knowReview FocusStart here: Verify the bundled outputs still work:
Non-obvious changes:
Potential gotchas:
|
There was a problem hiding this comment.
A couple of things need clarification before this can merge.
Registry split in package-lock.json: All 161 pre-existing packages in the lock file resolve from repox.jfrog.io (SonarSource's Artifactory proxy), but the 22 packages newly introduced by this upgrade resolve directly from registry.npmjs.org. If the Artifactory proxy exists for security scanning or to guarantee availability in restricted CI environments, those 22 packages are bypassing it. This looks like Renovate ran without the internal registry configured — worth confirming this is intentional or regenerating the lockfile with the proxy in place.
| "integrity": "sha512-hXJCSrkwfA46Vd9Z3q4cpEpHB1rL5NG04+/rbqW9d3+CSvtB1tYe8UTpAlixa1vj0m/ULglfEK2UKxMGxCxv5A==", | ||
| "version": "3.0.1", | ||
| "resolved": "https://registry.npmjs.org/@actions/core/-/core-3.0.1.tgz", | ||
| "integrity": "sha512-a6d/Nwahm9fliVGRhdhofo40HjHQasUPusmc7vBfyky+7Z+P2A1J68zyFVaNcEclc/Se+eO595oAr5nwEIoIUA==", |
There was a problem hiding this comment.
All packages already in the lock file resolve from repox.jfrog.io, but every package newly added or replaced by this upgrade (here and in the @actions/exec, @actions/http-client, @actions/io, vitest ecosystem, and undici entries) resolves directly from registry.npmjs.org. If this Artifactory proxy is used for security scanning, licence checks, or to air-gap CI from the public registry, the 22 newly resolved packages are bypassing it.
This looks like Renovate generated the lockfile without the internal registry configured. Please regenerate with the proxy in place, or confirm this split is intentional.
- Mark as noise
| "version": "1.11.1", | ||
| "resolved": "https://repox.jfrog.io/artifactory/api/npm/npm/@actions/core/-/core-1.11.1.tgz", | ||
| "integrity": "sha512-hXJCSrkwfA46Vd9Z3q4cpEpHB1rL5NG04+/rbqW9d3+CSvtB1tYe8UTpAlixa1vj0m/ULglfEK2UKxMGxCxv5A==", | ||
| "version": "3.0.1", |
There was a problem hiding this comment.
@actions/core v3.0.0 is ESM-only (the release notes explicitly state CommonJS consumers must switch to dynamic import()). The TypeScript source already uses ESM import syntax and ncc/esbuild handles ESM-to-CJS bundling, so the bundle itself should be fine. However, tsconfig.json sets "module": "commonjs" — TypeScript 6 has stricter module/resolution coupling and may fail to resolve types for ESM-only packages under the old node module resolution. Please confirm npm run build and tsc --noEmit both pass cleanly with this combination before merging.
- Mark as noise
e4dc528 to
8016e22
Compare
|
|



This PR contains the following updates:
^1.11.1→^3.0.0^5.7.0→^6.0.0^6.24.0→^8.0.0^7.3.2→^8.0.0^3.0.0→^4.0.0Release Notes
actions/toolkit (@actions/core)
v3.0.1undicifrom6.23.0to6.24.1#2348v3.0.0import()instead ofrequire()v2.0.3@actions/http-clientto3.0.2v2.0.1v2.0.0microsoft/TypeScript (typescript)
v6.0.3Compare Source
v6.0.2Compare Source
nodejs/undici (undici)
v8.3.0Compare Source
What's Changed
SnapshotAgent: AddnormalizeBodyandnormalizeQueryby @GeoffreyBooth in #5121New Contributors
Full Changelog: nodejs/undici@v8.2.0...v8.3.0
v8.2.0Compare Source
What's Changed
New Contributors
Full Changelog: nodejs/undici@v8.1.0...v8.2.0
v8.1.0Compare Source
What's Changed
Full Changelog: nodejs/undici@v8.0.3...v8.1.0
v8.0.3Compare Source
What's Changed
dump({ limit: Integer })default value by @samuel871211 in #4981dump.maxSizeby @samuel871211 in #4982New Contributors
Full Changelog: nodejs/undici@v8.0.2...v8.0.3
v8.0.2Compare Source
What's Changed
Full Changelog: nodejs/undici@v8.0.1...v8.0.2
v8.0.1Compare Source
What's Changed
New Contributors
Full Changelog: nodejs/undici@v7.24.7...v8.0.1
v8.0.0Compare Source
What's Changed
Full Changelog: nodejs/undici@v7.24.7...v8.0.0
v7.26.0Compare Source
What's Changed
Full Changelog: nodejs/undici@v7.25.0...v7.26.0
v7.25.0Compare Source
What's Changed
Full Changelog: nodejs/undici@v7.24.8...v7.25.0
v7.24.8Compare Source
What's Changed
Full Changelog: nodejs/undici@v7.24.7...v7.24.8
v7.24.7Compare Source
What's Changed
redirectionLimitReachedby @samuel871211 in #4933New Contributors
Full Changelog: nodejs/undici@v7.24.6...v7.24.7
v7.24.6Compare Source
What's Changed
New Contributors
Full Changelog: nodejs/undici@v7.24.5...v7.24.6
v7.24.5Compare Source
What's Changed
New Contributors
Full Changelog: nodejs/undici@v7.24.4...v7.24.5
v7.24.4Compare Source
What's Changed
Full Changelog: nodejs/undici@v7.24.3...v7.24.4
v7.24.3Compare Source
What's Changed
Full Changelog: nodejs/undici@v7.24.2...v7.24.3
v7.24.2Compare Source
What's Changed
Full Changelog: nodejs/undici@v7.24.1...v7.24.2
v7.24.1Compare Source
v7.24.0Compare Source
What's Changed
Full Changelog: nodejs/undici@v7.23.0...v7.24.0
v7.23.0Compare Source
v7.22.0Compare Source
What's Changed
New Contributors
Full Changelog: nodejs/undici@v7.21.0...v7.22.0
v7.21.0Compare Source
What's Changed
closemethod to WebSocketStream interface by @piotr-cz in #4802New Contributors
Full Changelog: nodejs/undici@v7.20.0...v7.21.0
v7.20.0Compare Source
What's Changed
Configuration
📅 Schedule: (in timezone Europe/Paris)
🚦 Automerge: Disabled by config. Please merge this manually once you are satisfied.
♻ Rebasing: Whenever PR becomes conflicted, or you tick the rebase/retry checkbox.
👻 Immortal: This PR will be recreated if closed unmerged. Get config help if that's undesired.
This PR was generated by Mend Renovate. View the repository job log.