test: make barrelClean chunk scan transitive (BFS) and CI-safe; map /stdio in quickstart tsconfig paths

- chunkImportsOf now BFS-walks the chunk import graph so the test name
  ('transitively imported') matches the implementation
- beforeAll builds the package if dist/ is missing (CI test job runs
  pnpm test:all without a build step)
- examples/{client,server}-quickstart/tsconfig.json: add /stdio to paths
  so tsc --noEmit resolves the new subpath without a prior build

Author: felixweinberger

examples/client-quickstart/tsconfig.json

@@ -12,6 +12,7 @@
         "forceConsistentCasingInFileNames": true,
         "paths": {
             "@modelcontextprotocol/client": ["./node_modules/@modelcontextprotocol/client/src/index.ts"],
+            "@modelcontextprotocol/client/stdio": ["./node_modules/@modelcontextprotocol/client/src/stdio.ts"],
             "@modelcontextprotocol/client/_shims": ["./node_modules/@modelcontextprotocol/client/src/shimsNode.ts"],
             "@modelcontextprotocol/core": [
                 "./node_modules/@modelcontextprotocol/client/node_modules/@modelcontextprotocol/core/src/index.ts"

examples/server-quickstart/tsconfig.json

@@ -11,6 +11,7 @@
         "forceConsistentCasingInFileNames": true,
         "paths": {
             "@modelcontextprotocol/server": ["./node_modules/@modelcontextprotocol/server/src/index.ts"],
+            "@modelcontextprotocol/server/stdio": ["./node_modules/@modelcontextprotocol/server/src/stdio.ts"],
             "@modelcontextprotocol/server/_shims": ["./node_modules/@modelcontextprotocol/server/src/shimsNode.ts"],
             "@modelcontextprotocol/core": [
                 "./node_modules/@modelcontextprotocol/server/node_modules/@modelcontextprotocol/core/src/index.ts"

packages/client/test/client/barrelClean.test.ts

@@ -1,18 +1,37 @@
-import { readFileSync } from 'node:fs';
+import { execFileSync } from 'node:child_process';
+import { existsSync, readFileSync } from 'node:fs';
 import { dirname, join } from 'node:path';
 import { fileURLToPath } from 'node:url';
 
-import { describe, expect, test } from 'vitest';
+import { beforeAll, describe, expect, test } from 'vitest';
 
-const distDir = join(dirname(fileURLToPath(import.meta.url)), '../../dist');
+const pkgDir = join(dirname(fileURLToPath(import.meta.url)), '../..');
+const distDir = join(pkgDir, 'dist');
 const NODE_ONLY = /\b(child_process|cross-spawn|node:stream|node:child_process)\b/;
 
 function chunkImportsOf(entryPath: string): string[] {
-    const src = readFileSync(entryPath, 'utf8');
-    return [...src.matchAll(/from\s+["']\.\/(.+?\.mjs)["']/g)].map(m => join(distDir, m[1]!));
+    const visited = new Set<string>();
+    const queue = [entryPath];
+    while (queue.length > 0) {
+        const file = queue.shift()!;
+        if (visited.has(file)) continue;
+        visited.add(file);
+        const src = readFileSync(file, 'utf8');
+        for (const m of src.matchAll(/from\s+["']\.\/(.+?\.mjs)["']/g)) {
+            queue.push(join(dirname(file), m[1]!));
+        }
+    }
+    visited.delete(entryPath);
+    return [...visited];
 }
 
 describe('@modelcontextprotocol/client root entry is browser-safe', () => {
+    beforeAll(() => {
+        if (!existsSync(join(distDir, 'index.mjs')) || !existsSync(join(distDir, 'stdio.mjs'))) {
+            execFileSync('pnpm', ['build'], { cwd: pkgDir, stdio: 'inherit' });
+        }
+    }, 60_000);
+
     test('dist/index.mjs contains no process-spawning runtime imports', () => {
         const entry = join(distDir, 'index.mjs');
         expect(readFileSync(entry, 'utf8')).not.toMatch(NODE_ONLY);

packages/server/test/server/barrelClean.test.ts

@@ -1,18 +1,37 @@
-import { readFileSync } from 'node:fs';
+import { execFileSync } from 'node:child_process';
+import { existsSync, readFileSync } from 'node:fs';
 import { dirname, join } from 'node:path';
 import { fileURLToPath } from 'node:url';
 
-import { describe, expect, test } from 'vitest';
+import { beforeAll, describe, expect, test } from 'vitest';
 
-const distDir = join(dirname(fileURLToPath(import.meta.url)), '../../dist');
+const pkgDir = join(dirname(fileURLToPath(import.meta.url)), '../..');
+const distDir = join(pkgDir, 'dist');
 const NODE_ONLY = /\b(node:stream|node:child_process)\b/;
 
 function chunkImportsOf(entryPath: string): string[] {
-    const src = readFileSync(entryPath, 'utf8');
-    return [...src.matchAll(/from\s+["']\.\/(.+?\.mjs)["']/g)].map(m => join(distDir, m[1]!));
+    const visited = new Set<string>();
+    const queue = [entryPath];
+    while (queue.length > 0) {
+        const file = queue.shift()!;
+        if (visited.has(file)) continue;
+        visited.add(file);
+        const src = readFileSync(file, 'utf8');
+        for (const m of src.matchAll(/from\s+["']\.\/(.+?\.mjs)["']/g)) {
+            queue.push(join(dirname(file), m[1]!));
+        }
+    }
+    visited.delete(entryPath);
+    return [...visited];
 }
 
 describe('@modelcontextprotocol/server root entry is browser-safe', () => {
+    beforeAll(() => {
+        if (!existsSync(join(distDir, 'index.mjs')) || !existsSync(join(distDir, 'stdio.mjs'))) {
+            execFileSync('pnpm', ['build'], { cwd: pkgDir, stdio: 'inherit' });
+        }
+    }, 60_000);
+
     test('dist/index.mjs contains no process-stdio runtime imports', () => {
         const entry = join(distDir, 'index.mjs');
         expect(readFileSync(entry, 'utf8')).not.toMatch(NODE_ONLY);