Skip to content

Commit ce335f6

Browse files
committed
test(integration): drop pre-installed @cfworker/json-schema from CF Workers test
PR #2088 adds @cfworker/json-schema to noExternal so the validator is bundled into the published @modelcontextprotocol/server tarball. The Cloudflare Workers integration test was still installing the package as a direct dep in the generated consumer package.json, which masked any future re-externalization regression — wrangler would resolve the bare import from the test's own install instead of failing. Removing the dep turns the test into a true regression guard that the bundle is genuinely self-contained, matching the migration docs. Closes the last open review thread on #2088.
1 parent 44e7b26 commit ce335f6

1 file changed

Lines changed: 4 additions & 3 deletions

File tree

test/integration/test/server/cloudflareWorkers.test.ts

Lines changed: 4 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -42,11 +42,12 @@ describe('Cloudflare Workers compatibility (no nodejs_compat)', () => {
4242
private: true,
4343
type: 'module',
4444
dependencies: {
45-
'@modelcontextprotocol/server': `file:./${tarballName}`,
46-
'@cfworker/json-schema': '^4.1.1'
45+
'@modelcontextprotocol/server': `file:./${tarballName}`
4746
},
4847
devDependencies: {
49-
wrangler: '^4.14.4'
48+
// Pinned exact: wrangler@4.95.0 ships a miniflare/workerd bump that races on Node 22.
49+
// Bumps need a deliberate refresh + verification across the Node matrix.
50+
wrangler: '4.94.0'
5051
}
5152
};
5253
fs.writeFileSync(path.join(tempDir, 'package.json'), JSON.stringify(pkgJson, null, 2));

0 commit comments

Comments
 (0)