golf-mcp 0.1.7__py3-none-any.whl → 0.1.9__py3-none-any.whl

golf/examples/basic/golf.json

@@ -3,5 +3,7 @@
   "description": "A GolfMCP project",
   "host": "127.0.0.1",
   "port": 3000,
-  "transport": "sse"
+  "transport": "sse",
+  "opentelemetry_enabled": true,
+  "opentelemetry_default_exporter": "otlp_http"
 } 

golf/telemetry/__init__.py

@@ -0,0 +1,19 @@
+"""Golf telemetry module for OpenTelemetry instrumentation."""
+
+from golf.telemetry.instrumentation import (
+    instrument_tool,
+    instrument_resource,
+    instrument_prompt,
+    telemetry_lifespan,
+    init_telemetry,
+    get_tracer,
+)
+
+__all__ = [
+    "instrument_tool",
+    "instrument_resource", 
+    "instrument_prompt",
+    "telemetry_lifespan",
+    "init_telemetry",
+    "get_tracer",
+] 

golf/telemetry/instrumentation.py

@@ -0,0 +1,540 @@
+"""Component-level OpenTelemetry instrumentation for Golf-built servers."""
+
+import os
+import sys
+import functools
+from typing import Callable, Optional, TypeVar
+from contextlib import asynccontextmanager
+import asyncio
+
+from opentelemetry import trace
+from opentelemetry.sdk.trace import TracerProvider
+from opentelemetry.sdk.trace.export import BatchSpanProcessor, ConsoleSpanExporter
+from opentelemetry.sdk.resources import Resource
+from opentelemetry.exporter.otlp.proto.http.trace_exporter import OTLPSpanExporter
+from opentelemetry.trace import Status, StatusCode, Span
+from opentelemetry import baggage
+
+T = TypeVar('T')
+
+# Global tracer instance
+_tracer: Optional[trace.Tracer] = None
+_provider: Optional[TracerProvider] = None
+_instrumented_tools = []
+
+def init_telemetry(service_name: str = "golf-mcp-server") -> Optional[TracerProvider]:
+    """Initialize OpenTelemetry with environment-based configuration.
+    
+    Returns None if required environment variables are not set.
+    """
+    global _provider
+    
+    # Check for required environment variables based on exporter type
+    exporter_type = os.environ.get("OTEL_TRACES_EXPORTER", "console").lower()
+    
+    # For OTLP HTTP exporter, check if endpoint is configured
+    if exporter_type == "otlp_http":
+        endpoint = os.environ.get("OTEL_EXPORTER_OTLP_ENDPOINT")
+        if not endpoint:
+            print(f"[WARNING] OpenTelemetry tracing is disabled: OTEL_EXPORTER_OTLP_ENDPOINT is not set for OTLP HTTP exporter")
+            return None
+    
+    # Create resource with service information
+    resource_attributes = {
+        "service.name": os.environ.get("OTEL_SERVICE_NAME", service_name),
+        "service.version": os.environ.get("SERVICE_VERSION", "1.0.0"),
+        "service.instance.id": os.environ.get("SERVICE_INSTANCE_ID", "default"),
+    }
+    resource = Resource.create(resource_attributes)
+    
+    # Create provider
+    provider = TracerProvider(resource=resource)
+    
+    # Configure exporter based on type
+    try:
+        if exporter_type == "otlp_http":
+            endpoint = os.environ.get("OTEL_EXPORTER_OTLP_ENDPOINT", "http://localhost:4318/v1/traces")
+            headers = os.environ.get("OTEL_EXPORTER_OTLP_HEADERS", "")
+            
+            # Parse headers if provided
+            header_dict = {}
+            if headers:
+                for header in headers.split(","):
+                    if "=" in header:
+                        key, value = header.split("=", 1)
+                        header_dict[key.strip()] = value.strip()
+            
+            exporter = OTLPSpanExporter(
+                endpoint=endpoint,
+                headers=header_dict if header_dict else None
+            )
+        else:
+            # Default to console exporter
+            exporter = ConsoleSpanExporter(out=sys.stderr)
+    except Exception as e:
+        import traceback
+        traceback.print_exc()
+        raise
+    
+    # Add batch processor for better performance
+    try:
+        processor = BatchSpanProcessor(
+            exporter,
+            max_queue_size=2048,
+            schedule_delay_millis=1000,  # Export every 1 second instead of default 5 seconds
+            max_export_batch_size=512,
+            export_timeout_millis=5000
+        )
+        provider.add_span_processor(processor)
+    except Exception as e:
+        import traceback
+        traceback.print_exc()
+        raise
+    
+    # Set as global provider
+    try:
+        # Check if a provider is already set to avoid the warning
+        existing_provider = trace.get_tracer_provider()
+        if existing_provider is None or str(type(existing_provider).__name__) == 'ProxyTracerProvider':
+            # Only set if no provider exists or it's the default proxy provider
+            trace.set_tracer_provider(provider)
+        _provider = provider
+    except Exception as e:
+        import traceback
+        traceback.print_exc()
+        raise
+    
+    # Create a test span to verify everything is working
+    try:
+        test_tracer = provider.get_tracer("golf.telemetry.test", "1.0.0")
+        with test_tracer.start_as_current_span("startup.test") as span:
+            span.set_attribute("test", True)
+            span.set_attribute("service.name", service_name)
+            span.set_attribute("exporter.type", exporter_type)
+            span.set_attribute("endpoint", os.environ.get("OTEL_EXPORTER_OTLP_ENDPOINT", "not set"))
+    except Exception as e:
+        import traceback
+        traceback.print_exc()
+    
+    return provider
+
+def get_tracer() -> trace.Tracer:
+    """Get or create the global tracer instance."""
+    global _tracer, _provider
+    
+    # If no provider is set, telemetry is disabled - return no-op tracer
+    if _provider is None:
+        return trace.get_tracer("golf.mcp.components.noop", "1.0.0")
+    
+    if _tracer is None:
+        _tracer = trace.get_tracer("golf.mcp.components", "1.0.0")
+    return _tracer
+
+def _add_component_attributes(span: Span, component_type: str, component_name: str, **kwargs):
+    """Add standard component attributes to a span."""
+    span.set_attribute("mcp.component.type", component_type)
+    span.set_attribute("mcp.component.name", component_name)
+    
+    # Add any additional attributes
+    for key, value in kwargs.items():
+        if value is not None:
+            span.set_attribute(f"mcp.component.{key}", str(value))
+
+def instrument_tool(func: Callable[..., T], tool_name: str) -> Callable[..., T]:
+    """Instrument a tool function with OpenTelemetry tracing."""
+    global _provider
+    
+    # If telemetry is disabled, return the original function
+    if _provider is None:
+        return func
+    
+    tracer = get_tracer()
+    
+    @functools.wraps(func)
+    async def async_wrapper(*args, **kwargs):
+        span = tracer.start_span(f"tool.{tool_name}")
+        
+        # Activate the span in the current context
+        from opentelemetry import context
+        token = context.attach(trace.set_span_in_context(span))
+        
+        try:
+            _add_component_attributes(span, "tool", tool_name, 
+                                     args_count=len(args),
+                                     kwargs_count=len(kwargs))
+            
+            # Extract Context parameter if present - this should have MCP session info
+            ctx = kwargs.get('ctx')
+            if ctx:
+                if hasattr(ctx, 'request_id'):
+                    span.set_attribute("mcp.request.id", ctx.request_id)
+                if hasattr(ctx, 'session_id'):
+                    span.set_attribute("mcp.session.id", ctx.session_id)
+                # Try to find any session-related attributes
+                for attr in dir(ctx):
+                    if 'session' in attr.lower() and not attr.startswith('_'):
+                        value = getattr(ctx, attr, None)
+                        if value:
+                            span.set_attribute(f"mcp.context.{attr}", str(value))
+            
+            # Also check baggage for session ID
+            session_id_from_baggage = baggage.get_baggage("mcp.session.id")
+            if session_id_from_baggage:
+                span.set_attribute("mcp.session.id", session_id_from_baggage)
+            
+            # Add tool arguments as span attributes (be careful with sensitive data)
+            for i, arg in enumerate(args):
+                if isinstance(arg, (str, int, float, bool)) or arg is None:
+                    span.set_attribute(f"tool.arg.{i}", str(arg))
+                elif hasattr(arg, '__dict__'):
+                    # For objects, just record the type
+                    span.set_attribute(f"tool.arg.{i}.type", type(arg).__name__)
+            
+            # Add named arguments
+            for key, value in kwargs.items():
+                if key != 'ctx':
+                    if value is None:
+                        span.set_attribute(f"tool.kwarg.{key}", "null")
+                    elif isinstance(value, (str, int, float, bool)):
+                        span.set_attribute(f"tool.kwarg.{key}", str(value))
+                    elif isinstance(value, (list, tuple)):
+                        span.set_attribute(f"tool.kwarg.{key}", f"[{len(value)} items]")
+                    elif isinstance(value, dict):
+                        span.set_attribute(f"tool.kwarg.{key}", f"{{dict with {len(value)} keys}}")
+                    else:
+                        # For other types, at least record the type
+                        span.set_attribute(f"tool.kwarg.{key}.type", type(value).__name__)
+            
+            try:
+                result = await func(*args, **kwargs)
+                span.set_status(Status(StatusCode.OK))
+                
+                # Capture result metadata
+                if result is not None:
+                    if isinstance(result, (str, int, float, bool)):
+                        span.set_attribute("tool.result", str(result))
+                    elif isinstance(result, list):
+                        span.set_attribute("tool.result.count", len(result))
+                        span.set_attribute("tool.result.type", "list")
+                    elif isinstance(result, dict):
+                        span.set_attribute("tool.result.keys", ",".join(result.keys()) if len(result) < 10 else f"{len(result)} keys")
+                        span.set_attribute("tool.result.type", "dict")
+                    elif hasattr(result, '__len__'):
+                        span.set_attribute("tool.result.length", len(result))
+                    
+                    # For any result, record its type
+                    span.set_attribute("tool.result.class", type(result).__name__)
+                
+                return result
+            except Exception as e:
+                span.record_exception(e)
+                span.set_status(Status(StatusCode.ERROR, str(e)))
+                raise
+        finally:
+            # End the span and detach context
+            span.end()
+            context.detach(token)
+            
+            # Force flush the provider to ensure spans are exported
+            global _provider
+            if _provider:
+                try:
+                    _provider.force_flush(timeout_millis=1000)
+                except Exception as e:
+                    pass
+    
+    @functools.wraps(func)
+    def sync_wrapper(*args, **kwargs):
+        span = tracer.start_span(f"tool.{tool_name}")
+        
+        # Activate the span in the current context
+        from opentelemetry import context
+        token = context.attach(trace.set_span_in_context(span))
+        
+        try:
+            _add_component_attributes(span, "tool", tool_name,
+                                     args_count=len(args),
+                                     kwargs_count=len(kwargs))
+            
+            # Extract Context parameter if present - this should have MCP session info
+            ctx = kwargs.get('ctx')
+            if ctx:
+                if hasattr(ctx, 'request_id'):
+                    span.set_attribute("mcp.request.id", ctx.request_id)
+                if hasattr(ctx, 'session_id'):
+                    span.set_attribute("mcp.session.id", ctx.session_id)
+                # Try to find any session-related attributes
+                for attr in dir(ctx):
+                    if 'session' in attr.lower() and not attr.startswith('_'):
+                        value = getattr(ctx, attr, None)
+                        if value:
+                            span.set_attribute(f"mcp.context.{attr}", str(value))
+            
+            # Also check baggage for session ID
+            session_id_from_baggage = baggage.get_baggage("mcp.session.id")
+            if session_id_from_baggage:
+                span.set_attribute("mcp.session.id", session_id_from_baggage)
+            
+            # Add tool arguments as span attributes (be careful with sensitive data)
+            for i, arg in enumerate(args):
+                if isinstance(arg, (str, int, float, bool)) or arg is None:
+                    span.set_attribute(f"tool.arg.{i}", str(arg))
+                elif hasattr(arg, '__dict__'):
+                    # For objects, just record the type
+                    span.set_attribute(f"tool.arg.{i}.type", type(arg).__name__)
+            
+            # Add named arguments
+            for key, value in kwargs.items():
+                if key != 'ctx':
+                    if value is None:
+                        span.set_attribute(f"tool.kwarg.{key}", "null")
+                    elif isinstance(value, (str, int, float, bool)):
+                        span.set_attribute(f"tool.kwarg.{key}", str(value))
+                    elif isinstance(value, (list, tuple)):
+                        span.set_attribute(f"tool.kwarg.{key}", f"[{len(value)} items]")
+                    elif isinstance(value, dict):
+                        span.set_attribute(f"tool.kwarg.{key}", f"{{dict with {len(value)} keys}}")
+                    else:
+                        # For other types, at least record the type
+                        span.set_attribute(f"tool.kwarg.{key}.type", type(value).__name__)
+            
+            try:
+                result = func(*args, **kwargs)
+                span.set_status(Status(StatusCode.OK))
+                
+                # Capture result metadata
+                if result is not None:
+                    if isinstance(result, (str, int, float, bool)):
+                        span.set_attribute("tool.result", str(result))
+                    elif isinstance(result, list):
+                        span.set_attribute("tool.result.count", len(result))
+                        span.set_attribute("tool.result.type", "list")
+                    elif isinstance(result, dict):
+                        span.set_attribute("tool.result.keys", ",".join(result.keys()) if len(result) < 10 else f"{len(result)} keys")
+                        span.set_attribute("tool.result.type", "dict")
+                    elif hasattr(result, '__len__'):
+                        span.set_attribute("tool.result.length", len(result))
+                    
+                    # For any result, record its type
+                    span.set_attribute("tool.result.class", type(result).__name__)
+                
+                return result
+            except Exception as e:
+                span.record_exception(e)
+                span.set_status(Status(StatusCode.ERROR, str(e)))
+                raise
+        finally:
+            # End the span and detach context
+            span.end()
+            context.detach(token)
+            
+            # Force flush the provider to ensure spans are exported
+            global _provider
+            if _provider:
+                try:
+                    _provider.force_flush(timeout_millis=1000)
+                except Exception as e:
+                    pass
+    
+    # Return appropriate wrapper based on function type
+    if asyncio.iscoroutinefunction(func):
+        return async_wrapper
+    else:
+        return sync_wrapper
+
+def instrument_resource(func: Callable[..., T], resource_uri: str) -> Callable[..., T]:
+    """Instrument a resource function with OpenTelemetry tracing."""
+    global _provider
+    
+    # If telemetry is disabled, return the original function
+    if _provider is None:
+        return func
+    
+    tracer = get_tracer()
+    
+    # Determine if this is a template based on URI pattern
+    is_template = '{' in resource_uri
+    
+    @functools.wraps(func)
+    async def async_wrapper(*args, **kwargs):
+        span_name = "resource.template.read" if is_template else "resource.read"
+        with tracer.start_as_current_span(span_name) as span:
+            _add_component_attributes(span, "resource", resource_uri,
+                                     is_template=is_template)
+            
+            # Extract Context parameter if present
+            ctx = kwargs.get('ctx')
+            if ctx and hasattr(ctx, 'request_id'):
+                span.set_attribute("mcp.request.id", ctx.request_id)
+            
+            try:
+                result = await func(*args, **kwargs)
+                span.set_status(Status(StatusCode.OK))
+                
+                # Add result size if applicable
+                if hasattr(result, '__len__'):
+                    span.set_attribute("mcp.resource.size", len(result))
+                
+                return result
+            except Exception as e:
+                span.record_exception(e)
+                span.set_status(Status(StatusCode.ERROR, str(e)))
+                raise
+    
+    @functools.wraps(func)
+    def sync_wrapper(*args, **kwargs):
+        span_name = "resource.template.read" if is_template else "resource.read"
+        with tracer.start_as_current_span(span_name) as span:
+            _add_component_attributes(span, "resource", resource_uri,
+                                     is_template=is_template)
+            
+            # Extract Context parameter if present
+            ctx = kwargs.get('ctx')
+            if ctx and hasattr(ctx, 'request_id'):
+                span.set_attribute("mcp.request.id", ctx.request_id)
+            
+            try:
+                result = func(*args, **kwargs)
+                span.set_status(Status(StatusCode.OK))
+                
+                # Add result size if applicable
+                if hasattr(result, '__len__'):
+                    span.set_attribute("mcp.resource.size", len(result))
+                    
+                return result
+            except Exception as e:
+                span.record_exception(e)
+                span.set_status(Status(StatusCode.ERROR, str(e)))
+                raise
+    
+    if asyncio.iscoroutinefunction(func):
+        return async_wrapper
+    else:
+        return sync_wrapper
+
+def instrument_prompt(func: Callable[..., T], prompt_name: str) -> Callable[..., T]:
+    """Instrument a prompt function with OpenTelemetry tracing."""
+    global _provider
+    
+    # If telemetry is disabled, return the original function
+    if _provider is None:
+        return func
+    
+    tracer = get_tracer()
+    
+    @functools.wraps(func)
+    async def async_wrapper(*args, **kwargs):
+        with tracer.start_as_current_span(f"prompt.{prompt_name}") as span:
+            _add_component_attributes(span, "prompt", prompt_name)
+            
+            # Extract Context parameter if present
+            ctx = kwargs.get('ctx')
+            if ctx and hasattr(ctx, 'request_id'):
+                span.set_attribute("mcp.request.id", ctx.request_id)
+            
+            try:
+                result = await func(*args, **kwargs)
+                span.set_status(Status(StatusCode.OK))
+                
+                # Add message count if result is a list
+                if isinstance(result, list):
+                    span.set_attribute("mcp.prompt.message_count", len(result))
+                    
+                return result
+            except Exception as e:
+                span.record_exception(e)
+                span.set_status(Status(StatusCode.ERROR, str(e)))
+                raise
+    
+    @functools.wraps(func)
+    def sync_wrapper(*args, **kwargs):
+        with tracer.start_as_current_span(f"prompt.{prompt_name}") as span:
+            _add_component_attributes(span, "prompt", prompt_name)
+            
+            # Extract Context parameter if present
+            ctx = kwargs.get('ctx')
+            if ctx and hasattr(ctx, 'request_id'):
+                span.set_attribute("mcp.request.id", ctx.request_id)
+            
+            try:
+                result = func(*args, **kwargs)
+                span.set_status(Status(StatusCode.OK))
+                
+                # Add message count if result is a list
+                if isinstance(result, list):
+                    span.set_attribute("mcp.prompt.message_count", len(result))
+                    
+                return result
+            except Exception as e:
+                span.record_exception(e)
+                span.set_status(Status(StatusCode.ERROR, str(e)))
+                raise
+    
+    if asyncio.iscoroutinefunction(func):
+        return async_wrapper
+    else:
+        return sync_wrapper
+
+@asynccontextmanager
+async def telemetry_lifespan(mcp_instance):
+    """Simplified lifespan for telemetry initialization and cleanup."""
+    global _provider, _instrumented_tools
+    
+    # Initialize telemetry with the server name
+    provider = init_telemetry(service_name=mcp_instance.name)
+    
+    # If provider is None, telemetry is disabled
+    if provider is None:
+        # Just yield without any telemetry setup
+        yield
+        return
+    
+    # Try to add session tracking middleware if possible
+    try:
+        from starlette.middleware.base import BaseHTTPMiddleware
+        from starlette.requests import Request
+        
+        class SessionTracingMiddleware(BaseHTTPMiddleware):
+            async def dispatch(self, request: Request, call_next):
+                # Extract session ID from query params
+                session_id = request.query_params.get('session_id')
+                if session_id:
+                    # Add to baggage for propagation
+                    ctx = baggage.set_baggage("mcp.session.id", session_id)
+                    from opentelemetry import context
+                    token = context.attach(ctx)
+                    
+                    # Also create a span for the HTTP request
+                    tracer = get_tracer()
+                    with tracer.start_as_current_span(f"http.{request.method} {request.url.path}") as span:
+                        span.set_attribute("http.method", request.method)
+                        span.set_attribute("http.url", str(request.url))
+                        span.set_attribute("http.session_id", session_id)
+                        span.set_attribute("mcp.session.id", session_id)
+                        
+                        try:
+                            response = await call_next(request)
+                            span.set_attribute("http.status_code", response.status_code)
+                            return response
+                        finally:
+                            context.detach(token)
+                else:
+                    return await call_next(request)
+        
+        # Try to add middleware to FastMCP app if it has Starlette app
+        if hasattr(mcp_instance, 'app') or hasattr(mcp_instance, '_app'):
+            app = getattr(mcp_instance, 'app', getattr(mcp_instance, '_app', None))
+            if app and hasattr(app, 'add_middleware'):
+                app.add_middleware(SessionTracingMiddleware)
+    except Exception:
+        pass
+    
+    try:
+        # Yield control back to FastMCP
+        yield
+    finally:
+        # Cleanup - shutdown the provider
+        if _provider and hasattr(_provider, 'shutdown'):
+            _provider.force_flush()
+            _provider.shutdown()
+            _provider = None 

{golf_mcp-0.1.7.dist-info → golf_mcp-0.1.9.dist-info}/METADATA

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: golf-mcp
-Version: 0.1.7
+Version: 0.1.9
 Summary: Framework for building MCP servers
 Author-email: Antoni Gmitruk <antoni@golf.dev>
 License-Expression: Apache-2.0
@@ -177,6 +177,11 @@ The `golf.json` file is the heart of your Golf project configuration. Here's wha
                                       // - "sse": Server-Sent Events (recommended for web clients)
                                       // - "streamable-http": HTTP with streaming support
                                       // - "stdio": Standard I/O (for CLI integration)
+  
+  // OpenTelemetry Configuration (optional)
+  "opentelemetry_enabled": false,     // Enable distributed tracing
+  "opentelemetry_default_exporter": "console"  // Default exporter if OTEL_TRACES_EXPORTER not set
+                                               // Options: "console", "otlp_http"
 }
 ```
 
@@ -188,12 +193,46 @@ The `golf.json` file is the heart of your Golf project configuration. Here's wha
   - `"streamable-http"` provides HTTP streaming for traditional API clients
   - `"stdio"` enables integration with command-line tools and scripts
 - **`host` & `port`**: Control where your server listens. Use `"127.0.0.1"` for local development or `"0.0.0.0"` to accept external connections.
+- **`opentelemetry_enabled`**: When true, enables distributed tracing for debugging and monitoring your MCP server
+- **`opentelemetry_default_exporter`**: Sets the default trace exporter. Can be overridden by the `OTEL_TRACES_EXPORTER` environment variable
+
+## Features
+
+### 🔍 OpenTelemetry Support
+
+Golf includes built-in OpenTelemetry instrumentation for distributed tracing. When enabled, it automatically traces:
+- Tool executions with arguments and results
+- Resource reads and template expansions
+- Prompt generations
+- HTTP requests and sessions
+
+#### Configuration
+
+Enable OpenTelemetry in your `golf.json`:
+```json
+{
+  "opentelemetry_enabled": true,
+  "opentelemetry_default_exporter": "otlp_http"
+}
+```
+
+Then configure via environment variables:
+```bash
+# For OTLP HTTP exporter (e.g., Jaeger, Grafana Tempo)
+OTEL_TRACES_EXPORTER=otlp_http
+OTEL_EXPORTER_OTLP_ENDPOINT=http://localhost:4318/v1/traces
+OTEL_SERVICE_NAME=my-golf-server  # Optional, defaults to project name
+
+# For console exporter (debugging)
+OTEL_TRACES_EXPORTER=console
+```
+
+**Note**: When using the OTLP HTTP exporter, you must set `OTEL_EXPORTER_OTLP_ENDPOINT`. If not configured, Golf will display a warning and disable tracing to avoid errors.
 
 ## Roadmap
 
 Here are the things we are working hard on:
 
-*   **Native OpenTelemetry implementation for tracing**
 *   **`golf deploy` command for one click deployments to Vercel, Blaxel and other providers**
 *   **Production-ready OAuth token management, to allow for persistent, encrypted token storage and client mapping**