Repository navigation
chore(deps-dev): bump the npm-deps group across 1 directory with 2 updates - #193
dependabot[bot] wants to merge 1 commit into
Conversation
|
Review the following changes in direct dependencies. Learn more about Socket for GitHub.
|
40cfac8 to
994ff03
Compare
| "@types/node": "^26.1.1", | ||
| "tsx": "^4.23.1", | ||
| "@types/node": "^26.6.5", | ||
| "tsx": "^4.23.15", |
There was a problem hiding this comment.
🟡 (optional) After this bump, CI tests the MCP bridge with tsx 4.23.15, but the bridge and agent images still install tsx 4.23.1. A green test run no longer covers the runtime that ships. test:mcp-bridge runs tsx --test shared/tempo/mcp-bridge/server.test.ts with the locked tsx. Both Dockerfiles still run npm install --global tsx@ 4.23.1. Fix: use one tsx version everywhere, e.g. change tsx@ 4.23.1 to tsx@ 4.23.15 in shared/tempo/mcp-bridge/Dockerfile:3 and shared/global/docker/agent/Dockerfile:16 in this PR, or have the images install from the root lockfile.
Why this was flagged
package.json:61 raises tsx to ^4.23.15, and package-lock.json now resolves 4.23.15. The test:mcp-bridge script in package.json and the check script that chains it run server.ts and server.test.ts under that version. The shipped bridge image runs CMD ["tsx", "server.ts"] after npm install --global tsx@ 4.23.1 (shared/tempo/mcp-bridge/Dockerfile:3-5). The agent image also pins tsx@ 4.23.1 (shared/global/docker/agent/Dockerfile:16). On the base branch the local lockfile and both images used tsx 4.23.1. After merge they differ by 14 patch releases, which include changes to import.meta shimming, the shared transform cache and CJS/ESM interop. Nothing ties the pins together: a repo-wide grep for tsx@ finds only these two Dockerfiles, and no check compares them with package.json. So code that relies on a 4.23.2+ fix passes npm run check and then fails inside the containers. server.ts does not use import.meta or tsImport today, so the impact is a future regression that CI cannot catch, not a current break.
Verification: On the base commit, package.json:61 had "tsx": "^4.23.1" and the lockfile resolved tsx-4.23.1.tgz. After the change, package.json:61 is "^4.23.15" and the lockfile resolves tsx-4.23.15.tgz. The diff does not touch either Dockerfile, so both still pin 4.23.1. A green npm run check therefore no longer exercises the tsx version that ships in either image.
…dates Bumps the npm-deps group with 2 updates in the / directory: [@types/node](https://github.com/DefinitelyTyped/DefinitelyTyped/tree/HEAD/types/node) and [tsx](https://github.com/privatenumber/tsx). Updates `@types/node` from 26.1.1 to 26.6.5 - [Release notes](https://github.com/DefinitelyTyped/DefinitelyTyped/releases) - [Commits](https://github.com/DefinitelyTyped/DefinitelyTyped/commits/HEAD/types/node) Updates `tsx` from 4.23.1 to 4.23.15 - [Release notes](https://github.com/privatenumber/tsx/releases) - [Changelog](https://github.com/privatenumber/tsx/blob/master/release.config.cjs) - [Commits](privatenumber/tsx@v4.23.1...v4.23.15) --- updated-dependencies: - dependency-name: "@types/node" dependency-version: 26.6.3 dependency-type: direct:development update-type: version-update:semver-minor dependency-group: npm-deps - dependency-name: tsx dependency-version: 4.23.15 dependency-type: direct:development update-type: version-update:semver-patch dependency-group: npm-deps ... Signed-off-by: dependabot[bot] <support@github.com>
994ff03 to
956b8a4
Compare
Bumps the npm-deps group with 2 updates in the / directory: @types/node and tsx.
Updates
@types/nodefrom 26.1.1 to 26.6.5Commits
Updates
tsxfrom 4.23.1 to 4.23.15Release notes
Sourced from tsx's releases.
... (truncated)
Commits
ca66105test: fix drive-less file URLs in ESM resolver fixtures2da3407fix: expose require.cache and require.extensions to tsImport CommonJS modules38e1588fix: exclude bare builtins from namespace inheritance562c434fix: make namespaced register() overloads portable for declaration emitedfb1f0build: upgrade pkgroll and externalize CJS loader reference70e7828test: upgrade tinyspy for disposable API9ed2022ci: avoid duplicate release notifications872e77frefactor: use disposables for cleanup6e5236bfix: restore the CJS bridge namespace for Node 24 require(esm) under tsImport...28e1f12fix(cache): bound shared transform cache memory (#835)