more lint

Author: lmolkova

dev-requirements.txt

@@ -1,6 +1,6 @@
 pylint==3.0.2
 httpretty==1.1.4
-pyright==v1.1.408
+pyright==v1.1.404
 sphinx==7.1.2
 sphinx-rtd-theme==2.0.0rc4
 sphinx-autodoc-typehints==1.25.2

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

@@ -259,10 +259,6 @@ class GenAIInvocation(ABC):
     start_workflow, start_tool) rather than constructing invocations directly.
     """
 
-    @property
-    def operation_name(self) -> str:
-        return self._operation_name
-
     def __init__(
         self,
         # Individual components instead of TelemetryHandler to avoid a circular

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

@@ -47,13 +47,6 @@ class TelemetryHandlerWorkflowTest(_WorkflowTestBase):
     # start_workflow
     # ------------------------------------------------------------------
 
-    def test_operation_name_is_readonly(self) -> None:
-        invocation = self.handler.start_workflow(name="wf")
-        with self.assertRaises(AttributeError):
-            setattr(invocation, "operation_name", "foo_Bar")
-        self.assertEqual(invocation.operation_name, "invoke_workflow")
-        invocation.stop()
-
     def test_start_workflow_creates_span(self) -> None:
         invocation = self.handler.start_workflow("my_workflow")
         self.assertIsNot(invocation.span, INVALID_SPAN)
@@ -182,7 +175,7 @@ def test_workflow_context_manager_default_invocation(self) -> None:
         with self.handler.workflow() as inv:
             self.assertIsInstance(inv, WorkflowInvocation)
             self.assertIsNone(inv.name)
-            self.assertEqual(inv.operation_name, "invoke_workflow")
+            self.assertEqual(inv._operation_name, "invoke_workflow")
 
         spans = self._get_finished_spans()
         self.assertEqual(len(spans), 1)

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

@@ -1,7 +1,5 @@
 import unittest
 
-import pytest
-
 from opentelemetry.sdk.trace import TracerProvider
 from opentelemetry.sdk.trace.export import SimpleSpanProcessor
 from opentelemetry.sdk.trace.export.in_memory_span_exporter import (
@@ -29,7 +27,7 @@ def test_default_values(self):
         invocation = self.handler.start_workflow(None)
         invocation.stop()
         assert invocation.name is None
-        assert invocation.operation_name == "invoke_workflow"
+        assert invocation._operation_name == "invoke_workflow"
         assert not invocation.input_messages
         assert not invocation.output_messages
         assert invocation.span is not INVALID_SPAN
@@ -85,12 +83,6 @@ def test_default_attributes_are_independent(self):
         inv1.stop()
         inv2.stop()
 
-    def test_operation_name_is_readonly(self):
-        invocation = self.handler.start_workflow("wf")
-        with pytest.raises(AttributeError):
-            setattr(invocation, "operation_name", "foo")
-        invocation.stop()
-
     def test_full_construction(self):
         inp = InputMessage(role="user", parts=[Text(content="query")])
         out = OutputMessage(