Skip to content

feat: add .mcp.json support to scaffold for mock MCP servers#49

Merged
GuyZivRH merged 1 commit into
mainfrom
feat/mcp-json-scaffold-support
Jul 8, 2026
Merged

feat: add .mcp.json support to scaffold for mock MCP servers#49
GuyZivRH merged 1 commit into
mainfrom
feat/mcp-json-scaffold-support

Conversation

@GuyZivRH

@GuyZivRH GuyZivRH commented Jul 8, 2026

Copy link
Copy Markdown
Collaborator

Summary

  • Enable skill submissions to include MCP server configuration that Harbor evaluations can use automatically
  • Scaffold detects .mcp.json at submission root or supportive/ (legacy location)
  • Copies .mcp.json to /workspace/.mcp.json for treatment variant only
  • Excludes .mcp.json from control variant (preserves A/B test validity)

Use Case

Skills requiring MCP tools (e.g., OpenShift Assisted Installer) can now include mock MCP servers that the agent discovers and uses during evaluation without external infrastructure dependencies.

Example Structure

submissions/my-skill/
├── .mcp.json                              ← Agent reads at /workspace/.mcp.json
├── supportive/
│   └── mcp-servers/
│       └── mock-server.py                 ← MCP server code
├── metadata.yaml
└── ...

Test Plan

  • 5 new tests added covering root and legacy location detection
  • All 49 scaffold tests pass

Made with Cursor

Enable skill submissions to include MCP server configuration that Harbor
evaluations can use automatically. The scaffold now:

- Detects .mcp.json at submission root or supportive/ (legacy location)
- Copies .mcp.json to /workspace/.mcp.json for treatment variant only
- Excludes .mcp.json from control variant (preserves A/B test validity)

This allows skills requiring MCP tools (e.g., OpenShift Assisted Installer)
to include mock MCP servers that the agent can discover and use during
evaluation without external infrastructure dependencies.

Co-authored-by: Cursor <cursoragent@cursor.com>
@GuyZivRH GuyZivRH merged commit 01cf4c7 into main Jul 8, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant