Revert "Fix bundling of Server Actions (#51367)"

This reverts commit 4698138.
vercel · Jun 16, 2023 · 9f810e1 · 9f810e1
1 parent cd68352
commit 9f810e1
Show file tree

Hide file tree

Showing 4 changed files with 2 additions and 24 deletions.
diff --git a/packages/next/src/build/webpack-config.ts b/packages/next/src/build/webpack-config.ts
@@ -1385,7 +1385,7 @@ export default async function getBaseWebpackConfig(
       // so that the DefinePlugin can inject process.env values.
 
       // Treat next internals as non-external for server layer
-      if (layer === WEBPACK_LAYERS.server || layer === WEBPACK_LAYERS.action) {
+      if (layer === WEBPACK_LAYERS.server) {
         return
       }
 
@@ -1520,7 +1520,7 @@ export default async function getBaseWebpackConfig(
       (isEsm && isAppLayer)
 
     if (/node_modules[/\\].*\.[mc]?js$/.test(res)) {
-      if (layer === WEBPACK_LAYERS.server || layer === WEBPACK_LAYERS.action) {
+      if (layer === WEBPACK_LAYERS.server) {
         // All packages should be bundled for the server layer if they're not opted out.
         // This option takes priority over the transpilePackages option.
 

diff --git a/test/e2e/app-dir/actions/app-action.test.ts b/test/e2e/app-dir/actions/app-action.test.ts
@@ -14,7 +14,6 @@ createNextDescribe(
     files: __dirname,
     dependencies: {
       react: 'latest',
-      nanoid: 'latest',
       'react-dom': 'latest',
       'server-only': 'latest',
     },
@@ -282,14 +281,6 @@ createNextDescribe(
           }
         })
       })
-
-      it('should bundle external libraries if they are on the action layer', async () => {
-        await next.fetch('/client')
-        const pageBundle = await fs.readFile(
-          join(next.testDir, '.next', 'server', 'app', 'client', 'page.js')
-        )
-        expect(pageBundle.toString()).toContain('node_modules/nanoid/index.js')
-      })
     }
 
     describe('Edge SSR', () => {

diff --git a/test/e2e/app-dir/actions/app/client/actions-lib.js b/test/e2e/app-dir/actions/app/client/actions-lib.js
diff --git a/test/e2e/app-dir/actions/app/client/page.js b/test/e2e/app-dir/actions/app/client/page.js
@@ -3,7 +3,6 @@
 import { useState } from 'react'
 
 import double, { inc, dec, redirectAction, getHeaders } from './actions'
-import { test } from './actions-lib'
 
 export default function Counter() {
   const [count, setCount] = useState(0)
@@ -59,9 +58,6 @@ export default function Counter() {
           submit
         </button>
       </form>
-      <form action={test}>
-        <button>test</button>
-      </form>
     </div>
   )
 }