fix: keep validator providers off root declarations

Author: Sehlani042

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

@@ -10,6 +10,7 @@ const distDir = join(pkgDir, 'dist');
 const NODE_ONLY = /\b(child_process|cross-spawn|node:stream|node:child_process)\b/;
 // Anchored at start-of-line so JSDoc-example `from 'ajv'` strings in vendored chunks don't match.
 const VALIDATOR_BACKEND_IMPORT = /^import[^\n]*?from\s+["'](?:ajv|ajv-formats|@cfworker\/json-schema)["']/m;
+const ROOT_VALIDATOR_EXPORTS = ['AjvJsonSchemaValidator', 'CfWorkerJsonSchemaValidator', 'CfWorkerSchemaDraft'];
 
 function chunkImportsOf(entryPath: string): string[] {
     const visited = new Set<string>();
@@ -27,6 +28,12 @@ function chunkImportsOf(entryPath: string): string[] {
     return [...visited];
 }
 
+function rootExportBlockOf(content: string): string {
+    const blocks = content.match(/(?:^|\n)export \{[\s\S]*?\};/g);
+    expect(blocks).toBeTruthy();
+    return blocks!.at(-1)!;
+}
+
 describe('@modelcontextprotocol/client root entry is browser-safe', () => {
     beforeAll(() => {
         if (!existsSync(join(distDir, 'index.mjs')) || !existsSync(join(distDir, 'stdio.mjs'))) {
@@ -67,4 +74,13 @@ describe('@modelcontextprotocol/client root entry is browser-safe', () => {
             }
         }
     });
+
+    test('root declarations do not advertise validator provider classes', () => {
+        for (const declaration of ['index.d.mts', 'index.d.cts']) {
+            const rootExportBlock = rootExportBlockOf(readFileSync(join(distDir, declaration), 'utf8'));
+            for (const symbol of ROOT_VALIDATOR_EXPORTS) {
+                expect(rootExportBlock).not.toMatch(new RegExp(`\\b(?:type\\s+)?${symbol}\\b`));
+            }
+        }
+    });
 });

packages/core-internal/src/exports/public/index.ts

@@ -131,10 +131,9 @@ export {
 export type { SpecTypeName, SpecTypes } from '../../types/specTypeSchema';
 export { isSpecType, specTypeSchemas } from '../../types/specTypeSchema';
 export type { StandardSchemaV1, StandardSchemaV1Sync, StandardSchemaWithJSON } from '../../util/standardSchema';
-// Validator providers are type-only here — import the runtime classes from the explicit
-// `@modelcontextprotocol/{client,server}/validators/{ajv,cf-worker}` subpaths to customise.
-export type { AjvJsonSchemaValidator } from '../../validators/ajvProvider';
-export type { CfWorkerJsonSchemaValidator, CfWorkerSchemaDraft } from '../../validators/cfWorkerProvider';
+// Validator provider classes stay subpath-only. Re-exporting them here, even as
+// `type`, can make generated root declarations advertise runtime-shaped root
+// exports that the package root does not provide.
 // fromJsonSchema is intentionally NOT exported here — the server and client packages
 // provide runtime-aware wrappers that default to the appropriate validator via _shims.
 export type { JsonSchemaType, JsonSchemaValidator, jsonSchemaValidator, JsonSchemaValidatorResult } from '../../validators/types';

packages/core-internal/src/index.ts

@@ -32,9 +32,8 @@ export * from './util/standardSchema';
 export * from './util/zodCompat';
 export { codecForVersion, MODERN_WIRE_REVISION } from './wire/codec';
 
-// Validator providers are type-only here — import the runtime classes from the explicit
-// `@modelcontextprotocol/{client,server}/validators/{ajv,cf-worker}` subpaths to customise.
-export type { AjvJsonSchemaValidator } from './validators/ajvProvider';
-export type { CfWorkerJsonSchemaValidator, CfWorkerSchemaDraft } from './validators/cfWorkerProvider';
+// Validator provider classes stay subpath-only. Re-exporting them here, even as
+// `type`, can make generated client/server root declarations advertise
+// runtime-shaped root exports that the package root does not provide.
 export * from './validators/fromJsonSchema';
 export type { JsonSchemaType, JsonSchemaValidator, jsonSchemaValidator, JsonSchemaValidatorResult } from './validators/types';

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

@@ -10,6 +10,7 @@ const distDir = join(pkgDir, 'dist');
 const NODE_ONLY = /\b(child_process|cross-spawn|node:stream|node:child_process)\b/;
 // Anchored at start-of-line so JSDoc-example `from 'ajv'` strings in vendored chunks don't match.
 const VALIDATOR_BACKEND_IMPORT = /^import[^\n]*?from\s+["'](?:ajv|ajv-formats|@cfworker\/json-schema)["']/m;
+const ROOT_VALIDATOR_EXPORTS = ['AjvJsonSchemaValidator', 'CfWorkerJsonSchemaValidator', 'CfWorkerSchemaDraft'];
 
 function chunkImportsOf(entryPath: string): string[] {
     const visited = new Set<string>();
@@ -27,6 +28,12 @@ function chunkImportsOf(entryPath: string): string[] {
     return [...visited];
 }
 
+function rootExportBlockOf(content: string): string {
+    const blocks = content.match(/(?:^|\n)export \{[\s\S]*?\};/g);
+    expect(blocks).toBeTruthy();
+    return blocks!.at(-1)!;
+}
+
 describe('@modelcontextprotocol/server root entry is browser-safe', () => {
     beforeAll(() => {
         if (!existsSync(join(distDir, 'index.mjs')) || !existsSync(join(distDir, 'stdio.mjs'))) {
@@ -68,4 +75,13 @@ describe('@modelcontextprotocol/server root entry is browser-safe', () => {
             }
         }
     });
+
+    test('root declarations do not advertise validator provider classes', () => {
+        for (const declaration of ['index.d.mts', 'index.d.cts']) {
+            const rootExportBlock = rootExportBlockOf(readFileSync(join(distDir, declaration), 'utf8'));
+            for (const symbol of ROOT_VALIDATOR_EXPORTS) {
+                expect(rootExportBlock).not.toMatch(new RegExp(`\\b(?:type\\s+)?${symbol}\\b`));
+            }
+        }
+    });
 });