Skip to content

Improve preview-stories consistency#176

Merged
JReinhold merged 5 commits intomainfrom
fix-non-apps-preview
Feb 26, 2026
Merged

Improve preview-stories consistency#176
JReinhold merged 5 commits intomainfrom
fix-non-apps-preview

Conversation

@JReinhold
Copy link
Copy Markdown
Contributor

@JReinhold JReinhold commented Feb 26, 2026

This PR modifies the description of the Preview Stories tool.
I discovered that non-MCP Apps compatible clients like Claude Code would often think the tool itself was previewing the story, and not understand that the intention of the tool was for the agent to return the URLs to the user for manual inspection.

MCP Apps-capable clients still work fine:

image

Also updated the preview-focused evals to assert that the agent's final response actually includes URLs

@changeset-bot
Copy link
Copy Markdown

changeset-bot bot commented Feb 26, 2026

🦋 Changeset detected

Latest commit: 316dfd4

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

@pkg-pr-new
Copy link
Copy Markdown

pkg-pr-new bot commented Feb 26, 2026

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

commit: 316dfd4

@codecov
Copy link
Copy Markdown

codecov bot commented Feb 26, 2026

Codecov Report

✅ All modified and coverable lines are covered by tests.
⚠️ Please upload report for BASE (main@94bb1c2). Learn more about missing BASE report.
⚠️ Report is 6 commits behind head on main.
✅ All tests successful. No failed tests found.

Additional details and impacted files
@@           Coverage Diff           @@
##             main     #176   +/-   ##
=======================================
  Coverage        ?   72.80%           
=======================================
  Files           ?       41           
  Lines           ?     1162           
  Branches        ?      325           
=======================================
  Hits            ?      846           
  Misses          ?      198           
  Partials        ?      118           

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@codecov
Copy link
Copy Markdown

codecov bot commented Feb 26, 2026

Bundle Report

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

Detailed changes
Bundle name Size Change
@storybook/addon-mcp-esm* 65.7kB 1.4kB (2.17%) ⬆️

ℹ️ *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.4kB 65.7kB 2.17%

Files in preset.js:

  • ./src/tools/preview-stories.ts → Total Size: 3.85kB

@JReinhold JReinhold self-assigned this Feb 26, 2026
Base automatically changed from id-story-input to main February 26, 2026 12:48
Copy link
Copy Markdown
Member

@kasperpeulen kasperpeulen left a comment

Choose a reason for hiding this comment

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

Great!

@JReinhold JReinhold merged commit 7fee33b into main Feb 26, 2026
12 checks passed
@JReinhold JReinhold deleted the fix-non-apps-preview branch February 26, 2026 17:46
@storybook-app-bot storybook-app-bot bot mentioned this pull request Feb 26, 2026
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.

2 participants