feat: add harness support to gcp-mcp-setup skill - #46
Conversation
- Add .copilot-plugin/plugin.json and marketplace.json - Add install-copilot / install-copilot-persistent justfile recipes - Fix justfile Windows compatibility (set windows-shell) - Document Copilot CLI install in INSTALL.md and README.md
- Add --harness flag to setup_onemcp.py and verify_setup.py - Harness registry with embedded format builders (setup) - Harness MCP list command registry (verify) - Centralise subprocess calls via run_command() for Windows .cmd compat - Bump MCP Setup Copilot CLI to tested (✅) in README
|
Linked Issue: #47 |
| os.path.expanduser("~/.gemini/antigravity/mcp_config.json"), | ||
| os.path.join(os.getcwd(), ".gemini", "config", "mcp_config.json"), | ||
| os.path.expanduser("~/.gemini/config/mcp_config.json") | ||
| os.path.expanduser("~/.gemini/config/mcp_config.json"), |
There was a problem hiding this comment.
@antoinemerc Thank you for introducing harness registry. We needed some central place for this pieces of information. Can we re-use it here?
There was a problem hiding this comment.
Yes, re-using and centralizing the path is better. I'll go a step further and centralize harness specific configuration into a single module
This way, harness support involve defining:
- a set of local/gloabl path
- an mcp format builder function
- any harness specific command that is needed
It lives in gcp-mcp-setup but can easily be moved elsewhere if harness specific support is needed in other skills
…y.py Introduce harness_registry.py as the single source of truth for harness-specific configuration as suggested in PR comment - Add HarnessName(str, Enum), HarnessCommand(str, Enum), and HarnessConfig frozen dataclass with .paths(scope) - HARNESS_REGISTRY controls config file paths, MCP entry builders, and CLI commands in both scripts; adding a new harness is a single registry entry - Fix Windows encoding errors when output is piped by adding UTF-8 reconfigure on stdout/stderr and explicit encoding='utf-8' in subprocess calls
…b.com/antoinemerc/sre into feature/260702-mercant-onemcp-copilot
|
This has become a bit bigger of a refactor than I originally intended! |
|
LGTM! Thanks Antoine! |
3 participants
This requires #44 to be merged first since it expand on copilot support
Add Copilot CLI harness support to gcp-mcp-setup skill: