dd-trace 5.52.0 → 5.54.0

package/packages/datadog-plugin-cucumber/src/index.js

@@ -34,7 +34,10 @@ const {
   TEST_MANAGEMENT_IS_ATTEMPT_TO_FIX,
   TEST_HAS_FAILED_ALL_RETRIES,
   TEST_MANAGEMENT_ATTEMPT_TO_FIX_PASSED,
-  TEST_RETRY_REASON_TYPES
+  TEST_RETRY_REASON_TYPES,
+  TEST_IS_MODIFIED,
+  isModifiedTest,
+  getTestEndLine
 } = require('../../dd-trace/src/plugins/util/test')
 const { RESOURCE_NAME } = require('../../../ext/tags')
 const { COMPONENT, ERROR_MESSAGE } = require('../../dd-trace/src/constants')
@@ -221,13 +224,8 @@ class CucumberPlugin extends CiPlugin {
       this.telemetry.ciVisEvent(TELEMETRY_CODE_COVERAGE_FINISHED, 'suite', { library: 'istanbul' })
     })
 
-    this.addSub('ci:cucumber:test:start', ({
-      testName,
-      testFileAbsolutePath,
-      testSourceLine,
-      isParallel,
-      promises
-    }) => {
+    this.addBind('ci:cucumber:test:start', (ctx) => {
+      const { testName, testFileAbsolutePath, testSourceLine, isParallel, promises } = ctx
       const store = storage('legacy').getStore()
       const testSuite = getTestSuitePath(testFileAbsolutePath, this.sourceRoot)
       const testSourceFile = getTestSuitePath(testFileAbsolutePath, this.repositoryRoot)
@@ -240,22 +238,23 @@ class CucumberPlugin extends CiPlugin {
         extraTags[CUCUMBER_IS_PARALLEL] = 'true'
       }
 
-      const testSpan = this.startTestSpan(testName, testSuite, extraTags)
+      const span = this.startTestSpan(testName, testSuite, extraTags)
 
-      this.enter(testSpan, store)
+      ctx.parentStore = store
+      ctx.currentStore = { ...store, span }
 
-      this.activeTestSpan = testSpan
+      this.activeTestSpan = span
       // Time we give the breakpoint to be hit
       if (promises && this.runningTestProbe) {
         promises.hitBreakpointPromise = new Promise((resolve) => {
           setTimeout(resolve, BREAKPOINT_HIT_GRACE_PERIOD_MS)
         })
       }
+
+      return ctx.currentStore
     })
 
-    this.addSub('ci:cucumber:test:retry', ({ isFirstAttempt, error, isAtrRetry }) => {
-      const store = storage('legacy').getStore()
-      const span = store.span
+    this.addSub('ci:cucumber:test:retry', ({ span, isFirstAttempt, error, isAtrRetry }) => {
       if (!isFirstAttempt) {
         span.setTag(TEST_IS_RETRY, 'true')
         if (isAtrRetry) {
@@ -284,7 +283,9 @@ class CucumberPlugin extends CiPlugin {
       finishAllTraceSpans(span)
     })
 
-    this.addSub('ci:cucumber:test-step:start', ({ resource }) => {
+    this.addBind('ci:cucumber:test-step:start', (ctx) => {
+      const { resource } = ctx
+
       const store = storage('legacy').getStore()
       const childOf = store ? store.span : store
       const span = this.tracer.startSpan('cucumber.step', {
@@ -295,7 +296,10 @@ class CucumberPlugin extends CiPlugin {
           [RESOURCE_NAME]: resource
         }
       })
-      this.enter(span, store)
+      ctx.parentStore = store
+      ctx.currentStore = { ...store, span }
+
+      return ctx.currentStore
     })
 
     this.addSub('ci:cucumber:worker-report:trace', (traces) => {
@@ -329,6 +333,7 @@ class CucumberPlugin extends CiPlugin {
     })
 
     this.addSub('ci:cucumber:test:finish', ({
+      span,
       isStep,
       status,
       skipReason,
@@ -343,9 +348,9 @@ class CucumberPlugin extends CiPlugin {
       hasPassedAllRetries,
       hasFailedAttemptToFix,
       isDisabled,
-      isQuarantined
+      isQuarantined,
+      isModified
     }) => {
-      const span = storage('legacy').getStore().span
       const statusTag = isStep ? 'step.status' : TEST_STATUS
 
       span.setTag(statusTag, status)
@@ -399,6 +404,14 @@ class CucumberPlugin extends CiPlugin {
         span.setTag(TEST_MANAGEMENT_IS_QUARANTINED, 'true')
       }
 
+      if (isModified) {
+        span.setTag(TEST_IS_MODIFIED, 'true')
+        if (isEfdRetry) {
+          span.setTag(TEST_IS_RETRY, 'true')
+          span.setTag(TEST_RETRY_REASON, TEST_RETRY_REASON_TYPES.efd)
+        }
+      }
+
       span.finish()
       if (!isStep) {
         const spanTags = span.context()._tags
@@ -425,11 +438,64 @@ class CucumberPlugin extends CiPlugin {
       }
     })
 
-    this.addSub('ci:cucumber:error', (err) => {
+    this.addBind('ci:cucumber:error', (ctx) => {
+      const { err } = ctx
       if (err) {
-        const span = storage('legacy').getStore().span
+        const span = ctx.currentStore.span
         span.setTag('error', err)
+
+        ctx.parentStore = ctx.currentStore
+        ctx.currentStore = { ...ctx.currentStore, span }
+      }
+
+      return ctx.currentStore
+    })
+
+    this.addBind('ci:cucumber:test:fn', (ctx) => {
+      return ctx.currentStore
+    })
+
+    this.addSub('ci:cucumber:is-modified-test', ({
+      scenarios,
+      testFileAbsolutePath,
+      modifiedTests,
+      stepIds,
+      stepDefinitions,
+      setIsModified
+    }) => {
+      const testScenarioPath = getTestSuitePath(testFileAbsolutePath, this.repositoryRoot || process.cwd())
+      for (const scenario of scenarios) {
+        const isModified = isModifiedTest(
+          testScenarioPath,
+          scenario.location.line,
+          scenario.steps[scenario.steps.length - 1].location.line,
+          modifiedTests,
+          'cucumber'
+        )
+        if (isModified) {
+          setIsModified(true)
+          return
+        }
+      }
+      for (const stepDefinition of stepDefinitions) {
+        if (!stepIds?.includes(stepDefinition.id)) {
+          continue
+        }
+        const testStartLineStep = stepDefinition.line
+        const testEndLineStep = getTestEndLine(stepDefinition.code, testStartLineStep)
+        const isModified = isModifiedTest(
+          stepDefinition.uri,
+          testStartLineStep,
+          testEndLineStep,
+          modifiedTests,
+          'cucumber'
+        )
+        if (isModified) {
+          setIsModified(true)
+          return
+        }
       }
+      setIsModified(false)
     })
   }
 

package/packages/datadog-plugin-cypress/src/cypress-plugin.js

@@ -41,7 +41,11 @@ const {
   TEST_MANAGEMENT_ATTEMPT_TO_FIX_PASSED,
   TEST_HAS_FAILED_ALL_RETRIES,
   getLibraryCapabilitiesTags,
-  TEST_RETRY_REASON_TYPES
+  TEST_RETRY_REASON_TYPES,
+  getPullRequestDiff,
+  getModifiedTestsFromDiff,
+  TEST_IS_MODIFIED,
+  getPullRequestBaseBranch
 } = require('../../dd-trace/src/plugins/util/test')
 const { isMarkedAsUnskippable } = require('../../datadog-plugin-jest/src/util')
 const { ORIGIN_KEY, COMPONENT } = require('../../dd-trace/src/constants')
@@ -67,7 +71,11 @@ const {
   GIT_BRANCH,
   CI_PROVIDER_NAME,
   CI_WORKSPACE_PATH,
-  GIT_COMMIT_MESSAGE
+  GIT_COMMIT_MESSAGE,
+  GIT_TAG,
+  GIT_PULL_REQUEST_BASE_BRANCH_SHA,
+  GIT_COMMIT_HEAD_SHA,
+  GIT_PULL_REQUEST_BASE_BRANCH
 } = require('../../dd-trace/src/plugins/util/tags')
 const {
   OS_VERSION,
@@ -76,6 +84,7 @@ const {
   RUNTIME_NAME,
   RUNTIME_VERSION
 } = require('../../dd-trace/src/plugins/util/env')
+const { DD_MAJOR } = require('../../../version')
 
 const TEST_FRAMEWORK_NAME = 'cypress'
 
@@ -175,6 +184,26 @@ function getTestManagementTests (tracer, testConfiguration) {
   })
 }
 
+function getModifiedTests (testEnvironmentMetadata) {
+  const {
+    [GIT_PULL_REQUEST_BASE_BRANCH]: pullRequestBaseBranch,
+    [GIT_PULL_REQUEST_BASE_BRANCH_SHA]: pullRequestBaseBranchSha,
+    [GIT_COMMIT_HEAD_SHA]: commitHeadSha
+  } = testEnvironmentMetadata
+
+  const baseBranchSha = pullRequestBaseBranchSha || getPullRequestBaseBranch(pullRequestBaseBranch)
+
+  if (baseBranchSha) {
+    const diff = getPullRequestDiff(baseBranchSha, commitHeadSha)
+    const modifiedTests = getModifiedTestsFromDiff(diff)
+    if (modifiedTests) {
+      return modifiedTests
+    }
+  }
+
+  throw new Error('Modified tests could not be retrieved')
+}
+
 function getSuiteStatus (suiteStats) {
   if (!suiteStats) {
     return 'skip'
@@ -205,10 +234,13 @@ class CypressPlugin {
       [GIT_BRANCH]: branch,
       [CI_PROVIDER_NAME]: ciProviderName,
       [CI_WORKSPACE_PATH]: repositoryRoot,
-      [GIT_COMMIT_MESSAGE]: commitMessage
+      [GIT_COMMIT_MESSAGE]: commitMessage,
+      [GIT_TAG]: tag,
+      [GIT_PULL_REQUEST_BASE_BRANCH_SHA]: pullRequestBaseSha,
+      [GIT_COMMIT_HEAD_SHA]: commitHeadSha
     } = this.testEnvironmentMetadata
 
-    this.repositoryRoot = repositoryRoot
+    this.repositoryRoot = repositoryRoot || process.cwd()
     this.ciProviderName = ciProviderName
     this.codeOwnersEntries = getCodeOwnersFileEntries(repositoryRoot)
 
@@ -222,7 +254,10 @@ class CypressPlugin {
       runtimeVersion,
       branch,
       testLevel: 'test',
-      commitMessage
+      commitMessage,
+      tag,
+      pullRequestBaseSha,
+      commitHeadSha
     }
     this.finishedTestsByFile = {}
     this.testStatuses = {}
@@ -242,6 +277,8 @@ class CypressPlugin {
     this.knownTests = []
     this.isTestManagementTestsEnabled = false
     this.testManagementAttemptToFixRetries = 0
+    this.isImpactedTestsEnabled = false
+    this.modifiedTests = []
   }
 
   // Init function returns a promise that resolves with the Cypress configuration
@@ -271,7 +308,8 @@ class CypressPlugin {
               flakyTestRetriesCount,
               isKnownTestsEnabled,
               isTestManagementEnabled,
-              testManagementAttemptToFixRetries
+              testManagementAttemptToFixRetries,
+              isImpactedTestsEnabled
             }
           } = libraryConfigurationResponse
           this.isSuitesSkippingEnabled = isSuitesSkippingEnabled
@@ -285,12 +323,25 @@ class CypressPlugin {
           }
           this.isTestManagementTestsEnabled = isTestManagementEnabled
           this.testManagementAttemptToFixRetries = testManagementAttemptToFixRetries
+          this.isImpactedTestsEnabled = isImpactedTestsEnabled
         }
         return this.cypressConfig
       })
     return this.libraryConfigurationPromise
   }
 
+  getIsTestModified (testSuiteAbsolutePath) {
+    const relativeTestSuitePath = getTestSuitePath(testSuiteAbsolutePath, this.repositoryRoot)
+    if (!this.modifiedTests) {
+      return false
+    }
+    const lines = this.modifiedTests[relativeTestSuitePath]
+    if (!lines) {
+      return false
+    }
+    return lines.length > 0
+  }
+
   getTestSuiteProperties (testSuite) {
     return this.testManagementTests?.cypress?.suites?.[testSuite]?.tests || {}
   }
@@ -455,6 +506,15 @@ class CypressPlugin {
       }
     }
 
+    if (this.isImpactedTestsEnabled) {
+      try {
+        this.modifiedTests = getModifiedTests(this.testEnvironmentMetadata)
+      } catch (error) {
+        log.error(error)
+        this.isImpactedTestsEnabled = false
+      }
+    }
+
     // `details.specs` are test files
     details.specs?.forEach(({ absolute, relative }) => {
       const isUnskippableSuite = isMarkedAsUnskippable({ path: absolute })
@@ -474,7 +534,13 @@ class CypressPlugin {
       testSessionSpanMetadata[TEST_EARLY_FLAKE_ENABLED] = 'true'
     }
 
-    const testSessionName = getTestSessionName(this.tracer._tracer._config, this.command, this.testEnvironmentMetadata)
+    const trimmedCommand = DD_MAJOR < 6 ? this.command : 'cypress run'
+
+    const testSessionName = getTestSessionName(
+      this.tracer._tracer._config,
+      trimmedCommand,
+      this.testEnvironmentMetadata
+    )
 
     if (this.tracer._tracer._exporter?.addMetadataTags) {
       const metadataTags = {}
@@ -601,13 +667,9 @@ class CypressPlugin {
       const isSkippedByItr = this.testsToSkip.find(test =>
         cypressTestName === test.name && spec.relative === test.suite
       )
-      let testSourceFile
-
-      if (spec.absolute && this.repositoryRoot) {
-        testSourceFile = getTestSuitePath(spec.absolute, this.repositoryRoot)
-      } else {
-        testSourceFile = spec.relative
-      }
+      const testSourceFile = spec.absolute && this.repositoryRoot
+        ? getTestSuitePath(spec.absolute, this.repositoryRoot)
+        : spec.relative
 
       const skippedTestSpan = this.getTestSpan({ testName: cypressTestName, testSuite: spec.relative, testSourceFile })
 
@@ -683,12 +745,9 @@ class CypressPlugin {
         if (this.itrCorrelationId) {
           finishedTest.testSpan.setTag(ITR_CORRELATION_ID, this.itrCorrelationId)
         }
-        let testSourceFile
-        if (spec.absolute && this.repositoryRoot) {
-          testSourceFile = getTestSuitePath(spec.absolute, this.repositoryRoot)
-        } else {
-          testSourceFile = spec.relative
-        }
+        const testSourceFile = spec.absolute && this.repositoryRoot
+          ? getTestSuitePath(spec.absolute, this.repositoryRoot)
+          : spec.relative
         if (testSourceFile) {
           finishedTest.testSpan.setTag(TEST_SOURCE_FILE, testSourceFile)
         }
@@ -725,7 +784,10 @@ class CypressPlugin {
           isKnownTestsEnabled: this.isKnownTestsEnabled,
           isTestManagementEnabled: this.isTestManagementTestsEnabled,
           testManagementAttemptToFixRetries: this.testManagementAttemptToFixRetries,
-          testManagementTests: this.getTestSuiteProperties(testSuite)
+          testManagementTests: this.getTestSuiteProperties(testSuite),
+          isImpactedTestsEnabled: this.isImpactedTestsEnabled,
+          isModifiedTest: this.getIsTestModified(testSuiteAbsolutePath),
+          repositoryRoot: this.repositoryRoot
         }
 
         if (this.testSuiteSpan) {
@@ -736,7 +798,7 @@ class CypressPlugin {
       },
       'dd:beforeEach': (test) => {
         const { testName, testSuite } = test
-        const shouldSkip = !!this.testsToSkip.find(test => {
+        const shouldSkip = this.testsToSkip.some(test => {
           return testName === test.name && testSuite === test.suite
         })
         const isUnskippable = this.unskippableSuites.includes(testSuite)
@@ -783,7 +845,8 @@ class CypressPlugin {
           testName,
           isNew,
           isEfdRetry,
-          isAttemptToFix
+          isAttemptToFix,
+          isModified
         } = test
         if (coverage && this.isCodeCoverageEnabled && this.tracer._tracer._exporter?.exportCoverage) {
           const coverageFiles = getCoveredFilenamesFromCoverage(coverage)
@@ -807,10 +870,10 @@ class CypressPlugin {
         this.activeTestSpan.setTag(TEST_STATUS, testStatus)
 
         // Save the test status to know if it has passed all retries
-        if (!this.testStatuses[testName]) {
-          this.testStatuses[testName] = [testStatus]
-        } else {
+        if (this.testStatuses[testName]) {
           this.testStatuses[testName].push(testStatus)
+        } else {
+          this.testStatuses[testName] = [testStatus]
         }
         const testStatuses = this.testStatuses[testName]
 
@@ -830,6 +893,13 @@ class CypressPlugin {
             this.activeTestSpan.setTag(TEST_RETRY_REASON, TEST_RETRY_REASON_TYPES.efd)
           }
         }
+        if (isModified) {
+          this.activeTestSpan.setTag(TEST_IS_MODIFIED, 'true')
+          if (isEfdRetry) {
+            this.activeTestSpan.setTag(TEST_IS_RETRY, 'true')
+            this.activeTestSpan.setTag(TEST_RETRY_REASON, TEST_RETRY_REASON_TYPES.efd)
+          }
+        }
         if (isAttemptToFix) {
           this.activeTestSpan.setTag(TEST_MANAGEMENT_IS_ATTEMPT_TO_FIX, 'true')
           if (testStatuses.length > 1) {
@@ -838,7 +908,7 @@ class CypressPlugin {
           }
           const isLastAttempt = testStatuses.length === this.testManagementAttemptToFixRetries + 1
           if (isLastAttempt) {
-            if (testStatuses.some(status => status === 'fail')) {
+            if (testStatuses.includes('fail')) {
               this.activeTestSpan.setTag(TEST_MANAGEMENT_ATTEMPT_TO_FIX_PASSED, 'false')
             }
             if (testStatuses.every(status => status === 'fail')) {

package/packages/datadog-plugin-cypress/src/plugin.js

@@ -1,6 +1,7 @@
 const NoopTracer = require('../../dd-trace/src/noop/tracer')
 const cypressPlugin = require('./cypress-plugin')
 const satisfies = require('semifies')
+const { DD_MAJOR } = require('../../../version')
 
 const noopTask = {
   'dd:testSuiteStart': () => {
@@ -17,10 +18,19 @@ const noopTask = {
   }
 }
 
-module.exports = (on, config) => {
+module.exports = function CypressPlugin (on, config) {
   const tracer = require('../../dd-trace')
 
   if (satisfies(config.version, '<10.2.0')) {
+    if (DD_MAJOR >= 6) {
+      // eslint-disable-next-line no-console
+      console.error(
+        'ERROR: dd-trace v6 has deleted support for Cypress<10.2.0.'
+      )
+      on('task', noopTask)
+      return config
+    }
+
     // console.warn does not seem to work in cypress, so using console.log instead
     // eslint-disable-next-line no-console
     console.log(

package/packages/datadog-plugin-cypress/src/support.js

@@ -1,12 +1,15 @@
+/* eslint-disable unicorn/no-abusive-eslint-disable */
 /* eslint-disable */
 let isEarlyFlakeDetectionEnabled = false
 let isKnownTestsEnabled = false
 let knownTestsForSuite = []
-let suiteTests = []
 let earlyFlakeDetectionNumRetries = 0
 let isTestManagementEnabled = false
 let testManagementAttemptToFixRetries = 0
 let testManagementTests = {}
+let isImpactedTestsEnabled = false
+let isModifiedTest = false
+
 // We need to grab the original window as soon as possible,
 // in case the test changes the origin. If the test does change the origin,
 // any call to `cy.window()` will result in a cross origin error.
@@ -48,10 +51,9 @@ function retryTest (test, suiteTests, numRetries, tags) {
   }
 }
 
-
 const oldRunTests = Cypress.mocha.getRunner().runTests
 Cypress.mocha.getRunner().runTests = function (suite, fn) {
-  if (!isKnownTestsEnabled && !isTestManagementEnabled) {
+  if (!isKnownTestsEnabled && !isTestManagementEnabled && !isImpactedTestsEnabled) {
     return oldRunTests.apply(this, arguments)
   }
   // We copy the new tests at the beginning of the suite run (runTests), so that they're run
@@ -67,10 +69,24 @@ Cypress.mocha.getRunner().runTests = function (suite, fn) {
         retryTest(test, suite.tests, testManagementAttemptToFixRetries, ['_ddIsAttemptToFix'])
       }
     }
+    if (isImpactedTestsEnabled && isModifiedTest) {
+      test._ddIsModified = true
+      if (isEarlyFlakeDetectionEnabled && !isAttemptToFix) {
+        retryTest(
+          test,
+          suite.tests,
+          earlyFlakeDetectionNumRetries,
+          ['_ddIsModified', '_ddIsEfdRetry', isKnownTestsEnabled && isNewTest(test) && '_ddIsNew']
+        )
+      }
+    }
     if (isKnownTestsEnabled) {
       if (!test._ddIsNew && !test.isPending() && isNewTest(test)) {
         test._ddIsNew = true
-        if (isEarlyFlakeDetectionEnabled && !isAttemptToFix) {
+        if (isImpactedTestsEnabled && isModifiedTest) {
+          test._ddIsModified = true
+        }
+        if (isEarlyFlakeDetectionEnabled && !isAttemptToFix && !isModifiedTest) {
           retryTest(test, suite.tests, earlyFlakeDetectionNumRetries, ['_ddIsNew', '_ddIsEfdRetry'])
         }
       }
@@ -108,6 +124,8 @@ before(function () {
       isTestManagementEnabled = suiteConfig.isTestManagementEnabled
       testManagementAttemptToFixRetries = suiteConfig.testManagementAttemptToFixRetries
       testManagementTests = suiteConfig.testManagementTests
+      isImpactedTestsEnabled = suiteConfig.isImpactedTestsEnabled
+      isModifiedTest = suiteConfig.isModifiedTest
     }
   })
 })
@@ -133,7 +151,8 @@ afterEach(function () {
     error: currentTest.err,
     isNew: currentTest._ddIsNew,
     isEfdRetry: currentTest._ddIsEfdRetry,
-    isAttemptToFix: currentTest._ddIsAttemptToFix
+    isAttemptToFix: currentTest._ddIsAttemptToFix,
+    isModified: currentTest._ddIsModified
   }
   try {
     testInfo.testSourceLine = Cypress.mocha.getRunner().currentRunnable.invocationDetails.line

package/packages/datadog-plugin-express/src/code_origin.js

@@ -0,0 +1,30 @@
+'use strict'
+
+const { entryTags } = require('../../datadog-code-origin')
+const Plugin = require('../../dd-trace/src/plugins/plugin')
+const web = require('../../dd-trace/src/plugins/util/web')
+
+class ExpressCodeOriginForSpansPlugin extends Plugin {
+  static get id () {
+    return 'express'
+  }
+
+  constructor (...args) {
+    super(...args)
+
+    const layerTags = new WeakMap()
+
+    this.addSub('apm:express:middleware:enter', ({ req, layer }) => {
+      const tags = layerTags.get(layer)
+      if (!tags) return
+      web.getContext(req).span?.addTags(tags)
+    })
+
+    this.addSub('apm:express:route:added', ({ topOfStackFunc, layer }) => {
+      if (layerTags.has(layer)) return
+      layerTags.set(layer, entryTags(topOfStackFunc))
+    })
+  }
+}
+
+module.exports = ExpressCodeOriginForSpansPlugin

package/packages/datadog-plugin-express/src/index.js

@@ -1,18 +1,16 @@
 'use strict'
 
-const RouterPlugin = require('../../datadog-plugin-router/src')
+const ExpressTracingPlugin = require('./tracing')
+const ExpressCodeOriginForSpansPlugin = require('./code_origin')
+const CompositePlugin = require('../../dd-trace/src/plugins/composite')
 
-class ExpressPlugin extends RouterPlugin {
-  static get id () {
-    return 'express'
-  }
-
-  constructor (...args) {
-    super(...args)
-
-    this.addSub('apm:express:request:handle', ({ req }) => {
-      this.setFramework(req, 'express', this.config)
-    })
+class ExpressPlugin extends CompositePlugin {
+  static get id () { return 'express' }
+  static get plugins () {
+    return {
+      tracing: ExpressTracingPlugin,
+      codeOriginForSpans: ExpressCodeOriginForSpansPlugin
+    }
   }
 }
 

package/packages/datadog-plugin-express/src/tracing.js

@@ -0,0 +1,19 @@
+'use strict'
+
+const RouterPlugin = require('../../datadog-plugin-router/src')
+
+class ExpressTracingPlugin extends RouterPlugin {
+  static get id () {
+    return 'express'
+  }
+
+  constructor (...args) {
+    super(...args)
+
+    this.addSub('apm:express:request:handle', ({ req }) => {
+      this.setFramework(req, 'express', this.config)
+    })
+  }
+}
+
+module.exports = ExpressTracingPlugin

package/packages/datadog-plugin-google-cloud-pubsub/src/client.js

@@ -7,18 +7,22 @@ class GoogleCloudPubsubClientPlugin extends ClientPlugin {
   static get type () { return 'messaging' }
   static get operation () { return 'request' }
 
-  start ({ request, api, projectId }) {
+  start (ctx) {
+    const { request, api, projectId } = ctx
+
     if (api === 'publish') return
 
     this.startSpan(this.operationName(), {
       service: this.config.service || this.serviceName(),
-      resource: [api, request.name].filter(x => x).join(' '),
+      resource: [api, request.name].filter(Boolean).join(' '),
       kind: this.constructor.kind,
       meta: {
         'pubsub.method': api,
         'gcloud.project_id': projectId
       }
-    })
+    }, ctx)
+
+    return ctx.currentStore
   }
 }