fix(deno): Avoid inferring invalid span op from Deno tracer

Author: Lms24

dev-packages/e2e-tests/test-applications/deno/tests/ai.test.ts

@@ -32,7 +32,7 @@ test('should create AI pipeline spans with Vercel AI SDK', async ({ baseURL }) =
     (span: any) =>
       span.op === 'gen_ai.invoke_agent' ||
       span.op === 'gen_ai.generate_content' ||
-      span.op === 'otel.span' ||
+      !span.op ||
       span.description?.includes('ai.generateText'),
   );
 

dev-packages/e2e-tests/test-applications/deno/tests/transactions.test.ts

@@ -33,7 +33,6 @@ test('Sends transaction with OTel tracer.startSpan despite pre-existing provider
     expect.arrayContaining([
       expect.objectContaining({
         description: 'test-otel-span',
-        op: 'otel.span',
         origin: 'manual',
       }),
     ]),
@@ -53,7 +52,6 @@ test('Sends transaction with OTel tracer.startActiveSpan', async ({ baseURL }) =
     expect.arrayContaining([
       expect.objectContaining({
         description: 'test-otel-active-span',
-        op: 'otel.span',
         origin: 'manual',
       }),
     ]),
@@ -77,7 +75,6 @@ test('OTel span appears as child of Sentry span (interop)', async ({ baseURL })
       }),
       expect.objectContaining({
         description: 'otel-child',
-        op: 'otel.span',
         origin: 'manual',
       }),
     ]),

packages/deno/src/opentelemetry/tracer.ts

@@ -43,7 +43,7 @@ class SentryDenoTracer implements Tracer {
       attributes: {
         ...options?.attributes,
         [SEMANTIC_ATTRIBUTE_SENTRY_ORIGIN]: 'manual',
-        [SEMANTIC_ATTRIBUTE_SENTRY_OP]: op,
+        ...(op ? { [SEMANTIC_ATTRIBUTE_SENTRY_OP]: op } : {}),
         'sentry.deno_tracer': true,
       },
     });
@@ -77,7 +77,7 @@ class SentryDenoTracer implements Tracer {
       attributes: {
         ...opts.attributes,
         [SEMANTIC_ATTRIBUTE_SENTRY_ORIGIN]: 'manual',
-        [SEMANTIC_ATTRIBUTE_SENTRY_OP]: op,
+        ...(op ? { [SEMANTIC_ATTRIBUTE_SENTRY_OP]: op } : {}),
         'sentry.deno_tracer': true,
       },
     };
@@ -96,7 +96,7 @@ class SentryDenoTracer implements Tracer {
     return startSpanManual(spanOpts, callback) as ReturnType<F>;
   }
 
-  private _mapSpanKindToOp(kind?: SpanKind): string {
+  private _mapSpanKindToOp(kind?: SpanKind): string | undefined {
     switch (kind) {
       case SpanKind.CLIENT:
         return 'http.client';
@@ -107,7 +107,7 @@ class SentryDenoTracer implements Tracer {
       case SpanKind.CONSUMER:
         return 'message.consume';
       default:
-        return 'otel.span';
+        return undefined;
     }
   }
 }