Merge @storybook/[email protected] and @storybook/[email protected]

[JReinhold]

Stable releases:

• https://github.com/storybookjs/mcp/releases/tag/%40storybook%2Fmcp%400.1.0
• https://github.com/storybookjs/mcp/releases/tag/%40storybook%2Faddon-mcp%400.1.4

Version PR: #80

[changeset-bot]

⚠️ No Changeset found

Latest commit: d8ce62a

Merging this PR will not cause a version bump for any packages. If these changes should not result in a new version, you're good to go. If these changes should result in a version bump, you need to add a changeset.

Click here to learn what changesets are, and how to add one.

Click here if you're a maintainer who wants to add a changeset to this PR

💥 An error occurred when fetching the changed packages and changesets in this PR

Some errors occurred when validating the changesets config:
The package or glob expression "@storybook/mcp-eval*" is specified in the `ignore` option but it is not found in the project. You may have misspelled the package name or provided an invalid glob expression. Note that glob expressions must be defined according to https://www.npmjs.com/package/micromatch.

[codecov]

Codecov Report

❌ Patch coverage is 92.17391% with 9 lines in your changes missing coverage. Please review.
✅ Project coverage is 89.13%. Comparing base (90a4952) to head (d8ce62a).
✅ All tests successful. No failed tests found.

Files with missing lines	Patch %	Lines
packages/addon-mcp/src/preset.ts	60.00%	1 Missing and 3 partials ⚠️
...kages/mcp/src/utils/manifest-formatter/markdown.ts	95.71%	1 Missing and 2 partials ⚠️
packages/mcp/src/utils/get-manifest.ts	83.33%	0 Missing and 2 partials ⚠️

Additional details and impacted files

@@            Coverage Diff             @@
##             main      #89      +/-   ##
==========================================
+ Coverage   81.91%   89.13%   +7.21%     
==========================================
  Files          15       19       +4     
  Lines         846      414     -432     
  Branches      159      116      -43     
==========================================
- Hits          693      369     -324     
+ Misses        145        6     -139     
- Partials        8       39      +31     

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

[Copilot]

Pull Request Overview

This PR merges stable releases of @storybook/[email protected] and @storybook/[email protected], introducing significant architectural improvements and new features.

Key changes:

• Replaced XML output format with markdown as the default for component documentation, with configurable format support
• Changed manifest loading from URL-based to Request-based architecture with improved context handling
• Modified get-component-documentation tool to handle single components instead of arrays
• Enhanced telemetry collection with toolset tracking

Reviewed Changes

Copilot reviewed 155 out of 177 changed files in this pull request and generated 5 comments.

Show a summary per file

File	Description
packages/mcp/src/utils/manifest-formatter/xml.ts	New XML formatter implementation extracted from format-manifest.ts
packages/mcp/src/utils/manifest-formatter/markdown.ts	New markdown formatter for token-efficient output
packages/mcp/src/utils/manifest-formatter/types.ts	Common interface for manifest formatters
packages/mcp/src/utils/get-manifest.ts	Changed from URL-based to Request-based manifest loading
packages/mcp/src/utils/format-manifest.ts	Refactored to use formatter abstraction with format parameter
packages/mcp/src/types.ts	Added OutputFormat type, replaced 'source' with 'request' in StorybookContext
packages/mcp/src/tools/*.ts	Updated tools to use request-based context and format parameter
packages/addon-mcp/src/mcp-handler.ts	Updated to pass request instead of source URL, improved telemetry handling
packages/addon-mcp/src/preset.ts	Improved HTML template handling and error handling
packages/addon-mcp/src/types.ts	Added experimentalFormat option
package.json	Updated dependencies and added new scripts for linting, testing

[Copilot]

The use of .at(-1)! with a non-null assertion is potentially unsafe. Consider adding a guard check or using strings[strings.length - 1] directly as was done before, since the original code was already safe.

Suggested change

	strings[strings.length - 1] = strings.at(-1)!.replace(/\r?\n([\t ]*)$/, '');
	strings[strings.length - 1] = strings[strings.length - 1].replace(/\r?\n([\t ]*)$/, '');

[Copilot]

The non-null assertion operator ! assumes strings[0] exists. Since strings is constructed from templ which could theoretically be empty, consider adding a guard check before accessing strings[0].

[Copilot]

Corrected grammar in comment from 'The can make' to 'This can make'.

Suggested change

	// There is error handling here. The can make the whole storybook app crash with.
	// There is error handling here. This can make the whole storybook app crash.

[Copilot]

Corrected spelling of 'now' to 'know' in the oxlint-disable comment.

Suggested change

	// oxlint-disable-next-line no-invalid-fetch-options -- We now req.method is always 'POST', linter doesn't
	// oxlint-disable-next-line no-invalid-fetch-options -- We know req.method is always 'POST', linter doesn't

[Copilot]

The 'source' property is being set in the context object, but according to the updated types in src/types.ts, StorybookContext no longer has a 'source' property - it was replaced with 'request'. This property should be removed as it's not part of the type definition and is unused.

Suggested change

source: args.values.manifestPath,