Skip to content

Extract review pipeline MCP tools#601

Merged
w1ne merged 1 commit into
developfrom
codex/mcp-review-pipeline
Jul 9, 2026
Merged

Extract review pipeline MCP tools#601
w1ne merged 1 commit into
developfrom
codex/mcp-review-pipeline

Conversation

@w1ne

@w1ne w1ne commented Jul 9, 2026

Copy link
Copy Markdown
Owner

Summary

  • Extract review/render pipeline MCP registry entries into src/agent/mcp/registry/reviewPipelineTools.ts.
  • Keep TOOL_REGISTRY behavior and public order by spreading reviewPipelineToolEntries at the original tail position.
  • Add a public contract assertion for the extracted review/render pipeline slice.

Moved tools

  • review_cad
  • review_paint_peek_latest
  • design_loop
  • flatten_pattern
  • evaluate_sdf
  • capture_animation
  • render_preview

Review

  • Subagent review: no findings. Reviewer confirmed moved entries preserve order, casts, schemas, handlers, public exports, and touched-file scope.

Verification

  • Baseline focused MCP registry suite before edits: 7 files / 47 tests passed.
  • TDD red: contract test failed on missing ./registry/reviewPipelineTools module.
  • Contract test after extraction: 14/14 tests passed.
  • Focused MCP registry suite after extraction: 7 files / 48 tests passed.
  • npm run typecheck passed; only existing TanStack route warnings appeared.
  • Touched-file ESLint passed.
  • git diff --check passed.
  • Mechanical block comparison: moved review/render block is byte-identical to the original tail block.
  • No docs files changed.

Signed-off-by: Andrii Shylenko <14119286+w1ne@users.noreply.github.com>
@w1ne w1ne force-pushed the codex/mcp-review-pipeline branch from e9732fb to 8884830 Compare July 9, 2026 10:37
@w1ne w1ne enabled auto-merge (squash) July 9, 2026 10:38
@w1ne w1ne merged commit 48dacd3 into develop Jul 9, 2026
13 checks passed
@w1ne w1ne deleted the codex/mcp-review-pipeline branch July 9, 2026 10:58
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