@netlify/edge-bundler 12.2.2 → 12.2.3

package/dist/node/bundler.test.js

@@ -355,6 +355,32 @@ test('Handles imports with the `node:` prefix', async () => {
     expect(func1).toBe('ok');
     await cleanup();
 });
+test('Handles Node builtin imports without the `node:` prefix', async () => {
+    const { basePath, cleanup, distPath } = await useFixture('imports_node_builtin');
+    const userDirectory = join(basePath, 'netlify', 'edge-functions');
+    const result = await bundle([userDirectory], distPath, [], {
+        basePath,
+        importMapPaths: [join(userDirectory, 'import_map.json')],
+    });
+    const generatedFiles = await readdir(distPath);
+    expect(result.functions.length).toBe(1);
+    expect(generatedFiles.length).toBe(2);
+    const manifestFile = await readFile(resolve(distPath, 'manifest.json'), 'utf8');
+    const manifest = JSON.parse(manifestFile);
+    expect(() => validateManifest(manifest)).not.toThrowError();
+    const { bundles, import_map: importMapURL, routes } = manifest;
+    expect(bundles.length).toBe(1);
+    expect(bundles[0].format).toBe('eszip2');
+    expect(generatedFiles.includes(bundles[0].asset)).toBe(true);
+    expect(importMapURL).toBe(importMapSpecifier);
+    expect(routes.length).toBe(1);
+    expect(routes[0].function).toBe('func1');
+    expect(routes[0].pattern).toBe('^/func1/?$');
+    const bundlePath = join(distPath, bundles[0].asset);
+    const { func1 } = await runESZIP(bundlePath);
+    expect(func1).toBe('ok');
+    await cleanup();
+});
 test('Loads npm modules from bare specifiers', async () => {
     const systemLogger = vi.fn();
     const { basePath, cleanup, distPath } = await useFixture('imports_npm_module');

package/dist/node/npm_dependencies.d.ts

@@ -18,5 +18,5 @@ export declare const vendorNPMSpecifiers: ({ basePath, directory, functions, imp
     };
     npmSpecifiersWithExtraneousFiles: string[];
     outputFiles: string[];
-} | undefined>;
+}>;
 export {};

package/dist/node/npm_dependencies.js

@@ -185,10 +185,6 @@ export const vendorNPMSpecifiers = async ({ basePath, directory, functions, impo
         environment,
         rootPath,
     });
-    // If we found no specifiers, there's nothing left to do here.
-    if (Object.keys(npmSpecifiers).length === 0) {
-        return;
-    }
     // To bundle an entire module and all its dependencies, create a entrypoint file
     // where we re-export everything from that specifier. We do this for every
     // specifier, and each of these files will become entry points to esbuild.
@@ -198,39 +194,43 @@ export const vendorNPMSpecifiers = async ({ basePath, directory, functions, impo
         await fs.writeFile(filePath, code);
         return { filePath, specifier, types };
     }));
-    const entryPoints = ops.map(({ filePath }) => filePath);
-    // Bundle each of the entrypoints we created. We'll end up with a compiled
-    // version of each, plus any chunks of shared code
-    // between them (such that a common module isn't bundled twice).
-    const { outputFiles } = await build({
-        allowOverwrite: true,
-        banner,
-        bundle: true,
-        entryPoints,
-        format: 'esm',
-        mainFields: ['module', 'browser', 'main'],
-        logLevel: 'error',
-        nodePaths,
-        outdir: temporaryDirectory.path,
-        platform: 'node',
-        splitting: true,
-        target: 'es2020',
-        write: false,
-        define: environment === 'production'
-            ? {
-                'process.env.NODE_ENV': '"production"',
+    const outputFiles = [];
+    if (ops.length !== 0) {
+        const entryPoints = ops.map(({ filePath }) => filePath);
+        // Bundle each of the entrypoints we created. We'll end up with a compiled
+        // version of each, plus any chunks of shared code
+        // between them (such that a common module isn't bundled twice).
+        const { outputFiles: outputFilesFromEsBuild } = await build({
+            allowOverwrite: true,
+            banner,
+            bundle: true,
+            entryPoints,
+            format: 'esm',
+            mainFields: ['module', 'browser', 'main'],
+            logLevel: 'error',
+            nodePaths,
+            outdir: temporaryDirectory.path,
+            platform: 'node',
+            splitting: true,
+            target: 'es2020',
+            write: false,
+            define: environment === 'production'
+                ? {
+                    'process.env.NODE_ENV': '"production"',
+                }
+                : undefined,
+        });
+        outputFiles.push(...outputFilesFromEsBuild.map((file) => file.path));
+        await Promise.all(outputFilesFromEsBuild.map(async (file) => {
+            var _a;
+            const types = (_a = ops.find((op) => path.basename(file.path) === path.basename(op.filePath))) === null || _a === void 0 ? void 0 : _a.types;
+            let content = file.text;
+            if (types) {
+                content = `/// <reference types="${path.relative(path.dirname(file.path), types)}" />\n${content}`;
             }
-            : undefined,
-    });
-    await Promise.all(outputFiles.map(async (file) => {
-        var _a;
-        const types = (_a = ops.find((op) => path.basename(file.path) === path.basename(op.filePath))) === null || _a === void 0 ? void 0 : _a.types;
-        let content = file.text;
-        if (types) {
-            content = `/// <reference types="${path.relative(path.dirname(file.path), types)}" />\n${content}`;
-        }
-        await fs.writeFile(file.path, content);
-    }));
+            await fs.writeFile(file.path, content);
+        }));
+    }
     // Add all Node.js built-ins to the import map, so any unprefixed specifiers
     // (e.g. `process`) resolve to the prefixed versions (e.g. `node:prefix`),
     // which Deno can process.
@@ -270,6 +270,6 @@ export const vendorNPMSpecifiers = async ({ basePath, directory, functions, impo
         directory: temporaryDirectory.path,
         importMap: newImportMap,
         npmSpecifiersWithExtraneousFiles,
-        outputFiles: outputFiles.map((file) => file.path),
+        outputFiles,
     };
 };

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@netlify/edge-bundler",
-  "version": "12.2.2",
+  "version": "12.2.3",
   "description": "Intelligently prepare Netlify Edge Functions for deployment",
   "type": "module",
   "main": "./dist/node/index.js",
@@ -84,5 +84,5 @@
     "urlpattern-polyfill": "8.0.2",
     "uuid": "^9.0.0"
   },
-  "gitHead": "11ec8d0f0bd573150a134cd9af077f5b56afdbb6"
+  "gitHead": "507a010535ba4028153a755b397501109fa872c9"
 }