Flip enableTruncation default to false

Author: andreiborza

dev-packages/node-integration-tests/suites/tracing/anthropic/scenario-media-truncation.mjs

@@ -46,7 +46,7 @@ async function run() {
       apiKey: 'mock-api-key',
     });
 
-    const client = instrumentAnthropicAiClient(mockClient);
+    const client = instrumentAnthropicAiClient(mockClient, { enableTruncation: true });
 
     // Send the image showing the number 3
     // Put the image in the last message so it doesn't get dropped

dev-packages/node-integration-tests/suites/tracing/anthropic/scenario-message-truncation.mjs

@@ -46,7 +46,7 @@ async function run() {
       apiKey: 'mock-api-key',
     });
 
-    const client = instrumentAnthropicAiClient(mockClient);
+    const client = instrumentAnthropicAiClient(mockClient, { enableTruncation: true });
 
     // Test 1: Given an array of messages only the last message should be kept
     // The last message should be truncated to fit within the 20KB limit

dev-packages/node-integration-tests/suites/tracing/anthropic/scenario-system-instructions.mjs

@@ -43,11 +43,18 @@ async function run() {
       baseURL: `http://localhost:${server.address().port}/anthropic`,
     });
 
+    // Multiple long messages verify default-off truncation: with `enableTruncation` unset,
+    // neither byte-truncation nor message popping should occur. The full array is preserved as-is.
+    const longContent = 'A'.repeat(50_000);
     await client.messages.create({
       model: 'claude-3-5-sonnet-20241022',
       max_tokens: 1024,
       system: 'You are a helpful assistant',
-      messages: [{ role: 'user', content: 'Hello' }],
+      messages: [
+        { role: 'user', content: longContent },
+        { role: 'assistant', content: 'Some reply' },
+        { role: 'user', content: 'Follow-up question' },
+      ],
     });
   });
 

dev-packages/node-integration-tests/suites/tracing/anthropic/test.ts

@@ -671,8 +671,13 @@ describe('Anthropic integration', () => {
     'scenario-system-instructions.mjs',
     'instrument-with-pii.mjs',
     (createRunner, test) => {
-      test('extracts system instructions from messages', async () => {
+      test('extracts system instructions and preserves full multi-message input by default (enableTruncation unset)', async () => {
         const expectedInstructions = JSON.stringify([{ type: 'text', content: 'You are a helpful assistant' }]);
+        const expectedMessages = JSON.stringify([
+          { role: 'user', content: 'A'.repeat(50_000) },
+          { role: 'assistant', content: 'Some reply' },
+          { role: 'user', content: 'Follow-up question' },
+        ]);
         await createRunner()
           .ignore('event')
           .expect({
@@ -687,6 +692,9 @@ describe('Anthropic integration', () => {
 
               // [0] messages.create — system instructions extracted into dedicated attribute
               expect(firstSpan!.attributes[GEN_AI_SYSTEM_INSTRUCTIONS_ATTRIBUTE].value).toBe(expectedInstructions);
+              // Default-off: no byte-truncation of the 50KB message and no message popping to keep-last-only.
+              expect(firstSpan!.attributes[GEN_AI_INPUT_MESSAGES_ATTRIBUTE].value).toBe(expectedMessages);
+              expect(firstSpan!.attributes[GEN_AI_INPUT_MESSAGES_ORIGINAL_LENGTH_ATTRIBUTE].value).toBe(3);
             },
           })
           .start()

dev-packages/node-integration-tests/suites/tracing/google-genai/scenario-message-truncation.mjs

@@ -41,7 +41,7 @@ async function run() {
       apiKey: 'mock-api-key',
     });
 
-    const client = instrumentGoogleGenAIClient(mockClient);
+    const client = instrumentGoogleGenAIClient(mockClient, { enableTruncation: true });
 
     // Test 1: Given an array of messages only the last message should be kept
     // The last message should be truncated to fit within the 20KB limit

dev-packages/node-integration-tests/suites/tracing/google-genai/scenario-system-instructions.mjs

@@ -37,12 +37,19 @@ async function run() {
     const mockClient = new MockGoogleGenAI({ apiKey: 'mock-api-key' });
     const client = instrumentGoogleGenAIClient(mockClient);
 
+    // Multiple long messages verify default-off truncation: with `enableTruncation` unset,
+    // neither byte-truncation nor message popping should occur. The full array is preserved as-is.
+    const longContent = 'A'.repeat(50_000);
     await client.models.generateContent({
       model: 'gemini-1.5-flash',
       config: {
         systemInstruction: 'You are a helpful assistant',
       },
-      contents: [{ role: 'user', parts: [{ text: 'Hello' }] }],
+      contents: [
+        { role: 'user', parts: [{ text: longContent }] },
+        { role: 'model', parts: [{ text: 'Some reply' }] },
+        { role: 'user', parts: [{ text: 'Follow-up question' }] },
+      ],
     });
   });
 }

dev-packages/node-integration-tests/suites/tracing/google-genai/test.ts

@@ -344,7 +344,12 @@ describe('Google GenAI integration', () => {
     'scenario-system-instructions.mjs',
     'instrument-with-pii.mjs',
     (createRunner, test) => {
-      test('extracts system instructions from messages', async () => {
+      test('extracts system instructions and preserves full multi-message input by default (enableTruncation unset)', async () => {
+        const expectedMessages = JSON.stringify([
+          { role: 'user', parts: [{ text: 'A'.repeat(50_000) }] },
+          { role: 'model', parts: [{ text: 'Some reply' }] },
+          { role: 'user', parts: [{ text: 'Follow-up question' }] },
+        ]);
         await createRunner()
           .ignore('event')
           .expect({ transaction: { transaction: 'main' } })
@@ -358,6 +363,9 @@ describe('Google GenAI integration', () => {
               expect(firstSpan!.attributes[GEN_AI_SYSTEM_INSTRUCTIONS_ATTRIBUTE].value).toBe(
                 JSON.stringify([{ type: 'text', content: 'You are a helpful assistant' }]),
               );
+              // Default-off: no byte-truncation of the 50KB message and no message popping to keep-last-only.
+              expect(firstSpan!.attributes[GEN_AI_INPUT_MESSAGES_ATTRIBUTE].value).toBe(expectedMessages);
+              expect(firstSpan!.attributes[GEN_AI_INPUT_MESSAGES_ORIGINAL_LENGTH_ATTRIBUTE].value).toBe(3);
             },
           })
           .start()

dev-packages/node-integration-tests/suites/tracing/langchain/instrument-with-pii.mjs

@@ -7,6 +7,11 @@ Sentry.init({
   tracesSampleRate: 1.0,
   sendDefaultPii: true,
   transport: loggingTransport,
+  integrations: [
+    Sentry.langChainIntegration({
+      enableTruncation: true,
+    }),
+  ],
   beforeSendTransaction: event => {
     // Filter out mock express server transactions
     if (event.transaction.includes('/v1/messages') || event.transaction.includes('/v1/embeddings')) {

dev-packages/node-integration-tests/suites/tracing/langchain/v1/instrument-with-pii.mjs

@@ -7,6 +7,11 @@ Sentry.init({
   tracesSampleRate: 1.0,
   sendDefaultPii: true,
   transport: loggingTransport,
+  integrations: [
+    Sentry.langChainIntegration({
+      enableTruncation: true,
+    }),
+  ],
   beforeSendTransaction: event => {
     // Filter out mock express server transactions
     if (event.transaction.includes('/v1/messages') || event.transaction.includes('/v1/chat/completions')) {

dev-packages/node-integration-tests/suites/tracing/langgraph/scenario-system-instructions.mjs

@@ -29,10 +29,15 @@ async function run() {
       .addEdge('agent', END)
       .compile({ name: 'test-agent' });
 
+    // Multiple long messages verify default-off truncation: with `enableTruncation` unset,
+    // neither byte-truncation nor message popping should occur. The full array is preserved as-is.
+    const longContent = 'A'.repeat(50_000);
     await graph.invoke({
       messages: [
         { role: 'system', content: 'You are a helpful assistant' },
-        { role: 'user', content: 'Hello' },
+        { role: 'user', content: longContent },
+        { role: 'assistant', content: 'Some reply' },
+        { role: 'user', content: 'Follow-up question' },
       ],
     });
   });