@newrelic/preflight 1.0.4 → 1.0.5

package/dist/shared/transport/otlp-event-bridge.js

@@ -2,31 +2,40 @@ import { OTLPLogExporter } from '@opentelemetry/exporter-logs-otlp-http';
 import { LoggerProvider, BatchLogRecordProcessor } from '@opentelemetry/sdk-logs';
 import { resourceFromAttributes } from '@opentelemetry/resources';
 import { createLogger } from '../logger.js';
-import { validateOtlpEndpoint, hasOtlpAuthHeader, DEFAULT_CLIENT_NAME } from './otlp-shared.js';
+import { validateOtlpEndpoint, hasOtlpAuthHeader, DEFAULT_CLIENT_NAME, buildUserAgent, sanitizeClientString, } from './otlp-shared.js';
 const logger = createLogger('otlp-event-bridge');
 export class OtlpEventBridge {
     loggerProvider;
     otelLogger;
+    hasAuth;
+    endpoint;
+    hasWarnedNoAuth = false;
     constructor(options) {
         validateOtlpEndpoint(options.endpoint, 'OtlpEventBridge');
-        // Warn once at construction time when no auth header is present (§TR4),
-        // matching the behaviour of OtlpTransport.exportMetrics().
-        if (!hasOtlpAuthHeader(options.headers ?? {})) {
-            logger.warn('OtlpEventBridge constructed with no auth header — collector may reject events', {
-                endpoint: options.endpoint,
-            });
-        }
+        const clientName = sanitizeClientString(options.clientName, DEFAULT_CLIENT_NAME);
+        const clientVersion = sanitizeClientString(options.clientVersion, '');
+        this.endpoint = options.endpoint;
+        this.hasAuth = hasOtlpAuthHeader(options.headers ?? {});
         const exporter = new OTLPLogExporter({
             url: `${options.endpoint}/v1/logs`,
-            headers: options.headers ?? {},
+            headers: {
+                ...(options.headers ?? {}),
+                'User-Agent': buildUserAgent(clientName, clientVersion),
+            },
         });
         this.loggerProvider = new LoggerProvider({
             resource: resourceFromAttributes({ 'service.name': options.appName }),
             processors: [new BatchLogRecordProcessor(exporter)],
         });
-        this.otelLogger = this.loggerProvider.getLogger(options.clientName || DEFAULT_CLIENT_NAME);
+        this.otelLogger = this.loggerProvider.getLogger(clientName, clientVersion || undefined);
     }
     sendEvents(events) {
+        if (!this.hasAuth && !this.hasWarnedNoAuth) {
+            this.hasWarnedNoAuth = true;
+            logger.warn('OtlpEventBridge sending events with no auth header — collector may reject', {
+                endpoint: this.endpoint,
+            });
+        }
         for (const event of events) {
             this.otelLogger.emit({
                 severityText: 'INFO',
@@ -34,7 +43,7 @@ export class OtlpEventBridge {
                 // Filter to scalar values only — the OTel SDK's AnyValue type also
                 // accepts arrays/objects/null, and a non-scalar value would produce a
                 // malformed log record. NrEventData is typed as all-scalar but callers
-                // may pass unexpected shapes (§TR2).
+                // may pass unexpected shapes.
                 attributes: Object.fromEntries(Object.entries(event).filter(([, v]) => typeof v === 'string' || typeof v === 'number' || typeof v === 'boolean')),
                 timestamp: typeof event.timestamp === 'number' ? event.timestamp : Date.now(),
             });

package/dist/shared/transport/otlp-event-bridge.js.map

@@ -1 +1 @@
-{"version":3,"file":"otlp-event-bridge.js","sourceRoot":"","sources":["../../../src/shared/transport/otlp-event-bridge.ts"],"names":[],"mappings":"AAAA,OAAO,EAAE,eAAe,EAAE,MAAM,wCAAwC,CAAC;AACzE,OAAO,EAAE,cAAc,EAAE,uBAAuB,EAAE,MAAM,yBAAyB,CAAC;AAClF,OAAO,EAAE,sBAAsB,EAAE,MAAM,0BAA0B,CAAC;AAElE,OAAO,EAAE,YAAY,EAAE,MAAM,cAAc,CAAC;AAC5C,OAAO,EAAE,oBAAoB,EAAE,iBAAiB,EAAE,mBAAmB,EAAE,MAAM,kBAAkB,CAAC;AAEhG,MAAM,MAAM,GAAG,YAAY,CAAC,mBAAmB,CAAC,CAAC;AAejD,MAAM,OAAO,eAAe;IACT,cAAc,CAAiB;IAC/B,UAAU,CAA0C;IAErE,YAAY,OAA+B;QACzC,oBAAoB,CAAC,OAAO,CAAC,QAAQ,EAAE,iBAAiB,CAAC,CAAC;QAE1D,wEAAwE;QACxE,2DAA2D;QAC3D,IAAI,CAAC,iBAAiB,CAAC,OAAO,CAAC,OAAO,IAAI,EAAE,CAAC,EAAE,CAAC;YAC9C,MAAM,CAAC,IAAI,CAAC,+EAA+E,EAAE;gBAC3F,QAAQ,EAAE,OAAO,CAAC,QAAQ;aAC3B,CAAC,CAAC;QACL,CAAC;QAED,MAAM,QAAQ,GAAG,IAAI,eAAe,CAAC;YACnC,GAAG,EAAE,GAAG,OAAO,CAAC,QAAQ,UAAU;YAClC,OAAO,EAAE,OAAO,CAAC,OAAO,IAAI,EAAE;SAC/B,CAAC,CAAC;QAEH,IAAI,CAAC,cAAc,GAAG,IAAI,cAAc,CAAC;YACvC,QAAQ,EAAE,sBAAsB,CAAC,EAAE,cAAc,EAAE,OAAO,CAAC,OAAO,EAAE,CAAC;YACrE,UAAU,EAAE,CAAC,IAAI,uBAAuB,CAAC,QAAQ,CAAC,CAAC;SACpD,CAAC,CAAC;QAEH,IAAI,CAAC,UAAU,GAAG,IAAI,CAAC,cAAc,CAAC,SAAS,CAAC,OAAO,CAAC,UAAU,IAAI,mBAAmB,CAAC,CAAC;IAC7F,CAAC;IAED,UAAU,CAAC,MAAqB;QAC9B,KAAK,MAAM,KAAK,IAAI,MAAM,EAAE,CAAC;YAC3B,IAAI,CAAC,UAAU,CAAC,IAAI,CAAC;gBACnB,YAAY,EAAE,MAAM;gBACpB,IAAI,EAAE,MAAM,CAAC,KAAK,CAAC,WAAW,CAAC,IAAI,SAAS,CAAC;gBAC7C,mEAAmE;gBACnE,sEAAsE;gBACtE,uEAAuE;gBACvE,qCAAqC;gBACrC,UAAU,EAAE,MAAM,CAAC,WAAW,CAC5B,MAAM,CAAC,OAAO,CAAC,KAAK,CAAC,CAAC,MAAM,CAC1B,CAAC,CAAC,EAAE,CAAC,CAAC,EAAE,EAAE,CAAC,OAAO,CAAC,KAAK,QAAQ,IAAI,OAAO,CAAC,KAAK,QAAQ,IAAI,OAAO,CAAC,KAAK,SAAS,CACpF,CACF;gBACD,SAAS,EAAE,OAAO,KAAK,CAAC,SAAS,KAAK,QAAQ,CAAC,CAAC,CAAC,KAAK,CAAC,SAAS,CAAC,CAAC,CAAC,IAAI,CAAC,GAAG,EAAE;aAC9E,CAAC,CAAC;QACL,CAAC;IACH,CAAC;IAED,KAAK,CAAC,KAAK;QACT,MAAM,IAAI,CAAC,cAAc,CAAC,UAAU,EAAE,CAAC;IACzC,CAAC;IAED,KAAK,CAAC,QAAQ;QACZ,MAAM,IAAI,CAAC,cAAc,CAAC,QAAQ,EAAE,CAAC;IACvC,CAAC;CACF"}
+{"version":3,"file":"otlp-event-bridge.js","sourceRoot":"","sources":["../../../src/shared/transport/otlp-event-bridge.ts"],"names":[],"mappings":"AAAA,OAAO,EAAE,eAAe,EAAE,MAAM,wCAAwC,CAAC;AACzE,OAAO,EAAE,cAAc,EAAE,uBAAuB,EAAE,MAAM,yBAAyB,CAAC;AAClF,OAAO,EAAE,sBAAsB,EAAE,MAAM,0BAA0B,CAAC;AAGlE,OAAO,EAAE,YAAY,EAAE,MAAM,cAAc,CAAC;AAC5C,OAAO,EACL,oBAAoB,EACpB,iBAAiB,EACjB,mBAAmB,EACnB,cAAc,EACd,oBAAoB,GACrB,MAAM,kBAAkB,CAAC;AAE1B,MAAM,MAAM,GAAG,YAAY,CAAC,mBAAmB,CAAC,CAAC;AAiBjD,MAAM,OAAO,eAAe;IACT,cAAc,CAAiB;IAC/B,UAAU,CAA0C;IACpD,OAAO,CAAU;IACjB,QAAQ,CAAS;IAC1B,eAAe,GAAG,KAAK,CAAC;IAEhC,YAAY,OAA+B;QACzC,oBAAoB,CAAC,OAAO,CAAC,QAAQ,EAAE,iBAAiB,CAAC,CAAC;QAE1D,MAAM,UAAU,GAAG,oBAAoB,CAAC,OAAO,CAAC,UAAU,EAAE,mBAAmB,CAAC,CAAC;QACjF,MAAM,aAAa,GAAG,oBAAoB,CAAC,OAAO,CAAC,aAAa,EAAE,EAAE,CAAC,CAAC;QAEtE,IAAI,CAAC,QAAQ,GAAG,OAAO,CAAC,QAAQ,CAAC;QACjC,IAAI,CAAC,OAAO,GAAG,iBAAiB,CAAC,OAAO,CAAC,OAAO,IAAI,EAAE,CAAC,CAAC;QAExD,MAAM,QAAQ,GAAG,IAAI,eAAe,CAAC;YACnC,GAAG,EAAE,GAAG,OAAO,CAAC,QAAQ,UAAU;YAClC,OAAO,EAAE;gBACP,GAAG,CAAC,OAAO,CAAC,OAAO,IAAI,EAAE,CAAC;gBAC1B,YAAY,EAAE,cAAc,CAAC,UAAU,EAAE,aAAa,CAAC;aACxD;SACF,CAAC,CAAC;QAEH,IAAI,CAAC,cAAc,GAAG,IAAI,cAAc,CAAC;YACvC,QAAQ,EAAE,sBAAsB,CAAC,EAAE,cAAc,EAAE,OAAO,CAAC,OAAO,EAAE,CAAC;YACrE,UAAU,EAAE,CAAC,IAAI,uBAAuB,CAAC,QAAQ,CAAC,CAAC;SACpD,CAAC,CAAC;QAEH,IAAI,CAAC,UAAU,GAAG,IAAI,CAAC,cAAc,CAAC,SAAS,CAAC,UAAU,EAAE,aAAa,IAAI,SAAS,CAAC,CAAC;IAC1F,CAAC;IAED,UAAU,CAAC,MAAqB;QAC9B,IAAI,CAAC,IAAI,CAAC,OAAO,IAAI,CAAC,IAAI,CAAC,eAAe,EAAE,CAAC;YAC3C,IAAI,CAAC,eAAe,GAAG,IAAI,CAAC;YAC5B,MAAM,CAAC,IAAI,CAAC,2EAA2E,EAAE;gBACvF,QAAQ,EAAE,IAAI,CAAC,QAAQ;aACxB,CAAC,CAAC;QACL,CAAC;QACD,KAAK,MAAM,KAAK,IAAI,MAAM,EAAE,CAAC;YAC3B,IAAI,CAAC,UAAU,CAAC,IAAI,CAAC;gBACnB,YAAY,EAAE,MAAM;gBACpB,IAAI,EAAE,MAAM,CAAC,KAAK,CAAC,WAAW,CAAC,IAAI,SAAS,CAAC;gBAC7C,mEAAmE;gBACnE,sEAAsE;gBACtE,uEAAuE;gBACvE,8BAA8B;gBAC9B,UAAU,EAAE,MAAM,CAAC,WAAW,CAC5B,MAAM,CAAC,OAAO,CAAC,KAAK,CAAC,CAAC,MAAM,CAC1B,CAAC,CAAC,EAAE,CAAC,CAAC,EAAE,EAAE,CAAC,OAAO,CAAC,KAAK,QAAQ,IAAI,OAAO,CAAC,KAAK,QAAQ,IAAI,OAAO,CAAC,KAAK,SAAS,CACpF,CACF;gBACD,SAAS,EAAE,OAAO,KAAK,CAAC,SAAS,KAAK,QAAQ,CAAC,CAAC,CAAC,KAAK,CAAC,SAAS,CAAC,CAAC,CAAC,IAAI,CAAC,GAAG,EAAE;aAC9E,CAAC,CAAC;QACL,CAAC;IACH,CAAC;IAED,KAAK,CAAC,KAAK;QACT,MAAM,IAAI,CAAC,cAAc,CAAC,UAAU,EAAE,CAAC;IACzC,CAAC;IAED,KAAK,CAAC,QAAQ;QACZ,MAAM,IAAI,CAAC,cAAc,CAAC,QAAQ,EAAE,CAAC;IACvC,CAAC;CACF"}

package/dist/shared/transport/otlp-shared.d.ts

@@ -4,10 +4,21 @@
  * import this constant rather than repeating the literal.
  */
 export declare const DEFAULT_CLIENT_NAME = "ai-telemetry";
+/**
+ * Strip HTTP header-injection characters and surrounding whitespace from a
+ * client name or version string. Falls back to `fallback` when the input is
+ * absent or reduces to an empty string after sanitization.
+ */
+export declare function sanitizeClientString(s: string | undefined, fallback: string): string;
+/**
+ * Builds the `User-Agent` header value for outbound NR ingest requests.
+ * Returns `name/version` when version is non-empty, `name` alone otherwise.
+ */
+export declare function buildUserAgent(clientName: string | undefined, clientVersion: string | undefined): string;
 /**
  * Returns true when `headers` contains at least one recognised auth header
  * (`api-key`, `authorization`, or `x-license-key`). Shared by OtlpTransport
- * and OtlpEventBridge so both warn when no auth header is present (§TR4).
+ * and OtlpEventBridge so both warn when no auth header is present.
  */
 export declare function hasOtlpAuthHeader(headers: Record<string, string>): boolean;
 export declare function validateOtlpEndpoint(endpoint: string, source: string): void;

package/dist/shared/transport/otlp-shared.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"otlp-shared.d.ts","sourceRoot":"","sources":["../../../src/shared/transport/otlp-shared.ts"],"names":[],"mappings":"AAIA;;;;GAIG;AACH,eAAO,MAAM,mBAAmB,iBAAiB,CAAC;AAElD;;;;GAIG;AACH,wBAAgB,iBAAiB,CAAC,OAAO,EAAE,MAAM,CAAC,MAAM,EAAE,MAAM,CAAC,GAAG,OAAO,CAM1E;AAOD,wBAAgB,oBAAoB,CAAC,QAAQ,EAAE,MAAM,EAAE,MAAM,EAAE,MAAM,GAAG,IAAI,CAsB3E"}
+{"version":3,"file":"otlp-shared.d.ts","sourceRoot":"","sources":["../../../src/shared/transport/otlp-shared.ts"],"names":[],"mappings":"AAIA;;;;GAIG;AACH,eAAO,MAAM,mBAAmB,iBAAiB,CAAC;AAElD;;;;GAIG;AACH,wBAAgB,oBAAoB,CAAC,CAAC,EAAE,MAAM,GAAG,SAAS,EAAE,QAAQ,EAAE,MAAM,GAAG,MAAM,CAEpF;AAED;;;GAGG;AACH,wBAAgB,cAAc,CAC5B,UAAU,EAAE,MAAM,GAAG,SAAS,EAC9B,aAAa,EAAE,MAAM,GAAG,SAAS,GAChC,MAAM,CAGR;AAED;;;;GAIG;AACH,wBAAgB,iBAAiB,CAAC,OAAO,EAAE,MAAM,CAAC,MAAM,EAAE,MAAM,CAAC,GAAG,OAAO,CAM1E;AAOD,wBAAgB,oBAAoB,CAAC,QAAQ,EAAE,MAAM,EAAE,MAAM,EAAE,MAAM,GAAG,IAAI,CAsB3E"}

package/dist/shared/transport/otlp-shared.js

@@ -6,10 +6,26 @@ const logger = createLogger('otlp-shared');
  * import this constant rather than repeating the literal.
  */
 export const DEFAULT_CLIENT_NAME = 'ai-telemetry';
+/**
+ * Strip HTTP header-injection characters and surrounding whitespace from a
+ * client name or version string. Falls back to `fallback` when the input is
+ * absent or reduces to an empty string after sanitization.
+ */
+export function sanitizeClientString(s, fallback) {
+    return (s ?? fallback).replace(/[\r\n\x00-\x1f]/g, '').trim() || fallback;
+}
+/**
+ * Builds the `User-Agent` header value for outbound NR ingest requests.
+ * Returns `name/version` when version is non-empty, `name` alone otherwise.
+ */
+export function buildUserAgent(clientName, clientVersion) {
+    const name = clientName || DEFAULT_CLIENT_NAME;
+    return clientVersion ? `${name}/${clientVersion}` : name;
+}
 /**
  * Returns true when `headers` contains at least one recognised auth header
  * (`api-key`, `authorization`, or `x-license-key`). Shared by OtlpTransport
- * and OtlpEventBridge so both warn when no auth header is present (§TR4).
+ * and OtlpEventBridge so both warn when no auth header is present.
  */
 export function hasOtlpAuthHeader(headers) {
     for (const key of Object.keys(headers)) {
@@ -19,11 +35,11 @@ export function hasOtlpAuthHeader(headers) {
     }
     return false;
 }
-// validate OTLP endpoint scheme. https:// is required for
+// Validate OTLP endpoint scheme. https:// is required for
 // any non-localhost destination because the payload may contain user prompt
 // fragments (PII). Plain http:// is allowed only against loopback to support
 // local development and testing. Shared by OtlpTransport and OtlpEventBridge
-// (§OT2 — deduplicates previously identical implementations).
+// (deduplicates previously identical implementations).
 export function validateOtlpEndpoint(endpoint, source) {
     let parsed;
     try {
@@ -40,7 +56,7 @@ export function validateOtlpEndpoint(endpoint, source) {
     // http://: only acceptable on loopback
     const host = parsed.hostname;
     // 0.0.0.0 is a wildcard that binds to ALL interfaces, not loopback only —
-    // cleartext traffic to it is reachable from any network (§TR5).
+    // cleartext traffic to it is reachable from any network.
     const isLoopback = host === 'localhost' || host === '127.0.0.1' || host === '::1';
     if (!isLoopback) {
         logger.warn(`${source}: OTLP endpoint uses plain http:// to a non-loopback host — payload may contain PII and should not be transmitted in cleartext`, { endpoint });

package/dist/shared/transport/otlp-shared.js.map

@@ -1 +1 @@
-{"version":3,"file":"otlp-shared.js","sourceRoot":"","sources":["../../../src/shared/transport/otlp-shared.ts"],"names":[],"mappings":"AAAA,OAAO,EAAE,YAAY,EAAE,MAAM,cAAc,CAAC;AAE5C,MAAM,MAAM,GAAG,YAAY,CAAC,aAAa,CAAC,CAAC;AAE3C;;;;GAIG;AACH,MAAM,CAAC,MAAM,mBAAmB,GAAG,cAAc,CAAC;AAElD;;;;GAIG;AACH,MAAM,UAAU,iBAAiB,CAAC,OAA+B;IAC/D,KAAK,MAAM,GAAG,IAAI,MAAM,CAAC,IAAI,CAAC,OAAO,CAAC,EAAE,CAAC;QACvC,MAAM,EAAE,GAAG,GAAG,CAAC,WAAW,EAAE,CAAC;QAC7B,IAAI,EAAE,KAAK,SAAS,IAAI,EAAE,KAAK,eAAe,IAAI,EAAE,KAAK,eAAe;YAAE,OAAO,IAAI,CAAC;IACxF,CAAC;IACD,OAAO,KAAK,CAAC;AACf,CAAC;AAED,0DAA0D;AAC1D,4EAA4E;AAC5E,6EAA6E;AAC7E,6EAA6E;AAC7E,8DAA8D;AAC9D,MAAM,UAAU,oBAAoB,CAAC,QAAgB,EAAE,MAAc;IACnE,IAAI,MAAW,CAAC;IAChB,IAAI,CAAC;QACH,MAAM,GAAG,IAAI,GAAG,CAAC,QAAQ,CAAC,CAAC;IAC7B,CAAC;IAAC,MAAM,CAAC;QACP,MAAM,IAAI,KAAK,CAAC,GAAG,MAAM,gCAAgC,QAAQ,EAAE,CAAC,CAAC;IACvE,CAAC;IACD,IAAI,MAAM,CAAC,QAAQ,KAAK,QAAQ;QAAE,OAAO;IACzC,IAAI,MAAM,CAAC,QAAQ,KAAK,OAAO,EAAE,CAAC;QAChC,MAAM,IAAI,KAAK,CAAC,GAAG,MAAM,yCAAyC,MAAM,CAAC,QAAQ,EAAE,CAAC,CAAC;IACvF,CAAC;IACD,uCAAuC;IACvC,MAAM,IAAI,GAAG,MAAM,CAAC,QAAQ,CAAC;IAC7B,0EAA0E;IAC1E,gEAAgE;IAChE,MAAM,UAAU,GAAG,IAAI,KAAK,WAAW,IAAI,IAAI,KAAK,WAAW,IAAI,IAAI,KAAK,KAAK,CAAC;IAClF,IAAI,CAAC,UAAU,EAAE,CAAC;QAChB,MAAM,CAAC,IAAI,CACT,GAAG,MAAM,gIAAgI,EACzI,EAAE,QAAQ,EAAE,CACb,CAAC;IACJ,CAAC;AACH,CAAC"}
+{"version":3,"file":"otlp-shared.js","sourceRoot":"","sources":["../../../src/shared/transport/otlp-shared.ts"],"names":[],"mappings":"AAAA,OAAO,EAAE,YAAY,EAAE,MAAM,cAAc,CAAC;AAE5C,MAAM,MAAM,GAAG,YAAY,CAAC,aAAa,CAAC,CAAC;AAE3C;;;;GAIG;AACH,MAAM,CAAC,MAAM,mBAAmB,GAAG,cAAc,CAAC;AAElD;;;;GAIG;AACH,MAAM,UAAU,oBAAoB,CAAC,CAAqB,EAAE,QAAgB;IAC1E,OAAO,CAAC,CAAC,IAAI,QAAQ,CAAC,CAAC,OAAO,CAAC,kBAAkB,EAAE,EAAE,CAAC,CAAC,IAAI,EAAE,IAAI,QAAQ,CAAC;AAC5E,CAAC;AAED;;;GAGG;AACH,MAAM,UAAU,cAAc,CAC5B,UAA8B,EAC9B,aAAiC;IAEjC,MAAM,IAAI,GAAG,UAAU,IAAI,mBAAmB,CAAC;IAC/C,OAAO,aAAa,CAAC,CAAC,CAAC,GAAG,IAAI,IAAI,aAAa,EAAE,CAAC,CAAC,CAAC,IAAI,CAAC;AAC3D,CAAC;AAED;;;;GAIG;AACH,MAAM,UAAU,iBAAiB,CAAC,OAA+B;IAC/D,KAAK,MAAM,GAAG,IAAI,MAAM,CAAC,IAAI,CAAC,OAAO,CAAC,EAAE,CAAC;QACvC,MAAM,EAAE,GAAG,GAAG,CAAC,WAAW,EAAE,CAAC;QAC7B,IAAI,EAAE,KAAK,SAAS,IAAI,EAAE,KAAK,eAAe,IAAI,EAAE,KAAK,eAAe;YAAE,OAAO,IAAI,CAAC;IACxF,CAAC;IACD,OAAO,KAAK,CAAC;AACf,CAAC;AAED,0DAA0D;AAC1D,4EAA4E;AAC5E,6EAA6E;AAC7E,6EAA6E;AAC7E,uDAAuD;AACvD,MAAM,UAAU,oBAAoB,CAAC,QAAgB,EAAE,MAAc;IACnE,IAAI,MAAW,CAAC;IAChB,IAAI,CAAC;QACH,MAAM,GAAG,IAAI,GAAG,CAAC,QAAQ,CAAC,CAAC;IAC7B,CAAC;IAAC,MAAM,CAAC;QACP,MAAM,IAAI,KAAK,CAAC,GAAG,MAAM,gCAAgC,QAAQ,EAAE,CAAC,CAAC;IACvE,CAAC;IACD,IAAI,MAAM,CAAC,QAAQ,KAAK,QAAQ;QAAE,OAAO;IACzC,IAAI,MAAM,CAAC,QAAQ,KAAK,OAAO,EAAE,CAAC;QAChC,MAAM,IAAI,KAAK,CAAC,GAAG,MAAM,yCAAyC,MAAM,CAAC,QAAQ,EAAE,CAAC,CAAC;IACvF,CAAC;IACD,uCAAuC;IACvC,MAAM,IAAI,GAAG,MAAM,CAAC,QAAQ,CAAC;IAC7B,0EAA0E;IAC1E,yDAAyD;IACzD,MAAM,UAAU,GAAG,IAAI,KAAK,WAAW,IAAI,IAAI,KAAK,WAAW,IAAI,IAAI,KAAK,KAAK,CAAC;IAClF,IAAI,CAAC,UAAU,EAAE,CAAC;QAChB,MAAM,CAAC,IAAI,CACT,GAAG,MAAM,gIAAgI,EACzI,EAAE,QAAQ,EAAE,CACb,CAAC;IACJ,CAAC;AACH,CAAC"}

package/dist/shared/transport/otlp-transport.d.ts

@@ -1,17 +1,19 @@
 import type { NrMetric } from './types.js';
 export interface OtlpTransportOptions {
-    endpoint: string;
-    headers?: Record<string, string>;
-    appName: string;
-    /** Override the default 30-second request timeout for exportMetrics (§TR7). */
-    requestTimeoutMs?: number;
+    readonly endpoint: string;
+    readonly headers?: Record<string, string>;
+    readonly appName: string;
+    /** Override the default 30-second request timeout for exportMetrics. */
+    readonly requestTimeoutMs?: number;
     /**
      * Identifies the consuming client in the `User-Agent` header and as the
      * OTel instrumentation scope name. Defaults to
-     * `'ai-telemetry'` when not provided. Pass `'preflight'`, `'my-agent'`,
+     * `'ai-telemetry'` when not provided. Pass `'preflight'`, `'nr-ai-agent'`,
      * etc. so telemetry from different consumers is distinguishable.
      */
-    clientName?: string;
+    readonly clientName?: string;
+    /** Version of the consuming client, stamped as the OTel instrumentation scope version. */
+    readonly clientVersion?: string;
 }
 export declare class OtlpTransport {
     private readonly traceExporter;
@@ -21,24 +23,21 @@ export declare class OtlpTransport {
     private readonly endpoint;
     private readonly headers;
     private readonly requestTimeoutMs;
-    private readonly clientName;
+    private readonly clientVersion;
+    private readonly userAgent;
+    private readonly hasAuth;
     private hasWarnedNoAuth;
-    /**
-     * Resource attributes shared by both the SDK-driven path (tracer/meter
-     * providers) and the manual `exportMetrics` payload.
-     * Keeping a single resolved attribute map prevents `service.name` from
-     * drifting between the two paths — previously `this.appName` was stored
-     * separately and re-encoded into the OTLP envelope, which would silently
-     * diverge if anyone added a second resource attribute.
-     */
-    private readonly resourceAttributes;
+    private readonly otlpResource;
+    private readonly otlpScope;
+    private readonly metricsHeaders;
     constructor(options: OtlpTransportOptions);
     flush(): Promise<void>;
     shutdown(): Promise<void>;
+    private settledOrThrow;
     /**
      * Return an OTel `Tracer` for the given instrumentation name. The returned
-     * value is the `Tracer` interface from `@opentelemetry/api`
-     *; consumers that bind to the type explicitly should
+     * value is the `Tracer` interface from `@opentelemetry/api`;
+     * consumers that bind to the type explicitly should
      * `import type { Tracer } from '@opentelemetry/api'`. `@opentelemetry/api`
      * is already a regular dependency of this package, so no extra install is
      * required. The type is intentionally NOT re-exported from this package's
@@ -54,5 +53,9 @@ export declare class OtlpTransport {
      */
     getMeter(name: string): import("@opentelemetry/api").Meter;
     exportMetrics(metrics: NrMetric[]): Promise<void>;
+    private otlpAttributes;
+    private numericDataPoint;
+    private summaryDataPoint;
+    private otlpMetric;
 }
 //# sourceMappingURL=otlp-transport.d.ts.map

package/dist/shared/transport/otlp-transport.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"otlp-transport.d.ts","sourceRoot":"","sources":["../../../src/shared/transport/otlp-transport.ts"],"names":[],"mappings":"AAQA,OAAO,KAAK,EAAE,QAAQ,EAAiD,MAAM,YAAY,CAAC;AAQ1F,MAAM,WAAW,oBAAoB;IACnC,QAAQ,EAAE,MAAM,CAAC;IACjB,OAAO,CAAC,EAAE,MAAM,CAAC,MAAM,EAAE,MAAM,CAAC,CAAC;IACjC,OAAO,EAAE,MAAM,CAAC;IAChB,+EAA+E;IAC/E,gBAAgB,CAAC,EAAE,MAAM,CAAC;IAC1B;;;;;OAKG;IACH,UAAU,CAAC,EAAE,MAAM,CAAC;CACrB;AAED,qBAAa,aAAa;IACxB,OAAO,CAAC,QAAQ,CAAC,aAAa,CAAoB;IAClD,OAAO,CAAC,QAAQ,CAAC,cAAc,CAAqB;IACpD,OAAO,CAAC,QAAQ,CAAC,cAAc,CAAsB;IACrD,OAAO,CAAC,QAAQ,CAAC,aAAa,CAAgB;IAC9C,OAAO,CAAC,QAAQ,CAAC,QAAQ,CAAS;IAClC,OAAO,CAAC,QAAQ,CAAC,OAAO,CAAyB;IACjD,OAAO,CAAC,QAAQ,CAAC,gBAAgB,CAAS;IAC1C,OAAO,CAAC,QAAQ,CAAC,UAAU,CAAS;IACpC,OAAO,CAAC,eAAe,CAAS;IAChC;;;;;;;OAOG;IACH,OAAO,CAAC,QAAQ,CAAC,kBAAkB,CAAmC;gBAE1D,OAAO,EAAE,oBAAoB;IAqCnC,KAAK,IAAI,OAAO,CAAC,IAAI,CAAC;IAKtB,QAAQ,IAAI,OAAO,CAAC,IAAI,CAAC;IAK/B;;;;;;;;;;OAUG;IACH,SAAS,CAAC,IAAI,EAAE,MAAM;IAItB;;;;OAIG;IACH,QAAQ,CAAC,IAAI,EAAE,MAAM;IAIf,aAAa,CAAC,OAAO,EAAE,QAAQ,EAAE,GAAG,OAAO,CAAC,IAAI,CAAC;CAwJxD"}
+{"version":3,"file":"otlp-transport.d.ts","sourceRoot":"","sources":["../../../src/shared/transport/otlp-transport.ts"],"names":[],"mappings":"AASA,OAAO,KAAK,EAAE,QAAQ,EAAiD,MAAM,YAAY,CAAC;AAa1F,MAAM,WAAW,oBAAoB;IACnC,QAAQ,CAAC,QAAQ,EAAE,MAAM,CAAC;IAC1B,QAAQ,CAAC,OAAO,CAAC,EAAE,MAAM,CAAC,MAAM,EAAE,MAAM,CAAC,CAAC;IAC1C,QAAQ,CAAC,OAAO,EAAE,MAAM,CAAC;IACzB,wEAAwE;IACxE,QAAQ,CAAC,gBAAgB,CAAC,EAAE,MAAM,CAAC;IACnC;;;;;OAKG;IACH,QAAQ,CAAC,UAAU,CAAC,EAAE,MAAM,CAAC;IAC7B,0FAA0F;IAC1F,QAAQ,CAAC,aAAa,CAAC,EAAE,MAAM,CAAC;CACjC;AAED,qBAAa,aAAa;IACxB,OAAO,CAAC,QAAQ,CAAC,aAAa,CAAoB;IAClD,OAAO,CAAC,QAAQ,CAAC,cAAc,CAAqB;IACpD,OAAO,CAAC,QAAQ,CAAC,cAAc,CAAsB;IACrD,OAAO,CAAC,QAAQ,CAAC,aAAa,CAAgB;IAC9C,OAAO,CAAC,QAAQ,CAAC,QAAQ,CAAS;IAClC,OAAO,CAAC,QAAQ,CAAC,OAAO,CAAyB;IACjD,OAAO,CAAC,QAAQ,CAAC,gBAAgB,CAAS;IAC1C,OAAO,CAAC,QAAQ,CAAC,aAAa,CAAS;IACvC,OAAO,CAAC,QAAQ,CAAC,SAAS,CAAS;IACnC,OAAO,CAAC,QAAQ,CAAC,OAAO,CAAU;IAClC,OAAO,CAAC,eAAe,CAAS;IAChC,OAAO,CAAC,QAAQ,CAAC,YAAY,CAAiE;IAC9F,OAAO,CAAC,QAAQ,CAAC,SAAS,CAAuD;IACjF,OAAO,CAAC,QAAQ,CAAC,cAAc,CAAmC;gBAEtD,OAAO,EAAE,oBAAoB;IAuDnC,KAAK,IAAI,OAAO,CAAC,IAAI,CAAC;IAOtB,QAAQ,IAAI,OAAO,CAAC,IAAI,CAAC;YAOjB,cAAc;IAS5B;;;;;;;;;;OAUG;IACH,SAAS,CAAC,IAAI,EAAE,MAAM;IAItB;;;;OAIG;IACH,QAAQ,CAAC,IAAI,EAAE,MAAM;IAIf,aAAa,CAAC,OAAO,EAAE,QAAQ,EAAE,GAAG,OAAO,CAAC,IAAI,CAAC;IA8DvD,OAAO,CAAC,cAAc;IAYtB,OAAO,CAAC,gBAAgB;IAsBxB,OAAO,CAAC,gBAAgB;IAyBxB,OAAO,CAAC,UAAU;CA0BnB"}

package/dist/shared/transport/otlp-transport.js

@@ -6,8 +6,7 @@ import { BasicTracerProvider, BatchSpanProcessor } from '@opentelemetry/sdk-trac
 import { MeterProvider, PeriodicExportingMetricReader } from '@opentelemetry/sdk-metrics';
 import { resourceFromAttributes } from '@opentelemetry/resources';
 import { createLogger } from '../logger.js';
-import { validateOtlpEndpoint, hasOtlpAuthHeader, DEFAULT_CLIENT_NAME } from './otlp-shared.js';
-import { VERSION } from '../version.js';
+import { validateOtlpEndpoint, hasOtlpAuthHeader, DEFAULT_CLIENT_NAME, buildUserAgent, sanitizeClientString, } from './otlp-shared.js';
 const gzipAsync = promisify(gzip);
 const logger = createLogger('otlp-transport');
 export class OtlpTransport {
@@ -18,32 +17,45 @@ export class OtlpTransport {
     endpoint;
     headers;
     requestTimeoutMs;
-    clientName;
+    clientVersion;
+    userAgent;
+    hasAuth;
     hasWarnedNoAuth = false;
-    /**
-     * Resource attributes shared by both the SDK-driven path (tracer/meter
-     * providers) and the manual `exportMetrics` payload.
-     * Keeping a single resolved attribute map prevents `service.name` from
-     * drifting between the two paths — previously `this.appName` was stored
-     * separately and re-encoded into the OTLP envelope, which would silently
-     * diverge if anyone added a second resource attribute.
-     */
-    resourceAttributes;
+    otlpResource;
+    otlpScope;
+    metricsHeaders;
     constructor(options) {
         validateOtlpEndpoint(options.endpoint, 'OtlpTransport');
-        this.clientName = options.clientName || DEFAULT_CLIENT_NAME;
-        this.resourceAttributes = Object.freeze({ 'service.name': options.appName });
-        const resource = resourceFromAttributes({ ...this.resourceAttributes });
+        const clientName = sanitizeClientString(options.clientName, DEFAULT_CLIENT_NAME);
+        this.clientVersion = sanitizeClientString(options.clientVersion, '');
+        this.userAgent = buildUserAgent(clientName, this.clientVersion);
+        const resourceAttributes = Object.freeze({ 'service.name': options.appName });
+        this.otlpResource = Object.entries(resourceAttributes).map(([key, value]) => ({
+            key,
+            value: { stringValue: value },
+        }));
+        this.otlpScope = this.clientVersion
+            ? { name: clientName, version: this.clientVersion }
+            : { name: clientName };
+        const resource = resourceFromAttributes({ ...resourceAttributes });
         this.endpoint = options.endpoint;
-        this.headers = options.headers ?? {};
+        this.headers = { ...(options.headers ?? {}) };
+        this.hasAuth = hasOtlpAuthHeader(this.headers);
         this.requestTimeoutMs = options.requestTimeoutMs ?? 30_000;
+        this.metricsHeaders = Object.freeze({
+            ...this.headers,
+            'Content-Type': 'application/json',
+            'Content-Encoding': 'gzip',
+            'User-Agent': this.userAgent,
+        });
+        const sdkHeaders = { ...this.headers, 'User-Agent': this.userAgent };
         this.traceExporter = new OTLPTraceExporter({
             url: `${options.endpoint}/v1/traces`,
-            headers: options.headers ?? {},
+            headers: sdkHeaders,
         });
         this.metricExporter = new OTLPMetricExporter({
             url: `${options.endpoint}/v1/metrics`,
-            headers: options.headers ?? {},
+            headers: sdkHeaders,
         });
         this.tracerProvider = new BasicTracerProvider({
             resource,
@@ -60,17 +72,25 @@ export class OtlpTransport {
         });
     }
     async flush() {
-        await this.tracerProvider.forceFlush();
-        await this.meterProvider.forceFlush();
+        await this.settledOrThrow([this.tracerProvider.forceFlush(), this.meterProvider.forceFlush()], 'OTLP flush failed on multiple providers');
     }
     async shutdown() {
-        await this.tracerProvider.shutdown();
-        await this.meterProvider.shutdown();
+        await this.settledOrThrow([this.tracerProvider.shutdown(), this.meterProvider.shutdown()], 'OTLP shutdown failed on multiple providers');
+    }
+    async settledOrThrow(ops, message) {
+        const results = await Promise.allSettled(ops);
+        const errors = results
+            .filter((r) => r.status === 'rejected')
+            .map((r) => (r.reason instanceof Error ? r.reason : new Error(String(r.reason))));
+        if (errors.length === 1)
+            throw errors[0];
+        if (errors.length > 1)
+            throw new AggregateError(errors, message);
     }
     /**
      * Return an OTel `Tracer` for the given instrumentation name. The returned
-     * value is the `Tracer` interface from `@opentelemetry/api`
-     *; consumers that bind to the type explicitly should
+     * value is the `Tracer` interface from `@opentelemetry/api`;
+     * consumers that bind to the type explicitly should
      * `import type { Tracer } from '@opentelemetry/api'`. `@opentelemetry/api`
      * is already a regular dependency of this package, so no extra install is
      * required. The type is intentionally NOT re-exported from this package's
@@ -79,7 +99,7 @@ export class OtlpTransport {
      * unrelated imports.
      */
     getTracer(name) {
-        return this.tracerProvider.getTracer(name);
+        return this.tracerProvider.getTracer(name, this.clientVersion || undefined);
     }
     /**
      * Return an OTel `Meter` for the given instrumentation name. See
@@ -87,106 +107,27 @@ export class OtlpTransport {
      * (`import type { Meter } from '@opentelemetry/api'`).
      */
     getMeter(name) {
-        return this.meterProvider.getMeter(name);
+        return this.meterProvider.getMeter(name, this.clientVersion || undefined);
     }
     async exportMetrics(metrics) {
         if (metrics.length === 0)
             return;
         // Warn ONCE when no auth header is present — emitting on every call would
-        // flood stderr with identical lines in long-running agents (§TR1).
-        if (!hasOtlpAuthHeader(this.headers) && !this.hasWarnedNoAuth) {
+        // flood stderr with identical lines in long-running agents.
+        if (!this.hasAuth && !this.hasWarnedNoAuth) {
             this.hasWarnedNoAuth = true;
             logger.warn('OTLP metric export attempted with no auth header — collector may reject', {
                 endpoint: this.endpoint,
             });
         }
-        const otlpAttributes = (attrs) => Object.entries(attrs ?? {}).map(([key, value]) => ({
-            key,
-            value: typeof value === 'number'
-                ? { doubleValue: value }
-                : typeof value === 'boolean'
-                    ? { boolValue: value }
-                    : { stringValue: String(value) },
-        }));
-        const numericDataPoint = (m) => ({
-            // For count (delta Sum) metrics, include startTimeUnixNano per the OTLP
-            // spec (§TR2). Gauge data points do not require it but it is harmless.
-            // Clamp to 0: if timestamp < intervalMs (misconfigured metric), a negative
-            // startTimeUnixNano would be rejected by strict OTLP collectors (§TR7).
-            startTimeUnixNano: m.type === 'count' ? Math.max(0, m.timestamp - m.intervalMs) * 1_000_000 : undefined,
-            timeUnixNano: m.timestamp * 1_000_000,
-            asDouble: m.value,
-            attributes: otlpAttributes(m.attributes),
-        });
-        // summary is now a first-class type with a structured
-        // value `{ count, sum, min, max }`. OTLP doesn't have a single
-        // "Summary"-shaped metric kind; the closest faithful mapping is OTLP
-        // Histogram with explicit `count` and `sum` fields plus per-data-point
-        // `min` / `max`. Bucket boundaries are intentionally omitted: NR doesn't
-        // need them for summary aggregation, and emitting empty `bucketCounts`
-        // alongside `min`/`max` is the documented OTLP shape for unbucketed
-        // summaries (`explicitBounds: []`, `bucketCounts: [<count>]`).
-        const summaryDataPoint = (m) => ({
-            // OTLP Histogram with DELTA temporality requires startTimeUnixNano (§TR2).
-            startTimeUnixNano: Math.max(0, m.timestamp - m.intervalMs) * 1_000_000,
-            timeUnixNano: m.timestamp * 1_000_000,
-            attributes: otlpAttributes(m.attributes),
-            count: m.value.count,
-            sum: m.value.sum,
-            min: m.value.min,
-            max: m.value.max,
-            bucketCounts: [m.value.count],
-            explicitBounds: [],
-        });
-        // Map NrMetric.type → OTLP metric kind:
-        //   - `gauge`   → OTLP Gauge (point-in-time numeric value)
-        //   - `count`   → OTLP Sum (monotonic, DELTA aggregation temporality = 1).
-        //     NrCountMetric carries intervalMs — it represents a bounded-interval
-        //     delta, not a cumulative running total. Using CUMULATIVE (2) would
-        //     cause downstream collectors to treat each harvest as a monotonically-
-        //     increasing total, producing incorrect rate calculations (§TR1).
-        //   - `summary` → OTLP Histogram (with explicit min/max/sum/count fields,
-        //     no buckets) — see §4.9 note above on why histogram is the closest
-        //     faithful mapping.
-        const otlpMetric = (m) => {
-            if (m.type === 'count') {
-                return {
-                    name: m.name,
-                    sum: {
-                        dataPoints: [numericDataPoint(m)],
-                        // 1 = DELTA — the value represents the count within intervalMs,
-                        // not a cumulative total from a fixed epoch.
-                        aggregationTemporality: 1,
-                        isMonotonic: true,
-                    },
-                };
-            }
-            if (m.type === 'summary') {
-                return {
-                    name: m.name,
-                    histogram: {
-                        dataPoints: [summaryDataPoint(m)],
-                        // Aggregation temporality 1 = DELTA (the count/sum/min/max
-                        // describe the harvest interval, not a cumulative total).
-                        aggregationTemporality: 1,
-                    },
-                };
-            }
-            return { name: m.name, gauge: { dataPoints: [numericDataPoint(m)] } };
-        };
         const payload = {
             resourceMetrics: [
                 {
-                    resource: {
-                        attributes: Object.entries(this.resourceAttributes).map(([key, value]) => ({
-                            key,
-                            value: { stringValue: value },
-                        })),
-                    },
+                    resource: { attributes: this.otlpResource },
                     scopeMetrics: [
                         {
-                            scope: { name: this.clientName },
-                            metrics: metrics.map(otlpMetric),
+                            scope: this.otlpScope,
+                            metrics: metrics.map((m) => this.otlpMetric(m)),
                         },
                     ],
                 },
@@ -203,34 +144,109 @@ export class OtlpTransport {
         // level retry queue is rarely engaged for events. For metrics, we
         // intentionally rely on the scheduler's retry queue instead — there
         // is no PeriodicExportingMetricReader in this code path.
-        // Gzip-compress the payload to match sendWithRetry (§TR3). The NR OTLP
+        // Gzip-compress the payload to match sendWithRetry. The NR OTLP
         // endpoint accepts gzip; for large metric batches this is a 5-10× size win.
         const compressed = await gzipAsync(JSON.stringify(payload));
         const response = await fetch(`${this.endpoint}/v1/metrics`, {
             method: 'POST',
-            headers: {
-                'Content-Type': 'application/json',
-                'Content-Encoding': 'gzip',
-                'User-Agent': `${this.clientName}/${VERSION}`,
-                ...this.headers,
-            },
+            headers: this.metricsHeaders,
             body: compressed,
             signal: AbortSignal.timeout(this.requestTimeoutMs),
-            // §5.20 — see http-client.ts for rationale.
+            // see http-client.ts for rationale.
             keepalive: true,
         });
         if (!response.ok) {
             const body = await response.text().catch(() => '');
             const msg = `OTLP metric export failed: HTTP ${response.status}${body ? ` — ${body.slice(0, 256)}` : ''}`;
             // 400 means the payload itself is malformed — retrying the same payload
-            // will always fail, so surface this as a distinct non-retryable error (§TR9).
+            // will always fail, so surface this as a distinct non-retryable error.
             if (response.status === 400) {
                 throw Object.assign(new Error(msg), { code: 'OTLP_BAD_REQUEST' });
             }
             throw new Error(msg);
         }
-        // Drain on success so undici returns the socket to the keep-alive pool (§HC1).
+        // Drain on success so undici returns the socket to the keep-alive pool.
         await response.body?.cancel().catch(() => { });
     }
+    otlpAttributes(attrs) {
+        return Object.entries(attrs ?? {}).map(([key, value]) => ({
+            key,
+            value: typeof value === 'number'
+                ? { doubleValue: value }
+                : typeof value === 'boolean'
+                    ? { boolValue: value }
+                    : { stringValue: String(value) },
+        }));
+    }
+    numericDataPoint(m) {
+        return {
+            // For count (delta Sum) metrics, include startTimeUnixNano per the OTLP
+            // spec. Gauge data points do not require it but it is harmless.
+            // Clamp to 0: if timestamp < intervalMs (misconfigured metric), a negative
+            // startTimeUnixNano would be rejected by strict OTLP collectors.
+            startTimeUnixNano: m.type === 'count' ? Math.max(0, m.timestamp - m.intervalMs) * 1_000_000 : undefined,
+            timeUnixNano: m.timestamp * 1_000_000,
+            asDouble: m.value,
+            attributes: this.otlpAttributes(m.attributes),
+        };
+    }
+    // summary is now a first-class type with a structured
+    // value `{ count, sum, min, max }`. OTLP doesn't have a single
+    // "Summary"-shaped metric kind; the closest faithful mapping is OTLP
+    // Histogram with explicit `count` and `sum` fields plus per-data-point
+    // `min` / `max`. Bucket boundaries are intentionally omitted: NR doesn't
+    // need them for summary aggregation, and emitting empty `bucketCounts`
+    // alongside `min`/`max` is the documented OTLP shape for unbucketed
+    // summaries (`explicitBounds: []`, `bucketCounts: [<count>]`).
+    summaryDataPoint(m) {
+        return {
+            // OTLP Histogram with DELTA temporality requires startTimeUnixNano.
+            startTimeUnixNano: Math.max(0, m.timestamp - m.intervalMs) * 1_000_000,
+            timeUnixNano: m.timestamp * 1_000_000,
+            attributes: this.otlpAttributes(m.attributes),
+            count: m.value.count,
+            sum: m.value.sum,
+            min: m.value.min,
+            max: m.value.max,
+            bucketCounts: [m.value.count],
+            explicitBounds: [],
+        };
+    }
+    // Map NrMetric.type → OTLP metric kind:
+    //   - `gauge`   → OTLP Gauge (point-in-time numeric value)
+    //   - `count`   → OTLP Sum (monotonic, DELTA aggregation temporality = 1).
+    //     NrCountMetric carries intervalMs — it represents a bounded-interval
+    //     delta, not a cumulative running total. Using CUMULATIVE (2) would
+    //     cause downstream collectors to treat each harvest as a monotonically-
+    //     increasing total, producing incorrect rate calculations.
+    //   - `summary` → OTLP Histogram (with explicit min/max/sum/count fields,
+    //     no buckets) — see note above on why histogram is the closest
+    //     faithful mapping.
+    otlpMetric(m) {
+        if (m.type === 'count') {
+            return {
+                name: m.name,
+                sum: {
+                    dataPoints: [this.numericDataPoint(m)],
+                    // 1 = DELTA — the value represents the count within intervalMs,
+                    // not a cumulative total from a fixed epoch.
+                    aggregationTemporality: 1,
+                    isMonotonic: true,
+                },
+            };
+        }
+        if (m.type === 'summary') {
+            return {
+                name: m.name,
+                histogram: {
+                    dataPoints: [this.summaryDataPoint(m)],
+                    // Aggregation temporality 1 = DELTA (the count/sum/min/max
+                    // describe the harvest interval, not a cumulative total).
+                    aggregationTemporality: 1,
+                },
+            };
+        }
+        return { name: m.name, gauge: { dataPoints: [this.numericDataPoint(m)] } };
+    }
 }
 //# sourceMappingURL=otlp-transport.js.map