More explicit __all__

Author: nagkumar91

instrumentation-genai/opentelemetry-instrumentation-openai-agents/src/opentelemetry/instrumentation/openai_agents/__init__.py

@@ -73,6 +73,15 @@
 from opentelemetry.semconv.schemas import Schemas
 from opentelemetry.trace import get_tracer
 
+__all__ = [
+    "OpenAIAgentsInstrumentor",
+    "GenAIProvider",
+    "GenAIOperationName",
+    "GenAIToolType",
+    "GenAIOutputType",
+    "GenAIEvaluationAttributes",
+]
+
 logger = logging.getLogger(__name__)
 
 
@@ -81,9 +90,6 @@ class OpenAIAgentsInstrumentor(BaseInstrumentor):
 
     def _instrument(self, **kwargs):
         """Instruments the OpenAI library for agent frameworks."""
-        # TODO: Implement your instrumentation logic here
-        # This is a stub implementation - add your custom instrumentation code
-
         tracer_provider = kwargs.get("tracer_provider")
         tracer = get_tracer(
             __name__,
@@ -106,17 +112,6 @@ def _instrument(self, **kwargs):
             )
         )
 
-        # Expose constants via module attributes for convenience
-        globals().update(
-            {
-                "GenAIProvider": GenAIProvider,
-                "GenAIOperationName": GenAIOperationName,
-                "GenAIToolType": GenAIToolType,
-                "GenAIOutputType": GenAIOutputType,
-                "GenAIEvaluationAttributes": GenAIEvaluationAttributes,
-            }
-        )
-
     def _uninstrument(self, **kwargs):
         """Uninstruments the OpenAI library for agent frameworks."""
         # No-op: optional processor registry may not be present.