Fix next.config.js resolution

Author: ehmicky

index.js

@@ -5,6 +5,7 @@ const util = require('util')
 const nextOnNetlify = require('next-on-netlify')
 const { PHASE_PRODUCTION_BUILD } = require('next/constants')
 const { default: loadConfig } = require('next/dist/next-server/server/config')
+const findUp = require('find-up')
 const makeDir = require('make-dir')
 const pathExists = require('path-exists')
 const cpx = require('cpx')
@@ -53,8 +54,8 @@ module.exports = {
       )
     }
 
-    const hasNextConfig = await pathExists('next.config.js')
-    if (hasNextConfig) {
+    const nextConfigPath = await findUp('next.config.js')
+    if (nextConfigPath !== undefined) {
       // If the next config exists, fail build if target isnt in acceptableTargets
       const acceptableTargets = ['serverless', 'experimental-serverless-trace']
       const nextConfig = loadConfig(PHASE_PRODUCTION_BUILD, path.resolve('.'))

package.json

@@ -25,6 +25,7 @@
   "homepage": "https://github.com/netlify/netlify-plugin-nextjs#readme",
   "dependencies": {
     "cpx": "^1.5.0",
+    "find-up": "^4.1.0",
     "make-dir": "^3.1.0",
     "next": "^9.5.3",
     "next-on-netlify": "^2.6.0",

test/fixtures/deep_invalid_next_config/base/.gitkeep

@@ -0,0 +1,3 @@
+module.exports = {
+  target: 'server',
+}

test/fixtures/deep_invalid_next_config/next.config.js

@@ -120,21 +120,24 @@ describe('preBuild()', () => {
     expect(await pathExists('next.config.js')).toBeTruthy()
   })
 
-  test(`fail build if the app's next config has an invalid target`, async () => {
-    const { restoreCwd } = useFixture('invalid_next_config')
-    await plugin.onPreBuild({
-      netlifyConfig: {},
-      packageJson: DUMMY_PACKAGE_JSON,
-      utils,
-      constants: { FUNCTIONS_SRC: 'out_functions' },
-    })
-    restoreCwd()
-
-    const acceptableTargets = ['serverless', 'experimental-serverless-trace']
-    expect(utils.build.failBuild.mock.calls[0][0]).toEqual(
-      `next.config.js must be one of: ${acceptableTargets.join(', ')}`,
-    )
-  })
+  test.each(['invalid_next_config', 'deep_invalid_next_config'])(
+    `fail build if the app's next config has an invalid target`,
+    async (fixtureName) => {
+      const { restoreCwd } = useFixture(fixtureName)
+      await plugin.onPreBuild({
+        netlifyConfig: {},
+        packageJson: DUMMY_PACKAGE_JSON,
+        utils,
+        constants: { FUNCTIONS_SRC: 'out_functions' },
+      })
+      restoreCwd()
+
+      const acceptableTargets = ['serverless', 'experimental-serverless-trace']
+      expect(utils.build.failBuild.mock.calls[0][0]).toEqual(
+        `next.config.js must be one of: ${acceptableTargets.join(', ')}`,
+      )
+    },
+  )
 })
 
 describe('onBuild()', () => {