hanikamu-rate-limit 0.3.1 → 0.3.2

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: ad248e2bab333b2d4ddd9fdf854f0856c68862ef9fc57dbdd80410fa821f82c0
-  data.tar.gz: 447847241345d838d88be7f5b908501fe4d72e720a6595f8c5dc1c8404a44fa3
+  metadata.gz: 9febd701c4dea4feea383f710cda6de80d26385f5e85b20971a2c5ed6825f5b9
+  data.tar.gz: 0d29be734df15391fd08a02eca7deddd46c754af3c4129683013ca9eed28c8dc
 SHA512:
-  metadata.gz: 8b4d7091043ef333f33ba67a2375cc36e615c830cd5e90ca6a1862d988394bcc6bb30fd64781390b1d0adf7799acfd95638431fac8822eb056382c79a68b8956
-  data.tar.gz: f056f4d0d29ab755e1262a3559f9e2514b0cfc5dad8546c27454330b003abca255863a04dfecde700ca85fffd06e2ec7da9f9dc1f6e3cbf3960b2e291d47e4b9
+  metadata.gz: 249ac5c8f437ea501a643e5a87c8d3263d80e3917ebada5140893b12ffb71804d78f44d8fcd4ee733fe04080b37834a0662b28a07b22e0c7ddd0bf81ff5321ce
+  data.tar.gz: eaa4bdc88f9704766dd1d5332c39d3450d654b440f53cb5316ec6686ed2e851f5e15047eddbfab5436b4ac74d13267dd508e0ba5e9c9f4cc4227528261d95031

data/CHANGELOG.md

@@ -1,5 +1,21 @@
 # Changelog
 
+## 0.3.2 - 2026-02-17
+
+### Fixed
+
+- **Short-reset overrides now visible in the dashboard.** Overrides with a very short reset (e.g. 2 seconds) previously expired before the SSE interval could push them, so the pill always showed "None". Expired overrides are now returned with `active: false` and displayed as a dimmed pill with a relative timestamp ("Registered 10s ago").
+- **Override pill no longer flickers** on SSE updates. The update path now patches the existing DOM element in-place instead of replacing `innerHTML`, and the "Registered" prefix slides in/out with a CSS transition.
+- Expired override pill uses neutral grey styling instead of the red accent colour.
+
+### Changed
+
+- Extracted `OverrideHelpers` module from `LimitPresenter` to keep class size under RuboCop limits.
+- Refactored `resolve_override` into `live_override_from` and `stored_override_from` for clarity.
+- Override snapshot hashes now always include an `"active"` boolean key.
+- Added `override_active?`, `override_updated_at`, and `override_age_label` to `LimitPresenter`.
+- Added footer with gem version and GitHub link to the dashboard.
+
 ## 0.3.1 - 2026-02-17
 
 ### Fixed

data/README.md

@@ -33,7 +33,7 @@ Requires Ruby 4.0 or later.
 
 ```ruby
 # Gemfile
-gem "hanikamu-rate-limit", "~> 0.2"
+gem "hanikamu-rate-limit", "~> 0.3"
 ```
 
 ```bash

data/lib/hanikamu/rate_limit/metrics.rb

@@ -333,22 +333,30 @@ module Hanikamu
       end
 
       def resolve_override(live_remaining, live_ttl, stored_meta)
-        # Try live override first
-        if live_remaining && live_ttl&.positive?
-          remaining = Integer(live_remaining, exception: false)
-          if remaining
-            return { "remaining" => remaining, "reset" => live_ttl,
-                     "updated_at" => Time.now.to_i }
-          end
-        end
-        # Fall back to stored meta
+        live_override_from(live_remaining, live_ttl) ||
+          stored_override_from(stored_meta)
+      end
+
+      def live_override_from(remaining_raw, ttl)
+        return nil unless remaining_raw && ttl&.positive?
+
+        remaining = Integer(remaining_raw, exception: false)
+        return nil unless remaining
+
+        { "remaining" => remaining, "reset" => ttl,
+          "updated_at" => Time.now.to_i, "active" => true }
+      end
+
+      def stored_override_from(stored_meta)
         return nil if stored_meta.nil? || stored_meta.empty?
 
         remaining_reset = compute_remaining_reset(stored_meta)
-        return nil unless remaining_reset
+        updated_at = stored_meta["updated_at"].to_i
+        active = !remaining_reset.nil?
 
-        { "remaining" => stored_meta["remaining"].to_i, "reset" => remaining_reset,
-          "updated_at" => stored_meta["updated_at"].to_i }
+        { "remaining" => stored_meta["remaining"].to_i,
+          "reset" => remaining_reset || stored_meta["reset"].to_i,
+          "updated_at" => updated_at, "active" => active }
       end
 
       def parse_override_history(raw, window, bucket_seconds)
@@ -407,7 +415,7 @@ module Hanikamu
         remaining = Integer(remaining_value, exception: false)
         return nil if remaining.nil?
 
-        { "remaining" => remaining, "reset" => ttl, "updated_at" => Time.now.to_i }
+        { "remaining" => remaining, "reset" => ttl, "updated_at" => Time.now.to_i, "active" => true }
       end
 
       def stored_override(registry)
@@ -415,10 +423,16 @@ module Hanikamu
         return nil if meta.empty?
 
         remaining_reset = compute_remaining_reset(meta)
-        return nil unless remaining_reset
+        updated_at = meta["updated_at"].to_i
 
-        { "remaining" => meta["remaining"].to_i, "reset" => remaining_reset,
-          "updated_at" => meta["updated_at"].to_i }
+        if remaining_reset
+          { "remaining" => meta["remaining"].to_i, "reset" => remaining_reset,
+            "updated_at" => updated_at, "active" => true }
+        else
+          { "remaining" => meta["remaining"].to_i,
+            "reset" => meta["reset"].to_i,
+            "updated_at" => updated_at, "active" => false }
+        end
       end
 
       def compute_remaining_reset(meta)

data/lib/hanikamu/rate_limit/ui/app/views/hanikamu/rate_limit/ui/dashboard/index.html.erb

@@ -156,6 +156,21 @@
         color: var(--accent);
         font-size: 12px;
         font-weight: 600;
+        transition: opacity 0.3s ease, border-color 0.3s ease;
+      }
+
+      .pill-age {
+        display: inline-flex;
+        overflow: hidden;
+        max-width: 0;
+        opacity: 0;
+        transition: max-width 0.4s ease, opacity 0.3s ease;
+        white-space: nowrap;
+      }
+
+      .pill--expired .pill-age {
+        max-width: 250px;
+        opacity: 1;
       }
 
       .stats {
@@ -278,6 +293,17 @@
         font-weight: 600;
       }
 
+      .pill--expired {
+        opacity: 0.55;
+        border: 1px dashed var(--line);
+        background: rgba(161, 161, 170, 0.10);
+        color: var(--muted);
+      }
+
+      .pill--expired strong {
+        color: var(--muted);
+      }
+
       .empty {
         padding: 40px 20px;
         background: var(--card);
@@ -287,6 +313,23 @@
         color: var(--muted);
       }
 
+      .page-footer {
+        text-align: center;
+        padding: 32px 0 16px;
+        color: #6b7280;
+        font-size: 0.8rem;
+      }
+
+      .page-footer a {
+        color: #6b7280;
+        text-decoration: none;
+      }
+
+      .page-footer a:hover {
+        color: #374151;
+        text-decoration: underline;
+      }
+
       @media (max-width: 720px) {
         header {
           padding-top: 24px;
@@ -415,7 +458,7 @@
               <div class="override">
                 <span>Override</span>
                 <% if limit.override? %>
-                  <span class="pill">Remaining <strong><%= limit.override_remaining %></strong> | Reset <strong><%= limit.override_reset %>s</strong></span>
+                  <span class="pill<%= ' pill--expired' unless limit.override_active? %>"><span class="pill-age">Registered&nbsp;<strong data-override-age><%= limit.override_age_label unless limit.override_active? %></strong>&nbsp;— </span>Remaining <strong data-override-remaining><%= limit.override_remaining %></strong> | Reset <strong data-override-reset><%= limit.override_reset %>s</strong></span>
                 <% else %>
                   <strong>None</strong>
                 <% end %>
@@ -432,6 +475,14 @@
       const grid = document.getElementById("limits-grid");
       const emptyState = document.getElementById("empty-state");
 
+      const timeAgo = (unixTs) => {
+        const seconds = Math.max(0, Math.floor(Date.now() / 1000) - Number(unixTs || 0));
+        if (seconds < 60) return `${seconds}s ago`;
+        if (seconds < 3600) return `${Math.floor(seconds / 60)}m ago`;
+        if (seconds < 86400) return `${Math.floor(seconds / 3600)}h ago`;
+        return `${Math.floor(seconds / 86400)}d ago`;
+      };
+
       const escapeHtml = (value) => {
         const str = String(value ?? "");
         return str
@@ -742,7 +793,9 @@
                 </div>
                 <div class="override">
                   <span>Override</span>
-                  ${override ? `<span class="pill">Remaining <strong>${escapeHtml(override.remaining)}</strong> | Reset <strong>${escapeHtml(override.reset)}s</strong></span>` : "<strong>None</strong>"}
+                  ${override
+                    ? `<span class="pill${override.active ? '' : ' pill--expired'}"><span class="pill-age">Registered&nbsp;<strong data-override-age>${override.active ? '' : timeAgo(override.updated_at)}</strong>&nbsp;— </span>Remaining <strong data-override-remaining>${escapeHtml(override.remaining)}</strong> | Reset <strong data-override-reset>${escapeHtml(override.reset)}s</strong></span>`
+                    : "<strong>None</strong>"}
                 </div>
               </article>
             `;
@@ -800,9 +853,28 @@
 
             const overrideEl = card.querySelector('.override');
             if (overrideEl) {
-              overrideEl.innerHTML = `<span>Override</span> ${override
-                ? `<span class="pill">Remaining <strong>${escapeHtml(override.remaining)}</strong> | Reset <strong>${escapeHtml(override.reset)}s</strong></span>`
-                : "<strong>None</strong>"}`;
+              if (override) {
+                let pillEl = overrideEl.querySelector('.pill');
+                if (!pillEl) {
+                  overrideEl.innerHTML = `<span>Override</span> <span class="pill${override.active ? '' : ' pill--expired'}"><span class="pill-age">Registered&nbsp;<strong data-override-age>${override.active ? '' : timeAgo(override.updated_at)}</strong>&nbsp;— </span>Remaining <strong data-override-remaining>${escapeHtml(override.remaining)}</strong> | Reset <strong data-override-reset>${escapeHtml(override.reset)}s</strong></span>`;
+                } else {
+                  pillEl.classList.toggle('pill--expired', !override.active);
+                  const ageEl = pillEl.querySelector('[data-override-age]');
+                  if (!override.active) {
+                    if (ageEl) ageEl.textContent = timeAgo(override.updated_at);
+                  } else {
+                    if (ageEl) ageEl.textContent = '';
+                  }
+                  const remEl = pillEl.querySelector('[data-override-remaining]');
+                  if (remEl) remEl.textContent = override.remaining;
+                  const resetEl = pillEl.querySelector('[data-override-reset]');
+                  if (resetEl) resetEl.textContent = `${override.reset}s`;
+                }
+              } else {
+                if (overrideEl.querySelector('.pill')) {
+                  overrideEl.innerHTML = '<span>Override</span> <strong>None</strong>';
+                }
+              }
             }
           });
         }
@@ -827,5 +899,9 @@
       // This prevents thread starvation on servers with few threads.
       window.addEventListener("beforeunload", () => source.close());
     </script>
+
+    <footer class="page-footer">
+      <a href="https://github.com/Hanikamu/hanikamu-rate-limit" target="_blank" rel="noopener noreferrer">hanikamu-rate-limit</a> v<%= Hanikamu::RateLimit::VERSION %>
+    </footer>
   </body>
 </html>

data/lib/hanikamu/rate_limit/ui/presenters/limit_presenter.rb

@@ -1,9 +1,13 @@
 # frozen_string_literal: true
 
+require_relative "override_helpers"
+
 module Hanikamu
   module RateLimit
     module UI
       class LimitPresenter
+        include OverrideHelpers
+
         EMPTY_HISTORY = { "allowed" => [], "blocked" => [], "buckets" => [] }.freeze
         EMPTY_LIFETIME = { "allowed" => 0, "blocked" => 0 }.freeze
 
@@ -110,24 +114,6 @@ module Hanikamu
           lifetime.fetch("blocked", 0)
         end
 
-        # ── Override ─────────────────────────────────────────────
-
-        def override
-          data["override"]
-        end
-
-        def override?
-          !override.nil?
-        end
-
-        def override_remaining
-          override&.fetch("remaining", nil)
-        end
-
-        def override_reset
-          override&.fetch("reset", nil)
-        end
-
         # ── Metrics ──────────────────────────────────────────────
 
         def metrics_enabled?

data/lib/hanikamu/rate_limit/ui/presenters/override_helpers.rb

@@ -0,0 +1,57 @@
+# frozen_string_literal: true
+
+module Hanikamu
+  module RateLimit
+    module UI
+      # Shared helpers for override presentation logic.
+      module OverrideHelpers
+        TIME_THRESHOLDS = [
+          [60,     ->(s) { "#{s}s ago" }],
+          [3600,   ->(s) { "#{s / 60}m ago" }],
+          [86_400, ->(s) { "#{s / 3600}h ago" }]
+        ].freeze
+
+        def override
+          data["override"]
+        end
+
+        def override?
+          !override.nil?
+        end
+
+        def override_active?
+          override&.fetch("active", false) == true
+        end
+
+        def override_remaining
+          override&.fetch("remaining", nil)
+        end
+
+        def override_reset
+          override&.fetch("reset", nil)
+        end
+
+        def override_updated_at
+          override&.fetch("updated_at", nil)
+        end
+
+        def override_age_label
+          ts = override_updated_at
+          return nil unless ts
+
+          seconds = [Time.now.to_i - ts, 0].max
+          time_ago_in_words(seconds)
+        end
+
+        private
+
+        def time_ago_in_words(seconds)
+          TIME_THRESHOLDS.each do |threshold, formatter|
+            return formatter.call(seconds) if seconds < threshold
+          end
+          "#{seconds / 86_400}d ago"
+        end
+      end
+    end
+  end
+end

data/lib/hanikamu/rate_limit/version.rb

@@ -2,6 +2,6 @@
 
 module Hanikamu
   module RateLimit
-    VERSION = "0.3.1"
+    VERSION = "0.3.2"
   end
 end

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: hanikamu-rate-limit
 version: !ruby/object:Gem::Version
-  version: 0.3.1
+  version: 0.3.2
 platform: ruby
 authors:
 - Nicolai Seerup
@@ -95,6 +95,7 @@ files:
 - lib/hanikamu/rate_limit/ui/engine.rb
 - lib/hanikamu/rate_limit/ui/presenters/dashboard_presenter.rb
 - lib/hanikamu/rate_limit/ui/presenters/limit_presenter.rb
+- lib/hanikamu/rate_limit/ui/presenters/override_helpers.rb
 - lib/hanikamu/rate_limit/version.rb
 homepage: https://github.com/Hanikamu/hanikamu-rate-limit
 licenses: