addressed comments

Author: wrisa

util/opentelemetry-util-genai/src/opentelemetry/util/genai/span_utils.py

@@ -111,9 +111,9 @@ def _get_embedding_span_name(invocation: EmbeddingInvocation) -> str:
 
 def _get_workflow_span_name(invocation: WorkflowInvocation) -> str:
     """Get the span name for an Workflow invocation."""
-    operation_name = getattr(invocation, "operation_name", None) or ""
-    name = getattr(invocation, "name", None) or ""
-    return f"{operation_name} {name}".strip()
+    operation_name = invocation.operation_name
+    name = invocation.name
+    return f"{operation_name} {name}" if name else operation_name
 
 
 def _get_llm_messages_attributes_for_span(

util/opentelemetry-util-genai/tests/test_handler_workflow.py

@@ -60,14 +60,23 @@ def test_start_workflow_creates_span(self) -> None:
 
     def test_start_workflow_span_name(self) -> None:
         invocation = WorkflowInvocation(
-            name="my_pipeline", operation_name="run_pipeline"
+            name="my_pipeline"
         )
         self.handler.start(invocation)
         self.handler.stop(invocation)
 
         spans = self._get_finished_spans()
         self.assertEqual(len(spans), 1)
-        self.assertEqual(spans[0].name, "run_pipeline my_pipeline")
+        self.assertEqual(spans[0].name, "invoke_workflow my_pipeline")
+
+    def test_start_workflow_span_name_without_name(self) -> None:
+        invocation = WorkflowInvocation()
+        self.handler.start(invocation)
+        self.handler.stop(invocation)
+
+        spans = self._get_finished_spans()
+        self.assertEqual(len(spans), 1)
+        self.assertEqual(spans[0].name, "invoke_workflow")
 
     def test_start_workflow_span_kind_is_internal(self) -> None:
         invocation = WorkflowInvocation(name="wf")