Fix a bug in the webpack-sources patch that happens sometimes when us…

…ing the webpack cache (#67928)
vercel · Jul 19, 2024 · 45928ce · 45928ce
1 parent c175c24
commit 45928ce
Show file tree

Hide file tree

Showing 5 changed files with 212 additions and 109 deletions.
diff --git a/packages/next/src/build/webpack-build/impl.ts b/packages/next/src/build/webpack-build/impl.ts
@@ -188,7 +188,7 @@ export async function webpackBuildImpl(
   await runWebpackSpan.traceAsyncFn(async () => {
     if (config.experimental.webpackMemoryOptimizations) {
       stringBufferUtils.disableDualStringBufferCaching()
-      stringBufferUtils.enableStringInterning()
+      stringBufferUtils.enterStringInterningRange()
     }
 
     // Run the server compilers first and then the client
@@ -261,7 +261,7 @@ export async function webpackBuildImpl(
     }
 
     if (config.experimental.webpackMemoryOptimizations) {
-      stringBufferUtils.disableStringInterning()
+      stringBufferUtils.exitStringInterningRange()
     }
     inputFileSystem?.purge?.()
 

diff --git a/packages/next/src/compiled/webpack-sources3/index.js b/packages/next/src/compiled/webpack-sources3/index.js
diff --git a/packages/next/types/$$compiled.internal.d.ts b/packages/next/types/$$compiled.internal.d.ts
@@ -453,9 +453,9 @@ declare module 'next/dist/compiled/loader-utils3'
 declare module 'next/dist/compiled/webpack-sources3' {
   interface StringBufferUtils {
     disableDualStringBufferCaching: () => boolean
-    disableStringInterning: () => boolean
     enableDualStringBufferCaching: () => boolean
-    enableStringInterning: () => boolean
+    enterStringInterningRange: () => boolean
+    exitStringInterningRange: () => boolean
   }
   export let stringBufferUtils: StringBufferUtils
 }