Conversation
✅ Deploy Preview for storybook-mcp-self-host-example ready!
To edit notification comments on pull requests, go to your Netlify project configuration. |
🦋 Changeset detectedLatest commit: 5af6b6d The changes in this PR will be included in the next version bump. This PR includes changesets to release 2 packages
Not sure what this means? Click here to learn what changesets are. Click here if you're a maintainer who wants to add another changeset to this PR |
commit: |
Codecov Report✅ All modified and coverable lines are covered by tests. Additional details and impacted files@@ Coverage Diff @@
## main #172 +/- ##
=======================================
Coverage 73.74% 73.74%
=======================================
Files 42 42
Lines 1177 1177
Branches 330 330
=======================================
Hits 868 868
Misses 193 193
Partials 116 116 ☔ View full report in Codecov by Sentry. |
Bundle ReportChanges will increase total bundle size by 65.45kB (147.95%) ⬆️
ℹ️ *Bundle size includes cached data from a previous commit |
There was a problem hiding this comment.
Pull request overview
This PR improves documentation across the MCP packages by introducing a docs-site-first approach. The changes simplify package READMEs and add comprehensive self-hosting documentation and examples.
Changes:
- Added a new self-hosting example app (
apps/self-host-mcp) demonstrating Node.js and Netlify Functions deployment of@storybook/mcp - Created draft self-hosting documentation (
packages/mcp/self-hosting.md) with API reference and implementation examples - Simplified package READMEs to point to future docs site (marked with "TK" placeholders)
- Moved existing addon README content to
packages/addon-mcp/docs.mdfor reference during docs site migration
Reviewed changes
Copilot reviewed 13 out of 14 changed files in this pull request and generated 4 comments.
Show a summary per file
| File | Description |
|---|---|
| packages/mcp/self-hosting.md | New comprehensive API documentation and self-hosting guide |
| packages/mcp/README.md | Simplified to single sentence with docs link placeholder |
| packages/addon-mcp/docs.md | Original README content moved here for migration reference |
| packages/addon-mcp/README.md | Simplified to brief description with docs link placeholder |
| apps/self-host-mcp/server.ts | Example Node.js server implementation with manifest provider |
| apps/self-host-mcp/netlify/functions/mcp.ts | Netlify Functions adapter for the MCP handler |
| apps/self-host-mcp/package.json | Standalone example package configuration |
| apps/self-host-mcp/README.md | Example app documentation and usage instructions |
| apps/self-host-mcp/manifests/*.json | Sample manifest files for testing |
| apps/self-host-mcp/netlify.toml | Netlify deployment configuration |
| apps/self-host-mcp/tsconfig.json | TypeScript configuration for example |
| apps/self-host-mcp/pnpm-lock.yaml | Lockfile for standalone dependencies |
| .changeset/bright-masks-hunt.md | Changeset describing README simplification |
Files not reviewed (1)
- apps/self-host-mcp/pnpm-lock.yaml: Language not supported
* main: (34 commits) Version Packages add back flag while core hasn't been released yet test: make /mcp GET handler selection explicit Apply suggestion from @JReinhold add changeset cleanup tests docs: clarify both flag names for older/newer Storybook in README and 404 hint docs: reflect that componentsManifest defaults to true and can be omitted feat: rename experimentalComponentsManifest → componentsManifest with backwards compat Initial plan test: format preset test assertions fix addon mcp page messaging and redirect behavior Initial plan Version Packages add more tests update task readmes update storybook improve preview stories tool description to make agents output the URLs to the user every time. Update eval/tasks/915-preview-story-by-id/README.md upgrade storybook ...
There was a problem hiding this comment.
Pull request overview
Copilot reviewed 11 out of 12 changed files in this pull request and generated 3 comments.
You can also share your feedback on Copilot code review. Take the survey.
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
There was a problem hiding this comment.
Pull request overview
Copilot reviewed 11 out of 12 changed files in this pull request and generated 2 comments.
You can also share your feedback on Copilot code review. Take the survey.
There was a problem hiding this comment.
Pull request overview
Copilot reviewed 11 out of 12 changed files in this pull request and generated no new comments.
Files not reviewed (1)
- apps/self-host-mcp/pnpm-lock.yaml: Language not supported
You can also share your feedback on Copilot code review. Take the survey.
There was a problem hiding this comment.
Pull request overview
Copilot reviewed 11 out of 12 changed files in this pull request and generated 2 comments.
Files not reviewed (1)
- apps/self-host-mcp/pnpm-lock.yaml: Language not supported
You can also share your feedback on Copilot code review. Take the survey.
There was a problem hiding this comment.
Pull request overview
Copilot reviewed 11 out of 12 changed files in this pull request and generated no new comments.
Files not reviewed (1)
- apps/self-host-mcp/pnpm-lock.yaml: Language not supported
You can also share your feedback on Copilot code review. Take the survey.
This PR improves the documentation for all the MCP packages.
@storybook/mcp, demonstrating how to run your own Storybook MCP server in a Node.js process or Netlify Functions.addon-mcphas been temporarily moved todocs.md, so it can be used as a reference for writing the docs on the docs-site~Both
docs.mdandself-hosting.mdshould be deleted when they have been migrated to the docs-site.~✅