fail build if app is using NoN already

Author: lindsaylevine

index.js

@@ -37,12 +37,16 @@ module.exports = {
       return failBuild(`** Static HTML export next.js projects do not require this plugin **`)
     }
 
-    // TO-DO: check scripts to make sure the app isn't manually running NoN
-    // For now, we'll make it clear in the README
-    // const isAlreadyUsingNextOnNetlify = Object.keys(dependencies).find((dep) => dep === 'next-on-netlify');
-    // if (isAlreadyUsingNextOnNetlify) {
-    //   return failBuild(`This plugin cannot support apps that manually use next-on-netlify. Uninstall next-on-netlify as a dependency to resolve.`);
-    // }
+    const hasNextOnNetlifyInstalled = Object.keys(dependencies).find((dep) => dep === 'next-on-netlify')
+    const hasNextOnNetlifyPostbuildScript = Object.keys(scripts).find(
+      (script) => script === 'postbuild' && scripts[script].includes('next-on-netlify'),
+    )
+    const isAlreadyUsingNextOnNetlify = hasNextOnNetlifyInstalled || hasNextOnNetlifyPostbuildScript
+    if (isAlreadyUsingNextOnNetlify) {
+      return failBuild(
+        `This plugin does not support apps that manually use next-on-netlify. Uninstall next-on-netlify as a dependency to resolve.`,
+      )
+    }
 
     const nextConfigPath = await findUp('next.config.js')
     if (nextConfigPath !== undefined) {

test/index.js

@@ -82,6 +82,36 @@ describe('preBuild()', () => {
     ).rejects.toThrow('** Static HTML export next.js projects do not require this plugin **')
   })
 
+  test('fail build if app has next-on-netlify installed', async () => {
+    const packageJson = {
+      dependencies: { 'next-on-netlify': '123' },
+    }
+    await expect(
+      plugin.onPreBuild({
+        netlifyConfig: {},
+        packageJson,
+        utils,
+      }),
+    ).rejects.toThrow(
+      `This plugin does not support apps that manually use next-on-netlify. Uninstall next-on-netlify as a dependency to resolve.`,
+    )
+  })
+
+  test('fail build if app has next-on-netlify postbuild script', async () => {
+    const packageJson = {
+      scripts: { postbuild: 'next-on-netlify' },
+    }
+    await expect(
+      plugin.onPreBuild({
+        netlifyConfig: {},
+        packageJson,
+        utils,
+      }),
+    ).rejects.toThrow(
+      `This plugin does not support apps that manually use next-on-netlify. Uninstall next-on-netlify as a dependency to resolve.`,
+    )
+  })
+
   test('fail build if the app has no package.json', async () => {
     await expect(
       plugin.onPreBuild({