Lower minimum refresh-interval from 30s to 5s

Upstream rate limiting is the job of the widget's cache duration,
not the client refresh interval. A refresh-interval tick that lands
inside the cache window simply re-renders existing data — no
upstream call. The 30s floor was over-cautious; 5s is enough to
prevent a 1s-typo footgun while letting users dial in a tight
local-feel cadence on widgets like monitor or server-stats whose
cache controls the actual fetch rate.

Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>

Author: jrnxf

docs/configuration.md

@@ -741,7 +741,7 @@ cache: 1d  # 1 day
 > Not all widgets can have their cache duration modified. The calendar and weather widgets update on the hour and this cannot be changed.
 
 #### `refresh-interval`
-How often the widget should automatically refresh in the browser without a full page reload. Same syntax as `cache` (a number followed by `s`, `m`, `h`, or `d`). Each tick force-fetches fresh data, bypassing the widget's cache. The minimum allowed interval is 30 seconds.
+How often the widget should automatically refresh in the browser without a full page reload. Same syntax as `cache` (a number followed by `s`, `m`, `h`, or `d`). Each tick re-renders the widget; if the widget's cache hasn't expired, the existing data is reused, so a tight refresh interval is safe and won't hammer upstream APIs. Use the manual refresh button to force a fresh fetch. The minimum allowed interval is 5 seconds.
 
 ```yaml
 refresh-interval: 1m

internal/glance/widget.go

@@ -161,7 +161,11 @@ var refreshIntervalDisallowedTypes = map[string]struct{}{
 	"bookmarks":       {},
 }
 
-const minRefreshInterval = 30 * time.Second
+// minRefreshInterval is a floor against typos (e.g. accidentally writing 1s).
+// Upstream rate limiting should be handled by the widget's cache duration,
+// not by this floor — refresh-interval ticks below the cache return the same
+// data without re-fetching, so a tight client interval is safe by default.
+const minRefreshInterval = 5 * time.Second
 
 type cacheType int
 

internal/glance/widget_test.go

@@ -14,9 +14,9 @@ func TestWidgetValidateRefreshInterval(t *testing.T) {
 		errContains string
 	}{
 		{name: "no interval is allowed on any type", widgetType: "clock", interval: 0},
-		{name: "valid interval on rss", widgetType: "rss", interval: 30 * time.Second},
+		{name: "valid interval on rss", widgetType: "rss", interval: 5 * time.Second},
 		{name: "valid interval on hacker-news", widgetType: "hacker-news", interval: 1 * time.Minute},
-		{name: "below minimum", widgetType: "rss", interval: 29 * time.Second, errContains: "at least 30s"},
+		{name: "below minimum", widgetType: "rss", interval: 4 * time.Second, errContains: "at least 5s"},
 		{name: "disallowed: clock", widgetType: "clock", interval: 1 * time.Minute, errContains: `type "clock"`},
 		{name: "disallowed: calendar", widgetType: "calendar", interval: 1 * time.Minute, errContains: `type "calendar"`},
 		{name: "disallowed: to-do", widgetType: "to-do", interval: 1 * time.Minute, errContains: `type "to-do"`},