Skip to content

Extract catalog MCP tools#595

Merged
w1ne merged 5 commits into
developfrom
codex/mcp-catalog-tools
Jul 9, 2026
Merged

Extract catalog MCP tools#595
w1ne merged 5 commits into
developfrom
codex/mcp-catalog-tools

Conversation

@w1ne

@w1ne w1ne commented Jul 9, 2026

Copy link
Copy Markdown
Owner

Summary

  • move lookup_cookbook, find_part, and fetch_part registry entries into src/agent/mcp/registry/catalogTools.ts
  • move internal MCP registry entry types into src/agent/mcp/registry/types.ts while preserving the public McpToolDefinition export from toolRegistry.ts
  • add a public-contract assertion that catalog entries compose back into the same registry slots

Review

  • subagent review: no findings; confirmed no docs/fixtures changed, no catalog module import cycle, public fixture coverage intact

Verification

  • red: focused public contract test failed before implementation because ./registry/catalogTools did not exist
  • npm test -- src/agent/mcp/toolRegistry.publicContract.test.ts
  • npm test -- src/agent/mcp/toolRegistry.publicContract.test.ts src/agent/mcp/toolNameConsistency.test.ts tests/integration/mcp/serverToolDispatch.test.ts tests/unit/mcp/toolRegistrySchema.test.ts tests/unit/scripts/distToolNameGate.test.ts tests/unit/skill/skillToolCountDrift.test.ts tests/integration/mcp/listApi.driftSentinel.test.ts
  • npm run typecheck

w1ne added 2 commits July 9, 2026 02:06
Signed-off-by: Andrii Shylenko <14119286+w1ne@users.noreply.github.com>
@w1ne w1ne enabled auto-merge July 9, 2026 00:07
@w1ne w1ne merged commit ee7f621 into develop Jul 9, 2026
13 checks passed
@w1ne w1ne deleted the codex/mcp-catalog-tools branch July 9, 2026 00:35
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant