Updates edge bundling to use --node-modules-dir=false flag

Adds the `--node-modules-dir=false` flag to deno bundle commands in both tarball and eszip bundling formats. This ensures consistent handling of node modules during edge function bundling and updates related tests to reflect the new flag usage. The change improves reliability of the bundling process by explicitly disabling node modules directory handling.

Author: JakeChampion

packages/build/tests/edge_functions/snapshots/tests.js.md

@@ -1279,7 +1279,7 @@ Generated by [AVA](https://avajs.dev).
     ────────────────────────────────────────────────────────────────␊
     ␊
       Error message␊
-      Error: Command failed with exit code 1: deno bundle --import-map BASE64_STRING --quiet --code-splitting --allow-import --outdir /external/path packages/build/tests/edge_functions/fixtures/functions_invalid/netlify/edge-functions/functions-1.ts␊
+      Error: Command failed with exit code 1: deno bundle --import-map BASE64_STRING --quiet --code-splitting --allow-import --node-modules-dir=manual --outdir /external/path packages/build/tests/edge_functions/fixtures/functions_invalid/netlify/edge-functions/functions-1.ts␊
     STACK TRACE␊
     ␊
         export async () => ␊
@@ -1292,9 +1292,9 @@ Generated by [AVA](https://avajs.dev).
     ␊
       Error properties␊
       {␊
-        shortMessage: 'Command failed with exit code 1: deno bundle --import-map BASE64_STRING --quiet --code-splitting --allow-import --outdir /external/path packages/build/tests/edge_functions/fixtures/functions_invalid/netlify/edge-functions/functions-1.ts',␊
-        command: 'deno bundle --import-map BASE64_STRING --quiet --code-splitting --allow-import --outdir /external/path packages/build/tests/edge_functions/fixtures/functions_invalid/netlify/edge-functions/functions-1.ts',␊
-        escapedCommand: 'deno bundle --import-map "BASE64_STRING" --quiet --code-splitting --allow-import --outdir "/external/path" "packages/build/tests/edge_functions/fixtures/functions_invalid/netlify/edge-functions/functions-1.ts"',␊
+        shortMessage: 'Command failed with exit code 1: deno bundle --import-map BASE64_STRING --quiet --code-splitting --allow-import --node-modules-dir=manual --outdir /external/path packages/build/tests/edge_functions/fixtures/functions_invalid/netlify/edge-functions/functions-1.ts',␊
+        command: 'deno bundle --import-map BASE64_STRING --quiet --code-splitting --allow-import --node-modules-dir=manual --outdir /external/path packages/build/tests/edge_functions/fixtures/functions_invalid/netlify/edge-functions/functions-1.ts',␊
+        escapedCommand: 'deno bundle --import-map "BASE64_STRING" --quiet --code-splitting --allow-import "--node-modules-dir=manual" --outdir "/external/path" "packages/build/tests/edge_functions/fixtures/functions_invalid/netlify/edge-functions/functions-1.ts"',␊
         exitCode: 1,␊
         signal: undefined,␊
         signalDescription: undefined,␊

packages/build/tests/edge_functions/snapshots/tests.js.snap

@@ -1159,70 +1159,3 @@ Generated by [AVA](https://avajs.dev).
     ␊
     (Netlify Build completed in 1ms)␊
     Build step duration: Netlify Build completed in 1ms`
-
-## In integration dev mode, install local plugins and install the integration when forcing build
-
-> Snapshot 1
-
-    `␊
-    Netlify Build                                                 ␊
-    ────────────────────────────────────────────────────────────────␊
-    ␊
-    > Version␊
-      @netlify/build 1.0.0␊
-    ␊
-    > Flags␊
-      context: dev␊
-      debug: true␊
-      repositoryRoot: packages/build/tests/install/fixtures/local_missing_integration␊
-      testOpts:␊
-        pluginsListUrl: test␊
-        silentLingeringProcesses: true␊
-    ␊
-    > Current directory␊
-      packages/build/tests/install/fixtures/local_missing_integration␊
-    ␊
-    > Config file␊
-      packages/build/tests/install/fixtures/local_missing_integration/netlify.toml␊
-    ␊
-    > Resolved config␊
-      build:␊
-        publish: packages/build/tests/install/fixtures/local_missing_integration␊
-        publishOrigin: default␊
-    ␊
-    > Context␊
-      dev␊
-    ␊
-    > Installing extensions␊
-       - functions-injection␊
-    ␊
-    > Loading extensions␊
-       - functions-injection␊
-    ␊
-    functions-injection-buildhooks (onPreBuild event)             ␊
-    ────────────────────────────────────────────────────────────────␊
-    ␊
-    Hello from onPreBuild␊
-    ␊
-    (functions-injection-buildhooks onPreBuild completed in 1ms)␊
-    Build step duration: functions-injection-buildhooks onPreBuild completed in 1ms␊
-    ␊
-    Functions bundling                                            ␊
-    ────────────────────────────────────────────────────────────────␊
-    ␊
-    Packaging Functions generated by functions-injection extension:␊
-     - my_unique_prefix_2_function-with-external-dependencies-directory␊
-     - my_unique_prefix_2_function-with-external-dependencies-directory-multifile␊
-     - my_unique_prefix_2_function-with-external-dependencies.ts␊
-     - my_unique_prefix_2_function-with-local-dependencies.ts␊
-     - my_unique_prefix_2_function-with-no-dependencies.ts␊
-    ␊
-    ␊
-    (Functions bundling completed in 1ms)␊
-    Build step duration: Functions bundling completed in 1ms␊
-    ␊
-    Netlify Build Complete                                        ␊
-    ────────────────────────────────────────────────────────────────␊
-    ␊
-    (Netlify Build completed in 1ms)␊
-    Build step duration: Netlify Build completed in 1ms`

packages/build/tests/install/snapshots/tests.js.md

@@ -845,6 +845,16 @@ describe.skipIf(lt(denoVersion, '2.4.3'))(
       const tarballResult = await runTarball(tarballPath)
       expect(tarballResult.func1).toBe(expectedOutput)
 
+      const tarballExtraction = await tmp.dir({ unsafeCleanup: true })
+      await tar.extract({
+        cwd: tarballExtraction.path,
+        file: tarballPath,
+      })
+      const denoConfigPath = join(tarballExtraction.path, 'deno.json')
+      const denoConfig = JSON.parse(await readFile(denoConfigPath, 'utf8'))
+      expect(denoConfig.imports).toHaveProperty('parent-1')
+      await tarballExtraction.cleanup()
+
       const eszipPath = join(distPath, manifest.bundles[1].asset)
       const eszipResult = await runESZIP(eszipPath, vendorDirectory.path)
       expect(eszipResult.func1).toBe(expectedOutput)

packages/build/tests/install/snapshots/tests.js.snap

@@ -58,7 +58,7 @@ export const bundle = async ({
     importMapData,
     vendorDirectory,
   }
-  const flags = ['--allow-all', '--no-config', `--import-map=${bundlerImportMap}`]
+  const flags = ['--allow-all', '--no-config', '--node-modules-dir=false', `--import-map=${bundlerImportMap}`]
 
   if (!debug) {
     flags.push('--quiet')

packages/edge-bundler/node/bundler.test.ts

@@ -88,6 +88,7 @@ export const bundle = async ({
       '--quiet',
       '--code-splitting',
       '--allow-import',
+      '--node-modules-dir=false',
       '--outdir',
       bundleDir.path,
       ...functions.map((func) => func.path),