@netlify/edge-bundler 1.10.0 → 1.11.0

package/dist/bridge.d.ts

@@ -1,3 +1,4 @@
+/// <reference types="node" />
 import { ExecaChildProcess } from 'execa';
 import { Logger } from './logger.js';
 declare type OnBeforeDownloadHook = () => void | Promise<void>;
@@ -17,6 +18,8 @@ interface ProcessRef {
 }
 interface RunOptions {
     pipeOutput?: boolean;
+    env?: NodeJS.ProcessEnv;
+    extendEnv?: boolean;
 }
 declare class DenoBridge {
     cacheDirectory: string;
@@ -41,9 +44,9 @@ declare class DenoBridge {
         global: boolean;
         path: string;
     }>;
-    getEnvironmentVariables(): Record<string, string>;
-    run(args: string[], { pipeOutput }?: RunOptions): Promise<import("execa").ExecaReturnValue<string>>;
-    runInBackground(args: string[], pipeOutput?: boolean, ref?: ProcessRef): Promise<void>;
+    getEnvironmentVariables(inputEnv?: NodeJS.ProcessEnv): NodeJS.ProcessEnv;
+    run(args: string[], { pipeOutput, env: inputEnv, extendEnv }?: RunOptions): Promise<import("execa").ExecaReturnValue<string>>;
+    runInBackground(args: string[], ref?: ProcessRef, { pipeOutput, env: inputEnv, extendEnv }?: RunOptions): Promise<void>;
 }
 export { DenoBridge };
 export type { DenoOptions, OnAfterDownloadHook, OnBeforeDownloadHook, ProcessRef };

package/dist/bridge.js

@@ -116,8 +116,8 @@ class DenoBridge {
         const downloadedPath = await this.getRemoteBinary();
         return { global: false, path: downloadedPath };
     }
-    getEnvironmentVariables() {
-        const env = {};
+    getEnvironmentVariables(inputEnv = {}) {
+        const env = { ...inputEnv };
         if (this.denoDir !== undefined) {
             env.DENO_DIR = this.denoDir;
         }
@@ -125,18 +125,18 @@ class DenoBridge {
     }
     // Runs the Deno CLI in the background and returns a reference to the child
     // process, awaiting its execution.
-    async run(args, { pipeOutput } = {}) {
+    async run(args, { pipeOutput, env: inputEnv, extendEnv = true } = {}) {
         const { path: binaryPath } = await this.getBinaryPath();
-        const env = this.getEnvironmentVariables();
-        const options = { env };
+        const env = this.getEnvironmentVariables(inputEnv);
+        const options = { env, extendEnv };
         return DenoBridge.runWithBinary(binaryPath, args, options, pipeOutput);
     }
     // Runs the Deno CLI in the background, assigning a reference of the child
     // process to a `ps` property in the `ref` argument, if one is supplied.
-    async runInBackground(args, pipeOutput, ref) {
+    async runInBackground(args, ref, { pipeOutput, env: inputEnv, extendEnv = true } = {}) {
         const { path: binaryPath } = await this.getBinaryPath();
-        const env = this.getEnvironmentVariables();
-        const options = { env };
+        const env = this.getEnvironmentVariables(inputEnv);
+        const options = { env, extendEnv };
         const ps = DenoBridge.runWithBinary(binaryPath, args, options, pipeOutput);
         if (ref !== undefined) {
             // eslint-disable-next-line no-param-reassign

package/dist/server/server.d.ts

@@ -1,3 +1,7 @@
+/// <reference types="node" />
+/// <reference types="node" />
+/// <reference types="node" />
+/// <reference types="node" />
 import { OnAfterDownloadHook, OnBeforeDownloadHook } from '../bridge.js';
 import type { EdgeFunction } from '../edge_function.js';
 import { ImportMapFile } from '../import_map.js';
@@ -21,7 +25,7 @@ interface ServeOptions {
     port: number;
     systemLogger?: LogFunction;
 }
-declare const serve: ({ certificatePath, debug, distImportMapPath, inspectSettings, formatExportTypeError, formatImportError, importMaps, onAfterDownload, onBeforeDownload, port, systemLogger, }: ServeOptions) => Promise<(newFunctions: EdgeFunction[]) => Promise<{
+declare const serve: ({ certificatePath, debug, distImportMapPath, inspectSettings, formatExportTypeError, formatImportError, importMaps, onAfterDownload, onBeforeDownload, port, systemLogger, }: ServeOptions) => Promise<(newFunctions: EdgeFunction[], env?: NodeJS.ProcessEnv) => Promise<{
     graph: any;
     success: boolean;
 }>>;

package/dist/server/server.js

@@ -7,7 +7,7 @@ import { ensureLatestTypes } from '../types.js';
 import { killProcess, waitForServer } from './util.js';
 const prepareServer = ({ deno, distDirectory, flags: denoFlags, formatExportTypeError, formatImportError, port, }) => {
     const processRef = {};
-    const startIsolate = async (newFunctions) => {
+    const startIsolate = async (newFunctions, env = {}) => {
         if ((processRef === null || processRef === void 0 ? void 0 : processRef.ps) !== undefined) {
             await killProcess(processRef.ps);
         }
@@ -32,7 +32,14 @@ const prepareServer = ({ deno, distDirectory, flags: denoFlags, formatExportType
             // no-op
         }
         const bootstrapFlags = ['--port', port.toString()];
-        await deno.runInBackground(['run', ...denoFlags, stage2Path, ...bootstrapFlags], true, processRef);
+        // We set `extendEnv: false` to avoid polluting the edge function context
+        // with variables from the user's system, since those will not be available
+        // in the production environment.
+        await deno.runInBackground(['run', ...denoFlags, stage2Path, ...bootstrapFlags], processRef, {
+            pipeOutput: true,
+            env,
+            extendEnv: false,
+        });
         const success = await waitForServer(port, processRef.ps);
         return {
             graph,

package/package.json

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