Add IP search and status filter to the Overview and Registry dashboard pages

Both pages rendered their full lists with no way to narrow them. Adds a
?q=&status= filter (substring match on address, exact match on result
status) applied dashboard-side, in Go, over the already-fetched list —
no control-api/db changes needed.

Overview: the filter form lives outside the polling target (#overview-live)
so the recurring poll never wipes out what's typed/selected; the poll and
both filter inputs share hx-sync="#overview-live:queue last" (the same
fix that resolved the earlier abandoned /ips auto-refresh races) and the
poll now carries hx-include="#overview-filter" so it keeps honoring the
current filter on every tick. Applies uniformly to both the "Текущая
проверка" and "Последние N завершённых" tables, per the confirmed design:
picking a specific status naturally hides in-progress rows, since they
have no result yet.

Registry: no polling exists there, so the filter form reuses the full page
via hx-select/hx-replace-url — simpler than adding a parallel fragment
endpoint, and gives a bookmarkable/shareable filtered URL.

Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com>

internal/dashboard/handlers_overview.go

@@ -3,6 +3,7 @@ package dashboard
 import (
 	"net/http"
 	"sort"
+	"strings"
 )
 
 type overviewData struct {
@@ -13,6 +14,8 @@ type overviewData struct {
 	Breakdown     map[string]int
 	LastN         int
 	PollSeconds   int
+	Query         string
+	StatusFilter  string
 }
 
 func (s *Server) loadOverview(r *http.Request) (overviewData, error) {
@@ -25,14 +28,18 @@ func (s *Server) loadOverview(r *http.Request) (overviewData, error) {
 	if err != nil {
 		return overviewData{}, err
 	}
+	q := strings.TrimSpace(r.URL.Query().Get("q"))
+	resultFilter := r.URL.Query().Get("status")
 	last := lastCompleted(ips, s.Cfg.LastCompletedCount)
 	return overviewData{
 		Status:        status,
-		CurrentItems:  currentlyChecking(ips),
-		LastCompleted: last,
+		CurrentItems:  filterQueueItems(currentlyChecking(ips), q, resultFilter),
+		LastCompleted: filterQueueItems(last, q, resultFilter),
 		Breakdown:     resultBreakdown(last),
 		LastN:         s.Cfg.LastCompletedCount,
 		PollSeconds:   s.Cfg.OverviewPollIntervalS,
+		Query:         q,
+		StatusFilter:  resultFilter,
 	}, nil
 }
 
@@ -89,3 +96,26 @@ func resultBreakdown(items []ipQueueItem) map[string]int {
 	}
 	return out
 }
+
+// filterQueueItems narrows items to those whose address contains q
+// (case-insensitive substring) and, if status is set, whose OverallResult
+// matches it exactly. A still-in-progress item always has an empty
+// OverallResult, so picking any specific status hides it — the intended
+// behavior for "Текущая проверка", which has no verdict yet.
+func filterQueueItems(items []ipQueueItem, q, status string) []ipQueueItem {
+	if q == "" && status == "" {
+		return items
+	}
+	q = strings.ToLower(q)
+	out := make([]ipQueueItem, 0, len(items))
+	for _, ip := range items {
+		if q != "" && !strings.Contains(strings.ToLower(ip.IPAddress), q) {
+			continue
+		}
+		if status != "" && ip.OverallResult != status {
+			continue
+		}
+		out = append(out, ip)
+	}
+	return out
+}

internal/dashboard/handlers_registry.go

@@ -1,10 +1,15 @@
 package dashboard
 
-import "net/http"
+import (
+	"net/http"
+	"strings"
+)
 
 type registryPageData struct {
 	PageData
-	Items []registryItem
+	Items        []registryItem
+	Query        string
+	StatusFilter string
 }
 
 type registryDetailData struct {
@@ -15,15 +20,45 @@ type registryDetailData struct {
 // handleRegistryPage lists every address ever submitted to the check
 // queue, with a summary of its accumulated check history — the durable
 // record that survives an address being deleted from /ips and later
-// re-added. See internal/db/migrations/0007_ip_registry.sql.
+// re-added. See internal/db/migrations/0007_ip_registry.sql. Optional
+// ?q=&status= query params narrow the list by address substring and by
+// LastResult — see filterRegistryItems.
 func (s *Server) handleRegistryPage(w http.ResponseWriter, r *http.Request) {
 	items, err := s.CA.ListRegistry(r.Context())
-	data := registryPageData{Items: items}
+	q := strings.TrimSpace(r.URL.Query().Get("q"))
+	status := r.URL.Query().Get("status")
+	data := registryPageData{
+		Items:        filterRegistryItems(items, q, status),
+		Query:        q,
+		StatusFilter: status,
+	}
 	data.ActiveNav = "registry"
 	data.Banner = bannerFor(err)
 	s.renderPage(w, "registry_page", data)
 }
 
+// filterRegistryItems narrows items to those whose address contains q
+// (case-insensitive substring) and, if status is set, whose LastResult
+// matches it exactly — the registry list's search-by-IP and
+// filter-by-status, mirroring filterQueueItems in handlers_overview.go.
+func filterRegistryItems(items []registryItem, q, status string) []registryItem {
+	if q == "" && status == "" {
+		return items
+	}
+	q = strings.ToLower(q)
+	out := make([]registryItem, 0, len(items))
+	for _, it := range items {
+		if q != "" && !strings.Contains(strings.ToLower(it.IPAddress), q) {
+			continue
+		}
+		if status != "" && it.LastResult != status {
+			continue
+		}
+		out = append(out, it)
+	}
+	return out
+}
+
 // handleRegistryDetail shows one address's full retained check history
 // across every cycle it has ever run, not just the current attempt — see
 // ip_detail_content in ip_detail.html for the attempt-scoped equivalent.

internal/dashboard/handlers_test.go

@@ -36,6 +36,51 @@ func TestOverviewFragment(t *testing.T) {
 	}
 }
 
+// TestOverviewFilterByQueryAndStatus proves the ?q=&status= params on both
+// /overview and /overview/fragment narrow the current-checking and
+// last-completed tables to matching rows only, and that the filter form
+// echoes the applied values back (so they survive the recurring poll,
+// which only re-renders overview_fragment, not the form around it).
+func TestOverviewFilterByQueryAndStatus(t *testing.T) {
+	fake, caURL := newFakeControlAPI(t)
+	now := time.Now()
+	fake.ips = []ipQueueItem{
+		{IPAddress: "1.1.1.1", State: "checking", Sequence: 1, AssignedAt: &now, UpdatedAt: now, CreatedAt: now},
+		{IPAddress: "1.1.1.2", State: "checking", Sequence: 2, AssignedAt: &now, UpdatedAt: now, CreatedAt: now},
+		{IPAddress: "2.2.2.2", State: "done", OverallResult: "pass", AggregatedAt: &now, UpdatedAt: now, CreatedAt: now},
+		{IPAddress: "3.3.3.3", State: "failed", OverallResult: "fail", AggregatedAt: &now, UpdatedAt: now, CreatedAt: now},
+	}
+	ts := newTestServer(t, caURL)
+
+	// Search narrows both tables by IP substring.
+	body := get(t, ts, "/overview/fragment?q=1.1.1.1")
+	if !strings.Contains(body, "1.1.1.1") || strings.Contains(body, "1.1.1.2") {
+		t.Fatalf("expected search to match only 1.1.1.1, got:\n%s", body)
+	}
+
+	// Status filter narrows the completed table and hides in-progress rows
+	// entirely (they have no OverallResult yet).
+	body = get(t, ts, "/overview/fragment?status=fail")
+	if strings.Contains(body, "1.1.1.1") || strings.Contains(body, "2.2.2.2") {
+		t.Fatalf("expected in-progress and passing rows hidden under status=fail, got:\n%s", body)
+	}
+	if !strings.Contains(body, "3.3.3.3") {
+		t.Fatalf("expected failing row to remain under status=fail, got:\n%s", body)
+	}
+	if !strings.Contains(body, "Ничего не найдено по текущему фильтру") {
+		t.Fatalf("expected the empty current-checking table to show the filtered empty state, got:\n%s", body)
+	}
+
+	// The full page echoes q/status back into the filter form's inputs.
+	page := get(t, ts, "/overview?q=1.1.1.1&status=pass")
+	if !strings.Contains(page, `value="1.1.1.1"`) {
+		t.Fatalf("expected search box to echo back the query, got:\n%s", page)
+	}
+	if !strings.Contains(page, `value="pass" selected`) {
+		t.Fatalf("expected status select to echo back the selection, got:\n%s", page)
+	}
+}
+
 func TestIPsSubmitAddAndForceRecheck(t *testing.T) {
 	fake, caURL := newFakeControlAPI(t)
 	now := time.Now()
@@ -526,6 +571,58 @@ func TestRegistryPageAndDetail(t *testing.T) {
 	}
 }
 
+// TestRegistryFilterByQueryAndStatus proves the ?q=&status= params on
+// /registry narrow the list by address substring and by LastResult, and
+// that the filter form echoes the applied values back into its inputs.
+func TestRegistryFilterByQueryAndStatus(t *testing.T) {
+	fake, caURL := newFakeControlAPI(t)
+	now := time.Now()
+	fake.registry["1.1.1.1"] = registryItem{IPAddress: "1.1.1.1", FirstSeenAt: now, LastSeenAt: now, TotalCycles: 1, LastResult: "pass"}
+	fake.registry["1.1.1.2"] = registryItem{IPAddress: "1.1.1.2", FirstSeenAt: now, LastSeenAt: now, TotalCycles: 1, LastResult: "fail"}
+	fake.registry["2.2.2.2"] = registryItem{IPAddress: "2.2.2.2", FirstSeenAt: now, LastSeenAt: now, TotalCycles: 1, LastResult: "pass"}
+	ts := newTestServer(t, caURL)
+
+	// Table rows link to /registry/{ip} — check for that instead of the
+	// bare IP substring, since the filter form echoes q/status back into
+	// its own input's value="..." attribute, which would otherwise cause
+	// a false match/mismatch for whatever address happens to equal q.
+	rowLink := func(ip string) string { return `href="/registry/` + ip + `"` }
+
+	// Search by IP substring.
+	body := get(t, ts, "/registry?q=1.1.1")
+	if !strings.Contains(body, rowLink("1.1.1.1")) || !strings.Contains(body, rowLink("1.1.1.2")) || strings.Contains(body, rowLink("2.2.2.2")) {
+		t.Fatalf("expected only the 1.1.1.x addresses to match the search, got:\n%s", body)
+	}
+
+	// Filter by status.
+	body = get(t, ts, "/registry?status=fail")
+	if strings.Contains(body, rowLink("1.1.1.1")) || strings.Contains(body, rowLink("2.2.2.2")) {
+		t.Fatalf("expected only the failing address under status=fail, got:\n%s", body)
+	}
+	if !strings.Contains(body, rowLink("1.1.1.2")) {
+		t.Fatalf("expected the failing address to remain under status=fail, got:\n%s", body)
+	}
+
+	// Both together (AND): 2.2.2.2 matches q but not status=fail (it's
+	// pass), so no row should match.
+	body = get(t, ts, "/registry?q=2.2.2.2&status=fail")
+	if strings.Contains(body, rowLink("1.1.1.2")) || strings.Contains(body, rowLink("2.2.2.2")) {
+		t.Fatalf("expected no match for q=2.2.2.2&status=fail (2.2.2.2 is pass), got:\n%s", body)
+	}
+	if !strings.Contains(body, "Ничего не найдено по текущему фильтру") {
+		t.Fatalf("expected the filtered-empty state, got:\n%s", body)
+	}
+
+	// The page echoes q/status back into the filter form's inputs.
+	page := get(t, ts, "/registry?q=1.1.1&status=pass")
+	if !strings.Contains(page, `value="1.1.1"`) {
+		t.Fatalf("expected search box to echo back the query, got:\n%s", page)
+	}
+	if !strings.Contains(page, `value="pass" selected`) {
+		t.Fatalf("expected status select to echo back the selection, got:\n%s", page)
+	}
+}
+
 func TestControlAPIUnreachable(t *testing.T) {
 	// Point the dashboard at an address nothing listens on, rather than a
 	// closed httptest.Server, to get a deterministic connection-refused
@@ -536,3 +633,61 @@ func TestControlAPIUnreachable(t *testing.T) {
 		t.Fatalf("expected server/transport error banner when control-api is unreachable, got:\n%s", body)
 	}
 }
+
+func TestFilterQueueItems(t *testing.T) {
+	items := []ipQueueItem{
+		{IPAddress: "1.1.1.1", OverallResult: "pass"},
+		{IPAddress: "1.1.1.2", OverallResult: "fail"},
+		{IPAddress: "2.2.2.2", OverallResult: "pass"},
+	}
+
+	if got := filterQueueItems(items, "", ""); len(got) != 3 {
+		t.Fatalf("expected no-op with empty q/status, got %+v", got)
+	}
+	if got := filterQueueItems(items, "1.1.1", ""); len(got) != 2 {
+		t.Fatalf("expected 2 matches for q=1.1.1, got %+v", got)
+	}
+	if got := filterQueueItems(items, "1.1.1.1", ""); len(got) != 1 || got[0].IPAddress != "1.1.1.1" {
+		t.Fatalf("expected exact-substring match, got %+v", got)
+	}
+	if got := filterQueueItems(items, "1.1.1.1", ""); len(got) != 1 {
+		t.Fatalf("expected search to be case/substring based, got %+v", got)
+	}
+	if got := filterQueueItems(items, "", "pass"); len(got) != 2 {
+		t.Fatalf("expected 2 matches for status=pass, got %+v", got)
+	}
+	if got := filterQueueItems(items, "1.1.1", "pass"); len(got) != 1 || got[0].IPAddress != "1.1.1.1" {
+		t.Fatalf("expected q+status combined with AND, got %+v", got)
+	}
+	if got := filterQueueItems(items, "9.9.9.9", ""); len(got) != 0 {
+		t.Fatalf("expected no matches, got %+v", got)
+	}
+
+	// Case-insensitivity, via a query with mixed-case letters (IP octets
+	// are numeric, so exercise it through IPv6-shaped input instead).
+	mixed := []ipQueueItem{{IPAddress: "fe80::AbCd"}}
+	if got := filterQueueItems(mixed, "abcd", ""); len(got) != 1 {
+		t.Fatalf("expected case-insensitive search to match, got %+v", got)
+	}
+}
+
+func TestFilterRegistryItems(t *testing.T) {
+	items := []registryItem{
+		{IPAddress: "1.1.1.1", LastResult: "pass"},
+		{IPAddress: "1.1.1.2", LastResult: "partial"},
+		{IPAddress: "2.2.2.2", LastResult: ""},
+	}
+
+	if got := filterRegistryItems(items, "", ""); len(got) != 3 {
+		t.Fatalf("expected no-op with empty q/status, got %+v", got)
+	}
+	if got := filterRegistryItems(items, "1.1.1", ""); len(got) != 2 {
+		t.Fatalf("expected 2 matches for q=1.1.1, got %+v", got)
+	}
+	if got := filterRegistryItems(items, "", "partial"); len(got) != 1 || got[0].IPAddress != "1.1.1.2" {
+		t.Fatalf("expected exactly the partial-result address, got %+v", got)
+	}
+	if got := filterRegistryItems(items, "2.2.2", "partial"); len(got) != 0 {
+		t.Fatalf("expected q+status combined with AND to exclude non-matching, got %+v", got)
+	}
+}

internal/dashboard/templates/overview.html

@@ -23,7 +23,36 @@
 <div class="topbar">
 <h1>Обзор</h1>
 </div>
-<div id="overview-live" hx-get="/overview/fragment" hx-trigger="every {{.PollSeconds}}s" hx-swap="innerHTML">
+
+<form id="overview-filter" class="panel" onsubmit="return false">
+<div class="panel-body field-row">
+<div class="field" style="flex:1 1 260px">
+<label for="overview-q">Поиск по IP</label>
+<input type="search" id="overview-q" name="q" value="{{.Query}}" placeholder="203.0.113.10"
+  hx-get="/overview/fragment" hx-target="#overview-live" hx-swap="innerHTML"
+  hx-include="#overview-filter" hx-trigger="input changed delay:300ms"
+  hx-sync="#overview-live:queue last">
+</div>
+<div class="field">
+<label for="overview-status">Статус</label>
+<select id="overview-status" name="status"
+  hx-get="/overview/fragment" hx-target="#overview-live" hx-swap="innerHTML"
+  hx-include="#overview-filter" hx-trigger="change" hx-sync="#overview-live:queue last">
+<option value="">Все</option>
+<option value="pass" {{if eq .StatusFilter "pass"}}selected{{end}}>pass</option>
+<option value="partial" {{if eq .StatusFilter "partial"}}selected{{end}}>partial</option>
+<option value="fail" {{if eq .StatusFilter "fail"}}selected{{end}}>fail</option>
+<option value="cancelled" {{if eq .StatusFilter "cancelled"}}selected{{end}}>cancelled</option>
+</select>
+</div>
+</div>
+<p class="muted" style="padding:0 16px 12px">Действует на обе таблицы ниже. Фильтр по статусу — это фильтр по
+итоговому результату, поэтому при выборе конкретного статуса строки «Текущей проверки» (у неё ещё нет результата)
+не показываются.</p>
+</form>
+
+<div id="overview-live" hx-get="/overview/fragment" hx-trigger="every {{.PollSeconds}}s"
+  hx-include="#overview-filter" hx-sync="#overview-live:queue last" hx-swap="innerHTML">
 {{template "overview_fragment" .}}
 </div>
 {{end}}

internal/dashboard/templates/overview_fragment.html

@@ -29,7 +29,8 @@
 </div>
 </div>
 {{else}}
-<p class="muted">Сейчас нет адресов в обработке.</p>
+{{if or .Query .StatusFilter}}<p class="muted">Ничего не найдено по текущему фильтру.</p>
+{{else}}<p class="muted">Сейчас нет адресов в обработке.</p>{{end}}
 {{end}}
 
 <h2 class="section-title">Последние {{.LastN}} завершённых</h2>
@@ -58,6 +59,7 @@
 </div>
 </div>
 {{else}}
-<p class="muted">Пока ничего не завершено.</p>
+{{if or .Query .StatusFilter}}<p class="muted">Ничего не найдено по текущему фильтру.</p>
+{{else}}<p class="muted">Пока ничего не завершено.</p>{{end}}
 {{end}}
 {{end}}

internal/dashboard/templates/registry.html

@@ -25,6 +25,37 @@
 сохраняется здесь даже после удаления адреса из <a href="/ips">очереди</a> и не теряется при повторном добавлении.
 Глубина хранимой истории на адрес настраивается на <a href="/settings">странице настроек</a>.</p>
 
+<form id="registry-filter" class="panel" onsubmit="return false" style="margin-bottom:16px">
+<div class="panel-body field-row">
+<div class="field" style="flex:1 1 260px">
+<label for="registry-q">Поиск по IP</label>
+<input type="search" id="registry-q" name="q" value="{{.Query}}" placeholder="203.0.113.10"
+  hx-get="/registry" hx-select="#registry-table-wrap" hx-target="#registry-table-wrap" hx-swap="outerHTML"
+  hx-include="#registry-filter" hx-trigger="input changed delay:300ms"
+  hx-replace-url="true" hx-sync="#registry-table-wrap:queue last">
+</div>
+<div class="field">
+<label for="registry-status">Статус</label>
+<select id="registry-status" name="status"
+  hx-get="/registry" hx-select="#registry-table-wrap" hx-target="#registry-table-wrap" hx-swap="outerHTML"
+  hx-include="#registry-filter" hx-trigger="change"
+  hx-replace-url="true" hx-sync="#registry-table-wrap:queue last">
+<option value="">Все</option>
+<option value="pass" {{if eq .StatusFilter "pass"}}selected{{end}}>pass</option>
+<option value="partial" {{if eq .StatusFilter "partial"}}selected{{end}}>partial</option>
+<option value="fail" {{if eq .StatusFilter "fail"}}selected{{end}}>fail</option>
+<option value="cancelled" {{if eq .StatusFilter "cancelled"}}selected{{end}}>cancelled</option>
+</select>
+</div>
+</div>
+</form>
+
+<div id="registry-table-wrap">
+{{template "registry_table" .}}
+</div>
+{{end}}
+
+{{define "registry_table"}}
 {{if .Items}}
 <div class="panel">
 <div class="table-scroll">
@@ -53,5 +84,6 @@
 </table>
 </div>
 </div>
+{{else if or .Query .StatusFilter}}<p class="muted">Ничего не найдено по текущему фильтру.</p>
 {{else}}<p class="muted">Реестр пуст — ни один адрес ещё не ставился на проверку.</p>{{end}}
 {{end}}