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.

Author: mattzcarey

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

@@ -42,8 +42,7 @@ describe('Cloudflare Workers compatibility (no nodejs_compat)', () => {
             private: true,
             type: 'module',
             dependencies: {
-                '@modelcontextprotocol/server': `file:./${tarballName}`,
-                '@cfworker/json-schema': '^4.1.1'
+                '@modelcontextprotocol/server': `file:./${tarballName}`
             },
             devDependencies: {
                 wrangler: '^4.14.4'
@@ -150,28 +149,29 @@ export default {
     it('should handle MCP requests', async () => {
         expect(env).not.toBeNull();
 
-        // Retry connection — wrangler may report "Ready" before it can handle requests
-        let client!: Client;
+        // Wrangler may report "Ready" before miniflare can fully service the user worker.
+        // `connect()` can succeed against the entry worker while the first real request still
+        // gets a 500 ("Network connection lost"), so both have to be retried together with a
+        // fresh client per attempt.
         let lastError: unknown;
         for (let attempt = 0; attempt < 5; attempt++) {
+            const client = new Client({ name: 'test-client', version: '1.0.0' });
             try {
-                client = new Client({ name: 'test-client', version: '1.0.0' });
                 const transport = new StreamableHTTPClientTransport(new URL(`http://127.0.0.1:${PORT}/`));
                 await client.connect(transport);
+                const result = await client.callTool({ name: 'greet', arguments: { name: 'World' } });
+                expect(result.content).toEqual([{ type: 'text', text: 'Hello, World!' }]);
+                await client.close();
                 lastError = undefined;
                 break;
             } catch (error) {
                 lastError = error;
+                await client.close().catch(() => {});
                 await new Promise(resolve => setTimeout(resolve, 1000));
             }
         }
         if (lastError) {
             throw lastError;
         }
-
-        const result = await client.callTool({ name: 'greet', arguments: { name: 'World' } });
-        expect(result.content).toEqual([{ type: 'text', text: 'Hello, World!' }]);
-
-        await client.close();
     }, 30_000);
 });