Skip to content

Add MCP registry metadata for @storybook/addon-mcp#202

Merged
JReinhold merged 4 commits intomainfrom
shilman/add-registry-server-json
Mar 19, 2026
Merged

Add MCP registry metadata for @storybook/addon-mcp#202
JReinhold merged 4 commits intomainfrom
shilman/add-registry-server-json

Conversation

@shilman
Copy link
Copy Markdown
Member

@shilman shilman commented Mar 18, 2026

See #200

Summary

  • add MCP registry verification metadata to packages/addon-mcp/package.json
  • add packages/addon-mcp/server.json for the registry entry and package metadata
  • co-locate the registry manifest with the publishable addon package

Testing

  • Not run (not requested)

@changeset-bot
Copy link
Copy Markdown

changeset-bot bot commented Mar 18, 2026

🦋 Changeset detected

Latest commit: 1a41f74

The changes in this PR will be included in the next version bump.

This PR includes changesets to release 1 package
Name Type
@storybook/addon-mcp Patch

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

@netlify
Copy link
Copy Markdown

netlify bot commented Mar 18, 2026

Deploy Preview for storybook-mcp-self-host-example canceled.

Name Link
🔨 Latest commit 1a41f74
🔍 Latest deploy log https://app.netlify.com/projects/storybook-mcp-self-host-example/deploys/69bc6520b90af200080f81a3

@pkg-pr-new
Copy link
Copy Markdown

pkg-pr-new bot commented Mar 18, 2026

npm i https://pkg.pr.new/storybookjs/mcp/@storybook/addon-mcp@202
npm i https://pkg.pr.new/storybookjs/mcp/@storybook/mcp@202

commit: 1a41f74

@codecov
Copy link
Copy Markdown

codecov bot commented Mar 18, 2026

❌ 1 Tests Failed:

Tests completed Failed Passed Skipped
448 1 447 0
View the full list of 1 ❄️ flaky test(s)
apps/internal-storybook/tests/check-deps.e2e.test.ts > Storybook Dependencies > should be using latest versions from registry

Flake rate in main: 29.41% (Passed 60 times, Failed 25 times)

Stack Traces | 5s run time
Error: Storybook dependencies are outdated. Update the catalog in pnpm-workspace.yaml:

  sed -i '' 's/10\.3\.0-alpha\.16/10.4.0-alpha.2/g' pnpm-workspace.yaml && pnpm install

Outdated packages:
  - @storybook/addon-docs: 10.3.0-alpha.16 → 10.4.0-alpha.2
  - @storybook/react-vite: 10.3.0-alpha.16 → 10.4.0-alpha.2
  - storybook: 10.3.0-alpha.16 → 10.4.0-alpha.2
 ❯ tests/check-deps.e2e.test.ts:42:10

To view more test analytics, go to the Test Analytics Dashboard
📋 Got 3 mins? Take this short survey to help us improve Test Analytics.

@shilman shilman force-pushed the shilman/add-registry-server-json branch from a8bf48c to 5adc647 Compare March 18, 2026 05:45
@shilman shilman changed the title Add evaluation logs for MCP and non-MCP trial runs Add MCP registry metadata for @storybook/addon-mcp Mar 18, 2026
@shilman shilman marked this pull request as ready for review March 18, 2026 05:49
Copilot AI review requested due to automatic review settings March 18, 2026 05:49
Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Copilot encountered an error and was unable to review this pull request. You can try again by re-requesting a review.

Copilot AI review requested due to automatic review settings March 19, 2026 21:04
@JReinhold
Copy link
Copy Markdown
Contributor

Added:

  • script to publish new versions to the registry when we publish new versions of addon-mcp
  • (including updating the version strings)
  • added more metadata to the server.json file, like icons and package arguments

Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

Copilot reviewed 4 out of 5 changed files in this pull request and generated 2 comments.

if: steps.changesets.outputs.published == 'true' && contains(steps.changesets.outputs.publishedPackages, '@storybook/addon-mcp')
working-directory: packages/addon-mcp
run: |
curl -L "https://github.com/modelcontextprotocol/registry/releases/latest/download/mcp-publisher_$(uname -s | tr '[:upper:]' '[:lower:]')_$(uname -m | sed 's/x86_64/amd64/;s/aarch64/arm64/').tar.gz" | tar xz mcp-publisher
Copy link

Copilot AI Mar 19, 2026

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

tar extraction is missing the archive file flag; tar xz mcp-publisher won’t read from stdin on Ubuntu’s GNU tar and will fail the install step. Use tar -xzf - mcp-publisher (or equivalent) when piping from curl.

Suggested change
curl -L "https://github.com/modelcontextprotocol/registry/releases/latest/download/mcp-publisher_$(uname -s | tr '[:upper:]' '[:lower:]')_$(uname -m | sed 's/x86_64/amd64/;s/aarch64/arm64/').tar.gz" | tar xz mcp-publisher
curl -L "https://github.com/modelcontextprotocol/registry/releases/latest/download/mcp-publisher_$(uname -s | tr '[:upper:]' '[:lower:]')_$(uname -m | sed 's/x86_64/amd64/;s/aarch64/arm64/').tar.gz" | tar -xzf - mcp-publisher

Copilot uses AI. Check for mistakes.
"type": "named",
"name": "--port",
"description": "Port of the Storybook dev server exposing the MCP endpoint.",
"default": "6006",
Copy link

Copilot AI Mar 19, 2026

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The packageArguments[0] declares format: "number" but the default value is a JSON string ("6006"). This is likely to fail schema validation or be mis-typed by tooling; use a numeric default (6006) if the schema expects a number.

Suggested change
"default": "6006",
"default": 6006,

Copilot uses AI. Check for mistakes.
@JReinhold JReinhold merged commit 2440718 into main Mar 19, 2026
12 of 13 checks passed
@JReinhold JReinhold deleted the shilman/add-registry-server-json branch March 19, 2026 21:08
@storybook-app-bot storybook-app-bot bot mentioned this pull request Mar 19, 2026
@codecov
Copy link
Copy Markdown

codecov bot commented Mar 19, 2026

Bundle Report

Changes will increase total bundle size by 1.94kB (1.77%) ⬆️. This is within the configured threshold ✅

Detailed changes
Bundle name Size Change
@storybook/addon-mcp-esm* 67.39kB 1.94kB (2.97%) ⬆️

ℹ️ *Bundle size includes cached data from a previous commit

Affected Assets, Files, and Routes:

view changes for bundle: @storybook/addon-mcp-esm

Assets Changed:

Asset Name Size Change Total Size Change (%)
preset.js 1.94kB 67.39kB 2.97%

Files in preset.js:

  • ./package.json → Total Size: 187 bytes

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.

3 participants