fix: force esbuild target in zisi (#681)

* fix: force esbuild target in zisi

* refactor: reduce esbuild log limit

* refactor: increase esbuild log limit in zisi

src/runtimes/node/list_imports.js

@@ -4,6 +4,11 @@ const { tmpName } = require('tmp-promise')
 
 const { safeUnlink } = require('../../utils/fs')
 
+// Maximum number of log messages that an esbuild instance will produce. This
+// limit is important to avoid out-of-memory errors due to too much data being
+// sent in the Go<>Node IPC channel.
+const ESBUILD_LOG_LIMIT = 10
+
 const getListImportsPlugin = ({ imports, path }) => ({
   name: 'list-imports',
   setup(build) {
@@ -35,11 +40,14 @@ const listImports = async ({ path }) => {
 
   try {
     await esbuild.build({
-      entryPoints: [path],
       bundle: true,
+      entryPoints: [path],
+      logLevel: 'error',
+      logLimit: ESBUILD_LOG_LIMIT,
       outfile: targetPath,
       platform: 'node',
       plugins: [getListImportsPlugin({ imports, path })],
+      target: 'esnext',
     })
   } finally {
     await safeUnlink(targetPath)

tests/fixtures/local-node-module-destructure-require/function.js

@@ -0,0 +1,3 @@
+const { stack } = require('@netlify/mock-package/stack')
+
+module.exports = { stack }

tests/fixtures/local-node-module-destructure-require/package.json

@@ -0,0 +1,7 @@
+{
+  "name": "local-node-module-destructure-require",
+  "version": "1.0.0",
+  "dependencies": {
+    "@netlify/mock-package": "1.0.0"
+  }
+}

tests/fixtures/local-node-module-destructure-require/tsconfig.json

@@ -0,0 +1,5 @@
+{
+  "compilerOptions": {
+    "target": "ES5"
+  }
+}

tests/main.js

@@ -173,6 +173,21 @@ testBundlers('Can require deep paths in node modules', [ESBUILD, ESBUILD_ZISI, D
   }
 })
 
+testBundlers(
+  'Can require Node modules with destructuring expressions',
+  [ESBUILD, ESBUILD_ZISI, DEFAULT],
+  async (bundler, t) => {
+    await zipNode(t, `local-node-module-destructure-require`, {
+      opts: { config: { '*': { nodeBundler: bundler } } },
+    })
+
+    // TO DO: Remove when `parseWithEsbuild` feature flag is decommissioned.
+    await zipNode(t, `local-node-module-destructure-require`, {
+      opts: { config: { '*': { nodeBundler: bundler } }, featureFlags: { parseWithEsbuild: true } },
+    })
+  },
+)
+
 testBundlers('Can require scoped node modules', [ESBUILD, ESBUILD_ZISI, DEFAULT], async (bundler, t) => {
   await zipNode(t, 'node-module-scope', { opts: { config: { '*': { nodeBundler: bundler } } } })
 })