Admin dashboard: pages, Lane observability, poll pass log, per-Series intervention (#134) #148

2026-08-22T07:37:06+07:00

sulthan commented

2026-08-22 07:37:06 +07:00

Spec #134, all ten tickets. Closes #134.

What ships

The admin surface becomes four bookmarkable addresses behind one nav row, and Lane observability stops dying with the process.

#138 /admin splits into Overview, Lanes, Readers, Series, each a real route with the active tab underlined.
#139 poll_passes and poll_lanes land as durable tables with their store surface.
#140 cross-Series admin read model, with the privacy boundary in the projection: the Reader id that raised a Latest Chapter never leaves the store package.
#141 the poller records exactly one pass row per exit, with a skip reason and outcome counts.
#142 Series list: eight hygiene filters, Site and Library narrowing, paging — all of it in the query string, so a filtered list is a bookmark.
#143 Overview: a three-state verdict line and a stats block where every non-zero figure links to the list that counts it.
#144 per-Series detail page, keyed by the site:series_id composite the rest of the system already uses.
#145 the Lanes page reads the database; the in-memory Lane state, web.LaneReporter and latest.Status are deleted.
#146 Forced Poll: Check now stamps series.force_poll_at and never commands the poller.
#147 pause and resume one Site's Lane, with a mandatory 1h/6h/24h expiry.

Shape of the design

Two decisions carry the rest. Commands go through the database, never at the poller: both Check now and a Lane pause write a row the next pass reads, so they survive a restart and the whole surface stays testable with no poller running. And pending is derived, never stored — the request stamp being newer than the check stamp — which self-clears on the check stamp with no second write and no sweeper, because the check stamp is written before the fetch.

ADRs: docs/adr/0012-persisted-lane-state.md, docs/adr/0013-commands-through-the-database.md.

Verification

go test ./... green on the merged base (264839e), all packages, Docker-backed. gofmt -l and go vet clean.

Every ticket was reviewed on both axes (cr-spec + cr-standards) before merge.

Known, non-blocking

#143 the verdict ignores never-reported Lanes when other Lanes have reported, and the per-Site table lists Sites that have Series rather than the whole registry. The ticket prose asks for eight hygiene figures per Site; the design mock and the landed .tbl.sites grid both say six columns, and the mock won.
#146 two SeriesPage scans per press instead of a keyed read — ponytail:-commented in-tree with the upgrade path.
#147 a paused Site with no pass row yet renders no row and so no control, since the Lanes page lists Sites that have passed.
#141 a sibling browser Lane declining at the top of a pass records as sidecar-down. Specified deliberately; the later spec in this series settles it.

Spec #134, all ten tickets. Closes #134. ## What ships The admin surface becomes four bookmarkable addresses behind one nav row, and Lane observability stops dying with the process. - **#138** `/admin` splits into Overview, Lanes, Readers, Series, each a real route with the active tab underlined. - **#139** `poll_passes` and `poll_lanes` land as durable tables with their store surface. - **#140** cross-Series admin read model, with the privacy boundary in the projection: the Reader id that raised a Latest Chapter never leaves the store package. - **#141** the poller records exactly one pass row per exit, with a skip reason and outcome counts. - **#142** Series list: eight hygiene filters, Site and Library narrowing, paging — all of it in the query string, so a filtered list is a bookmark. - **#143** Overview: a three-state verdict line and a stats block where every non-zero figure links to the list that counts it. - **#144** per-Series detail page, keyed by the `site:series_id` composite the rest of the system already uses. - **#145** the Lanes page reads the database; the in-memory Lane state, `web.LaneReporter` and `latest.Status` are deleted. - **#146** Forced Poll: *Check now* stamps `series.force_poll_at` and never commands the poller. - **#147** pause and resume one Site's Lane, with a mandatory 1h/6h/24h expiry. ## Shape of the design Two decisions carry the rest. **Commands go through the database, never at the poller**: both *Check now* and a Lane pause write a row the next pass reads, so they survive a restart and the whole surface stays testable with no poller running. And **pending is derived, never stored** — the request stamp being newer than the check stamp — which self-clears on the check stamp with no second write and no sweeper, because the check stamp is written before the fetch. ADRs: `docs/adr/0012-persisted-lane-state.md`, `docs/adr/0013-commands-through-the-database.md`. ## Verification `go test ./...` green on the merged base (`264839e`), all packages, Docker-backed. `gofmt -l` and `go vet` clean. Every ticket was reviewed on both axes (`cr-spec` + `cr-standards`) before merge. ## Known, non-blocking - **#143** the verdict ignores never-reported Lanes when other Lanes have reported, and the per-Site table lists Sites that have Series rather than the whole registry. The ticket prose asks for eight hygiene figures per Site; the design mock and the landed `.tbl.sites` grid both say six columns, and the mock won. - **#146** two `SeriesPage` scans per press instead of a keyed read — `ponytail:`-commented in-tree with the upgrade path. - **#147** a paused Site with no pass row yet renders no row and so no control, since the Lanes page lists Sites that have passed. - **#141** a sibling browser Lane declining at the top of a pass records as `sidecar-down`. Specified deliberately; the later spec in this series settles it.

sulthan added 22 commits 2026-08-22 07:37:07 +07:00

feat: split admin surface into bookmarkable pages (#138 ) 030ffdc26e

feat(store): persist poll lane state fd1131d11d

feat(store): cross-series admin read model with privacy in the projection (#140 ) e0b9063d9e

SeriesPage returns one 50-row page of Series matching one of eight named hygiene filters (all, no_series_url, never_read_a_chapter, no_readers, never_checked, stale, no_cover, reader_report), with a window-count total; SeriesShapes returns the per-Site aggregate, one grouped pass. The compound filter value object takes Site, Kind, Name, a caller-supplied staleness cutoff, and a 1-based page.

The privacy boundary lives in the projection: adminSeriesColumns never selects latest_raised_by, and AdminSeries has no field for it — a SQL-computed boolean is all that crosses. LEFT JOIN surfaces orphans (reader_count 0); (site, series_id) tie-breaks the zero-stamp boundary so pages stay stable.

Also lands 0014: the series(latest_checked_at) index and series.force_poll_at, both expand-step schema for later tickets in the series.

feat(latest): record one poll pass per exit with skip reason and outcome counts 503fb49d0a

Every way a Lane pass can end now writes exactly one durable row: a skip
value naming the exit (paused, refusing, sidecar-down, no-fetcher,
due-query, asleep, eligible-count, nothing-eligible, or empty for the
loop), five outcome counts from the classification the Series read already
makes, and carry-forward of the previous pass's figures exactly when the
pass's own gap is zero. Refusal is durable through the poll_lanes row, so
a restart does not re-probe a Site inside its backoff. Retention is 14
days. The mid-loop browser-unreachable return writes an empty skip by
design: a tenth value is not invented here. (#141)

feat(web): per-Series detail page keyed by site:series_id (#144 ) 14990bde21

#142 : series list page with eight hygiene filters, site/library narrowing, paging e8a3c5f826

fix(web): detail row lookup walks all pages of the Site read (#144 ) 09a094ca67

#142 : fix stale filter count cutoff, tighten comments (review fixes) 134c9307b1

feat(web): read admin lanes from the durable pass log, not a poller snapshot (#145 ) cb2b104e63

The Lanes page now projects store.LatestLanePasses and the owner-window
outcomes (store.LanePassOutcomes) into per-Site rows instead of reading an
in-memory Poller snapshot, so a deploy answers the instant the store is up.
Browser configuration is a config fact and reachability is derived from
recent browser-Site passes inside latest.RefuseBackoff.

This atomically deletes the in-memory path in the same commit that makes the
page read the DB: latest/status.go (LaneState, Status, LaneStatus,
recordLaneState) and the web.LaneReporter seam plus fakeLanes are gone, and
latest.refuseBackoff is renamed latest.RefuseBackoff at every callsite.
ownerWindow (#142) is referenced, never declared (contract C2)

fix(latest,web): finish the atomic deletion and honest empty states (#145 ) fc8a40cfc6

Review fixes on top of cb2b104:
- Delete the write-only in-memory refuseUntil map and setRefusalBackoff now
  that status.go is gone: the pass gate reads the durable stamp, so the map
  was half-deleted dead state (spec review C-1).
- An outcome-query failure no longer blanks the Lane table into the false
  'no data yet': rows render with 'none observed' chips instead (I-1).
- due-query and eligible-count skips get their own sentences instead of the
  merged 'check failed' (I-2).
- ADR-0012 constrain wording corrected and trailing newline added (M-1)

fix(web): statusline tokens, unreachable dot, reachability test spans (#145 ) d72c48295d

Second review round (spec + standards):
- Polling-off and unreachable render as statusline tokens; the unreachable
  span drops mark-strong so the patina dot never sits next to red text.
- Reachability tests assert on the rendered span, which 'reachable' and
  'unreachable' substrings never could.
- A pass-log query failure now reads as reachable (no evidence rule) instead
  of condemning the browser, and admin.go loses its dead time import.
- startLatestPoller's doc no longer claims the admin page reads the poller;
  AGENTS.md carries the RefuseBackoff rename.

Merge ticket/142-series-list bf08d6e65c

Merge ticket/144-series-detail f5b52e48c8

Merge ticket/145-lanes-from-db 6f9109c885

feat(web): overview page with verdict line and stats doors (#143 ) c432a3be30

feat(web,store,latest): forced poll — Check now stamps a Series for the next Lane pass (#146 ) fe2cd12049

The control writes series.force_poll_at (column landed in migration 0014)
and never commands the poller: pending is derived as force_poll_at >
latest_checked_at and self-clears because the check stamp is written before
the fetch. The due query's forced flag overrides the rest cutoff, the
Sighting-deferral and finished-only clauses, jumps the queue, and wakes a
sleeping browser Lane; it never overrides an empty series_url, the Bookmarks
join, the refusal backoff, the sidecar-down skip or the Lane gap.

ADRs: 0013-commands-through-the-database.

fix(web): verdict space per design mock, single door() fig builder (#143 ) 00fa237151

refactor(web): share poll-state derivation; keep zebra parity across a row swap (#146 ) 5b7adf5f2c

Review round: pollState() unifies the CanPoll/Pending/Requested derivation
used by the list row and the detail page, and the row anchor carries its
band parity with the press (hx-vals) so the swapped answer keeps the
alternation.

Merge ticket/143-overview 4a95657425

Merge ticket/146-forced-poll e45445cb20

feat(web,latest): pause and resume one Site's Lane with a mandatory expiry (#147 ) cbe0a28921

Two owner-gated POST routes write the durable poll_lanes pause stamp the
poller's top-of-pass gate already reads: /admin/lanes/{site}/pause validates
the duration against the fixed 1h/6h/24h allow-list and the Site against the
registry, and /admin/lanes/{site}/resume zeroes the stamp. Both cap the form
body like the API path, answer with the freshly rendered Lanes block, and
never command the poller — the pause is a fact about the Site, so it
survives a restart. The Lanes page's c-ctrl slot now carries the pausebar:
Resume while paused, the duration select plus Pause while running, with the
paused phrase read from the live poll_lanes stamp so a press renders as
paused with no pass having run. Tests cover the round trips, rejections,
body caps, the pause-before-refusal ordering, fresh-poller survival, resume
restoring the full queue, and acquisition being unaffected.

Merge ticket/147-lane-pause 264839e798

sulthan merged commit 889f0f3f38 into main

2026-08-22 08:27:20 +07:00

Sign in to join this conversation.

1 Participants

Notifications

Due Date

No due date set.

Dependencies

No dependencies set.

Reference: sulthan/mangaBookmark#148