Merge pull request #30190 from storybookjs/danny/rnw-doesnt-require-v…

…ite-conf

Automigrations: Skip vite config file migration for react native web

code/lib/cli-storybook/src/automigrate/fixes/vite-config-file.test.ts

@@ -0,0 +1,48 @@
+import { describe, expect, it } from 'vitest';
+
+import type { StorybookConfig } from 'storybook/internal/types';
+
+import { viteConfigFile } from './vite-config-file';
+
+const check = async ({
+  packageManager,
+  main: mainConfig,
+  storybookVersion = '8.0.0',
+}: {
+  packageManager: any;
+  main: Partial<StorybookConfig> & Record<string, unknown>;
+  storybookVersion?: string;
+}) => {
+  return viteConfigFile.check({
+    packageManager,
+    configDir: '',
+    mainConfig: mainConfig as any,
+    storybookVersion,
+  });
+};
+
+describe('no-ops', () => {
+  it('skips when react-native-web-vite', async () => {
+    await expect(
+      check({
+        packageManager: {},
+        main: {
+          framework: '@storybook/react-native-web-vite',
+        },
+      })
+    ).resolves.toBeFalsy();
+  });
+});
+
+describe('continue', () => {
+  it('executes for vite framework', async () => {
+    await expect(
+      check({
+        packageManager: {},
+        main: {
+          framework: '@storybook/react-vite',
+        },
+      })
+    ).resolves.toBeTruthy();
+  });
+});

code/lib/cli-storybook/src/automigrate/fixes/vite-config-file.ts

@@ -42,6 +42,12 @@ export const viteConfigFile = {
       return null;
     }
     const frameworkName = frameworkPackages[frameworkPackageName];
+
+    if (frameworkName === 'react-native-web-vite') {
+      // we don't expect a vite config file for this framework
+      return null;
+    }
+
     const isUsingViteBuilder =
       mainConfig.core?.builder === 'vite' ||
       frameworkPackageName?.includes('vite') ||