feat(node)!: Use function.gcp span op for Firebase functions

Author: msonnb

MIGRATION.md

@@ -254,6 +254,16 @@ Affected SDKs: All SDKs.
 
 If you reference these attributes in custom instrumentation, `beforeSendSpan`, dashboards, or alerts, update them to the new names.
 
+### Span operation changes
+
+Affected SDKs: All SDKs.
+
+Span operations (`sentry.op`) were aligned with the values defined in `@sentry/conventions`:
+
+- `@sentry/node`: Firebase Functions spans now use `function.gcp` instead of `http.request`. The wrapper instruments all function trigger types (HTTP, Firestore, scheduler, storage), so a fixed HTTP op was wrong. The trigger type remains available on the `faas.trigger` attribute.
+
+If you reference these operations in dashboards, alerts, or `beforeSendSpan`, update them to the new values.
+
 ### `thirdPartyErrorFilterIntegration` filters internal frames by default
 
 Affected SDKs: All SDKs.

dev-packages/e2e-tests/test-applications/node-firebase/tests/functions.test.ts

@@ -20,12 +20,12 @@ test('should only call the function once without any extra calls', async () => {
           'faas.provider': 'firebase',
           'faas.trigger': 'http.request',
           'sentry.kind': 'server',
-          'sentry.op': 'http.request',
+          'sentry.op': 'function.gcp',
           'sentry.origin': 'auto.firebase.otel.functions',
           'sentry.sample_rate': expect.any(Number),
           'sentry.source': 'route',
         },
-        op: 'http.request',
+        op: 'function.gcp',
         origin: 'auto.firebase.otel.functions',
         span_id: expect.any(String),
         status: 'ok',
@@ -97,12 +97,12 @@ test('should create a document and trigger onDocumentCreated and another with au
       'faas.provider': 'firebase',
       'faas.trigger': 'http.request',
       'sentry.kind': 'server',
-      'sentry.op': 'http.request',
+      'sentry.op': 'function.gcp',
       'sentry.origin': 'auto.firebase.otel.functions',
       'sentry.sample_rate': expect.any(Number),
       'sentry.source': 'route',
     },
-    op: 'http.request',
+    op: 'function.gcp',
     origin: 'auto.firebase.otel.functions',
     span_id: expect.any(String),
     status: 'ok',

packages/node/src/integrations/tracing/firebase/otel/patches/functions.ts

@@ -2,6 +2,7 @@ import type { InstrumentationBase } from '@opentelemetry/instrumentation';
 import { InstrumentationNodeModuleDefinition, isWrapped } from '@opentelemetry/instrumentation';
 import { InstrumentationNodeModuleFile } from '../../../InstrumentationNodeModuleFile';
 import { SENTRY_KIND } from '@sentry/conventions/attributes';
+import { FAAS_FUNCTION_GCP_SPAN_OP } from '@sentry/conventions/op';
 import type { SpanAttributes } from '@sentry/core';
 import {
   captureException,
@@ -87,7 +88,7 @@ export function patchV2Functions<T extends FirebaseFunctions = FirebaseFunctions
         return startSpanManual(
           {
             name: `firebase.function.${triggerType}`,
-            op: 'http.request',
+            op: FAAS_FUNCTION_GCP_SPAN_OP,
             attributes,
           },
           async span => {