feat(mcp): embed main requests configuration reference and resource - #56
Conversation
- Add comprehensive assets/examples/https-wrench-mcp-main-config.yaml demonstrating all schema options against os76.xyz endpoints - Embed main configuration into MCP assets via go:generate - Expose https-wrench://main-config static resource and https-wrench://examples/mcp-main-config example template - Add reference hints in author_requests_config prompt - Update unit/coverage tests and CHANGELOG
|
No actionable comments were generated in the recent review. 🎉 ℹ️ Recent review info⚙️ Run configurationConfiguration used: defaults Review profile: CHILL Plan: Advanced Run ID: 📒 Files selected for processing (8)
Included review availability: Your plan provides up to 1 included review per hour; 0 remain after this review. 📝 WalkthroughWalkthroughThe PR adds a complete MCP configuration example, embeds it as two MCP resources, includes the resources in prompt references and hints, adds resource-read coverage, and updates the changelog. ChangesMCP main configuration
Priority: ⬇️ Low Estimated code review effort: 2 (Simple) | ~10 minutes Change: Feature Sequence Diagram(s)sequenceDiagram
participant MCPRequest
participant authorRequestsConfigPrompt
participant registerResources
participant EmbeddedMainConfig
MCPRequest->>authorRequestsConfigPrompt: request configuration guidance
authorRequestsConfigPrompt->>registerResources: include main-config resource references
registerResources->>EmbeddedMainConfig: serve main YAML configuration
MCPRequest->>registerResources: read main-config or example URI
registerResources-->>MCPRequest: return configuration content
Merge Risk: ⚪ Minimal · up to The new MCP configuration resources and prompt references have no supported current-head issue requiring changes before merge. 🚥 Pre-merge checks | ✅ 4 | ❌ 1❌ Failed checks (1 warning)
✅ Passed checks (4 passed)
Full details: Docstring CoverageExplanation Docstring coverage is 0.00% which is insufficient. The required threshold is 80.00%. Docstring coverage is scoped to functions touched by this diff. Analyzed 9 functions across 5 files. (3 skipped: 3 unsupported.)
✨ Finishing Touches 💡 1📝 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 |
Summary by CodeRabbit
New Features
Documentation