Skip to content

Commit 088dfd3

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 85ce2e5 commit 088dfd3

1 file changed

Lines changed: 1 addition & 2 deletions

File tree

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

Lines changed: 1 addition & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -42,8 +42,7 @@ 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: {
4948
wrangler: '^4.14.4'

0 commit comments

Comments
 (0)