-
Notifications
You must be signed in to change notification settings - Fork 26
feat: simple MCP server #712
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
base: main
Are you sure you want to change the base?
Conversation
|
☂️ Python Coverage
Overall Coverage
New Files
Modified Files
|
import pytest | ||
|
||
# Skip entire module if fastmcp is not available (requires Python 3.10+) | ||
pytest.importorskip("fastmcp", reason="fastmcp library not installed (requires Python 3.10+)") |
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.
Need this to make sure we don't try to run this test on tests being run on python<3.10
Available top-level commands: | ||
{commands_list} |
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.
❤️ . Great stuff. Makes this evergreen
Can you please add info about this feature in the readme, docs, etc? |
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.
Looks good and works! Approved but would recommend adding the necessary docs. Good work!
@mconflitti-pbc Do you think we need to list the various ways to add this to some of the popular LLM clients? (i.e claude code, vscode, cursor). While useful, I worry about this being susceptible to going stale quickly. Here's the current state of the docs for the command: https://posit-dev.github.io/rsconnect-python/pr-preview/pr-712/commands/mcp-server/ |
c3e39ca
to
1f6ef33
Compare
Maybe just link out to the docs on how to do that for the most popular tools: codex, claude, vs code |
1f6ef33
to
d68caea
Compare
d68caea
to
fc38445
Compare
Intent
Add a simple MCP server with the goal of improving LLM interactions with the rsconnect cli
Resolves https://github.com/posit-dev/connect/issues/33653
Type of Change
Approach
Compile structured context about each cli command by walking the click decorators at MCP server startup
deploy shiny
,content build add
, etc)Automated Tests
Added unit tests for command parameter schema discovery
Directions for Reviewers
Manually inspect using
try out for yourself locally using claude:
Checklist