Skip to content

Extract sketch assembly MCP tools#598

Merged
w1ne merged 1 commit into
developfrom
codex/mcp-sketch-assembly
Jul 9, 2026
Merged

Extract sketch assembly MCP tools#598
w1ne merged 1 commit into
developfrom
codex/mcp-sketch-assembly

Conversation

@w1ne

@w1ne w1ne commented Jul 9, 2026

Copy link
Copy Markdown
Owner

Summary

  • move sketch/assembly MCP registry entries into src/agent/mcp/registry/sketchAssemblyTools.ts
  • compose that module back into TOOL_REGISTRY at the same slot
  • add a public-contract guard for the sketch/assembly slice

Moved tools: solve_sketch, add_constraint, add_part, add_connector, add_mate, add_workspace_target, set_scene_return, solve_mates.

Review

  • subagent review: no findings; confirmed no docs/fixtures changed, no import cycle, handlers/metadata/order preserved

Verification

  • baseline: focused MCP registry suite passed on origin/develop (7 files / 46 tests)
  • red: public contract test failed because ./registry/sketchAssemblyTools 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
  • npx eslint src/agent/mcp/toolRegistry.ts src/agent/mcp/registry/sketchAssemblyTools.ts src/agent/mcp/toolRegistry.publicContract.test.ts

Signed-off-by: Andrii Shylenko <14119286+w1ne@users.noreply.github.com>
@w1ne w1ne force-pushed the codex/mcp-sketch-assembly branch from 81d6b7a to 81125c4 Compare July 9, 2026 09:53
@w1ne w1ne enabled auto-merge (squash) July 9, 2026 09:57
@w1ne w1ne merged commit a47f8d6 into develop Jul 9, 2026
13 checks passed
@w1ne w1ne deleted the codex/mcp-sketch-assembly branch July 9, 2026 10:08
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