swap linkTraces arguments, adjust comment

Author: Lms24

packages/browser/src/tracing/browserTracingIntegration.ts

@@ -425,7 +425,7 @@ export const browserTracingIntegration = ((_options: Partial<BrowserTracingOptio
       });
 
       if (linkPreviousTrace !== 'off') {
-        linkTraces({ linkPreviousTrace, consistentTraceSampling }, client);
+        linkTraces(client, { linkPreviousTrace, consistentTraceSampling });
       }
 
       if (WINDOW.location) {

packages/browser/src/tracing/linkedTraces.ts

@@ -48,14 +48,14 @@ export const PREVIOUS_TRACE_TMP_SPAN_ATTRIBUTE = 'sentry.previous_trace';
  * @param client - Sentry client
  */
 export function linkTraces(
+  client: Client,
   {
     linkPreviousTrace,
     consistentTraceSampling,
   }: {
     linkPreviousTrace: 'session-storage' | 'in-memory';
     consistentTraceSampling: boolean;
   },
-  client: Client,
 ): void {
   const useSessionStorage = linkPreviousTrace === 'session-storage';
 
@@ -79,7 +79,7 @@ export function linkTraces(
     /*
     When users opt into `consistentTraceSampling`, we need to ensure that we propagate
     the previous trace's sample rate and rand to the current trace. This is necessary because otherwise, span
-    metric extrapolation is inaccurate, as we'd propagate a too high sample rate for the subsequent traces.
+    metric extrapolation is inaccurate, as we'd propagate too high of a sample rate for the subsequent traces.
 
     So therefore, we pretend that the previous trace was the parent trace of the newly started trace. To do that,
     we mutate the propagation context of the current trace and set the sample rate and sample rand of the previous trace.

packages/browser/test/tracing/linkedTraces.test.ts

@@ -35,7 +35,7 @@ describe('linkTraces', () => {
     });
 
     beforeEach(() => {
-      linkTraces({ linkPreviousTrace: 'in-memory', consistentTraceSampling: false }, client);
+      linkTraces(client, { linkPreviousTrace: 'in-memory', consistentTraceSampling: false });
     });
 
     it("doesn't add a link if the passed span is not the root span", () => {
@@ -131,7 +131,7 @@ describe('linkTraces', () => {
     const clientOnSpy = vi.spyOn(client, 'on');
 
     beforeEach(() => {
-      linkTraces({ linkPreviousTrace: 'in-memory', consistentTraceSampling: true }, client);
+      linkTraces(client, { linkPreviousTrace: 'in-memory', consistentTraceSampling: true });
     });
 
     it('registers a beforeSampling handler', () => {