scout_apm_mcp 0.1.1 → 0.1.2

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: ee786e2a50c8985e1423731c5acf85264c5dce178d366238a317685762170c57
-  data.tar.gz: 8fda9d886c840af02bc5caeb0375b4d963bfef95c4c9c12492f415256a21d361
+  metadata.gz: fe2bbad61c1acff6ae4c31b52a7e820acdc64449be4ae998a574c1937ac4775c
+  data.tar.gz: a2c68cf35aa37975a99cf98b68fa29a91673c449f983bbb96547b6403df8401f
 SHA512:
-  metadata.gz: 3434d01dfa3783ba3a1364caadd17aae11deb0170557fba1277d3b9b72e3706cbb5eed30a984ca9b0222f85933caff79242a05b19d400693e27b1b0a8adc9a08
-  data.tar.gz: 87540aa4f25083d54b2aee4c9080c3b3544f3c87fe39462198949814ce12a82aae9dae266f73bbbf940ad0f643c78cf5435c40520a221d642b3859d8ab1b6683
+  metadata.gz: e591d27a417285348f7257f150aeb11a9fa4390227893fc996fadf65d08dd856fd4121f3780e7d5403ecb3e985d5c3f2389dce5a99c3d4f57ad9f47bb4123876
+  data.tar.gz: 6112164513df8a51d4e0c039f9245621869ea4533a54ad493df3ec3e4aa67b738fd4952a0507c740a35a1c8568705ed72f8c2cf9d927603bd85fb9080b671f46

data/CHANGELOG.md

@@ -1,5 +1,14 @@
 # CHANGELOG
 
+## 0.1.2 (2025-11-21)
+
+- Enhanced SSL certificate handling with support for `SSL_CERT_FILE` environment variable and automatic fallback to system certificates
+- Improved error handling for SSL verification failures with clearer error messages
+- Extended `Helpers.parse_scout_url` to support parsing multiple URL types (endpoints, error_groups, insights, apps) beyond just traces
+- Added `FetchScoutURLTool` MCP tool for automatically detecting and fetching data from any ScoutAPM URL
+- Fixed MCP error handling to ensure error responses always have valid IDs for strict MCP client validation
+- Improved URL parsing to return `url_type` field for better resource type detection
+
 ## 0.1.1 (2025-11-21)
 
 - Fixed `NullLogger` missing `set_client_initialized` method that caused MCP initialization errors

data/README.md

@@ -24,22 +24,6 @@ Sponsored by [Kisko Labs](https://www.kiskolabs.com).
 
 For Cursor IDE, create or update `.cursor/mcp.json` in your project:
 
-```json
-{
-  "mcpServers": {
-    "scout-apm": {
-      "command": "bundle",
-      "args": ["exec", "scout_apm_mcp"],
-      "env": {
-        "OP_ENV_ENTRY_PATH": "op://Vault Name/Item Name"
-      }
-    }
-  }
-}
-```
-
-Or if installed globally:
-
 ```json
 {
   "mcpServers": {
@@ -64,21 +48,10 @@ For Claude Desktop, edit the MCP configuration file:
 {
   "mcpServers": {
     "scout-apm": {
-      "command": "bundle",
-      "args": ["exec", "scout_apm_mcp"],
-      "cwd": "/path/to/your/project"
-    }
-  }
-}
-```
-
-Or if installed globally:
-
-```json
-{
-  "mcpServers": {
-    "scout-apm": {
-      "command": "scout_apm_mcp"
+      "command": "scout_apm_mcp",
+      "env": {
+        "OP_ENV_ENTRY_PATH": "op://Vault Name/Item Name"
+      }
     }
   }
 }

data/lib/scout_apm_mcp/client.rb

@@ -1,5 +1,6 @@
 require "uri"
 require "net/http"
+require "openssl"
 require "json"
 require "base64"
 
@@ -254,6 +255,10 @@ module ScoutApmMcp
       else
         raise "API request failed: #{response.code} #{response.message}"
       end
+    rescue OpenSSL::SSL::SSLError => e
+      raise "SSL verification failed: #{e.message}. This may be due to system certificate configuration issues."
+    rescue => e
+      raise "Request failed: #{e.class} - #{e.message}"
     end
 
     private
@@ -264,9 +269,24 @@ module ScoutApmMcp
     # @return [Net::HTTP] Configured HTTP client
     def build_http_client(uri)
       http = Net::HTTP.new(uri.host, uri.port)
-      http.use_ssl = uri.scheme == "https"
       http.read_timeout = 10
       http.open_timeout = 10
+
+      if uri.scheme == "https"
+        http.use_ssl = true
+        http.verify_mode = OpenSSL::SSL::VERIFY_PEER
+
+        # Set ca_file directly - this is the simplest and most reliable approach
+        # Try SSL_CERT_FILE first, then default cert file
+        ca_file = if ENV["SSL_CERT_FILE"] && File.file?(ENV["SSL_CERT_FILE"])
+          ENV["SSL_CERT_FILE"]
+        elsif File.exist?(OpenSSL::X509::DEFAULT_CERT_FILE)
+          OpenSSL::X509::DEFAULT_CERT_FILE
+        end
+
+        http.ca_file = ca_file if ca_file
+      end
+
       http
     end
 
@@ -297,6 +317,10 @@ module ScoutApmMcp
       else
         raise "API request failed: #{response.code} #{response.message}\n#{response.body}"
       end
+    rescue OpenSSL::SSL::SSLError => e
+      raise "SSL verification failed: #{e.message}. This may be due to system certificate configuration issues."
+    rescue => e
+      raise "Request failed: #{e.class} - #{e.message}"
     end
   end
 end

data/lib/scout_apm_mcp/helpers.rb

@@ -63,30 +63,55 @@ module ScoutApmMcp
             "or provide OP_ENV_ENTRY_PATH, or op_vault and op_item parameters for 1Password integration"
     end
 
-    # Parse a ScoutAPM trace URL and extract app_id, endpoint_id, and trace_id
+    # Parse a ScoutAPM URL and extract resource information
     #
-    # @param url [String] Full ScoutAPM trace URL
-    # @return [Hash] Hash containing :app_id, :endpoint_id, :trace_id, :query_params, and :decoded_endpoint
+    # @param url [String] Full ScoutAPM URL
+    # @return [Hash] Hash containing resource type and extracted IDs
+    #   Possible keys: :url_type, :app_id, :endpoint_id, :trace_id, :error_id, :insight_type,
+    #   :query_params, :decoded_endpoint
     def self.parse_scout_url(url)
       uri = URI.parse(url)
       path_parts = uri.path.split("/").reject(&:empty?)
 
-      # Extract from URL: /apps/{app_id}/endpoints/{endpoint_id}/trace/{trace_id}
+      result = {}
       app_index = path_parts.index("apps")
-      endpoints_index = path_parts.index("endpoints")
-      trace_index = path_parts.index("trace")
 
-      result = {}
+      return result unless app_index
+
+      result[:app_id] = path_parts[app_index + 1].to_i
 
-      if app_index && endpoints_index && trace_index
-        result[:app_id] = path_parts[app_index + 1].to_i
-        result[:endpoint_id] = path_parts[endpoints_index + 1]
-        result[:trace_id] = path_parts[trace_index + 1].to_i
+      # Detect URL type and extract IDs
+      # Pattern: /apps/{app_id}/endpoints/{endpoint_id}/trace/{trace_id}
+      if path_parts.include?("trace")
+        result[:url_type] = :trace
+        endpoints_index = path_parts.index("endpoints")
+        trace_index = path_parts.index("trace")
+        if endpoints_index && trace_index
+          result[:endpoint_id] = path_parts[endpoints_index + 1]
+          result[:trace_id] = path_parts[trace_index + 1].to_i
+        end
+      # Pattern: /apps/{app_id}/endpoints/{endpoint_id}
+      elsif path_parts.include?("endpoints")
+        result[:url_type] = :endpoint
+        endpoints_index = path_parts.index("endpoints")
+        result[:endpoint_id] = path_parts[endpoints_index + 1] if endpoints_index
+      # Pattern: /apps/{app_id}/error_groups/{error_id}
+      elsif path_parts.include?("error_groups")
+        result[:url_type] = :error_group
+        error_groups_index = path_parts.index("error_groups")
+        result[:error_id] = path_parts[error_groups_index + 1].to_i if error_groups_index
+      # Pattern: /apps/{app_id}/insights or /apps/{app_id}/insights/{insight_type}
+      elsif path_parts.include?("insights")
+        result[:url_type] = :insight
+        insights_index = path_parts.index("insights")
+        if insights_index && path_parts.length > insights_index + 1
+          result[:insight_type] = path_parts[insights_index + 1]
+        end
+      # Pattern: /apps/{app_id}
+      elsif path_parts.length == 2 && path_parts[0] == "apps"
+        result[:url_type] = :app
       else
-        # Fallback: try to extract by position
-        result[:app_id] = path_parts[1].to_i if path_parts[0] == "apps"
-        result[:endpoint_id] = path_parts[3] if path_parts[2] == "endpoints"
-        result[:trace_id] = path_parts[5].to_i if path_parts[4] == "trace"
+        result[:url_type] = :unknown
       end
 
       # Parse query parameters

data/lib/scout_apm_mcp/server.rb

@@ -16,6 +16,21 @@ FastMcp = MCP unless defined?(FastMcp)
 module MCP
   module Transports
     class StdioTransport
+      if method_defined?(:send_error)
+        alias_method :original_send_error, :send_error
+
+        def send_error(code, message, id = nil)
+          # Use placeholder id if nil to satisfy strict MCP client validation
+          # JSON-RPC 2.0 allows null for notifications, but MCP clients require valid id
+          id = "error_#{SecureRandom.hex(8)}" if id.nil?
+          original_send_error(code, message, id)
+        end
+      end
+    end
+  end
+
+  class Server
+    if method_defined?(:send_error)
       alias_method :original_send_error, :send_error
 
       def send_error(code, message, id = nil)
@@ -26,17 +41,6 @@ module MCP
       end
     end
   end
-
-  class Server
-    alias_method :original_send_error, :send_error
-
-    def send_error(code, message, id = nil)
-      # Use placeholder id if nil to satisfy strict MCP client validation
-      # JSON-RPC 2.0 allows null for notifications, but MCP clients require valid id
-      id = "error_#{SecureRandom.hex(8)}" if id.nil?
-      original_send_error(code, message, id)
-    end
-  end
 end
 
 module ScoutApmMcp
@@ -141,6 +145,7 @@ module ScoutApmMcp
       server.register_tool(GetInsightsHistoryTool)
       server.register_tool(GetInsightsHistoryByTypeTool)
       server.register_tool(ParseScoutURLTool)
+      server.register_tool(FetchScoutURLTool)
       server.register_tool(FetchOpenAPISchemaTool)
     end
 
@@ -427,10 +432,10 @@ module ScoutApmMcp
 
     # Utility Tools
     class ParseScoutURLTool < BaseTool
-      description "Parse a ScoutAPM trace URL and extract app_id, endpoint_id, and trace_id"
+      description "Parse a ScoutAPM URL and extract resource information (app_id, endpoint_id, trace_id, etc.)"
 
       arguments do
-        required(:url).filled(:string).description("Full ScoutAPM trace URL (e.g., https://scoutapm.com/apps/123/endpoints/.../trace/456)")
+        required(:url).filled(:string).description("Full ScoutAPM URL (e.g., https://scoutapm.com/apps/123/endpoints/.../trace/456)")
       end
 
       def call(url:)
@@ -438,6 +443,84 @@ module ScoutApmMcp
       end
     end
 
+    class FetchScoutURLTool < BaseTool
+      description "Fetch data from a ScoutAPM URL by automatically detecting the resource type and fetching the appropriate data"
+
+      arguments do
+        required(:url).filled(:string).description("Full ScoutAPM URL (e.g., https://scoutapm.com/apps/123/endpoints/.../trace/456)")
+        optional(:include_endpoint).filled(:bool).description("For trace URLs, also fetch endpoint details for context (default: false)")
+      end
+
+      def call(url:, include_endpoint: false)
+        parsed = Helpers.parse_scout_url(url)
+        client = get_client
+
+        result = {
+          url: url,
+          parsed: parsed,
+          data: nil
+        }
+
+        case parsed[:url_type]
+        when :trace
+          if parsed[:app_id] && parsed[:trace_id]
+            trace_data = client.fetch_trace(parsed[:app_id], parsed[:trace_id])
+            result[:data] = {trace: trace_data}
+
+            if include_endpoint && parsed[:endpoint_id]
+              endpoint_data = client.get_endpoint(parsed[:app_id], parsed[:endpoint_id])
+              result[:data][:endpoint] = endpoint_data
+              result[:data][:decoded_endpoint] = parsed[:decoded_endpoint]
+            end
+          else
+            raise "Invalid trace URL: missing app_id or trace_id"
+          end
+        when :endpoint
+          if parsed[:app_id] && parsed[:endpoint_id]
+            endpoint_data = client.get_endpoint(parsed[:app_id], parsed[:endpoint_id])
+            result[:data] = {
+              endpoint: endpoint_data,
+              decoded_endpoint: parsed[:decoded_endpoint]
+            }
+          else
+            raise "Invalid endpoint URL: missing app_id or endpoint_id"
+          end
+        when :error_group
+          if parsed[:app_id] && parsed[:error_id]
+            error_data = client.get_error_group(parsed[:app_id], parsed[:error_id])
+            result[:data] = {error_group: error_data}
+          else
+            raise "Invalid error group URL: missing app_id or error_id"
+          end
+        when :insight
+          if parsed[:app_id]
+            if parsed[:insight_type]
+              insight_data = client.get_insight_by_type(parsed[:app_id], parsed[:insight_type])
+              result[:data] = {insight: insight_data, insight_type: parsed[:insight_type]}
+            else
+              insights_data = client.get_all_insights(parsed[:app_id])
+              result[:data] = {insights: insights_data}
+            end
+          else
+            raise "Invalid insight URL: missing app_id"
+          end
+        when :app
+          if parsed[:app_id]
+            app_data = client.get_app(parsed[:app_id])
+            result[:data] = {app: app_data}
+          else
+            raise "Invalid app URL: missing app_id"
+          end
+        when :unknown
+          raise "Unknown or unsupported ScoutAPM URL format: #{url}"
+        else
+          raise "Unable to determine URL type from: #{url}"
+        end
+
+        result
+      end
+    end
+
     class FetchOpenAPISchemaTool < BaseTool
       description "Fetch the ScoutAPM OpenAPI schema from the API and optionally validate it"
 

data/lib/scout_apm_mcp/version.rb

@@ -1,3 +1,3 @@
 module ScoutApmMcp
-  VERSION = "0.1.1"
+  VERSION = "0.1.2"
 end

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: scout_apm_mcp
 version: !ruby/object:Gem::Version
-  version: 0.1.1
+  version: 0.1.2
 platform: ruby
 authors:
 - Andrei Makarov