Conversation
CodSpeed Performance ReportMerging #5395 will degrade performances by 3.15%Comparing Summary
Benchmarks breakdown
|
|
Great, I'll test it. Small fix for the README:
Cursor doesn't pick up the config there. It expects the config to be in .cursor/mcp.json instead https://docs.cursor.com/context/model-context-protocol#configuration-locations |
Let me share my honest thought on this: So this can all be very frustrating for newbies. If someone is evaluating new frameworks to use in future projects and the first experience is not smooth and promising, he goes on to the next framework. This is why, in my opinion, it would be a strategic mistake to make this a "Pro" feature if you want to grow. |
Fair enough we'll consider this - do you have any feedback on functionality or improvements you want |
There was a problem hiding this comment.
PR Summary
Added MCP_README documenting the Model Context Protocol (MCP) Server for Reflex, which provides programmatic access to framework documentation and component information.
- New MCP Server documentation details endpoints and tools for accessing Reflex framework information
- Includes client configuration examples for different platforms
- Outlines future pricing plans and service tiers
- Documentation structure needs consistency improvements in configuration examples
- Missing version requirements and security considerations need to be addressed
1 file reviewed, no comments
Edit PR Review Bot Settings | Greptile
All Submissions:
Type of change
Please delete options that are not relevant.
New Feature Submission:
Changes To Core Features:
After these steps, you're ready to open a pull request.