Skip to content

chore(v2): deprecate legacy MCP wrapper entrypoints#55

Merged
blackms merged 1 commit intomainfrom
codex/mcp-wrapper-deprecation-20260208
Feb 8, 2026
Merged

chore(v2): deprecate legacy MCP wrapper entrypoints#55
blackms merged 1 commit intomainfrom
codex/mcp-wrapper-deprecation-20260208

Conversation

@blackms
Copy link
Owner

@blackms blackms commented Feb 8, 2026

Summary

  • classify legacy MCP wrapper entrypoint files as deprecate-first (not delete)
  • add explicit @deprecated markers and runtime notices in:
    • v2/src/mcp/server-with-wrapper.ts
    • v2/src/mcp/server-wrapper-mode.ts
    • v2/src/mcp/integrate-wrapper.ts
  • document supported MCP startup contract (claude-flow mcp start) in:
    • v2/src/mcp/README.md
    • v2/docs/MCP_WRAPPER_ENTRYPOINTS_DEPRECATION.md
  • record deprecation in v2/CHANGELOG.md

Evidence

  • v2/package.json bin wiring only exposes bin/claude-flow.js
  • no script/bin wiring for wrapper entrypoint files
  • wrapper-specific env flags only appear in wrapper files and wrapper class messaging

Validation

  • wrapper wiring check:
    • node -e "const pkg=require('./v2/package.json'); console.log(JSON.stringify({bin:pkg.bin,mcpScriptKeys:Object.keys(pkg.scripts).filter(k=>k.includes('mcp'))},null,2));"
  • deprecation markers check:
    • rg -n "\[DEPRECATED\]|@deprecated|claude-flow mcp start" v2/src/mcp/server-with-wrapper.ts v2/src/mcp/server-wrapper-mode.ts v2/src/mcp/integrate-wrapper.ts v2/src/mcp/README.md v2/docs/MCP_WRAPPER_ENTRYPOINTS_DEPRECATION.md
  • cd v2 && npm run typecheck -- --pretty false (baseline repo has many pre-existing TS errors)

Closes #51

- mark wrapper entrypoint files as deprecated internal paths

- add deprecation runtime notices with supported replacement command

- document staged removal plan and entrypoint contract

Refs: #51
@coderabbitai
Copy link

coderabbitai bot commented Feb 8, 2026

Warning

Rate limit exceeded

@blackms has exceeded the limit for the number of commits that can be reviewed per hour. Please wait 10 minutes and 54 seconds before requesting another review.

⌛ How to resolve this issue?

After the wait time has elapsed, a review can be triggered using the @coderabbitai review command as a PR comment. Alternatively, push new commits to this PR.

We recommend that you space out your commits to avoid hitting the rate limit.

🚦 How do rate limits work?

CodeRabbit enforces hourly rate limits for each developer per organization.

Our paid plans have higher rate limits than the trial, open-source and free plans. In all cases, we re-allow further reviews after a brief timeout.

Please see our FAQ for further information.

✨ Finishing touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Post copyable unit tests in a comment
  • Commit unit tests in branch codex/mcp-wrapper-deprecation-20260208

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

@blackms blackms merged commit 676d912 into main Feb 8, 2026
5 of 6 checks passed
@blackms blackms deleted the codex/mcp-wrapper-deprecation-20260208 branch February 8, 2026 16:55
@github-actions
Copy link

github-actions bot commented Feb 8, 2026

🔗 Integration Test Results

🔗 Cross-Agent Integration Test Report

Session ID: integration-20260208-165542-a258adb5154d17be61b6c35511811575e7472349
Timestamp: 2026-02-08T16:57:05.951Z
Overall Status: ✅ PASSED

Summary

  • Total Tests: 4
  • Passed: 4
  • Failed: 0
  • Success Rate: 100.0%

Test Results

Component Status Details
Agent Coordination Multi-agent communication and task distribution
Memory Integration Shared memory operations and synchronization
Fault Tolerance Failure recovery and system resilience
Performance Multi-agent performance and scalability

Recommendations

  • All integration tests passed successfully!

Next Steps

  1. Review detailed test artifacts
  2. Address any failed test scenarios
  3. Monitor integration performance in production

Generated by Cross-Agent Integration Test Pipeline

@blackms blackms restored the codex/mcp-wrapper-deprecation-20260208 branch February 8, 2026 17:17
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

1 participant