fix(builder-rsbuild): make options.cache optional for Nx compatibility

fi3ework · fi3ework · commit 208d543a3a9d · 2026-01-16T15:20:39.000+08:00
- Remove hard throw when Storybook options.cache is missing - Align cache handling behavior with official builder-webpack5 - Add unit test to verify no-cache scenario works - Fixes issue #33 (Nx executor passes undefined cache)
diff --git a/packages/builder-rsbuild/src/preview/iframe-rsbuild.config.ts b/packages/builder-rsbuild/src/preview/iframe-rsbuild.config.ts
@@ -179,10 +179,6 @@ export default async (
   const { virtualModules: virtualModuleMapping, entries: dynamicEntries } =
     await getVirtualModules(options)
 
-  if (!options.cache) {
-    throw new Error('Cache is required')
-  }
-
   let contentFromConfig: RsbuildConfig = {}
   const { content } = await loadConfig({
     cwd: workingDir,
diff --git a/packages/builder-rsbuild/tests/preview/iframe-rsbuild.config.test.ts b/packages/builder-rsbuild/tests/preview/iframe-rsbuild.config.test.ts
@@ -98,6 +98,16 @@ const createOptions = (
   return { options, apply }
 }
 
+const createOptionsWithoutCache = (
+  lazyCompilation: LazyCompilationOption | 'unset' = false,
+) => {
+  const { options, apply } = createOptions(lazyCompilation)
+
+  delete options.cache
+
+  return { options, apply }
+}
+
 describe('iframe-rsbuild.config', () => {
   it('overrides rsbuild source.entry with Storybook entry', async () => {
     const { options } = createOptions()
@@ -116,6 +126,16 @@ describe('iframe-rsbuild.config', () => {
     })
   })
 
+  it('does not require cache from storybook options', async () => {
+    const { options } = createOptionsWithoutCache()
+
+    const config = await createIframeRsbuildConfig(
+      options as RsbuildBuilderOptions,
+    )
+
+    expect(config.source?.entry).toBeDefined()
+  })
+
   const runRspackTool = async (
     lazyCompilation: LazyCompilationOption | 'unset',
   ) => {
