Merge branch 'main' into mk/wasm-support

Author: ascorbic

.github/pull_request_template.md

@@ -1,5 +1,3 @@
-<!--Please tag yourself as the Assignee and netlify/frameworks as the Reviewer -->
-
 ### Summary
 
 <!-- Provide a brief summary of the change. -->

.release-please-manifest.json

@@ -1,4 +1,4 @@
 {
-  "packages/runtime": "4.27.0",
+  "packages/runtime": "4.27.1",
   "packages/next": "1.3.1"
 }

packages/runtime/CHANGELOG.md

@@ -1,5 +1,12 @@
 # Changelog
 
+## [4.27.1](https://github.com/netlify/next-runtime/compare/plugin-nextjs-v4.27.0...plugin-nextjs-v4.27.1) (2022-10-18)
+
+
+### Bug Fixes
+
+* gracefully handle missing static analysis tools ([#1691](https://github.com/netlify/next-runtime/issues/1691)) ([34a039e](https://github.com/netlify/next-runtime/commit/34a039ec80a7c7f050fb5f2dab6f4b8ffddda38a))
+
 ## [4.27.0](https://github.com/netlify/next-runtime/compare/plugin-nextjs-v4.26.0...plugin-nextjs-v4.27.0) (2022-10-17)
 
 

packages/runtime/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@netlify/plugin-nextjs",
-  "version": "4.27.0",
+  "version": "4.27.1",
   "description": "Run Next.js seamlessly on Netlify",
   "main": "index.js",
   "files": [

packages/runtime/src/helpers/analysis.ts

@@ -1,7 +1,5 @@
 import fs, { existsSync } from 'fs'
 
-import { extractExportedConstValue, UnsupportedValueError } from 'next/dist/build/analysis/extract-const-value'
-import { parseModule } from 'next/dist/build/analysis/parse-module'
 import { relative } from 'pathe'
 
 // I have no idea what eslint is up to here but it gives an error
@@ -81,13 +79,39 @@ export const validateConfigValue = (config: ApiConfig, apiFilePath: string): con
   return false
 }
 
+let extractConstValue
+let parseModule
+let hasWarnedAboutNextVersion = false
 /**
  * Uses Next's swc static analysis to extract the config values from a file.
  */
 export const extractConfigFromFile = async (apiFilePath: string): Promise<ApiConfig> => {
   if (!apiFilePath || !existsSync(apiFilePath)) {
     return {}
   }
+
+  try {
+    if (!extractConstValue) {
+      extractConstValue = require('next/dist/build/analysis/extract-const-value')
+    }
+    if (!parseModule) {
+      // eslint-disable-next-line prefer-destructuring, @typescript-eslint/no-var-requires
+      parseModule = require('next/dist/build/analysis/parse-module').parseModule
+    }
+  } catch (error) {
+    if (error.code === 'MODULE_NOT_FOUND') {
+      if (!hasWarnedAboutNextVersion) {
+        console.log("This version of Next.js doesn't support advanced API routes. Skipping...")
+        hasWarnedAboutNextVersion = true
+      }
+      // Old Next.js version
+      return {}
+    }
+    throw error
+  }
+
+  const { extractExportedConstValue, UnsupportedValueError } = extractConstValue
+
   const fileContent = await fs.promises.readFile(apiFilePath, 'utf8')
   // No need to parse if there's no "config"
   if (!fileContent.includes('config')) {
@@ -99,7 +123,7 @@ export const extractConfigFromFile = async (apiFilePath: string): Promise<ApiCon
   try {
     config = extractExportedConstValue(ast, 'config')
   } catch (error) {
-    if (error instanceof UnsupportedValueError) {
+    if (UnsupportedValueError && error instanceof UnsupportedValueError) {
       console.warn(`Unsupported config value in ${relative(process.cwd(), apiFilePath)}`)
     }
     return {}