newrelic 13.4.0 → 13.5.0

package/NEWS.md

@@ -1,3 +1,35 @@
+### v13.5.0 (2025-10-14)
+
+#### Code refactoring
+
+* Updated `fastify` instrumentation to subscribe to events emitted ([#3407](https://github.com/newrelic/node-newrelic/pull/3407)) ([51e361e](https://github.com/newrelic/node-newrelic/commit/51e361ed736fa1235eeb16f1c5ec833a1b56a0bf))
+* Updated `pg` and `pg-native` instrumentation to subscribe to events emitted ([#3397](https://github.com/newrelic/node-newrelic/pull/3397)) ([04af258](https://github.com/newrelic/node-newrelic/commit/04af2580a554ecabedab447d5edc5cec836d7f40))
+
+* Updated `config.instrumentation.timers.enabled` to false to disable `timers.setTimeout` and `timers.setInterval` by default. ([#3414](https://github.com/newrelic/node-newrelic/pull/3414)) ([b925d3d](https://github.com/newrelic/node-newrelic/commit/b925d3d178886e6d4bbd0f2eaa6f51b6fddd7878))
+    * To re-enable, set `config.instrumentation.timers.enabled` to `true` in config or set `NEW_RELIC_INSTRUMENTATION_TIMERS_ENABLED=true` 
+
+#### Documentation
+
+* Updated compatibility report ([#3413](https://github.com/newrelic/node-newrelic/pull/3413)) ([fef98dd](https://github.com/newrelic/node-newrelic/commit/fef98ddffddf60b32f162ea87dee2a3a0bdf8048))
+
+#### Miscellaneous chores
+
+* Added a new diagnostics base class and migrated undici to use new class ([#3408](https://github.com/newrelic/node-newrelic/pull/3408)) ([8047bf1](https://github.com/newrelic/node-newrelic/commit/8047bf165ae5105bb612c1a834d574078d52724d))
+* Created directory for samplers ([#3419](https://github.com/newrelic/node-newrelic/pull/3419)) ([dd2d3df](https://github.com/newrelic/node-newrelic/commit/dd2d3dfe1bb2582de51e07a1cce6077df84f7108))
+
+#### Styles
+
+* Require `check-types` and `require-returns-type` jsdoc ([#3400](https://github.com/newrelic/node-newrelic/pull/3400)) ([e4cd687](https://github.com/newrelic/node-newrelic/commit/e4cd687c59aeaba6b627403bcb29b4b6576fe4d9))
+* Require jsdoc `check*`, `implements-on-classes`, and some `no-undefined-types` ([#3404](https://github.com/newrelic/node-newrelic/pull/3404)) ([4fa645b](https://github.com/newrelic/node-newrelic/commit/4fa645b91900ff0acbd24b8f50c6fffcd07f3b82))
+* Require jsdoc `require-returns-check` and `require-property-*` ([#3403](https://github.com/newrelic/node-newrelic/pull/3403)) ([1008073](https://github.com/newrelic/node-newrelic/commit/100807342b9c9283f43042024f15bb575de219ee))
+
+#### Tests
+
+* Added helper to get version of package when package.json not exported, and updated tests that need it ([#3411](https://github.com/newrelic/node-newrelic/pull/3411)) ([5ae02c7](https://github.com/newrelic/node-newrelic/commit/5ae02c7411cd7ae0a45203b025559c5442a7171e))
+* Fixed fastify segment tree assertion when security agent is enabled ([#3422](https://github.com/newrelic/node-newrelic/pull/3422)) ([b2cf6ed](https://github.com/newrelic/node-newrelic/commit/b2cf6ed5d8debae23d0bb31e9a72b325464fadc2))
+* Refactored mcp tests to listen on new port for every streaming test, removed unncessary shutdown of process ([#3420](https://github.com/newrelic/node-newrelic/pull/3420)) ([17db813](https://github.com/newrelic/node-newrelic/commit/17db8133e8e13c970362330885dccf4870942aea))
+* Updated an openai assertion with setTimeout disabled ([#3421](https://github.com/newrelic/node-newrelic/pull/3421)) ([ceec8f9](https://github.com/newrelic/node-newrelic/commit/ceec8f9ef2ac95285dcc1619ef2ee8e187252beb))
+
 ### v13.4.0 (2025-09-29)
 
 #### Code refactoring

package/api.js

@@ -1937,7 +1937,7 @@ API.prototype.ignoreApdex = function ignoreApdex() {
  *      { role: 'user', content: 'Tell me about Node.js.'}
  *    ]})
  * })
- * @param {Object} context LLM custom attributes context
+ * @param {object} context LLM custom attributes context
  * @param {Function} callback The function to execute in context.
  */
 API.prototype.withLlmCustomAttributes = function withLlmCustomAttributes(context, callback) {

package/lib/agent.js

@@ -5,7 +5,7 @@
 
 'use strict'
 
-const AdaptiveSampler = require('./adaptive-sampler')
+const AdaptiveSampler = require('./samplers/adaptive-sampler')
 const CollectorAPI = require('./collector/api')
 const ServerlessCollector = require('./collector/serverless')
 const DESTINATIONS = require('./config/attribute-filter').DESTINATIONS
@@ -169,7 +169,7 @@ const DEFAULT_HARVEST_INTERVAL_MS = 60000
  * available. Don't try to recover here, because without configuration the
  * agent can't be brought up to a useful state.
  *
- * @param {object} config Agent configuration object
+ * @param {Config} config Agent configuration object
  */
 function Agent(config) {
   EventEmitter.call(this)

package/lib/aggregators/event-aggregator.js

@@ -85,8 +85,6 @@ class EventAggregator extends Aggregator {
 
   /**
    * Resets the contents of the aggregator and all counters.
-   *
-   * @returns {PriorityQueue} The old collection of aggregated events.
    */
   clearEvents() {
     // ???: might be more efficient to clear here and come up with an efficient way to

package/lib/attributes.js

@@ -176,8 +176,8 @@ class Attributes {
  * Creates a filter function for the given scope.
  *
  * @param {string} scope - The scope of the filter to make.
- * @returns {Function} A function that performs attribute filtering for the given
- *  scope.
+ * @returns {Function|undefined} A function that performs attribute filtering for the given
+ *  scope, or undefined if the scope is not recognized.
  */
 function makeFilter(scope) {
   const { attributeFilter } = Config.getInstance()

package/lib/collector/api.js

@@ -153,7 +153,7 @@ CollectorAPI.prototype._updateEndpoints = function _updateEndpoints(endpoint) {
 /**
  * Connect to the data collector.
  *
- * @param {function} callback A typical error first callback to be invoked
+ * @param {Function} callback A typical error first callback to be invoked
  * upon successful or unsuccessful connection. The second parameter will be
  * an instance of {@link CollectorResponse}.
  *

package/lib/collector/http-agents.js

@@ -19,7 +19,7 @@ let agentProxyWithKeepAlive = null
 /**
  * Returns an HTTP agent with keep-alive enabled
  *
- * @param {Object} config configuration for HTTP agent
+ * @param {object} config configuration for HTTP agent
  */
 exports.keepAliveAgent = function keepAliveAgent(config) {
   config = config ? config : {}
@@ -45,7 +45,7 @@ exports.keepAliveAgent = function keepAliveAgent(config) {
  * to the proxy server as to how its connection is made
  * with New Relic's servers.
  *
- * @param {Object} config configuration for proxy agent
+ * @param {object} config configuration for proxy agent
  */
 exports.proxyAgent = function proxyAgent(config) {
   if (agentProxyWithKeepAlive !== null) {

package/lib/collector/serverless.js

@@ -110,7 +110,7 @@ class ServerlessCollector {
    * Constructs, serializes, and prints the final consolidated payload to stdout.
    *
    * @param {Function} cb The callback to invoke when finished.
-   * @returns {boolean} indicating if callback was defined and successfully executed
+   * @returns {boolean|undefined} indicating if callback was defined and successfully executed
    */
   flushPayload(cb) {
     if (!this.enabled) {

package/lib/config/build-instrumentation-config.js

@@ -12,18 +12,28 @@ const corePkgs = Object.keys(coreLibraries)
 const subscriptions = require('../subscriber-configs')
 const subscribers = Object.keys(subscriptions)
 pkgNames.push(...subscribers)
-// Manually adding undici as it is registered separately in shimmer
-corePkgs.push('undici')
 // Manually adding domain as it is registered separately in shimmer
 corePkgs.push('domain')
 pkgNames.push(...corePkgs)
 
+// Packages are normally enabled without any extra
+// configuration. This list is a set of packages that
+// we want to be disabled without any extra configuration.
+// Typically, this is because the instrumentation no longer
+// provides useful data. Users can still enable them if they
+// are interested in the instrumentation they provide.
+const disabledPkgs = ['timers']
+
 /**
  * Builds the stanza for config.instrumentation.*
  * It defaults every library to true and assigns a boolean
  * formatter for the environment variable conversion of the values
  */
 module.exports = pkgNames.reduce((config, pkg) => {
-  config[pkg] = { enabled: { formatter: boolean, default: true } }
+  let defaultValue = true
+  if (disabledPkgs.includes(pkg)) {
+    defaultValue = false
+  }
+  config[pkg] = { enabled: { formatter: boolean, default: defaultValue } }
   return config
 }, {})

package/lib/config/default.js

@@ -1156,16 +1156,27 @@ defaultConfig.definition = () => {
       },
 
       sampler: {
-      /**
-       * When set to `always_on`, the sampled flag in the `traceparent` header
-       * being set to "true" will result in the local transaction being sampled
-       * with a priority value of "2". When set to `always_off`, the local
-       * transaction will never be sampled. At the default setting, the sampling
-       * decision will be determined according to the normal algorithm.
-       *
-       * This setting takes precedence over the `remote_parent_not_sampled`
-       * setting.
-       */
+        /**
+         * The sampling target for adaptive sampling is controlled via this attribute when configuring the default/adaptive sampler.
+         * The default sampling target is 10 transactions/min when it is not specified but **MUST** be within the range of [1, 120] (inclusive).
+         * Upon agent connect, the connect response **MUST** provide the value of `sampling_target` based on this configuration setting's value.
+         * The `sampling_target` value from the connect response **SHOULD** be used as the sampling target value for adaptive sampling in the agent.
+         */
+        adaptive_sampling_target: {
+          formatter: int,
+          default: 10
+        },
+
+        /**
+         * When set to `always_on`, the sampled flag in the `traceparent` header
+         * being set to "true" will result in the local transaction being sampled
+         * with a priority value of "2". When set to `always_off`, the local
+         * transaction will never be sampled. At the default setting, the sampling
+         * decision will be determined according to the normal algorithm.
+         *
+         * This setting takes precedence over the `remote_parent_not_sampled`
+         * setting.
+         */
         remote_parent_sampled: {
           formatter: allowList.bind(null, ['always_on', 'always_off', 'default']),
           default: 'default'

package/lib/config/formatters.js

@@ -67,7 +67,7 @@ formatters.boolean = function boolean(setting) {
  *
  * @param {string} val config setting
  * @param {logger} logger agent logger instance
- * @returns {object}
+ * @returns {object|undefined} the parsed value, or undefined if an error occurred
  */
 formatters.object = function object(val, logger) {
   try {
@@ -83,7 +83,7 @@ formatters.object = function object(val, logger) {
  *
  * @param {string} val config setting
  * @param {logger} logger agent logger instance
- * @returns {Array}
+ * @returns {Array|undefined} The parsed array of objects, or undefined if an error occurred
  */
 formatters.objectList = function objectList(val, logger) {
   try {
@@ -111,7 +111,7 @@ formatters.allowList = function allowList(list, val) {
  *
  * @param {string} val valid regex
  * @param {logger} logger agent logger instance
- * @returns {RegExp} regex
+ * @returns {RegExp|undefined} regex, or undefined if an error occurred
  */
 formatters.regex = function regex(val, logger) {
   try {

package/lib/config/index.js

@@ -123,7 +123,8 @@ function Config(config) {
   this.primary_application_id = null
   this.trusted_account_ids = null
   this.trusted_account_key = null
-  this.sampling_target = 10
+
+  this.sampling_target = this.distributed_tracing.sampler.adaptive_sampling_target
   this.sampling_target_period_in_seconds = 60
   this.max_payload_size_in_bytes = DEFAULT_MAX_PAYLOAD_SIZE_IN_BYTES
 
@@ -184,9 +185,9 @@ function Config(config) {
     if (this.security_policies_token) {
       throw new Error(
         'Security Policies and High Security Mode cannot both be present ' +
-          'in the agent configuration. If Security Policies have been set ' +
-          'for your account, please ensure the security_policies_token is ' +
-          'set but high_security is disabled (default).'
+        'in the agent configuration. If Security Policies have been set ' +
+        'for your account, please ensure the security_policies_token is ' +
+        'set but high_security is disabled (default).'
       )
     }
     this._applyHighSecurity()
@@ -199,8 +200,20 @@ function Config(config) {
   this.parsedLabels = parseLabels(this.labels, logger)
   this.loggingLabels = this._setApplicationLoggingLabels()
 }
+
 util.inherits(Config, EventEmitter)
 
+// Expose sampling_target as an accessor that proxies to distributed_tracing.sampler.adaptive_sampling_target.
+Object.defineProperty(Config.prototype, 'sampling_target', {
+  get: function () {
+    return this.distributed_tracing.sampler.adaptive_sampling_target
+  },
+  set: function (value) {
+    this.distributed_tracing.sampler.adaptive_sampling_target = value
+    this.emit('sampling_target', value)
+  }
+})
+
 /**
  * Compares the labels list to the application logging excluded label list and removes any labels that need to be excluded.
  * Then prefixing each label with "tags."
@@ -694,7 +707,7 @@ function _validateThenUpdateStatusCodes(remote, local, remoteKey, localKey) {
     if (!(typeof thingToTest === 'string' || typeof thingToTest === 'number')) {
       logger.warn(
         'Saw SSC (ignore|expect)_status_code that is not a number or string,' +
-          'will not merge: %s',
+        'will not merge: %s',
         thingToTest
       )
       valid = false
@@ -858,7 +871,7 @@ Config.prototype.logDisabled = function logDisabled(json, key) {
   if (value != null) {
     logger.debug(
       'Server-side configuration of %s is currently disabled by local configuration. ' +
-        '(Server sent value of %s.)',
+      '(Server sent value of %s.)',
       key,
       value
     )
@@ -877,7 +890,7 @@ Config.prototype.logUnsupported = function logUnsupported(json, key) {
   if (value !== null && value !== undefined) {
     logger.debug(
       'Server-side configuration of %s is currently not supported by the ' +
-        'Node.js agent. (Server sent value of %s.)',
+      'Node.js agent. (Server sent value of %s.)',
       key,
       value
     )
@@ -1218,8 +1231,8 @@ Config.prototype._serverlessLogging = function _serverlessLogging(inputConfig) {
 
     logger.info(
       'Logging is disabled by default when serverless_mode is enabled. ' +
-        'If desired, enable logging via config file or environment variable and ' +
-        'set filepath to a valid path for current environment, stdout or stderr.'
+      'If desired, enable logging via config file or environment variable and ' +
+      'set filepath to a valid path for current environment, stdout or stderr.'
     )
   }
 }
@@ -1241,17 +1254,17 @@ Config.prototype._serverlessNativeMetrics = function _serverlessNativeMetrics(in
   ) {
     logger.info(
       'Enabling the native-metrics module when in serverless mode may greatly ' +
-        'increase cold-start times. Given the limited benefit of the VM metrics' +
-        'and general lack of control in a serverless environment, we do not ' +
-        'recommend this trade-off.'
+      'increase cold-start times. Given the limited benefit of the VM metrics' +
+      'and general lack of control in a serverless environment, we do not ' +
+      'recommend this trade-off.'
     )
   } else {
     this.plugins.native_metrics.enabled = false
 
     logger.info(
       'The native-metrics module is disabled by default when serverless_mode ' +
-        'is enabled.  If desired, enable the native-metrics module via config file ' +
-        'or environment variable.'
+      'is enabled.  If desired, enable the native-metrics module via config file ' +
+      'or environment variable.'
     )
   }
 }
@@ -1305,8 +1318,8 @@ Config.prototype._serverlessDT = function _serverlessDT() {
     if (this.distributed_tracing.enabled) {
       logger.warn(
         'Using distributed tracing in serverless mode requires account_id be ' +
-          'defined, either in your newrelic.js file or via environment variables. ' +
-          'Disabling distributed tracing.'
+        'defined, either in your newrelic.js file or via environment variables. ' +
+        'Disabling distributed tracing.'
       )
       this.distributed_tracing.enabled = false
     }
@@ -1332,8 +1345,8 @@ Config.prototype._preventServerlessDT = function _preventServerlessDT() {
     if (this[key]) {
       logger.warn(
         key +
-          ' was configured locally without enabling serverless_mode. ' +
-          'This local value will be ignored and set by the New Relic servers.'
+        ' was configured locally without enabling serverless_mode. ' +
+        'This local value will be ignored and set by the New Relic servers.'
       )
       this[key] = null
     }
@@ -1530,7 +1543,7 @@ function _laspReponse(keys) {
  * Applies the server side LASP policies to a local configuration object
  *
  * @param {Agent} agent Agent instance
- * @param {bject} policies server side LASP policy
+ * @param {object} policies server side LASP policy
  * @returns {object} { missingRequired, finalPolicies } list of missing required fields and finalized LASP policy
  */
 Config.prototype._buildLaspPolicy = function _buildLaspPolicy(agent, policies) {
@@ -1670,7 +1683,7 @@ function redactValue(value) {
  * Get a JSONifiable object containing all settings we want to report to the
  * collector and store in the environment_values table.
  *
- * @returns {object} containing simple key-value pairs of settings
+ * @returns {object|undefined} containing simple key-value pairs of settings, or undefined if an error occurred
  */
 Config.prototype.publicSettings = function publicSettings() {
   let settings = Object.create(null)

package/lib/harvester.js

@@ -74,7 +74,7 @@ module.exports = class Harvester {
   /**
    * Registers an aggregator with the Harvester instance.
    *
-   * @param {Object} aggregator Aggregator instance
+   * @param {object} aggregator Aggregator instance
    */
   add(aggregator) {
     this.aggregators.push(aggregator)

package/lib/header-processing.js

@@ -15,8 +15,8 @@ const CONTENT_LENGTH_REGEX = /^Content-Length$/i
  * Queue time is provided by certain providers by stamping the request
  * header with the time the request arrived at the router.
  *
- * @param {Object} logger Logger instance
- * @param {Object} requestHeaders Incoming request headers
+ * @param {object} logger Logger instance
+ * @param {object} requestHeaders Incoming request headers
  */
 function getQueueTime(logger, requestHeaders) {
   const headerValue = requestHeaders[REQUEST_START_HEADER] || requestHeaders[QUEUE_HEADER]
@@ -62,7 +62,7 @@ function convertUnit(time) {
  *
  * If no header is found, returns -1
  *
- * @param {Object} headers Incoming request headers
+ * @param {object} headers Incoming request headers
  */
 function getContentLengthFromHeaders(headers) {
   const contentLength = -1

package/lib/health-reporter.js

@@ -112,7 +112,7 @@ class HealthReporter {
    * object that has an `agent_control` property which is an instance of
    * {@link AgentControlConfig}.
    * @param {object} [params.logger] A standard logger instance.
-   * @param {function} [params.setInterval] A function to use as `setInterval`.
+   * @param {Function} [params.setInterval] A function to use as `setInterval`.
    * Must return an interval object that supports the `unref()` method.
    */
   constructor({
@@ -217,7 +217,7 @@ class HealthReporter {
    * initiate an immediate write of the status file, and then invoke the
    * provided callback.
    *
-   * @param {function} done Callback to be invoked after the status file has
+   * @param {Function} done Callback to be invoked after the status file has
    * been updated.
    */
   stop(done) {

package/lib/instrumentation/@prisma/client.js

@@ -75,7 +75,7 @@ function extractQueryArgs(args, pkgVersion) {
  *
  * @param {Array} args arguments to a prisma operation
  * @param {string} pkgVersion prisma version
- * @returns {string} query raw query string or model call <collection>.<operation>
+ * @returns {string|undefined} query raw query string or model call <collection>.<operation>
  */
 function retrieveQuery(args, pkgVersion) {
   if (Array.isArray(args)) {

package/lib/instrumentation/aws-sdk/util.js

@@ -20,9 +20,9 @@ function grabLastUrlSegment(url = '/') {
 /**
  * Retrieves the db segment params from endpoint and command parameters
  *
- * @param {Object} endpoint instance of ddb endpoint
- * @param {Object} params parameters passed to a ddb command
- * @returns {Object}
+ * @param {object} endpoint instance of ddb endpoint
+ * @param {object} params parameters passed to a ddb command
+ * @returns {object}
  */
 function setDynamoParameters(endpoint, params) {
   return new DatastoreParameters({

package/lib/instrumentation/aws-sdk/v2/instrumentation-helper.js

@@ -8,7 +8,7 @@
 /**
  * Series of tests to determine if the library
  * has the features needed to provide instrumentation.
- * @param {Object} AWS AWS SDK object
+ * @param {object} AWS AWS SDK object
  */
 const instrumentationSupported = function instrumentationSupported(AWS) {
   // instrumentation requires the serviceClientOperationsMap property

package/lib/instrumentation/aws-sdk/v3/bedrock.js

@@ -43,7 +43,7 @@ const CONVERSE_COMMANDS = new Set([
 /**
  * Helper to determine if we should instrument the bedrock middleware call
  *
- * @param {Object} config agent configuration
+ * @param {object} config agent configuration
  * @returns {boolean} to instrument or not to instrument
  */
 function shouldSkipInstrumentation(config) {
@@ -251,9 +251,9 @@ function createBedrockResponse({ bedrockCommand, response, err }) {
  * @param {object} params { config, commandName } aws config and command name
  * @param {string} params.commandName name of command
  * @param {Shim} shim instance of shim
- * @param {function} _original original middleware function
+ * @param {Function} _original original middleware function
  * @param {string} _name function name
- * @param {array} args argument passed to middleware
+ * @param {Array} args argument passed to middleware
  * @returns {object} specification object that records middleware as promise
  * with an after hook to create LLM events
  */
@@ -357,10 +357,10 @@ function handleResponse({ shim, err, response, segment, transaction, bedrockComm
  *
  * @param {Shim} shim instance of shim
  * @param {object} config AWS configuration object
- * @param {function} next the next middleware function in stack
+ * @param {Function} next the next middleware function in stack
  * @param {object} context AWS client context info
  *
- * @returns {function} the next middleware function in stack
+ * @returns {Function} the next middleware function in stack
  */
 function bedrockMiddleware(shim, config, next, context) {
   const { commandName } = context

package/lib/instrumentation/aws-sdk/v3/common.js

@@ -13,9 +13,9 @@ const InstrumentationDescriptor = require('../../../instrumentation-descriptor')
  * header to all outgoing requests
  *
  * @param {Shim} shim instance
- * @param {Object} config configuration options
- * @param {function} next next function in middleware chain
- * @returns {function}
+ * @param {object} config configuration options
+ * @param {Function} next next function in middleware chain
+ * @returns {Function}
  *
  */
 function headerMiddleware(shim, config, next) {
@@ -32,10 +32,10 @@ function headerMiddleware(shim, config, next) {
  * appropriate segment attributes for the AWS command
  *
  * @param {Shim} shim instance
- * @param {Object} config AWS command configuration
- * @param {function} next next function in middleware chain
- * @param {Object} context AWS command context
- * @returns {function}
+ * @param {object} config AWS command configuration
+ * @param {Function} next next function in middleware chain
+ * @param {object} context AWS command context
+ * @returns {Function}
  */
 function attrMiddleware(shim, config, next, context) {
   return async function wrappedMiddleware(args) {
@@ -55,12 +55,12 @@ function attrMiddleware(shim, config, next, context) {
  * Adds the necessary aws.* attributes to either the External or first
  * class operation segment
  *
- * @param {Object} params params object
- * @param {Object} params.result result from middleware
- * @param {Object} params.config AWS config
+ * @param {object} params params object
+ * @param {object} params.result result from middleware
+ * @param {object} params.config AWS config
  * @param {string} params.region AWS region
  * @param {Shim} params.shim Shim instance
- * @param {Object} params.context smithy client context
+ * @param {object} params.context smithy client context
  */
 function addAwsAttributes({ result, config, region, shim, context }) {
   try {

package/lib/instrumentation/aws-sdk/v3/dynamodb.js

@@ -13,9 +13,9 @@ const InstrumentationDescriptor = require('../../../instrumentation-descriptor')
  *
  * @param {Shim} shim Shim instance
  * @param {original} original Original middleware function
- * @param {String} name Name of the calling function
+ * @param {string} name Name of the calling function
  * @param {Array} args Arguments for the middleware function
- * @returns {Object}
+ * @returns {object}
  */
 function getDynamoSpec(shim, original, name, args) {
   const [{ input }] = args
@@ -35,7 +35,7 @@ function getDynamoSpec(shim, original, name, args) {
  * unless you provide an endpoint override to your library constructor
  * This function will derive the endpoint in that scenario by grabbing the region
  * and building the URL
- * @param {Object} config configuration options
+ * @param {object} config configuration options
  */
 async function getEndpoint(config) {
   if (typeof config.endpoint === 'function') {
@@ -51,10 +51,10 @@ async function getEndpoint(config) {
  * when command is in a list of monitored commands.
  *
  * @param {Shim} shim Shim instance
- * @param {Object} config AWS SDK client configuration
- * @param {function} next middleware function
- * @param {Object} context Context for the running command
- * @returns {function}
+ * @param {object} config AWS SDK client configuration
+ * @param {Function} next middleware function
+ * @param {object} context Context for the running command
+ * @returns {Function}
  */
 function dynamoMiddleware(shim, config, next, context) {
   const { commandName } = context
@@ -77,9 +77,9 @@ function dynamoMiddleware(shim, config, next, context) {
  * cloud.resource_id segment attributes for the AWS command
  *
  * @param {Shim} shim Shim instance
- * @param {Object} config AWS command configuration
- * @param {function} next next function in middleware chain
- * @returns {function}
+ * @param {object} config AWS command configuration
+ * @param {Function} next next function in middleware chain
+ * @returns {Function}
  */
 function resourceIdMiddlerware(shim, config, next) {
   return async function wrappedResourceIdMiddlerware(args) {

package/lib/instrumentation/aws-sdk/v3/lambda.js

@@ -11,9 +11,9 @@ const InstrumentationDescriptor = require('../../../instrumentation-descriptor')
  * cloud.resource_id segment attributes for the AWS command
  *
  * @param {Shim} shim New Relic agent shim
- * @param {Object} config AWS command configuration
- * @param {function} next next function in middleware chain
- * @returns {function} wrapped version of middleware function
+ * @param {object} config AWS command configuration
+ * @param {Function} next next function in middleware chain
+ * @returns {Function} wrapped version of middleware function
  */
 function resourceIdMiddleware(shim, config, next) {
   return async function wrappedResourceIdMiddleware(args) {

package/lib/instrumentation/aws-sdk/v3/sns.js

@@ -12,10 +12,10 @@ const InstrumentationDescriptor = require('../../../instrumentation-descriptor')
  * when command is `PublishCommand`
  *
  * @param {Shim} shim instance
- * @param {Object} config configuration options
- * @param {function} next middleware function
- * @param {Object} context context
- * @returns {function}
+ * @param {object} config configuration options
+ * @param {Function} next middleware function
+ * @param {object} context context
+ * @returns {Function}
  */
 function snsMiddleware(shim, config, next, context) {
   if (context.commandName === 'PublishCommand') {
@@ -30,10 +30,10 @@ function snsMiddleware(shim, config, next, context) {
  * Returns the spec for PublishCommand
  *
  * @param {Shim} shim instance
- * @param {function} original original middleware function
+ * @param {Function} original original middleware function
  * @param {string} name function name
  * @param {Array} args to the middleware function
- * @returns {Object}
+ * @returns {object}
  */
 function getSnsSpec(shim, original, name, args) {
   const [command] = args
@@ -50,7 +50,7 @@ function getSnsSpec(shim, original, name, args) {
  * Helper to set the appropriate destinationName based on
  * the command input
  *
- * @param {Object} params params object
+ * @param {object} params params object
  * @param {string} params.TopicArn TopicArn if available
  * @param {string} params.TargetArn TargetArn if available
  */