ruby_llm-agents 3.1.0 → 3.2.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 552fe8427ac0b47f41b14e4652ac65eac1c7b0389a603b71576f8171aba504f7
-  data.tar.gz: d356d7fc391f93194bc0d4dc1cd179178361af5db8c6b00270c8cbd9bcea19b8
+  metadata.gz: d8c4a83ecc9e39e7df7243b98a51d1c249a963f3a1f96551ebefae13becb50c5
+  data.tar.gz: d042bda1737b7593187896e879b3065cc855e990e1406410e99d1d853819f3a9
 SHA512:
-  metadata.gz: 8d7d722cc72e9baa30a3da29f234ed039965f12743f541ae354f7d477aa7a9a5798c0639ddf37872fa1b67c58c7244b7490d0d9f548a03f3734e101a717153a9
-  data.tar.gz: 33f0e37172dbd4b607f86ca27d27b2be997b1650544dcbf41877dd00e094f66bde86b81afaa7aacbf31d7049eda2f9567f3a27b3970fad64c01113a6e436785b
+  metadata.gz: 78b6fa31a8a656c36e0bb51f6e7a405101e90f47afc3ca35f87c392878b6a2d986b2b275c6424826ffb6b5d4bfb059f4632e8976aa5b2a473eab540619bd18cb
+  data.tar.gz: 3d5890ea864aea3531e96571b6010c9fdcedc5a15e8966c7974138cde0ebaec89a771e33fd20e3fd2d5097a1ecfc398236d833f9852f046fd3b1f720eaf7fb6e

data/README.md

@@ -135,6 +135,7 @@ result.save("logo.png")
 | **Attachments** | Images, PDFs, and multimodal support | [Attachments](https://github.com/adham90/ruby_llm-agents/wiki/Attachments) |
 | **Embeddings** | Vector embeddings with batching, caching, and preprocessing | [Embeddings](https://github.com/adham90/ruby_llm-agents/wiki/Embeddings) |
 | **Image Operations** | Generation, analysis, editing, pipelines with cost tracking | [Images](https://github.com/adham90/ruby_llm-agents/wiki/Image-Generation) |
+| **Audio** | Text-to-speech (OpenAI, ElevenLabs) and speech-to-text with cost tracking | [Audio](https://github.com/adham90/ruby_llm-agents/wiki/Audio) |
 | **Alerts** | Slack, webhook, and custom notifications | [Alerts](https://github.com/adham90/ruby_llm-agents/wiki/Alerts) |
 
 ## Quick Start

data/app/controllers/ruby_llm/agents/agents_controller.rb

@@ -55,14 +55,14 @@ module RubyLLM
 
         @agent_count = @agents.size
         @deleted_count = @deleted_agents.size
-      rescue StandardError => e
+      rescue => e
         Rails.logger.error("[RubyLLM::Agents] Error loading agents: #{e.message}")
         @agents = []
         @deleted_agents = []
-        @agents_by_type = { agent: [], embedder: [], speaker: [], transcriber: [], image_generator: [] }
+        @agents_by_type = {agent: [], embedder: [], speaker: [], transcriber: [], image_generator: []}
         @agent_count = 0
         @deleted_count = 0
-        @sort_params = { column: DEFAULT_AGENT_SORT_COLUMN, direction: DEFAULT_AGENT_SORT_DIRECTION }
+        @sort_params = {column: DEFAULT_AGENT_SORT_COLUMN, direction: DEFAULT_AGENT_SORT_DIRECTION}
         flash.now[:alert] = "Error loading agents list"
       end
 
@@ -88,7 +88,7 @@ module RubyLLM
           # Only load circuit breaker status for base agents
           load_circuit_breaker_status if @agent_type_kind == "agent"
         end
-      rescue StandardError => e
+      rescue => e
         Rails.logger.error("[RubyLLM::Agents] Error loading agent #{@agent_type}: #{e.message}")
         redirect_to ruby_llm_agents.agents_path, alert: "Error loading agent details"
       end
@@ -118,9 +118,9 @@ module RubyLLM
       def load_filter_options
         # Single query to get all filter options (fixes N+1)
         filter_data = Execution.by_agent(@agent_type)
-                               .where.not(model_id: nil)
-                               .or(Execution.by_agent(@agent_type).where.not(temperature: nil))
-                               .pluck(:model_id, :temperature)
+          .where.not(model_id: nil)
+          .or(Execution.by_agent(@agent_type).where.not(temperature: nil))
+          .pluck(:model_id, :temperature)
 
         @models = filter_data.map(&:first).compact.uniq.sort
         @temperatures = filter_data.map(&:last).compact.uniq.sort
@@ -167,9 +167,7 @@ module RubyLLM
 
         # Apply time range filter with validation
         days = parse_days_param
-        scope = apply_time_filter(scope, days)
-
-        scope
+        apply_time_filter(scope, days)
       end
 
       # Loads chart data for agent performance visualization
@@ -300,7 +298,7 @@ module RubyLLM
       def safe_config_call(method)
         return nil unless @agent_class&.respond_to?(method)
         @agent_class.public_send(method)
-      rescue StandardError
+      rescue
         nil
       end
 
@@ -313,7 +311,11 @@ module RubyLLM
       def load_circuit_breaker_status
         return unless @agent_class.respond_to?(:reliability_config)
 
-        config = @agent_class.reliability_config rescue nil
+        config = begin
+          @agent_class.reliability_config
+        rescue
+          nil
+        end
         return unless config
 
         # Collect all models: primary + fallbacks
@@ -355,7 +357,7 @@ module RubyLLM
 
           @circuit_breaker_status[model_id] = status
         end
-      rescue StandardError => e
+      rescue => e
         Rails.logger.debug("[RubyLLM::Agents] Could not load circuit breaker status: #{e.message}")
         @circuit_breaker_status = {}
       end
@@ -394,7 +396,7 @@ module RubyLLM
           end
         end
 
-        direction == "desc" ? sorted.reverse : sorted
+        (direction == "desc") ? sorted.reverse : sorted
       end
     end
   end

data/app/controllers/ruby_llm/agents/dashboard_controller.rb

@@ -135,9 +135,9 @@ module RubyLLM
             total_cost: model_cost,
             total_tokens: model_tokens,
             avg_duration_ms: durations[model_id]&.round || 0,
-            success_rate: count > 0 ? (successful.to_f / count * 100).round(1) : 0,
-            cost_per_1k_tokens: model_tokens > 0 ? (model_cost / model_tokens * 1000).round(4) : 0,
-            cost_percentage: total_cost > 0 ? (model_cost / total_cost * 100).round(1) : 0
+            success_rate: (count > 0) ? (successful.to_f / count * 100).round(1) : 0,
+            cost_per_1k_tokens: (model_tokens > 0) ? (model_cost / model_tokens * 1000).round(4) : 0,
+            cost_percentage: (total_cost > 0) ? (model_cost / total_cost * 100).round(1) : 0
           }
         end.sort_by { |m| -(m[:total_cost] || 0) }
       end
@@ -151,16 +151,16 @@ module RubyLLM
         total_errors = scope.count
 
         scope.group(:error_class)
-             .select("error_class, COUNT(*) as count, MAX(created_at) as last_seen")
-             .order("count DESC")
-             .limit(5)
-             .map do |row|
-          {
-            error_class: row.error_class || "Unknown Error",
-            count: row.count,
-            percentage: total_errors > 0 ? (row.count.to_f / total_errors * 100).round(1) : 0,
-            last_seen: row.last_seen
-          }
+          .select("error_class, COUNT(*) as count, MAX(created_at) as last_seen")
+          .order("count DESC")
+          .limit(5)
+          .map do |row|
+            {
+              error_class: row.error_class || "Unknown Error",
+              count: row.count,
+              percentage: (total_errors > 0) ? (row.count.to_f / total_errors * 100).round(1) : 0,
+              last_seen: row.last_seen
+            }
         end
       end
 
@@ -249,7 +249,7 @@ module RubyLLM
         end
 
         open_breakers
-      rescue StandardError => e
+      rescue => e
         Rails.logger.debug("[RubyLLM::Agents] Error loading open breakers: #{e.message}")
         []
       end
@@ -277,8 +277,8 @@ module RubyLLM
           monthly_limit: tenant.effective_monthly_limit,
           daily_spend: daily_spend,
           monthly_spend: monthly_spend,
-          daily_percentage: tenant.effective_daily_limit.to_f > 0 ? (daily_spend / tenant.effective_daily_limit * 100).round(1) : 0,
-          monthly_percentage: tenant.effective_monthly_limit.to_f > 0 ? (monthly_spend / tenant.effective_monthly_limit * 100).round(1) : 0,
+          daily_percentage: (tenant.effective_daily_limit.to_f > 0) ? (daily_spend / tenant.effective_daily_limit * 100).round(1) : 0,
+          monthly_percentage: (tenant.effective_monthly_limit.to_f > 0) ? (monthly_spend / tenant.effective_monthly_limit * 100).round(1) : 0,
           enforcement: tenant.effective_enforcement,
           per_agent_daily: tenant.per_agent_daily || {}
         }
@@ -308,7 +308,7 @@ module RubyLLM
 
         # Open circuit breakers
         load_open_breakers.each do |breaker|
-          alerts << { type: :breaker, data: breaker }
+          alerts << {type: :breaker, data: breaker}
         end
 
         # Budget breaches (>100% of limit)
@@ -341,7 +341,7 @@ module RubyLLM
         # Error spike detection (>5 errors in last 15 minutes)
         error_count_15m = base_scope.status_error.where("created_at > ?", 15.minutes.ago).count
         if error_count_15m >= 5
-          alerts << { type: :error_spike, data: { count: error_count_15m } }
+          alerts << {type: :error_spike, data: {count: error_count_15m}}
         end
 
         alerts.take(3)
@@ -366,9 +366,9 @@ module RubyLLM
           hash[agent_type] = {
             count: count,
             total_cost: total_cost,
-            avg_cost: count > 0 ? (total_cost / count).round(6) : 0,
+            avg_cost: (count > 0) ? (total_cost / count).round(6) : 0,
             avg_duration_ms: durations[agent_type]&.round || 0,
-            success_rate: count > 0 ? (successful.to_f / count * 100).round(1) : 0
+            success_rate: (count > 0) ? (successful.to_f / count * 100).round(1) : 0
           }
         end
       end

data/app/controllers/ruby_llm/agents/executions_controller.rb

@@ -17,7 +17,7 @@ module RubyLLM
       include Sortable
 
       CSV_COLUMNS = %w[id agent_type status model_id total_tokens total_cost
-                       duration_ms created_at error_class error_message].freeze
+        duration_ms created_at error_class error_message].freeze
 
       # Lists all executions with filtering and pagination
       #
@@ -55,7 +55,7 @@ module RubyLLM
             render turbo_stream: turbo_stream.replace(
               "executions_list",
               partial: "ruby_llm/agents/executions/list",
-              locals: { executions: @executions, pagination: @pagination, filter_stats: @filter_stats }
+              locals: {executions: @executions, pagination: @pagination, filter_stats: @filter_stats}
             )
           end
         end
@@ -92,8 +92,8 @@ module RubyLLM
       # @return [String] CSV row string
       def generate_csv_row(execution)
         redacted_error_message = if execution.error_message.present?
-                                   Redactor.redact_string(execution.error_message)
-                                 end
+          Redactor.redact_string(execution.error_message)
+        end
 
         CSV.generate_line([
           execution.id,
@@ -207,9 +207,7 @@ module RubyLLM
         scope = scope.where(parent_execution_id: nil)
 
         # Eager load children for grouping
-        scope = scope.includes(:child_executions)
-
-        scope
+        scope.includes(:child_executions)
       end
     end
   end

data/app/helpers/ruby_llm/agents/application_helper.rb

@@ -12,7 +12,7 @@ module RubyLLM
       include Chartkick::Helper if defined?(Chartkick)
 
       # Wiki base URL for documentation links
-      WIKI_BASE_URL = "https://github.com/adham90/ruby_llm-agents/wiki/".freeze
+      WIKI_BASE_URL = "https://github.com/adham90/ruby_llm-agents/wiki/"
 
       # Page to documentation mapping
       DOC_PAGES = {
@@ -176,7 +176,7 @@ module RubyLLM
       def render_sparkline(trend_data, metric_key, color_class: "text-blue-500")
         return "".html_safe if trend_data.blank? || trend_data.length < 2
 
-        values = trend_data.map { |d| d[metric_key].to_f || 0 }
+        values = trend_data.map { |d| d[metric_key].to_f }
         max_val = values.max || 1
         min_val = values.min || 0
         range = max_val - min_val
@@ -197,8 +197,7 @@ module RubyLLM
             "stroke-width": "2",
             "stroke-linecap": "round",
             "stroke-linejoin": "round",
-            class: color_class
-          )
+            class: color_class)
         end
       end
 
@@ -212,27 +211,27 @@ module RubyLLM
       # @return [ActiveSupport::SafeBuffer] HTML badge element
       def comparison_badge(change_pct, metric_type)
         threshold = case metric_type
-                    when :success_rate then 5
-                    when :cost, :tokens then 15
-                    when :duration then 20
-                    when :count then 25
-                    else 10
-                    end
+        when :success_rate then 5
+        when :cost, :tokens then 15
+        when :duration then 20
+        when :count then 25
+        else 10
+        end
 
         # Determine what "improvement" means for this metric
         # For cost/tokens/duration: negative change is good (lower is better)
         # For success_rate/count: positive change is good (higher is better)
         is_improvement = case metric_type
-                         when :success_rate, :count then change_pct > threshold
-                         when :cost, :tokens, :duration then change_pct < -threshold
-                         else false
-                         end
+        when :success_rate, :count then change_pct > threshold
+        when :cost, :tokens, :duration then change_pct < -threshold
+        else false
+        end
 
         is_regression = case metric_type
-                        when :success_rate, :count then change_pct < -threshold
-                        when :cost, :tokens, :duration then change_pct > threshold
-                        else false
-                        end
+        when :success_rate, :count then change_pct < -threshold
+        when :cost, :tokens, :duration then change_pct > threshold
+        else false
+        end
 
         if is_improvement
           content_tag(:span, class: "inline-flex items-center gap-1 px-2 py-0.5 text-xs font-medium text-green-700 dark:text-green-300 bg-green-100 dark:bg-green-500/20 rounded-full") do
@@ -279,7 +278,7 @@ module RubyLLM
         positive_is_good = metric_type.in?(%i[success tokens count])
         is_improvement = positive_is_good ? change_pct > 0 : change_pct < 0
 
-        arrow = change_pct > 0 ? "\u2191" : "\u2193"
+        arrow = (change_pct > 0) ? "\u2191" : "\u2193"
         color = is_improvement ? "text-green-600 dark:text-green-400" : "text-red-600 dark:text-red-400"
 
         content_tag(:span, "#{arrow}#{change_pct.abs}%", class: "text-xs font-medium #{color} ml-1")
@@ -323,24 +322,24 @@ module RubyLLM
       # @return [String] Tailwind CSS classes for row background
       def comparison_row_class(change_pct, metric_type)
         threshold = case metric_type
-                    when :success_rate then 5
-                    when :cost, :tokens then 15
-                    when :duration then 20
-                    when :count then 25
-                    else 10
-                    end
+        when :success_rate then 5
+        when :cost, :tokens then 15
+        when :duration then 20
+        when :count then 25
+        else 10
+        end
 
         is_improvement = case metric_type
-                         when :success_rate, :count then change_pct > threshold
-                         when :cost, :tokens, :duration then change_pct < -threshold
-                         else false
-                         end
+        when :success_rate, :count then change_pct > threshold
+        when :cost, :tokens, :duration then change_pct < -threshold
+        else false
+        end
 
         is_regression = case metric_type
-                        when :success_rate, :count then change_pct < -threshold
-                        when :cost, :tokens, :duration then change_pct > threshold
-                        else false
-                        end
+        when :success_rate, :count then change_pct < -threshold
+        when :cost, :tokens, :duration then change_pct > threshold
+        else false
+        end
 
         if is_improvement
           "bg-green-50 dark:bg-green-900/20 border border-green-200 dark:border-green-800"
@@ -433,7 +432,7 @@ module RubyLLM
             str_start = i
             i += 1
             while i < chars.length
-              if chars[i] == '\\'
+              if chars[i] == "\\"
                 i += 2
               elsif chars[i] == '"'
                 i += 1
@@ -442,49 +441,49 @@ module RubyLLM
                 i += 1
               end
             end
-            tokens << { type: :string, value: chars[str_start...i].join }
-          when /[0-9\-]/
+            tokens << {type: :string, value: chars[str_start...i].join}
+          when /[0-9-]/
             # Number token: starts with digit or minus, continues with digits/decimals/exponents
             num_start = i
             i += 1
-            while i < chars.length && chars[i] =~ /[0-9.eE+\-]/
+            while i < chars.length && chars[i] =~ /[0-9.eE+-]/
               i += 1
             end
-            tokens << { type: :number, value: chars[num_start...i].join }
-          when 't'
+            tokens << {type: :number, value: chars[num_start...i].join}
+          when "t"
             # Boolean token: check for "true" keyword
-            if chars[i, 4].join == 'true'
-              tokens << { type: :boolean, value: 'true' }
+            if chars[i, 4].join == "true"
+              tokens << {type: :boolean, value: "true"}
               i += 4
             else
-              tokens << { type: :text, value: char }
+              tokens << {type: :text, value: char}
               i += 1
             end
-          when 'f'
+          when "f"
             # Boolean token: check for "false" keyword
-            if chars[i, 5].join == 'false'
-              tokens << { type: :boolean, value: 'false' }
+            if chars[i, 5].join == "false"
+              tokens << {type: :boolean, value: "false"}
               i += 5
             else
-              tokens << { type: :text, value: char }
+              tokens << {type: :text, value: char}
               i += 1
             end
-          when 'n'
+          when "n"
             # Null token: check for "null" keyword
-            if chars[i, 4].join == 'null'
-              tokens << { type: :null, value: 'null' }
+            if chars[i, 4].join == "null"
+              tokens << {type: :null, value: "null"}
               i += 4
             else
-              tokens << { type: :text, value: char }
+              tokens << {type: :text, value: char}
               i += 1
             end
-          when ':', ',', '{', '}', '[', ']', ' ', "\n", "\t"
+          when ":", ",", "{", "}", "[", "]", " ", "\n", "\t"
             # Punctuation token: structural characters and whitespace
-            tokens << { type: :punct, value: char }
+            tokens << {type: :punct, value: char}
             i += 1
           else
             # Fallback for unexpected characters
-            tokens << { type: :text, value: char }
+            tokens << {type: :text, value: char}
             i += 1
           end
         end
@@ -503,10 +502,10 @@ module RubyLLM
             is_key = false
             (idx + 1...tokens.length).each do |j|
               if tokens[j][:type] == :punct
-                if tokens[j][:value] == ':'
+                if tokens[j][:value] == ":"
                   is_key = true
                   break
-                elsif tokens[j][:value] !~ /\s/
+                elsif !/\s/.match?(tokens[j][:value])
                   # Non-whitespace punct that isn't colon - not a key
                   break
                 end
@@ -517,10 +516,10 @@ module RubyLLM
             end
 
             escaped_value = ERB::Util.html_escape(token[:value])
-            if is_key
-              result << %(<span class="text-purple-600 dark:text-purple-400">#{escaped_value}</span>)
+            result << if is_key
+              %(<span class="text-purple-600 dark:text-purple-400">#{escaped_value}</span>)
             else
-              result << %(<span class="text-green-600 dark:text-green-400">#{escaped_value}</span>)
+              %(<span class="text-green-600 dark:text-green-400">#{escaped_value}</span>)
             end
           when :number
             result << %(<span class="text-blue-600 dark:text-blue-400">#{token[:value]}</span>)

data/app/models/ruby_llm/agents/execution/analytics.rb

@@ -72,7 +72,7 @@ module RubyLLM
               period: period,
               count: count,
               total_cost: total_cost,
-              avg_cost: count > 0 ? (total_cost / count).round(6) : 0,
+              avg_cost: (count > 0) ? (total_cost / count).round(6) : 0,
               total_tokens: scope.total_tokens_sum || 0,
               avg_tokens: scope.avg_tokens&.round || 0,
               avg_duration_ms: scope.avg_duration&.round || 0,
@@ -179,7 +179,7 @@ module RubyLLM
             total_duration_count = 0
             total_tokens = 0
 
-            (23.downto(0)).each do |hours_ago|
+            23.downto(0).each do |hours_ago|
               bucket_time = (reference_time - hours_ago.hours).beginning_of_hour
               rows = results[bucket_time] || []
 
@@ -206,7 +206,7 @@ module RubyLLM
               total_duration_count += duration_rows.count
             end
 
-            avg_duration_ms = total_duration_count > 0 ? (total_duration_sum / total_duration_count).round : 0
+            avg_duration_ms = (total_duration_count > 0) ? (total_duration_sum / total_duration_count).round : 0
 
             {
               range: "today",
@@ -218,11 +218,11 @@ module RubyLLM
                 tokens: total_tokens
               },
               series: [
-                { name: "Success", data: success_data },
-                { name: "Errors", data: failed_data },
-                { name: "Cost", data: cost_data },
-                { name: "Duration", data: duration_data },
-                { name: "Tokens", data: tokens_data }
+                {name: "Success", data: success_data},
+                {name: "Errors", data: failed_data},
+                {name: "Cost", data: cost_data},
+                {name: "Duration", data: duration_data},
+                {name: "Tokens", data: tokens_data}
               ]
             }
           end
@@ -283,7 +283,7 @@ module RubyLLM
               total_duration_count += duration_rows.count
             end
 
-            avg_duration_ms = total_duration_count > 0 ? (total_duration_sum / total_duration_count).round : 0
+            avg_duration_ms = (total_duration_count > 0) ? (total_duration_sum / total_duration_count).round : 0
 
             {
               range: "#{days}d",
@@ -296,11 +296,11 @@ module RubyLLM
                 tokens: total_tokens
               },
               series: [
-                { name: "Success", data: success_data },
-                { name: "Errors", data: failed_data },
-                { name: "Cost", data: cost_data },
-                { name: "Duration", data: duration_data },
-                { name: "Tokens", data: tokens_data }
+                {name: "Success", data: success_data},
+                {name: "Errors", data: failed_data},
+                {name: "Cost", data: cost_data},
+                {name: "Duration", data: duration_data},
+                {name: "Tokens", data: tokens_data}
               ]
             }
           end
@@ -359,7 +359,7 @@ module RubyLLM
               total_duration_count += duration_rows.count
             end
 
-            avg_duration_ms = total_duration_count > 0 ? (total_duration_sum / total_duration_count).round : 0
+            avg_duration_ms = (total_duration_count > 0) ? (total_duration_sum / total_duration_count).round : 0
 
             {
               range: "custom",
@@ -374,11 +374,11 @@ module RubyLLM
                 tokens: total_tokens
               },
               series: [
-                { name: "Success", data: success_data },
-                { name: "Errors", data: failed_data },
-                { name: "Cost", data: cost_data },
-                { name: "Duration", data: duration_data },
-                { name: "Tokens", data: tokens_data }
+                {name: "Success", data: success_data},
+                {name: "Errors", data: failed_data},
+                {name: "Cost", data: cost_data},
+                {name: "Duration", data: duration_data},
+                {name: "Tokens", data: tokens_data}
               ]
             }
           end
@@ -398,7 +398,7 @@ module RubyLLM
             reference_time = Time.current.beginning_of_hour
 
             # Create entries for the last 24 hours ending at current hour
-            (23.downto(0)).each do |hours_ago|
+            23.downto(0).each do |hours_ago|
               start_time = reference_time - hours_ago.hours
               end_time = start_time + 1.hour
               time_label = start_time.in_time_zone.strftime("%H:%M")
@@ -409,8 +409,8 @@ module RubyLLM
             end
 
             [
-              { name: "Success", data: success_data },
-              { name: "Failed", data: failed_data }
+              {name: "Success", data: success_data},
+              {name: "Failed", data: failed_data}
             ]
           end
 
@@ -447,8 +447,8 @@ module RubyLLM
             end
 
             [
-              { name: "Input Cost", data: input_cost_data },
-              { name: "Output Cost", data: output_cost_data }
+              {name: "Input Cost", data: input_cost_data},
+              {name: "Output Cost", data: output_cost_data}
             ]
           end
 
@@ -514,7 +514,7 @@ module RubyLLM
             (rate_limited_count.to_f / total * 100).round(1)
           end
 
-        private
+          private
 
           # Calculates success rate percentage for a scope
           #
@@ -547,7 +547,7 @@ module RubyLLM
             {
               count: count,
               total_cost: total_cost,
-              avg_cost: count > 0 ? (total_cost / count).round(6) : 0,
+              avg_cost: (count > 0) ? (total_cost / count).round(6) : 0,
               avg_tokens: scope.avg_tokens&.round || 0,
               avg_duration_ms: scope.avg_duration&.round || 0,
               success_rate: calculate_success_rate(scope)

data/app/models/ruby_llm/agents/execution/scopes.rb

@@ -149,12 +149,10 @@ module RubyLLM
               else
                 joined.where("json_extract(#{detail_table}.parameters, ?) IS NOT NULL", "$.#{key}")
               end
+            elsif value
+              joined.where("#{detail_table}.parameters @> ?", {key => value}.to_json)
             else
-              if value
-                joined.where("#{detail_table}.parameters @> ?", { key => value }.to_json)
-              else
-                joined.where("#{detail_table}.parameters ? :key", key: key.to_s)
-              end
+              joined.where("#{detail_table}.parameters ? :key", key: key.to_s)
             end
           end
 
@@ -299,7 +297,7 @@ module RubyLLM
             if connection.adapter_name.downcase.include?("sqlite")
               where("json_extract(metadata, ?) = 1", "$.#{key}")
             else
-              where("metadata @> ?", { key.to_s => true }.to_json)
+              where("metadata @> ?", {key.to_s => true}.to_json)
             end
           end