feat: add remote MCP to cloud skills with tests - #5
Merged
Conversation
There was a problem hiding this comment.
Pull request overview
This PR updates the skill docs to support a “remote MCP-first” execution path (StreamNative Cloud sncloud_* domain tools + feature-gated pulsar_* / kafka_* tools), while keeping snctl as a fallback for gaps and CLI-specific workflows.
Changes:
- Add a central remote MCP reference doc and wire it into all domain skills as the preferred execution base.
- Update each skill’s prerequisites/pitfalls to document JSON-string manifest + dry-run-first remote MCP write workflows and cluster-selection via
sncloud_context_*. - Add a parallel set of remote MCP smoke-test walkthroughs and update top-level docs/plugin metadata to reflect remote MCP support.
Reviewed changes
Copilot reviewed 23 out of 24 changed files in this pull request and generated 2 comments.
Show a summary per file
| File | Description |
|---|---|
| skills/workspaces/SKILL.md | Adds remote MCP execution-base guidance for Workspace control-plane + feature-gated runtime tools, bumps version. |
| skills/pulsar-clusters/SKILL.md | Documents sncloud_clusters_* remote MCP flow for Pulsar instance/cluster/gateway management, bumps version. |
| skills/pulsar-admin/SKILL.md | Adds remote MCP route for Pulsar data-plane admin/client tools and cluster selection, bumps version. |
| skills/kafka-clusters/SKILL.md | Adds remote MCP route for Instance/KafkaCluster control-plane and data-plane context selection, bumps version. |
| skills/kafka-admin/SKILL.md | Adds remote MCP route for Kafka admin/client tools and cluster selection, bumps version. |
| skills/identity/SKILL.md | Adds remote MCP route for identity resources + explicitly documents snctl-only gaps (APIKey/Secret), bumps version. |
| skills/cloud-core/SKILL.md | Introduces remote MCP discovery/context/log workflows alongside snctl setup guidance, bumps version. |
| skills/cloud-core/references/troubleshooting.md | Adjusts troubleshooting section title to remove snctl mcp mention. |
| skills/cloud-core/references/remote-mcp.md | New reference describing remote MCP session model, tool families, and safe mutation workflow. |
| skills/byoc-infrastructure/SKILL.md | Adds remote MCP route for BYOC resources and dry-run-first JSON manifest guidance, bumps version. |
| README.md | Updates project description/feature list to include remote MCP and adds prerequisite guidance for both routes. |
| docs/smoke-tests/remote-mcp/README.md | New remote MCP smoke-test README and acceptance criteria. |
| docs/smoke-tests/remote-mcp/cloud-core.md | New remote MCP smoke walkthrough for cloud-core flows. |
| docs/smoke-tests/remote-mcp/byoc-infrastructure.md | New remote MCP smoke walkthrough for BYOC domain. |
| docs/smoke-tests/remote-mcp/identity.md | New remote MCP smoke walkthrough for identity domain. |
| docs/smoke-tests/remote-mcp/pulsar-clusters.md | New remote MCP smoke walkthrough for Pulsar cluster lifecycle. |
| docs/smoke-tests/remote-mcp/pulsar-admin.md | New remote MCP smoke walkthrough for Pulsar data-plane admin/client. |
| docs/smoke-tests/remote-mcp/kafka-clusters.md | New remote MCP smoke walkthrough for Kafka cluster lifecycle + selection for data-plane step. |
| docs/smoke-tests/remote-mcp/kafka-admin.md | New remote MCP smoke walkthrough for Kafka data-plane admin/client. |
| docs/smoke-tests/remote-mcp/workspaces.md | New remote MCP smoke walkthrough for Workspace + packages/connectors/agents. |
| docs/smoke-tests/README.md | Clarifies that top-level smoke tests are snctl route and remote-mcp/ is MCP route. |
| CONTRIBUTING.md | Updates contribution guidance to include remote MCP domain tools and JSON-manifest/dry-run workflow. |
| .gitignore | Adds /PLAN.md ignore entry. |
| .claude-plugin/plugin.json | Updates plugin description and keywords to include remote MCP/MCP. |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
…cp-server # Conflicts: # README.md # skills/kafka-clusters/SKILL.md
There was a problem hiding this comment.
Pull request overview
Copilot reviewed 56 out of 57 changed files in this pull request and generated 2 comments.
Comments suppressed due to low confidence (1)
docker/smoke/README.md:31
- These steps
cdinto/Users/rui/playground/sn/streamnative-cloud-skills, which doesn’t match this repository name/path. Update the example to use a generic repo path (or$PWD) so readers don’t copy/paste an incorrect directory.
mkdir -p /tmp/snctl-bin
cp /Users/rui/playground/sn/cloud-cli/bin/snctl-linux-arm64 /tmp/snctl-bin/snctl
cd /Users/rui/playground/sn/streamnative-cloud-skills
jiangpengcheng
approved these changes
May 15, 2026
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Adds StreamNative Cloud remote MCP as a first-class execution path for the agent skills while keeping
snctlas the portable fallback/baseline.sncloud_*_read/writeusage, catalog/schema discovery, JSON-string manifests, dry-run-first mutations,and explicit fallback limits.
data-plane tool gates, retry/error handling, and unsupported workflows.
required/forbidden tool calls.
scripts/smoke-runner.py, Claude Code adapter, Docker smokeimage,
snctltest config helper, and serverless cluster-name resolver.snctlsmoke specs for identity, Pulsar cluster/admin, and Kafka cluster/admin coverage.snctlbehavior, shorter cluster names, organization flags,Kafka admin commands, output formats, and smoke-test spec format.