@netlify/edge-bundler 7.1.0 → 8.0.0

package/deno/config.ts

@@ -19,22 +19,12 @@ if (func.config === undefined) {
   Deno.exit(exitCodes.NoConfig)
 }
 
-if (typeof func.config !== 'function') {
+if (typeof func.config !== 'object') {
   Deno.exit(exitCodes.InvalidExport)
 }
 
-let config
-
-try {
-  config = await func.config()
-} catch (error) {
-  console.error(error)
-
-  Deno.exit(exitCodes.RuntimeError)
-}
-
 try {
-  const result = JSON.stringify(config)
+  const result = JSON.stringify(func.config)
 
   await Deno.writeTextFile(new URL(collectorURL), result)
 } catch (error) {

package/dist/node/bundler.test.js

@@ -110,7 +110,28 @@ test('Prints a nice error message when user tries importing NPM module', async (
     }
     catch (error) {
         expect(error).toBeInstanceOf(BundleError);
-        expect(error.message).toEqual(`It seems like you're trying to import an npm module. This is only supported in Deno via CDNs like esm.sh. Have you tried 'import mod from "https://esm.sh/p-retry"'?`);
+        expect(error.message).toEqual(`It seems like you're trying to import an npm module. This is only supported via CDNs like esm.sh. Have you tried 'import mod from "https://esm.sh/p-retry"'?`);
+    }
+    finally {
+        await cleanup();
+    }
+});
+test('Prints a nice error message when user tries importing NPM module with npm: scheme', async () => {
+    expect.assertions(2);
+    const { basePath, cleanup, distPath } = await useFixture('imports_npm_module_scheme');
+    const sourceDirectory = join(basePath, 'functions');
+    const declarations = [
+        {
+            function: 'func1',
+            path: '/func1',
+        },
+    ];
+    try {
+        await bundle([sourceDirectory], distPath, declarations, { basePath });
+    }
+    catch (error) {
+        expect(error).toBeInstanceOf(BundleError);
+        expect(error.message).toEqual(`It seems like you're trying to import an npm module. This is only supported via CDNs like esm.sh. Have you tried 'import mod from "https://esm.sh/p-retry"'?`);
     }
     finally {
         await cleanup();

package/dist/node/config.js

@@ -11,9 +11,8 @@ var ConfigExitCode;
     ConfigExitCode[ConfigExitCode["ImportError"] = 2] = "ImportError";
     ConfigExitCode[ConfigExitCode["NoConfig"] = 3] = "NoConfig";
     ConfigExitCode[ConfigExitCode["InvalidExport"] = 4] = "InvalidExport";
-    ConfigExitCode[ConfigExitCode["RuntimeError"] = 5] = "RuntimeError";
-    ConfigExitCode[ConfigExitCode["SerializationError"] = 6] = "SerializationError";
-    ConfigExitCode[ConfigExitCode["InvalidDefaultExport"] = 7] = "InvalidDefaultExport";
+    ConfigExitCode[ConfigExitCode["SerializationError"] = 5] = "SerializationError";
+    ConfigExitCode[ConfigExitCode["InvalidDefaultExport"] = 6] = "InvalidDefaultExport";
 })(ConfigExitCode || (ConfigExitCode = {}));
 const getConfigExtractor = () => {
     const packagePath = getPackagePath();
@@ -75,14 +74,10 @@ const logConfigError = (func, exitCode, stderr, log) => {
             log.system(`No in-source config found for edge function at '${func.path}'`);
             break;
         case ConfigExitCode.InvalidExport:
-            log.user(`'config' export in edge function at '${func.path}' must be a function`);
-            break;
-        case ConfigExitCode.RuntimeError:
-            log.user(`Error while running 'config' function in edge function at '${func.path}'`);
-            log.user(stderr);
+            log.user(`'config' export in edge function at '${func.path}' must be an object`);
             break;
         case ConfigExitCode.SerializationError:
-            log.user(`'config' function in edge function at '${func.path}' must return an object with primitive values only`);
+            log.user(`'config' object in edge function at '${func.path}' must contain primitive values only`);
             break;
         case ConfigExitCode.InvalidDefaultExport:
             throw new BundleError(new Error(`Default export in '${func.path}' must be a function. More on the Edge Functions API at https://ntl.fyi/edge-api.`));

package/dist/node/config.test.js

@@ -35,7 +35,7 @@ test('`getFunctionConfig` extracts configuration properties from function file',
             source: `
         export default async () => new Response("Hello from function two")
 
-        export const config = () => ({})
+        export const config = {}
       `,
         },
         // Config with the wrong type
@@ -45,9 +45,9 @@ test('`getFunctionConfig` extracts configuration properties from function file',
             source: `
       export default async () => new Response("Hello from function two")
 
-      export const config = {}
+      export const config = () => ({})
     `,
-            userLog: /^'config' export in edge function at '(.*)' must be a function$/,
+            userLog: /^'config' export in edge function at '(.*)' must be an object$/,
         },
         // Config with a syntax error
         {
@@ -60,19 +60,6 @@ test('`getFunctionConfig` extracts configuration properties from function file',
     `,
             userLog: /^Could not load edge function at '(.*)'$/,
         },
-        // Config that throws
-        {
-            expectedConfig: {},
-            name: 'func5',
-            source: `
-          export default async () => new Response("Hello from function two")
-
-          export const config = () => {
-            throw new Error('uh-oh')
-          }
-        `,
-            userLog: /^Error while running 'config' function in edge function at '(.*)'$/,
-        },
         // Config with `path`
         {
             expectedConfig: { path: '/home' },
@@ -80,23 +67,8 @@ test('`getFunctionConfig` extracts configuration properties from function file',
             source: `
         export default async () => new Response("Hello from function three")
 
-        export const config = () => ({ path: "/home" })
-      `,
-        },
-        // Config that prints to stdout
-        {
-            expectedConfig: { path: '/home' },
-            name: 'func7',
-            source: `
-        export default async () => new Response("Hello from function three")
-
-        export const config = () => {
-          console.log("Hello from config!")
-
-          return { path: "/home" }
-        }
+        export const config = { path: "/home" }
       `,
-            userLog: /^Hello from config!$/,
         },
     ];
     for (const func of functions) {

package/dist/node/npm_import_error.js

@@ -1,6 +1,6 @@
 class NPMImportError extends Error {
     constructor(originalError, moduleName) {
-        super(`It seems like you're trying to import an npm module. This is only supported in Deno via CDNs like esm.sh. Have you tried 'import mod from "https://esm.sh/${moduleName}"'?`);
+        super(`It seems like you're trying to import an npm module. This is only supported via CDNs like esm.sh. Have you tried 'import mod from "https://esm.sh/${moduleName}"'?`);
         this.name = 'NPMImportError';
         this.stack = originalError.stack;
         // https://github.com/microsoft/TypeScript-wiki/blob/8a66ecaf77118de456f7cd9c56848a40fe29b9b4/Breaking-Changes.md#implicit-any-error-raised-for-un-annotated-callback-arguments-with-no-matching-overload-arguments
@@ -14,6 +14,11 @@ const wrapNpmImportError = (input) => {
             const [, moduleName] = match;
             return new NPMImportError(input, moduleName);
         }
+        const schemeMatch = input.message.match(/Error: Module not found "npm:(.*)"/);
+        if (schemeMatch !== null) {
+            const [, moduleName] = schemeMatch;
+            return new NPMImportError(input, moduleName);
+        }
     }
     return input;
 };

package/package.json

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