@foam-ai/node-cliengo 0.1.0-alpha.2 → 0.1.0-alpha.20

package/dist/node-cliengo/src/init.js

@@ -25,6 +25,7 @@
  *     features (trace propagation, log enrichment, NR bridging) still work.
  *   - Token undefined + forceExport: throws (fail-fast — you asked to force
  *     export but provided no credentials).
+ *   - Endpoint is always the production Foam endpoint — not configurable.
  *   - Auto-shutdown (SIGTERM/SIGINT): flushes all providers with 5s timeout,
  *     then calls process.exit(0). Without the explicit exit, registering
  *     process.on('SIGTERM') replaces Node's default terminate behavior — services
@@ -50,6 +51,8 @@
  */
 Object.defineProperty(exports, "__esModule", { value: true });
 exports.init = init;
+exports.getFoam = getFoam;
+exports.createFastifyLoggerConfig = createFastifyLoggerConfig;
 const api_1 = require("@opentelemetry/api");
 const exporter_logs_otlp_http_1 = require("@opentelemetry/exporter-logs-otlp-http");
 const exporter_metrics_otlp_http_1 = require("@opentelemetry/exporter-metrics-otlp-http");
@@ -73,6 +76,7 @@ const sns_1 = require("./sns");
 const sqs_1 = require("./sqs");
 const trace_bridge_1 = require("./trace-bridge");
 const instances = new Map();
+let _processHandlersRegistered = false;
 function createInertInstance(serviceName) {
     const resource = (0, resources_1.resourceFromAttributes)({ 'service.name': serviceName });
     const traceProvider = new sdk_trace_base_1.BasicTracerProvider({ resource });
@@ -84,7 +88,6 @@ function createInertInstance(serviceName) {
     return {
         tracer,
         meter,
-        logger: loggerProvider,
         traceProvider,
         meterProvider,
         loggerProvider,
@@ -98,14 +101,24 @@ function createInertInstance(serviceName) {
         }),
         createWinstonTransport: () => ({}),
         createPinoMixin: () => () => ({}),
+        createPinoHttpOptions: () => ({ customProps: () => ({}) }),
         createPinoDestination: () => new node_stream_1.Writable({
             write: (_c, _e, cb) => cb(),
         }),
-        buildTraceparent: () => undefined,
-        injectSnsAttributes: (attrs) => attrs,
-        injectJobData: (data) => data,
-        wrapSqsConsumer: async (_t, _s, _m, fn) => fn(),
-        wrapJobConsumer: async (_t, _s, _j, fn) => fn(),
+        createPinoLogger: () => console,
+        createPinoHttpConfig: (opts) => (opts?.logger ? { logger: opts.logger } : {}),
+        createFastifyLoggerConfig: (opts) => ({ level: opts?.level ?? 'info' }),
+        buildTraceparent: trace_bridge_1.buildTraceparent,
+        injectSnsAttributes: sns_1.injectSnsAttributes,
+        injectJobData: job_1.injectJobData,
+        wrapSqsConsumer: async (...args) => {
+            const fn = typeof args[0] === 'string' ? args[2] : args[3];
+            return fn();
+        },
+        wrapJobConsumer: async (...args) => {
+            const fn = typeof args[0] === 'string' ? args[2] : args[3];
+            return fn();
+        },
         extractParentContext: () => {
             const { ROOT_CONTEXT } = require('@opentelemetry/api');
             return ROOT_CONTEXT;
@@ -114,7 +127,7 @@ function createInertInstance(serviceName) {
     };
 }
 /* eslint-disable @typescript-eslint/no-explicit-any, @typescript-eslint/no-unsafe-member-access, @typescript-eslint/no-unsafe-assignment, @typescript-eslint/no-unsafe-call */
-function init(serviceName, token, options = {}) {
+function init(serviceName, options = {}) {
     const existing = instances.get(serviceName);
     if (existing) {
         // eslint-disable-next-line no-console
@@ -131,20 +144,21 @@ function init(serviceName, token, options = {}) {
     }
     // Resolve NR
     (0, nr_1.resolveNewRelic)(options.newrelic);
+    const token = options.token;
     // Production gate
     const isProduction = process.env.NODE_ENV === 'production';
     const shouldExport = isProduction || (options.forceExport === true);
     // Token validation
     if (shouldExport && !token) {
         if (options.forceExport) {
-            throw new Error('[foam] forceExport is true but no FOAM_OTEL_TOKEN provided — cannot export telemetry');
+            throw new Error('[foam] forceExport is true but no token provided — cannot export telemetry');
         }
         // eslint-disable-next-line no-console
-        console.warn('[foam] no FOAM_OTEL_TOKEN — OTLP export disabled');
+        console.warn('[foam] no token provided — OTLP export disabled');
     }
     const hasToken = Boolean(token);
     const canExport = shouldExport && hasToken;
-    const endpoint = options.endpoint ?? constants_1.FOAM_OTEL_ENDPOINT;
+    const endpoint = constants_1.FOAM_OTEL_ENDPOINT;
     const resource = (0, resources_1.resourceFromAttributes)({ 'service.name': serviceName });
     const authHeaders = token
         ? { Authorization: `Bearer ${token}` }
@@ -218,46 +232,54 @@ function init(serviceName, token, options = {}) {
         process.on('SIGTERM', () => void handler());
         process.on('SIGINT', () => void handler());
     }
-    // Process-level exception capture — recorded as span exceptions so they
-    // land in otel_traces (not otel_logs). Each creates a short-lived span,
-    // records the exception, ends it, then force-flushes the trace provider.
-    process.on('uncaughtException', (err) => {
-        try {
-            const span = tracer.startSpan('uncaughtException', {
-                attributes: { 'process.event': 'uncaughtException' },
-            });
-            span.recordException({
-                name: err.constructor.name,
-                message: err.message,
-                stack: err.stack,
-            });
-            span.setStatus({ code: api_1.SpanStatusCode.ERROR, message: err.message });
-            span.end();
-            void traceProvider.forceFlush().then(() => shutdown());
-        }
-        catch {
-            void shutdown();
-        }
-    });
-    process.on('unhandledRejection', (reason) => {
-        try {
-            const err = reason instanceof Error ? reason : new Error(String(reason));
-            const span = tracer.startSpan('unhandledRejection', {
-                attributes: { 'process.event': 'unhandledRejection' },
-            });
-            span.recordException({
-                name: err.constructor.name,
-                message: err.message,
-                stack: err.stack,
-            });
-            span.setStatus({ code: api_1.SpanStatusCode.ERROR, message: err.message });
-            span.end();
-            void traceProvider.forceFlush();
-        }
-        catch {
-            /* best effort */
-        }
-    });
+    else {
+        process.once('beforeExit', () => {
+            if (!_isShutdown) {
+                // eslint-disable-next-line no-console
+                console.warn('[foam] autoShutdown is false but shutdown() was never called — OTel data may be lost');
+            }
+        });
+    }
+    if (!_processHandlersRegistered) {
+        _processHandlersRegistered = true;
+        process.on('uncaughtException', (err) => {
+            try {
+                const span = tracer.startSpan('uncaughtException', {
+                    attributes: { 'process.event': 'uncaughtException' },
+                });
+                span.recordException({
+                    name: err.constructor.name,
+                    message: err.message,
+                    stack: err.stack,
+                });
+                span.setStatus({ code: api_1.SpanStatusCode.ERROR, message: err.message });
+                span.end();
+                void traceProvider.forceFlush().then(() => shutdown());
+            }
+            catch {
+                void shutdown();
+            }
+        });
+        process.on('unhandledRejection', (reason) => {
+            try {
+                const err = reason instanceof Error ? reason : new Error(String(reason));
+                const span = tracer.startSpan('unhandledRejection', {
+                    attributes: { 'process.event': 'unhandledRejection' },
+                });
+                span.recordException({
+                    name: err.constructor.name,
+                    message: err.message,
+                    stack: err.stack,
+                });
+                span.setStatus({ code: api_1.SpanStatusCode.ERROR, message: err.message });
+                span.end();
+                void traceProvider.forceFlush();
+            }
+            catch {
+                /* best effort */
+            }
+        });
+    }
     // Winston auto-wiring
     if (options.winston) {
         const winstonLogger = options.winston;
@@ -322,7 +344,6 @@ function init(serviceName, token, options = {}) {
     const foam = {
         tracer,
         meter,
-        logger: loggerProvider,
         traceProvider,
         meterProvider,
         loggerProvider,
@@ -334,12 +355,51 @@ function init(serviceName, token, options = {}) {
         createWinstonFormat: () => (0, winston_format_1.createWinstonFormat)(),
         createWinstonTransport: () => (0, winston_transport_1.createWinstonTransport)(loggerProvider, serviceName),
         createPinoMixin: () => (0, pino_mixin_1.createPinoMixin)(),
+        createPinoHttpOptions: () => (0, express_1.createPinoHttpOptions)(),
         createPinoDestination: () => (0, pino_destination_1.createPinoDestination)(loggerProvider, serviceName),
+        createPinoLogger: (opts = {}) => {
+            const pino = require('pino');
+            const mixin = (0, pino_mixin_1.createPinoMixin)();
+            const dest = (0, pino_destination_1.createPinoDestination)(loggerProvider, serviceName);
+            const { streams: userStreams, ...pinoOpts } = opts;
+            const outputStreams = userStreams ?? [
+                { stream: process.stdout },
+            ];
+            return pino({ ...pinoOpts, mixin }, pino.multistream([...outputStreams, { stream: dest }]));
+        },
+        createPinoHttpConfig: (opts) => {
+            const pinoHttpOpts = (0, express_1.createPinoHttpOptions)();
+            return { ...pinoHttpOpts, ...(opts?.logger && { logger: opts.logger }) };
+        },
+        createFastifyLoggerConfig: (opts = {}) => {
+            const pino = require('pino');
+            const mixin = (0, pino_mixin_1.createPinoMixin)();
+            const dest = (0, pino_destination_1.createPinoDestination)(loggerProvider, serviceName);
+            const { streams: userStreams, ...pinoOpts } = opts;
+            const outputStreams = userStreams ?? [
+                { stream: process.stdout },
+            ];
+            return {
+                ...pinoOpts,
+                mixin,
+                stream: pino.multistream([...outputStreams, { stream: dest }]),
+            };
+        },
         buildTraceparent: trace_bridge_1.buildTraceparent,
         injectSnsAttributes: sns_1.injectSnsAttributes,
         injectJobData: job_1.injectJobData,
-        wrapSqsConsumer: (t, s, m, fn) => (0, sqs_1.wrapSqsConsumer)(t, s, m, fn, metrics),
-        wrapJobConsumer: (t, s, j, fn) => (0, job_1.wrapJobConsumer)(t, s, j, fn, metrics),
+        wrapSqsConsumer: (...args) => {
+            if (typeof args[0] === 'string') {
+                return (0, sqs_1.wrapSqsConsumer)(tracer, args[0], args[1], args[2], metrics);
+            }
+            return (0, sqs_1.wrapSqsConsumer)(tracer, args[1], args[2], args[3], metrics);
+        },
+        wrapJobConsumer: (...args) => {
+            if (typeof args[0] === 'string') {
+                return (0, job_1.wrapJobConsumer)(tracer, args[0], args[1], args[2], metrics);
+            }
+            return (0, job_1.wrapJobConsumer)(tracer, args[1], args[2], args[3], metrics);
+        },
         extractParentContext: trace_bridge_1.extractParentContext,
         shutdown,
     };
@@ -347,3 +407,40 @@ function init(serviceName, token, options = {}) {
     return foam;
 }
 /* eslint-enable @typescript-eslint/no-explicit-any, @typescript-eslint/no-unsafe-member-access, @typescript-eslint/no-unsafe-assignment, @typescript-eslint/no-unsafe-call */
+/**
+ * Returns the Foam instance for the given service name (or the only instance
+ * if there's just one). Call this from any module after init() has run —
+ * no circular dependency, no holder pattern needed.
+ */
+function getFoam(serviceName) {
+    if (serviceName) {
+        return instances.get(serviceName);
+    }
+    if (instances.size === 1) {
+        return instances.values().next().value;
+    }
+    return undefined;
+}
+/**
+ * Standalone Fastify logger config that handles the getFoam() lookup internally.
+ * If Foam is initialized, returns { mixin, stream (multistream with OTLP), ...opts }.
+ * If Foam is not initialized (disabled, not yet called), returns opts as-is —
+ * Fastify gets a plain Pino logger with no OTLP destination.
+ *
+ *   // No getFoam() check or ternary needed:
+ *   Fastify({ logger: createFastifyLoggerConfig({ level: 'info' }) })
+ *
+ *   // Dev with pino-pretty:
+ *   Fastify({ logger: createFastifyLoggerConfig({
+ *     level: 'debug',
+ *     streams: [{ stream: pinoPretty() }],
+ *   })})
+ */
+function createFastifyLoggerConfig(opts = {}) {
+    const foam = getFoam();
+    if (foam) {
+        return foam.createFastifyLoggerConfig(opts);
+    }
+    const { streams: _streams, ...pinoOpts } = opts;
+    return { ...pinoOpts };
+}

package/dist/node-cliengo/src/job.d.ts

@@ -26,6 +26,6 @@
 import { type Tracer } from '@opentelemetry/api';
 import type { FoamMetrics } from './types';
 export declare function injectJobData<T extends Record<string, unknown>>(data: T): T & {
-    traceparent?: string;
+    traceparent: string;
 };
 export declare function wrapJobConsumer(tracer: Tracer, spanName: string, jobData: Record<string, unknown>, fn: () => Promise<void>, metrics?: FoamMetrics): Promise<void>;

package/dist/node-cliengo/src/job.js

@@ -31,11 +31,7 @@ const api_1 = require("@opentelemetry/api");
 const nr_1 = require("./nr");
 const trace_bridge_1 = require("./trace-bridge");
 function injectJobData(data) {
-    const tp = (0, trace_bridge_1.buildTraceparent)();
-    if (!tp) {
-        return data;
-    }
-    return { ...data, traceparent: tp };
+    return { ...data, traceparent: (0, trace_bridge_1.buildTraceparent)() };
 }
 async function wrapJobConsumer(tracer, spanName, jobData, fn, metrics) {
     const nr = (0, nr_1.getNr)();

package/dist/node-cliengo/src/logs/pino-destination.d.ts

@@ -21,8 +21,13 @@
  *     60→FATAL. Unknown numeric levels default to INFO.
  *   - Pino uses `msg` for the message field (not `message` like Winston). We
  *     check both for compatibility with custom Pino configs.
- *   - Internal Pino fields (pid, hostname, time) are excluded from attributes
- *     to avoid noise. All other fields are forwarded.
+ *   - Trace context (traceId, spanId, traceparent) is read from the parsed
+ *     JSON line, NOT from getTraceContext(). The pino mixin injects these at
+ *     log-write time (when NR's transaction is still active). The stream write
+ *     happens asynchronously — by then NR's transaction has ended, so calling
+ *     getTraceContext() here would return empty strings.
+ *   - Internal Pino fields (pid, hostname, time) and trace fields (traceId,
+ *     spanId, traceparent) are excluded from forwarded attributes.
  *   - Callback always called — never blocks the Pino stream pipeline.
  *   - Buffer chunks: converted to string before parsing (handles both Buffer
  *     and string inputs from Node streams).

package/dist/node-cliengo/src/logs/pino-destination.js

@@ -22,17 +22,22 @@
  *     60→FATAL. Unknown numeric levels default to INFO.
  *   - Pino uses `msg` for the message field (not `message` like Winston). We
  *     check both for compatibility with custom Pino configs.
- *   - Internal Pino fields (pid, hostname, time) are excluded from attributes
- *     to avoid noise. All other fields are forwarded.
+ *   - Trace context (traceId, spanId, traceparent) is read from the parsed
+ *     JSON line, NOT from getTraceContext(). The pino mixin injects these at
+ *     log-write time (when NR's transaction is still active). The stream write
+ *     happens asynchronously — by then NR's transaction has ended, so calling
+ *     getTraceContext() here would return empty strings.
+ *   - Internal Pino fields (pid, hostname, time) and trace fields (traceId,
+ *     spanId, traceparent) are excluded from forwarded attributes.
  *   - Callback always called — never blocks the Pino stream pipeline.
  *   - Buffer chunks: converted to string before parsing (handles both Buffer
  *     and string inputs from Node streams).
  */
 Object.defineProperty(exports, "__esModule", { value: true });
 exports.createPinoDestination = createPinoDestination;
+const api_1 = require("@opentelemetry/api");
 const api_logs_1 = require("@opentelemetry/api-logs");
 const node_stream_1 = require("node:stream");
-const trace_bridge_1 = require("../trace-bridge");
 const PINO_LEVEL_MAP = {
     10: api_logs_1.SeverityNumber.TRACE,
     20: api_logs_1.SeverityNumber.DEBUG,
@@ -66,16 +71,23 @@ function createPinoDestination(loggerProvider, serviceName) {
                 const msg = parsed.msg ?? parsed.message ?? '';
                 const severity = PINO_LEVEL_MAP[level] ?? api_logs_1.SeverityNumber.INFO;
                 const severityText = PINO_LEVEL_TEXT[level] ?? 'INFO';
-                const ctx = (0, trace_bridge_1.getTraceContext)();
+                // Read trace context from the parsed JSON line — the pino mixin already
+                // injected traceId/spanId/traceparent at log-write time (when the NR
+                // transaction was still active). Calling getTraceContext() here would be
+                // too late — the stream write happens asynchronously after NR's
+                // transaction has ended.
+                const traceId = parsed.traceId ?? '';
+                const spanId = parsed.spanId ?? '';
                 const attributes = {};
-                if (ctx.traceId) {
-                    attributes['trace.id'] = ctx.traceId;
+                if (traceId) {
+                    attributes['trace.id'] = traceId;
                 }
-                if (ctx.spanId) {
-                    attributes['span.id'] = ctx.spanId;
+                if (spanId) {
+                    attributes['span.id'] = spanId;
                 }
+                const SKIP_KEYS = new Set(['level', 'msg', 'message', 'time', 'pid', 'hostname', 'traceId', 'spanId', 'traceparent']);
                 for (const [key, val] of Object.entries(parsed)) {
-                    if (key !== 'level' && key !== 'msg' && key !== 'message' && key !== 'time' && key !== 'pid' && key !== 'hostname') {
+                    if (!SKIP_KEYS.has(key)) {
                         try {
                             attributes[key] = typeof val === 'string' ? val : JSON.stringify(val);
                         }
@@ -84,11 +96,24 @@ function createPinoDestination(loggerProvider, serviceName) {
                         }
                     }
                 }
+                // Build an OTel Context carrying the SpanContext so the SDK stamps the
+                // native traceId/spanId fields on the exported OTLP LogRecord. Without
+                // this, backends can't correlate logs to traces.
+                let logContext;
+                if (traceId && spanId) {
+                    logContext = api_1.trace.setSpanContext(api_1.ROOT_CONTEXT, {
+                        traceId,
+                        spanId,
+                        traceFlags: 1,
+                        isRemote: true,
+                    });
+                }
                 otelLogger.emit({
                     body: msg,
                     severityNumber: severity,
                     severityText,
                     attributes,
+                    ...(logContext && { context: logContext }),
                 });
             }
             catch {

package/dist/node-cliengo/src/logs/pino-mixin.d.ts

@@ -11,10 +11,11 @@
  * Used by kori (Fastify built-in Pino) and cb-proxy (Pino + pino-http).
  *
  * Edge cases covered:
- *   - NR not loaded or no active transaction: returns { traceId: '', spanId: '',
- *     traceparent: '' }. Consistent schema — log parsers always see the fields.
- *   - getTraceContext throws: catch returns the same empty-string object.
- *     Never crashes logging.
+ *   - NR not loaded or no active transaction: returns {} so it doesn't
+ *     overwrite valid trace context injected by pino-http's customProps
+ *     (Pino serializes mixin fields after customProps — duplicate keys
+ *     in JSON take the last value).
+ *   - getTraceContext throws: catch returns {}. Never crashes logging.
  *   - Lazy evaluation: the mixin is called at log-write time, not at Pino
  *     construction time, so trace context is always current.
  *   - Works with pino-pretty in development — fields appear in pretty output.

package/dist/node-cliengo/src/logs/pino-mixin.js

@@ -12,10 +12,11 @@
  * Used by kori (Fastify built-in Pino) and cb-proxy (Pino + pino-http).
  *
  * Edge cases covered:
- *   - NR not loaded or no active transaction: returns { traceId: '', spanId: '',
- *     traceparent: '' }. Consistent schema — log parsers always see the fields.
- *   - getTraceContext throws: catch returns the same empty-string object.
- *     Never crashes logging.
+ *   - NR not loaded or no active transaction: returns {} so it doesn't
+ *     overwrite valid trace context injected by pino-http's customProps
+ *     (Pino serializes mixin fields after customProps — duplicate keys
+ *     in JSON take the last value).
+ *   - getTraceContext throws: catch returns {}. Never crashes logging.
  *   - Lazy evaluation: the mixin is called at log-write time, not at Pino
  *     construction time, so trace context is always current.
  *   - Works with pino-pretty in development — fields appear in pretty output.
@@ -27,6 +28,9 @@ function createPinoMixin() {
     return () => {
         try {
             const ctx = (0, trace_bridge_1.getTraceContext)();
+            if (!ctx.traceId) {
+                return {};
+            }
             return {
                 traceId: ctx.traceId,
                 spanId: ctx.spanId,
@@ -34,7 +38,7 @@ function createPinoMixin() {
             };
         }
         catch {
-            return { traceId: '', spanId: '', traceparent: '' };
+            return {};
         }
     };
 }

package/dist/node-cliengo/src/logs/winston-transport.js

@@ -29,6 +29,7 @@
  */
 Object.defineProperty(exports, "__esModule", { value: true });
 exports.createWinstonTransport = createWinstonTransport;
+const api_1 = require("@opentelemetry/api");
 const api_logs_1 = require("@opentelemetry/api-logs");
 const node_stream_1 = require("node:stream");
 const trace_bridge_1 = require("../trace-bridge");
@@ -69,11 +70,21 @@ function createWinstonTransport(loggerProvider, serviceName) {
                         }
                     }
                 }
+                let logContext;
+                if (ctx.traceId && ctx.spanId) {
+                    logContext = api_1.trace.setSpanContext(api_1.ROOT_CONTEXT, {
+                        traceId: ctx.traceId,
+                        spanId: ctx.spanId,
+                        traceFlags: 1,
+                        isRemote: true,
+                    });
+                }
                 otelLogger.emit({
                     body: message,
                     severityNumber: severity,
                     severityText: level.toUpperCase(),
                     attributes,
+                    ...(logContext && { context: logContext }),
                 });
             }
             catch {

package/dist/node-cliengo/src/nr.d.ts

@@ -33,3 +33,20 @@ import type { NewRelicAgent } from './types';
 export declare function resolveNewRelic(explicit?: NewRelicAgent): NewRelicAgent;
 export declare function getNr(): NewRelicAgent;
 export declare function isNrLoaded(): boolean;
+/**
+ * Loads the New Relic agent via CJS require — safe to call from ESM services.
+ * Eliminates the createRequire boilerplate:
+ *
+ *   // Before (ESM service):
+ *   import { createRequire } from 'node:module';
+ *   const require = createRequire(import.meta.url);
+ *   let nr; try { nr = require('newrelic'); } catch {}
+ *   init('kori', { newrelic: nr });
+ *
+ *   // After:
+ *   import { init, loadNewRelic } from '@foam-ai/node-cliengo';
+ *   init('kori', { newrelic: loadNewRelic() });
+ *
+ * Returns undefined if the newrelic package is not installed.
+ */
+export declare function loadNewRelic(): NewRelicAgent | undefined;

package/dist/node-cliengo/src/nr.js

@@ -34,6 +34,7 @@ Object.defineProperty(exports, "__esModule", { value: true });
 exports.resolveNewRelic = resolveNewRelic;
 exports.getNr = getNr;
 exports.isNrLoaded = isNrLoaded;
+exports.loadNewRelic = loadNewRelic;
 const NOOP_NR = {
     getTransaction: () => null,
     startBackgroundTransaction: (_name, _group, handler) => typeof handler === 'function' ? handler() : undefined,
@@ -69,3 +70,31 @@ function getNr() {
 function isNrLoaded() {
     return cachedNr !== null && cachedNr !== NOOP_NR;
 }
+/**
+ * Loads the New Relic agent via CJS require — safe to call from ESM services.
+ * Eliminates the createRequire boilerplate:
+ *
+ *   // Before (ESM service):
+ *   import { createRequire } from 'node:module';
+ *   const require = createRequire(import.meta.url);
+ *   let nr; try { nr = require('newrelic'); } catch {}
+ *   init('kori', { newrelic: nr });
+ *
+ *   // After:
+ *   import { init, loadNewRelic } from '@foam-ai/node-cliengo';
+ *   init('kori', { newrelic: loadNewRelic() });
+ *
+ * Returns undefined if the newrelic package is not installed.
+ */
+function loadNewRelic() {
+    try {
+        /* eslint-disable @typescript-eslint/no-require-imports */
+        const nr = require('newrelic');
+        /* eslint-enable @typescript-eslint/no-require-imports */
+        cachedNr = nr;
+        return nr;
+    }
+    catch {
+        return undefined;
+    }
+}

package/dist/node-cliengo/src/sns.d.ts

@@ -8,10 +8,7 @@
  *   alongside the existing ones. This function spread-merges it in.
  *
  * Edge cases covered:
- *   - No active NR transaction (buildTraceparent returns undefined): the
- *     original attributes are returned unchanged — no traceparent key added.
- *   - Empty attrs {}: returns just { traceparent: ... } if NR is active,
- *     or {} if not. Never crashes.
+ *   - Empty attrs {}: returns { traceparent: ... }. Never crashes.
  *   - Existing traceparent in attrs: overwritten with the current value
  *     (spread puts our key last).
  */

package/dist/node-cliengo/src/sns.js

@@ -9,10 +9,7 @@
  *   alongside the existing ones. This function spread-merges it in.
  *
  * Edge cases covered:
- *   - No active NR transaction (buildTraceparent returns undefined): the
- *     original attributes are returned unchanged — no traceparent key added.
- *   - Empty attrs {}: returns just { traceparent: ... } if NR is active,
- *     or {} if not. Never crashes.
+ *   - Empty attrs {}: returns { traceparent: ... }. Never crashes.
  *   - Existing traceparent in attrs: overwritten with the current value
  *     (spread puts our key last).
  */
@@ -20,12 +17,8 @@ Object.defineProperty(exports, "__esModule", { value: true });
 exports.injectSnsAttributes = injectSnsAttributes;
 const trace_bridge_1 = require("./trace-bridge");
 function injectSnsAttributes(attrs) {
-    const tp = (0, trace_bridge_1.buildTraceparent)();
-    if (!tp) {
-        return attrs;
-    }
     return {
         ...attrs,
-        traceparent: { DataType: 'String', StringValue: tp },
+        traceparent: { DataType: 'String', StringValue: (0, trace_bridge_1.buildTraceparent)() },
     };
 }