update with rollup bundle path

RulaKhaled · RulaKhaled · commit 78adfa8eba0c · 2025-10-13T14:50:05.000+02:00
diff --git a/dev-packages/browser-integration-tests/suites/tracing/ai-providers/anthropic/mocks.js b/dev-packages/browser-integration-tests/suites/tracing/ai-providers/anthropic/mocks.js
@@ -1,15 +1,11 @@
 // Mock Anthropic client for browser testing
 export class MockAnthropic {
   constructor(config) {
-    // eslint-disable-next-line no-console
-    console.log('[Mock Anthropic] Constructor called with config:', config);
     this.apiKey = config.apiKey;
 
     // Main focus: messages.create functionality
     this.messages = {
       create: async (...args) => {
-        // eslint-disable-next-line no-console
-        console.log('[Mock Anthropic] messages.create called with args:', args);
         const params = args[0];
         // Simulate processing time
         await new Promise(resolve => setTimeout(resolve, 10));
@@ -41,8 +37,6 @@ export class MockAnthropic {
             cache_read_input_tokens: 0,
           },
         };
-        // eslint-disable-next-line no-console
-        console.log('[Mock Anthropic] Returning response:', response);
         return response;
       },
       countTokens: async (..._args) => ({ id: 'mock', type: 'model', model: 'mock', input_tokens: 0 }),
diff --git a/dev-packages/browser-integration-tests/suites/tracing/ai-providers/anthropic/subject.js b/dev-packages/browser-integration-tests/suites/tracing/ai-providers/anthropic/subject.js
@@ -1,43 +1,22 @@
 import * as Sentry from '@sentry/browser';
 import { MockAnthropic } from './mocks.js';
 
-// eslint-disable-next-line no-console
-console.log('[Anthropic Test] Starting test...');
-
-// eslint-disable-next-line no-console
-console.log('[Anthropic Test] Creating mock client...');
 const mockClient = new MockAnthropic({
   apiKey: 'mock-api-key',
 });
 
-// eslint-disable-next-line no-console
-console.log('[Anthropic Test] Mock client created:', mockClient);
-
-// eslint-disable-next-line no-console
-console.log('[Anthropic Test] Instrumenting client with Sentry...');
 const client = Sentry.instrumentAnthropicAiClient(mockClient);
 
-// eslint-disable-next-line no-console
-console.log('[Anthropic Test] Client instrumented:', client);
-
 // Test that manual instrumentation doesn't crash the browser
 // The instrumentation automatically creates spans
-// eslint-disable-next-line no-console
-console.log('[Anthropic Test] Calling messages.create...');
 const response = await client.messages.create({
   model: 'claude-3-haiku-20240307',
   messages: [{ role: 'user', content: 'What is the capital of France?' }],
   temperature: 0.7,
   max_tokens: 100,
 });
 
-// eslint-disable-next-line no-console
-console.log('[Anthropic Test] Response received:', JSON.stringify(response));
+console.log("Received response", response)
 
-// eslint-disable-next-line no-console
-console.log('[Anthropic Test] Flushing Sentry...');
 // Ensure transaction is flushed in CI
 await Sentry.flush(2000);
-
-// eslint-disable-next-line no-console
-console.log('[Anthropic Test] Test completed!');
diff --git a/dev-packages/browser-integration-tests/suites/tracing/ai-providers/anthropic/test.ts b/dev-packages/browser-integration-tests/suites/tracing/ai-providers/anthropic/test.ts
@@ -8,38 +8,16 @@ import { envelopeRequestParser, waitForTransactionRequest } from '../../../../ut
 // and that gen_ai transactions are sent.
 
 sentryTest('manual Anthropic instrumentation sends gen_ai transactions', async ({ getLocalTestUrl, page }) => {
-  // Listen for console logs
-  page.on('console', msg => {
-    // eslint-disable-next-line no-console
-    console.log(`[Browser Console ${msg.type()}]`, msg.text());
-  });
-
-  // Listen for page errors
-  page.on('pageerror', error => {
-    // eslint-disable-next-line no-console
-    console.error('[Browser Error]', error);
-  });
-
   const transactionPromise = waitForTransactionRequest(page, event => {
-    // eslint-disable-next-line no-console
-    console.log('[Test] Received transaction event:', JSON.stringify(event, null, 2));
     return !!event.transaction?.includes('claude-3-haiku-20240307');
   });
 
   const url = await getLocalTestUrl({ testDir: __dirname });
-  // eslint-disable-next-line no-console
-  console.log('[Test] Navigating to URL:', url);
   await page.goto(url);
 
-  // eslint-disable-next-line no-console
-  console.log('[Test] Waiting for transaction...');
   const req = await transactionPromise;
-  // eslint-disable-next-line no-console
-  console.log('[Test] Transaction received!');
 
   const eventData = envelopeRequestParser(req);
-  // eslint-disable-next-line no-console
-  console.log('[Test] Parsed event data:', JSON.stringify(eventData, null, 2));
 
   // Verify it's a gen_ai transaction
   expect(eventData.transaction).toBe('messages claude-3-haiku-20240307');
diff --git a/dev-packages/browser-integration-tests/suites/tracing/ai-providers/google-genai/mocks.js b/dev-packages/browser-integration-tests/suites/tracing/ai-providers/google-genai/mocks.js
@@ -1,15 +1,11 @@
 // Mock Google GenAI client for browser testing
 export class MockGoogleGenAI {
   constructor(config) {
-    // eslint-disable-next-line no-console
-    console.log('[Mock Google GenAI] Constructor called with config:', config);
     this.apiKey = config.apiKey;
 
     // models.generateContent functionality
     this.models = {
       generateContent: async (...args) => {
-        // eslint-disable-next-line no-console
-        console.log('[Mock Google GenAI] models.generateContent called with args:', args);
         const params = args[0];
         // Simulate processing time
         await new Promise(resolve => setTimeout(resolve, 10));
@@ -65,16 +61,12 @@ export class MockGoogleGenAI {
     // chats.create implementation
     this.chats = {
       create: (...args) => {
-        // eslint-disable-next-line no-console
-        console.log('[Mock Google GenAI] chats.create called with args:', args);
         const params = args[0];
         const model = params.model;
 
         return {
           modelVersion: model,
           sendMessage: async (..._messageArgs) => {
-            // eslint-disable-next-line no-console
-            console.log('[Mock Google GenAI] chat.sendMessage called with args:', _messageArgs);
             // Simulate processing time
             await new Promise(resolve => setTimeout(resolve, 10));
 
@@ -100,8 +92,6 @@ export class MockGoogleGenAI {
               },
               modelVersion: model, // Include model version in response
             };
-            // eslint-disable-next-line no-console
-            console.log('[Mock Google GenAI] Returning response:', response);
             return response;
           },
           sendMessageStream: async () => {
diff --git a/dev-packages/browser-integration-tests/suites/tracing/ai-providers/google-genai/subject.js b/dev-packages/browser-integration-tests/suites/tracing/ai-providers/google-genai/subject.js
@@ -1,30 +1,15 @@
 import * as Sentry from '@sentry/browser';
 import { MockGoogleGenAI } from './mocks.js';
 
-// eslint-disable-next-line no-console
-console.log('[Google GenAI Test] Starting test...');
-
-// eslint-disable-next-line no-console
-console.log('[Google GenAI Test] Creating mock client...');
 const mockClient = new MockGoogleGenAI({
   apiKey: 'mock-api-key',
 });
 
-// eslint-disable-next-line no-console
-console.log('[Google GenAI Test] Mock client created:', mockClient);
-
-// eslint-disable-next-line no-console
-console.log('[Google GenAI Test] Instrumenting client with Sentry...');
 const client = Sentry.instrumentGoogleGenAIClient(mockClient);
 
-// eslint-disable-next-line no-console
-console.log('[Google GenAI Test] Client instrumented:', client);
-
 // Test that manual instrumentation doesn't crash the browser
 // The instrumentation automatically creates spans
 // Test both chats and models APIs
-// eslint-disable-next-line no-console
-console.log('[Google GenAI Test] Creating chat...');
 const chat = client.chats.create({
   model: 'gemini-1.5-pro',
   config: {
@@ -40,19 +25,11 @@ const chat = client.chats.create({
   ],
 });
 
-// eslint-disable-next-line no-console
-console.log('[Google GenAI Test] Sending message...');
 const response = await chat.sendMessage({
   message: 'Tell me a joke',
 });
 
-// eslint-disable-next-line no-console
-console.log('[Google GenAI Test] Response received:', JSON.stringify(response));
+console.log("Received response", response)
 
-// eslint-disable-next-line no-console
-console.log('[Google GenAI Test] Flushing Sentry...');
 // Ensure transaction is flushed in CI
 await Sentry.flush(2000);
-
-// eslint-disable-next-line no-console
-console.log('[Google GenAI Test] Test completed!');
diff --git a/dev-packages/browser-integration-tests/suites/tracing/ai-providers/google-genai/test.ts b/dev-packages/browser-integration-tests/suites/tracing/ai-providers/google-genai/test.ts
@@ -8,38 +8,16 @@ import { envelopeRequestParser, waitForTransactionRequest } from '../../../../ut
 // and that gen_ai transactions are sent.
 
 sentryTest('manual Google GenAI instrumentation sends gen_ai transactions', async ({ getLocalTestUrl, page }) => {
-  // Listen for console logs
-  page.on('console', msg => {
-    // eslint-disable-next-line no-console
-    console.log(`[Browser Console ${msg.type()}]`, msg.text());
-  });
-
-  // Listen for page errors
-  page.on('pageerror', error => {
-    // eslint-disable-next-line no-console
-    console.error('[Browser Error]', error);
-  });
-
   const transactionPromise = waitForTransactionRequest(page, event => {
-    // eslint-disable-next-line no-console
-    console.log('[Test] Received transaction event:', JSON.stringify(event, null, 2));
     return !!event.transaction?.includes('gemini-1.5-pro');
   });
 
   const url = await getLocalTestUrl({ testDir: __dirname });
-  // eslint-disable-next-line no-console
-  console.log('[Test] Navigating to URL:', url);
   await page.goto(url);
 
-  // eslint-disable-next-line no-console
-  console.log('[Test] Waiting for transaction...');
   const req = await transactionPromise;
-  // eslint-disable-next-line no-console
-  console.log('[Test] Transaction received!');
 
   const eventData = envelopeRequestParser(req);
-  // eslint-disable-next-line no-console
-  console.log('[Test] Parsed event data:', JSON.stringify(eventData, null, 2));
 
   // Verify it's a gen_ai transaction
   expect(eventData.transaction).toBe('chat gemini-1.5-pro create');
diff --git a/dev-packages/browser-integration-tests/suites/tracing/ai-providers/openai/mocks.js b/dev-packages/browser-integration-tests/suites/tracing/ai-providers/openai/mocks.js
@@ -1,15 +1,11 @@
 // Mock OpenAI client for browser testing
 export class MockOpenAi {
   constructor(config) {
-    // eslint-disable-next-line no-console
-    console.log('[Mock OpenAI] Constructor called with config:', config);
     this.apiKey = config.apiKey;
 
     this.chat = {
       completions: {
         create: async (...args) => {
-          // eslint-disable-next-line no-console
-          console.log('[Mock OpenAI] chat.completions.create called with args:', args);
           const params = args[0];
           // Simulate processing time
           await new Promise(resolve => setTimeout(resolve, 10));
@@ -43,8 +39,6 @@ export class MockOpenAi {
               total_tokens: 25,
             },
           };
-          // eslint-disable-next-line no-console
-          console.log('[Mock OpenAI] Returning response:', response);
           return response;
         },
       },
diff --git a/dev-packages/browser-integration-tests/suites/tracing/ai-providers/openai/subject.js b/dev-packages/browser-integration-tests/suites/tracing/ai-providers/openai/subject.js
@@ -1,29 +1,14 @@
 import * as Sentry from '@sentry/browser';
 import { MockOpenAi } from './mocks.js';
 
-// eslint-disable-next-line no-console
-console.log('[OpenAI Test] Starting test...');
-
-// eslint-disable-next-line no-console
-console.log('[OpenAI Test] Creating mock client...');
 const mockClient = new MockOpenAi({
   apiKey: 'mock-api-key',
 });
 
-// eslint-disable-next-line no-console
-console.log('[OpenAI Test] Mock client created:', mockClient);
-
-// eslint-disable-next-line no-console
-console.log('[OpenAI Test] Instrumenting client with Sentry...');
 const client = Sentry.instrumentOpenAiClient(mockClient);
 
-// eslint-disable-next-line no-console
-console.log('[OpenAI Test] Client instrumented:', client);
-
 // Test that manual instrumentation doesn't crash the browser
 // The instrumentation automatically creates spans
-// eslint-disable-next-line no-console
-console.log('[OpenAI Test] Calling chat.completions.create...');
 const response = await client.chat.completions.create({
   model: 'gpt-3.5-turbo',
   messages: [
@@ -34,13 +19,7 @@ const response = await client.chat.completions.create({
   max_tokens: 100,
 });
 
-// eslint-disable-next-line no-console
-console.log('[OpenAI Test] Response received:', JSON.stringify(response));
+console.log("Received response", response)
 
-// eslint-disable-next-line no-console
-console.log('[OpenAI Test] Flushing Sentry...');
 // Ensure transaction is flushed in CI
 await Sentry.flush(2000);
-
-// eslint-disable-next-line no-console
-console.log('[OpenAI Test] Test completed!');
diff --git a/dev-packages/browser-integration-tests/suites/tracing/ai-providers/openai/test.ts b/dev-packages/browser-integration-tests/suites/tracing/ai-providers/openai/test.ts
@@ -8,38 +8,16 @@ import { envelopeRequestParser, waitForTransactionRequest } from '../../../../ut
 // and that gen_ai transactions are sent.
 
 sentryTest('manual OpenAI instrumentation sends gen_ai transactions', async ({ getLocalTestUrl, page }) => {
-  // Listen for console logs
-  page.on('console', msg => {
-    // eslint-disable-next-line no-console
-    console.log(`[Browser Console ${msg.type()}]`, msg.text());
-  });
-
-  // Listen for page errors
-  page.on('pageerror', error => {
-    // eslint-disable-next-line no-console
-    console.error('[Browser Error]', error);
-  });
-
   const transactionPromise = waitForTransactionRequest(page, event => {
-    // eslint-disable-next-line no-console
-    console.log('[Test] Received transaction event:', JSON.stringify(event, null, 2));
     return !!event.transaction?.includes('gpt-3.5-turbo');
   });
 
   const url = await getLocalTestUrl({ testDir: __dirname });
-  // eslint-disable-next-line no-console
-  console.log('[Test] Navigating to URL:', url);
   await page.goto(url);
 
-  // eslint-disable-next-line no-console
-  console.log('[Test] Waiting for transaction...');
   const req = await transactionPromise;
-  // eslint-disable-next-line no-console
-  console.log('[Test] Transaction received!');
 
   const eventData = envelopeRequestParser(req);
-  // eslint-disable-next-line no-console
-  console.log('[Test] Parsed event data:', JSON.stringify(eventData, null, 2));
 
   // Verify it's a gen_ai transaction
   expect(eventData.transaction).toBe('chat gpt-3.5-turbo');
diff --git a/packages/browser/rollup.bundle.config.mjs b/packages/browser/rollup.bundle.config.mjs
@@ -13,6 +13,9 @@ const reexportedPluggableIntegrationFiles = [
   'modulemetadata',
   'graphqlclient',
   'spotlight',
+  'instrumentanthropicaiclient',
+  'instrumentopenaiclient',
+  'instrumentgooglegenaiclient',
 ];
 
 browserPluggableIntegrationFiles.forEach(integrationName => {
