chore: allow custom agent name (strands-agents#347)

Author: poshinchen

src/strands/agent/agent.py

@@ -56,6 +56,7 @@ class _DefaultCallbackHandlerSentinel:
 
 
 _DEFAULT_CALLBACK_HANDLER = _DefaultCallbackHandlerSentinel()
+_DEFAULT_AGENT_NAME = "Strands Agents"
 
 
 class Agent:
@@ -320,7 +321,7 @@ def __init__(
             self.state = AgentState()
 
         self.tool_caller = Agent.ToolCaller(self)
-        self.name = name
+        self.name = name or _DEFAULT_AGENT_NAME
         self.description = description
 
         self._hooks = HookRegistry()
@@ -710,6 +711,7 @@ def _start_agent_trace_span(self, prompt: str) -> None:
 
         self.trace_span = self.tracer.start_agent_span(
             prompt=prompt,
+            agent_name=self.name,
             model_id=model_id,
             tools=self.tool_names,
             system_prompt=self.system_prompt,

src/strands/telemetry/tracer.py

@@ -406,7 +406,7 @@ def end_event_loop_cycle_span(
     def start_agent_span(
         self,
         prompt: str,
-        agent_name: str = "Strands Agent",
+        agent_name: str,
         model_id: Optional[str] = None,
         tools: Optional[list] = None,
         custom_trace_attributes: Optional[Mapping[str, AttributeValue]] = None,

tests/strands/agent/test_agent.py

@@ -1231,6 +1231,7 @@ def test_agent_call_creates_and_ends_span_on_success(mock_get_tracer, mock_model
     # Verify span was created
     mock_tracer.start_agent_span.assert_called_once_with(
         prompt="test prompt",
+        agent_name="Strands Agents",
         model_id=unittest.mock.ANY,
         tools=agent.tool_names,
         system_prompt=agent.system_prompt,
@@ -1264,6 +1265,7 @@ async def test_event_loop(*args, **kwargs):
     # Verify span was created
     mock_tracer.start_agent_span.assert_called_once_with(
         prompt="test prompt",
+        agent_name="Strands Agents",
         model_id=unittest.mock.ANY,
         tools=agent.tool_names,
         system_prompt=agent.system_prompt,
@@ -1301,6 +1303,7 @@ def test_agent_call_creates_and_ends_span_on_exception(mock_get_tracer, mock_mod
     # Verify span was created
     mock_tracer.start_agent_span.assert_called_once_with(
         prompt="test prompt",
+        agent_name="Strands Agents",
         model_id=unittest.mock.ANY,
         tools=agent.tool_names,
         system_prompt=agent.system_prompt,
@@ -1336,6 +1339,7 @@ async def test_agent_stream_async_creates_and_ends_span_on_exception(mock_get_tr
     # Verify span was created
     mock_tracer.start_agent_span.assert_called_once_with(
         prompt="test prompt",
+        agent_name="Strands Agents",
         model_id=unittest.mock.ANY,
         tools=agent.tool_names,
         system_prompt=agent.system_prompt,