@contrail/telemetry 2.0.5 → 2.0.6-alpha.0

package/lib/logger/stdout-writer.js

@@ -1,20 +1,20 @@
 "use strict";
 Object.defineProperty(exports, "__esModule", { value: true });
 exports.stdoutWriter = void 0;
-// Capture the original console.log before app-framework's hijackConsole() replaces it.
+// Capture console.log so we can use it from the pino stdout stream.
 // Lambda treats each console.log() call as a single CloudWatch log event (even with
-// embedded newlines), unlike process.stdout.write() which splits on \n. Using the
-// original lets us pretty-print objects without them scattering across log events.
+// embedded newlines), unlike process.stdout.write() which splits on \n.
 //
-// Guard: if telemetry is imported after hijackConsole(), the captured function would be
-// the hijacked one, creating an infinite loop. Detect by checking for [native code] —
-// falls back to process.stdout.write if console.log has been replaced by anything.
+// Guard: app-framework's hijackConsole() replaces console.log with overridenLog,
+// which routes back into pino — using it here would create an infinite loop.
+// We detect that specific replacement by name. Any other replacement (e.g. the
+// OTel Lambda layer's console patch) is safe to use.
 const _capturedConsoleLog = console.log;
-const isNative = Function.prototype.toString.call(_capturedConsoleLog).includes('[native code]');
+const isAppFrameworkHijack = _capturedConsoleLog.name === 'overridenLog';
 // Wrapped in an object so tests can jest.spyOn(stdoutWriter, 'log') regardless
 // of Jest's console patching strategy (BufferedConsole vs CustomConsole).
 exports.stdoutWriter = {
-    log: isNative
-        ? _capturedConsoleLog
-        : (...args) => process.stdout.write(args.map(String).join(' ') + '\n'),
+    log: isAppFrameworkHijack
+        ? (...args) => process.stdout.write(args.map(String).join(' ') + '\n')
+        : _capturedConsoleLog,
 };

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@contrail/telemetry",
-  "version": "2.0.5",
+  "version": "2.0.6-alpha.0",
   "description": "Telemetry and monitoring utilities for contrail services",
   "main": "lib/index.js",
   "types": "lib/index.d.ts",