ref(aws-severless): Slightly adjust aws-serverless mechanism type (#17614)

should have used snake_case instead of kebab-case

Author: Lms24

dev-packages/e2e-tests/test-applications/aws-serverless/tests/layer.test.ts

@@ -160,7 +160,7 @@ test.describe('Lambda layer', () => {
         type: 'Error',
         value: 'test',
         mechanism: {
-          type: 'auto.function.aws-serverless.otel',
+          type: 'auto.function.aws_serverless.otel',
           handled: false,
         },
       }),
@@ -188,7 +188,7 @@ test.describe('Lambda layer', () => {
         type: 'Error',
         value: 'test esm',
         mechanism: {
-          type: 'auto.function.aws-serverless.otel',
+          type: 'auto.function.aws_serverless.otel',
           handled: false,
         },
       }),

packages/aws-serverless/src/integration/awslambda.ts

@@ -29,7 +29,7 @@ export const instrumentAwsLambda = generateInstrumentOnce(
       },
       responseHook(_span, { err }) {
         if (err) {
-          captureException(err, scope => markEventUnhandled(scope, 'auto.function.aws-serverless.otel'));
+          captureException(err, scope => markEventUnhandled(scope, 'auto.function.aws_serverless.otel'));
         }
       },
     };

packages/aws-serverless/src/sdk.ts

@@ -235,13 +235,13 @@ export function wrapHandler<TEvent, TResult>(
         if (options.captureAllSettledReasons && Array.isArray(rv) && isPromiseAllSettledResult(rv)) {
           const reasons = getRejectedReasons(rv);
           reasons.forEach(exception => {
-            captureException(exception, scope => markEventUnhandled(scope, 'auto.function.aws-serverless.promise'));
+            captureException(exception, scope => markEventUnhandled(scope, 'auto.function.aws_serverless.promise'));
           });
         }
       } catch (e) {
         // Errors should already captured in the instrumentation's `responseHook`,
         // we capture them here just to be safe. Double captures are deduplicated by the SDK.
-        captureException(e, scope => markEventUnhandled(scope, 'auto.function.aws-serverless.handler'));
+        captureException(e, scope => markEventUnhandled(scope, 'auto.function.aws_serverless.handler'));
         throw e;
       } finally {
         clearTimeout(timeoutWarningTimer);
@@ -281,14 +281,14 @@ function wrapStreamingHandler<TEvent, TResult>(
         enhanceScopeWithEnvironmentData(scope, context, startTime);
 
         responseStream.on('error', error => {
-          captureException(error, scope => markEventUnhandled(scope, 'auto.function.aws-serverless.stream'));
+          captureException(error, scope => markEventUnhandled(scope, 'auto.function.aws_serverless.stream'));
         });
 
         return await handler(event, responseStream, context);
       } catch (e) {
         // Errors should already captured in the instrumentation's `responseHook`,
         // we capture them here just to be safe. Double captures are deduplicated by the SDK.
-        captureException(e, scope => markEventUnhandled(scope, 'auto.function.aws-serverless.handler'));
+        captureException(e, scope => markEventUnhandled(scope, 'auto.function.aws_serverless.handler'));
         throw e;
       } finally {
         if (timeoutWarningTimer) {

packages/aws-serverless/test/sdk.test.ts

@@ -533,7 +533,7 @@ describe('AWSLambda', () => {
         // @ts-expect-error just mocking around...
         expect(evtProcessor!(event).exception.values[0]?.mechanism).toEqual({
           handled: false,
-          type: 'auto.function.aws-serverless.handler',
+          type: 'auto.function.aws_serverless.handler',
         });
       }
     });
@@ -576,7 +576,7 @@ describe('AWSLambda', () => {
       // @ts-expect-error just mocking around...
       expect(evtProcessor(event).exception.values[0]?.mechanism).toEqual({
         handled: false,
-        type: 'auto.function.aws-serverless.handler',
+        type: 'auto.function.aws_serverless.handler',
       });
     }
   });