From ad72d86329dc396a06c3d2e9597c2b058a075857 Mon Sep 17 00:00:00 2001
From: Alden Quimby <aldenquimby@gmail.com>
Date: Thu, 12 Oct 2023 13:57:57 -0400
Subject: [PATCH] fix: rename startTransaction -> startTrace

---
 packages/serverless/src/awslambda.ts       | 8 ++++----
 packages/serverless/test/awslambda.test.ts | 6 +++---
 2 files changed, 7 insertions(+), 7 deletions(-)

diff --git a/packages/serverless/src/awslambda.ts b/packages/serverless/src/awslambda.ts
index d915ba7f5af7..e29f77aaf879 100644
--- a/packages/serverless/src/awslambda.ts
+++ b/packages/serverless/src/awslambda.ts
@@ -52,7 +52,7 @@ export interface WrapperOptions {
    * You may want to disable this if you use express within Lambda (use tracingHandler instead).
    * @default true
    */
-  startTransaction: boolean;
+  startTrace: boolean;
 }
 
 export const defaultIntegrations: Integration[] = [...Sentry.defaultIntegrations, new AWSServices({ optional: true })];
@@ -235,7 +235,7 @@ export function wrapHandler<TEvent, TResult>(
     captureTimeoutWarning: true,
     timeoutWarningLimit: 500,
     captureAllSettledReasons: false,
-    startTransaction: true,
+    startTrace: true,
     ...wrapOptions,
   };
   let timeoutWarningTimer: NodeJS.Timeout;
@@ -294,7 +294,7 @@ export function wrapHandler<TEvent, TResult>(
     const hub = getCurrentHub();
 
     let transaction: Sentry.Transaction | undefined;
-    if (options.startTransaction) {
+    if (options.startTrace) {
       const eventWithHeaders = event as { headers?: { [key: string]: string } };
 
       const sentryTrace =
@@ -324,7 +324,7 @@ export function wrapHandler<TEvent, TResult>(
     let rv: TResult;
     try {
       enhanceScopeWithEnvironmentData(scope, context, START_TIME);
-      if (options.startTransaction) {
+      if (options.startTrace) {
         enhanceScopeWithTransactionData(scope, context);
         // We put the transaction on the scope so users can attach children to it
         scope.setSpan(transaction);
diff --git a/packages/serverless/test/awslambda.test.ts b/packages/serverless/test/awslambda.test.ts
index 2641c4dccd6d..199d0ac295ab 100644
--- a/packages/serverless/test/awslambda.test.ts
+++ b/packages/serverless/test/awslambda.test.ts
@@ -183,12 +183,12 @@ describe('AWSLambda', () => {
       expect(SentryNode.captureException).toBeCalledTimes(2);
     });
 
-    // "wrapHandler() ... successful execution" tests the default of startTransaction enabled
-    test('startTransaction disabled', async () => {
+    // "wrapHandler() ... successful execution" tests the default of startTrace enabled
+    test('startTrace disabled', async () => {
       expect.assertions(3);
 
       const handler: Handler = async (_event, _context) => 42;
-      const wrappedHandler = wrapHandler(handler, { startTransaction: false });
+      const wrappedHandler = wrapHandler(handler, { startTrace: false });
       await wrappedHandler(fakeEvent, fakeContext, fakeCallback);
 
       // @ts-expect-error see "Why @ts-expect-error" note