@netlify/build 27.9.0 → 27.11.0

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@netlify/build",
-  "version": "27.9.0",
+  "version": "27.11.0",
   "description": "Netlify build module",
   "type": "module",
   "exports": "./src/core/main.js",
@@ -56,7 +56,7 @@
   "license": "MIT",
   "dependencies": {
     "@bugsnag/js": "^7.0.0",
-    "@netlify/edge-bundler": "^1.8.0",
+    "@netlify/edge-bundler": "^1.10.0",
     "@netlify/cache-utils": "^4.0.0",
     "@netlify/config": "^18.1.2",
     "@netlify/functions-utils": "^4.2.2",

package/src/core/feature_flags.js

@@ -18,4 +18,5 @@ export const DEFAULT_FEATURE_FLAGS = {
   buildbot_zisi_esbuild_parser: false,
   edge_functions_cache_cli: false,
   edge_functions_produce_eszip: false,
+  edge_functions_system_logger: false,
 }

package/src/log/logger.js

@@ -113,11 +113,21 @@ export const logWarningSubHeader = function (logs, string, opts) {
 const reduceLogLines = function (lines) {
   return lines
     .map((input) => {
+      if (input instanceof Error) {
+        return `${input.message} ${input.stack}`
+      }
+
       if (typeof input === 'object') {
-        return JSON.stringify(input)
+        try {
+          return JSON.stringify(input)
+        } catch {
+          // Value could not be serialized to JSON, so we return the string
+          // representation.
+          return String(input)
+        }
       }
 
-      return input.toString()
+      return String(input)
     })
     .join(' ')
 }
@@ -125,18 +135,27 @@ const reduceLogLines = function (lines) {
 // Builds a function for logging data to the system logger (i.e. hidden from
 // the user-facing build logs)
 export const getSystemLogger = function (logs, debug, systemLogFile) {
-  // If there's not a file descriptor (or it's set to 0, matching stdout), we
-  // send debug lines to the normal logger. The same happens if the `debug`
-  // flag is used, since that means we want to print logs to stdout.
-  if (!systemLogFile || debug) {
+  // If the `debug` flag is used, we return a function that pipes system logs
+  // to the regular logger, as the intention is for them to end up in stdout.
+  if (debug) {
     return (...args) => log(logs, reduceLogLines(args))
   }
 
+  // If there's not a file descriptor configured for system logs and `debug`
+  // is not set, we return a no-op function that will swallow the errors.
+  if (!systemLogFile) {
+    return () => {
+      // no-op
+    }
+  }
+
+  // Return a function that writes to the file descriptor configured for system
+  // logs.
   const fileDescriptor = createWriteStream(null, { fd: systemLogFile })
 
   fileDescriptor.on('error', () => {
     logError(logs, 'Could not write to system log file')
   })
 
-  return (...args) => fileDescriptor.write(reduceLogLines(args))
+  return (...args) => fileDescriptor.write(`${reduceLogLines(args)}\n`)
 }

package/src/plugins_core/edge_functions/index.js

@@ -14,7 +14,7 @@ import { validateEdgeFunctionsManifest } from './validate_manifest/validate_edge
 const DENO_CLI_CACHE_DIRECTORY = '.netlify/plugins/deno-cli'
 const IMPORT_MAP_FILENAME = 'edge-functions-import-map.json'
 
-// eslint-disable-next-line max-statements
+// eslint-disable-next-line complexity, max-statements
 const coreStep = async function ({
   buildDir,
   constants: {
@@ -57,6 +57,7 @@ const coreStep = async function ({
       distImportMapPath,
       featureFlags,
       importMaps: [importMap].filter(Boolean),
+      systemLogger: featureFlags.edge_functions_system_logger ? systemLog : undefined,
     })
 
     systemLog('Edge Functions manifest:', manifest)