llm_cost_tracker 0.14.1 → 0.15.0

data/app/services/llm_cost_tracker/dashboard/filter.rb

@@ -11,6 +11,7 @@ module LlmCostTracker
         ["Streaming only", "yes"],
         ["Non-streaming only", "no"]
       ].freeze
+      STREAM_FILTER_LABELS = { "yes" => "Streaming", "no" => "Non-streaming" }.freeze
 
       class << self
         def call(scope: LlmCostTracker::Call.all, params: {}, tags: {})
@@ -39,17 +40,13 @@ module LlmCostTracker
       attr_reader :scope, :params, :extra_tags
 
       def apply_date_filters(relation)
-        from_date = Dashboard::DateRange.parse(params, :from)
-        to_date = Dashboard::DateRange.parse(params, :to)
-        Dashboard::DateRange.validate!(from: from_date, to: to_date)
-
-        default_range = Dashboard::DateRange.call(params: params)
-        from_date ||= default_range.from
-        to_date ||= default_range.to
+        Dashboard::DateRange.validate!(from: Dashboard::DateRange.parse(params, :from),
+                                       to: Dashboard::DateRange.parse(params, :to))
+        range = Dashboard::DateRange.call(params: params)
 
         relation
-          .where(tracked_at: from_date.beginning_of_day..)
-          .where(tracked_at: ..to_date.end_of_day)
+          .where(tracked_at: range.from.beginning_of_day..)
+          .where(tracked_at: ..range.to.end_of_day)
       end
 
       def apply_exact_filter(relation, key)
@@ -94,7 +91,7 @@ module LlmCostTracker
 
       def matchable?(values)
         values.none? { |value| value.include?("\0") } ||
-          !LlmCostTracker::Ledger::Schema::Adapter.postgresql?(scope.connection)
+          !LlmCostTracker::Ledger::Schema::Adapter.postgresql?(scope.lease_connection)
       end
 
       def normalized_string(value, name)

data/app/services/llm_cost_tracker/dashboard/monthly_budget.rb

@@ -2,29 +2,22 @@
 
 module LlmCostTracker
   module Dashboard
-    module MonthlyBudget
-      module_function
-
-      def status
+    class MonthlyBudget
+      def self.status
         budget = LlmCostTracker.configuration.budgets.monthly
         return nil unless budget
 
-        budget = budget.to_f
-        now = Time.now.utc
-        month_start = now.beginning_of_month
-        month_end = now.end_of_month
-        spent = LlmCostTracker::Ledger::Period::Totals.call(%i[month], time: now).fetch(:month)
-        elapsed_seconds = now - month_start
-        total_seconds = month_end - month_start
-        projected_spent = if spent.zero? || !elapsed_seconds.positive?
-                            spent
-                          else
-                            spent * (total_seconds / elapsed_seconds)
-                          end
-        percent_used = budget.positive? ? (spent / budget) * 100.0 : 0.0
-        projected_percent_used = budget.positive? ? (projected_spent / budget) * 100.0 : 0.0
-        projected_delta = projected_spent - budget
+        new(budget.to_f, Time.now.utc).status
+      end
 
+      def initialize(budget, now)
+        @budget = budget
+        @now = now
+        @spent = LlmCostTracker::Ledger::Period::Totals.call(%i[month], time: now).fetch(:month)
+        @projected_spent = project(@spent)
+      end
+
+      def status
         {
           budget: budget,
           spent: spent,
@@ -32,31 +25,45 @@ module LlmCostTracker
           projected_spent: projected_spent,
           projected_percent_used: projected_percent_used,
           projected_delta: projected_delta,
-          projection_end_label: month_end.strftime("%b %-d"),
-          fill_modifier: fill_modifier(percent_used),
-          progress_percent: clamped_percent(percent_used),
-          projected_marker_percent: clamped_percent(projected_percent_used),
-          projected_delta_amount: projected_delta.abs,
-          projected_delta_direction: projected_delta.positive? ? "over" : "under",
-          projected_delta_status_class: projected_delta_status_class(projected_delta)
+          projection_end_label: now.end_of_month.strftime("%b %-d"),
+          fill_modifier: fill_modifier,
+          progress_percent: percent_used.clamp(0.0, 100.0),
+          projected_marker_percent: projected_percent_used.clamp(0.0, 100.0),
+          **projected_delta_labels
         }
       end
 
-      def clamped_percent(value)
-        value.clamp(0.0, 100.0)
+      private
+
+      attr_reader :budget, :now, :spent, :projected_spent
+
+      def project(amount)
+        month_start = now.beginning_of_month
+        elapsed_seconds = now - month_start
+        return amount if amount.zero? || !elapsed_seconds.positive?
+
+        amount * ((now.end_of_month - month_start) / elapsed_seconds)
       end
 
-      def fill_modifier(percent)
-        return "lct-budget-fill--over" if percent >= 100.0
-        return "lct-budget-fill--warn" if percent >= 80.0
+      def percent_used = share_of_budget(spent)
+      def projected_percent_used = share_of_budget(projected_spent)
+      def share_of_budget(amount) = budget.positive? ? (amount / budget) * 100.0 : 0.0
+      def projected_delta = projected_spent - budget
+
+      def fill_modifier
+        return "lct-budget-fill--over" if percent_used >= 100.0
+        return "lct-budget-fill--warn" if percent_used >= 80.0
 
         ""
       end
 
-      def projected_delta_status_class(delta)
-        return "lct-budget-projection-status--over" if delta.positive?
-
-        "lct-budget-projection-status--under"
+      def projected_delta_labels
+        direction = projected_delta.positive? ? "over" : "under"
+        {
+          projected_delta_amount: projected_delta.abs,
+          projected_delta_direction: direction,
+          projected_delta_status_class: "lct-budget-projection-status--#{direction}"
+        }
       end
     end
   end

data/app/services/llm_cost_tracker/dashboard/pagination.rb

@@ -19,11 +19,8 @@ module LlmCostTracker
         )
       end
 
-      def self.integer_param(params, key, default:, min:, max: nil)
-        value = Integer(params[key], 10)
-        value = [value, min].max
-        value = [value, max].min if max
-        value
+      def self.integer_param(params, key, default:, min:, max:)
+        Integer(params[key], 10).clamp(min, max)
       rescue ArgumentError, TypeError
         default
       end

data/app/services/llm_cost_tracker/dashboard/percent.rb

@@ -0,0 +1,12 @@
+# frozen_string_literal: true
+
+module LlmCostTracker
+  module Dashboard
+    module Percent
+      def self.of(part, whole)
+        whole = whole.to_f
+        whole.positive? ? (part.to_f / whole) * 100.0 : 0.0
+      end
+    end
+  end
+end

data/app/services/llm_cost_tracker/dashboard/pricing_overview.rb

@@ -63,9 +63,10 @@ module LlmCostTracker
       end
 
       def build_rows(prices)
+        today = Time.now.utc.to_date.iso8601
         rows = prices.map do |key, rates|
           provider, model = split_key(key.to_s)
-          Row.new(provider: provider, model: model, rates: rates)
+          Row.new(provider: provider, model: model, rates: Pricing::Matcher.prices_on(rates, today))
         end
         rows.sort_by { |row| [row.provider || "~", row.model] }
       end

data/app/services/llm_cost_tracker/dashboard/sort.rb

@@ -4,6 +4,15 @@ module LlmCostTracker
   module Dashboard
     module Sort
       DIRECTIONS = %w[asc desc].freeze
+
+      Choice = Data.define(:column, :direction)
+
+      def self.resolve(column, direction, natural_directions:, fallback:)
+        column = natural_directions.key?(column.to_s) ? column.to_s : fallback
+        direction = direction.to_s
+        direction = natural_directions.fetch(column) unless DIRECTIONS.include?(direction)
+        Choice.new(column: column, direction: direction)
+      end
     end
   end
 end

data/app/services/llm_cost_tracker/dashboard/spend_anomaly.rb

@@ -28,18 +28,14 @@ module LlmCostTracker
       attr_reader :scope, :from, :to
 
       def alerts
-        window_days = WINDOW_DAYS.to_f
-        daily_spend_by_model.each_with_object([]) do |((provider, model), daily_costs), rows|
+        daily_spend_by_model.filter_map do |(provider, model), daily_costs|
           latest_spend = daily_costs.fetch(to, 0.0)
           next unless latest_spend.positive?
 
-          baseline_days = ((to - WINDOW_DAYS)...to).map { |day| daily_costs.fetch(day, 0.0) }
-          mean = baseline_days.sum / window_days
-          variance = baseline_days.sum { |value| (value - mean)**2 } / window_days
-          threshold = mean + (2 * Math.sqrt(variance))
-          next unless latest_spend > threshold
+          mean, deviation = baseline(daily_costs)
+          next unless latest_spend > mean + (2 * deviation)
 
-          rows << {
+          {
             provider: provider,
             model: model,
             day: to,
@@ -50,22 +46,28 @@ module LlmCostTracker
         end
       end
 
-      def daily_spend_by_model
-        window = (to - WINDOW_DAYS).beginning_of_day..to.end_of_day
+      def baseline(daily_costs)
+        days = ((to - WINDOW_DAYS)...to).map { |day| daily_costs.fetch(day, 0.0) }
+        mean = days.sum / WINDOW_DAYS.to_f
+        variance = days.sum { |value| (value - mean)**2 } / WINDOW_DAYS.to_f
+        [mean, Math.sqrt(variance)]
+      end
 
-        grouped = Hash.new { |hash, key| hash[key] = Hash.new(0.0) }
+      def daily_spend_by_model
+        daily_costs = Hash.new { |hash, key| hash[key] = Hash.new(0.0) }
+        daily_totals.each do |(provider, model, day), total_cost|
+          daily_costs[[provider, model]][Date.iso8601(day.to_s)] += total_cost.to_f
+        end
+        daily_costs
+      end
 
+      def daily_totals
         scope
-          .where(tracked_at: window)
+          .where(tracked_at: (to - WINDOW_DAYS).beginning_of_day..to.end_of_day)
           .where.not(total_cost: nil)
           .group(:provider, :model)
-          .group_by_period(:day)
+          .group_by_period(:day, time_zone: Time.zone)
           .sum(:total_cost)
-          .each do |(provider, model, day), total_cost|
-            grouped[[provider, model]][Date.iso8601(day.to_s)] += total_cost.to_f
-          end
-
-        grouped
       end
     end
   end

data/app/services/llm_cost_tracker/dashboard/spend_chart.rb

@@ -0,0 +1,50 @@
+# frozen_string_literal: true
+
+module LlmCostTracker
+  module Dashboard
+    class SpendChart
+      WIDTH = 1180
+      PADDING = { top: 16, right: 16, bottom: 28, left: 56 }.freeze
+      MIN_MAX_COST = 0.0001
+
+      attr_reader :points, :height, :y_ticks, :max_cost, :coords, :comparison_coords
+
+      def initialize(points, comparison_points: nil, height: 180, y_ticks: 3)
+        @points = points
+        @height = height
+        @y_ticks = y_ticks
+        costs = points.map { |point| point[:cost].to_f } + Array(comparison_points).map { |point| point[:cost].to_f }
+        @max_cost = [costs.max.to_f, MIN_MAX_COST].max
+        @coords = coords_for(points)
+        @comparison_coords = coords_for(comparison_points) if comparison_points.present?
+      end
+
+      def peak_index
+        @peak_index ||= points.each_with_index.max_by { |point, _| point[:cost].to_f }.last
+      end
+
+      def left = PADDING[:left]
+      def right = left + plot_width
+      def baseline = PADDING[:top] + plot_height
+      def tick_y(index) = PADDING[:top] + (plot_height * index.to_f / y_ticks)
+      def tick_value(index) = max_cost * (y_ticks - index).to_f / y_ticks
+
+      def label_indexes
+        count = points.size
+        count <= 2 ? (0...count).to_a : [0, count / 2, count - 1].uniq
+      end
+
+      private
+
+      def plot_width = WIDTH - PADDING[:left] - PADDING[:right]
+      def plot_height = height - PADDING[:top] - PADDING[:bottom]
+
+      def coords_for(series)
+        step = series.size > 1 ? plot_width.to_f / (series.size - 1) : 0.0
+        series.each_with_index.map do |point, index|
+          [left + (index * step), baseline - ((point[:cost].to_f / max_cost) * plot_height)]
+        end
+      end
+    end
+  end
+end

data/app/services/llm_cost_tracker/dashboard/tag_breakdown.rb

@@ -4,7 +4,6 @@ module LlmCostTracker
   module Dashboard
     class TagBreakdown
       DEFAULT_LIMIT = 100
-      SORT_OPTIONS = %w[value calls cost avg_cost].freeze
       DEFAULT_DIRECTIONS = { "value" => "asc", "calls" => "desc", "cost" => "desc", "avg_cost" => "desc" }.freeze
       Row = Data.define(:value, :calls, :total_cost, :average_cost_per_call, :share_percent)
 
@@ -21,8 +20,7 @@ module LlmCostTracker
         @key = LlmCostTracker::Tags::Key.validate!(key, error_class: LlmCostTracker::InvalidFilterError)
         limit = limit.to_i
         @limit = limit.positive? ? [limit, DEFAULT_LIMIT].min : DEFAULT_LIMIT
-        @sort = SORT_OPTIONS.include?(sort.to_s) ? sort.to_s : "cost"
-        @direction = Sort::DIRECTIONS.include?(direction.to_s) ? direction.to_s : DEFAULT_DIRECTIONS[@sort]
+        @order = Sort.resolve(sort, direction, natural_directions: DEFAULT_DIRECTIONS, fallback: "cost")
       end
 
       def rows
@@ -35,7 +33,7 @@ module LlmCostTracker
               calls: calls,
               total_cost: row.total_cost,
               average_cost_per_call: row.average_cost_per_call,
-              share_percent: total.positive? ? (calls.to_f / total) * 100.0 : 0.0
+              share_percent: Percent.of(calls, total)
             )
           end
         end
@@ -55,7 +53,7 @@ module LlmCostTracker
 
       private
 
-      attr_reader :scope, :key, :sort, :direction
+      attr_reader :scope, :key, :order
 
       def summary_counts
         @summary_counts ||= scope.klass.find_by_sql(summary_sql).first
@@ -77,8 +75,8 @@ module LlmCostTracker
       end
 
       def order_clause
-        dir = direction.upcase
-        case sort
+        dir = order.direction.upcase
+        case order.column
         when "value"    then "#{tag_value_column} #{dir}"
         when "calls"    then "COUNT(*) #{dir}, total_cost DESC"
         when "avg_cost" then "average_cost_per_call #{dir}, total_cost DESC"
@@ -89,7 +87,7 @@ module LlmCostTracker
       def summary_sql
         <<~SQL.squish
           SELECT COUNT(*) AS total_calls,
-                 COUNT(t.#{quote_column('value')}) AS tagged_calls,
+                 COUNT(CASE WHEN #{tag_present_predicate} THEN 1 END) AS tagged_calls,
                  COUNT(DISTINCT CASE WHEN #{tag_present_predicate} THEN #{tag_value_column} END) AS distinct_values
           FROM (#{scope.to_sql}) AS sub
           LEFT OUTER JOIN #{call_tag_table} t ON t.llm_cost_tracker_call_id = sub.id AND t.#{quote_column('key')} = #{quoted_key}
@@ -109,11 +107,11 @@ module LlmCostTracker
       end
 
       def quote_column(name)
-        scope.connection.quote_column_name(name)
+        scope.lease_connection.quote_column_name(name)
       end
 
       def quoted_key
-        scope.connection.quote(key)
+        scope.lease_connection.quote(key)
       end
     end
   end

data/app/services/llm_cost_tracker/dashboard/tag_key_explorer.rb

@@ -13,16 +13,13 @@ module LlmCostTracker
 
       def initialize(scope:, limit:)
         @scope = scope
-        @connection = LlmCostTracker::Call.connection
+        @connection = LlmCostTracker::Call.lease_connection
         limit = limit.to_i
         @limit = limit.positive? ? [limit, DEFAULT_LIMIT].min : DEFAULT_LIMIT
       end
 
       def rows
         scope.klass.find_by_sql(build_sql)
-      rescue ActiveRecord::StatementInvalid => e
-        LlmCostTracker::Logging.warn("Tag key discovery failed (#{connection.adapter_name}): #{e.class}: #{e.message}")
-        []
       end
 
       private
@@ -38,6 +35,7 @@ module LlmCostTracker
                  COUNT(DISTINCT t.#{value_column}) AS distinct_values
           FROM (#{scope.to_sql}) AS sub
           INNER JOIN #{tags_table} t ON t.llm_cost_tracker_call_id = sub.id
+          WHERE t.#{value_column} != ''
           GROUP BY t.#{key_column}
           ORDER BY calls_count DESC
           LIMIT #{limit}

data/app/services/llm_cost_tracker/dashboard/time_series.rb

@@ -5,18 +5,16 @@ require "date"
 module LlmCostTracker
   module Dashboard
     class TimeSeries
-      DEFAULT_DAYS = 30
-
       class << self
-        def call(scope: LlmCostTracker::Call.all, from: nil, to: Date.current)
+        def call(from:, to:, scope: LlmCostTracker::Call.all)
           new(scope: scope, from: from, to: to).points
         end
       end
 
       def initialize(scope:, from:, to:)
         @scope = scope
+        @from = from.to_date
         @to = to.to_date
-        @from = from ? from.to_date : (@to - (DEFAULT_DAYS - 1))
       end
 
       def points
@@ -35,7 +33,7 @@ module LlmCostTracker
       def scoped_costs
         scope
           .where(tracked_at: from.beginning_of_day..to.end_of_day)
-          .group_by_period(:day)
+          .group_by_period(:day, time_zone: Time.zone)
           .sum(:total_cost)
           .transform_values(&:to_f)
       end

data/app/services/llm_cost_tracker/dashboard/top_models.rb

@@ -4,7 +4,6 @@ module LlmCostTracker
   module Dashboard
     class TopModels
       DEFAULT_LIMIT = 5
-      SORT_OPTIONS = %w[cost calls avg_cost latency tokens provider name].freeze
       DEFAULT_SORT = "cost"
       DEFAULT_DIRECTIONS = {
         "provider" => "asc",
@@ -43,21 +42,20 @@ module LlmCostTracker
       def initialize(scope:, limit:, sort: DEFAULT_SORT, direction: nil)
         @scope = scope
         @limit = limit
-        @sort = SORT_OPTIONS.include?(sort.to_s) ? sort.to_s : DEFAULT_SORT
-        @direction = Sort::DIRECTIONS.include?(direction.to_s) ? direction.to_s : DEFAULT_DIRECTIONS[@sort]
+        @order = Sort.resolve(sort, direction, natural_directions: DEFAULT_DIRECTIONS, fallback: DEFAULT_SORT)
       end
 
       def rows
         scope
           .group(:provider, :model)
           .select(selects)
-          .order(*ORDER_NODES.fetch([sort, direction]))
+          .order(*ORDER_NODES.fetch([order.column, order.direction]))
           .then { |r| limit ? r.limit(limit) : r }
       end
 
       private
 
-      attr_reader :scope, :limit, :sort, :direction
+      attr_reader :scope, :limit, :order
 
       def selects
         columns = [

data/app/views/layouts/llm_cost_tracker/application.html.erb

@@ -54,28 +54,28 @@
         <aside class="lct-sidebar" aria-label="Dashboard">
           <div class="lct-sidebar-inner">
           <div class="lct-sidebar-section">Insights</div>
-          <%= link_to root_path, class: ("lct-sidebar-link lct-active" if dashboard_section == :overview) || "lct-sidebar-link", aria: (dashboard_section == :overview ? { current: "page" } : {}) do %>
+          <%= link_to root_path, sidebar_link_options(:overview) do %>
             <svg viewBox="0 0 24 24" fill="none" stroke="currentColor" stroke-width="2"><rect x="3" y="3" width="7" height="9"/><rect x="14" y="3" width="7" height="5"/><rect x="14" y="12" width="7" height="9"/><rect x="3" y="16" width="7" height="5"/></svg>
             Overview
           <% end %>
-          <%= link_to models_path, class: ("lct-sidebar-link lct-active" if dashboard_section == :models) || "lct-sidebar-link", aria: (dashboard_section == :models ? { current: "page" } : {}) do %>
+          <%= link_to models_path, sidebar_link_options(:models) do %>
             <svg viewBox="0 0 24 24" fill="none" stroke="currentColor" stroke-width="2" stroke-linecap="round" stroke-linejoin="round"><path d="M21 8a2 2 0 0 0-1-1.73l-7-4a2 2 0 0 0-2 0l-7 4A2 2 0 0 0 3 8v8a2 2 0 0 0 1 1.73l7 4a2 2 0 0 0 2 0l7-4A2 2 0 0 0 21 16Z"/><path d="m3.3 7 8.7 5 8.7-5"/><path d="M12 22V12"/></svg>
             Models
           <% end %>
-          <%= link_to calls_path, class: ("lct-sidebar-link lct-active" if dashboard_section == :calls) || "lct-sidebar-link", aria: (dashboard_section == :calls ? { current: "page" } : {}) do %>
+          <%= link_to calls_path, sidebar_link_options(:calls) do %>
             <svg viewBox="0 0 24 24" fill="none" stroke="currentColor" stroke-width="2"><path d="M21 15a2 2 0 0 1-2 2H7l-4 4V5a2 2 0 0 1 2-2h14a2 2 0 0 1 2 2z"/></svg>
             Calls
           <% end %>
-          <%= link_to tags_path, class: ("lct-sidebar-link lct-active" if dashboard_section == :tags) || "lct-sidebar-link", aria: (dashboard_section == :tags ? { current: "page" } : {}) do %>
+          <%= link_to tags_path, sidebar_link_options(:tags) do %>
             <svg viewBox="0 0 24 24" fill="none" stroke="currentColor" stroke-width="2"><path d="M20.59 13.41l-7.17 7.17a2 2 0 0 1-2.83 0L2 12V2h10l8.59 8.59a2 2 0 0 1 0 2.82z"/><line x1="7" y1="7" x2="7.01" y2="7"/></svg>
             Tags
           <% end %>
-          <%= link_to data_quality_path, class: ("lct-sidebar-link lct-active" if dashboard_section == :data_quality) || "lct-sidebar-link", aria: (dashboard_section == :data_quality ? { current: "page" } : {}) do %>
+          <%= link_to data_quality_path, sidebar_link_options(:data_quality) do %>
             <svg viewBox="0 0 24 24" fill="none" stroke="currentColor" stroke-width="2"><path d="M22 11.08V12a10 10 0 1 1-5.93-9.14"/><polyline points="22 4 12 14.01 9 11.01"/></svg>
             Data Quality
           <% end %>
           <div class="lct-sidebar-section">Reference</div>
-          <%= link_to pricing_path, class: ("lct-sidebar-link lct-active" if dashboard_section == :pricing) || "lct-sidebar-link", aria: (dashboard_section == :pricing ? { current: "page" } : {}) do %>
+          <%= link_to pricing_path, sidebar_link_options(:pricing) do %>
             <svg viewBox="0 0 24 24" fill="none" stroke="currentColor" stroke-width="2" stroke-linecap="round" stroke-linejoin="round"><line x1="12" y1="1" x2="12" y2="23"/><path d="M17 5H9.5a3.5 3.5 0 0 0 0 7h5a3.5 3.5 0 0 1 0 7H6"/></svg>
             Pricing
           <% end %>

data/app/views/llm_cost_tracker/calls/index.html.erb

@@ -19,7 +19,8 @@
   <% end %>
 
   <span class="lct-filter-row-meta"><%= number_with_delimiter(@calls_count) %> call<%= "s" unless @calls_count == 1 %> · <%= money(@calls_total_cost) %></span>
-  <%= link_to "Export CSV", calls_path(current_query(format: :csv)), class: "lct-page-link", title: "Capped at #{number_with_delimiter(LlmCostTracker::CallsController::CSV_EXPORT_LIMIT)} rows per request" %>
+  <% export_limit = LlmCostTracker::CallsController::CSV_EXPORT_LIMIT %>
+  <%= link_to(@calls_count > export_limit ? "Export CSV (first #{number_with_delimiter(export_limit)})" : "Export CSV", calls_path(current_query(format: :csv)), class: "lct-page-link", title: "Capped at #{number_with_delimiter(export_limit)} rows per request") %>
 </div>
 
 <% if @calls_count.zero? %>

data/app/views/llm_cost_tracker/calls/show.html.erb

@@ -1,18 +1,4 @@
-<% priced_components = token_usage_stack_components %>
-<% line_item_costs_by_component = call_line_item_costs_by_component(@call) %>
-<% token_segments = priced_components.map do |component|
-  token_key = component.fetch(:token_key)
-  value = @call[token_key]
-  { label: component.fetch(:label), value: value, formatted_value: number_with_delimiter(value), css_class: component.fetch(:css_class) }
-end %>
-<% cost_segments = [] %>
-<% unless @call.total_cost.nil? %>
-  <% cost_segments = priced_components.map do |component|
-    value = line_item_costs_by_component[component.fetch(:price_key)]
-    { label: component.fetch(:label), value: value, formatted_value: optional_money(value), css_class: component.fetch(:css_class) }
-  end %>
-<% end %>
-<% service_line_items = @call.line_items.reject { |li| li.unit == "token" }.sort_by(&:position) %>
+<% service_line_items = call_service_line_items(@call) %>
 
 <p class="lct-breadcrumb-back"><%= link_to "← All calls", calls_path %></p>
 <h2 class="lct-page-title"><code class="lct-code-id"><%= @call.model %></code> <span class="lct-page-title-meta">#<%= @call.id %></span></h2>
@@ -45,7 +31,7 @@ end %>
   <section class="lct-panel">
     <div class="lct-panel-head"><h2 class="lct-panel-title">Token mix</h2></div>
     <div class="lct-panel-body">
-      <%= render "llm_cost_tracker/shared/metric_stack", segments: token_segments, empty_message: "No token data for this call." %>
+      <%= render "llm_cost_tracker/shared/metric_stack", segments: call_token_segments(@call), empty_message: "No token data for this call." %>
     </div>
   </section>
 
@@ -55,7 +41,7 @@ end %>
       <% if @call.total_cost.nil? %>
         <p class="lct-stack-empty">Pricing not available for this call.</p>
       <% else %>
-        <%= render "llm_cost_tracker/shared/metric_stack", segments: cost_segments, empty_message: "No cost breakdown for this call." %>
+        <%= render "llm_cost_tracker/shared/metric_stack", segments: call_cost_segments(@call), empty_message: "No cost breakdown for this call." %>
       <% end %>
     </div>
   </section>
@@ -84,22 +70,11 @@ end %>
         <dd><%= content_tag(:code, @call.provider_response_id, class: "lct-code-id") %></dd>
       </div>
     <% end %>
-    <% if @call.provider_project_id.present? %>
-      <div class="lct-meta-strip-item">
-        <dt>Project ID</dt>
-        <dd><%= content_tag(:code, LlmCostTracker::Dashboard::Masking.mask_value(:provider_project_id, @call.provider_project_id), class: "lct-code-id") %></dd>
-      </div>
-    <% end %>
-    <% if @call.provider_api_key_id.present? %>
-      <div class="lct-meta-strip-item">
-        <dt>API Key ID</dt>
-        <dd><%= content_tag(:code, LlmCostTracker::Dashboard::Masking.mask_value(:provider_api_key_id, @call.provider_api_key_id), class: "lct-code-id") %></dd>
-      </div>
-    <% end %>
-    <% if @call.provider_workspace_id.present? %>
+    <% { provider_project_id: "Project ID", provider_api_key_id: "API Key ID", provider_workspace_id: "Workspace ID" }.each do |attribute, label| %>
+      <% next if @call[attribute].blank? %>
       <div class="lct-meta-strip-item">
-        <dt>Workspace ID</dt>
-        <dd><%= content_tag(:code, LlmCostTracker::Dashboard::Masking.mask_value(:provider_workspace_id, @call.provider_workspace_id), class: "lct-code-id") %></dd>
+        <dt><%= label %></dt>
+        <dd><%= content_tag(:code, LlmCostTracker::Dashboard::Masking.mask_value(attribute, @call[attribute]), class: "lct-code-id") %></dd>
       </div>
     <% end %>
   </dl>

data/app/views/llm_cost_tracker/data_quality/index.html.erb

@@ -271,7 +271,7 @@
         <h2 class="lct-panel-title">Incomplete pricing by model</h2>
         <span class="lct-panel-meta"><%= link_to "View calls →", calls_path(current_query(cost_status: "incomplete", sort: nil)) %></span>
       </div>
-      <p class="lct-panel-intro">These models have line items without configured rates, so totals undercount (a full row is missing all rates; a partial row has some line items priced and others not). After the next price refresh or a <code>pricing_overrides</code> update, run <code>bin/rails llm_cost_tracker:backfill_unknown_pricing</code> to recompute these calls.</p>
+      <p class="lct-panel-intro">These models have line items without configured rates, so totals undercount (a full row is missing all rates; a partial row has some line items priced and others not). After the next price refresh or a <code>pricing_overrides</code> update, run <code>bin/rails llm_cost_tracker:backfill_unknown_pricing</code> to recompute these calls. A partial call is left as recorded when a rate it already used has changed since.</p>
 
       <div class="lct-table-wrap">
         <table class="lct-tbl">

data/app/views/llm_cost_tracker/errors/query.html.erb

@@ -0,0 +1,6 @@
+<section class="lct-panel lct-empty">
+  <h2 class="lct-state-title">Query failed</h2>
+  <p class="lct-state-copy">
+    A dashboard query failed, for example on a statement timeout or a lock. The error is in your application log.
+  </p>
+</section>

data/app/views/llm_cost_tracker/shared/_bar.html.erb

@@ -1,5 +1,3 @@
-<% variant_class = local_assigns[:variant] == "budget" ? " lct-budget-fill" : "" %>
-
 <div class="lct-bar-track" aria-hidden="true">
-  <div data-lct-style="<%= inline_style("width: #{bar_width(value, max)}") %>" class="lct-bar-fill<%= variant_class %>"></div>
+  <div data-lct-style="<%= inline_style("width: #{bar_width(value, max)}") %>" class="lct-bar-fill"></div>
 </div>

data/app/views/llm_cost_tracker/shared/_filter_pill_date.html.erb

@@ -1,5 +1,3 @@
-<% extra_hidden = local_assigns.fetch(:extra_hidden, {}) %>
-<% extra_except = local_assigns.fetch(:extra_except, []) %>
 <details class="lct-filter-pop" name="lct-filter">
   <summary class="lct-filter-pill">
     <svg width="12" height="12" viewBox="0 0 24 24" fill="none" stroke="currentColor" stroke-width="2"><rect x="3" y="4" width="18" height="18" rx="2"/><path d="M16 2v4M8 2v4M3 10h18"/></svg>
@@ -8,8 +6,7 @@
     <svg class="lct-chev" width="10" height="6" viewBox="0 0 10 6" fill="none" stroke="currentColor" stroke-width="1.5" stroke-linecap="round" stroke-linejoin="round"><path d="M1 1l4 4 4-4"/></svg>
   </summary>
   <%= form_with url: path, method: :get, local: true, html: { class: "lct-filter-pop-body" } do %>
-    <% extra_hidden.each do |k, v| %><%= hidden_field_tag k, v %><% end %>
-    <%= hidden_query_fields(current_query.except(:from, :to, :page, :per, *extra_except)) %>
+    <%= hidden_query_fields(current_query.except(:from, :to, :page, :per)) %>
     <div class="lct-filter-pop-field"><label for="lct-filter-from">From</label><input type="date" name="from" id="lct-filter-from" value="<%= @from_date.iso8601 %>"></div>
     <div class="lct-filter-pop-field"><label for="lct-filter-to">To</label><input type="date" name="to" id="lct-filter-to" value="<%= @to_date.iso8601 %>"></div>
     <button type="submit" class="lct-button lct-button-primary">Apply</button>