ref(nextjs): Remove metadata builder class (#4263)

* ref(nextjs): Remove metadata builder class

Refactor the class into functions to save on bundle size. Extracted from
changes in #4196.

* add tests for nextjs metadata

Author: AbhiPrasad

Diff for: packages/nextjs/src/index.client.ts

@@ -2,7 +2,7 @@ import { configureScope, init as reactInit, Integrations as BrowserIntegrations
 import { BrowserTracing, defaultRequestInstrumentationOptions } from '@sentry/tracing';
 
 import { nextRouterInstrumentation } from './performance/client';
-import { MetadataBuilder } from './utils/metadataBuilder';
+import { buildMetadata } from './utils/metadata';
 import { NextjsOptions } from './utils/nextjsOptions';
 import { addIntegration, UserIntegrations } from './utils/userIntegrations';
 
@@ -13,8 +13,7 @@ export const Integrations = { ...BrowserIntegrations, BrowserTracing };
 
 /** Inits the Sentry NextJS SDK on the browser with the React SDK. */
 export function init(options: NextjsOptions): void {
-  const metadataBuilder = new MetadataBuilder(options, ['nextjs', 'react']);
-  metadataBuilder.addSdkMetadata();
+  buildMetadata(options, ['nextjs', 'react']);
   options.environment = options.environment || process.env.NODE_ENV;
 
   // Only add BrowserTracing if a tracesSampleRate or tracesSampler is set

Diff for: packages/nextjs/src/index.server.ts

@@ -6,7 +6,7 @@ import { escapeStringForRegex, logger } from '@sentry/utils';
 import * as domainModule from 'domain';
 import * as path from 'path';
 
-import { MetadataBuilder } from './utils/metadataBuilder';
+import { buildMetadata } from './utils/metadata';
 import { NextjsOptions } from './utils/nextjsOptions';
 import { addIntegration } from './utils/userIntegrations';
 
@@ -43,8 +43,7 @@ export function init(options: NextjsOptions): void {
     return;
   }
 
-  const metadataBuilder = new MetadataBuilder(options, ['nextjs', 'node']);
-  metadataBuilder.addSdkMetadata();
+  buildMetadata(options, ['nextjs', 'node']);
   options.environment = options.environment || process.env.NODE_ENV;
   addServerIntegrations(options);
   // Right now we only capture frontend sessions for Next.js

Diff for: packages/nextjs/src/utils/metadata.ts

@@ -0,0 +1,23 @@
+import { SDK_VERSION } from '@sentry/core';
+import { Options, SdkInfo } from '@sentry/types';
+
+const PACKAGE_NAME_PREFIX = 'npm:@sentry/';
+
+/**
+ * A builder for the SDK metadata in the options for the SDK initialization.
+ * @param options sdk options object that gets mutated
+ * @param names list of package names
+ */
+export function buildMetadata(options: Options, names: string[]): void {
+  options._metadata = options._metadata || {};
+  options._metadata.sdk =
+    options._metadata.sdk ||
+    ({
+      name: 'sentry.javascript.nextjs',
+      packages: names.map(name => ({
+        name: `${PACKAGE_NAME_PREFIX}${name}`,
+        version: SDK_VERSION,
+      })),
+      version: SDK_VERSION,
+    } as SdkInfo);
+}

Diff for: packages/nextjs/src/utils/metadataBuilder.ts

@@ -32,7 +32,16 @@ describe('Client init()', () => {
           sdk: {
             name: 'sentry.javascript.nextjs',
             version: expect.any(String),
-            packages: expect.any(Array),
+            packages: [
+              {
+                name: 'npm:@sentry/nextjs',
+                version: expect.any(String),
+              },
+              {
+                name: 'npm:@sentry/react',
+                version: expect.any(String),
+              },
+            ],
           },
         },
         environment: 'test',

Diff for: packages/nextjs/test/index.client.test.ts

@@ -37,7 +37,16 @@ describe('Server init()', () => {
           sdk: {
             name: 'sentry.javascript.nextjs',
             version: expect.any(String),
-            packages: expect.any(Array),
+            packages: [
+              {
+                name: 'npm:@sentry/nextjs',
+                version: expect.any(String),
+              },
+              {
+                name: 'npm:@sentry/node',
+                version: expect.any(String),
+              },
+            ],
           },
         },
         autoSessionTracking: false,