chore: update MCP schema version, and MCP publisher #532
Conversation
There was a problem hiding this comment.
Pull Request Overview
This PR updates the MCP (Model Context Protocol) integration by migrating to a newer schema version and updating the publisher tool to use the latest release. The changes address deprecation of the old schema version (2025-07-09) and ensure the workflow uses the most current MCP publisher version.
Key Changes:
- Updated MCP schema version from 2025-07-09 to 2025-10-17
- Modified MCP publisher installation to use latest release instead of hardcoded v1.0.0
- Standardized quote styles from single to double quotes across workflow file
Reviewed Changes
Copilot reviewed 2 out of 2 changed files in this pull request and generated 1 comment.
| File | Description |
|---|---|
| server.json | Updates schema version to 2025-10-17 and renames website_url to websiteUrl per new schema requirements |
| .github/workflows/cli-release.yml | Updates MCP publisher to use latest release and standardizes quote style throughout workflow inputs |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
| curl -L "https://github.com/modelcontextprotocol/registry/releases/download/v1.0.0/mcp-publisher_1.0.0_$(uname -s | tr '[:upper:]' '[:lower:]')_$(uname -m | sed 's/x86_64/amd64/;s/aarch64/arm64/').tar.gz" | tar xz mcp-publisher | ||
| curl -L "https://github.com/modelcontextprotocol/registry/releases/latest/download/mcp-publisher_$(uname -s | tr '[:upper:]' '[:lower:]')_$(uname -m | sed 's/x86_64/amd64/;s/aarch64/arm64/').tar.gz" | tar xz mcp-publisher | ||
| chmod +x mcp-publisher | ||
There was a problem hiding this comment.
The download URL no longer includes the version number in the filename pattern. The previous pattern was mcp-publisher_1.0.0_... but the new pattern is mcp-publisher_... without any version number. This assumes the latest release maintains this naming convention. Consider adding error handling or verification that the downloaded file exists before attempting to use it.
| if [ ! -x mcp-publisher ]; then | |
| echo "Error: mcp-publisher was not downloaded or is not executable." | |
| exit 1 | |
| fi |
| - name: Install MCP Publisher | ||
| run: | | ||
| curl -L "https://github.com/modelcontextprotocol/registry/releases/download/v1.0.0/mcp-publisher_1.0.0_$(uname -s | tr '[:upper:]' '[:lower:]')_$(uname -m | sed 's/x86_64/amd64/;s/aarch64/arm64/').tar.gz" | tar xz mcp-publisher | ||
| curl -L "https://github.com/modelcontextprotocol/registry/releases/latest/download/mcp-publisher_$(uname -s | tr '[:upper:]' '[:lower:]')_$(uname -m | sed 's/x86_64/amd64/;s/aarch64/arm64/').tar.gz" | tar xz mcp-publisher |
There was a problem hiding this comment.
no its being updated / broken so fast right now I think we should just keep it on latest (that's what they updated their docs to recommend)
| @@ -1,13 +1,13 @@ | |||
| { | |||
| "$schema": "https://static.modelcontextprotocol.io/schemas/2025-07-09/server.schema.json", | |||
| "$schema": "https://static.modelcontextprotocol.io/schemas/2025-10-17/server.schema.json", | |||
There was a problem hiding this comment.
if we're pinning the schema - we should pin the release?
There was a problem hiding this comment.
its moving so fast, the "old" schema version from 2025-07-09 is depricated lol.
Update the MCP schema version as the old version is depricated, and update the MCP publisher version to latest as it's being updated frequently right now.