-
Notifications
You must be signed in to change notification settings - Fork 4
chore: update MCP schema version, and MCP publisher #532
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Changes from all commits
File filter
Filter by extension
Conversations
Jump to
Diff view
Diff view
There are no files selected for viewing
| Original file line number | Diff line number | Diff line change | ||||||||||
|---|---|---|---|---|---|---|---|---|---|---|---|---|
|
|
@@ -4,24 +4,24 @@ on: | |||||||||||
| workflow_dispatch: | ||||||||||||
| inputs: | ||||||||||||
| draft-release: | ||||||||||||
| description: 'Draft release?' | ||||||||||||
| description: "Draft release?" | ||||||||||||
| required: false | ||||||||||||
| type: boolean | ||||||||||||
| default: true | ||||||||||||
| prerelease: | ||||||||||||
| description: 'Prerelease?' | ||||||||||||
| description: "Prerelease?" | ||||||||||||
| required: false | ||||||||||||
| type: boolean | ||||||||||||
| default: true | ||||||||||||
| npm-version: | ||||||||||||
| description: 'NPM version bump?' | ||||||||||||
| description: "NPM version bump?" | ||||||||||||
| required: false | ||||||||||||
| type: choice | ||||||||||||
| options: | ||||||||||||
| - major | ||||||||||||
| - minor | ||||||||||||
| - patch | ||||||||||||
| default: 'patch' | ||||||||||||
| default: "patch" | ||||||||||||
|
|
||||||||||||
| # schedule: | ||||||||||||
| # - cron: '0 12 * * 2' # Runs every Tuesday at 12:00 PM (noon) ET | ||||||||||||
|
|
@@ -43,7 +43,7 @@ jobs: | |||||||||||
| uses: DevCycleHQ/aws-secrets-action@main | ||||||||||||
| with: | ||||||||||||
| secrets_map: '{"MCP_KEY": "DEVCYCLE_GITHUB_cli_MCP_REGISTRY_PKEY"}' | ||||||||||||
| aws_account_id: '134377926370' | ||||||||||||
| aws_account_id: "134377926370" | ||||||||||||
| - name: Set Git author | ||||||||||||
| run: | | ||||||||||||
| git config --global user.email "foundation-admin@devcycle.com" | ||||||||||||
|
|
@@ -60,8 +60,8 @@ jobs: | |||||||||||
| uses: actions/setup-node@v4 | ||||||||||||
| with: | ||||||||||||
| node-version: "${{ steps.get_node_version.outputs.NVMRC }}" | ||||||||||||
| registry-url: 'https://registry.npmjs.org' | ||||||||||||
| cache: 'yarn' | ||||||||||||
| registry-url: "https://registry.npmjs.org" | ||||||||||||
| cache: "yarn" | ||||||||||||
|
|
||||||||||||
| - name: Install dependencies | ||||||||||||
| run: yarn install | ||||||||||||
|
|
@@ -99,9 +99,10 @@ jobs: | |||||||||||
| prerelease: ${{ inputs.prerelease }} | ||||||||||||
|
|
||||||||||||
| # --- Official MCP Registry (registry.modelcontextprotocol.io) Publish --- | ||||||||||||
| - name: Install MCP Publisher (registry.modelcontextprotocol.io) | ||||||||||||
| # See docs here: https://raw.githubusercontent.com/modelcontextprotocol/registry/refs/heads/main/docs/guides/publishing/github-actions.md | ||||||||||||
| - 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 | ||||||||||||
| chmod +x mcp-publisher | ||||||||||||
|
|
||||||||||||
|
||||||||||||
| if [ ! -x mcp-publisher ]; then | |
| echo "Error: mcp-publisher was not downloaded or is not executable." | |
| exit 1 | |
| fi |
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -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", | ||
|
Member
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. if we're pinning the schema - we should pin the release?
Member
Author
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. its moving so fast, the "old" schema version from |
||
| "name": "com.devcycle/mcp", | ||
| "description": "DevCycle MCP server for feature flag management", | ||
| "version": "6.1.2", | ||
| "repository": { | ||
| "url": "https://github.com/DevCycleHQ/cli", | ||
| "source": "github" | ||
| }, | ||
| "website_url": "https://docs.devcycle.com/cli-mcp/mcp-getting-started", | ||
| "websiteUrl": "https://docs.devcycle.com/cli-mcp/mcp-getting-started", | ||
| "remotes": [ | ||
| { | ||
| "type": "streamable-http", | ||
|
|
||
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
not version pinning?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
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)