-
Notifications
You must be signed in to change notification settings - Fork 126
docs(skill): add CSP location and CORS pitfalls to common mistakes #407
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
base: main
Are you sure you want to change the base?
Conversation
Add three new items to the 'Common Mistakes to Avoid' section: - CSP _meta must be in contents array, not registerResource config - Localhost/private IPs blocked in CSP validation - CORS vs CSP confusion with ui.domain guidance
@modelcontextprotocol/ext-apps
@modelcontextprotocol/server-arcade
@modelcontextprotocol/server-basic-react
@modelcontextprotocol/server-basic-vanillajs
@modelcontextprotocol/server-budget-allocator
@modelcontextprotocol/server-cohort-heatmap
@modelcontextprotocol/server-customer-segmentation
@modelcontextprotocol/server-map
@modelcontextprotocol/server-pdf
@modelcontextprotocol/server-scenario-modeler
@modelcontextprotocol/server-shadertoy
@modelcontextprotocol/server-sheet-music
@modelcontextprotocol/server-system-monitor
@modelcontextprotocol/server-threejs
@modelcontextprotocol/server-transcript
@modelcontextprotocol/server-video-resource
@modelcontextprotocol/server-wiki-explorer
commit: |
|
I've added some more guidance for these in Line 299 in ec0f217
Lines 545 to 550 in ec0f217
Line 273 in ec0f217
Line 302 in ec0f217
I've also opened #416 to refactor the skill to lean more on
If you have some troublesome prompts, I can try them with the new version of the skill and iterate. |
Add three new items to the 'Common Mistakes to Avoid' section in the create-mcp-app skill:
_metain wrong location - Must be incontentsarray (readCallback), NOT in registerResource configui.domainfor CORS allowlists