minor comment and type fixes (#100)

Author: JReinhold

packages/addon-mcp/src/mcp-handler.ts

@@ -148,7 +148,7 @@ export async function incomingMessageToWebRequest(
 	return new Request(url, {
 		method: req.method,
 		headers: req.headers as HeadersInit,
-		// 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
 		body: bodyBuffer.length > 0 ? new Uint8Array(bodyBuffer) : undefined,
 	});
 }

packages/addon-mcp/src/preset.ts

@@ -8,7 +8,7 @@ import htmlTemplate from './template.html';
 export const experimental_devServer: PresetProperty<
 	'experimental_devServer'
 > = async (app, options) => {
-	// There is error handling here. The can make the whole storybook app crash with.
+	// There is no error handling here. This can make the whole storybook app crash with:
 	// ValiError: Invalid type: Expected boolean but received "false"
 	const addonOptions = v.parse(AddonOptions, {
 		toolsets: 'toolsets' in options ? options.toolsets : {},

packages/addon-mcp/src/types.ts

@@ -24,7 +24,7 @@ export type AddonOptionsOutput = v.InferOutput<typeof AddonOptions>;
  * Contains Storybook-specific configuration and runtime information.
  * Extends StorybookContext to be compatible with @storybook/mcp tools.
  */
-export interface AddonContext extends StorybookContext {
+export type AddonContext = StorybookContext & {
 	/**
 	 * The Storybook options object containing configuration,
 	 * port, presets, and other runtime information.
@@ -43,7 +43,7 @@ export interface AddonContext extends StorybookContext {
 	disableTelemetry: boolean;
 
 	toolsets?: NonNullable<AddonOptionsOutput>['toolsets'];
-}
+};
 
 /**
  * Schema for a single story input when requesting story URLs.

packages/mcp/bin.ts

@@ -57,7 +57,6 @@ const args = parseArgs({
 const format = args.values.format as OutputFormat;
 
 transport.listen({
-	source: args.values.manifestPath,
 	format,
 	manifestProvider: async () => {
 		const { manifestPath } = args.values;

packages/mcp/src/types.ts

@@ -10,7 +10,7 @@ export type OutputFormat = 'xml' | 'markdown';
  * Custom context passed to MCP server and tools.
  * Contains the request object and optional manifest provider.
  */
-export interface StorybookContext extends Record<string, unknown> {
+export type StorybookContext = {
 	/**
 	 * The incoming HTTP request being processed.
 	 */
@@ -50,7 +50,7 @@ export interface StorybookContext extends Record<string, unknown> {
 		input: { componentId: string };
 		foundComponent?: ComponentManifest;
 	}) => void | Promise<void>;
-}
+};
 
 const JSDocTag = v.record(v.string(), v.array(v.string()));