sasai-common-utils 1.0.53 → 1.0.55

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "sasai-common-utils",
-  "version": "1.0.53",
+  "version": "1.0.55",
   "description": "Reusable utility library for common logging and other shared features.",
   "main": "src/index.js",
   "scripts": {
@@ -13,12 +13,12 @@
     "@elastic/ecs-winston-format": "^1.5.3",
     "@opentelemetry/api-logs": "^0.211.0",
     "@opentelemetry/exporter-logs-otlp-http": "^0.211.0",
+    "@opentelemetry/instrumentation-pino": "^0.55.0",
     "@opentelemetry/resources": "^2.5.0",
     "@opentelemetry/sdk-logs": "^0.211.0",
     "jsonwebtoken": "^9.0.2",
     "klona": "^2.0.6",
     "pino": "^9.6.0",
-    "pino-opentelemetry-transport": "^1.1.0",
     "pino-pretty": "^13.0.0",
     "traverse": "^0.6.11",
     "winston": "^3.17.0"

package/src/features/logger/index.js

@@ -7,7 +7,6 @@ let contextProvider = () => ({});
 let globalConfig = {};
 let loggerInstance;
 let pinoLogger;
-let pinoTransport;
 
 function setContextProvider(provider) {
   contextProvider = typeof provider === "function" ? provider : () => ({});
@@ -20,9 +19,6 @@ function setGlobalConfig(config) {
     DEBUG_MODE: config?.DEBUG_MODE,
     SENSITIVE_KEYS: config?.SENSITIVE_KEYS,
     NODE_ENV: config?.NODE_ENV,
-    OTEL_EXPORTER_OTLP_LOGS_ENDPOINT: config?.OTEL_EXPORTER_OTLP_LOGS_ENDPOINT,
-    OTEL_EXPORTER_OTLP_LOGS_PROTOCOL:
-      config?.OTEL_EXPORTER_OTLP_LOGS_PROTOCOL || "http/protobuf",
   };
 }
 
@@ -86,6 +82,7 @@ function getTraceContext() {
 function initPinoLogger() {
   const pinoOptions = {
     timestamp: false,
+    messageKey: "message",
     level:
       globalConfig.DEBUG_MODE === DEBUG_MODES.DISABLE
         ? "silent"
@@ -101,54 +98,23 @@ function initPinoLogger() {
     },
   };
 
-  if (globalConfig.OTEL_EXPORTER_OTLP_LOGS_ENDPOINT) {
-    const stdoutTarget = {
-      target: require.resolve("pino/file"),
-      options: { destination: 1 },
-    };
-
-    const otelTarget = {
-      target: require.resolve("pino-opentelemetry-transport"),
-      options: {
-        url: globalConfig.OTEL_EXPORTER_OTLP_LOGS_ENDPOINT,
-        protocol: globalConfig.OTEL_EXPORTER_OTLP_LOGS_PROTOCOL,
-        resourceAttributes: {
-          "service.name": globalConfig.SERVICE_NAME,
-          "deployment.environment": globalConfig.NODE_ENV,
-        },
-      },
-    };
-
-    pinoTransport = pino.transport({
-      targets: [stdoutTarget, otelTarget],
-    });
-
-    pinoTransport.on("error", (err) => {
-      console.error("Pino OTEL transport error:", err);
-    });
-
-    pinoTransport.on("close", () => {
-      console.warn("Pino OTEL transport closed");
-    });
-
-    pinoLogger = pino(pinoOptions, pinoTransport);
+  // OTel log export is handled by PinoInstrumentation inside the NodeSDK (tracing.js).
+  // Pino only needs to write to its local destination (stdout or file).
+  // PinoInstrumentation hooks into pino internally and routes records through
+  // the OTel Logs SDK pipeline automatically — no separate transport needed.
+  let destination;
+  if (globalConfig.DEBUG_MODE === DEBUG_MODES.FILE) {
+    destination = pino.destination("app.log");
   } else {
-    let destination;
-    if (globalConfig.DEBUG_MODE === DEBUG_MODES.FILE) {
-      destination = pino.destination("app.log");
-    } else {
-      destination = pino.destination(1);
-    }
-    pinoLogger = pino(pinoOptions, destination);
+    destination = pino.destination(1); // stdout
   }
+  pinoLogger = pino(pinoOptions, destination);
 
   try {
     pinoLogger.info({
       message: "Pino logger initialized",
       "service.name": globalConfig.SERVICE_NAME,
       "deployment.environment": globalConfig.NODE_ENV,
-      otel_logs_endpoint: globalConfig.OTEL_EXPORTER_OTLP_LOGS_ENDPOINT || "",
-      otel_logs_protocol: globalConfig.OTEL_EXPORTER_OTLP_LOGS_PROTOCOL || "",
     });
   } catch (error) {
     console.error("Failed to write startup log:", error);
@@ -165,13 +131,21 @@ function emitLog(level, data) {
       ? cleanString(message)
       : "Application log";
 
-  pinoLogger[level]({
+  const logPayload = {    
+    app: globalConfig.SERVICE_NAME,
     trace_id: trace || undefined,
     span_id: span || undefined,
     parent_span_id: parentSpan || undefined,
-    body: finalMessage,
-    message: finalMessage,
     ...rest,
+  };
+
+  // OTel body (shown in SigNoz list view) = full serialized payload so the
+  // entire structured data is visible at a glance, not just the label string.
+  const fullBody = safeJsonStringify({ message: finalMessage, ...logPayload });
+
+  pinoLogger[level]({
+    ...logPayload,    
+    message: fullBody, // PinoInstrumentation reads messageKey ("message") as OTel body
   });
 }
 
@@ -274,11 +248,11 @@ function createLogger(config) {
 
     flush: async () => {
       try {
-        if (pinoTransport && typeof pinoTransport.flush === "function") {
-          await pinoTransport.flush();
+        if (pinoLogger && typeof pinoLogger.flush === "function") {
+          pinoLogger.flush();
         }
       } catch (error) {
-        console.error("Failed to flush pino transport:", error);
+        console.error("Failed to flush pino logger:", error);
       }
     },
   };