chore: remove MCP conformance CI - #84
Conversation
The conformance test suite is designed for MCP SDK maintainers to verify protocol implementations, not for production MCP servers. Removing it from CI and the template simplifies the project and avoids misleading people who fork this into thinking conformance tooling belongs in their own servers. Removes: - .github/workflows/conformance.yml - conformance-baseline.yml - Conformance section from README.md - Step 5 (conformance baseline) from the create-mcp-tool skill Generated with [Devin](https://devin.ai) Co-Authored-By: Devin <158243242+devin-ai-integration[bot]@users.noreply.github.com>
📝 WalkthroughWalkthroughRemoves MCP conformance testing infrastructure: deletes the ChangesConformance Testing Removal
Estimated code review effort: 1 (Trivial) | ~5 minutes Related PRs: None identified. Suggested labels: documentation, ci Suggested reviewers: nickytonline 🐰 *Hops away from tests that failed,* *Conformance checks now unveiled—* *gone from workflows, gone from docs,* *no more baseline logs to log,* *a simpler path the rabbit trailed.*🚥 Pre-merge checks | ✅ 5✅ Passed checks (5 passed)
✨ Finishing Touches📝 Generate docstrings
🧪 Generate unit tests (beta)
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
Greptile SummaryThis PR removes all MCP conformance testing infrastructure from the template, cleaning up files that were intended for MCP SDK maintainers rather than end-users who fork the template. A full-repo search confirms no stale
Confidence Score: 5/5Safe to merge — purely removes files with no functional production code changes. All four changes are straightforward deletions or doc edits. No functional source code is touched, no cross-references to the removed files remain in the repo, and the SKILL.md step renumbering is correct. Nothing new is introduced that could regress. No files require special attention. Important Files Changed
Flowchart%%{init: {'theme': 'neutral'}}%%
flowchart TD
subgraph Before["CI Pipeline - Before"]
A[Push or PR] --> B[lint-and-test]
A --> C[conformance]
C --> D[docker compose up]
D --> E[MCP conformance action]
E --> F[conformance-baseline.yml]
end
subgraph After["CI Pipeline - After"]
G[Push or PR] --> H[lint-and-test]
end
%%{init: {'theme': 'base', 'themeVariables': {"darkMode": true, "background": "#0d1117", "primaryColor": "#21262d", "primaryTextColor": "#e6edf3", "primaryBorderColor": "#8b949e", "lineColor": "#8b949e", "textColor": "#e6edf3", "edgeLabelBackground": "#161b22", "actorBkg": "#21262d", "actorBorder": "#8b949e", "actorTextColor": "#e6edf3", "actorLineColor": "#8b949e", "signalColor": "#8b949e", "signalTextColor": "#e6edf3", "noteBkgColor": "#373320", "noteBorderColor": "#d4a72c", "noteTextColor": "#f0e6c0", "labelBoxBkgColor": "#21262d", "labelBoxBorderColor": "#8b949e", "labelTextColor": "#e6edf3", "loopTextColor": "#e6edf3", "activationBkgColor": "#30363d", "activationBorderColor": "#8b949e"}}}%%
flowchart TD
subgraph Before["CI Pipeline - Before"]
A[Push or PR] --> B[lint-and-test]
A --> C[conformance]
C --> D[docker compose up]
D --> E[MCP conformance action]
E --> F[conformance-baseline.yml]
end
subgraph After["CI Pipeline - After"]
G[Push or PR] --> H[lint-and-test]
end
Reviews (1): Last reviewed commit: "chore: remove MCP conformance CI" | Re-trigger Greptile |
|
🎉 This PR is included in version 1.4.0 🎉 The release is available on GitHub release Your semantic-release bot 📦🚀 |
Summary
The conformance test suite is designed for MCP SDK maintainers to verify protocol implementations, not for production MCP servers. Removing it from CI and the template simplifies the project and avoids misleading people who fork this into thinking conformance tooling belongs in their own servers.
I initially thought it made sense as it was a template where I wanted to validate some things, but I didn't completely realize how the conformance tests behave so it's really not a fit for this template.
Removes:
Generated with Devin
Related Issues
Relates to #76
AI Disclosure
Co-Authored-By: Devin <158243242+devin-ai-integration[bot]@users.noreply.github.com>
I reviewed the changes.