ref: Remove redundant forceTransaction: true usages

Removes `forceTransaction: true` where the span is already a root span, so the
option had no effect:

- `aws-serverless`: the `function.aws` invocation span
- `browser`: the navigation span, which already passes `parentSpan: null`
- `nestjs`: the BullMQ process span, since queue jobs run outside a request
- `react-router`: the server request span, on the branch that only runs when
  there is no root span

Co-Authored-By: Opus 5 <noreply@anthropic.com>

Author: andreiborza

packages/aws-serverless/src/requestSpanOptions.ts

@@ -46,7 +46,6 @@ export function getRequestSpanOptions(event: unknown, context: Context, requestI
   // The span is started within the surrounding `continueTrace`, so it continues the incoming trace.
   return {
     name: context.functionName,
-    forceTransaction: true,
     attributes: {
       [SENTRY_OP]: FUNCTION_AWS,
       [SEMANTIC_ATTRIBUTE_SENTRY_ORIGIN]: 'auto.aws_lambda',

packages/browser/src/tracing/browserTracingIntegration.ts

@@ -520,7 +520,6 @@ export const browserTracingIntegration = ((options: Partial<BrowserTracingOption
             ...startSpanOptions,
             // Navigation starts a new trace and is NOT parented under any active interaction (e.g. ui.action.click)
             parentSpan: null,
-            forceTransaction: true,
           },
           true,
           navigationOptions?.url,

packages/nestjs/src/integrations/helpers.ts

@@ -130,7 +130,6 @@ const PROCESS_OPERATION = 'process';
 export function getBullMQProcessSpanOptions(queueName: string | undefined): {
   name: string;
   attributes: Record<string, string | undefined>;
-  forceTransaction: boolean;
 } {
   const client = getClient();
   const isStreamed = !!client && hasSpanStreamingEnabled(client);
@@ -151,7 +150,6 @@ export function getBullMQProcessSpanOptions(queueName: string | undefined): {
       [MESSAGING_OPERATION_TYPE]: PROCESS_OPERATION,
       [MESSAGING_DESTINATION_NAME]: queueName,
     },
-    forceTransaction: true,
   };
 }
 

packages/react-router/src/server/createServerInstrumentation.ts

@@ -104,7 +104,6 @@ export function createSentryServerInstrumentation(
             await startSpan(
               {
                 name: unparameterizedName,
-                forceTransaction: true,
                 attributes: {
                   [SEMANTIC_ATTRIBUTE_SENTRY_OP]: 'http.server',
                   [SEMANTIC_ATTRIBUTE_SENTRY_ORIGIN]: 'auto.http.react_router.instrumentation_api',

packages/react-router/test/server/createServerInstrumentation.test.ts

@@ -165,11 +165,10 @@ describe('createSentryServerInstrumentation', () => {
 
     await hooks.request(mockHandleRequest, { request: mockRequest, context: undefined });
 
-    // Should create a new root span with forceTransaction
+    // Should create a new root span
     expect(core.startSpan).toHaveBeenCalledWith(
       expect.objectContaining({
         name: 'GET /api/users',
-        forceTransaction: true,
         attributes: expect.objectContaining({
           'sentry.op': 'http.server',
           'sentry.origin': 'auto.http.react_router.instrumentation_api',