-
Notifications
You must be signed in to change notification settings - Fork 2k
feat(skill): enable advanced MCP tool use with mcpx #950
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Conversation
Adds mcpx as a builtin skill providing: - On-demand MCP tool discovery without upfront token cost - Daemon mode for stateful operations (browser, database) - Clear decision tree for when to use daemon vs direct calls Addresses pain points from: - code-yeongyu#882: MCP startup delay (daemon provides instant availability) - code-yeongyu#878: Tools not exposed despite connection (reliable discovery) - code-yeongyu#949: No tool preference system (mcpx grep enables selection)
|
All contributors have signed the CLA. Thank you! ✅ |
|
I have read the CLA Document and I hereby sign the CLA |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
No issues found across 1 file
Confidence score: 5/5
- Automated review surfaced no issues in the provided summaries.
- No files require special attention.
Since this is your first cubic review, here's how it works:
- cubic automatically reviews your code and comments on bugs and improvements
- Teach cubic by replying to its comments. cubic learns from your replies and gets better over time
- Ask questions if you need clarification on any suggestion
|
Thank you for this contribution! I need some time to think about whether this approach aligns with the project direction. Will get back to you soon. |
|
sounds good, i personally switched to using the cli for all mcps and haven't found a reason to switch back since. |
|
Thank you for this contribution! I've reviewed the PR and decided to close it for the following reasons:
The mcpx CLI approach doesn't align with the project's direction of building native, integrated solutions rather than relying on external CLI tools. Feel free to reopen if you'd like to discuss an alternative approach that integrates natively or if you have questions about the decision. |
|
understandable. Appreciate the feedback. |
Summary
Adds mcpx as a builtin skill providing lightweight MCP tool access via CLI.
What mcpx provides:
mcpx server/tool '{"args": ...}'Pain points addressed:
mcpx/mcpx grepmcpx grep "*pattern*"enables tool selectionChanges
mcpxSkilltosrc/features/builtin-skills/skills.tsTest plan
bun run typecheckpasses/skill mcpxin OpenCodeNote
mcpx is available at: https://github.com/cs50victor/mcpx
Install:
brew install cs50victor/tap/mcpxorbun install -g mcpxSummary by cubic
Adds mcpx as a builtin skill to discover and invoke MCP tools via a simple CLI, with a daemon mode for stateful workflows. Improves tool availability and selection, addressing #882 (startup delay), #878 (tools not exposed), and #949 (tool preference).
Written for commit 58a659c. Summary will update on new commits.