Merge pull request #128 from storybookjs/fix-feature-detect

Fix manifest detection for Storybook versions lower than `10.2.0-alpha.10`

Author: JReinhold

.changeset/tricky-rabbits-rush.md

@@ -0,0 +1,5 @@
+---
+'@storybook/addon-mcp': patch
+---
+
+Fix disabled docs toolset in Storybook version prior to v10.2.0-alpha.10

packages/addon-mcp/src/tools/is-manifest-available.test.ts

@@ -5,10 +5,12 @@ import type { Options } from 'storybook/internal/types';
 function createMockOptions({
 	featureFlag = false,
 	hasManifests = false,
+	hasLegacyComponentManifestGenerator = false,
 	hasFeaturesObject = true,
 }: {
 	featureFlag?: boolean;
 	hasManifests?: boolean;
+	hasLegacyComponentManifestGenerator?: boolean;
 	hasFeaturesObject?: boolean;
 } = {}): Options {
 	return {
@@ -20,7 +22,12 @@ function createMockOptions({
 						: {};
 				}
 				if (key === 'experimental_manifests') {
-					return hasManifests ? vi.fn() : undefined;
+					return hasManifests
+						? { components: { v: 1, components: {} } }
+						: undefined;
+				}
+				if (key === 'experimental_componentManifestGenerator') {
+					return hasLegacyComponentManifestGenerator ? vi.fn() : undefined;
 				}
 				return undefined;
 			}),
@@ -31,12 +38,18 @@ function createMockOptions({
 describe('getManifestStatus', () => {
 	it.each([
 		{
-			description: 'both feature flag and generator are present',
+			description: 'both feature flag and manifests are present',
 			options: { featureFlag: true, hasManifests: true },
 			expected: { available: true, hasManifests: true, hasFeatureFlag: true },
 		},
 		{
-			description: 'missing generator (unsupported framework)',
+			description:
+				'both feature flag and legacy component manifest generator are present',
+			options: { featureFlag: true, hasLegacyComponentManifestGenerator: true },
+			expected: { available: true, hasManifests: true, hasFeatureFlag: true },
+		},
+		{
+			description: 'missing manifests (unsupported framework)',
 			options: { featureFlag: true, hasManifests: false },
 			expected: { available: false, hasManifests: false, hasFeatureFlag: true },
 		},

packages/addon-mcp/src/tools/is-manifest-available.ts

@@ -9,14 +9,21 @@ export type ManifestStatus = {
 export const getManifestStatus = async (
 	options: Options,
 ): Promise<ManifestStatus> => {
-	const [features, manifests] = await Promise.all([
+	const [
+		features,
+		manifests,
+		// Added for backwards compatibility with Storybook versions prior to v10.2.0-alpha.10
+		// Should be removed once support for Storybook version < 10.2.0 is dropped
+		legacyComponentManifestGenerator,
+	] = await Promise.all([
 		options.presets.apply('features') as any,
 		options.presets.apply('experimental_manifests', undefined, {
 			manifestEntries: [],
 		}),
+		options.presets.apply('experimental_componentManifestGenerator'),
 	]);
 
-	const hasManifests = !!manifests;
+	const hasManifests = !!manifests || !!legacyComponentManifestGenerator;
 	const hasFeatureFlag = !!features?.experimentalComponentsManifest;
 
 	return {