Archived and finished buckets, userscript nav chips (#4)

Gives every bookmark a lifecycle bucket — `reading`, `archived`, or `finished` — so on-hold series leave the main list while still being polled for new chapters, completed series get a web-only bucket, and the userscript panel gains quick links to the web UI and both manga sites.

Design: `docs/superpowers/specs/2026-07-27-status-buckets-design.md`

## Data model

One additive column through the existing `addedColumns` migration list:

```sql
ALTER TABLE bookmarks ADD COLUMN status TEXT NOT NULL DEFAULT 'reading'
```

The `DEFAULT` backfills every pre-existing row as `reading`, so there is no separate migration step. `favorite` is unchanged and orthogonal — a series can be an archived favourite.

Rollback is safe: an old binary against the new database omits `status` from its INSERT (it gets the DEFAULT) and never mentions it in the conflict clause, so buckets survive.

## The write rule

`PUT /bookmarks/{key}` decodes a whole `Bookmark` and `Upsert` writes every column it knows about. `latest_checked_at` escaped this by staying out of `bookmarkColumns` entirely — `status` cannot, because the userscript must be able to archive and restore.

So an empty incoming status means **"no opinion"**, not a value, and resolves on the `VALUES` side of the upsert:

```sql
COALESCE(NULLIF(?, ''), (SELECT status FROM bookmarks WHERE key = ?), 'reading')
```

with `DO UPDATE SET status = excluded.status`.

It has to be this way round. `excluded.*` is the row *after* the `VALUES` expressions are evaluated, so applying the default there and then reading `excluded.status` in the conflict clause would see `'reading'` rather than the empty string — and would overwrite an archived row on every progress PUT from a client that knows nothing about the column. One expression, evaluated once, covers insert and update alike. The subquery runs inside the transaction, so it sees the row the statement is about to conflict with.

`TestUpsertEmptyStatusPreservesStored` is the guard on this.

`updated_at` behaviour is unchanged: it moves only when `last_chapter_num` changes, so archiving, finishing, restoring, and favouriting never reorder the list.

## Validation

`PUT /bookmarks/{key}` returns 400 for any status outside `{"", "reading", "archived", "finished"}`, and for `"finished"` specifically. Finishing a series is a web-UI decision, enforced server-side rather than by trusting every client to leave the value alone. The `/ui/*` endpoints have their own session-guarded route and are unaffected.

## Visibility

| Surface | All | Updated | Favourites | Archived | Finished |
|---|---|---|---|---|---|
| Web | reading | reading | reading | archived | finished |
| Userscript | reading | — | reading | archived | not shown |

Archived and finished appear in their own tab and nowhere else — including the web UI's "Continue reading" strip, which is now built from reading-only rows before tab filtering. An archived favourite shows up under Archived only: Favourites means "favourites I am currently reading".

## Backend

- **`store.go`** — `Bookmark.Status`, the column in `schema` / `addedColumns` / `bookmarkColumns` / `scanBookmark` / `Upsert`. `scanBookmark` normalises anything outside the three known buckets to `reading`, so no row can land in no list at all.
- **`store.go`** — `DueForLatestCheck` gains `AND status IS NOT 'finished'`. Archived series keep being polled; that is the whole point of archiving rather than deleting. Finished ones have nothing coming, so polling them only burns fetches and risks a spurious "new chapter" badge. `IS NOT` is null-safe, so a hand-edited NULL still qualifies.
- **`handlers.go`** — status validation on `PUT`, before any write.
- **`web.go`** — `buildListView` filters the new tabs and excludes both buckets from `all` / `new` / `fav` and the recent strip; new `POST /ui/bookmarks/{key}/status`, session-guarded like its siblings, read-modify-writing through `Store.Get` + `Store.Upsert` so the `updated_at` rule stays in one place.
- **`templates/`** — two more tabs; per-card controls (reading → Archive + Finish, archived → Restore + Finish, finished → Restore); empty-state copy for both new tabs.
- **`static/style.css`** — five tabs no longer divide a phone's width legibly, so the row scrolls sideways instead of squeezing.

## Userscript (1.3.0)

- Third tab **Archived** beside All and Favourites. A missing `status` reads as `reading`, so a list cached by the previous version still renders. `finished` matches no tab and is invisible everywhere.
- Per-item **Archive / Unarchive** button on the existing optimistic path: mutate local state and cache, `apiPut`, adopt the server's returned row.
- Header chip row linking the web UI and both manga sites, each `target="_blank" rel="noopener"`.
- `apiPut` now omits `status` unless the caller opts in — see below.

Still free of every `GM_*` API: plain `fetch`, page `localStorage`, on-page UI only.

## One bug worth calling out

The userscript's other mutations (`updateToCurrentChapter`, `setChapterManual`, `toggleFavorite`, `applyLatestChapterIfChanged`) build their payload with `Object.assign({}, existing, …)`, so they echoed the cached `status` back to the server. `GET /bookmarks` has no status filter — finished rows are in `state.list` and only hidden at render time — which made two failures reachable:

1. Reading a chapter of a series marked finished sent `"status":"finished"`, which the API rejects with 400. Progress never synced, behind a misleading "Offline — saved locally, will retry" toast, permanently.
2. Archiving on desktop and then reading on a phone whose cache predated the archive sent `"status":"reading"` and silently un-archived the series — contradicting the README's "reading an archived series leaves it archived".

Fixed at the single choke point: `apiPut(key, obj, { sendStatus = false })` strips `status` from a copy of the body unless the caller opts in, and only `toggleArchive` opts in. Only an explicit archive/restore has an opinion about the bucket; everything else omits the field so the server's keep-on-empty rule applies. Stripping merely the *invalid* values would not have been enough — a stale cached `"reading"` still clobbers a remote archive.

Also: the userscript's `backgroundRefreshLatest` now skips finished series, matching the server poller, instead of spending batch slots fetching pages for a series that has nothing coming.

## Known limitations, deliberate

Both are marked in-code with `ponytail:` comments naming the ceiling and the upgrade path:

- The poller's `Store.Get` + `Store.Upsert` is not wrapped in a transaction, so a client PUT that commits between the two is lost to the stale re-read. Already documented for read progress in `CLAUDE.md`; it now costs a status change too. Accepted for a single-user deployment.
- A card whose new status no longer matches the active tab stays on screen until the next list load. The alternative is an out-of-band swap or a full list refresh per toggle, and the card visibly showing its new state is enough feedback.

## Testing

`go test ./...` passes; `CGO_ENABLED=0 go build ./...` clean.

- **`store_test.go`** — a fresh row defaults to `reading`; a legacy database gains the column with every row `reading`; an `Upsert` carrying `""` preserves the stored bucket while a value replaces it; a status change does not move `updated_at`; `DueForLatestCheck` returns archived and skips finished; the poller's `Get` → `Upsert` round trip preserves `archived`.
- **`main_test.go`** — `PUT` with `finished` or garbage is 400, `""` / `reading` / `archived` round-trip; a PUT that omits the `status` key entirely (what a pre-1.3.0 userscript sends) preserves an archived bucket *and* applies the chapter progress in the same request.
- **`web_test.go`** — each tab returns only its bucket; the recent strip excludes archived and finished; the status endpoint requires a session, rejects unknown values, and does not move `updated_at`; the card renders the right controls per bucket.

Userscript has no automated harness, so it was checked against a live `https://asurascans.com` page: the chips resolve, Archive moves a series out of All and Favourites into Archived, the state survives a full reload (so it came from the server, not local optimism), Unarchive returns it, a series marked finished in the web UI appears in no tab, and — captured on the wire — the archive PUT carries `"status":"archived"` while a favourite toggle on that same archived series carries no `status` key at all.

Reviewed-on: #4
Co-authored-by: Sulthan Zaki <sultankiki05@gmail.com>
Co-committed-by: Sulthan Zaki <sultankiki05@gmail.com>

backend/handlers.go

@@ -60,6 +60,21 @@ func (h *bookmarkHandler) put(w http.ResponseWriter, r *http.Request) {
 			}
 		}
 	}
+
+	// An empty status is "no opinion" and Upsert keeps the stored bucket.
+	// Finishing a series is a web-UI decision, so the JSON API refuses it
+	// rather than trusting every client to leave it alone.
+	switch b.Status {
+	case "", statusReading, statusArchived:
+	case statusFinished:
+		http.Error(w, "status "+statusFinished+" can only be set from the web UI",
+			http.StatusBadRequest)
+		return
+	default:
+		http.Error(w, "invalid status", http.StatusBadRequest)
+		return
+	}
+
 	// Candidate timestamp, not a decision: Upsert keeps the stored one unless
 	// reading progress actually moved. Any client value is ignored.
 	b.UpdatedAt = time.Now().UnixMilli()

backend/latest.go

@@ -148,6 +148,13 @@ func (p *latestPoller) checkOne(ctx context.Context, b Bookmark) {
 
 	// Re-read: the row may have been updated or deleted while the fetch was in
 	// flight, and writing b back wholesale would undo that.
+	//
+	// ponytail: non-transactional read-modify-write, wrap Get+Upsert in a tx if
+	// this ever runs for more than one user. A client PUT that commits between
+	// these two statements is lost to the stale re-read — reverting read
+	// progress or a status change, and moving updated_at because the stored
+	// value now differs. Accepted for a single-user deployment: the window is
+	// milliseconds and the loser is one poll cycle.
 	cur, found, err := p.store.Get(b.Key)
 	if err != nil {
 		log.Printf("latest poll %q: reread: %v", b.Key, err)

backend/main_test.go

@@ -1,6 +1,11 @@
 package main
 
 import (
+	"encoding/json"
+	"fmt"
+	"net/http"
+	"net/http/httptest"
+	"strings"
 	"testing"
 	"time"
 )
@@ -113,3 +118,81 @@ func TestLoadLatestPollClampsAndFallsBack(t *testing.T) {
 		})
 	}
 }
+
+func TestPutStatusValidation(t *testing.T) {
+	cases := []struct {
+		name   string
+		status string
+		want   int
+	}{
+		{"empty is no opinion", "", http.StatusOK},
+		{"reading", "reading", http.StatusOK},
+		{"archived", "archived", http.StatusOK},
+		{"finished is web-only", "finished", http.StatusBadRequest},
+		{"garbage", "dropped", http.StatusBadRequest},
+	}
+	for _, tc := range cases {
+		t.Run(tc.name, func(t *testing.T) {
+			srv := newTestServer(t)
+			body := fmt.Sprintf(`{"title":"Solo","status":%q}`, tc.status)
+			req := auth(httptest.NewRequest(http.MethodPut, "/bookmarks/asura:solo",
+				strings.NewReader(body)))
+			rr := httptest.NewRecorder()
+			srv.ServeHTTP(rr, req)
+
+			if rr.Code != tc.want {
+				t.Fatalf("status = %d, want %d (body %s)", rr.Code, tc.want, rr.Body.String())
+			}
+			if tc.want != http.StatusOK {
+				return
+			}
+			var got Bookmark
+			if err := json.Unmarshal(rr.Body.Bytes(), &got); err != nil {
+				t.Fatalf("decode: %v", err)
+			}
+			want := tc.status
+			if want == "" {
+				want = "reading"
+			}
+			if got.Status != want {
+				t.Fatalf("stored status = %q, want %q", got.Status, want)
+			}
+		})
+	}
+}
+
+// A PUT that omits the status field entirely (what a userscript build
+// predating the column sends) is the actual preserve path — the "" case
+// above only exercises the fresh-INSERT default and never touches an
+// existing bucket. This must both keep the archived bucket and still apply
+// the chapter progress carried in the same request.
+func TestPutOmittedStatusPreservesArchivedAndAppliesProgress(t *testing.T) {
+	srv := newTestServer(t)
+
+	seed := httptest.NewRequest(http.MethodPut, "/bookmarks/asura:solo",
+		strings.NewReader(`{"title":"Solo","status":"archived"}`))
+	rr := httptest.NewRecorder()
+	srv.ServeHTTP(rr, auth(seed))
+	if rr.Code != http.StatusOK {
+		t.Fatalf("seed status = %d, want 200 (body %s)", rr.Code, rr.Body.String())
+	}
+
+	req := httptest.NewRequest(http.MethodPut, "/bookmarks/asura:solo",
+		strings.NewReader(`{"title":"Solo","last_chapter":"12","last_chapter_num":12}`))
+	rr = httptest.NewRecorder()
+	srv.ServeHTTP(rr, auth(req))
+	if rr.Code != http.StatusOK {
+		t.Fatalf("status = %d, want 200 (body %s)", rr.Code, rr.Body.String())
+	}
+
+	var got Bookmark
+	if err := json.Unmarshal(rr.Body.Bytes(), &got); err != nil {
+		t.Fatalf("decode: %v", err)
+	}
+	if got.Status != "archived" {
+		t.Fatalf("stored status = %q, want %q", got.Status, "archived")
+	}
+	if got.LastChapterNum != 12 {
+		t.Fatalf("stored last_chapter_num = %v, want 12", got.LastChapterNum)
+	}
+}

backend/static/style.css

@@ -131,10 +131,19 @@ button, .primary, [role="tab"] {
 
 .search:focus-visible { outline: 2px solid var(--accent); }
 
-.tabs { display: flex; gap: 8px; margin-bottom: 16px; }
+/* Five buttons no longer divide a phone's width legibly, so the row scrolls
+   sideways instead of squeezing. The scrollbar is hidden: on mobile there is
+   none anyway, and on desktop it would sit under the tab labels. */
+.tabs {
+  display: flex; gap: 8px; margin-bottom: 16px;
+  overflow-x: auto; scrollbar-width: none; -webkit-overflow-scrolling: touch;
+}
+.tabs::-webkit-scrollbar { display: none; }
 
 .tabs [role="tab"] {
-  flex: 1;
+  flex: 1 0 auto;
+  padding-inline: 14px;
+  white-space: nowrap;
   display: grid;
   place-items: center;
   background: var(--surface);

backend/store.go

@@ -27,6 +27,10 @@ type Bookmark struct {
 	LatestChapter    string   `json:"latest_chapter"`
 	LatestChapterNum *float64 `json:"latest_chapter_num"` // nil until first captured
 	UpdatedAt        int64    `json:"updated_at"`         // unix ms; see Upsert
+	// Status is the lifecycle bucket: reading, archived, or finished.
+	// Archived series stay polled for new chapters; finished ones do not.
+	// Empty on the way in means "no opinion" — see Upsert.
+	Status string `json:"status"`
 }
 
 // HasNewChapter reports whether the site has published past the read point.
@@ -45,6 +49,13 @@ func (b Bookmark) ContinueURL() string {
 	return b.SeriesURL
 }
 
+// Lifecycle buckets. A bookmark is in exactly one; favorite is orthogonal.
+const (
+	statusReading  = "reading"
+	statusArchived = "archived"
+	statusFinished = "finished"
+)
+
 const schema = `
 CREATE TABLE IF NOT EXISTS bookmarks (
   key                TEXT PRIMARY KEY,
@@ -60,6 +71,7 @@ CREATE TABLE IF NOT EXISTS bookmarks (
   latest_chapter     TEXT NOT NULL DEFAULT '',
   latest_chapter_num REAL,
   latest_checked_at  INTEGER NOT NULL DEFAULT 0,
+  status             TEXT NOT NULL DEFAULT 'reading',
   updated_at         INTEGER NOT NULL
 );`
 
@@ -73,11 +85,14 @@ var addedColumns = []struct{ name, ddl string }{
 	// sorts first so a new bookmark is picked up on the next tick with no
 	// special case. Deliberately NOT in bookmarkColumns — see MarkLatestChecked.
 	{"latest_checked_at", `ALTER TABLE bookmarks ADD COLUMN latest_checked_at INTEGER NOT NULL DEFAULT 0`},
+	// Lifecycle bucket. The DEFAULT backfills every pre-existing row as
+	// 'reading', so there is no separate migration step.
+	{"status", `ALTER TABLE bookmarks ADD COLUMN status TEXT NOT NULL DEFAULT 'reading'`},
 }
 
 const bookmarkColumns = `key, site, series_id, title, series_url, cover,
 	last_chapter, last_chapter_num, last_chapter_url,
-	favorite, latest_chapter, latest_chapter_num, updated_at`
+	favorite, latest_chapter, latest_chapter_num, updated_at, status`
 
 // Store is the SQLite-backed bookmark store.
 type Store struct {
@@ -156,13 +171,14 @@ func scanBookmark(scan func(...any) error) (Bookmark, error) {
 		b                                          Bookmark
 		title, seriesURL, cover                    sql.NullString
 		lastChapter, lastChapterURL, latestChapter sql.NullString
+		status                                     sql.NullString
 		lastChapterNum, latestChapterNum           sql.NullFloat64
 		favorite                                   sql.NullInt64
 	)
 	if err := scan(
 		&b.Key, &b.Site, &b.SeriesID, &title, &seriesURL, &cover,
 		&lastChapter, &lastChapterNum, &lastChapterURL,
-		&favorite, &latestChapter, &latestChapterNum, &b.UpdatedAt,
+		&favorite, &latestChapter, &latestChapterNum, &b.UpdatedAt, &status,
 	); err != nil {
 		return Bookmark{}, err
 	}
@@ -177,6 +193,13 @@ func scanBookmark(scan func(...any) error) (Bookmark, error) {
 	if latestChapterNum.Valid {
 		b.LatestChapterNum = &latestChapterNum.Float64
 	}
+	// A NULL, empty, or unrecognised bucket (e.g. a hand-edited row) would
+	// leave the row in no list at all, so anything outside the three known
+	// buckets reads as the default rather than being passed through.
+	b.Status = status.String
+	if b.Status != statusReading && b.Status != statusArchived && b.Status != statusFinished {
+		b.Status = statusReading
+	}
 	return b, nil
 }
 
@@ -242,9 +265,19 @@ func (s *Store) Upsert(b Bookmark) (Bookmark, error) {
 	// IS NOT is SQLite's null-safe comparison. Within DO UPDATE, a bare column
 	// is the stored row and excluded.* is the incoming one; a brand-new key
 	// never reaches this clause, so it keeps the fresh timestamp from VALUES.
+	//
+	// The status column resolves on the VALUES side, not in the conflict
+	// clause: excluded.* is the row *after* these expressions are evaluated,
+	// so a default applied there would look identical to a real 'reading' and
+	// would overwrite an archived row on every PUT from a client that knows
+	// nothing about the column. Resolved once here, an empty incoming status
+	// means "keep what is stored", and only a brand-new row falls through to
+	// the literal default. The subquery runs inside this transaction, so it
+	// sees the row this statement is about to conflict with.
 	if _, err := tx.Exec(`
 		INSERT INTO bookmarks (`+bookmarkColumns+`)
-		VALUES (?, ?, ?, ?, ?, ?, ?, ?, ?, ?, ?, ?, ?)
+		VALUES (?, ?, ?, ?, ?, ?, ?, ?, ?, ?, ?, ?, ?,
+		        COALESCE(NULLIF(?, ''), (SELECT status FROM bookmarks WHERE key = ?), 'reading'))
 		ON CONFLICT(key) DO UPDATE SET
 		  site=excluded.site, series_id=excluded.series_id, title=excluded.title,
 		  series_url=excluded.series_url, cover=excluded.cover,
@@ -253,6 +286,7 @@ func (s *Store) Upsert(b Bookmark) (Bookmark, error) {
 		  favorite=excluded.favorite,
 		  latest_chapter=excluded.latest_chapter,
 		  latest_chapter_num=excluded.latest_chapter_num,
+		  status=excluded.status,
 		  updated_at=CASE
 		    WHEN bookmarks.last_chapter_num IS NOT excluded.last_chapter_num
 		    THEN excluded.updated_at
@@ -260,7 +294,8 @@ func (s *Store) Upsert(b Bookmark) (Bookmark, error) {
 		  END`,
 		b.Key, b.Site, b.SeriesID, b.Title, b.SeriesURL, b.Cover,
 		b.LastChapter, b.LastChapterNum, b.LastChapterURL,
-		b.Favorite, b.LatestChapter, latestNum, b.UpdatedAt); err != nil {
+		b.Favorite, b.LatestChapter, latestNum, b.UpdatedAt,
+		b.Status, b.Key); err != nil {
 		return Bookmark{}, fmt.Errorf("upsert %q: %w", b.Key, err)
 	}
 
@@ -293,10 +328,15 @@ func (s *Store) Delete(key string) error {
 //
 // Bookmarks with no series_url are skipped — there is nothing to fetch, which
 // is the same filter the userscript applies at L452.
+//
+// Finished series are excluded: nothing more is coming, so fetching them only
+// burns requests. Archived ones are deliberately still polled — knowing what a
+// shelved series is up to is the whole reason for archiving instead of deleting.
 func (s *Store) DueForLatestCheck(cutoffMs int64, limit int) ([]Bookmark, error) {
 	rows, err := s.db.Query(`SELECT `+bookmarkColumns+`
 		FROM bookmarks
 		WHERE series_url IS NOT NULL AND series_url <> ''
+		  AND status IS NOT 'finished'
 		  AND latest_checked_at <= ?
 		ORDER BY latest_checked_at ASC
 		LIMIT ?`, cutoffMs, limit)

backend/store_test.go

@@ -722,3 +722,186 @@ func TestPutDoesNotClobberLatestCheckedAt(t *testing.T) {
 		t.Fatalf("latest_checked_at = %d after client PUT, want 777 preserved", got)
 	}
 }
+
+func TestUpsertDefaultsStatusToReading(t *testing.T) {
+	store := newTestStore(t)
+	stored, err := store.Upsert(Bookmark{
+		Key: "asura:solo", Site: "asura", SeriesID: "solo",
+		UpdatedAt: time.Now().UnixMilli(),
+	})
+	if err != nil {
+		t.Fatalf("Upsert: %v", err)
+	}
+	if stored.Status != statusReading {
+		t.Fatalf("Status = %q, want %q", stored.Status, statusReading)
+	}
+}
+
+// An empty status means "no opinion" — a userscript build that predates the
+// column must not reset the bucket on every progress PUT.
+func TestUpsertEmptyStatusPreservesStored(t *testing.T) {
+	store := newTestStore(t)
+	base := Bookmark{
+		Key: "asura:solo", Site: "asura", SeriesID: "solo",
+		Status: statusArchived, UpdatedAt: time.Now().UnixMilli(),
+	}
+	if _, err := store.Upsert(base); err != nil {
+		t.Fatalf("seed: %v", err)
+	}
+
+	base.Status = ""
+	base.LastChapterNum = 12
+	stored, err := store.Upsert(base)
+	if err != nil {
+		t.Fatalf("Upsert: %v", err)
+	}
+	if stored.Status != statusArchived {
+		t.Fatalf("Status = %q, want it preserved as %q", stored.Status, statusArchived)
+	}
+}
+
+// Mirrors the poller's read-modify-write in latest.go: Get the current row,
+// mutate only the latest-chapter fields, and Upsert the whole struct back.
+// cur.Status comes from Get (never empty — see scanBookmark), so it must
+// round-trip through Upsert unchanged rather than being reset.
+func TestLatestPollRoundTripPreservesArchived(t *testing.T) {
+	store := newTestStore(t)
+	base := Bookmark{
+		Key: "asura:solo", Site: "asura", SeriesID: "solo",
+		Status: statusArchived, UpdatedAt: time.Now().UnixMilli(),
+	}
+	if _, err := store.Upsert(base); err != nil {
+		t.Fatalf("seed: %v", err)
+	}
+
+	cur, found, err := store.Get(base.Key)
+	if err != nil || !found {
+		t.Fatalf("Get: found=%v err=%v", found, err)
+	}
+
+	num := 7.0
+	cur.LatestChapter = "7"
+	cur.LatestChapterNum = &num
+	cur.UpdatedAt = time.Now().UnixMilli()
+
+	stored, err := store.Upsert(cur)
+	if err != nil {
+		t.Fatalf("Upsert: %v", err)
+	}
+	if stored.Status != statusArchived {
+		t.Fatalf("Status = %q, want it preserved as %q", stored.Status, statusArchived)
+	}
+}
+
+func TestUpsertReplacesStatusWhenGiven(t *testing.T) {
+	store := newTestStore(t)
+	base := Bookmark{
+		Key: "asura:solo", Site: "asura", SeriesID: "solo",
+		Status: statusArchived, UpdatedAt: time.Now().UnixMilli(),
+	}
+	if _, err := store.Upsert(base); err != nil {
+		t.Fatalf("seed: %v", err)
+	}
+
+	base.Status = statusReading
+	stored, err := store.Upsert(base)
+	if err != nil {
+		t.Fatalf("Upsert: %v", err)
+	}
+	if stored.Status != statusReading {
+		t.Fatalf("Status = %q, want %q", stored.Status, statusReading)
+	}
+}
+
+// A status change is not reading progress, so the list must not reorder.
+func TestUpsertStatusChangeKeepsUpdatedAt(t *testing.T) {
+	store := newTestStore(t)
+	base := Bookmark{
+		Key: "asura:solo", Site: "asura", SeriesID: "solo",
+		LastChapter: "45", LastChapterNum: 45,
+		UpdatedAt: time.Now().UnixMilli(),
+	}
+	first, err := store.Upsert(base)
+	if err != nil {
+		t.Fatalf("seed: %v", err)
+	}
+
+	base.Status = statusArchived
+	base.UpdatedAt = first.UpdatedAt + 60_000
+	stored, err := store.Upsert(base)
+	if err != nil {
+		t.Fatalf("Upsert: %v", err)
+	}
+	if stored.UpdatedAt != first.UpdatedAt {
+		t.Fatalf("UpdatedAt = %d, want it frozen at %d", stored.UpdatedAt, first.UpdatedAt)
+	}
+}
+
+// A database written before the column existed must gain it, with every
+// pre-existing row landing in the reading bucket.
+func TestMigrationAddsStatusToLegacyDatabase(t *testing.T) {
+	path := filepath.Join(t.TempDir(), "legacy.db")
+	db, err := sql.Open("sqlite", path)
+	if err != nil {
+		t.Fatalf("open: %v", err)
+	}
+	if _, err := db.Exec(`
+		CREATE TABLE bookmarks (
+		  key TEXT PRIMARY KEY, site TEXT NOT NULL, series_id TEXT NOT NULL,
+		  title TEXT, series_url TEXT, cover TEXT,
+		  last_chapter TEXT, last_chapter_num REAL, last_chapter_url TEXT,
+		  updated_at INTEGER NOT NULL);
+		INSERT INTO bookmarks (key, site, series_id, updated_at)
+		VALUES ('asura:old', 'asura', 'old', 1)`); err != nil {
+		t.Fatalf("seed legacy: %v", err)
+	}
+	db.Close()
+
+	store, err := OpenStore(path)
+	if err != nil {
+		t.Fatalf("OpenStore: %v", err)
+	}
+	defer store.Close()
+
+	b, ok, err := store.Get("asura:old")
+	if err != nil || !ok {
+		t.Fatalf("Get: ok=%v err=%v", ok, err)
+	}
+	if b.Status != statusReading {
+		t.Fatalf("Status = %q, want %q", b.Status, statusReading)
+	}
+}
+
+// Archiving is the reason to keep polling — the point is to come back to a
+// series that has moved on. A finished series has nothing left to publish.
+func TestDueForLatestCheckSkipsFinishedKeepsArchived(t *testing.T) {
+	store := newTestStore(t)
+	for _, tc := range []struct{ key, status string }{
+		{"asura:reading", statusReading},
+		{"asura:archived", statusArchived},
+		{"asura:finished", statusFinished},
+	} {
+		if _, err := store.Upsert(Bookmark{
+			Key: tc.key, Site: "asura", SeriesID: tc.key,
+			SeriesURL: "https://asurascans.com/comics/" + tc.key,
+			Status:    tc.status, UpdatedAt: time.Now().UnixMilli(),
+		}); err != nil {
+			t.Fatalf("seed %s: %v", tc.key, err)
+		}
+	}
+
+	due, err := store.DueForLatestCheck(time.Now().UnixMilli(), 10)
+	if err != nil {
+		t.Fatalf("DueForLatestCheck: %v", err)
+	}
+	got := map[string]bool{}
+	for _, b := range due {
+		got[b.Key] = true
+	}
+	if !got["asura:reading"] || !got["asura:archived"] {
+		t.Fatalf("due = %v, want reading and archived present", got)
+	}
+	if got["asura:finished"] {
+		t.Fatal("due included a finished series")
+	}
+}

backend/templates/app.html

@@ -31,6 +31,12 @@
     <a role="tab" href="/?tab=fav" class="{{if eq .Tab "fav"}}active{{end}}"
        hx-get="/ui/list?tab=fav" hx-target="#list" hx-swap="innerHTML"
        hx-push-url="/?tab=fav" hx-on::after-request="setActiveTab(this)">Favourites</a>
+    <a role="tab" href="/?tab=archived" class="{{if eq .Tab "archived"}}active{{end}}"
+       hx-get="/ui/list?tab=archived" hx-target="#list" hx-swap="innerHTML"
+       hx-push-url="/?tab=archived" hx-on::after-request="setActiveTab(this)">Archived</a>
+    <a role="tab" href="/?tab=finished" class="{{if eq .Tab "finished"}}active{{end}}"
+       hx-get="/ui/list?tab=finished" hx-target="#list" hx-swap="innerHTML"
+       hx-push-url="/?tab=finished" hx-on::after-request="setActiveTab(this)">Finished</a>
   </nav>
 
   {{if .Recent}}

backend/templates/card.html

@@ -38,6 +38,48 @@
           <path d="M13.8 7.2l3 3"/>
         </svg>
       </button>
+      {{if eq .Status "finished"}}
+      <button class="icon on" title="Restore to reading" aria-label="Restore to reading"
+              hx-post="/ui/bookmarks/{{.Key}}/status" hx-vals='{"status":"reading"}'
+              hx-target="[id='card-{{.Key}}']" hx-swap="outerHTML"
+              hx-indicator="[id='card-{{.Key}}']" hx-disabled-elt="this">
+        <svg viewBox="0 0 24 24" fill="none" stroke="currentColor" stroke-width="1.7"
+             stroke-linecap="round" stroke-linejoin="round" aria-hidden="true" focusable="false">
+          <path d="M4 12a8 8 0 1 1 2.6 5.9M4 12V7M4 12h5"/>
+        </svg>
+      </button>
+      {{else}}
+      {{if eq .Status "archived"}}
+      <button class="icon on" title="Restore to reading" aria-label="Restore to reading"
+              hx-post="/ui/bookmarks/{{.Key}}/status" hx-vals='{"status":"reading"}'
+              hx-target="[id='card-{{.Key}}']" hx-swap="outerHTML"
+              hx-indicator="[id='card-{{.Key}}']" hx-disabled-elt="this">
+        <svg viewBox="0 0 24 24" fill="none" stroke="currentColor" stroke-width="1.7"
+             stroke-linecap="round" stroke-linejoin="round" aria-hidden="true" focusable="false">
+          <path d="M4 12a8 8 0 1 1 2.6 5.9M4 12V7M4 12h5"/>
+        </svg>
+      </button>
+      {{else}}
+      <button class="icon" title="Archive" aria-label="Archive"
+              hx-post="/ui/bookmarks/{{.Key}}/status" hx-vals='{"status":"archived"}'
+              hx-target="[id='card-{{.Key}}']" hx-swap="outerHTML"
+              hx-indicator="[id='card-{{.Key}}']" hx-disabled-elt="this">
+        <svg viewBox="0 0 24 24" fill="none" stroke="currentColor" stroke-width="1.7"
+             stroke-linecap="round" stroke-linejoin="round" aria-hidden="true" focusable="false">
+          <path d="M3.5 6.4h17v3.2h-17zM5 9.6h14V19H5zM10 13h4"/>
+        </svg>
+      </button>
+      {{end}}
+      <button class="icon" title="Mark finished" aria-label="Mark finished"
+              hx-post="/ui/bookmarks/{{.Key}}/status" hx-vals='{"status":"finished"}'
+              hx-target="[id='card-{{.Key}}']" hx-swap="outerHTML"
+              hx-indicator="[id='card-{{.Key}}']" hx-disabled-elt="this">
+        <svg viewBox="0 0 24 24" fill="none" stroke="currentColor" stroke-width="1.7"
+             stroke-linecap="round" stroke-linejoin="round" aria-hidden="true" focusable="false">
+          <path d="M4.5 12.5l5 5 10-11"/>
+        </svg>
+      </button>
+      {{end}}
       <button class="icon danger" title="Remove" aria-label="Remove"
               onclick="toggleConfirmRow('{{.Key}}')">
         <svg viewBox="0 0 24 24" fill="none" stroke="currentColor" stroke-width="1.7"

backend/templates/list.html

@@ -5,6 +5,10 @@
   <p class="empty">No favourites yet. Star a series to pin it here.</p>
 {{else if eq .Tab "new"}}
   <p class="empty">Nothing new. Every series is caught up to its latest chapter.</p>
+{{else if eq .Tab "archived"}}
+  <p class="empty">Nothing archived. Shelve a series to park it here — it keeps getting checked for new chapters.</p>
+{{else if eq .Tab "finished"}}
+  <p class="empty">Nothing finished yet.</p>
 {{else}}
   <p class="empty">
     Nothing here yet. Bookmarks appear once the userscript records a chapter.

backend/web.go

@@ -79,6 +79,7 @@ func (h *webHandler) register(mux *http.ServeMux) {
 
 	mux.HandleFunc("GET /ui/list", h.requireSession(h.uiList))
 	mux.HandleFunc("POST /ui/bookmarks/{key}/favorite", h.requireSession(h.uiFavorite))
+	mux.HandleFunc("POST /ui/bookmarks/{key}/status", h.requireSession(h.uiStatus))
 	mux.HandleFunc("POST /ui/bookmarks/{key}/chapter", h.requireSession(h.uiChapter))
 	mux.HandleFunc("DELETE /ui/bookmarks/{key}", h.requireSession(h.uiDelete))
 }
@@ -145,38 +146,51 @@ func (h *webHandler) index(w http.ResponseWriter, r *http.Request) {
 	h.render(w, http.StatusOK, "app", view)
 }
 
+// filterBookmarks returns the subset keep reports true for, preserving order.
+// It always returns a non-nil slice so an empty tab renders its empty state.
+func filterBookmarks(all []Bookmark, keep func(Bookmark) bool) []Bookmark {
+	out := []Bookmark{}
+	for _, b := range all {
+		if keep(b) {
+			out = append(out, b)
+		}
+	}
+	return out
+}
+
 // buildListView loads the list once and derives both the tab-filtered items and
-// the recent strip from it. The strip always reflects overall recency, not the
-// active tab, so it is built before filtering.
+// the recent strip from it.
+//
+// Archived and finished series appear in their own tab and nowhere else — not
+// in All, not in Updated, not in Favourites, and not in the recent strip. An
+// archived favourite therefore shows only under Archived: Favourites means
+// "favourites I am currently reading".
 func (h *webHandler) buildListView(tab string) (listView, error) {
 	all, err := h.store.List() // already ordered updated_at DESC
 	if err != nil {
 		return listView{}, err
 	}
+	reading := filterBookmarks(all, func(b Bookmark) bool { return b.Status == statusReading })
 
-	recent := all
+	// The strip reflects overall reading recency, not the active tab.
+	recent := reading
 	if len(recent) > recentCount {
 		recent = recent[:recentCount]
 	}
 
-	items := all
+	var items []Bookmark
 	switch tab {
 	case "fav":
-		items = []Bookmark{}
-		for _, b := range all {
-			if b.Favorite {
-				items = append(items, b)
-			}
-		}
+		items = filterBookmarks(reading, func(b Bookmark) bool { return b.Favorite })
 	case "new":
-		items = []Bookmark{}
-		for _, b := range all {
-			if b.HasNewChapter() {
-				items = append(items, b)
-			}
-		}
+		items = filterBookmarks(reading, func(b Bookmark) bool { return b.HasNewChapter() })
+	case "archived":
+		items = filterBookmarks(all, func(b Bookmark) bool { return b.Status == statusArchived })
+	case "finished":
+		items = filterBookmarks(all, func(b Bookmark) bool { return b.Status == statusFinished })
 	default:
 		tab = "all"
+		items = reading
 	}
 	return listView{Tab: tab, Recent: recent, Items: items}, nil
 }
@@ -248,6 +262,13 @@ func (h *webHandler) loadForMutation(w http.ResponseWriter, r *http.Request) (Bo
 // saveAndRenderCard upserts and renders the row as stored. Upsert decides
 // whether updated_at moves, so the argument's timestamp is only a candidate and
 // the response must come from the return value.
+//
+// ponytail: the swapped card stays put even when its new status no longer
+// matches the active tab, add an hx-swap-oob list refresh if that reads as a
+// bug rather than as feedback. Archiving from the All tab leaves the card on
+// screen until the next list load. The alternative costs a full list round
+// trip on every toggle, and the card visibly showing its new state is the
+// feedback the user needs.
 func (h *webHandler) saveAndRenderCard(w http.ResponseWriter, b Bookmark) {
 	stored, err := h.store.Upsert(b)
 	if err != nil {
@@ -270,6 +291,32 @@ func (h *webHandler) uiFavorite(w http.ResponseWriter, r *http.Request) {
 	h.saveAndRenderCard(w, b)
 }
 
+// uiStatus moves a bookmark between lifecycle buckets. This is the only place
+// a series can be marked finished — the JSON API refuses that value, so the
+// userscript cannot set it even by accident.
+//
+// last_chapter_num is untouched, so Upsert keeps the stored updated_at and the
+// list does not reorder.
+func (h *webHandler) uiStatus(w http.ResponseWriter, r *http.Request) {
+	b, ok := h.loadForMutation(w, r)
+	if !ok {
+		return
+	}
+	if err := r.ParseForm(); err != nil {
+		http.Error(w, "invalid form", http.StatusBadRequest)
+		return
+	}
+	switch s := r.PostFormValue("status"); s {
+	case statusReading, statusArchived, statusFinished:
+		b.Status = s
+	default:
+		http.Error(w, "invalid status", http.StatusBadRequest)
+		return
+	}
+	b.UpdatedAt = time.Now().UnixMilli()
+	h.saveAndRenderCard(w, b)
+}
+
 // uiChapter forces the read chapter to a value the user typed.
 //
 // Writing the number also clears last_chapter_url: that URL points at the

backend/web_test.go

@@ -308,8 +308,8 @@ func TestCardHxTargetIsValidSelectorForColonKey(t *testing.T) {
 	body := rr.Body.String()
 
 	want := `hx-target="[id='card-asura:solo']"`
-	if strings.Count(body, want) != 3 {
-		t.Fatalf("body has %d occurrences of %s, want 3 (favorite button, delete button, chapter form)",
+	if strings.Count(body, want) != 5 {
+		t.Fatalf("body has %d occurrences of %s, want 5 (favorite, archive, finish, delete buttons, chapter form)",
 			strings.Count(body, want), want)
 	}
 	if strings.Contains(body, `hx-target="#card-asura:solo"`) {
@@ -513,3 +513,220 @@ func TestUIListNewTab(t *testing.T) {
 		t.Fatal("new tab included a series already caught up")
 	}
 }
+
+// seedStatusRows puts one series in each bucket, the archived one also
+// favourited and with a new chapter out, so a leak into any reading-bucket tab
+// shows up as a failure rather than passing by accident.
+func seedStatusRows(t *testing.T, store *Store) {
+	t.Helper()
+	// floatPtr already exists in store_test.go — same package, reuse it.
+	rows := []Bookmark{
+		{Key: "asura:reading", Site: "asura", SeriesID: "reading", Title: "ReadingOne",
+			Status: statusReading, LastChapterNum: 10, Favorite: true,
+			LatestChapter: "11", LatestChapterNum: floatPtr(11)},
+		{Key: "asura:archived", Site: "asura", SeriesID: "archived", Title: "ArchivedOne",
+			Status: statusArchived, LastChapterNum: 5, Favorite: true,
+			LatestChapter: "99", LatestChapterNum: floatPtr(99)},
+		{Key: "asura:finished", Site: "asura", SeriesID: "finished", Title: "FinishedOne",
+			Status: statusFinished, LastChapterNum: 200, Favorite: true},
+	}
+	for _, b := range rows {
+		b.UpdatedAt = time.Now().UnixMilli()
+		if _, err := store.Upsert(b); err != nil {
+			t.Fatalf("seed %s: %v", b.Key, err)
+		}
+	}
+}
+
+func TestTabsShowOnlyTheirBucket(t *testing.T) {
+	cfg := webConfig()
+	srv, store := newWebTestServer(t, cfg)
+	seedStatusRows(t, store)
+
+	cases := []struct {
+		tab            string
+		want, dontWant []string
+	}{
+		{"all", []string{"ReadingOne"}, []string{"ArchivedOne", "FinishedOne"}},
+		{"new", []string{"ReadingOne"}, []string{"ArchivedOne", "FinishedOne"}},
+		{"fav", []string{"ReadingOne"}, []string{"ArchivedOne", "FinishedOne"}},
+		{"archived", []string{"ArchivedOne"}, []string{"ReadingOne", "FinishedOne"}},
+		{"finished", []string{"FinishedOne"}, []string{"ReadingOne", "ArchivedOne"}},
+	}
+	for _, tc := range cases {
+		t.Run(tc.tab, func(t *testing.T) {
+			req := httptest.NewRequest(http.MethodGet, "/ui/list?tab="+tc.tab, nil)
+			req.AddCookie(sessionCookie(t, cfg))
+			rr := httptest.NewRecorder()
+			srv.ServeHTTP(rr, req)
+
+			if rr.Code != http.StatusOK {
+				t.Fatalf("status = %d, want 200", rr.Code)
+			}
+			body := rr.Body.String()
+			for _, w := range tc.want {
+				if !strings.Contains(body, w) {
+					t.Fatalf("tab %s missing %s", tc.tab, w)
+				}
+			}
+			for _, d := range tc.dontWant {
+				if strings.Contains(body, d) {
+					t.Fatalf("tab %s leaked %s", tc.tab, d)
+				}
+			}
+		})
+	}
+}
+
+func TestRecentStripExcludesArchivedAndFinished(t *testing.T) {
+	cfg := webConfig()
+	srv, store := newWebTestServer(t, cfg)
+	seedStatusRows(t, store)
+
+	req := httptest.NewRequest(http.MethodGet, "/?tab=archived", nil)
+	req.AddCookie(sessionCookie(t, cfg))
+	rr := httptest.NewRecorder()
+	srv.ServeHTTP(rr, req)
+
+	strip := rr.Body.String()
+	if i := strings.Index(strip, `id="list"`); i >= 0 {
+		strip = strip[:i] // everything above the list is the recent section
+	}
+	if strings.Contains(strip, "ArchivedOne") || strings.Contains(strip, "FinishedOne") {
+		t.Fatal("recent strip included an archived or finished series")
+	}
+	if !strings.Contains(strip, "ReadingOne") {
+		t.Fatal("recent strip dropped the reading series")
+	}
+}
+
+func postStatus(t *testing.T, srv http.Handler, cfg Config, key, status string) *httptest.ResponseRecorder {
+	t.Helper()
+	form := url.Values{"status": {status}}
+	req := httptest.NewRequest(http.MethodPost, "/ui/bookmarks/"+key+"/status",
+		strings.NewReader(form.Encode()))
+	req.Header.Set("Content-Type", "application/x-www-form-urlencoded")
+	req.AddCookie(sessionCookie(t, cfg))
+	rr := httptest.NewRecorder()
+	srv.ServeHTTP(rr, req)
+	return rr
+}
+
+func TestUIStatusSetsBucket(t *testing.T) {
+	cfg := webConfig()
+	srv, store := newWebTestServer(t, cfg)
+	seedStatusRows(t, store)
+
+	for _, want := range []string{statusArchived, statusFinished, statusReading} {
+		if rr := postStatus(t, srv, cfg, "asura:reading", want); rr.Code != http.StatusOK {
+			t.Fatalf("set %s: status = %d, body %s", want, rr.Code, rr.Body.String())
+		}
+		b, ok, err := store.Get("asura:reading")
+		if err != nil || !ok {
+			t.Fatalf("Get: ok=%v err=%v", ok, err)
+		}
+		if b.Status != want {
+			t.Fatalf("stored status = %q, want %q", b.Status, want)
+		}
+	}
+}
+
+func TestUIStatusRejectsUnknownValue(t *testing.T) {
+	cfg := webConfig()
+	srv, store := newWebTestServer(t, cfg)
+	seedStatusRows(t, store)
+
+	if rr := postStatus(t, srv, cfg, "asura:reading", "dropped"); rr.Code != http.StatusBadRequest {
+		t.Fatalf("status = %d, want 400", rr.Code)
+	}
+	b, _, _ := store.Get("asura:reading")
+	if b.Status != statusReading {
+		t.Fatalf("stored status = %q, want it untouched", b.Status)
+	}
+}
+
+func TestUIStatusRequiresSession(t *testing.T) {
+	srv, store := newWebTestServer(t, webConfig())
+	seedStatusRows(t, store)
+
+	req := httptest.NewRequest(http.MethodPost, "/ui/bookmarks/asura:reading/status",
+		strings.NewReader("status=archived"))
+	req.Header.Set("Content-Type", "application/x-www-form-urlencoded")
+	rr := httptest.NewRecorder()
+	srv.ServeHTTP(rr, req)
+
+	if rr.Code != http.StatusUnauthorized {
+		t.Fatalf("status = %d, want 401", rr.Code)
+	}
+}
+
+func TestUIStatusDoesNotReorderList(t *testing.T) {
+	cfg := webConfig()
+	srv, store := newWebTestServer(t, cfg)
+	seedStatusRows(t, store)
+
+	before, _, _ := store.Get("asura:reading")
+	time.Sleep(2 * time.Millisecond)
+	if rr := postStatus(t, srv, cfg, "asura:reading", statusArchived); rr.Code != http.StatusOK {
+		t.Fatalf("status = %d", rr.Code)
+	}
+	after, _, _ := store.Get("asura:reading")
+	if after.UpdatedAt != before.UpdatedAt {
+		t.Fatalf("UpdatedAt moved %d -> %d", before.UpdatedAt, after.UpdatedAt)
+	}
+}
+
+func TestCardShowsStatusControls(t *testing.T) {
+	cfg := webConfig()
+	srv, store := newWebTestServer(t, cfg)
+	seedStatusRows(t, store)
+
+	cases := []struct {
+		tab            string
+		want, dontWant []string
+	}{
+		// A series being read can be shelved or completed, not restored.
+		{"all", []string{`hx-vals='{"status":"archived"}'`, `hx-vals='{"status":"finished"}'`}, nil},
+		// An archived one can come back or be completed.
+		{"archived", []string{`hx-vals='{"status":"reading"}'`, `hx-vals='{"status":"finished"}'`}, nil},
+		// A finished one can only come back.
+		{"finished", []string{`hx-vals='{"status":"reading"}'`}, []string{`hx-vals='{"status":"finished"}'`}},
+	}
+	for _, tc := range cases {
+		t.Run(tc.tab, func(t *testing.T) {
+			req := httptest.NewRequest(http.MethodGet, "/ui/list?tab="+tc.tab, nil)
+			req.AddCookie(sessionCookie(t, cfg))
+			rr := httptest.NewRecorder()
+			srv.ServeHTTP(rr, req)
+
+			body := rr.Body.String()
+			for _, w := range tc.want {
+				if !strings.Contains(body, w) {
+					t.Fatalf("tab %s missing control %s", tc.tab, w)
+				}
+			}
+			for _, d := range tc.dontWant {
+				if strings.Contains(body, d) {
+					t.Fatalf("tab %s offered %s", tc.tab, d)
+				}
+			}
+		})
+	}
+}
+
+func TestAppRendersNewTabs(t *testing.T) {
+	cfg := webConfig()
+	srv, store := newWebTestServer(t, cfg)
+	seedStatusRows(t, store)
+
+	req := httptest.NewRequest(http.MethodGet, "/", nil)
+	req.AddCookie(sessionCookie(t, cfg))
+	rr := httptest.NewRecorder()
+	srv.ServeHTTP(rr, req)
+
+	for _, want := range []string{`href="/?tab=archived"`, `href="/?tab=finished"`} {
+		if !strings.Contains(rr.Body.String(), want) {
+			t.Fatalf("app page missing %s", want)
+		}
+	}
+}