Skip to content

docs(csp-cors): add a pre-publish checklist for the declared CSP - #800

Open
shteynu wants to merge 1 commit into
modelcontextprotocol:mainfrom
shteynu:docs/check-csp-before-publishing
Open

shteynu wants to merge 1 commit into
modelcontextprotocol:mainfrom
shteynu:docs/check-csp-before-publishing

Conversation

@shteynu

@shteynu shteynu commented Oct 8, 2026 •

Copy link
Copy Markdown

Adds a short pre-publish checklist to docs/csp-cors.md for the CSP a server declares in _meta.ui.csp: undeclared origins fail silently in the sandbox, */scheme-only entries open the App to any host, broad subdomain wildcards on shared services, and localhost leaking into published metadata. Also adds one optional verification step to the add-app-to-server skill.

Docs-only, two Markdown files; prettier --check passes locally.

Disclosure: I maintain mcp-app-lint, which the last paragraph mentions. If you'd rather keep the docs tool-neutral, I'll drop that paragraph and the skill step and keep only the checklist.

@shteynu

shteynu commented Oct 8, 2026

Copy link
Copy Markdown
Author

Hi, a note for review: this is a docs-only change (two Markdown files). The CI, Publish Any Commit and Docs Preview workflows are waiting for approval because the branch comes from a fork. Locally, prettier --check docs/csp-cors.md passes. If you would rather keep the docs tool-neutral, I'm happy to drop the mcp-app-lint paragraph and the skill step and keep only the checklist; just let me know. Thanks!

This branch has not been deployed

No deployments
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant