Release: Prerelease 10.1.0-alpha.7

.circleci/config.yml

@@ -1011,7 +1011,7 @@ workflows:
                 requires:
                     - create-sandboxes
             - vitest-integration:
-                parallelism: 11
+                parallelism: 13
                 requires:
                     - create-sandboxes
             - test-portable-stories:
@@ -1106,7 +1106,7 @@ workflows:
                 requires:
                     - create-sandboxes
             - vitest-integration:
-                parallelism: 6
+                parallelism: 7
                 requires:
                     - create-sandboxes
             - test-portable-stories:

.circleci/src/workflows/daily.yml

@@ -54,7 +54,7 @@ jobs:
       requires:
         - create-sandboxes
   - vitest-integration:
-      parallelism: 11
+      parallelism: 13
       requires:
         - create-sandboxes
   - test-portable-stories:

.circleci/src/workflows/merged.yml

@@ -54,7 +54,7 @@ jobs:
       requires:
         - create-sandboxes
   - vitest-integration:
-      parallelism: 6
+      parallelism: 7
       requires:
         - create-sandboxes
   - test-portable-stories:

CHANGELOG.md

@@ -1,3 +1,8 @@
+## 10.0.5
+
+- Core: Add reentry guard to focus patch - [#32655](https://github.com/storybookjs/storybook/pull/32655), thanks @ia319!
+- Nextjs Vite: Update internal plugin to support `svgr` use cases - [#32957](https://github.com/storybookjs/storybook/pull/32957), thanks @yannbf!
+
 ## 10.0.4
 
 - CLI: Fix issue with running Storybook after being initialized - [#32929](https://github.com/storybookjs/storybook/pull/32929), thanks @yannbf!

CHANGELOG.prerelease.md

@@ -1,3 +1,11 @@
+## 10.1.0-alpha.7
+
+- CSF: Fix export interface declaration for NextPreview - [#32914](https://github.com/storybookjs/storybook/pull/32914), thanks @icopp!
+- Controls: Add range validation in Number Control - [#32539](https://github.com/storybookjs/storybook/pull/32539), thanks @ia319!
+- Fix: Export interface declaration for ReactMeta - [#32915](https://github.com/storybookjs/storybook/pull/32915), thanks @icopp!
+- React: Improve error messages in component manifest - [#32954](https://github.com/storybookjs/storybook/pull/32954), thanks @kasperpeulen!
+- Vitest Addon: Add support for Preact - [#32948](https://github.com/storybookjs/storybook/pull/32948), thanks @yannbf!
+
 ## 10.1.0-alpha.6
 
 - Core: Add reentry guard to focus patch - [#32655](https://github.com/storybookjs/storybook/pull/32655), thanks @ia319!

code/addons/docs/src/blocks/controls/Number.tsx

@@ -49,11 +49,31 @@ export const NumberControl: FC<NumberProps> = ({
       if (Number.isNaN(result)) {
         setParseError(new Error(`'${event.target.value}' is not a number`));
       } else {
-        onChange(result);
+        // Initialize the final value as the user's input
+        let finalValue = result;
+
+        // Clamp to minimum: if finalValue is less than min, use min
+        if (typeof min === 'number' && finalValue < min) {
+          finalValue = min;
+        }
+
+        // Clamp to maximum: if finalValue is greater than max, use max
+        if (typeof max === 'number' && finalValue > max) {
+          finalValue = max;
+        }
+
+        // Pass the clamped final value to the onChange callback
+        onChange(finalValue);
+        // Clear any previous parse errors
         setParseError(null);
+
+        // If the value was clamped, update the input display to the final value
+        if (finalValue !== result) {
+          setInputValue(String(finalValue));
+        }
       }
     },
-    [onChange, setParseError]
+    [onChange, setParseError, min, max]
   );
 
   const onForceVisible = useCallback(() => {

code/addons/vitest/src/constants.ts

@@ -22,6 +22,7 @@ export const SUPPORTED_FRAMEWORKS = [
   '@storybook/nextjs',
   '@storybook/nextjs-vite',
   '@storybook/react-vite',
+  '@storybook/preact-vite',
   '@storybook/svelte-vite',
   '@storybook/vue3-vite',
   '@storybook/html-vite',

code/frameworks/nextjs/src/index.ts

@@ -23,4 +23,4 @@ export function definePreview<Addons extends PreviewAddon<never>[]>(
   });
 }
 
-interface NextPreview<T extends AddonTypes> extends ReactPreview<NextJsTypes & T> {}
+export interface NextPreview<T extends AddonTypes> extends ReactPreview<NextJsTypes & T> {}

code/lib/cli-storybook/src/sandbox-templates.ts

@@ -615,7 +615,7 @@ export const baseTemplates = {
     modifications: {
       extraDependencies: ['preact-render-to-string'],
     },
-    skipTasks: ['e2e-tests', 'bench', 'vitest-integration'],
+    skipTasks: ['e2e-tests', 'bench'],
   },
   'preact-vite/default-ts': {
     name: 'Preact Latest (Vite | TypeScript)',
@@ -628,7 +628,7 @@ export const baseTemplates = {
     modifications: {
       extraDependencies: ['preact-render-to-string'],
     },
-    skipTasks: ['e2e-tests', 'bench', 'vitest-integration'],
+    skipTasks: ['e2e-tests', 'bench'],
   },
   'qwik-vite/default-ts': {
     name: 'Qwik CLI Latest (Vite | TypeScript)',

code/package.json

@@ -283,5 +283,6 @@
         "Dependency Upgrades"
       ]
     ]
-  }
+  },
+  "deferredNextVersion": "10.1.0-alpha.7"
 }

code/renderers/react/src/componentManifest/generateCodeSnippet.ts

@@ -118,17 +118,26 @@ export function getCodeSnippet(
     ? metaPath.get('properties').filter((p) => p.isObjectProperty())
     : [];
 
-  const getRenderPath = (object: NodePath<t.ObjectProperty>[]) =>
-    object
-      .filter((p) => keyOf(p.node) === 'render')
-      .map((p) => p.get('value'))
-      .find(
-        (v): v is NodePath<t.ArrowFunctionExpression | t.FunctionExpression> =>
-          v.isArrowFunctionExpression() || v.isFunctionExpression()
+  const getRenderPath = (object: NodePath<t.ObjectProperty>[]) => {
+    const renderPath = object.find((p) => keyOf(p.node) === 'render')?.get('value');
+
+    if (renderPath?.isIdentifier()) {
+      componentName = renderPath.node.name;
+    }
+    if (
+      renderPath &&
+      !(renderPath.isArrowFunctionExpression() || renderPath.isFunctionExpression())
+    ) {
+      throw renderPath.buildCodeFrameError(
+        'Expected render to be an arrow function or function expression'
       );
+    }
+
+    return renderPath;
+  };
 
-  const renderPath = getRenderPath(storyProps);
   const metaRenderPath = getRenderPath(metaProps);
+  const renderPath = getRenderPath(storyProps);
 
   storyFn ??= renderPath ?? metaRenderPath;
 

code/renderers/react/src/componentManifest/generator.ts

@@ -132,21 +132,23 @@ export const componentManifestGenerator: PresetPropertyFn<
       } satisfies Partial<ComponentManifest>;
 
       if (!component?.reactDocgen) {
-        const error = !component
+        const error = !csf._meta?.component
           ? {
-              name: 'No meta.component specified',
-              message: 'Specify meta.component for the component to be included in the manifest.',
+              name: 'No component found',
+              message:
+                'We could not detect the component from your story file. Specify meta.component.',
             }
           : {
               name: 'No component import found',
-              message: `No component file found for the "${component.componentName}" component.`,
+              message: `No component file found for the "${csf.meta.component}" component.`,
             };
         return {
           ...base,
           error: {
             name: error.name,
             message:
-              csf._metaStatementPath?.buildCodeFrameError(error.message).message ?? error.message,
+              (csf._metaStatementPath?.buildCodeFrameError(error.message).message ??
+                error.message) + `\n\n${entry.importPath}:\n${storyFile}`,
           },
         };
       }

code/renderers/react/src/componentManifest/getComponentImports.ts

@@ -198,7 +198,7 @@ export const getComponents = ({
           });
         }
       } catch (e) {
-        logger.error(e);
+        logger.debug(e);
       }
       if (path) {
         const reactDocgen = getReactDocgen(path, component);