feat: package skills as the pcm plugin - #7
Merged
Conversation
- Introduced the CDB Editor plugin for Pro Cycling Manager, allowing users to explore and edit game databases (.cdb) and build race startlists. - Added plugin metadata in `plugin.json` with details about the plugin, author, and capabilities. - Created skills for database editing (`pcm-db-editor`) with comprehensive documentation on usage, constraints, and workflows. - Implemented a script (`open-cdb.sh`) to facilitate opening and converting .cdb files to SQLite for easier exploration. - Developed a startlist generation skill (`pcm-startlist`) to create and export race startlists in XML format, detailing the workflow for user interaction. - Included references for database constraints and schema to guide users in editing and querying the database effectively.
Plugin metadata is duplicated across four manifests, one per agent format, with nothing validating them against each other. List them in AGENTS.md so a name/description/version change updates all of them. Co-Authored-By: Claude Opus 5 <noreply@anthropic.com>
This reverts commit ee10671.
There was a problem hiding this comment.
Pull request overview
Packages the existing Pro Cycling Manager database/startlist skills into a distributable cdb-editor plugin, including plugin manifests, marketplace manifests, and bundled MCP server configuration.
Changes:
- Adds
cdb-editorplugin manifests for Claude and Codex, plus marketplace manifests for Claude and Agents. - Moves/introduces the two skills (
pcm-db-editor,pcm-startlist) underplugins/cdb-editor/skills/, including supporting scripts and references. - Adds branding assets (
logo.svg,app-icon.png) and an MCP server launcher config (.mcp.json).
Reviewed changes
Copilot reviewed 5 out of 13 changed files in this pull request and generated 1 comment.
Show a summary per file
| File | Description |
|---|---|
| plugins/cdb-editor/skills/pcm-startlist/SKILL.md | Startlist-building skill definition moved under the plugin. |
| plugins/cdb-editor/skills/pcm-db-editor/SKILL.md | Database editing skill definition moved under the plugin. |
| plugins/cdb-editor/skills/pcm-db-editor/scripts/open-cdb.sh | Adds helper script for backing up + converting .cdb to SQLite and printing table inventory. |
| plugins/cdb-editor/skills/pcm-db-editor/references/database-schema.md | Adds schema reference documentation used by the db editor skill. |
| plugins/cdb-editor/skills/pcm-db-editor/references/database-constraints.md | Adds constraints/reference documentation for safe edits. |
| plugins/cdb-editor/assets/logo.svg | Adds plugin logo SVG. |
| plugins/cdb-editor/assets/app-icon.png | Adds plugin app icon PNG. |
| plugins/cdb-editor/.mcp.json | Adds MCP server config for running pcm-mcp via npx. |
| plugins/cdb-editor/.codex-plugin/plugin.json | Adds/updates Codex plugin manifest for cdb-editor. |
| plugins/cdb-editor/.claude-plugin/plugin.json | Adds Claude plugin manifest for cdb-editor. |
| .claude/skills/create-agentsmd/SKILL.md | Adds a Claude skill vendored from github/awesome-copilot (not plugin-related). |
| .claude-plugin/marketplace.json | Adds Claude marketplace manifest entry for cdb-editor. |
| .agents/plugins/marketplace.json | Adds Agents marketplace manifest entry for cdb-editor. |
💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.
There was a problem hiding this comment.
Pull request overview
Copilot reviewed 4 out of 12 changed files in this pull request and generated no new comments.
Suppressed comments (1)
plugins/cdb-editor/.codex-plugin/plugin.json:41
interface.defaultPromptis an array with 4 entries. Codex/ChatGPT plugin manifest validation commonly limits this list to a maximum of 3 prompts; having 4 can cause schema validation errors or the prompt list to be rejected.
There was a problem hiding this comment.
Pull request overview
Copilot reviewed 6 out of 13 changed files in this pull request and generated no new comments.
Suppressed comments (2)
plugins/pcm/.codex-plugin/plugin.json:2
- PR metadata describes repackaging as the "cdb-editor" plugin, but this manifest (and the plugin directory path) still uses the plugin name "pcm". If the intended plugin ID is actually "cdb-editor", it should be renamed consistently across plugin.json files and marketplace manifests; otherwise the PR title/description should be updated to match the "pcm" plugin name to avoid release/installation confusion.
plugins/pcm/.claude-plugin/plugin.json:2 - This manifest also uses the plugin name "pcm", which appears to conflict with the PR metadata calling this the "cdb-editor" plugin. Please ensure the plugin ID is consistent with the intended published name across all manifests (or update the PR title/description accordingly).
"name": "pcm",
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.
Summary
Repackages the two existing skills (
pcm-database,pcm-startlist) as a distributablepcmplugin..claude-plugin/) and Agents (.agents/plugins/) formats, under thepcmstackmarketplaceplugins/pcm/), with display names "Pro Cycling Manager" (Claude / Agents marketplace) and "CDB Editor" (Codex interface), plus branding assets (logo.svg,app-icon.png)pcm-mcpMCP server via.mcp.jsonplugins/pcm/skills/—pcm-db-editorrenamed topcm-database, content otherwise unchangeddescriptionkept byte-identical across the three manifests that carry itNotes for review
.mcp.jsonrunsnpx -y pcm-mcpunpinned, so every launch pulls the latest published version — worth pinning a major before release..agents/plugins/marketplace.jsonis the only manifest whose plugin entry has nodescription; left as-is pending confirmation of what that schema accepts.skills/at the root). Both need a follow-up.