brainzlab 0.1.33 → 0.1.35

data/lib/brainzlab/testing/matchers.rb

@@ -112,10 +112,10 @@ module BrainzLab
             "expected event '#{@event_name}' to have been tracked, but no such event was found.\n" \
               "Tracked events: #{BrainzLab::Testing.event_store.events.map { |e| e[:name] }.inspect}"
           else
-            "expected event '#{@event_name}' to have been tracked with properties:\n" \
-              "  #{@expected_properties.inspect}\n" \
-              "but was tracked with:\n" \
-              "  #{actual_events.map { |e| e[:properties] }.inspect}"
+            "expected event '#{@event_name}' to have been tracked with properties:\n  " \
+              "#{@expected_properties.inspect}\n" \
+              "but was tracked with:\n  " \
+              "#{actual_events.map { |e| e[:properties] }.inspect}"
           end
         end
 

data/lib/brainzlab/testing.rb

@@ -198,16 +198,18 @@ module BrainzLab
         end
 
         # Stub distributed tracing methods
-        BrainzLab::Pulse.define_singleton_method(:inject) do |headers, **opts|
+        BrainzLab::Pulse.define_singleton_method(:inject) do |headers, **_opts|
           headers['traceparent'] = '00-test-trace-id-12345-test-span-id-67890-01'
           headers
         end
 
         BrainzLab::Pulse.define_singleton_method(:extract) do |headers|
+          return unless headers['traceparent']
+
           BrainzLab::Pulse::Propagation::Context.new(
             trace_id: 'test-trace-id-12345',
             span_id: 'test-span-id-67890'
-          ) if headers['traceparent']
+          )
         end
 
         BrainzLab::Pulse.define_singleton_method(:extract!) do |headers|
@@ -304,7 +306,7 @@ module BrainzLab
         BrainzLab::Cortex.define_singleton_method(:read) { |*| nil }
         BrainzLab::Cortex.define_singleton_method(:write) { |*| true }
         BrainzLab::Cortex.define_singleton_method(:delete) { |*| true }
-        BrainzLab::Cortex.define_singleton_method(:fetch) { |key, **opts, &block| block&.call }
+        BrainzLab::Cortex.define_singleton_method(:fetch) { |_key, **_opts, &block| block&.call }
       end
 
       def stub_vault!

data/lib/brainzlab/utilities/route_path_normalizer.rb

@@ -0,0 +1,91 @@
+# frozen_string_literal: true
+
+module BrainzLab
+  module Utilities
+    # Collapses concrete request paths into route patterns, so that
+    # /users/1, /users/2 ... /users/99999 aggregate as a single /users/:id
+    # endpoint instead of one endpoint per record.
+    #
+    # Rails instrumentation names traces by controller#action and does not need
+    # this. Grape and plain Rack do: when the router does not expose a route
+    # pattern the SDK falls back to the raw PATH_INFO, which is where the
+    # cardinality blowup comes from.
+    #
+    # Two modes:
+    #   * with path params — substitute the exact values the router resolved
+    #   * without them — fall back to recognising id-shaped segments
+    module RoutePathNormalizer
+      # Params that describe the dispatch rather than the path itself.
+      SKIP_PARAMS = %i[controller action format].freeze
+
+      UUID_PATTERN = /\A\h{8}-\h{4}-\h{4}-\h{4}-\h{12}\z/
+      NUMERIC_PATTERN = /\A\d+\z/
+      # 16+ hex chars: git SHAs, hex tokens. Shorter hex strings are too easily
+      # a real word segment ("beef", "cafe") to risk substituting.
+      LONG_HEX_PATTERN = /\A\h{16,}\z/
+
+      module_function
+
+      # @param path [String, nil] the concrete request path
+      # @param path_params [Hash, nil] params the router resolved, when available
+      # @return [String, nil] the path with variable segments replaced
+      def normalize(path, path_params = nil)
+        return path if path.nil? || path.empty?
+
+        params = normalize_params(path_params)
+
+        # -1 keeps trailing empty segments, so a trailing slash survives.
+        segments = path.split('/', -1)
+        normalized = segments.map do |segment|
+          next segment if segment.empty?
+
+          params.empty? ? substitute_by_shape(segment) : substitute_by_param(segment, params)
+        end
+
+        normalized.join('/')
+      end
+
+      def normalize_params(path_params)
+        return {} if path_params.nil? || path_params.empty?
+
+        # Map value => [param names]. A value claimed by more than one param is
+        # ambiguous, so it is left alone rather than guessed at.
+        path_params.each_with_object({}) do |(key, value), map|
+          next if SKIP_PARAMS.include?(key.to_sym)
+
+          (map[value.to_s] ||= []) << key
+        end
+      end
+
+      def substitute_by_param(segment, params)
+        keys = params[segment]
+        return ":#{keys.first}" if keys && keys.length == 1
+
+        # "42.json" when the id param is "42"
+        if segment.include?('.')
+          base, extension = segment.split('.', 2)
+          keys = params[base]
+          return ":#{keys.first}.#{extension}" if keys && keys.length == 1
+        end
+
+        segment
+      end
+
+      def substitute_by_shape(segment)
+        base, extension = segment.include?('.') ? segment.split('.', 2) : [segment, nil]
+        placeholder = placeholder_for(base)
+        return segment if placeholder.nil?
+
+        extension ? "#{placeholder}.#{extension}" : placeholder
+      end
+
+      def placeholder_for(segment)
+        return ':uuid' if segment.match?(UUID_PATTERN)
+        return ':id' if segment.match?(NUMERIC_PATTERN)
+        return ':id' if segment.match?(LONG_HEX_PATTERN)
+
+        nil
+      end
+    end
+  end
+end

data/lib/brainzlab/utilities.rb

@@ -4,6 +4,7 @@ require_relative 'utilities/rate_limiter'
 require_relative 'utilities/circuit_breaker'
 require_relative 'utilities/health_check'
 require_relative 'utilities/log_formatter'
+require_relative 'utilities/route_path_normalizer'
 
 module BrainzLab
   module Utilities

data/lib/brainzlab/vault/client.rb

@@ -193,9 +193,9 @@ module BrainzLab
         BrainzLab.debug_log("[Vault::Client] #{operation} failed: #{structured_error.message}")
 
         # Call on_error callback if configured
-        if @config.on_error
-          @config.on_error.call(structured_error, { service: 'Vault', operation: operation })
-        end
+        return unless @config.on_error
+
+        @config.on_error.call(structured_error, { service: 'Vault', operation: operation })
       end
 
       def handle_response_error(response, operation)
@@ -205,9 +205,7 @@ module BrainzLab
         BrainzLab.debug_log("[Vault::Client] #{operation} failed: #{structured_error.message}")
 
         # Call on_error callback if configured
-        if @config.on_error
-          @config.on_error.call(structured_error, { service: 'Vault', operation: operation })
-        end
+        @config.on_error&.call(structured_error, { service: 'Vault', operation: operation })
 
         structured_error
       end

data/lib/brainzlab/version.rb

@@ -1,5 +1,5 @@
 # frozen_string_literal: true
 
 module BrainzLab
-  VERSION = '0.1.33'
+  VERSION = '0.1.35'
 end

data/lib/brainzlab/vision/client.rb

@@ -154,9 +154,9 @@ module BrainzLab
         BrainzLab.debug_log("[Vision::Client] #{operation} failed: #{error.message}")
 
         # Call on_error callback if configured
-        if @config.on_error
-          @config.on_error.call(error, { service: 'Vision', operation: operation })
-        end
+        return unless @config.on_error
+
+        @config.on_error.call(error, { service: 'Vision', operation: operation })
       end
 
       def auth_key

data/lib/brainzlab.rb

@@ -63,6 +63,7 @@ module BrainzLab
 
     def configure
       yield(configuration) if block_given?
+      configuration.validate!
       configuration
     end
 

data/lib/fluyenta-ruby.rb

@@ -1,3 +1,3 @@
 # frozen_string_literal: true
 
-require_relative 'brainzlab'
+require_relative 'brainzlab'

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: brainzlab
 version: !ruby/object:Gem::Version
-  version: 0.1.33
+  version: 0.1.35
 platform: ruby
 authors:
 - BrainzLab
@@ -177,6 +177,7 @@ files:
 - lib/brainzlab/nerve/provisioner.rb
 - lib/brainzlab/pulse.rb
 - lib/brainzlab/pulse/client.rb
+- lib/brainzlab/pulse/explain_capture.rb
 - lib/brainzlab/pulse/instrumentation.rb
 - lib/brainzlab/pulse/propagation.rb
 - lib/brainzlab/pulse/provisioner.rb
@@ -211,6 +212,7 @@ files:
 - lib/brainzlab/utilities/health_check.rb
 - lib/brainzlab/utilities/log_formatter.rb
 - lib/brainzlab/utilities/rate_limiter.rb
+- lib/brainzlab/utilities/route_path_normalizer.rb
 - lib/brainzlab/vault.rb
 - lib/brainzlab/vault/cache.rb
 - lib/brainzlab/vault/client.rb