ref(openai-agents): Stop setting gen_ai.request.available_tools on Invoke Agent spans (#6955)

The attribute is not well-defined for agents as different tools can be available in an agent's lifetime.

Author: alexander-alderman-webb

sentry_sdk/integrations/openai_agents/spans/invoke_agent.py

@@ -38,8 +38,6 @@ def invoke_agent_span(
                 SPANDATA.GEN_AI_OPERATION_NAME: "invoke_agent",
             },
         )
-
-        set_on_span = span.set_attribute
     else:
         start_span_function = get_start_span_function()
         span = start_span_function(
@@ -51,8 +49,6 @@ def invoke_agent_span(
 
         span.set_data(SPANDATA.GEN_AI_OPERATION_NAME, "invoke_agent")
 
-        set_on_span = span.set_data
-
     if should_send_default_pii():
         messages = []
         if agent.instructions:
@@ -101,12 +97,6 @@ def invoke_agent_span(
 
     _set_agent_data(span, agent)
 
-    if len(agent.tools) > 0:
-        set_on_span(
-            SPANDATA.GEN_AI_REQUEST_AVAILABLE_TOOLS,
-            safe_serialize([vars(tool) for tool in agent.tools]),
-        )
-
     return span
 
 

tests/integrations/openai_agents/test_openai_agents.py

@@ -1998,12 +1998,6 @@ def simple_test_tool(message: str) -> str:
         assert agent_span["attributes"]["gen_ai.agent.name"] == "test_agent"
         assert agent_span["attributes"]["gen_ai.operation.name"] == "invoke_agent"
 
-        agent_span_available_tool = json.loads(
-            agent_span["attributes"]["gen_ai.request.available_tools"]
-        )[0]
-
-        assert all(agent_span_available_tool[k] == v for k, v in available_tool.items())
-
         assert agent_span["attributes"]["gen_ai.request.max_tokens"] == 100
         assert agent_span["attributes"]["gen_ai.request.model"] == "gpt-4"
         assert agent_span["attributes"]["gen_ai.request.temperature"] == 0.7
@@ -2224,12 +2218,6 @@ def simple_test_tool(message: str) -> str:
         assert agent_span["attributes"]["gen_ai.agent.name"] == "test_agent"
         assert agent_span["attributes"]["gen_ai.operation.name"] == "invoke_agent"
 
-        agent_span_available_tool = json.loads(
-            agent_span["attributes"]["gen_ai.request.available_tools"]
-        )[0]
-
-        assert all(agent_span_available_tool[k] == v for k, v in available_tool.items())
-
         assert agent_span["attributes"]["gen_ai.request.max_tokens"] == 100
         assert agent_span["attributes"]["gen_ai.request.model"] == "gpt-4"
         assert agent_span["attributes"]["gen_ai.request.temperature"] == 0.7
@@ -2444,11 +2432,6 @@ def simple_test_tool(message: str) -> str:
         assert agent_span["data"]["gen_ai.agent.name"] == "test_agent"
         assert agent_span["data"]["gen_ai.operation.name"] == "invoke_agent"
 
-        agent_span_available_tool = json.loads(
-            agent_span["data"]["gen_ai.request.available_tools"]
-        )[0]
-        assert all(agent_span_available_tool[k] == v for k, v in available_tool.items())
-
         assert agent_span["data"]["gen_ai.request.max_tokens"] == 100
         assert agent_span["data"]["gen_ai.request.model"] == "gpt-4"
         assert agent_span["data"]["gen_ai.request.temperature"] == 0.7