newrelic 12.22.0 → 12.23.0

package/NEWS.md

@@ -1,3 +1,24 @@
+### v12.23.0 (2025-06-30)
+
+#### Features
+
+* Added ability to report only on entry and exit spans ([#3184](https://github.com/newrelic/node-newrelic/pull/3184)) ([1f909d3](https://github.com/newrelic/node-newrelic/commit/1f909d389f790733c8787a9db8b0ee71c26bb5ed))
+* Added Node.js 24 support ([#3080](https://github.com/newrelic/node-newrelic/pull/3080)) ([a538c2a](https://github.com/newrelic/node-newrelic/commit/a538c2a5e23b96be40fa3c014e60b912f695423e))
+
+#### Documentation
+
+* Updated compatibility report ([#3186](https://github.com/newrelic/node-newrelic/pull/3186)) ([5498c15](https://github.com/newrelic/node-newrelic/commit/5498c15bb6cf3329fe50595da46ab4f2b1385170))
+
+#### Miscellaneous chores
+
+* Fix issues on Node 24 ([#3181](https://github.com/newrelic/node-newrelic/pull/3181)) ([24832df](https://github.com/newrelic/node-newrelic/commit/24832df2984df960a4e44375525a8127a00a637c))
+* Removed force labels on main CI workflow ([#3183](https://github.com/newrelic/node-newrelic/pull/3183)) ([3aaee17](https://github.com/newrelic/node-newrelic/commit/3aaee17586f817ae1a2ab0573570a740bd6397ef))
+* Updated undici tests to unblock CI ([#3185](https://github.com/newrelic/node-newrelic/pull/3185)) ([08261eb](https://github.com/newrelic/node-newrelic/commit/08261ebcdbafdb625a451f4ccec9b0cba7641cc2))
+
+#### Continuous integration
+
+* Fixed the name of todo colum in add to board reusable workflow ([#3178](https://github.com/newrelic/node-newrelic/pull/3178)) ([f216e0c](https://github.com/newrelic/node-newrelic/commit/f216e0c48adf0291bfa38b3edbb04c9dcc96f92c))
+
 ### v12.22.0 (2025-06-24)
 
 #### Features

package/index.js

@@ -5,7 +5,7 @@
 
 'use strict'
 
-const HealthReporter = require('./lib/health-reporter')
+const HealthReporter = require('./lib/health-reporter.js')
 
 // Record opening times before loading any other files.
 const preAgentTime = process.uptime()
@@ -64,8 +64,8 @@ function initialize() {
       throw new Error(message)
     }
 
-    // TODO: Update this check when Node v24 support is added
-    if (psemver.satisfies('>=23.0.0')) {
+    // TODO: Update this check when Node v26 support is added
+    if (psemver.satisfies('>=25.0.0')) {
       logger.warn(
         'New Relic for Node.js %s has not been tested on Node.js %s. Please ' +
           'update the agent or downgrade your version of Node.js',

package/lib/config/default.js

@@ -1120,6 +1120,19 @@ defaultConfig.definition = () => ({
       default: false
     },
 
+    /**
+     * Controls whether the agent will generate spans for in-process actions.
+     * When disabled, this will only create spans for entry and exit spans.
+     * entry spans - initial actions for web servers and messsage queue consumption.
+     * exit spans - all outgoing calls to external services(external and database calls)
+     */
+    in_process_spans: {
+      enabled: {
+        formatter: boolean,
+        default: true
+      }
+    },
+
     sampler: {
       /**
        * When set to `always_on`, the sampled flag in the `traceparent` header

package/lib/spans/helpers.js

@@ -60,10 +60,72 @@ function addSpanKind({ segment, span }) {
   }
 }
 
+/**
+ * Checks if the segment is an entry point span.
+ * An entry point span is defined as the base segment of a transaction.
+ * @param {object} params to function
+ * @param {Transaction} params.transaction active transaction
+ * @param {TraceSegment} params.segment segment that is creating span
+ * @returns {boolean} true if the segment is an entry point span
+ */
+function isEntryPointSpan({ transaction, segment }) {
+  return transaction?.baseSegment === segment
+}
+
+/**
+ * Checks if the segment is an exit span.
+ * An exit span is defined as a segment that is an external call,
+ * datastore operation, or message broker operation.
+ * @param {TraceSegment} segment segment that is creating span
+ * @returns {boolean} true if the segment is an exit span
+ */
+function isExitSpan(segment) {
+  return REGEXS.CLIENT.EXTERNAL.test(segment.name) || REGEXS.CLIENT.DATASTORE.test(segment.name) || REGEXS.PRODUCER.test(segment.name)
+}
+
+/**
+ * Determines if a span should be created based on the segment and transaction.
+ * If the segment is an entry point span or an exit span, a span should be created.
+ * @param {object} params to function
+ * @param {boolean} params.entryPoint true if the segment is an entry point span
+ * @param {TraceSegment} params.segment segment that is creating span
+ * @returns {boolean} true if a span should be created
+ */
+function shouldCreateSpan({ entryPoint, segment }) {
+  return entryPoint ||
+         isExitSpan(segment)
+}
+
+/**
+ * Reparents a span based on the transaction and inProcessSpans.
+ * If inProcessSpans is true or the transaction has accepted distributed trace and it's the root segment,
+ * the parentId is not changed.
+ * If there is a parentId and the transaction has a base segment, the span is reparented to the base segment.
+ * If there is no parentId, the span is not reparented(indicates an entry root span for a DT trace).
+ * @param {object} params to function
+ * @param {boolean} params.inProcessSpans true if in-process spans are enabled
+ * @param {boolean} params.isRoot true if the segment is the root segment
+ * @param {string} params.parentId the parent id of the segment
+ * @param {Transaction} params.transaction active transaction
+ * @returns {string|null} the new parentId for the span
+ */
+function reparentSpan({ inProcessSpans, isRoot, parentId, transaction }) {
+  if (inProcessSpans || (transaction.acceptedDistributedTrace && isRoot)) {
+    return parentId
+  } else if (parentId && transaction?.baseSegment?.id) {
+    return transaction.baseSegment.id
+  } else {
+    return null
+  }
+}
+
 module.exports = {
   HTTP_LIBRARY,
   CATEGORIES,
   SPAN_KIND,
   REGEXS,
-  addSpanKind
+  addSpanKind,
+  isEntryPointSpan,
+  reparentSpan,
+  shouldCreateSpan
 }

package/lib/spans/span-event-aggregator.js

@@ -21,6 +21,7 @@ class SpanEventAggregator extends EventAggregator {
     opts.metricNames = opts.metricNames || NAMES.SPAN_EVENTS
 
     super(opts, agent)
+    this.inProcessSpans = agent.config.distributed_tracing.in_process_spans.enabled
   }
 
   _toPayloadSync() {
@@ -66,7 +67,6 @@ class SpanEventAggregator extends EventAggregator {
    * @param {Transaction} params.transaction active transaction
    * @param {string} [params.parentId] GUID of the parent span.
    * @param {boolean} params.isRoot if segment is root segment
-   * @returns {boolean} True if the segment was added, or false if it was discarded.
    */
   addSegment({ segment, transaction, parentId, isRoot }) {
     // Check if the priority would be accepted before creating the event object.
@@ -77,8 +77,11 @@ class SpanEventAggregator extends EventAggregator {
 
       return false
     }
-    const span = SpanEvent.fromSegment(segment, transaction, parentId || null, isRoot)
-    return this.add(span, transaction.priority)
+    const span = SpanEvent.fromSegment({ segment, transaction, parentId, isRoot, inProcessSpans: this.inProcessSpans })
+
+    if (span) {
+      this.add(span, transaction.priority)
+    }
   }
 
   /**

package/lib/spans/span-event.js

@@ -9,7 +9,7 @@ const Config = require('../config')
 const { truncate } = require('../util/byte-limit')
 
 const { DESTINATIONS } = require('../config/attribute-filter')
-const { addSpanKind, HTTP_LIBRARY, REGEXS, SPAN_KIND, CATEGORIES } = require('./helpers')
+const { addSpanKind, isEntryPointSpan, reparentSpan, shouldCreateSpan, HTTP_LIBRARY, REGEXS, SPAN_KIND, CATEGORIES } = require('./helpers')
 const EMPTY_USER_ATTRS = Object.freeze(Object.create(null))
 const SERVER_ADDRESS = 'server.address'
 
@@ -93,9 +93,15 @@ class SpanEvent {
    * @param {Transaction} transaction active transaction
    * @param {?string} [parentId] ID of the segment's parent.
    * @param {boolean} isRoot if segment is root segment
+   * @param {boolean} inProcessSpans if the segment is in-process, create span
    * @returns {SpanEvent} The constructed event.
    */
-  static fromSegment(segment, transaction, parentId = null, isRoot = false) {
+  static fromSegment({ segment, transaction, parentId = null, isRoot = false, inProcessSpans }) {
+    const entryPoint = isEntryPointSpan({ segment, transaction })
+    if (!inProcessSpans && !shouldCreateSpan({ entryPoint, segment, transaction })) {
+      return null
+    }
+
     const spanContext = segment.getSpanContext()
 
     // Since segments already hold span agent attributes and we want to leverage
@@ -128,7 +134,7 @@ class SpanEvent {
 
     span.intrinsics.traceId = transaction.traceId
     span.intrinsics.guid = segment.id
-    span.intrinsics.parentId = parentId
+    span.intrinsics.parentId = reparentSpan({ inProcessSpans, isRoot, segment, transaction, parentId })
     span.intrinsics.transactionId = transaction.id
     span.intrinsics.sampled = transaction.sampled
     span.intrinsics.priority = transaction.priority
@@ -142,7 +148,7 @@ class SpanEvent {
     }
 
     // Only set this if it will be `true`. Must be `null` otherwise.
-    if (transaction.baseSegment === segment) {
+    if (entryPoint) {
       span.intrinsics['nr.entryPoint'] = true
     }
 

package/lib/spans/streaming-span-event-aggregator.js

@@ -44,6 +44,7 @@ class StreamingSpanEventAggregator extends Aggregator {
     this.metrics = metrics
     this.started = false
     this.isStream = true
+    this.inProcessSpans = agent.config.distributed_tracing.in_process_spans.enabled
   }
 
   /**
@@ -112,8 +113,11 @@ class StreamingSpanEventAggregator extends Aggregator {
       return
     }
 
-    const span = StreamingSpanEvent.fromSegment(segment, transaction, parentId, isRoot)
-    this.stream.write(span)
+    const span = StreamingSpanEvent.fromSegment({ segment, transaction, parentId, isRoot, inProcessSpans: this.inProcessSpans })
+
+    if (span) {
+      this.stream.write(span)
+    }
   }
 
   reconfigure(config) {

package/lib/spans/streaming-span-event.js

@@ -10,7 +10,7 @@ const { truncate } = require('../util/byte-limit')
 const Config = require('../config')
 
 const { DESTINATIONS } = require('../config/attribute-filter')
-const { addSpanKind, HTTP_LIBRARY, REGEXS, SPAN_KIND, CATEGORIES } = require('./helpers')
+const { addSpanKind, isEntryPointSpan, reparentSpan, shouldCreateSpan, HTTP_LIBRARY, REGEXS, SPAN_KIND, CATEGORIES } = require('./helpers')
 
 /**
  * Specialized span event class for use with infinite streaming.
@@ -108,7 +108,12 @@ class StreamingSpanEvent {
     }
   }
 
-  static fromSegment(segment, transaction, parentId = null, isRoot = false) {
+  static fromSegment({ segment, transaction, parentId = null, isRoot = false, inProcessSpans }) {
+    const entryPoint = isEntryPointSpan({ segment, transaction })
+    if (!inProcessSpans && !shouldCreateSpan({ entryPoint, segment, transaction })) {
+      return null
+    }
+
     const spanContext = segment.getSpanContext()
 
     // Since segments already hold span agent attributes and we want to leverage
@@ -141,8 +146,9 @@ class StreamingSpanEvent {
       span.addIntrinsicAttribute(key, value)
     }
 
+    const newParentId = reparentSpan({ inProcessSpans, isRoot, segment, transaction, parentId })
     span.addIntrinsicAttribute('guid', segment.id)
-    span.addIntrinsicAttribute('parentId', parentId)
+    span.addIntrinsicAttribute('parentId', newParentId)
     span.addIntrinsicAttribute('transactionId', transaction.id)
     span.addIntrinsicAttribute('sampled', transaction.sampled)
     span.addIntrinsicAttribute('priority', transaction.priority)
@@ -156,7 +162,7 @@ class StreamingSpanEvent {
     }
 
     // Only set this if it will be `true`. Must be `null` otherwise.
-    if (transaction.baseSegment === segment) {
+    if (entryPoint) {
       span.addIntrinsicAttribute('nr.entryPoint', true)
     }
 

package/lib/util/code-level-metrics.js

@@ -52,7 +52,7 @@ clmUtils.addCLMAttributes = function addCLMAttributes(fn, segment) {
   }
 
   try {
-    const { funcInfo } = require('@contrast/fn-inspect')
+    const { funcInfo } = require('@newrelic/fn-inspect')
     const { lineNumber, method, file: filePath, column } = funcInfo(fn)
     const fnName = setFunctionName(method)
 

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "newrelic",
-  "version": "12.22.0",
+  "version": "12.23.0",
   "author": "New Relic Node.js agent team <nodejs@newrelic.com>",
   "license": "Apache-2.0",
   "contributors": [
@@ -223,8 +223,8 @@
     "winston-transport": "^4.5.0"
   },
   "optionalDependencies": {
-    "@contrast/fn-inspect": "^4.2.0",
-    "@newrelic/native-metrics": "^11.0.0",
+    "@newrelic/fn-inspect": "^4.4.0",
+    "@newrelic/native-metrics": "^11.1.0",
     "@prisma/prisma-fmt-wasm": "^4.17.0-16.27eb2449f178cd9fe1a4b892d732cc4795f75085"
   },
   "devDependencies": {