llm_cost_tracker 0.7.3 → 0.9.0

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

@@ -1,64 +1,14 @@
-<% total = @stats.total_calls.to_i %>
-<% unknown_pricing_count = @stats.unknown_pricing_count.to_i %>
-<% untagged_calls_count = @stats.untagged_calls_count.to_i %>
-<% missing_latency_count = @stats.missing_latency_count&.to_i %>
-<% streaming_count = @stats.streaming_count&.to_i %>
-<% streaming_missing_usage = @stats.streaming_missing_usage_count&.to_i %>
-<% missing_provider_response_id_count = @stats.missing_provider_response_id_count&.to_i %>
-<% calls_with_provider_response_id = total - missing_provider_response_id_count %>
-
 <section class="lct-panel lct-toolbar">
   <div class="lct-toolbar-head">
     <h2 class="lct-section-title">Data Quality</h2>
   </div>
 
-  <form class="lct-filters" action="<%= data_quality_path %>" method="get">
-    <div class="lct-filter-row lct-filter-row-basic">
-      <div class="lct-field">
-        <label for="lct-quality-from">From</label>
-        <input id="lct-quality-from" type="date" name="from" value="<%= params[:from] %>">
-      </div>
-
-      <div class="lct-field">
-        <label for="lct-quality-to">To</label>
-        <input id="lct-quality-to" type="date" name="to" value="<%= params[:to] %>">
-      </div>
-
-      <div class="lct-field">
-        <label for="lct-quality-provider">Provider</label>
-        <%= select_tag :provider,
-                       options_for_select(provider_filter_options, params[:provider]),
-                       include_blank: "All providers",
-                       id: "lct-quality-provider" %>
-      </div>
-
-      <div class="lct-field">
-        <label for="lct-quality-model">Model</label>
-        <%= select_tag :model,
-                       options_for_select(model_filter_options, params[:model]),
-                       include_blank: "All models",
-                       id: "lct-quality-model" %>
-      </div>
-
-      <div class="lct-field">
-        <label for="lct-quality-stream">Stream</label>
-        <%= select_tag :stream,
-                       options_for_select(LlmCostTracker::DashboardFilterHelper::STREAM_FILTER_OPTIONS, params[:stream]),
-                       include_blank: "All calls",
-                       id: "lct-quality-stream" %>
-      </div>
-
-      <div class="lct-filter-actions">
-        <button class="lct-button" type="submit">Apply</button>
-        <%= link_to("Reset", data_quality_path, class: "lct-button lct-button-secondary") if any_filter_applied? %>
-      </div>
-    </div>
-  </form>
+  <%= render "llm_cost_tracker/shared/filters", path: data_quality_path %>
 
   <%= render "llm_cost_tracker/shared/active_filters", chips: active_tag_filters, clear_path: data_quality_path %>
 </section>
 
-<% if total.zero? %>
+<% if @summary.total.zero? %>
   <section class="lct-panel lct-empty">
     <h2 class="lct-state-title">No data yet</h2>
     <p class="lct-state-copy">Quality metrics will appear here once calls are recorded in the current slice.</p>
@@ -67,125 +17,107 @@
     </div>
   </section>
 <% else %>
-  <section class="lct-hero">
-    <article class="lct-panel lct-hero-primary">
-      <div>
-        <p class="lct-stat-label">Calls inspected</p>
-        <p class="lct-hero-value"><%= number(total) %></p>
-      </div>
+  <section class="lct-stat-grid lct-stat-grid-spaced">
+    <article class="lct-stat">
+      <p class="lct-stat-label">Calls inspected</p>
+      <p class="lct-stat-value"><%= number(@summary.total) %></p>
+      <p class="lct-stat-sub">in current slice</p>
     </article>
 
-    <div class="lct-hero-side">
-      <div class="lct-stat-grid">
-        <article class="lct-stat">
-          <p class="lct-stat-label">Unknown pricing</p>
-          <p class="lct-stat-value"><%= number(unknown_pricing_count) %></p>
-          <p class="lct-stat-sub"><%= percent(coverage_percent(unknown_pricing_count, total)) %> of calls</p>
-        </article>
+    <article class="lct-stat">
+      <p class="lct-stat-label">Unknown pricing</p>
+      <p class="lct-stat-value"><%= number(@summary.unknown_pricing_count) %></p>
+      <p class="lct-stat-sub"><%= percent(@summary.unknown_pricing_share) %> of calls</p>
+    </article>
 
-        <article class="lct-stat">
-          <p class="lct-stat-label">Calls without tags</p>
-          <p class="lct-stat-value"><%= number(untagged_calls_count) %></p>
-          <p class="lct-stat-sub"><%= percent(coverage_percent(untagged_calls_count, total)) %> of calls</p>
-        </article>
+    <article class="lct-stat">
+      <p class="lct-stat-label">Calls without tags</p>
+      <p class="lct-stat-value"><%= number(@summary.untagged_calls_count) %></p>
+      <p class="lct-stat-sub"><%= percent(@summary.untagged_share) %> of calls</p>
+    </article>
 
-        <article class="lct-stat">
-          <p class="lct-stat-label">Missing latency</p>
-          <p class="lct-stat-value"><%= number(missing_latency_count) %></p>
-          <p class="lct-stat-sub"><%= percent(coverage_percent(missing_latency_count, total)) %> of calls</p>
-        </article>
+    <article class="lct-stat">
+      <p class="lct-stat-label">Missing latency</p>
+      <p class="lct-stat-value"><%= number(@summary.missing_latency_count) %></p>
+      <p class="lct-stat-sub"><%= percent(@summary.missing_latency_share) %> of calls</p>
+    </article>
 
-        <article class="lct-stat">
-          <p class="lct-stat-label">Streaming calls</p>
-          <p class="lct-stat-value"><%= number(streaming_count) %></p>
-          <p class="lct-stat-sub"><%= percent(coverage_percent(streaming_count, total)) %> of calls</p>
-        </article>
+    <article class="lct-stat">
+      <p class="lct-stat-label">Streaming calls</p>
+      <p class="lct-stat-value"><%= number(@summary.streaming_count) %></p>
+      <p class="lct-stat-sub"><%= percent(@summary.streaming_share) %> of calls</p>
+    </article>
 
-        <% if streaming_count.positive? %>
-          <article class="lct-stat">
-            <p class="lct-stat-label">Streams without usage</p>
-            <p class="lct-stat-value"><%= number(streaming_missing_usage) %></p>
-            <p class="lct-stat-sub"><%= percent(coverage_percent(streaming_missing_usage, streaming_count)) %> of streams</p>
-          </article>
-        <% end %>
+    <% if @summary.streaming_count.positive? %>
+      <article class="lct-stat">
+        <p class="lct-stat-label">Streams without usage</p>
+        <p class="lct-stat-value"><%= number(@summary.streaming_missing_usage) %></p>
+        <p class="lct-stat-sub"><%= percent(@summary.streaming_missing_usage_share) %> of streams</p>
+      </article>
+    <% end %>
 
-        <article class="lct-stat">
-          <p class="lct-stat-label">Calls with provider response ID</p>
-          <p class="lct-stat-value"><%= number(calls_with_provider_response_id) %></p>
-          <p class="lct-stat-sub"><%= percent(coverage_percent(calls_with_provider_response_id, total)) %> of calls</p>
-        </article>
+    <article class="lct-stat">
+      <p class="lct-stat-label">Calls with provider response ID</p>
+      <p class="lct-stat-value"><%= number(@summary.calls_with_provider_response_id) %></p>
+      <p class="lct-stat-sub"><%= percent(@summary.provider_response_id_coverage) %> of calls</p>
+    </article>
 
-        <% if @hidden_output_summary %>
-          <article class="lct-stat">
-            <p class="lct-stat-label">Hidden output share</p>
-            <p class="lct-stat-value"><%= percent(@hidden_output_summary.fetch(:share_percent)) %></p>
-            <p class="lct-stat-sub"><%= number(@hidden_output_summary.fetch(:hidden_output_tokens)) %> of <%= number(@hidden_output_summary.fetch(:output_tokens)) %> output tokens</p>
-          </article>
-        <% end %>
-      </div>
-    </div>
+    <% if @hidden_output_summary %>
+      <article class="lct-stat">
+        <p class="lct-stat-label">Hidden output share</p>
+        <p class="lct-stat-value"><%= percent(@hidden_output_summary.fetch(:share_percent)) %></p>
+        <p class="lct-stat-sub"><%= number(@hidden_output_summary.fetch(:hidden_output_tokens)) %> of <%= number(@hidden_output_summary.fetch(:output_tokens)) %> output tokens</p>
+      </article>
+    <% end %>
   </section>
 
   <section class="lct-grid lct-two-col">
     <section class="lct-panel">
       <div class="lct-section-head">
         <div>
-          <h2 class="lct-section-title">Coverage summary</h2>
-          <p class="lct-section-copy">Good dashboards start with clean pricing, tags, and latency coverage.</p>
+          <h2 class="lct-section-title">Next actions</h2>
+          <p class="lct-section-copy">Use these fixes to improve the trustworthiness of downstream cost reports.</p>
         </div>
       </div>
 
       <table class="lct-table lct-table-compact">
         <thead>
           <tr>
-            <th>Dimension</th>
-            <th class="lct-num">Coverage</th>
-            <th class="lct-num">Calls with data</th>
-            <th>Visual</th>
+            <th>Issue</th>
+            <th>Why it matters</th>
+            <th>Suggested action</th>
           </tr>
         </thead>
         <tbody>
-          <% cost_coverage = coverage_percent(total - unknown_pricing_count, total) %>
           <tr>
-            <td>Cost (pricing known)</td>
-            <td class="lct-num"><%= percent(cost_coverage) %></td>
-            <td class="lct-num"><%= number(total - unknown_pricing_count) %></td>
-            <td><%= render "llm_cost_tracker/shared/bar", value: cost_coverage, max: 100.0 %></td>
+            <td>Unknown pricing</td>
+            <td>Cost stays <code class="lct-code">nil</code>, so totals undercount.</td>
+            <td>Update <code class="lct-code">pricing_overrides</code> or <code class="lct-code">prices_file</code>.</td>
           </tr>
-
-          <% tag_coverage = coverage_percent(total - untagged_calls_count, total) %>
           <tr>
-            <td>Tags (at least one tag)</td>
-            <td class="lct-num"><%= percent(tag_coverage) %></td>
-            <td class="lct-num"><%= number(total - untagged_calls_count) %></td>
-            <td><%= render "llm_cost_tracker/shared/bar", value: tag_coverage, max: 100.0 %></td>
+            <td>Missing tags</td>
+            <td>Attribution by tenant, user, or feature becomes less useful.</td>
+            <td>Pass <code class="lct-code">tags:</code> from middleware using request context.</td>
           </tr>
-
-          <% latency_coverage = coverage_percent(total - missing_latency_count, total) %>
           <tr>
-            <td>Latency</td>
-            <td class="lct-num"><%= percent(latency_coverage) %></td>
-            <td class="lct-num"><%= number(total - missing_latency_count) %></td>
-            <td><%= render "llm_cost_tracker/shared/bar", value: latency_coverage, max: 100.0 %></td>
+            <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 streaming_count.to_i.positive? %>
-            <% stream_coverage = coverage_percent(streaming_count - streaming_missing_usage, streaming_count) %>
+          <% if @summary.streaming_missing_usage.positive? %>
             <tr>
-              <td>Streaming usage captured</td>
-              <td class="lct-num"><%= percent(stream_coverage) %></td>
-              <td class="lct-num"><%= number(streaming_count - streaming_missing_usage) %> / <%= number(streaming_count) %></td>
-              <td><%= render "llm_cost_tracker/shared/bar", value: stream_coverage, max: 100.0 %></td>
+              <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">stream_options: { include_usage: true }</code>, or wrap custom clients with <code class="lct-code">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">provider_response_id:</code> for custom clients when the provider exposes one.</td>
             </tr>
           <% end %>
-
-          <% provider_response_id_coverage = coverage_percent(calls_with_provider_response_id, total) %>
-          <tr>
-            <td>Provider response ID</td>
-            <td class="lct-num"><%= percent(provider_response_id_coverage) %></td>
-            <td class="lct-num"><%= number(calls_with_provider_response_id) %></td>
-            <td><%= render "llm_cost_tracker/shared/bar", value: provider_response_id_coverage, max: 100.0 %></td>
-          </tr>
         </tbody>
       </table>
     </section>
@@ -193,49 +125,57 @@
     <section class="lct-panel">
       <div class="lct-section-head">
         <div>
-          <h2 class="lct-section-title">Next actions</h2>
-          <p class="lct-section-copy">Use these fixes to improve the trustworthiness of downstream cost reports.</p>
+          <h2 class="lct-section-title">Coverage summary</h2>
+          <p class="lct-section-copy">Good dashboards start with clean pricing, tags, and latency coverage.</p>
         </div>
       </div>
 
       <table class="lct-table lct-table-compact">
         <thead>
           <tr>
-            <th>Issue</th>
-            <th>Why it matters</th>
-            <th>Suggested action</th>
+            <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>Unknown pricing</td>
-            <td>Cost stays <code class="lct-code">nil</code>, so totals undercount.</td>
-            <td>Update <code class="lct-code">pricing_overrides</code> or <code class="lct-code">prices_file</code>.</td>
+            <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>Missing tags</td>
-            <td>Attribution by tenant, user, or feature becomes less useful.</td>
-            <td>Pass <code class="lct-code">tags:</code> from middleware using request context.</td>
+            <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>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>
+            <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>
           </tr>
-          <% if streaming_missing_usage.to_i.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">stream_options: { include_usage: true }</code>, or wrap custom clients with <code class="lct-code">LlmCostTracker.track_stream</code>.</td>
-            </tr>
-          <% end %>
-          <% if missing_provider_response_id_count.to_i.positive? %>
+
+          <% if @summary.streaming_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">provider_response_id:</code> for custom clients when the provider exposes one.</td>
+              <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>
@@ -264,7 +204,7 @@
             <% cost_key = row.fetch(:cost_key) %>
             <% cost_value = row.fetch(:cost_value) %>
             <tr>
-              <td><%= token_usage_quality_label(token_key) %></td>
+              <td><%= LlmCostTracker::TokenUsageHelper::QUALITY_LABELS.fetch(token_key) %></td>
               <td class="lct-num"><%= number(row.fetch(:token_value)) %></td>
               <% if row.fetch(:share_basis) == :output %>
                 <td class="lct-num"><%= percent(row.fetch(:share_percent)) %> of output</td>
@@ -281,6 +221,80 @@
     </div>
   </section>
 
+  <% if @service_charge_rows.any? %>
+    <section class="lct-panel">
+      <div class="lct-section-head">
+        <div>
+          <h2 class="lct-section-title">Service charges</h2>
+        </div>
+      </div>
+
+      <div class="lct-table-wrap">
+        <table class="lct-table lct-table-compact">
+          <thead>
+            <tr>
+              <th>Provider</th>
+              <th>Component</th>
+              <th>Status</th>
+              <th class="lct-num">Rows</th>
+              <th class="lct-num">Quantity</th>
+              <th class="lct-num">Cost</th>
+            </tr>
+          </thead>
+          <tbody>
+            <% @service_charge_rows.each do |row| %>
+              <% unknown_cost = row.cost_status.to_s == LlmCostTracker::Billing::CostStatus::UNKNOWN %>
+              <tr>
+                <td><code class="lct-code"><%= row.provider %></code></td>
+                <td><code class="lct-code"><%= 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<%= ' lct-num-muted' if unknown_cost %>"><%= unknown_cost ? "n/a" : money(row.total_cost) %></td>
+              </tr>
+            <% end %>
+          </tbody>
+        </table>
+      </div>
+    </section>
+  <% end %>
+
+  <% if @streaming_health_rows.any? %>
+    <section class="lct-panel">
+      <div class="lct-section-head">
+        <div>
+          <h2 class="lct-section-title">Streaming health by provider</h2>
+          <p class="lct-section-copy">Streams without a final usage chunk land as <code class="lct-code">usage_source: unknown</code> and undercount tokens. A high unknown share for an OpenAI-compatible provider usually means <code class="lct-code">stream_options: { include_usage: true }</code> is not being injected for that host.</p>
+        </div>
+      </div>
+
+      <div class="lct-table-wrap">
+        <table class="lct-table lct-table-compact">
+          <thead>
+            <tr>
+              <th>Provider</th>
+              <th class="lct-num">Streams</th>
+              <th class="lct-num">With usage</th>
+              <th class="lct-num">Unknown</th>
+              <th class="lct-num">Unknown share</th>
+            </tr>
+          </thead>
+          <tbody>
+            <% @streaming_health_rows.each do |row| %>
+              <tr>
+                <td><code class="lct-code"><%= 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"><%= percent(row.unknown_share) %></td>
+              </tr>
+            <% end %>
+          </tbody>
+        </table>
+      </div>
+    </section>
+  <% end %>
+
   <% unless @unknown_pricing_by_model.empty? %>
     <section class="lct-panel">
       <div class="lct-section-head">
@@ -288,13 +302,14 @@
           <h2 class="lct-section-title">Unknown pricing by model</h2>
           <p class="lct-section-copy">These models have token counts but no configured rates, so cost stays unknown.</p>
         </div>
-        <%= link_to "Review calls", calls_path(current_query(sort: "unknown_pricing")), class: "lct-button lct-button-secondary" %>
+        <%= link_to "Calls", calls_path(current_query(sort: "unknown_pricing")), class: "lct-button lct-button-secondary" %>
       </div>
 
       <div class="lct-table-wrap">
         <table class="lct-table lct-table-compact">
           <thead>
             <tr>
+              <th>Provider</th>
               <th>Model</th>
               <th class="lct-num">Calls without cost</th>
               <th class="lct-num">Share of total</th>
@@ -303,9 +318,10 @@
           <tbody>
             <% @unknown_pricing_by_model.each do |row| %>
               <tr>
+                <td><code class="lct-code"><%= row.provider %></code></td>
                 <td><code class="lct-code"><%= row.model %></code></td>
                 <td class="lct-num"><%= number(row.calls) %></td>
-                <td class="lct-num"><%= percent(total.positive? ? (row.calls.to_f / total) * 100.0 : 0.0) %></td>
+                <td class="lct-num"><%= percent(row.share_percent) %></td>
               </tr>
             <% end %>
           </tbody>

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

@@ -1,7 +1,7 @@
 <section class="lct-panel lct-empty">
   <h2 class="lct-state-title">Database unavailable</h2>
   <p class="lct-state-copy">
-    llm_cost_tracker could not read the <span class="lct-code">llm_api_calls</span> table.
+    llm_cost_tracker could not read the <span class="lct-code">llm_cost_tracker_calls</span> table.
     Check that ActiveRecord is connected, then run
     <span class="lct-code">rails generate llm_cost_tracker:install</span> and migrate your database.
   </p>

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

@@ -3,53 +3,9 @@
     <h2 class="lct-section-title">Models</h2>
   </div>
 
-  <form class="lct-filters" action="<%= models_path %>" method="get">
-    <div class="lct-filter-row lct-filter-row-with-sort">
-      <div class="lct-field">
-        <label for="lct-models-from">From</label>
-        <input id="lct-models-from" type="date" name="from" value="<%= params[:from] %>">
-      </div>
-
-      <div class="lct-field">
-        <label for="lct-models-to">To</label>
-        <input id="lct-models-to" type="date" name="to" value="<%= params[:to] %>">
-      </div>
-
-      <div class="lct-field">
-        <label for="lct-models-provider">Provider</label>
-        <%= select_tag :provider,
-                       options_for_select(provider_filter_options, params[:provider]),
-                       include_blank: "All providers",
-                       id: "lct-models-provider" %>
-      </div>
-
-      <div class="lct-field">
-        <label for="lct-models-model">Model</label>
-        <%= select_tag :model,
-                       options_for_select(model_filter_options, params[:model]),
-                       include_blank: "All models",
-                       id: "lct-models-model" %>
-      </div>
-
-      <div class="lct-field">
-        <label for="lct-models-sort">Sort</label>
-        <%= select_tag :sort,
-                       options_for_select(
-                         [["Total spend", "cost"],
-                          ["Call volume", "calls"],
-                          ["Avg cost / call", "avg_cost"],
-                          ["Avg latency", "latency"]],
-                         @sort.presence || "cost"
-                       ),
-                       id: "lct-models-sort" %>
-      </div>
-
-      <div class="lct-filter-actions">
-        <button class="lct-button" type="submit">Apply</button>
-        <%= link_to("Reset", models_path, class: "lct-button lct-button-secondary") if any_filter_applied? %>
-      </div>
-    </div>
-  </form>
+  <%= render "llm_cost_tracker/shared/filters",
+             path: models_path,
+             fields: %i[from to provider model] %>
 
   <%= render "llm_cost_tracker/shared/active_filters", chips: active_tag_filters, clear_path: models_path %>
 </section>
@@ -64,6 +20,18 @@
   </section>
 <% else %>
   <section class="lct-panel">
+    <div class="lct-results-toolbar">
+      <%= render "llm_cost_tracker/shared/sort",
+                 current: (@sort.presence || "cost"),
+                 options: [
+                   ["Total spend", "cost"],
+                   ["Call volume", "calls"],
+                   ["Avg cost / call", "avg_cost"],
+                   ["Avg latency", "latency"]
+                 ],
+                 path_for_sort: ->(value) { models_path(current_query(sort: value)) } %>
+    </div>
+
     <div class="lct-table-wrap">
       <table class="lct-table lct-table-compact">
         <thead>
@@ -86,9 +54,9 @@
               <td><%= row.provider %></td>
               <td><code class="lct-code"><%= row.model %></code></td>
               <td class="lct-num"><%= number(row.calls) %></td>
-              <td class="lct-num"><%= format_tokens(row.total_tokens) %></td>
-              <td class="lct-num"><%= format_tokens(row.input_tokens) %></td>
-              <td class="lct-num"><%= format_tokens(row.output_tokens) %></td>
+              <td class="lct-num"><%= number(row.total_tokens) %></td>
+              <td class="lct-num"><%= number(row.input_tokens) %></td>
+              <td class="lct-num"><%= number(row.output_tokens) %></td>
               <td class="lct-num"><%= money(row.total_cost) %></td>
               <td class="lct-num"><%= money(row.average_cost_per_call) %></td>
               <% average_latency_ms = row.average_latency_ms %>