@payloops/observability 0.0.5 → 0.0.7

package/README.md

@@ -5,7 +5,7 @@ Shared observability package for PayLoops services. Provides OpenTelemetry traci
 ## Installation
 
 ```bash
-pnpm add @payloops/observability
+npm install @payloops/observability
 ```
 
 ## Features

package/dist/index.d.ts

@@ -18,6 +18,8 @@ declare function shutdownTelemetry(): Promise<void>;
 
 /**
  * Base logger with trace context mixin
+ * In development: pretty print to console + OTLP
+ * In production: JSON to stdout + OTLP
  */
 declare const logger: pino.Logger<never, boolean>;
 /**
@@ -94,7 +96,7 @@ declare function recordWebhookDelivery(status: 'success' | 'failed', attempt: nu
 declare function recordWebhookLatency(durationMs: number, status: 'success' | 'failed'): void;
 declare function recordHttpRequest(method: string, path: string, statusCode: number, durationMs: number): void;
 declare function recordWorkflowStarted(workflowType: string, taskQueue: string): void;
-declare function recordWorkflowCompleted(workflowType: string, taskQueue: string, durationMs: number): void;
+declare function recordWorkflowCompleted(workflowType: string, taskQueue: string, _durationMs: number): void;
 declare function recordWorkflowFailed(workflowType: string, taskQueue: string, errorType: string): void;
 declare function recordActivityLatency(activityType: string, durationMs: number, status: 'success' | 'failed'): void;
 

package/dist/index.js

@@ -2,6 +2,7 @@
 import { NodeSDK } from "@opentelemetry/sdk-node";
 import { OTLPTraceExporter } from "@opentelemetry/exporter-trace-otlp-http";
 import { OTLPMetricExporter } from "@opentelemetry/exporter-metrics-otlp-http";
+import { OTLPLogExporter } from "@opentelemetry/exporter-logs-otlp-http";
 import { getNodeAutoInstrumentations } from "@opentelemetry/auto-instrumentations-node";
 
 // node_modules/@opentelemetry/resources/build/esm/Resource.js
@@ -229,18 +230,31 @@ var Resource = (
 // src/lib/otel.ts
 import { ATTR_SERVICE_NAME, ATTR_SERVICE_VERSION } from "@opentelemetry/semantic-conventions";
 import { PeriodicExportingMetricReader } from "@opentelemetry/sdk-metrics";
+import {
+  LoggerProvider,
+  BatchLogRecordProcessor
+} from "@opentelemetry/sdk-logs";
+import { logs } from "@opentelemetry/api-logs";
 var sdk = null;
+var loggerProvider = null;
 function initTelemetry(config, serviceVersion = "0.0.1") {
   if (sdk) return sdk;
   const cfg = typeof config === "string" ? { serviceName: config, serviceVersion } : config;
   const otlpEndpoint = cfg.otlpEndpoint || process.env.OTEL_EXPORTER_OTLP_ENDPOINT || "http://localhost:4318";
   const environment = cfg.environment || process.env.NODE_ENV || "development";
+  const resource = new Resource({
+    [ATTR_SERVICE_NAME]: cfg.serviceName,
+    [ATTR_SERVICE_VERSION]: cfg.serviceVersion || "0.0.1",
+    "deployment.environment": environment
+  });
+  const logExporter = new OTLPLogExporter({
+    url: `${otlpEndpoint}/v1/logs`
+  });
+  loggerProvider = new LoggerProvider({ resource });
+  loggerProvider.addLogRecordProcessor(new BatchLogRecordProcessor(logExporter));
+  logs.setGlobalLoggerProvider(loggerProvider);
   sdk = new NodeSDK({
-    resource: new Resource({
-      [ATTR_SERVICE_NAME]: cfg.serviceName,
-      [ATTR_SERVICE_VERSION]: cfg.serviceVersion || "0.0.1",
-      "deployment.environment": environment
-    }),
+    resource,
     traceExporter: new OTLPTraceExporter({
       url: `${otlpEndpoint}/v1/traces`
     }),
@@ -260,7 +274,7 @@ function initTelemetry(config, serviceVersion = "0.0.1") {
   });
   sdk.start();
   process.on("SIGTERM", () => {
-    sdk?.shutdown().then(() => console.log("Telemetry shut down")).catch((err) => console.error("Telemetry shutdown error", err));
+    Promise.all([sdk?.shutdown(), loggerProvider?.shutdown()]).then(() => console.log("Telemetry shut down")).catch((err) => console.error("Telemetry shutdown error", err));
   });
   return sdk;
 }
@@ -272,6 +286,7 @@ function shutdownTelemetry() {
 // src/lib/logger.ts
 import pino from "pino";
 import { trace } from "@opentelemetry/api";
+import { logs as logs2, SeverityNumber as SeverityNumber2 } from "@opentelemetry/api-logs";
 
 // src/lib/context.ts
 import { AsyncLocalStorage } from "async_hooks";
@@ -318,6 +333,20 @@ function createContextFromMemo(memo) {
 // src/lib/logger.ts
 var NODE_ENV = process.env.NODE_ENV || "development";
 var SERVICE_NAME = process.env.OTEL_SERVICE_NAME || "loop";
+var pinoLevelToOtelSeverity = {
+  10: SeverityNumber2.TRACE,
+  // trace
+  20: SeverityNumber2.DEBUG,
+  // debug
+  30: SeverityNumber2.INFO,
+  // info
+  40: SeverityNumber2.WARN,
+  // warn
+  50: SeverityNumber2.ERROR,
+  // error
+  60: SeverityNumber2.FATAL
+  // fatal
+};
 var traceMixin = () => {
   const mixinData = {};
   const span = trace.getActiveSpan();
@@ -335,16 +364,112 @@ var traceMixin = () => {
   }
   return mixinData;
 };
-var logger = pino({
-  level: NODE_ENV === "production" ? "info" : "debug",
-  mixin: traceMixin,
-  base: {
-    service: SERVICE_NAME,
-    env: NODE_ENV
+function createOtelDestination() {
+  const stdout = pino.destination(1);
+  return {
+    write(msg) {
+      stdout.write(msg);
+      try {
+        const logRecord = JSON.parse(msg);
+        const otelLogger = logs2.getLogger(SERVICE_NAME);
+        const span = trace.getActiveSpan();
+        const spanContext = span?.spanContext();
+        otelLogger.emit({
+          severityNumber: pinoLevelToOtelSeverity[logRecord.level] || SeverityNumber2.INFO,
+          severityText: pino.levels.labels[logRecord.level] || "INFO",
+          body: logRecord.msg,
+          attributes: {
+            ...logRecord,
+            // Remove fields that are part of the log record structure
+            msg: void 0,
+            level: void 0,
+            time: void 0
+          },
+          timestamp: logRecord.time ? new Date(logRecord.time).getTime() * 1e6 : Date.now() * 1e6,
+          // nanoseconds
+          ...spanContext && {
+            spanId: spanContext.spanId,
+            traceId: spanContext.traceId,
+            traceFlags: spanContext.traceFlags
+          }
+        });
+      } catch {
+      }
+    }
+  };
+}
+var logger = pino(
+  {
+    level: NODE_ENV === "production" ? "info" : "debug",
+    mixin: traceMixin,
+    base: {
+      service: SERVICE_NAME,
+      env: NODE_ENV
+    },
+    timestamp: pino.stdTimeFunctions.isoTime,
+    // In development, use pino-pretty for console but also send to OTLP
+    ...NODE_ENV !== "production" && {
+      transport: {
+        targets: [
+          {
+            target: "pino-pretty",
+            options: { colorize: true },
+            level: "debug"
+          },
+          {
+            target: "pino/file",
+            options: { destination: 1 },
+            // stdout for OTLP bridge
+            level: "debug"
+          }
+        ]
+      }
+    }
   },
-  timestamp: pino.stdTimeFunctions.isoTime,
-  transport: NODE_ENV !== "production" ? { target: "pino-pretty", options: { colorize: true } } : void 0
-});
+  // In production, use the OTLP destination
+  NODE_ENV === "production" ? createOtelDestination() : void 0
+);
+if (NODE_ENV !== "production") {
+  const originalWrite = process.stdout.write.bind(process.stdout);
+  let buffer = "";
+  process.stdout.write = function(chunk, encodingOrCallback, callback) {
+    const str = typeof chunk === "string" ? chunk : chunk.toString();
+    buffer += str;
+    const lines = buffer.split("\n");
+    buffer = lines.pop() || "";
+    for (const line of lines) {
+      if (line.startsWith("{") && line.includes('"level"')) {
+        try {
+          const logRecord = JSON.parse(line);
+          const otelLogger = logs2.getLogger(SERVICE_NAME);
+          otelLogger.emit({
+            severityNumber: pinoLevelToOtelSeverity[logRecord.level] || SeverityNumber2.INFO,
+            severityText: pino.levels.labels[logRecord.level] || "INFO",
+            body: logRecord.msg,
+            attributes: {
+              service: logRecord.service,
+              env: logRecord.env,
+              trace_id: logRecord.trace_id,
+              span_id: logRecord.span_id,
+              correlation_id: logRecord.correlation_id,
+              merchant_id: logRecord.merchant_id,
+              order_id: logRecord.order_id,
+              workflow_id: logRecord.workflow_id
+            },
+            timestamp: logRecord.time ? new Date(logRecord.time).getTime() * 1e6 : Date.now() * 1e6
+          });
+        } catch {
+        }
+      }
+    }
+    if (typeof encodingOrCallback === "function") {
+      return originalWrite(chunk, encodingOrCallback);
+    } else if (encodingOrCallback) {
+      return originalWrite(chunk, encodingOrCallback, callback);
+    }
+    return originalWrite(chunk);
+  };
+}
 function createActivityLogger(activityName, correlationId) {
   return logger.child({
     activity: activityName,
@@ -453,7 +578,7 @@ function recordHttpRequest(method, path, statusCode, durationMs) {
 function recordWorkflowStarted(workflowType, taskQueue) {
   workflowStartedCounter.add(1, { workflow_type: workflowType, task_queue: taskQueue });
 }
-function recordWorkflowCompleted(workflowType, taskQueue, durationMs) {
+function recordWorkflowCompleted(workflowType, taskQueue, _durationMs) {
   workflowCompletedCounter.add(1, { workflow_type: workflowType, task_queue: taskQueue });
 }
 function recordWorkflowFailed(workflowType, taskQueue, errorType) {

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@payloops/observability",
-  "version": "0.0.5",
+  "version": "0.0.7",
   "type": "module",
   "repository": {
     "type": "git",
@@ -27,9 +27,12 @@
   },
   "dependencies": {
     "@opentelemetry/api": "^1.9.0",
+    "@opentelemetry/api-logs": "^0.57.0",
     "@opentelemetry/auto-instrumentations-node": "^0.53.0",
+    "@opentelemetry/exporter-logs-otlp-http": "^0.57.0",
     "@opentelemetry/exporter-metrics-otlp-http": "^0.57.0",
     "@opentelemetry/exporter-trace-otlp-http": "^0.57.0",
+    "@opentelemetry/sdk-logs": "^0.57.0",
     "@opentelemetry/sdk-metrics": "^1.30.1",
     "@opentelemetry/sdk-node": "^0.57.0",
     "@opentelemetry/semantic-conventions": "^1.28.0",
@@ -37,10 +40,12 @@
     "pino": "^9.6.0"
   },
   "devDependencies": {
+    "@eslint/js": "^9.39.2",
     "@types/node": "^22.0.0",
     "eslint": "^9.0.0",
     "pino-pretty": "^13.0.0",
     "tsup": "^8.3.5",
-    "typescript": "^5.9.3"
+    "typescript": "^5.9.3",
+    "typescript-eslint": "^8.53.0"
   }
 }