fix: remove cyclic workspace dependency between test-helpers and client/server (#1444)

felixweinberger · web-flow · commit 64f27820fa63 · 2026-02-02T17:30:09.000Z
diff --git a/pnpm-lock.yaml b/pnpm-lock.yaml
diff --git a/test/helpers/package.json b/test/helpers/package.json
@@ -31,8 +31,6 @@
     },
     "devDependencies": {
         "@modelcontextprotocol/core": "workspace:^",
-        "@modelcontextprotocol/client": "workspace:^",
-        "@modelcontextprotocol/server": "workspace:^",
         "zod": "catalog:runtimeShared",
         "vitest": "catalog:devTools",
         "@modelcontextprotocol/tsconfig": "workspace:^",
diff --git a/test/helpers/tsconfig.json b/test/helpers/tsconfig.json
@@ -1,13 +1,11 @@
 {
     "extends": "@modelcontextprotocol/tsconfig",
-    "include": ["./", "../integration/test/helpers/mcp.ts"],
+    "include": ["./"],
     "exclude": ["node_modules", "dist"],
     "compilerOptions": {
         "paths": {
             "*": ["./*"],
             "@modelcontextprotocol/core": ["./node_modules/@modelcontextprotocol/core/src/index.ts"],
-            "@modelcontextprotocol/client": ["./node_modules/@modelcontextprotocol/client/src/index.ts"],
-            "@modelcontextprotocol/server": ["./node_modules/@modelcontextprotocol/server/src/index.ts"],
             "@modelcontextprotocol/vitest-config": ["./node_modules/@modelcontextprotocol/vitest-config/tsconfig.json"]
         }
     }

Original file line number	Diff line number	Diff line change
`@@ -1,13 +1,11 @@`
`1`	`1`	`{`
`2`	`2`	`"extends": "@modelcontextprotocol/tsconfig",`
`3`		`- "include": ["./", "../integration/test/helpers/mcp.ts"],`
	`3`	`+ "include": ["./"],`
`4`	`4`	`"exclude": ["node_modules", "dist"],`
`5`	`5`	`"compilerOptions": {`
`6`	`6`	`"paths": {`
`7`	`7`	`"": ["./"],`
`8`	`8`	`"@modelcontextprotocol/core": ["./node_modules/@modelcontextprotocol/core/src/index.ts"],`
`9`		`- "@modelcontextprotocol/client": ["./node_modules/@modelcontextprotocol/client/src/index.ts"],`
`10`		`- "@modelcontextprotocol/server": ["./node_modules/@modelcontextprotocol/server/src/index.ts"],`
`11`	`9`	`"@modelcontextprotocol/vitest-config": ["./node_modules/@modelcontextprotocol/vitest-config/tsconfig.json"]`
`12`	`10`	`}`
`13`	`11`	`}`