Skip to content

Commit 87dccb1

Browse files
committed
refactor(codegen): remove all MCP functionality from codegen
Remove MCP (Model Context Protocol) tool generation, mcp.json file generation, and related config/types/tests. MCP was generating static JSON tool definition files that weren't providing value. Removed: - getCliMcpTools(), getMultiTargetCliMcpTools() from cli/docs-generator.ts - getOrmMcpTools() from orm/docs-generator.ts - getHooksMcpTools() from hooks-docs-generator.ts - generateCombinedMcpConfig() from target-docs-generator.ts - McpTool interface from docs-utils.ts - mcp option from DocsConfig interface and resolveDocsConfig() - All MCP imports and if(docsConfig.mcp) blocks from generate.ts - 5 MCP test cases and 2 snapshots from cli-generator.test.ts - McpTool re-exports from cli/index.ts barrel
1 parent e834f20 commit 87dccb1

10 files changed

Lines changed: 15 additions & 2665 deletions

File tree

graphql/codegen/src/__tests__/codegen/__snapshots__/cli-generator.test.ts.snap

Lines changed: 0 additions & 1628 deletions
Large diffs are not rendered by default.

graphql/codegen/src/__tests__/codegen/cli-generator.test.ts

Lines changed: 5 additions & 64 deletions
Original file line numberDiff line numberDiff line change
@@ -1,27 +1,22 @@
11
import { generateCli, generateMultiTargetCli, resolveBuiltinNames } from '../../core/codegen/cli';
22
import {
33
generateReadme as generateCliReadme,
4-
getCliMcpTools,
54
generateSkills as generateCliSkills,
65
generateMultiTargetReadme,
7-
getMultiTargetCliMcpTools,
86
generateMultiTargetSkills,
97
} from '../../core/codegen/cli/docs-generator';
108
import type { MultiTargetDocsInput } from '../../core/codegen/cli/docs-generator';
119
import { resolveDocsConfig } from '../../core/codegen/docs-utils';
1210
import {
1311
generateOrmReadme,
14-
getOrmMcpTools,
1512
generateOrmSkills,
1613
} from '../../core/codegen/orm/docs-generator';
1714
import {
1815
generateHooksReadme,
19-
getHooksMcpTools,
2016
generateHooksSkills,
2117
} from '../../core/codegen/hooks-docs-generator';
2218
import {
2319
generateTargetReadme,
24-
generateCombinedMcpConfig,
2520
generateRootRootReadme,
2621
} from '../../core/codegen/target-docs-generator';
2722
import type {
@@ -264,16 +259,6 @@ describe('cli docs generator', () => {
264259
expect(readme.content).toMatchSnapshot();
265260
});
266261

267-
it('generates CLI MCP tools', () => {
268-
const tools = getCliMcpTools([carTable, driverTable], allCustomOps, 'myapp');
269-
expect(tools.length).toBeGreaterThan(0);
270-
for (const tool of tools) {
271-
expect(tool.name).toBeDefined();
272-
expect(tool.description).toBeDefined();
273-
expect(tool.inputSchema).toBeDefined();
274-
}
275-
});
276-
277262
it('generates CLI skill files', () => {
278263
const skills = generateCliSkills([carTable, driverTable], allCustomOps, 'myapp', 'default');
279264
expect(skills.length).toBeGreaterThan(0);
@@ -290,16 +275,6 @@ describe('orm docs generator', () => {
290275
expect(readme.content).toMatchSnapshot();
291276
});
292277

293-
it('generates ORM MCP tools', () => {
294-
const tools = getOrmMcpTools([carTable, driverTable], allCustomOps);
295-
expect(tools.length).toBeGreaterThan(0);
296-
for (const tool of tools) {
297-
expect(tool.name).toBeDefined();
298-
expect(tool.description).toBeDefined();
299-
expect(tool.inputSchema).toBeDefined();
300-
}
301-
});
302-
303278
it('generates ORM skill files', () => {
304279
const skills = generateOrmSkills([carTable, driverTable], allCustomOps, 'default');
305280
expect(skills.length).toBeGreaterThan(0);
@@ -316,16 +291,6 @@ describe('hooks docs generator', () => {
316291
expect(readme.content).toMatchSnapshot();
317292
});
318293

319-
it('generates hooks MCP tools', () => {
320-
const tools = getHooksMcpTools([carTable, driverTable], allCustomOps);
321-
expect(tools.length).toBeGreaterThan(0);
322-
for (const tool of tools) {
323-
expect(tool.name).toBeDefined();
324-
expect(tool.description).toBeDefined();
325-
expect(tool.inputSchema).toBeDefined();
326-
}
327-
});
328-
329294
it('generates hooks skill files', () => {
330295
const skills = generateHooksSkills([carTable, driverTable], allCustomOps, 'default');
331296
expect(skills.length).toBeGreaterThan(0);
@@ -350,18 +315,6 @@ describe('target docs generator', () => {
350315
expect(readme.content).toMatchSnapshot();
351316
});
352317

353-
it('generates combined MCP config', () => {
354-
const cliTools = getCliMcpTools([carTable, driverTable], allCustomOps, 'myapp');
355-
const ormTools = getOrmMcpTools([carTable, driverTable], allCustomOps);
356-
const allTools = [...cliTools, ...ormTools];
357-
const mcp = generateCombinedMcpConfig(allTools, 'myapp');
358-
expect(mcp.fileName).toBe('mcp.json');
359-
const parsed = JSON.parse(mcp.content);
360-
expect(parsed.name).toBe('myapp');
361-
expect(parsed.tools.length).toBe(allTools.length);
362-
expect(mcp.content).toMatchSnapshot();
363-
});
364-
365318
it('generates root-root README for multi-target', () => {
366319
const readme = generateRootRootReadme([
367320
{ name: 'auth', output: './generated/auth', endpoint: 'http://auth.localhost/graphql', generators: ['ORM'] },
@@ -375,22 +328,22 @@ describe('target docs generator', () => {
375328
describe('resolveDocsConfig', () => {
376329
it('defaults to readme + agents', () => {
377330
const config = resolveDocsConfig(undefined);
378-
expect(config).toEqual({ readme: true, agents: true, mcp: false, skills: false });
331+
expect(config).toEqual({ readme: true, agents: true, skills: false });
379332
});
380333

381334
it('docs: true enables all', () => {
382335
const config = resolveDocsConfig(true);
383-
expect(config).toEqual({ readme: true, agents: true, mcp: true, skills: true });
336+
expect(config).toEqual({ readme: true, agents: true, skills: true });
384337
});
385338

386339
it('docs: false disables all', () => {
387340
const config = resolveDocsConfig(false);
388-
expect(config).toEqual({ readme: false, agents: false, mcp: false, skills: false });
341+
expect(config).toEqual({ readme: false, agents: false, skills: false });
389342
});
390343

391344
it('partial config fills defaults', () => {
392-
const config = resolveDocsConfig({ mcp: true });
393-
expect(config).toEqual({ readme: true, agents: true, mcp: true, skills: false });
345+
const config = resolveDocsConfig({ skills: true });
346+
expect(config).toEqual({ readme: true, agents: true, skills: true });
394347
});
395348
});
396349

@@ -713,18 +666,6 @@ describe('multi-target cli docs', () => {
713666
expect(readme.content).toMatchSnapshot();
714667
});
715668

716-
it('generates multi-target MCP tools', () => {
717-
const tools = getMultiTargetCliMcpTools(docsInput);
718-
expect(tools.length).toBeGreaterThan(0);
719-
const toolNames = tools.map((t) => t.name);
720-
expect(toolNames).toContain('myapp_credentials_set_token');
721-
expect(toolNames).toContain('myapp_context_create');
722-
expect(toolNames.some((n) => n.includes('auth_user'))).toBe(true);
723-
expect(toolNames.some((n) => n.includes('members_member'))).toBe(true);
724-
expect(toolNames.some((n) => n.includes('app_car'))).toBe(true);
725-
expect(tools).toMatchSnapshot();
726-
});
727-
728669
it('generates multi-target skills', () => {
729670
const skills = generateMultiTargetSkills(docsInput);
730671
expect(skills.length).toBeGreaterThan(0);

0 commit comments

Comments
 (0)