From 62d734fefd50fb7c6c2d8972a56011ad3d64741c Mon Sep 17 00:00:00 2001 From: Nicolas Hrubec Date: Thu, 19 Mar 2026 13:34:11 +0100 Subject: [PATCH 1/2] fix(serverless): Add node to metadata --- packages/aws-serverless/src/init.ts | 2 +- packages/aws-serverless/test/init.test.ts | 22 +++++++++++++++++++ packages/google-cloud-serverless/src/sdk.ts | 2 +- .../google-cloud-serverless/test/sdk.test.ts | 4 ++++ 4 files changed, 28 insertions(+), 2 deletions(-) diff --git a/packages/aws-serverless/src/init.ts b/packages/aws-serverless/src/init.ts index 25180a41f6e6..fe069d7ff6f5 100644 --- a/packages/aws-serverless/src/init.ts +++ b/packages/aws-serverless/src/init.ts @@ -109,7 +109,7 @@ export function init(options: AwsServerlessOptions = {}): NodeClient | undefined } } - applySdkMetadata(opts, 'aws-serverless', ['aws-serverless'], sdkSource); + applySdkMetadata(opts, 'aws-serverless', ['aws-serverless', 'node'], sdkSource); return initWithoutDefaultIntegrations(opts); } diff --git a/packages/aws-serverless/test/init.test.ts b/packages/aws-serverless/test/init.test.ts index 500338dc7144..f73a51828940 100644 --- a/packages/aws-serverless/test/init.test.ts +++ b/packages/aws-serverless/test/init.test.ts @@ -28,6 +28,28 @@ describe('init', () => { delete process.env.SENTRY_LAYER_EXTENSION; }); + describe('sdk metadata', () => { + test('should include aws-serverless and node packages in metadata', () => { + mockGetSDKSource.mockReturnValue('npm'); + init({}); + + expect(mockInitWithoutDefaultIntegrations).toHaveBeenCalledWith( + expect.objectContaining({ + _metadata: { + sdk: { + name: 'sentry.javascript.aws-serverless', + packages: [ + { name: 'npm:@sentry/aws-serverless', version: expect.any(String) }, + { name: 'npm:@sentry/node', version: expect.any(String) }, + ], + version: expect.any(String), + }, + }, + }), + ); + }); + }); + describe('Lambda extension setup', () => { test('should preserve user-provided tunnel option when Lambda extension is enabled', () => { mockGetSDKSource.mockReturnValue('aws-lambda-layer'); diff --git a/packages/google-cloud-serverless/src/sdk.ts b/packages/google-cloud-serverless/src/sdk.ts index 1161ab60300e..6eb80aed2f64 100644 --- a/packages/google-cloud-serverless/src/sdk.ts +++ b/packages/google-cloud-serverless/src/sdk.ts @@ -29,7 +29,7 @@ export function init(options: NodeOptions = {}): NodeClient | undefined { ...options, }; - applySdkMetadata(opts, 'google-cloud-serverless'); + applySdkMetadata(opts, 'google-cloud-serverless', ['google-cloud-serverless', 'node']); return initNode(opts); } diff --git a/packages/google-cloud-serverless/test/sdk.test.ts b/packages/google-cloud-serverless/test/sdk.test.ts index 5b41bc4abb21..553911ead646 100644 --- a/packages/google-cloud-serverless/test/sdk.test.ts +++ b/packages/google-cloud-serverless/test/sdk.test.ts @@ -31,6 +31,10 @@ describe('init()', () => { name: 'npm:@sentry/google-cloud-serverless', version: expect.any(String), }, + { + name: 'npm:@sentry/node', + version: expect.any(String), + }, ], version: expect.any(String), }, From 9fbc1b979b6b3f2b3f7b0f1b0ead5f83395b6a30 Mon Sep 17 00:00:00 2001 From: Nicolas Hrubec Date: Thu, 19 Mar 2026 13:37:54 +0100 Subject: [PATCH 2/2] . --- packages/aws-serverless/test/init.test.ts | 22 ---------------------- packages/aws-serverless/test/sdk.test.ts | 4 ++++ 2 files changed, 4 insertions(+), 22 deletions(-) diff --git a/packages/aws-serverless/test/init.test.ts b/packages/aws-serverless/test/init.test.ts index f73a51828940..500338dc7144 100644 --- a/packages/aws-serverless/test/init.test.ts +++ b/packages/aws-serverless/test/init.test.ts @@ -28,28 +28,6 @@ describe('init', () => { delete process.env.SENTRY_LAYER_EXTENSION; }); - describe('sdk metadata', () => { - test('should include aws-serverless and node packages in metadata', () => { - mockGetSDKSource.mockReturnValue('npm'); - init({}); - - expect(mockInitWithoutDefaultIntegrations).toHaveBeenCalledWith( - expect.objectContaining({ - _metadata: { - sdk: { - name: 'sentry.javascript.aws-serverless', - packages: [ - { name: 'npm:@sentry/aws-serverless', version: expect.any(String) }, - { name: 'npm:@sentry/node', version: expect.any(String) }, - ], - version: expect.any(String), - }, - }, - }), - ); - }); - }); - describe('Lambda extension setup', () => { test('should preserve user-provided tunnel option when Lambda extension is enabled', () => { mockGetSDKSource.mockReturnValue('aws-lambda-layer'); diff --git a/packages/aws-serverless/test/sdk.test.ts b/packages/aws-serverless/test/sdk.test.ts index c0f508eb3859..aa16049e2e2b 100644 --- a/packages/aws-serverless/test/sdk.test.ts +++ b/packages/aws-serverless/test/sdk.test.ts @@ -618,6 +618,10 @@ describe('AWSLambda', () => { name: 'npm:@sentry/aws-serverless', version: expect.any(String), }, + { + name: 'npm:@sentry/node', + version: expect.any(String), + }, ], version: expect.any(String), },