Skip to content

Commit dfec646

Browse files
committed
Fix CI
1 parent 9f2930e commit dfec646

4 files changed

Lines changed: 2 additions & 24 deletions

File tree

.cursor-plugin/plugin.json

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1,7 +1,6 @@
11
{
22
"name": "amd-skills",
33
"skills": "skills",
4-
"mcpServers": ".mcp.json",
54
"description": "Agent Skills for AMD-optimized workflows.",
65
"version": "0.1.0",
76
"author": {

.github/workflows/validate.yml

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -10,7 +10,6 @@ on:
1010
- "skills/**"
1111
- ".claude-plugin/**"
1212
- ".cursor-plugin/**"
13-
- ".mcp.json"
1413
- ".github/workflows/validate.yml"
1514
workflow_dispatch:
1615

README.md

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -155,7 +155,7 @@ AMD Skills are compatible with Cursor, Claude Code, OpenAI Codex, and Gemini CLI
155155

156156
### Cursor
157157

158-
Install the AMD plugin from this repository through the Cursor plugin flow. The repo ships a `.cursor-plugin/plugin.json` and an `.mcp.json` so skills are discoverable as soon as the plugin is enabled.
158+
Install the AMD plugin from this repository through the Cursor plugin flow. The repo ships a `.cursor-plugin/plugin.json` so skills are discoverable as soon as the plugin is enabled.
159159

160160
### Claude Code
161161

scripts/generate_cursor_plugin.py

Lines changed: 1 addition & 21 deletions
Original file line numberDiff line numberDiff line change
@@ -13,7 +13,6 @@
1313
- Reuse `.claude-plugin/plugin.json` as the primary metadata source.
1414
- Discover skills from `skills/*/SKILL.md` so the manifest tracks the
1515
catalog automatically.
16-
- Treat `.mcp.json` as hand-maintained (no MCP server is generated here).
1716
1817
Usage:
1918
uv run scripts/generate_cursor_plugin.py # write
@@ -32,7 +31,6 @@
3231
CLAUDE_PLUGIN_MANIFEST = ROOT / ".claude-plugin" / "plugin.json"
3332
CURSOR_PLUGIN_DIR = ROOT / ".cursor-plugin"
3433
CURSOR_PLUGIN_MANIFEST = CURSOR_PLUGIN_DIR / "plugin.json"
35-
MCP_CONFIG = ROOT / ".mcp.json"
3634

3735
# Fields copied verbatim from the Claude plugin manifest into the Cursor
3836
# manifest so the two stay in lock-step.
@@ -107,9 +105,7 @@ def build_cursor_plugin_manifest() -> dict:
107105
if not skills:
108106
raise ValueError("No skills discovered under skills/*/SKILL.md")
109107

110-
# `mcpServers` points at .mcp.json so future MCP servers added there
111-
# are picked up by Cursor without a manifest change.
112-
manifest: dict = {"name": name, "skills": "skills", "mcpServers": ".mcp.json"}
108+
manifest: dict = {"name": name, "skills": "skills"}
113109
for key in COPIED_FIELDS:
114110
if key in src:
115111
manifest[key] = src[key]
@@ -139,21 +135,6 @@ def write_or_check(path: Path, content: str, check: bool) -> bool:
139135
return True
140136

141137

142-
def validate_mcp_config() -> None:
143-
"""Make sure .mcp.json is at least valid JSON shaped like an MCP config."""
144-
if not MCP_CONFIG.exists():
145-
raise FileNotFoundError(
146-
f"Missing required file: {MCP_CONFIG.relative_to(ROOT)}. "
147-
'Create it with `{"mcpServers": {}}` if there are no servers yet.'
148-
)
149-
data = load_json(MCP_CONFIG)
150-
if not isinstance(data, dict) or "mcpServers" not in data:
151-
raise ValueError(
152-
f"{MCP_CONFIG.relative_to(ROOT)} must be a JSON object with an "
153-
"`mcpServers` key (use `{}` if no servers are configured)."
154-
)
155-
156-
157138
def main() -> None:
158139
parser = argparse.ArgumentParser(
159140
description="Generate Cursor plugin manifest from .claude-plugin/plugin.json"
@@ -165,7 +146,6 @@ def main() -> None:
165146
)
166147
args = parser.parse_args()
167148

168-
validate_mcp_config()
169149
plugin_manifest = render_json(build_cursor_plugin_manifest())
170150
ok_plugin = write_or_check(CURSOR_PLUGIN_MANIFEST, plugin_manifest, check=args.check)
171151

0 commit comments

Comments
 (0)