@netlify/edge-bundler 10.1.0 → 10.1.2

package/dist/node/bundler.js

@@ -156,7 +156,7 @@ const safelyVendorNPMSpecifiers = async ({ basePath, functions, importMap, logge
             functions: functions.map(({ path }) => path),
             importMap,
             logger,
-            referenceTypes: false,
+            environment: 'production',
             rootPath,
         });
     }

package/dist/node/bundler.test.js

@@ -1,3 +1,4 @@
+import { Buffer } from 'buffer';
 import { access, readdir, readFile, rm, writeFile } from 'fs/promises';
 import { join, resolve } from 'path';
 import process from 'process';
@@ -430,3 +431,27 @@ test('Loads JSON modules', async () => {
     await cleanup();
     await rm(vendorDirectory.path, { force: true, recursive: true });
 });
+test('Supports TSX and process.env', async () => {
+    const { basePath, cleanup, distPath } = await useFixture('tsx');
+    const sourceDirectory = join(basePath, 'functions');
+    const declarations = [
+        {
+            function: 'func1',
+            path: '/func1',
+        },
+    ];
+    const vendorDirectory = await tmp.dir();
+    await bundle([sourceDirectory], distPath, declarations, {
+        basePath,
+        vendorDirectory: vendorDirectory.path,
+    });
+    const manifestFile = await readFile(resolve(distPath, 'manifest.json'), 'utf8');
+    const manifest = JSON.parse(manifestFile);
+    const bundlePath = join(distPath, manifest.bundles[0].asset);
+    process.env.FOO = 'bar';
+    const { func1 } = await runESZIP(bundlePath, vendorDirectory.path);
+    expect(Buffer.from(func1, 'base64').toString()).toBe(`hippedy hoppedy, createElement is now a production property. Here, take this env var: FOO=bar`);
+    await cleanup();
+    await rm(vendorDirectory.path, { force: true, recursive: true });
+    delete process.env.FOO;
+});

package/dist/node/npm_dependencies.d.ts

@@ -7,10 +7,10 @@ interface VendorNPMSpecifiersOptions {
     functions: string[];
     importMap: ImportMap;
     logger: Logger;
-    referenceTypes: boolean;
+    environment: 'production' | 'development';
     rootPath?: string;
 }
-export declare const vendorNPMSpecifiers: ({ basePath, directory, functions, importMap, referenceTypes, rootPath, }: VendorNPMSpecifiersOptions) => Promise<{
+export declare const vendorNPMSpecifiers: ({ basePath, directory, functions, importMap, environment, rootPath, }: VendorNPMSpecifiersOptions) => Promise<{
     cleanup: () => Promise<void>;
     directory: string;
     importMap: {

package/dist/node/npm_dependencies.js

@@ -9,7 +9,7 @@ import { findUp } from 'find-up';
 import getPackageName from 'get-package-name';
 import tmp from 'tmp-promise';
 import { pathsBetween } from './utils/fs.js';
-const TYPESCRIPT_EXTENSIONS = new Set(['.ts', '.cts', '.mts']);
+const TYPESCRIPT_EXTENSIONS = new Set(['.ts', '.tsx', '.cts', '.ctsx', '.mts', '.mtsx']);
 const slugifyPackageName = (specifier) => {
     if (!specifier.startsWith('@'))
         return specifier;
@@ -66,19 +66,26 @@ const safelyDetectTypes = async (packageJsonPath) => {
 // Workaround for https://github.com/evanw/esbuild/issues/1921.
 const banner = {
     js: `
+  import __nfyProcess from "node:process";
+  import {setImmediate as __nfySetImmediate, clearImmediate as __nfyClearImmediate} from "node:timers";
+  import {Buffer as __nfyBuffer} from "node:buffer";
   import {createRequire as ___nfyCreateRequire} from "node:module";
   import {fileURLToPath as ___nfyFileURLToPath} from "node:url";
   import {dirname as ___nfyPathDirname} from "node:path";
   let __filename=___nfyFileURLToPath(import.meta.url);
   let __dirname=___nfyPathDirname(___nfyFileURLToPath(import.meta.url));
   let require=___nfyCreateRequire(import.meta.url);
+  globalThis.process = __nfyProcess;
+  globalThis.setImmediate = __nfySetImmediate;
+  globalThis.clearImmediate = __nfyClearImmediate;
+  globalThis.Buffer = __nfyBuffer;
   `,
 };
 /**
  * Parses a set of functions and returns a list of specifiers that correspond
  * to npm modules.
  */
-const getNPMSpecifiers = async ({ basePath, functions, importMap, referenceTypes, rootPath, }) => {
+const getNPMSpecifiers = async ({ basePath, functions, importMap, environment, rootPath }) => {
     const baseURL = pathToFileURL(basePath);
     const { reasons } = await nodeFileTrace(functions, {
         base: rootPath,
@@ -153,7 +160,7 @@ const getNPMSpecifiers = async ({ basePath, functions, importMap, referenceTypes
         if (isDirectDependency) {
             npmSpecifiers.push({
                 specifier: packageName,
-                types: referenceTypes ? await safelyDetectTypes(path.join(basePath, filePath)) : undefined,
+                types: environment === 'development' ? await safelyDetectTypes(path.join(basePath, filePath)) : undefined,
             });
         }
     }
@@ -162,7 +169,7 @@ const getNPMSpecifiers = async ({ basePath, functions, importMap, referenceTypes
         npmSpecifiersWithExtraneousFiles: [...npmSpecifiersWithExtraneousFiles],
     };
 };
-export const vendorNPMSpecifiers = async ({ basePath, directory, functions, importMap, referenceTypes, rootPath = basePath, }) => {
+export const vendorNPMSpecifiers = async ({ basePath, directory, functions, importMap, environment, rootPath = basePath, }) => {
     // The directories that esbuild will use when resolving Node modules. We must
     // set these manually because esbuild will be operating from a temporary
     // directory that will not live inside the project root, so the normal
@@ -176,7 +183,7 @@ export const vendorNPMSpecifiers = async ({ basePath, directory, functions, impo
         basePath,
         functions,
         importMap: importMap.getContentsWithURLObjects(),
-        referenceTypes,
+        environment,
         rootPath,
     });
     // If we found no specifiers, there's nothing left to do here.
@@ -210,6 +217,11 @@ export const vendorNPMSpecifiers = async ({ basePath, directory, functions, impo
         splitting: true,
         target: 'es2020',
         write: false,
+        define: environment === 'production'
+            ? {
+                'process.env.NODE_ENV': '"production"',
+            }
+            : undefined,
     });
     await Promise.all(outputFiles.map(async (file) => {
         var _a;

package/dist/node/server/server.js

@@ -48,7 +48,7 @@ const prepareServer = ({ basePath, bootstrapURL, deno, distDirectory, distImport
             functions: functions.map(({ path }) => path),
             importMap,
             logger,
-            referenceTypes: true,
+            environment: 'development',
             rootPath,
         });
         if (vendor) {

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@netlify/edge-bundler",
-  "version": "10.1.0",
+  "version": "10.1.2",
   "description": "Intelligently prepare Netlify Edge Functions for deployment",
   "type": "module",
   "main": "./dist/node/index.js",