fix: Do not load pino-pretty in production bundles

yarn-project/accounts/package.json

@@ -72,7 +72,10 @@
         }
       ]
     ],
-    "testTimeout": 30000
+    "testTimeout": 30000,
+    "setupFiles": [
+      "../../foundation/src/jest/setup.mjs"
+    ]
   },
   "dependencies": {
     "@aztec/aztec.js": "workspace:^",

yarn-project/archiver/package.json

@@ -62,7 +62,10 @@
         }
       ]
     ],
-    "testTimeout": 30000
+    "testTimeout": 30000,
+    "setupFiles": [
+      "../../foundation/src/jest/setup.mjs"
+    ]
   },
   "dependencies": {
     "@aztec/circuit-types": "workspace:^",

yarn-project/aztec-faucet/package.json

@@ -56,7 +56,10 @@
         }
       ]
     ],
-    "testTimeout": 30000
+    "testTimeout": 30000,
+    "setupFiles": [
+      "../../foundation/src/jest/setup.mjs"
+    ]
   },
   "dependencies": {
     "@aztec/ethereum": "workspace:^",

yarn-project/aztec-node/package.json

@@ -58,7 +58,10 @@
         }
       ]
     ],
-    "testTimeout": 30000
+    "testTimeout": 30000,
+    "setupFiles": [
+      "../../foundation/src/jest/setup.mjs"
+    ]
   },
   "dependencies": {
     "@aztec/archiver": "workspace:^",

yarn-project/aztec.js/package.json

@@ -74,7 +74,10 @@
         }
       ]
     ],
-    "testTimeout": 30000
+    "testTimeout": 30000,
+    "setupFiles": [
+      "../../foundation/src/jest/setup.mjs"
+    ]
   },
   "dependencies": {
     "@aztec/circuit-types": "workspace:^",

yarn-project/aztec/package.json

@@ -110,7 +110,10 @@
         }
       ]
     ],
-    "testTimeout": 30000
+    "testTimeout": 30000,
+    "setupFiles": [
+      "../../foundation/src/jest/setup.mjs"
+    ]
   },
   "engines": {
     "node": ">=18"

yarn-project/bb-prover/package.json

@@ -63,7 +63,10 @@
         }
       ]
     ],
-    "testTimeout": 30000
+    "testTimeout": 30000,
+    "setupFiles": [
+      "../../foundation/src/jest/setup.mjs"
+    ]
   },
   "dependencies": {
     "@aztec/circuit-types": "workspace:^",

yarn-project/bot/package.json

@@ -50,7 +50,10 @@
         }
       ]
     ],
-    "testTimeout": 30000
+    "testTimeout": 30000,
+    "setupFiles": [
+      "../../foundation/src/jest/setup.mjs"
+    ]
   },
   "dependencies": {
     "@aztec/accounts": "workspace:^",

yarn-project/builder/package.json

@@ -66,7 +66,10 @@
         }
       ]
     ],
-    "testTimeout": 30000
+    "testTimeout": 30000,
+    "setupFiles": [
+      "../../foundation/src/jest/setup.mjs"
+    ]
   },
   "dependencies": {
     "@aztec/foundation": "workspace:^",

yarn-project/circuit-types/package.json

@@ -64,7 +64,10 @@
         }
       ]
     ],
-    "testTimeout": 30000
+    "testTimeout": 30000,
+    "setupFiles": [
+      "../../foundation/src/jest/setup.mjs"
+    ]
   },
   "dependencies": {
     "@aztec/circuits.js": "workspace:^",

yarn-project/circuits.js/package.json

@@ -98,6 +98,9 @@
         }
       ]
     ],
-    "testTimeout": 30000
+    "testTimeout": 30000,
+    "setupFiles": [
+      "../../foundation/src/jest/setup.mjs"
+    ]
   }
 }

yarn-project/cli-wallet/package.json

@@ -64,7 +64,10 @@
         }
       ]
     ],
-    "testTimeout": 30000
+    "testTimeout": 30000,
+    "setupFiles": [
+      "../../foundation/src/jest/setup.mjs"
+    ]
   },
   "dependencies": {
     "@aztec/accounts": "workspace:^",

yarn-project/cli/package.json

@@ -64,7 +64,10 @@
         }
       ]
     ],
-    "testTimeout": 30000
+    "testTimeout": 30000,
+    "setupFiles": [
+      "../../foundation/src/jest/setup.mjs"
+    ]
   },
   "dependencies": {
     "@aztec/archiver": "workspace:^",

yarn-project/end-to-end/package.json

@@ -156,6 +156,9 @@
     },
     "testRegex": "./src/.*\\.test\\.(js|mjs|ts)$",
     "rootDir": "./src",
-    "testTimeout": 30000
+    "testTimeout": 30000,
+    "setupFiles": [
+      "../../foundation/src/jest/setup.mjs"
+    ]
   }
 }

yarn-project/end-to-end/src/shared/browser.ts

@@ -97,7 +97,7 @@ export const browserTestSuite = (
         pageLogger.info(msg.text());
       });
       page.on('pageerror', err => {
-        pageLogger.error(err.toString());
+        pageLogger.error(`Error on web page`, err);
       });
       await page.goto(`${webServerURL}/index.html`);
       while (!(await page.evaluate(() => !!window.AztecJs))) {

yarn-project/entrypoints/package.json

@@ -60,7 +60,10 @@
         }
       ]
     ],
-    "testTimeout": 30000
+    "testTimeout": 30000,
+    "setupFiles": [
+      "../../foundation/src/jest/setup.mjs"
+    ]
   },
   "dependencies": {
     "@aztec/aztec.js": "workspace:^",

yarn-project/epoch-cache/package.json

@@ -87,7 +87,10 @@
         }
       ]
     ],
-    "testTimeout": 30000
+    "testTimeout": 30000,
+    "setupFiles": [
+      "../../foundation/src/jest/setup.mjs"
+    ]
   },
   "engines": {
     "node": ">=18"

yarn-project/ethereum/package.json

@@ -86,9 +86,12 @@
         }
       ]
     ],
-    "testTimeout": 30000
+    "testTimeout": 30000,
+    "setupFiles": [
+      "../../foundation/src/jest/setup.mjs"
+    ]
   },
   "engines": {
     "node": ">=18"
   }
-}
+}

yarn-project/foundation/package.json

@@ -97,7 +97,13 @@
         }
       ]
     ],
-    "testTimeout": 30000
+    "testTimeout": 30000,
+    "setupFilesAfterEnv": [
+      "../../foundation/src/jest/setup.mjs"
+    ],
+    "setupFiles": [
+      "../../foundation/src/jest/setup.mjs"
+    ]
   },
   "dependencies": {
     "@aztec/bb.js": "portal:../../barretenberg/ts",

yarn-project/foundation/src/jest/setup.mjs

@@ -0,0 +1,9 @@
+import { overwriteLoggingStream, pinoPrettyOpts } from '@aztec/foundation/log';
+
+import pretty from 'pino-pretty';
+
+// Overwrite logging stream with pino-pretty. We define this as a separate
+// file so we don't mess up with dependencies in non-testing environments,
+// since pino-pretty messes up with browser bundles.
+// See also https://www.npmjs.com/package/pino-pretty?activeTab=readme#user-content-usage-with-jest
+overwriteLoggingStream(pretty(pinoPrettyOpts));

yarn-project/foundation/src/log/pino-logger.ts

@@ -1,7 +1,6 @@
 import { createColors } from 'colorette';
 import isNode from 'detect-node';
 import { pino, symbols } from 'pino';
-import pretty from 'pino-pretty';
 import { type Writable } from 'stream';
 import { inspect } from 'util';
 
@@ -68,7 +67,7 @@ const [logLevel, logFilters] = parseEnv(process.env.LOG_LEVEL, defaultLogLevel);
 // Transport options for pretty logging to stderr via pino-pretty.
 const useColor = true;
 const { bold, reset } = createColors({ useColor });
-const pinoPrettyOpts = {
+export const pinoPrettyOpts = {
   destination: 2,
   sync: true,
   colorize: useColor,
@@ -78,6 +77,7 @@ const pinoPrettyOpts = {
   customColors: 'fatal:bgRed,error:red,warn:yellow,info:green,verbose:magenta,debug:blue,trace:gray',
   minimumLevel: 'trace' as const,
 };
+
 const prettyTransport: pino.TransportSingleOptions = {
   target: 'pino-pretty',
   options: pinoPrettyOpts,
@@ -113,11 +113,13 @@ const otelTransport: pino.TransportSingleOptions = {
 
 function makeLogger() {
   if (!isNode) {
-    // We are on the browser
+    // We are on the browser.
     return pino({ ...pinoOpts, browser: { asObject: false } });
   } else if (process.env.JEST_WORKER_ID) {
-    // We are on jest, so we need sync logging. We stream to stderr with pretty.
-    return pino(pinoOpts, pretty(pinoPrettyOpts));
+    // We are on jest, so we need sync logging and stream to stderr.
+    // We expect jest/setup.mjs to kick in later and replace set up a pretty logger,
+    // but if for some reason it doesn't, at least we're covered with a default logger.
+    return pino(pinoOpts, pino.destination(2));
   } else {
     // Regular nodejs with transports on worker thread, using pino-pretty for console logging if LOG_JSON
     // is not set, and an optional OTLP transport if the OTLP endpoint is provided.
@@ -142,6 +144,14 @@ logger.verbose(
     : `Browser console logger initialized with level ${logLevel}`,
 );
 
+/**
+ * Overwrites the logging stream with a different destination.
+ * Used by jest/setup.mjs to set up a pretty logger.
+ */
+export function overwriteLoggingStream(stream: Writable): void {
+  (logger as any)[symbols.streamSym] = stream;
+}
+
 /**
  * Registers an additional destination to the pino logger.
  * Use only when working with destinations, not worker transports.

yarn-project/ivc-integration/package.json

@@ -59,7 +59,10 @@
         }
       ]
     ],
-    "testTimeout": 30000
+    "testTimeout": 30000,
+    "setupFiles": [
+      "../../foundation/src/jest/setup.mjs"
+    ]
   },
   "dependencies": {
     "@aztec/bb.js": "../../ts",

yarn-project/key-store/package.json

@@ -54,7 +54,10 @@
         }
       ]
     ],
-    "testTimeout": 30000
+    "testTimeout": 30000,
+    "setupFiles": [
+      "../../foundation/src/jest/setup.mjs"
+    ]
   },
   "dependencies": {
     "@aztec/circuit-types": "workspace:^",

yarn-project/kv-store/package.json

@@ -91,6 +91,9 @@
     ],
     "testRegex": "./src/.*\\.test\\.(js|mjs|ts)$",
     "rootDir": "./src",
-    "testTimeout": 30000
+    "testTimeout": 30000,
+    "setupFiles": [
+      "../../foundation/src/jest/setup.mjs"
+    ]
   }
 }

yarn-project/merkle-tree/package.json

@@ -55,7 +55,10 @@
         }
       ]
     ],
-    "testTimeout": 30000
+    "testTimeout": 30000,
+    "setupFiles": [
+      "../../foundation/src/jest/setup.mjs"
+    ]
   },
   "dependencies": {
     "@aztec/circuit-types": "workspace:^",

yarn-project/noir-contracts.js/package.json

@@ -54,7 +54,10 @@
         }
       ]
     ],
-    "testTimeout": 30000
+    "testTimeout": 30000,
+    "setupFiles": [
+      "../../foundation/src/jest/setup.mjs"
+    ]
   },
   "dependencies": {
     "@aztec/aztec.js": "workspace:^",

yarn-project/noir-protocol-circuits-types/package.json

@@ -58,7 +58,10 @@
         }
       ]
     ],
-    "testTimeout": 30000
+    "testTimeout": 30000,
+    "setupFiles": [
+      "../../foundation/src/jest/setup.mjs"
+    ]
   },
   "dependencies": {
     "@aztec/circuits.js": "workspace:^",

yarn-project/p2p-bootstrap/package.json

@@ -82,7 +82,10 @@
         }
       ]
     ],
-    "testTimeout": 30000
+    "testTimeout": 30000,
+    "setupFiles": [
+      "../../foundation/src/jest/setup.mjs"
+    ]
   },
   "engines": {
     "node": ">=18"

yarn-project/p2p/package.json

@@ -60,7 +60,10 @@
         }
       ]
     ],
-    "testTimeout": 30000
+    "testTimeout": 30000,
+    "setupFiles": [
+      "../../foundation/src/jest/setup.mjs"
+    ]
   },
   "dependencies": {
     "@aztec/circuit-types": "workspace:^",