Update auth ops

Author: onurtemizkan

dev-packages/browser-integration-tests/suites/integrations/supabase/auth/test.ts

@@ -76,7 +76,7 @@ sentryTest('should capture Supabase authentication spans', async ({ getLocalTest
   const url = await getLocalTestUrl({ testDir: __dirname });
 
   const eventData = await getFirstSentryEnvelopeRequest<Event>(page, url);
-  const supabaseSpans = eventData.spans?.filter(({ op }) => op?.startsWith('db.supabase.auth'));
+  const supabaseSpans = eventData.spans?.filter(({ op }) => op?.startsWith('db.auth'));
 
   expect(supabaseSpans).toHaveLength(2);
   expect(supabaseSpans![0]).toMatchObject({
@@ -88,7 +88,7 @@ sentryTest('should capture Supabase authentication spans', async ({ getLocalTest
     trace_id: eventData.contexts?.trace?.trace_id,
     status: 'ok',
     data: expect.objectContaining({
-      'sentry.op': 'db.supabase.auth.signInWithPassword',
+      'sentry.op': 'db.auth.signInWithPassword',
       'sentry.origin': 'auto.db.supabase',
     }),
   });
@@ -102,7 +102,7 @@ sentryTest('should capture Supabase authentication spans', async ({ getLocalTest
     trace_id: eventData.contexts?.trace?.trace_id,
     status: 'ok',
     data: expect.objectContaining({
-      'sentry.op': 'db.supabase.auth.signOut',
+      'sentry.op': 'db.auth.signOut',
       'sentry.origin': 'auto.db.supabase',
     }),
   });
@@ -119,7 +119,7 @@ sentryTest('should capture Supabase authentication errors', async ({ getLocalTes
 
   const [errorEvent, transactionEvent] = await getMultipleSentryEnvelopeRequests<Event>(page, 2, { url });
 
-  const supabaseSpans = transactionEvent.spans?.filter(({ op }) => op?.startsWith('db.supabase.auth'));
+  const supabaseSpans = transactionEvent.spans?.filter(({ op }) => op?.startsWith('db.auth'));
 
   expect(errorEvent.exception?.values?.[0].value).toBe('Invalid email or password');
 
@@ -133,7 +133,7 @@ sentryTest('should capture Supabase authentication errors', async ({ getLocalTes
     trace_id: transactionEvent.contexts?.trace?.trace_id,
     status: 'unknown_error',
     data: expect.objectContaining({
-      'sentry.op': 'db.supabase.auth.signInWithPassword',
+      'sentry.op': 'db.auth.signInWithPassword',
       'sentry.origin': 'auto.db.supabase',
     }),
   });

dev-packages/e2e-tests/test-applications/supabase-nextjs/tests/performance.test.ts

@@ -19,7 +19,7 @@ test('Sends server-side Supabase auth admin `createUser` span', async ({ page, b
   expect(transactionEvent.spans).toContainEqual({
     data: expect.any(Object),
     description: 'createUser',
-    op: 'db.supabase.auth.admin.createUser',
+    op: 'db.auth.admin.createUser',
     parent_span_id: expect.stringMatching(/[a-f0-9]{16}/),
     span_id: expect.stringMatching(/[a-f0-9]{16}/),
     start_timestamp: expect.any(Number),
@@ -165,7 +165,7 @@ test('Sends server-side Supabase auth admin `listUsers` span', async ({ page, ba
   expect(transactionEvent.spans).toContainEqual({
     data: expect.any(Object),
     description: 'listUsers',
-    op: 'db.supabase.auth.admin.listUsers',
+    op: 'db.auth.admin.listUsers',
     parent_span_id: expect.stringMatching(/[a-f0-9]{16}/),
     span_id: expect.stringMatching(/[a-f0-9]{16}/),
     start_timestamp: expect.any(Number),

packages/core/src/integrations/supabase.ts

@@ -224,11 +224,11 @@ function instrumentAuthOperation(operation: AuthOperationFn, isAdmin = false): A
           name: operation.name,
           attributes: {
             [SEMANTIC_ATTRIBUTE_SENTRY_ORIGIN]: 'auto.db.supabase',
-            [SEMANTIC_ATTRIBUTE_SENTRY_OP]: `db.supabase.auth.${isAdmin ? 'admin.' : ''}${operation.name}`,
+            [SEMANTIC_ATTRIBUTE_SENTRY_OP]: `db.auth.${isAdmin ? 'admin.' : ''}${operation.name}`,
           },
         },
         span => {
-          return Reflect.apply(target, thisArg, argumentsList)
+          return Reflect.apply(target, thisArg, [])
             .then((res: unknown) => {
               if (res && typeof res === 'object' && 'error' in res && res.error) {
                 span.setStatus({ code: SPAN_STATUS_ERROR });