chore(v2): deprecate legacy MCP wrapper entrypoints#55
Conversation
- 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
|
Warning Rate limit exceeded
⌛ How to resolve this issue?After the wait time has elapsed, a review can be triggered using the 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)
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. Comment |
🔗 Integration Test Results🔗 Cross-Agent Integration Test ReportSession ID: integration-20260208-165542-a258adb5154d17be61b6c35511811575e7472349 Summary
Test Results
Recommendations
Next Steps
Generated by Cross-Agent Integration Test Pipeline |
Summary
@deprecatedmarkers and runtime notices in:v2/src/mcp/server-with-wrapper.tsv2/src/mcp/server-wrapper-mode.tsv2/src/mcp/integrate-wrapper.tsclaude-flow mcp start) in:v2/src/mcp/README.mdv2/docs/MCP_WRAPPER_ENTRYPOINTS_DEPRECATION.mdv2/CHANGELOG.mdEvidence
v2/package.jsonbin wiring only exposesbin/claude-flow.jsValidation
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));"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.mdcd v2 && npm run typecheck -- --pretty false(baseline repo has many pre-existing TS errors)Closes #51