Skip to content

Extract geometry authoring MCP tools#597

Merged
w1ne merged 2 commits into
developfrom
codex/mcp-geometry-authoring
Jul 9, 2026
Merged

Extract geometry authoring MCP tools#597
w1ne merged 2 commits into
developfrom
codex/mcp-geometry-authoring

Conversation

@w1ne

@w1ne w1ne commented Jul 9, 2026

Copy link
Copy Markdown
Owner

Summary

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

Moved tools: add_feature, add_surface, add_curve, add_path_segment, trace_from_image, add_variable_sweep, add_text, project_curve, add_pattern_feature, remove_feature.

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 / 45 tests)
  • red: public contract test failed because ./registry/geometryAuthoringTools 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/geometryAuthoringTools.ts src/agent/mcp/toolRegistry.publicContract.test.ts

w1ne added 2 commits July 9, 2026 02:50
Signed-off-by: Andrii Shylenko <14119286+w1ne@users.noreply.github.com>
@w1ne
w1ne merged commit 2457f51 into develop Jul 9, 2026
13 checks passed
@w1ne
w1ne deleted the codex/mcp-geometry-authoring branch July 9, 2026 01:11
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