Don't assume frameworkOptions will be an object

storybookjs · Mar 16, 2023 · bc77248 · bc77248
1 parent 8d09434
commit bc77248
Showing 1 changed file with 4 additions and 6 deletions.
diff --git a/code/lib/react-dom-shim/src/preset.ts b/code/lib/react-dom-shim/src/preset.ts
@@ -3,9 +3,8 @@ import type { Options } from '@storybook/types';
 import { version } from 'react-dom/package.json';
 
 export const webpackFinal = async (config: any, options: Options) => {
-  const { legacyRootApi } = await options.presets.apply<{ legacyRootApi?: boolean }>(
-    'frameworkOptions'
-  );
+  const { legacyRootApi } =
+    (await options.presets.apply<{ legacyRootApi?: boolean }>('frameworkOptions')) || {};
 
   const isReact18 = version.startsWith('18') || version.startsWith('0.0.0');
   const useReact17 = legacyRootApi ?? !isReact18;
@@ -24,9 +23,8 @@ export const webpackFinal = async (config: any, options: Options) => {
 };
 
 export const viteFinal = async (config: any, options: Options) => {
-  const { legacyRootApi } = await options.presets.apply<{ legacyRootApi?: boolean }>(
-    'frameworkOptions'
-  );
+  const { legacyRootApi } =
+    (await options.presets.apply<{ legacyRootApi?: boolean }>('frameworkOptions')) || {};
 
   const isReact18 = version.startsWith('18') || version.startsWith('0.0.0');
   const useReact17 = legacyRootApi || !isReact18;