feat: Merge pull request #57 from seamapi/fix-skip-generate-routes

always generate routes on nsm build command

lib/build.js

@@ -21,7 +21,7 @@ const build = async ({
   skipBuild = false,
   nextless = false,
   onlyApiFiles = false,
-  skipNextBuild = false,
+  skipNextBuild = false
 }) => {
   if (skipNextBuild) {
     console.log("Deprecated. Use --skip-build and/or --nextless instead")
@@ -58,36 +58,26 @@ const build = async ({
 
   logv({ skipBuild, onlyApiFiles })
 
-  if (!skipBuild) {
-    const executionOptions = nextless
-      ? getNextlessOptions({
-          dir,
-        })
-      : getNextjsOptions({
-          packageDirWithNext,
-          dir,
-        })
+  const executionOptions = nextless
+    ? getNextlessOptions({
+        dir,
+      })
+    : getNextjsOptions({
+        packageDirWithNext,
+        dir,
+      })
 
+  if (!skipBuild) {
     // Run the build command
     logv("building...")
     await executionOptions.build(dir, null, false, false, false)
-    // Run the generateRoutes command
-    logv("generating routes...")
-    await executionOptions.generateRoutes({
-      onlyApiFiles,
-      skipBuild,
-    })
   }
 
-  // The "generate_routes.ts" file will be generated by searching for source files instead of transpiled ones.
-  // super useful for tests
-  if (nextless && skipBuild) {
-    logv("generating routes...")
-    await getNextlessOptions({ dir }).generateRoutes({
-      onlyApiFiles,
-      skipBuild,
-    })
-  }
+  logv("generating routes...")
+  await executionOptions.generateRoutes({
+    onlyApiFiles,
+    skipBuild,
+  })
 
   logv("copying nextjs config...")
   await require(`${dir}/.nsm/scripts/copy-nextjs-config.js`).copyNextjsConfig()

lib/cli.js

@@ -49,7 +49,7 @@ async function main() {
         skipBuild: argv["skip-build"],
         nextless: argv["nextless"],
         onlyApiFiles: argv["only-api-files"],
-        skipNextBuild: argv["skip-next-build"],
+        skipNextBuild: argv["skip-next-build"]
       })
       break
     }