fixes #67 do not fail build for unused export script

Author: lindsaylevine

.gitignore

@@ -7,6 +7,7 @@ test/sample/netlify-automatic-functions
 test/sample/my-publish-dir
 test/sample/.next
 test/sample/.netlify
+next.config.js
 
 # Logs
 logs

.prettierignore

@@ -20,3 +20,4 @@ node_modules
 __mocks__
 test/fixtures
 test/sample
+next.config.js

helpers/isStaticExportProject.js

@@ -4,9 +4,14 @@
 
 const isStaticExportProject = ({ build, scripts }) => {
   const NEXT_EXPORT_COMMAND = 'next export'
-  const isSetInNetlifyConfig = build && build.command && build.command.includes(NEXT_EXPORT_COMMAND)
+
+  if (!build || !build.command) return
+
+  const isSetInNetlifyConfig = build.command.includes(NEXT_EXPORT_COMMAND)
+
   const isSetInNpmScript = Object.keys(scripts).find((script) => {
-    return scripts[script].includes(NEXT_EXPORT_COMMAND)
+    const scriptValue = scripts[script]
+    return build.command.includes(script) && scriptValue.includes(NEXT_EXPORT_COMMAND)
   })
   return isSetInNetlifyConfig || isSetInNpmScript
 }

index.js

@@ -29,7 +29,9 @@ module.exports = {
     const { name, scripts = {}, dependencies = {} } = packageJson
 
     if (isStaticExportProject({ build, scripts })) {
-      return failBuild(`** Static HTML export next.js projects do not require this plugin **`)
+      return failBuild(
+        `Static HTML export Next.js projects do not require this plugin. Check your project's build command for 'next export'.`,
+      )
     }
 
     const hasNextOnNetlifyInstalled = dependencies['next-on-netlify'] !== undefined

test/index.js

@@ -63,12 +63,25 @@ describe('preBuild()', () => {
   test('fail build if the app has static html export in npm script', async () => {
     await expect(
       plugin.onPreBuild({
-        netlifyConfig: {},
+        netlifyConfig: { build: { command: 'npm run build' } },
         packageJson: { ...DUMMY_PACKAGE_JSON, scripts: { build: 'next export' } },
         utils,
         constants: { FUNCTIONS_SRC: 'out_functions' },
       }),
-    ).rejects.toThrow('** Static HTML export next.js projects do not require this plugin **')
+    ).rejects.toThrow(
+      `Static HTML export Next.js projects do not require this plugin. Check your project's build command for 'next export'.`,
+    )
+  })
+
+  test('do not fail build if the app has next export in an unused script', async () => {
+    await expect(
+      plugin.onPreBuild({
+        netlifyConfig: {},
+        packageJson: { ...DUMMY_PACKAGE_JSON, scripts: { export: 'next export' } },
+        utils,
+        constants: {},
+      }),
+    ).resolves
   })
 
   test('fail build if the app has static html export in toml/ntl config', async () => {
@@ -79,7 +92,9 @@ describe('preBuild()', () => {
         utils,
         constants: { FUNCTIONS_SRC: 'out_functions' },
       }),
-    ).rejects.toThrow('** Static HTML export next.js projects do not require this plugin **')
+    ).rejects.toThrow(
+      `Static HTML export Next.js projects do not require this plugin. Check your project's build command for 'next export'.`,
+    )
   })
 
   test('fail build if app has next-on-netlify installed', async () => {