@contrail/telemetry 2.0.7 → 2.0.8-alpha-formatting.1

package/lib/logger/index.js

@@ -57,6 +57,11 @@ if (globalRecord[TELEMETRY_SINGLETON_KEY]) {
 else {
     globalRecord[TELEMETRY_SINGLETON_KEY] = true;
 }
+// Capture the real console.log before app-framework's hijackConsole() replaces it.
+// This module loads first (as a dependency), so this is always the original.
+// Using it in the stdout stream avoids the infinite loop (hijacked console → pino → console → …)
+// while preserving single-event CloudWatch output (process.stdout.write splits on newlines).
+const originalConsoleLog = console.log;
 // --- Environment & Resource Setup ---
 const semantic_conventions_4 = require("../semantic-conventions");
 const LAMBDA_ENV_OTEL_ATTRIBUTES = getLambdaEnvAttributes();
@@ -82,7 +87,6 @@ const otelLoggerProvider = new sdk_logs_1.LoggerProvider({
 });
 const otelLogger = otelLoggerProvider.getLogger(serviceName);
 // --- Pino Stream Destinations ---
-const stdout_writer_1 = require("./stdout-writer");
 function isPinoPrettyInstalled() {
     try {
         require.resolve('pino-pretty');
@@ -125,7 +129,7 @@ function createStdoutStream() {
                 catch {
                     line = chunk.toString().replace(/\n$/, '');
                 }
-                stdout_writer_1.stdoutWriter.log(line);
+                originalConsoleLog(line);
                 callback();
             },
         }),

package/package.json

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

package/lib/logger/stdout-writer.d.ts

@@ -1,6 +0,0 @@
-export declare const stdoutWriter: {
-    log: {
-        (...data: any[]): void;
-        (message?: any, ...optionalParams: any[]): void;
-    };
-};

package/lib/logger/stdout-writer.js

@@ -1,19 +0,0 @@
-"use strict";
-Object.defineProperty(exports, "__esModule", { value: true });
-exports.stdoutWriter = void 0;
-// 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.
-//
-// Guard: app-framework's hijackConsole() replaces console.log with overridenLog,
-// which routes back into pino — using it here would create an infinite loop.
-// Any other console.log replacement (e.g. the OTel Lambda layer) is safe to use.
-const _capturedConsoleLog = console.log;
-const isConsoleHijacked = _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: isConsoleHijacked
-        ? (...args) => process.stdout.write(args.map(String).join(' ') + '\n')
-        : _capturedConsoleLog,
-};