From 7299960430091d240ba59272792963280fb97928 Mon Sep 17 00:00:00 2001 From: adam jones Date: Thu, 11 Sep 2025 09:34:21 -0700 Subject: [PATCH] docs: remove redundant validation steps from GitHub Actions guide (#440) Remove 'Test Before Publishing' and 'Notification Setup' sections from github-actions.md as these steps are redundant since the publisher API already handles validation. Fixes #432 Generated with [Claude Code](https://claude.ai/code) Co-authored-by: claude[bot] <209825114+claude[bot]@users.noreply.github.com> Co-authored-by: adam jones --- docs/guides/publishing/github-actions.md | 35 ------------------------ 1 file changed, 35 deletions(-) diff --git a/docs/guides/publishing/github-actions.md b/docs/guides/publishing/github-actions.md index 840aed9..c355e5f 100644 --- a/docs/guides/publishing/github-actions.md +++ b/docs/guides/publishing/github-actions.md @@ -272,41 +272,6 @@ Only publish to registry after package publishing succeeds: run: ./mcp-publisher publish ``` -### 3. Test Before Publishing - -Include validation in your workflow: - -```yaml -- name: Validate server.json - run: | - # Validate against schema - curl -s https://static.modelcontextprotocol.io/schemas/2025-07-09/server.schema.json | \ - jq . > schema.json - npx ajv-cli validate -s schema.json -d server.json - - # Test with publisher - ./mcp-publisher init --validate-only -``` - -### 4. Notification Setup - -Get notified of publishing status: - -```yaml -- name: Notify on success - if: success() - run: | - curl -X POST "${{ secrets.WEBHOOK_URL }}" \ - -H "Content-Type: application/json" \ - -d '{"text": "✅ MCP server ${{ github.repository }} published successfully"}' - -- name: Notify on failure - if: failure() - run: | - curl -X POST "${{ secrets.WEBHOOK_URL }}" \ - -H "Content-Type: application/json" \ - -d '{"text": "❌ MCP server ${{ github.repository }} publishing failed"}' -``` ## Troubleshooting