llm_cost_tracker 0.7.3 → 0.8.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,63 +17,58 @@
     </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">
@@ -145,46 +90,41 @@
           </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 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>
 
-          <% 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 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>
 
-          <% 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 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_count.to_i.positive? %>
-            <% stream_coverage = coverage_percent(streaming_count - streaming_missing_usage, streaming_count) %>
+          <% if @summary.streaming_count.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 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 %>
 
-          <% 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>
+            <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>
@@ -222,14 +162,14 @@
             <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_missing_usage.to_i.positive? %>
+          <% 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">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.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>
@@ -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,43 @@
     </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| %>
+              <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"><%= optional_money(row.total_cost) %></td>
+              </tr>
+            <% end %>
+          </tbody>
+        </table>
+      </div>
+    </section>
+  <% end %>
+
   <% unless @unknown_pricing_by_model.empty? %>
     <section class="lct-panel">
       <div class="lct-section-head">
@@ -305,7 +282,7 @@
               <tr>
                 <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 %>

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

@@ -0,0 +1,63 @@
+<%
+  fields = local_assigns.fetch(:fields, %i[from to provider model stream])
+  default_range = LlmCostTracker::Dashboard::DateRange.call(params: params)
+  defaults = {
+    from: default_range.from.iso8601,
+    to: default_range.to.iso8601
+  }.merge(local_assigns.fetch(:defaults, {}))
+  reset_path = local_assigns.fetch(:reset_path, path)
+  filter_scope = local_assigns.fetch(:filter_scope, params)
+%>
+
+<form class="lct-filters" action="<%= path %>" method="get">
+  <div class="lct-filter-row">
+    <% if fields.include?(:from) %>
+      <div class="lct-field">
+        <label for="lct-filter-from">From</label>
+        <input id="lct-filter-from" type="date" name="from" value="<%= params[:from] || defaults[:from] %>">
+      </div>
+    <% end %>
+
+    <% if fields.include?(:to) %>
+      <div class="lct-field">
+        <label for="lct-filter-to">To</label>
+        <input id="lct-filter-to" type="date" name="to" value="<%= params[:to] || defaults[:to] %>">
+      </div>
+    <% end %>
+
+    <% if fields.include?(:provider) %>
+      <div class="lct-field">
+        <label for="lct-filter-provider">Provider</label>
+        <%= select_tag :provider,
+                       options_for_select(provider_filter_options(filter_params: filter_scope), params[:provider]),
+                       include_blank: "All providers",
+                       id: "lct-filter-provider" %>
+      </div>
+    <% end %>
+
+    <% if fields.include?(:model) %>
+      <div class="lct-field">
+        <label for="lct-filter-model">Model</label>
+        <%= select_tag :model,
+                       options_for_select(model_filter_options(filter_params: filter_scope), params[:model]),
+                       include_blank: "All models",
+                       id: "lct-filter-model" %>
+      </div>
+    <% end %>
+
+    <% if fields.include?(:stream) %>
+      <div class="lct-field">
+        <label for="lct-filter-stream">Stream</label>
+        <%= select_tag :stream,
+                       options_for_select(LlmCostTracker::DashboardFilterHelper::STREAM_FILTER_OPTIONS, params[:stream]),
+                       include_blank: "All calls",
+                       id: "lct-filter-stream" %>
+      </div>
+    <% end %>
+
+    <div class="lct-filter-actions">
+      <button class="lct-button" type="submit">Apply</button>
+      <%= link_to("Reset", reset_path, class: "lct-button lct-button-secondary") if any_filter_applied? %>
+    </div>
+  </div>
+</form>

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

@@ -0,0 +1,13 @@
+<%
+  current = local_assigns.fetch(:current).to_s
+  options = local_assigns.fetch(:options)
+%>
+
+<nav class="lct-sort" aria-label="Sort by">
+  <% options.each do |label, value| %>
+    <%= link_to label,
+                path_for_sort.call(value),
+                class: ["lct-sort-option", ("is-active" if current == value.to_s)].compact.join(" "),
+                aria: ({ current: "true" } if current == value.to_s) %>
+  <% end %>
+</nav>

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

@@ -1,7 +1,7 @@
 <section class="lct-panel lct-empty">
   <h2 class="lct-state-title">Setup required</h2>
   <p class="lct-state-copy">
-    <%= @setup_message || "The llm_api_calls table is not available yet." %>
+    <%= @setup_message || "The llm_cost_tracker_calls table is not available yet." %>
     <% if @setup_details.present? %>
       Run <span class="lct-code">bin/rails llm_cost_tracker:doctor</span>, apply the listed migrations, and migrate your database.
     <% else %>

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

@@ -3,40 +3,9 @@
     <h2 class="lct-section-title">Tag keys</h2>
   </div>
 
-  <form class="lct-filters" action="<%= tags_path %>" method="get">
-    <div class="lct-filter-row lct-filter-row-basic">
-      <div class="lct-field">
-        <label for="lct-tags-from">From</label>
-        <input id="lct-tags-from" type="date" name="from" value="<%= params[:from] %>">
-      </div>
-
-      <div class="lct-field">
-        <label for="lct-tags-to">To</label>
-        <input id="lct-tags-to" type="date" name="to" value="<%= params[:to] %>">
-      </div>
-
-      <div class="lct-field">
-        <label for="lct-tags-provider">Provider</label>
-        <%= select_tag :provider,
-                       options_for_select(provider_filter_options, params[:provider]),
-                       include_blank: "All providers",
-                       id: "lct-tags-provider" %>
-      </div>
-
-      <div class="lct-field">
-        <label for="lct-tags-model">Model</label>
-        <%= select_tag :model,
-                       options_for_select(model_filter_options, params[:model]),
-                       include_blank: "All models",
-                       id: "lct-tags-model" %>
-      </div>
-
-      <div class="lct-filter-actions">
-        <button class="lct-button" type="submit">Apply</button>
-        <%= link_to("Reset", tags_path, class: "lct-button lct-button-secondary") if any_filter_applied? %>
-      </div>
-    </div>
-  </form>
+  <%= render "llm_cost_tracker/shared/filters",
+             path: tags_path,
+             fields: %i[from to provider model] %>
 
   <%= render "llm_cost_tracker/shared/active_filters", chips: active_tag_filters, clear_path: tags_path %>
 </section>

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

@@ -1,5 +1,3 @@
-<% share_base = @breakdown.tagged_calls.positive? ? @breakdown.tagged_calls.to_f : 1.0 %>
-
 <section class="lct-panel lct-toolbar">
   <div class="lct-toolbar-head">
     <div>
@@ -8,40 +6,10 @@
     </div>
   </div>
 
-  <form class="lct-filters" action="<%= tag_path(params[:key]) %>" method="get">
-    <div class="lct-filter-row lct-filter-row-basic">
-      <div class="lct-field">
-        <label for="lct-tag-show-from">From</label>
-        <input id="lct-tag-show-from" type="date" name="from" value="<%= params[:from] %>">
-      </div>
-
-      <div class="lct-field">
-        <label for="lct-tag-show-to">To</label>
-        <input id="lct-tag-show-to" type="date" name="to" value="<%= params[:to] %>">
-      </div>
-
-      <div class="lct-field">
-        <label for="lct-tag-show-provider">Provider</label>
-        <%= select_tag :provider,
-                       options_for_select(provider_filter_options, params[:provider]),
-                       include_blank: "All providers",
-                       id: "lct-tag-show-provider" %>
-      </div>
-
-      <div class="lct-field">
-        <label for="lct-tag-show-model">Model</label>
-        <%= select_tag :model,
-                       options_for_select(model_filter_options, params[:model]),
-                       include_blank: "All models",
-                       id: "lct-tag-show-model" %>
-      </div>
-
-      <div class="lct-filter-actions">
-        <button class="lct-button" type="submit">Apply</button>
-        <%= link_to("Reset", tag_path(params[:key]), class: "lct-button lct-button-secondary") if any_filter_applied? %>
-      </div>
-    </div>
-  </form>
+  <%= render "llm_cost_tracker/shared/filters",
+             path: tag_path(params[:key]),
+             fields: %i[from to provider model],
+             reset_path: tag_path(params[:key]) %>
 
   <%= render "llm_cost_tracker/shared/active_filters", chips: active_tag_filters, clear_path: tag_path(params[:key]) %>
 
@@ -103,7 +71,7 @@
             <tr>
               <td><code class="lct-code"><%= row.value %></code></td>
               <td class="lct-num"><%= number(row.calls) %></td>
-              <td class="lct-num"><%= percent((row.calls / share_base) * 100.0) %></td>
+              <td class="lct-num"><%= percent(row.share_percent) %></td>
               <td class="lct-num"><%= money(row.total_cost) %></td>
               <td class="lct-num"><%= money(row.average_cost_per_call) %></td>
               <td>