llm_cost_tracker 0.11.0 → 0.13.0

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

@@ -35,7 +35,7 @@ module LlmCostTracker
               calls: calls,
               total_cost: row.total_cost,
               average_cost_per_call: row.average_cost_per_call,
-              share_percent: percentage(calls, total)
+              share_percent: total.positive? ? (calls.to_f / total) * 100.0 : 0.0
             )
           end
         end
@@ -115,12 +115,6 @@ module LlmCostTracker
       def quoted_key
         scope.connection.quote(key)
       end
-
-      def percentage(numerator, denominator)
-        return 0.0 unless denominator.positive?
-
-        (numerator / denominator.to_f) * 100.0
-      end
     end
   end
 end

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

@@ -20,7 +20,7 @@ module LlmCostTracker
 
       def rows
         scope.klass.find_by_sql(build_sql)
-      rescue StandardError => e
+      rescue ActiveRecord::StatementInvalid => e
         LlmCostTracker::Logging.warn("Tag key discovery failed (#{connection.adapter_name}): #{e.class}: #{e.message}")
         []
       end

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

@@ -79,12 +79,6 @@
             <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 %>
-          <% if LlmCostTracker.reconciliation_enabled? %>
-            <%= link_to reconciliation_path, class: ("lct-sidebar-link lct-active" if dashboard_section == :reconciliation) || "lct-sidebar-link", aria: (dashboard_section == :reconciliation ? { current: "page" } : {}) do %>
-              <svg viewBox="0 0 24 24" fill="none" stroke="currentColor" stroke-width="2"><polyline points="23 4 23 10 17 10"/><polyline points="1 20 1 14 7 14"/><path d="M3.51 9a9 9 0 0 1 14.85-3.36L23 10M1 14l4.64 4.36A9 9 0 0 0 20.49 15"/></svg>
-              Reconciliation
-            <% end %>
-          <% end %>
           </div>
         </aside>
 

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

@@ -18,8 +18,8 @@
     <%= link_to "× Clear filters", calls_path(current_query(provider: nil, model: nil, stream: nil, cost_status: nil, page: nil)), class: "lct-filter-clear" %>
   <% end %>
 
-  <span class="lct-filter-row-meta"><%= number(@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(LlmCostTracker::CallsController::CSV_EXPORT_LIMIT)} rows per request" %>
+  <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" %>
 </div>
 
 <% if @calls_count.zero? %>
@@ -32,7 +32,7 @@
     <table class="lct-tbl">
       <thead>
         <tr>
-          <%= sortable_header("Tracked at", "tracked_at", num: true) %>
+          <%= sortable_header("Tracked at", "tracked_at", num: true, default: true) %>
           <%= sortable_header("Provider", "provider") %>
           <%= sortable_header("Model", "model") %>
           <%= sortable_header("Input", "input", num: true) %>
@@ -49,11 +49,11 @@
             <td><%= format_date(call.tracked_at) %></td>
             <td><span class="lct-model-cell"><span class="lct-provider-dot lct-provider-dot-<%= call.provider %>"></span><%= call.provider %></span></td>
             <td><code class="lct-code-id"><%= call.model %></code></td>
-            <td class="lct-num"><%= number(call.input_tokens) %></td>
-            <td class="lct-num"><%= number(call.output_tokens) %></td>
+            <td class="lct-num"><%= number_with_delimiter(call.input_tokens) %></td>
+            <td class="lct-num"><%= number_with_delimiter(call.output_tokens) %></td>
             <td class="lct-num<%= ' lct-num-muted' if call.total_cost.nil? %>"><%= optional_money(call.total_cost) %></td>
-            <td class="lct-num<%= ' lct-num-muted' if call.latency_ms.nil? %>"><%= call.latency_ms ? "#{number(call.latency_ms)}ms" : "n/a" %></td>
-            <td><%= render "llm_cost_tracker/shared/tag_chips", tags: call.parsed_tags %></td>
+            <td class="lct-num<%= ' lct-num-muted' if call.latency_ms.nil? %>"><%= call.latency_ms ? "#{number_with_delimiter(call.latency_ms)}ms" : "n/a" %></td>
+            <td><%= render "llm_cost_tracker/shared/tag_chips", tags: call.tag_pairs %></td>
             <td class="lct-num"><%= link_to "Details", call_path(call), class: "lct-page-link" %></td>
           </tr>
         <% end %>
@@ -64,7 +64,7 @@
     <% last_row = [@page.offset + @calls.length, @calls_count].min %>
     <% total_pages = @page.total_pages(@calls_count) %>
     <div class="lct-pagination">
-      <span class="lct-pagination-info">Showing <strong><%= number(first_row) %></strong>–<strong><%= number(last_row) %></strong> of <strong><%= number(@calls_count) %></strong></span>
+      <span class="lct-pagination-info">Showing <strong><%= number_with_delimiter(first_row) %></strong>–<strong><%= number_with_delimiter(last_row) %></strong> of <strong><%= number_with_delimiter(@calls_count) %></strong></span>
       <span class="lct-seg lct-pagination-perpage" title="Rows per page">
         <% LlmCostTracker::PaginationHelper::PER_PAGE_CHOICES.each do |choice| %>
           <% if choice == @page.per %>

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

@@ -3,7 +3,7 @@
 <% token_segments = priced_components.map do |component|
   token_key = component.fetch(:token_key)
   value = @call.has_attribute?(token_key) ? @call[token_key] : 0
-  { label: component.fetch(:label), value: value, formatted_value: number(value), css_class: component.fetch(:css_class) }
+  { 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? %>
@@ -33,11 +33,11 @@ end %>
   </div>
   <div class="lct-stat">
     <div class="lct-stat-head"><p class="lct-stat-label">Total tokens</p></div>
-    <p class="lct-stat-value"><%= number(@call.total_tokens) %></p>
+    <p class="lct-stat-value"><%= number_with_delimiter(@call.total_tokens) %></p>
   </div>
   <div class="lct-stat">
     <div class="lct-stat-head"><p class="lct-stat-label">Latency</p></div>
-    <p class="lct-stat-value<%= ' lct-num-muted' if @call.latency_ms.nil? %>"><%= @call.latency_ms ? number(@call.latency_ms) : "n/a" %><% if @call.latency_ms %><span class="lct-stat-unit">ms</span><% end %></p>
+    <p class="lct-stat-value<%= ' lct-num-muted' if @call.latency_ms.nil? %>"><%= @call.latency_ms ? number_with_delimiter(@call.latency_ms) : "n/a" %><% if @call.latency_ms %><span class="lct-stat-unit">ms</span><% end %></p>
   </div>
 </div>
 
@@ -80,33 +80,41 @@ end %>
         <dd><code class="lct-code-id"><%= @call.usage_source %></code></dd>
       </div>
     <% end %>
-    <div class="lct-meta-strip-item">
-      <dt>Response ID</dt>
-      <dd><%= @call.provider_response_id.present? ? content_tag(:code, @call.provider_response_id, class: "lct-code-id") : "n/a" %></dd>
-    </div>
-    <div class="lct-meta-strip-item">
-      <dt>Project ID</dt>
-      <dd><%= @call.provider_project_id.present? ? content_tag(:code, LlmCostTracker::Masking.mask_value(:provider_project_id, @call.provider_project_id), class: "lct-code-id") : "n/a" %></dd>
-    </div>
-    <div class="lct-meta-strip-item">
-      <dt>API Key ID</dt>
-      <dd><%= @call.provider_api_key_id.present? ? content_tag(:code, LlmCostTracker::Masking.mask_value(:provider_api_key_id, @call.provider_api_key_id), class: "lct-code-id") : "n/a" %></dd>
-    </div>
-    <div class="lct-meta-strip-item">
-      <dt>Workspace ID</dt>
-      <dd><%= @call.provider_workspace_id.present? ? content_tag(:code, LlmCostTracker::Masking.mask_value(:provider_workspace_id, @call.provider_workspace_id), class: "lct-code-id") : "n/a" %></dd>
-    </div>
+    <% if @call.provider_response_id.present? %>
+      <div class="lct-meta-strip-item">
+        <dt>Response ID</dt>
+        <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? %>
+      <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>
+      </div>
+    <% end %>
   </dl>
 </section>
 
 <section class="lct-panel">
   <div class="lct-panel-head"><h2 class="lct-panel-title">Tags</h2></div>
   <div class="lct-panel-body">
-    <% if @call.parsed_tags.empty? %>
+    <% if @call.tag_pairs.empty? %>
       <p class="lct-stack-empty">(untagged)</p>
     <% else %>
       <span class="lct-tag-chips">
-        <% @call.parsed_tags.each do |k, v| %>
+        <% @call.tag_pairs.each do |k, v| %>
           <span class="lct-tag-chip"><span class="lct-tag-chip-key"><%= k %></span>=<%= v %></span>
         <% end %>
       </span>
@@ -134,9 +142,9 @@ end %>
             <td><code class="lct-code-id"><%= line_item.kind %></code></td>
             <td><%= line_item.unit %></td>
             <td class="lct-num"><%= line_item.quantity %></td>
-            <td class="lct-num"><%= line_item.rate_amount ? "#{optional_money(line_item.rate_amount)} / #{line_item.rate_quantity}" : "n/a" %></td>
-            <% unknown_cost = line_item.cost.nil? || line_item.cost_status.to_s == LlmCostTracker::Billing::CostStatus::UNKNOWN %>
-            <td class="lct-num<%= ' lct-num-muted' if unknown_cost %>"><%= unknown_cost ? "n/a" : money(line_item.cost) %></td>
+            <td class="lct-num"><%= line_item.rate_amount ? "#{optional_money(line_item.rate_amount, currency: line_item.currency)} / #{line_item.rate_quantity}" : "n/a" %></td>
+            <% unknown_cost = line_item.cost.nil? || line_item.cost_status.to_s == LlmCostTracker::Charges::CostStatus::UNKNOWN %>
+            <td class="lct-num<%= ' lct-num-muted' if unknown_cost %>"><%= unknown_cost ? "n/a" : money(line_item.cost, currency: line_item.currency) %></td>
             <td><%= line_item.cost_status %></td>
           </tr>
         <% end %>
@@ -161,6 +169,6 @@ end %>
       <h2 class="lct-panel-title">Metadata</h2>
       <span class="lct-disclose-hint">show JSON</span>
     </summary>
-    <pre class="lct-pre"><%= safe_json(LlmCostTracker::Masking.mask_hash(masked_metadata_hash(@call.read_attribute("metadata")))) %></pre>
+    <pre class="lct-pre"><%= safe_json(LlmCostTracker::Dashboard::Masking.mask_hash(masked_metadata_hash(@call.read_attribute("metadata")))) %></pre>
   </details>
 <% end %>

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

@@ -8,7 +8,7 @@
   <% end %>
 
   <span class="lct-filter-row-meta">
-    <%= number(@stats.total_calls) %> call<%= "s" unless @stats.total_calls.to_i == 1 %> · <%= money(@stats.total_cost) %>
+    <%= number_with_delimiter(@stats.total_calls) %> call<%= "s" unless @stats.total_calls.to_i == 1 %> · <%= money(@stats.total_cost) %>
   </span>
 </div>
 
@@ -22,7 +22,7 @@
     <div class="lct-alert lct-alert-warn">
       <svg width="14" height="14" viewBox="0 0 24 24" fill="none" stroke="currentColor" stroke-width="2"><path d="M12 9v4M12 17h.01M10.29 3.86L1.82 18a2 2 0 0 0 1.71 3h16.94a2 2 0 0 0 1.71-3L13.71 3.86a2 2 0 0 0-3.42 0z"/></svg>
       <span>
-        <strong><%= number(@stats.unknown_pricing_count) %> call<%= "s" unless @stats.unknown_pricing_count.to_i == 1 %> with incomplete pricing</strong>
+        <strong><%= number_with_delimiter(@stats.unknown_pricing_count) %> call<%= "s" unless @stats.unknown_pricing_count.to_i == 1 %> with incomplete pricing</strong>
         · <%= percent(coverage_percent(@stats.unknown_pricing_count, @stats.total_calls)) %> of slice
         — update <code>pricing_overrides</code> or <code>prices_file</code>.
       </span>
@@ -64,12 +64,12 @@
         <% badge = delta_badge(@stats.calls_delta_percent, mode: :neutral) %>
         <span class="<%= badge[:css_class] %>"><%= badge[:text] %></span>
       </div>
-      <p class="lct-stat-value"><%= number(@stats.total_calls) %></p>
+      <p class="lct-stat-value"><%= number_with_delimiter(@stats.total_calls) %></p>
     </div>
     <% if @stats.average_latency_ms %>
       <div class="lct-stat lct-stat-ok">
         <div class="lct-stat-head"><p class="lct-stat-label">Avg latency</p></div>
-        <p class="lct-stat-value"><%= number(@stats.average_latency_ms.round) %><span class="lct-stat-unit">ms</span></p>
+        <p class="lct-stat-value"><%= number_with_delimiter(@stats.average_latency_ms.round) %><span class="lct-stat-unit">ms</span></p>
       </div>
     <% else %>
       <div class="lct-stat">
@@ -79,7 +79,7 @@
     <% end %>
     <div class="lct-stat <%= @stats.unknown_pricing_count.to_i.positive? ? 'lct-stat-warn' : 'lct-stat-ok' %>">
       <div class="lct-stat-head"><p class="lct-stat-label">Incomplete pricing</p></div>
-      <p class="lct-stat-value"><%= number(@stats.unknown_pricing_count) %></p>
+      <p class="lct-stat-value"><%= number_with_delimiter(@stats.unknown_pricing_count) %></p>
       <p class="lct-stat-foot"><%= percent(coverage_percent(@stats.unknown_pricing_count, @stats.total_calls)) %> of slice</p>
     </div>
   </div>
@@ -115,7 +115,7 @@
       <h2 class="lct-panel-title">Daily spend</h2>
       <span class="lct-panel-meta">
         <span class="lct-legend"><span class="lct-legend-dot lct-legend-dot-current"></span>current</span>
-        <% if @comparison_series.any? %><span class="lct-legend"><span class="lct-legend-dot lct-legend-dot-prior"></span>prior <%= number(@comparison_series.size) %>d</span><% end %>
+        <% if @comparison_series.any? %><span class="lct-legend"><span class="lct-legend-dot lct-legend-dot-prior"></span>prior <%= number_with_delimiter(@comparison_series.size) %>d</span><% end %>
       </span>
     </div>
     <%= render "llm_cost_tracker/shared/spend_chart", series: @time_series, comparison_series: @comparison_series %>
@@ -133,7 +133,7 @@
           <% @top_models.first(5).each do |row| %>
             <tr>
               <td><span class="lct-model-cell"><span class="lct-provider-dot lct-provider-dot-<%= row.provider %>"></span><%= link_to calls_path(calls_query_for_model(provider: row.provider, model: row.model)), class: "lct-code-id" do %><%= row.model %><% end %></span></td>
-              <td class="lct-num"><%= number(row.calls) %></td>
+              <td class="lct-num"><%= number_with_delimiter(row.calls) %></td>
               <td class="lct-num"><%= money(row.total_cost) %></td>
             </tr>
           <% end %>
@@ -150,7 +150,7 @@
             <% @providers.each do |row| %>
               <tr>
                 <td><span class="lct-model-cell"><span class="lct-provider-dot lct-provider-dot-<%= row.provider %>"></span><%= row.provider %></span></td>
-                <td class="lct-num"><%= number(row.calls) %></td>
+                <td class="lct-num"><%= number_with_delimiter(row.calls) %></td>
                 <td class="lct-num"><%= money(row.total_cost) %></td>
                 <td class="lct-num"><%= percent(row.share_percent) %></td>
               </tr>

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

@@ -7,9 +7,20 @@
     <%= link_to "× Clear filters", data_quality_path(current_query(provider: nil, model: nil, page: nil)), class: "lct-filter-clear" %>
   <% end %>
 
-  <span class="lct-filter-row-meta"><%= number(@summary.total) %> call<%= "s" unless @summary.total == 1 %> inspected</span>
+  <span class="lct-filter-row-meta"><%= number_with_delimiter(@summary.total) %> call<%= "s" unless @summary.total == 1 %> inspected</span>
 </div>
 
+<% if @quarantined_inbox && @quarantined_inbox.count.positive? %>
+  <h3 class="lct-stat-section-label">Async inbox</h3>
+  <div class="lct-stat-grid">
+    <div class="lct-stat lct-stat-warn">
+      <div class="lct-stat-head"><p class="lct-stat-label">Quarantined inbox rows</p></div>
+      <p class="lct-stat-value"><%= number_with_delimiter(@quarantined_inbox.count) %></p>
+      <p class="lct-stat-foot"><%= money(@quarantined_inbox.total_cost) %> excluded from the ledger and totals</p>
+    </div>
+  </div>
+<% end %>
+
 <% if @summary.total.zero? %>
   <section class="lct-panel lct-empty">
     <h2 class="lct-state-title">No data yet</h2>
@@ -20,19 +31,19 @@
   <div class="lct-stat-grid">
     <div class="lct-stat">
       <div class="lct-stat-head"><p class="lct-stat-label">Calls inspected</p></div>
-      <p class="lct-stat-value"><%= number(@summary.total) %></p>
+      <p class="lct-stat-value"><%= number_with_delimiter(@summary.total) %></p>
       <p class="lct-stat-foot">in current slice</p>
     </div>
 
     <div class="lct-stat">
       <div class="lct-stat-head"><p class="lct-stat-label">Streaming calls</p></div>
-      <p class="lct-stat-value"><%= number(@summary.streaming_count) %></p>
+      <p class="lct-stat-value"><%= number_with_delimiter(@summary.streaming_count) %></p>
       <p class="lct-stat-foot"><%= percent(@summary.streaming_share) %> of calls</p>
     </div>
 
     <div class="lct-stat">
       <div class="lct-stat-head"><p class="lct-stat-label">Calls with provider response ID</p></div>
-      <p class="lct-stat-value"><%= number(@summary.calls_with_provider_response_id) %></p>
+      <p class="lct-stat-value"><%= number_with_delimiter(@summary.calls_with_provider_response_id) %></p>
       <p class="lct-stat-foot"><%= percent(@summary.provider_response_id_coverage) %> of calls</p>
     </div>
   </div>
@@ -50,7 +61,7 @@
       <% if incomplete_pricing.positive? %>
         <div class="lct-stat lct-stat-warn">
           <div class="lct-stat-head"><p class="lct-stat-label">Incomplete pricing</p></div>
-          <p class="lct-stat-value"><%= number(incomplete_pricing) %></p>
+          <p class="lct-stat-value"><%= number_with_delimiter(incomplete_pricing) %></p>
           <p class="lct-stat-foot"><%= percent(@summary.unknown_pricing_share) %> of calls</p>
         </div>
       <% end %>
@@ -58,7 +69,7 @@
       <% if untagged.positive? %>
         <div class="lct-stat lct-stat-warn">
           <div class="lct-stat-head"><p class="lct-stat-label">Calls without tags</p></div>
-          <p class="lct-stat-value"><%= number(untagged) %></p>
+          <p class="lct-stat-value"><%= number_with_delimiter(untagged) %></p>
           <p class="lct-stat-foot"><%= percent(@summary.untagged_share) %> of calls</p>
         </div>
       <% end %>
@@ -66,7 +77,7 @@
       <% if missing_latency.positive? %>
         <div class="lct-stat lct-stat-warn">
           <div class="lct-stat-head"><p class="lct-stat-label">Missing latency</p></div>
-          <p class="lct-stat-value"><%= number(missing_latency) %></p>
+          <p class="lct-stat-value"><%= number_with_delimiter(missing_latency) %></p>
           <p class="lct-stat-foot"><%= percent(@summary.missing_latency_share) %> of calls</p>
         </div>
       <% end %>
@@ -74,7 +85,7 @@
       <% if streams_missing_usage.positive? %>
         <div class="lct-stat lct-stat-warn">
           <div class="lct-stat-head"><p class="lct-stat-label">Streams without usage</p></div>
-          <p class="lct-stat-value"><%= number(streams_missing_usage) %></p>
+          <p class="lct-stat-value"><%= number_with_delimiter(streams_missing_usage) %></p>
           <p class="lct-stat-foot"><%= percent(@summary.streaming_missing_usage_share) %> of streams</p>
         </div>
       <% end %>
@@ -83,118 +94,64 @@
         <div class="lct-stat lct-stat-warn">
           <div class="lct-stat-head"><p class="lct-stat-label">Hidden output share</p></div>
           <p class="lct-stat-value"><%= percent(hidden_output_share) %></p>
-          <p class="lct-stat-foot"><%= number(@hidden_output_summary.fetch(:hidden_output_tokens)) %> of <%= number(@hidden_output_summary.fetch(:output_tokens)) %> output tokens</p>
+          <p class="lct-stat-foot"><%= number_with_delimiter(@hidden_output_summary.fetch(:hidden_output_tokens)) %> of <%= number_with_delimiter(@hidden_output_summary.fetch(:output_tokens)) %> output tokens</p>
         </div>
       <% end %>
     </div>
-  <% else %>
-    <div class="lct-alert lct-alert-info">
-      <svg width="14" height="14" viewBox="0 0 24 24" fill="none" stroke="currentColor" stroke-width="2" stroke-linecap="round" stroke-linejoin="round"><polyline points="20 6 9 17 4 12"/></svg>
-      <span>No data-quality issues in this slice.</span>
-    </div>
   <% end %>
 
-  <div class="lct-grid-2">
-    <section class="lct-panel">
-      <div class="lct-panel-head"><h2 class="lct-panel-title">Next actions</h2></div>
-      <p class="lct-panel-intro">Use these fixes to improve the trustworthiness of downstream cost reports.</p>
-
-      <table class="lct-tbl">
-        <thead>
-          <tr>
-            <th>Issue</th>
-            <th>Why it matters</th>
-            <th>Suggested action</th>
-          </tr>
-        </thead>
-        <tbody>
-          <tr>
-            <td>Incomplete pricing</td>
-            <td>Some or all line items have no configured rate, so totals undercount.</td>
-            <td>Update <code class="lct-code-id">pricing_overrides</code> or <code class="lct-code-id">prices_file</code>.</td>
-          </tr>
-          <tr>
-            <td>Missing tags</td>
-            <td>Attribution by tenant, user, or feature becomes less useful.</td>
-            <td>Pass <code class="lct-code-id">tags:</code> from middleware using request context.</td>
-          </tr>
-          <tr>
-            <td>Missing latency</td>
-            <td>Slow requests become harder to isolate on the calls page.</td>
-            <td>Make sure latency capture is enabled on every tracked request.</td>
-          </tr>
-          <% if @summary.streaming_missing_usage.positive? %>
-            <tr>
-              <td>Streams without usage</td>
-              <td>Token totals undercount when streaming responses drop the final usage event.</td>
-              <td>Send OpenAI requests with <code class="lct-code-id">stream_options: { include_usage: true }</code>, or wrap custom clients with <code class="lct-code-id">LlmCostTracker.track_stream</code>.</td>
-            </tr>
-          <% end %>
-          <% if @summary.missing_provider_response_id_count.positive? %>
-            <tr>
-              <td>Missing provider response IDs</td>
-              <td>Proof of provider-issued responses is weaker when calls cannot be tied back to provider objects.</td>
-              <td>Upgrade to the latest parser coverage and pass <code class="lct-code-id">provider_response_id:</code> for custom clients when the provider exposes one.</td>
-            </tr>
-          <% end %>
-        </tbody>
-      </table>
-    </section>
+  <section class="lct-panel">
+    <div class="lct-panel-head"><h2 class="lct-panel-title">Coverage summary</h2></div>
 
-    <section class="lct-panel">
-      <div class="lct-panel-head"><h2 class="lct-panel-title">Coverage summary</h2></div>
-      <p class="lct-panel-intro">Good dashboards start with clean pricing, tags, and latency coverage.</p>
+    <table class="lct-tbl">
+      <thead>
+        <tr>
+          <th>Dimension</th>
+          <th class="lct-num">Coverage</th>
+          <th class="lct-num">Calls with data</th>
+          <th>Visual</th>
+        </tr>
+      </thead>
+      <tbody>
+        <tr>
+          <td>Cost (pricing known)</td>
+          <td class="lct-num"><%= percent(@summary.cost_coverage) %></td>
+          <td class="lct-num"><%= number_with_delimiter(@summary.calls_with_pricing) %></td>
+          <td><%= render "llm_cost_tracker/shared/bar", value: @summary.cost_coverage, max: 100.0 %></td>
+        </tr>
 
-      <table class="lct-tbl">
-        <thead>
-          <tr>
-            <th>Dimension</th>
-            <th class="lct-num">Coverage</th>
-            <th class="lct-num">Calls with data</th>
-            <th>Visual</th>
-          </tr>
-        </thead>
-        <tbody>
-          <tr>
-            <td>Cost (pricing known)</td>
-            <td class="lct-num"><%= percent(@summary.cost_coverage) %></td>
-            <td class="lct-num"><%= number(@summary.calls_with_pricing) %></td>
-            <td><%= render "llm_cost_tracker/shared/bar", value: @summary.cost_coverage, max: 100.0 %></td>
-          </tr>
+        <tr>
+          <td>Tags (at least one tag)</td>
+          <td class="lct-num"><%= percent(@summary.tag_coverage) %></td>
+          <td class="lct-num"><%= number_with_delimiter(@summary.tagged_calls) %></td>
+          <td><%= render "llm_cost_tracker/shared/bar", value: @summary.tag_coverage, max: 100.0 %></td>
+        </tr>
 
-          <tr>
-            <td>Tags (at least one tag)</td>
-            <td class="lct-num"><%= percent(@summary.tag_coverage) %></td>
-            <td class="lct-num"><%= number(@summary.tagged_calls) %></td>
-            <td><%= render "llm_cost_tracker/shared/bar", value: @summary.tag_coverage, max: 100.0 %></td>
-          </tr>
+        <tr>
+          <td>Latency</td>
+          <td class="lct-num"><%= percent(@summary.latency_coverage) %></td>
+          <td class="lct-num"><%= number_with_delimiter(@summary.calls_with_latency) %></td>
+          <td><%= render "llm_cost_tracker/shared/bar", value: @summary.latency_coverage, max: 100.0 %></td>
+        </tr>
 
+        <% if @summary.streaming_count.positive? %>
           <tr>
-            <td>Latency</td>
-            <td class="lct-num"><%= percent(@summary.latency_coverage) %></td>
-            <td class="lct-num"><%= number(@summary.calls_with_latency) %></td>
-            <td><%= render "llm_cost_tracker/shared/bar", value: @summary.latency_coverage, max: 100.0 %></td>
+            <td>Streaming usage captured</td>
+            <td class="lct-num"><%= percent(@summary.stream_coverage) %></td>
+            <td class="lct-num"><%= number_with_delimiter(@summary.streams_with_usage) %> / <%= number_with_delimiter(@summary.streaming_count) %></td>
+            <td><%= render "llm_cost_tracker/shared/bar", value: @summary.stream_coverage, max: 100.0 %></td>
           </tr>
+        <% end %>
 
-          <% if @summary.streaming_count.positive? %>
-            <tr>
-              <td>Streaming usage captured</td>
-              <td class="lct-num"><%= percent(@summary.stream_coverage) %></td>
-              <td class="lct-num"><%= number(@summary.streams_with_usage) %> / <%= number(@summary.streaming_count) %></td>
-              <td><%= render "llm_cost_tracker/shared/bar", value: @summary.stream_coverage, max: 100.0 %></td>
-            </tr>
-          <% end %>
-
-          <tr>
-            <td>Provider response ID</td>
-            <td class="lct-num"><%= percent(@summary.provider_response_id_coverage) %></td>
-            <td class="lct-num"><%= number(@summary.calls_with_provider_response_id) %></td>
-            <td><%= render "llm_cost_tracker/shared/bar", value: @summary.provider_response_id_coverage, max: 100.0 %></td>
-          </tr>
-        </tbody>
-      </table>
-    </section>
-  </div>
+        <tr>
+          <td>Provider response ID</td>
+          <td class="lct-num"><%= percent(@summary.provider_response_id_coverage) %></td>
+          <td class="lct-num"><%= number_with_delimiter(@summary.calls_with_provider_response_id) %></td>
+          <td><%= render "llm_cost_tracker/shared/bar", value: @summary.provider_response_id_coverage, max: 100.0 %></td>
+        </tr>
+      </tbody>
+    </table>
+  </section>
 
   <section class="lct-panel">
     <div class="lct-panel-head"><h2 class="lct-panel-title">Token usage</h2></div>
@@ -216,7 +173,7 @@
             <% cost_value = row.fetch(:cost_value) %>
             <tr>
               <td><%= LlmCostTracker::TokenUsageHelper::QUALITY_LABELS.fetch(token_key) %></td>
-              <td class="lct-num"><%= number(row.fetch(:token_value)) %></td>
+              <td class="lct-num"><%= number_with_delimiter(row.fetch(:token_value)) %></td>
               <% if row.fetch(:share_basis) == :output %>
                 <td class="lct-num"><%= percent(row.fetch(:share_percent)) %> of output</td>
               <% else %>
@@ -250,13 +207,13 @@
           </thead>
           <tbody>
             <% @service_charge_rows.each do |row| %>
-              <% unknown_cost = row.cost_status.to_s == LlmCostTracker::Billing::CostStatus::UNKNOWN %>
+              <% unknown_cost = row.cost_status.to_s == LlmCostTracker::Charges::CostStatus::UNKNOWN %>
               <tr>
                 <td><code class="lct-code-id"><%= row.provider %></code></td>
                 <td><code class="lct-code-id"><%= row.component %></code></td>
                 <td><%= row.cost_status %></td>
-                <td class="lct-num"><%= number(row.charges_count) %></td>
-                <td class="lct-num"><%= number(row.quantity) %></td>
+                <td class="lct-num"><%= number_with_delimiter(row.charges_count) %></td>
+                <td class="lct-num"><%= number_with_delimiter(row.quantity) %></td>
                 <td class="lct-num<%= ' lct-num-muted' if unknown_cost %>"><%= unknown_cost ? "n/a" : money(row.total_cost) %></td>
               </tr>
             <% end %>
@@ -269,7 +226,6 @@
   <% if @streaming_health_rows.any? %>
     <section class="lct-panel">
       <div class="lct-panel-head"><h2 class="lct-panel-title">Streaming health by provider</h2></div>
-      <p class="lct-panel-intro">Streams without a final usage chunk land as <code>usage_source: unknown</code> and undercount tokens. A high unknown share for an OpenAI-compatible provider usually means <code>stream_options: { include_usage: true }</code> is not being injected for that host.</p>
 
       <div class="lct-table-wrap">
         <table class="lct-tbl">
@@ -286,9 +242,9 @@
             <% @streaming_health_rows.each do |row| %>
               <tr>
                 <td><code class="lct-code-id"><%= row.provider %></code></td>
-                <td class="lct-num"><%= number(row.streams) %></td>
-                <td class="lct-num"><%= number(row.with_usage) %></td>
-                <td class="lct-num"><%= number(row.unknown) %></td>
+                <td class="lct-num"><%= number_with_delimiter(row.streams) %></td>
+                <td class="lct-num"><%= number_with_delimiter(row.with_usage) %></td>
+                <td class="lct-num"><%= number_with_delimiter(row.unknown) %></td>
                 <td class="lct-num"><%= percent(row.unknown_share) %></td>
               </tr>
             <% end %>
@@ -321,7 +277,7 @@
               <tr>
                 <td><code class="lct-code-id"><%= row.provider %></code></td>
                 <td><code class="lct-code-id"><%= row.model %></code></td>
-                <td class="lct-num"><%= number(row.calls) %></td>
+                <td class="lct-num"><%= number_with_delimiter(row.calls) %></td>
                 <td class="lct-num"><%= percent(row.share_percent) %></td>
               </tr>
             <% end %>

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

@@ -8,7 +8,7 @@
     <%= link_to "× Clear filters", models_path(current_query(provider: nil, model: nil, page: nil)), class: "lct-filter-clear" %>
   <% end %>
 
-  <span class="lct-filter-row-meta"><%= number(rows.size) %> model<%= "s" unless rows.size == 1 %></span>
+  <span class="lct-filter-row-meta"><%= number_with_delimiter(rows.size) %> model<%= "s" unless rows.size == 1 %></span>
 </div>
 
 <% if rows.empty? %>
@@ -27,7 +27,7 @@
           <%= sortable_header("Tokens", "tokens", num: true) %>
           <%= sortable_header("Avg latency", "latency", num: true) %>
           <%= sortable_header("Avg cost / call", "avg_cost", num: true) %>
-          <%= sortable_header("Total cost", "cost", num: true) %>
+          <%= sortable_header("Total cost", "cost", num: true, default: true) %>
           <th></th>
         </tr>
       </thead>
@@ -36,10 +36,10 @@
           <tr>
             <td><span class="lct-model-cell"><span class="lct-provider-dot lct-provider-dot-<%= row.provider %>"></span><%= row.provider %></span></td>
             <td><code class="lct-code-id"><%= row.model %></code></td>
-            <td class="lct-num"><%= number(row.calls) %></td>
-            <td class="lct-num"><%= number(row.total_tokens) %></td>
+            <td class="lct-num"><%= number_with_delimiter(row.calls) %></td>
+            <td class="lct-num"><%= number_with_delimiter(row.total_tokens) %></td>
             <% avg_latency = row.average_latency_ms %>
-            <td class="lct-num<%= ' lct-num-muted' if avg_latency.nil? %>"><%= avg_latency ? "#{number(avg_latency.round)}ms" : "n/a" %></td>
+            <td class="lct-num<%= ' lct-num-muted' if avg_latency.nil? %>"><%= avg_latency ? "#{number_with_delimiter(avg_latency.round)}ms" : "n/a" %></td>
             <td class="lct-num"><%= money(row.average_cost_per_call) %></td>
             <td class="lct-num"><%= money(row.total_cost) %></td>
             <td class="lct-num"><%= link_to "Calls", calls_path(calls_query_for_model(provider: row.provider, model: row.model)), class: "lct-page-link" %></td>