test: Drop tool definitions from the genAI-disabled openai assertions

Resolve the recording options in `instrumentStateGraphCompile` too, so the
exported entry point keeps the `dataCollection` defaults.

Co-Authored-By: Claude Opus 5 <noreply@anthropic.com>

Author: s1gr1d

dev-packages/node-integration-tests/suites/tracing/openai/openai-tool-calls/test.ts

@@ -105,10 +105,8 @@ describe('OpenAI Tool Calls integration', () => {
               type: 'string',
               value: 'gpt-4',
             });
-            expect(chatToolsSpan!.attributes[GEN_AI_TOOL_DEFINITIONS]).toEqual({
-              type: 'string',
-              value: WEATHER_TOOL_DEFINITION,
-            });
+            // Tool definitions are gen AI input data, so `genAI.inputs: false` drops them.
+            expect(chatToolsSpan!.attributes[GEN_AI_TOOL_DEFINITIONS]).toBeUndefined();
             expect(chatToolsSpan!.attributes[GEN_AI_RESPONSE_MODEL]).toEqual({
               type: 'string',
               value: 'gpt-4',
@@ -164,10 +162,7 @@ describe('OpenAI Tool Calls integration', () => {
               type: 'boolean',
               value: true,
             });
-            expect(streamingChatToolsSpan!.attributes[GEN_AI_TOOL_DEFINITIONS]).toEqual({
-              type: 'string',
-              value: WEATHER_TOOL_DEFINITION,
-            });
+            expect(streamingChatToolsSpan!.attributes[GEN_AI_TOOL_DEFINITIONS]).toBeUndefined();
             expect(streamingChatToolsSpan!.attributes[GEN_AI_RESPONSE_MODEL]).toEqual({
               type: 'string',
               value: 'gpt-4',
@@ -224,10 +219,7 @@ describe('OpenAI Tool Calls integration', () => {
               value: 'gpt-4',
             });
             expect(responsesToolsSpan!.attributes[GEN_AI_REQUEST_STREAM_ATTRIBUTE]).toBeUndefined();
-            expect(responsesToolsSpan!.attributes[GEN_AI_TOOL_DEFINITIONS]).toEqual({
-              type: 'string',
-              value: WEATHER_TOOL_DEFINITION,
-            });
+            expect(responsesToolsSpan!.attributes[GEN_AI_TOOL_DEFINITIONS]).toBeUndefined();
             expect(responsesToolsSpan!.attributes[GEN_AI_RESPONSE_MODEL]).toEqual({
               type: 'string',
               value: 'gpt-4',
@@ -283,10 +275,7 @@ describe('OpenAI Tool Calls integration', () => {
               type: 'boolean',
               value: true,
             });
-            expect(streamingResponsesToolsSpan!.attributes[GEN_AI_TOOL_DEFINITIONS]).toEqual({
-              type: 'string',
-              value: WEATHER_TOOL_DEFINITION,
-            });
+            expect(streamingResponsesToolsSpan!.attributes[GEN_AI_TOOL_DEFINITIONS]).toBeUndefined();
             expect(streamingResponsesToolsSpan!.attributes[GEN_AI_RESPONSE_MODEL]).toEqual({
               type: 'string',
               value: 'gpt-4',

packages/server-utils/src/ai/langgraph/index.ts

@@ -37,12 +37,15 @@ const SENTRY_PATCHED = '__sentry_patched__';
  */
 export function instrumentStateGraphCompile(
   originalCompile: (...args: unknown[]) => CompiledGraph,
-  options: LangGraphOptions,
+  rawOptions: LangGraphOptions,
 ): (...args: unknown[]) => CompiledGraph {
   if (Object.prototype.hasOwnProperty.call(originalCompile, SENTRY_PATCHED)) {
     return originalCompile;
   }
 
+  // This is exported, so callers can hand us an options object with no recording flags set. Resolving
+  // here (rather than only in `instrumentStateGraph`) keeps that path on the `dataCollection` defaults.
+  const options = resolveAIRecordingOptions(rawOptions);
   const sentryHandler = createLangChainCallbackHandler(options);
 
   const wrapped = new Proxy(originalCompile, {