refactor(server): move schema-derivation helper out of src to test-only

specToToolDefinition was no longer on the live path (McpServer derives each
tool's advertised JSON Schema from its Zod inputShape), surviving only to let
unit tests assert that advertised schema. Move it to a test helper
(packages/server/test/tool-schema.ts as toToolDefinition) and drop the 80
per-tool xToolDefinition exports it fed. spec.ts is now just the ToolSpec /
ToolKind types. Per-tool tests re-derive their definition through the helper, so
their assertions (required fields, property types a client sees) are unchanged.

No production code path changes — live tool registration still runs off the Zod
specs in ALL_TOOL_SPECS. 504 tests green, typecheck/lint/build clean; knip's
unused-export count drops from 94 to 46.

Author: awdr74100

packages/server/src/tools/add-page.ts

@@ -1,6 +1,6 @@
 import { z } from 'zod';
 
-import { specToToolDefinition, type ToolSpec } from './spec.js';
+import type { ToolSpec } from './spec.js';
 
 export const ADD_PAGE_TOOL_NAME = 'add_page';
 
@@ -12,5 +12,3 @@ export const addPageTool: ToolSpec = {
   },
   kind: 'write',
 };
-
-export const addPageToolDefinition = specToToolDefinition(addPageTool);

packages/server/src/tools/add-variable-mode.ts

@@ -1,6 +1,6 @@
 import { z } from 'zod';
 
-import { specToToolDefinition, type ToolSpec } from './spec.js';
+import type { ToolSpec } from './spec.js';
 
 export const ADD_VARIABLE_MODE_TOOL_NAME = 'add_variable_mode';
 
@@ -13,5 +13,3 @@ export const addVariableModeTool: ToolSpec = {
   },
   kind: 'write',
 };
-
-export const addVariableModeToolDefinition = specToToolDefinition(addVariableModeTool);

packages/server/src/tools/analyze-project.ts

@@ -1,7 +1,7 @@
 import { z } from 'zod';
 
 import { analyzeProject, type ProjectProfile } from '../profile/profile.js';
-import { specToToolDefinition, type ToolSpec } from './spec.js';
+import type { ToolSpec } from './spec.js';
 
 export const ANALYZE_PROJECT_TOOL_NAME = 'analyze_project';
 
@@ -21,9 +21,6 @@ export const analyzeProjectTool: ToolSpec = {
   inputShape,
   kind: 'local',
 };
-
-export const analyzeProjectToolDefinition = specToToolDefinition(analyzeProjectTool);
-
 export const handleAnalyzeProject = async (rawArgs: unknown): Promise<ProjectProfile> => {
   const args = z.object(inputShape).parse(rawArgs);
   return analyzeProject(args.rootDir ?? process.cwd());

packages/server/src/tools/apply-style-to-node.ts

@@ -1,6 +1,6 @@
 import { z } from 'zod';
 
-import { specToToolDefinition, type ToolSpec } from './spec.js';
+import type { ToolSpec } from './spec.js';
 
 export const APPLY_STYLE_TO_NODE_TOOL_NAME = 'apply_style_to_node';
 
@@ -16,5 +16,3 @@ export const applyStyleToNodeTool: ToolSpec = {
   },
   kind: 'write',
 };
-
-export const applyStyleToNodeToolDefinition = specToToolDefinition(applyStyleToNodeTool);

packages/server/src/tools/batch-rename-nodes.ts

@@ -1,6 +1,6 @@
 import { z } from 'zod';
 
-import { specToToolDefinition, type ToolSpec } from './spec.js';
+import type { ToolSpec } from './spec.js';
 
 export const BATCH_RENAME_NODES_TOOL_NAME = 'batch_rename_nodes';
 
@@ -16,5 +16,3 @@ export const batchRenameNodesTool: ToolSpec = {
   },
   kind: 'write',
 };
-
-export const batchRenameNodesToolDefinition = specToToolDefinition(batchRenameNodesTool);

packages/server/src/tools/batch.ts

@@ -1,6 +1,6 @@
 import { z } from 'zod';
 
-import { specToToolDefinition, type ToolSpec } from './spec.js';
+import type { ToolSpec } from './spec.js';
 
 export const BATCH_TOOL_NAME = 'batch';
 
@@ -32,5 +32,3 @@ export const batchTool: ToolSpec = {
   },
   kind: 'write',
 };
-
-export const batchToolDefinition = specToToolDefinition(batchTool);

packages/server/src/tools/bind-variable-to-node.ts

@@ -1,6 +1,6 @@
 import { z } from 'zod';
 
-import { specToToolDefinition, type ToolSpec } from './spec.js';
+import type { ToolSpec } from './spec.js';
 
 export const BIND_VARIABLE_TO_NODE_TOOL_NAME = 'bind_variable_to_node';
 
@@ -16,5 +16,3 @@ export const bindVariableToNodeTool: ToolSpec = {
   },
   kind: 'write',
 };
-
-export const bindVariableToNodeToolDefinition = specToToolDefinition(bindVariableToNodeTool);

packages/server/src/tools/clone-node.ts

@@ -1,6 +1,6 @@
 import { z } from 'zod';
 
-import { specToToolDefinition, type ToolSpec } from './spec.js';
+import type { ToolSpec } from './spec.js';
 
 export const CLONE_NODE_TOOL_NAME = 'clone_node';
 
@@ -13,5 +13,3 @@ export const cloneNodeTool: ToolSpec = {
   },
   kind: 'write',
 };
-
-export const cloneNodeToolDefinition = specToToolDefinition(cloneNodeTool);

packages/server/src/tools/component-map.ts

@@ -19,7 +19,7 @@ import { analyzeProject, type ProjectProfile } from '../profile/profile.js';
 import { scanComponents } from '../scan/scan.js';
 import { GET_DESIGN_CONTEXT_TOOL_NAME } from './get-design-context.js';
 import { GET_LOCAL_COMPONENTS_TOOL_NAME } from './get-local-components.js';
-import { specToToolDefinition, type ToolSpec } from './spec.js';
+import type { ToolSpec } from './spec.js';
 
 export const COMPONENT_MAP_TOOL_NAME = 'component_map';
 
@@ -58,9 +58,6 @@ export const componentMapTool: ToolSpec = {
   inputShape,
   kind: 'local',
 };
-
-export const componentMapToolDefinition = specToToolDefinition(componentMapTool);
-
 export type ToolDispatcher = (toolName: string, args: unknown) => Promise<unknown>;
 
 const readOverrides = async (rootDir: string): Promise<ReturnType<typeof parseMapFile>> => {

packages/server/src/tools/create-component.ts

@@ -1,6 +1,6 @@
 import { z } from 'zod';
 
-import { specToToolDefinition, type ToolSpec } from './spec.js';
+import type { ToolSpec } from './spec.js';
 
 export const CREATE_COMPONENT_TOOL_NAME = 'create_component';
 
@@ -19,5 +19,3 @@ export const createComponentTool: ToolSpec = {
   },
   kind: 'write',
 };
-
-export const createComponentToolDefinition = specToToolDefinition(createComponentTool);