release: 0.109.0 (improvement/e43-identity-diet)

chore(release): 0.109.0 — one account per browser, settings that follow the account, a tab that names its account
fix(frontend): a page opened at a //path boots, and the account-switch copy reads right in Hungarian
2026-10-21 03:05:48 +02:00 · 2026-10-21 02:55:46 +02:00 · 2026-10-21 02:51:48 +02:00 · 2026-10-21 02:05:33 +02:00 · 2026-10-21 01:05:12 +02:00 · 2026-10-20 23:34:43 +02:00
151 changed files with 4385 additions and 1570 deletions
@@ -7,6 +7,32 @@ edit the release notes and re-run the generator.

 Upgrading, pinning a version and rolling back: [docs/self-hosting.md](docs/self-hosting.md#upgrading).

+## 0.109.0 — 2026-10-21
+
+### New
+
+- Accounts: a browser is signed in to one account at a time. The account menu's "Sign in as someone else" replaces it; the account list and switcher are gone. After this update everyone signs in once more.
+- Settings follow you: the Home start page, the channel manager and Plex playlist layouts, the discovery sorts and a dismissed onboarding or release banner are saved to your account, on every device. Performance mode stays per device.
+- Development: every key the browser stores declares its kind, held by a contract test; one Google connect route serves the sign-in and the YouTube access grants.
+
+### Fixed
+
+- Accounts: a tab still showing another account than the one signed in now reloads as that one and says why; a failed "Sign in as someone else" says what happened.
+- A link whose path starts with // no longer opens the error screen.
+- A member following a link to an admin page lands on the feed, with the rail and the address bar in step.
+- Google: an unknown access request is refused instead of quietly asking for read access, and a YouTube grant on an instance without Google says so.
+
+## 0.108.1 — 2026-10-20
+
+### New
+
+- Development: the backend is split into four units (downloads, reader, catalog, Plex), each owning its tables. A contract test holds what each one reads and writes, and that every query names whose rows it asks for.
+
+### Fixed
+
+- Watch later: saving the first two videos at once no longer makes two lists. An existing second list is merged into the first, its videos and its comfort-shelf pin with it.
+- Playback: the first resume point of a video or a Plex item no longer fails when it is saved together with the watched mark, a whole season marked at once, or a Plex sync.
+
 ## 0.108.0 — 2026-10-18

 ### New
@@ -70,8 +70,9 @@ The domain is four **units**, each owning its tables: Capture (`downloads/`), Re
 sync), Catalog (`catalog/`, the `sync/` ingests) and Plex (`plex/`). A unit's route does no database work: it reads through
 the unit's `queries`, writes through its `service` (which flushes) and commits. A unit's code reads another unit's tables
 only along a declared edge and writes them only through that unit's service; core reaches a unit's tables through its
-`queries`. **Fence-held, with no exemptions** (`backend/tests/test_unit_fences.py`); conventions in
-[`backend/app/README.md`](backend/app/README.md).
+`queries`. **Fence-held, with no exemptions** (`backend/tests/test_unit_fences.py`) — as far as an AST sees: what it
+cannot (an ORM write on another unit's loaded row, SQL in `text()`, a commit by `begin()`) is review's to catch, listed in
+[`backend/app/README.md`](backend/app/README.md) with the conventions.
 Frontend: `main.tsx` provider tree → `App.tsx` shell (a shared `/watch` link boots from its own
 entry, `watch.html` → `watchMain.tsx`, held app-free by a contract test) → `moduleRegistry` route-chunks → module
 components → the **platform layer `components/ui/`** (domain-free primitives: the popover engine and
@@ -80,44 +81,44 @@ boundary is one-way and **lint-enforced**, not just described: a platform compon
 feature component or a domain API slice, and one layer down `lib/` may not import from `components/` at
 all, not even a type (`no-restricted-imports` in `eslint.config.js`, with its own test). A new generic
 component goes in `ui/`; see [`frontend/src/components/ui/README.md`](frontend/src/components/ui/README.md).
-Data via react-query (`qk` factory), per-account `localStorage`, and server prefs; API through
-`lib/api/*` slices over `core.req<T>`.
+Data via react-query (`qk` factory), the signed-in account's `localStorage` (`storage.ts`), and server
+prefs; API through `lib/api/*` slices over `core.req<T>`.

 **Not wanted** (reviewed and rejected as over-engineering for this size): microservices, event bus,
 CQRS, a repository/DI framework, a router library, a generic CRUD-router factory or base serializer.

 ## Canonical patterns — when you have a choice, pick THIS

-| Area                                 | Use                                                                                                                                                                                               | Not                                                                                                                                                     |
-| ------------------------------------ | ------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------- | ------------------------------------------------------------------------------------------------------------------------------------------------------- |
-| ORM                                  | SQLAlchemy 2.0 `select()`                                                                                                                                                                         | `db.query(...)`                                                                                                                                         |
-| Request body                         | a Pydantic model                                                                                                                                                                                  | `payload: dict`                                                                                                                                         |
-| Request SIZE                         | nothing — `bodylimit` caps every route (SEC-20); need more? add a ceiling to `ROUTE_MAX_BODY_BYTES`                                                                                               | `capped_body` on a route that does not read raw bytes itself                                                                                            |
-| Ownership check                      | `userscope.owned_by` (a filter) / `owned_row` (a row by id): a unit's `queries.own_x` returns None and its route answers `not_found`; a core route `owned_or_404`                                 | an inline `Model.user_id == …` (fence-held, in every module) or a `db.get` and a compare                                                                |
-| Admin gate import                    | `app.auth`                                                                                                                                                                                        | the `app.routes.admin` re-export                                                                                                                        |
-| Upstream failure status              | `422` with JSON detail                                                                                                                                                                            | `502` (reserve 502 for raw byte-proxies)                                                                                                                |
-| User-facing error                    | structured `detail` (`{code, reason}`)                                                                                                                                                            | a bare English string under a bilingual UI                                                                                                              |
-| Mutation response                    | `{"ok": true, …}` (`{"deleted": id}`), test-held                                                                                                                                                  | a bare `{"unsubscribed"}`/`{"cleared": n}`, no `ok`                                                                                                     |
-| Transaction commit                   | the handler commits; a unit's `service` and every helper `flush` (fence-held)                                                                                                                     | a helper that commits (only `quota`, `jobruns` and the jobs the fence's `UNIT_COMMITS` names own their session)                                         |
-| Admin knob storage                   | `sysconfig` (values)                                                                                                                                                                              | env / `app_state` / a table of its own (the job intervals had one)                                                                                      |
-| Job status word                      | the `downloads/status.py` StrEnum                                                                                                                                                                 | loose `"canceled"`/`"cancelled"` literals                                                                                                               |
-| Current time                         | inline `datetime.now(timezone.utc)`                                                                                                                                                               | new `now_utc`/`_now` helpers                                                                                                                            |
-| FE per-account state                 | `usePref`                                                                                                                                                                                         | raw `readAccount`/`writeAccount` or bespoke hooks                                                                                                       |
-| FE keyboard shortcut                 | a row in `lib/keymap.ts` + `dispatch()`                                                                                                                                                           | a private `keydown` listener with its own key literals                                                                                                  |
-| FE list/grid keys                    | `NavGrid`/`keyboardNav` + `navItemAttrs` + `data-nav-open`                                                                                                                                        | a private arrow-key / roving-tabindex handler                                                                                                           |
-| FE view push (history)               | `rememberNavPosition()` right before `history.pushState`                                                                                                                                          | a push that forgets: Back strands focus on `<body>`                                                                                                     |
-| FE static label / button-like link   | `Badge` (flat, no edge) · `LinkButton` / `IconLink`                                                                                                                                               | a glass pill that reads as a chip; an `<a>` hand-painted like a button                                                                                  |
-| FE loading/empty/error               | `QueryState` / `StateMessage`                                                                                                                                                                     | ad-hoc conditionals                                                                                                                                     |
-| FE failed write                      | `meta.failTitle`, else `notifyActionError(err, title)`; a YouTube write action (subscribe, sync, …): `notifyYouTubeActionError` — a 403 offers Connect                                            | a bare `catch {}` or `notify()` with no error attached                                                                                                  |
-| FE floating or full-screen layer     | `ui/Overlay` (portals to `<body>`) or the popover engine (`AnchoredPopover` portals itself); stacking by the z-index scale (`z-menu` … `z-popover`, docs/DESIGN.md "Layers")                      | one rendered in place inside a virtualized row or a masked/transformed scroller — clipped or painted over; a magic `z-[n]`                              |
-| FE player on a native media element  | `lib/mediaSession` (`useMediaSession`) + `lib/mediaKeys` + `lib/resume` (`useResume`), the element held as STATE                                                                                  | a private `navigator.mediaSession` / `keydown` / position-saving effect per player                                                                      |
-| FE diagnostics                       | `lib/log.ts` (`log.info("event.code", msg, attrs)`); a filter change → `lib/filterLog.ts`, a player → `lib/mediaLog.ts`                                                                           | `console.*` (lint-enforced); a value only a higher level may carry, logged unguarded                                                                    |
-| FE state that MIRRORS a prop/query   | derive it, or hold a draft (`draft ?? server`)                                                                                                                                                    | a copy an effect re-syncs — that paints the stale value first                                                                                           |
-| FE state reset on a key change       | adjust it DURING render against a remembered key                                                                                                                                                  | `useEffect(() => setX(init), [key])`                                                                                                                    |
-| FE effect that must not re-subscribe | `useEffectEvent` (React 19.2+); a handler or timer: `useLatest` / `useStableCallback`                                                                                                             | a render-phase `ref.current = x` — the React Compiler skips the component                                                                               |
-| FE hook suppression                  | an inline `eslint-disable-next-line` NAMING the invariant                                                                                                                                         | `eslint-suppressions.json` (deliberately absent — E27 S2)                                                                                               |
-| FE popover / scroll-fade ref         | destructure it: `const { setReference } = pop.refs`, then `ref={setReference}`                                                                                                                    | `ref={pop.refs.setReference}` / `ref={fade.ref}` — the compiler then treats the whole object as a ref and skips                                         |
-| FE async handler with cleanup        | `try { … } catch { … }`, then the cleanup after it (steps that `return` early go in their own function); a conditional, loop or `?.` the `try` needs goes before it or into a module-level helper | `try/finally`, or a `?:` / `??` / loop / `?.` inside `try` — the React Compiler skips the whole component (and `react-hooks/todo` misses most of these) |
+| Area                                 | Use                                                                                                                                                                                                                                           | Not                                                                                                                                                                                         |
+| ------------------------------------ | --------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------- | ------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------- |
+| ORM                                  | SQLAlchemy 2.0 `select()`                                                                                                                                                                                                                     | `db.query(...)`                                                                                                                                                                             |
+| Request body                         | a Pydantic model                                                                                                                                                                                                                              | `payload: dict`                                                                                                                                                                             |
+| Request SIZE                         | nothing — `bodylimit` caps every route (SEC-20); need more? add a ceiling to `ROUTE_MAX_BODY_BYTES`                                                                                                                                           | `capped_body` on a route that does not read raw bytes itself                                                                                                                                |
+| Ownership check                      | `userscope.owned_by` (a filter) / `owned_row` (a row by id): a unit's `queries.own_x` returns None and its route answers `not_found`; a core route `owned_or_404`. A composite rule (own or shared, built-in or own) says so in its `queries` | an inline owner filter — `Model.user_id == …`, an alias's, `.in_()`, `filter_by(user_id=…)` (fence-held, every module) — or a `db.get` and a compare where `owned_row` fits                 |
+| Admin gate import                    | `app.auth`                                                                                                                                                                                                                                    | the `app.routes.admin` re-export                                                                                                                                                            |
+| Upstream failure status              | `422` with JSON detail                                                                                                                                                                                                                        | `502` (reserve 502 for raw byte-proxies)                                                                                                                                                    |
+| User-facing error                    | structured `detail` (`{code, reason}`)                                                                                                                                                                                                        | a bare English string under a bilingual UI                                                                                                                                                  |
+| Mutation response                    | `{"ok": true, …}` (`{"deleted": id}`), test-held                                                                                                                                                                                              | a bare `{"unsubscribed"}`/`{"cleared": n}`, no `ok`                                                                                                                                         |
+| Transaction commit                   | the handler commits; a unit's `service` and every helper `flush` (fence-held, every module)                                                                                                                                                   | a helper that commits — only a listed one (what owns its session, or a core step that must land whatever its caller does next), each with its reason: the fence's `UNIT_COMMITS` / `CORE_COMMITS`, a route's `COMMITTING_HELPERS` (`test_route_conventions`) |
+| Admin knob storage                   | `sysconfig` (values)                                                                                                                                                                                                                          | env / `app_state` / a table of its own (the job intervals had one)                                                                                                                          |
+| Job status word                      | the `downloads/status.py` StrEnum                                                                                                                                                                                                             | loose `"canceled"`/`"cancelled"` literals                                                                                                                                                   |
+| Current time                         | inline `datetime.now(timezone.utc)`                                                                                                                                                                                                           | new `now_utc`/`_now` helpers                                                                                                                                                                |
+| FE browser storage                   | a key in `storage.ts` with its kind (contract-held): a `pref` through `usePref` (the server's, follows the account); `device` / `view` / `cache` this browser's; `public` kept for the /watch page                                            | a `"siftlode.x"` literal; a preference kept only in `localStorage` (it stays on one device)                                                                                                 |
+| FE keyboard shortcut                 | a row in `lib/keymap.ts` + `dispatch()`                                                                                                                                                                                                       | a private `keydown` listener with its own key literals                                                                                                                                      |
+| FE list/grid keys                    | `NavGrid`/`keyboardNav` + `navItemAttrs` + `data-nav-open`                                                                                                                                                                                    | a private arrow-key / roving-tabindex handler                                                                                                                                               |
+| FE view push (history)               | `rememberNavPosition()` right before `history.pushState`                                                                                                                                                                                      | a push that forgets: Back strands focus on `<body>`                                                                                                                                         |
+| FE static label / button-like link   | `Badge` (flat, no edge) · `LinkButton` / `IconLink`                                                                                                                                                                                           | a glass pill that reads as a chip; an `<a>` hand-painted like a button                                                                                                                      |
+| FE loading/empty/error               | `QueryState` / `StateMessage`                                                                                                                                                                                                                 | ad-hoc conditionals                                                                                                                                                                         |
+| FE failed write                      | `meta.failTitle`, else `notifyActionError(err, title)`; a YouTube write action (subscribe, sync, …): `notifyYouTubeActionError` — a 403 offers Connect                                                                                        | a bare `catch {}` or `notify()` with no error attached                                                                                                                                      |
+| FE floating or full-screen layer     | `ui/Overlay` (portals to `<body>`) or the popover engine (`AnchoredPopover` portals itself); stacking by the z-index scale (`z-menu` … `z-popover`, docs/DESIGN.md "Layers")                                                                  | one rendered in place inside a virtualized row or a masked/transformed scroller — clipped or painted over; a magic `z-[n]`                                                                  |
+| FE player on a native media element  | `lib/mediaSession` (`useMediaSession`) + `lib/mediaKeys` + `lib/resume` (`useResume`), the element held as STATE                                                                                                                              | a private `navigator.mediaSession` / `keydown` / position-saving effect per player                                                                                                          |
+| FE diagnostics                       | `lib/log.ts` (`log.info("event.code", msg, attrs)`); a filter change → `lib/filterLog.ts`, a player → `lib/mediaLog.ts`                                                                                                                       | `console.*` (lint-enforced); a value only a higher level may carry, logged unguarded                                                                                                        |
+| FE state that MIRRORS a prop/query   | derive it, or hold a draft (`draft ?? server`)                                                                                                                                                                                                | a copy an effect re-syncs — that paints the stale value first                                                                                                                               |
+| FE state reset on a key change       | adjust it DURING render against a remembered key                                                                                                                                                                                              | `useEffect(() => setX(init), [key])`                                                                                                                                                        |
+| FE effect that must not re-subscribe | `useEffectEvent` (React 19.2+); a handler or timer: `useLatest` / `useStableCallback`                                                                                                                                                         | a render-phase `ref.current = x` — the React Compiler skips the component                                                                                                                   |
+| FE hook suppression                  | an inline `eslint-disable-next-line` NAMING the invariant                                                                                                                                                                                     | `eslint-suppressions.json` (deliberately absent — E27 S2)                                                                                                                                   |
+| FE popover / scroll-fade ref         | destructure it: `const { setReference } = pop.refs`, then `ref={setReference}`                                                                                                                                                                | `ref={pop.refs.setReference}` / `ref={fade.ref}` — the compiler then treats the whole object as a ref and skips                                                                             |
+| FE async handler with cleanup        | `try { … } catch { … }`, then the cleanup after it (steps that `return` early go in their own function); a conditional, loop or `?.` the `try` needs goes before it or into a module-level helper                                             | `try/finally`, or a `?:` / `??` / loop / `?.` inside `try` — the React Compiler skips the whole component (and `react-hooks/todo` misses most of these)                                     |

 Not repeated here, because a lint rule refuses the wrong one and its message names the right one:
 buttons (`ui/Button`, `IconButton`, `Pressable`, `Chip`, `DragHandle`), text fields and native
@@ -1 +1 @@
-0.108.0
+0.109.0
@@ -0,0 +1,70 @@
+"""One Watch later list per user, held by the database
+
+Revision ID: 0112_one_watch_later
+Revises: 0111_audit_request_id
+Create Date: 2026-10-20
+
+Watch later was made on a user's first save by a read then an insert, so two first saves at once
+each made a list, and the reader then picked between them at random. A partial unique index holds
+"one per user" now. An instance that already has a second list keeps the oldest: the others' videos
+move into it (after its own, in their order; one it already holds is not added twice), a Home pin
+of one of them moves to it (unless it is pinned itself; of several, the first on the shelf), then
+they go — a pin left on them goes with them, as the kept list already holds one.
+"""
+
+import sqlalchemy as sa
+from alembic import op
+
+revision: str = "0112_one_watch_later"
+down_revision: str | None = "0111_audit_request_id"
+branch_labels: str | None = None
+depends_on: str | None = None
+
+_EXTRA = """
+    SELECT p.id, k.keep_id
+    FROM playlists p
+    JOIN (
+        SELECT user_id, min(id) AS keep_id FROM playlists WHERE kind = 'watch_later' GROUP BY user_id
+    ) k ON k.user_id = p.user_id
+    WHERE p.kind = 'watch_later' AND p.id <> k.keep_id
+"""
+
+
+def upgrade() -> None:
+    op.execute(
+        f"""
+        INSERT INTO playlist_items (playlist_id, video_id, position, added_at)
+        SELECT e.keep_id, i.video_id,
+               (SELECT coalesce(max(position), -1) FROM playlist_items WHERE playlist_id = e.keep_id)
+               + row_number() OVER (PARTITION BY e.keep_id ORDER BY i.playlist_id, i.position, i.id),
+               i.added_at
+        FROM playlist_items i
+        JOIN ({_EXTRA}) e ON e.id = i.playlist_id
+        ON CONFLICT ON CONSTRAINT uq_playlist_video DO NOTHING
+        """
+    )
+    op.execute(
+        f"""
+        UPDATE comfort_pins c SET playlist_id = m.keep_id
+        FROM (
+            SELECT DISTINCT ON (e.keep_id) p.id AS pin_id, e.keep_id
+            FROM comfort_pins p
+            JOIN ({_EXTRA}) e ON e.id = p.playlist_id
+            WHERE NOT EXISTS (SELECT 1 FROM comfort_pins k WHERE k.playlist_id = e.keep_id)
+            ORDER BY e.keep_id, p.position, p.id
+        ) m
+        WHERE c.id = m.pin_id
+        """
+    )
+    op.execute(f"DELETE FROM playlists WHERE id IN (SELECT id FROM ({_EXTRA}) e)")
+    op.create_index(
+        "uq_playlist_watch_later",
+        "playlists",
+        ["user_id"],
+        unique=True,
+        postgresql_where=sa.text("kind = 'watch_later'"),
+    )
+
+
+def downgrade() -> None:
+    op.drop_index("uq_playlist_watch_later", table_name="playlists")
@@ -56,11 +56,19 @@ The domain is four units, each owning its tables (`models.py` in its package):
 - **Whose a row is, `userscope` says** — in every module, core included: `owned_by(Model.user_id,
  user)` for a filter, `owned_row(db, Model, id, user)` for a row by id (someone else's row is the
  same answer as a missing one). A unit's `queries.own_x` returns None and its route answers
-  `not_found`; a core route calls `owned_or_404`.
+  `not_found`; a core route calls `owned_or_404`. A rule `owned_row` cannot say — a job its owner
+  shared, a built-in profile nobody owns, a folder read under `FOR SHARE` — is written out in the
+  unit's own code, beside the reason.

 All of it is held by `tests/test_unit_fences.py`, with no exemptions: the unit map, each unit's
-packages and edges (`UNIT_PACKAGES`, `READS`) and the jobs that commit (`UNIT_COMMITS`) are listed
-there.
+packages and edges (`UNIT_PACKAGES`, `READS`) and what may commit (`UNIT_COMMITS`, `CORE_COMMITS`)
+are listed there. It reads the code's AST, so it holds the forms written here: an owner column
+compared inline (a model's, an import alias's or an `aliased()` one's, by `==`, `!=`, `.in_()`, or a
+`filter_by(user_id=…)`), a model built or an `insert`/`update`/`delete` (under any import name, of a
+model or its `__table__`) across units, and a `.commit()` call off the lists all fail it. What an AST
+cannot tell is review's to catch: an ORM write on another unit's loaded row (an attribute, a
+`db.delete`), SQL in `text()`, a column or model reached another way (bound to a variable, through
+`__table__.c` or a module path), and a commit made by a `begin()` block or a `commit` handed in.

 ## Transaction ownership

@@ -73,10 +81,12 @@ mutation it records, so an audit entry can never exist without its change (or vi

 - The **operation owner** is the route handler, or a job. A unit's `service` flushes even when it
  makes a whole transition (`service.cancel`): the handler commits it, together with whatever else the
-  request changed. What owns its OWN session commits it: `quota` and `jobruns`' run rows, and a unit's
-  jobs (the worker's `jobdb` writes, the syncs and ingests that commit per page or batch so a later
-  failure loses nothing already stored) — the fence's `UNIT_COMMITS` names each. Everything else
-  flushes.
+  request changed. What owns its OWN session commits it: `quota` and `jobruns`' run rows, the
+  scheduler's jobs, a unit's jobs (the worker's `jobdb` writes, the syncs and ingests that commit per
+  page or batch so a later failure loses nothing already stored), and a few core steps that must land
+  whatever their caller does next (a token's last-used stamp, a refreshed YouTube grant) — the fence's
+  `UNIT_COMMITS` and `CORE_COMMITS` name each with its reason, `test_route_conventions` a route's own
+  committing helpers. Everything else flushes.
 - `SessionLocal` is `expire_on_commit=False`: after a raw `UPDATE` + commit, an ORM object you already
  loaded is **stale**. Read the fresh value from the row (RETURNING, or a re-`SELECT`), not the ORM
  attribute — see `service.decr_ref_count`.
@@ -110,10 +110,9 @@ def record(

 def gc(db: Session, retention_days: int) -> int:
    """Delete audit rows older than `retention_days`. Returns the number removed. A
-    retention_days <= 0 disables pruning (keep forever)."""
+    retention_days <= 0 disables pruning (keep forever). Flushes; the job commits."""
    if retention_days <= 0:
        return 0
    cutoff = datetime.now(timezone.utc) - timedelta(days=retention_days)
    result = db.execute(delete(AuditLog).where(AuditLog.created_at < cutoff))
-    db.commit()
    return result.rowcount or 0
@@ -25,8 +25,6 @@ def operator_contact() -> str | None:
    return next(iter(sorted(settings.admin_email_set)), None)


-MAX_SESSION_ACCOUNTS = 6
-
 # OAuth scope vocabulary — the atoms + the request bundles, kept together as core. has_*_scope check
 # the atoms against a user's stored grant; the *_SCOPES bundles are what the login/callback endpoints
 # request (imported by app/routes/auth.py).
@@ -253,94 +251,63 @@ def purge_user(db: Session, user: User, background: BackgroundTasks) -> None:
    background.add_task(email_mod.send_account_deleted, email, lang, operator_contact())


-ACTIVE_ACCOUNT_HEADER = "x-siftlode-account"
+def session_user_id(conn: HTTPConnection) -> int | None:
+    """The account this browser session signed in as, read from the signed cookie alone (no DB).
+    One account per session: signing in as another replaces it (`routes.auth._establish_session`)."""
+    return conn.session.get("user_id")


-def resolved_user_id(conn: HTTPConnection, *, strict: bool = True) -> tuple[int | None, bool]:
-    """Which account this connection acts as, and whether that's the session's default account.
-    Takes an HTTPConnection rather than a Request: it reads only the session and the headers.
-
-    The signed cookie holds the *wallet* (`account_ids`, every account signed into this browser)
-    plus a default `user_id`. A caller may name its account with the X-Siftlode-Account header (or
-    `?account=`), but ONLY an account in the wallet — so a tab can't impersonate an account that
-    never authenticated here. A connection that names no account acts as the cookie default.
-
-    A connection naming an account NOT in the wallet acts as nobody (`None`) — typically a tab still
-    pinned to an account that was signed out in another tab. It must not fall back to the default:
-    that tab would go on acting as an account it never showed, its player checkpoints and pref saves
-    landing there and its Sign out signing THAT account out. Only the bootstrap /api/me probe passes
-    `strict=False` and gets the default, which is how such a tab learns who it now is (the frontend
-    re-pins it and reloads). An account whose session ended while it stayed in the wallet (deleted,
-    suspended, revoked) leaves the wallet when it is refused (`_session_user`), so a tab pinned to it
-    takes the same way back.
-    """
-    default_id = conn.session.get("user_id")
-    wallet = conn.session.get("account_ids") or []
-    # Header for normal XHR; ?account= for a context that can't set headers (a plain <a> file
-    # download). Both are wallet-gated below.
-    hdr = conn.headers.get(ACTIVE_ACCOUNT_HEADER) or conn.query_params.get("account")
-    if hdr:
-        try:
-            hid = int(hdr)
-        except ValueError:
-            # Under `strict` this becomes a 401 and the tab authenticates as nobody, which reads
-            # in the browser exactly like a session that expired. The value itself stays out —
-            # it came from a request, and `wallet` says which accounts were actually on offer.
-            log.debug(
-                "the active-account header could not be read as an id",
-                extra=logs.event("auth.account_header.invalid", accounts=len(wallet)),
-            )
-            hid = None
-        if hid is not None and hid in wallet:
-            return hid, hid == default_id
-        if strict:
-            return None, False
-    return default_id, True
+def stamp_session(request: Request, user: User) -> None:
+    """Record the account's current `session_epoch` in the signed cookie, so `current_user` can later
+    reject the cookie once the epoch moves (password reset/change, or "log out everywhere")."""
+    request.session["epoch"] = user.session_epoch or 0


-def drop_from_wallet(session: dict, user_id: int) -> None:
-    """Take one account out of this browser's wallet (`account_ids`); the session default is untouched."""
-    session["account_ids"] = [a for a in (session.get("account_ids") or []) if a != user_id]
+# The account a tab was built for, sent with every request it makes (`identityHeaders`, the
+# frontend). The cookie alone says who is asking, and one browser holds one account: a sign-in as
+# someone else in another tab replaces it under every open tab. A tab still showing the previous
+# account would otherwise act as the new one — its player checkpoints, pref saves, Sign out and
+# Delete account landing there — until it noticed. It never selects an account; a mismatch is
+# refused, and the tab reloads as whoever is signed in now.
+EXPECTED_ACCOUNT_HEADER = "x-siftlode-expect-account"


-def _end_account_session(request: Request, user_id: int, is_default: bool) -> None:
-    """An account whose session here is no longer valid. Only nuke the whole session when it is the
-    SESSION DEFAULT; a per-tab account just leaves the wallet (other tabs' default may still be valid).
-    Leaving is what lets a tab pinned to it recover: its next /api/me names an account no longer signed
-    in here and gets the default. Left in, it answered "signed out" — the landing page, with the default
-    still signed in, and no way off it but closing the tab."""
-    if is_default:
-        request.session.clear()
-    else:
-        drop_from_wallet(request.session, user_id)
+def built_for_another_account(user_id: int, expected: str | None) -> bool:
+    """Whether a tab naming `expected` is showing another account than the one signed in. No name is
+    no claim (a caller from before this check, a script): only a mismatch is refused, and the
+    session stays — it is the other tab's sign-in, and it holds. The one comparison for every way a
+    tab names its account: this header on a request, `expect` on a navigation."""
+    if expected is None or expected == str(user_id):
+        return False
+    log.info(
+        "a request from a tab built for another account was refused",
+        extra=logs.event("auth.session.account_changed", user_id=user_id),
+    )
+    return True


 def current_user(request: Request, db: Session = Depends(get_db)) -> User:
-    return _session_user(request, db, *resolved_user_id(request))
+    user = _session_user(request, db, session_user_id(request))
+    if built_for_another_account(user.id, request.headers.get(EXPECTED_ACCOUNT_HEADER)):
+        raise refused(401, "account_changed")
+    return user


-def _session_user(request: Request, db: Session, user_id: int | None, is_default: bool) -> User:
+def _session_user(request: Request, db: Session, user_id: int | None) -> User:
    if not user_id:
        raise refused(401, "not_authenticated")
    user = db.get(User, user_id)
    if user is None or not user.is_active or user.is_suspended:
        # Suspended/deactivated mid-session → end it so the block takes effect at once.
-        _end_account_session(request, user_id, is_default)
+        request.session.clear()
        raise refused(401, "not_authenticated")
-    # SA4: reject a cookie whose recorded epoch is behind the account's current one (revoked by a
-    # password reset/change or "log out everywhere"). A cookie with no recorded epoch (pre-SA4, or an
-    # account signed in before this shipped) is treated as epoch 0 — so the FIRST bump revokes it too.
-    epochs = request.session.get("epochs") or {}
-    if int(epochs.get(str(user_id), 0)) != (user.session_epoch or 0):
-        _end_account_session(request, user_id, is_default)
+    # A cookie whose epoch is behind the account's (revoked by a password reset/change or "log out
+    # everywhere") is refused — and so is one with no epoch at all, which only a build before the
+    # one-account session wrote: its holder signs in once more.
+    epoch = request.session.get("epoch")
+    if epoch is None or int(epoch) != (user.session_epoch or 0):
+        request.session.clear()
        raise refused(401, "not_authenticated")
-    # Always keep the session default in the switchable list — covers sessions created before
-    # multi-session existed (their account_ids was never seeded), so the switcher isn't empty.
-    default_id = request.session.get("user_id")
-    accounts = request.session.get("account_ids") or []
-    if default_id and default_id not in accounts:
-        accounts.append(default_id)
-        request.session["account_ids"] = accounts[-MAX_SESSION_ACCOUNTS:]
    # Attribute this request's log lines to the account. Here, not in the request middleware:
    # this is the one place a session becomes an account, and the middleware runs outside both the
    # session middleware and this task's context. `request.state` is shared across middleware layers
@@ -358,25 +325,17 @@ def _session_user(request: Request, db: Session, user_id: int | None, is_default
 def optional_current_user(request: Request, db: Session = Depends(get_db)) -> User | None:
    """Like `current_user` but returns None instead of raising 401 when there's no valid session.
    Used by the bootstrap /api/me probe so the logged-out landing doesn't log a failed request to
-    the browser console (a non-2xx fetch is a console error no matter how the app handles it).
-    The one lenient resolution (`strict=False`): a tab naming an account no longer signed in here gets
-    the default account, which is how it learns who it now is — see `resolved_user_id`. That includes an
-    account refused right here, which `_session_user` has just taken out of the wallet."""
-    user_id, is_default = resolved_user_id(request, strict=False)
+    the browser console (a non-2xx fetch is a console error no matter how the app handles it)."""
+    user_id = session_user_id(request)
+    if user_id is None:
+        return None
    try:
-        return _session_user(request, db, user_id, is_default)
-    # silent-ok: this is the anonymous case as much as the refused one — every landing page hit
-    # comes through here. The second attempt below is the handling, and it reports.
+        return _session_user(request, db, user_id)
    except HTTPException:
-        if is_default:
-            return None
-    try:
-        return _session_user(request, db, *resolved_user_id(request, strict=False))
-    except HTTPException:
-        # Both the pinned account AND the default one were refused, so the tab is signed out
-        # without being told why — and it got here from a session that named a real account.
+        # A session that named an account, refused: the browser is signed out without being told
+        # why, so say it here.
        log.info(
-            "a session was resolved to nobody: neither the named nor the default account held",
+            "a session was refused: its account no longer holds",
            extra=logs.event("auth.session.probe_rejected", user_id=user_id),
        )
        return None
@@ -26,7 +26,7 @@ from uuid import uuid4
 from sqlalchemy import and_, delete, func, not_, select, update
 from sqlalchemy.orm import Session, defer

-from app.userscope import owned_by
+from app.userscope import owned_by, owned_row
 from app import diagnostics
 from app.models import DebugScreenshot, DebugSession, LogEvent, User

@@ -129,12 +129,8 @@ def from_header(db: Session, user: User, value: str | None) -> DebugSession | No
    """The session a request names, if it is the caller's own and still live; else None."""
    if not value or len(value) != _ID_LEN or not all(c in "0123456789abcdef" for c in value):
        return None
-    session = db.get(DebugSession, value)
-    if (
-        session is None
-        or session.user_id != user.id
-        or not is_live(session, datetime.now(timezone.utc))
-    ):
+    session = owned_row(db, DebugSession, value, user)
+    if session is None or not is_live(session, datetime.now(timezone.utc)):
        return None
    return session

@@ -72,11 +72,11 @@ def resolve_ref(db: Session, user: User, kind: str, ref: str) -> tuple[str, obje
    # such pin target", which is the same answer this function gives for one that does not exist.
    except ValueError:
        return None
-    pl = db.get(Playlist, pid)
    # A playlist is per-user, unlike the other four (which are shared catalog rows whose PRIVACY
    # lives in the state tables). Someone else's list must read as "no such thing", not as a 403 —
    # the same reason `owned_or_404` 404s rather than 403s.
-    return (col, pl.id) if pl is not None and pl.user_id == user.id else None
+    pl = owned_row(db, Playlist, pid, user)
+    return (col, pl.id) if pl is not None else None


 def reorder(db: Session, user: User, ids: list[int]) -> None:
@@ -93,7 +93,7 @@ def yt_in_progress(db: Session, user: User, limit: int = RAIL_LIMIT) -> list[tup
    q = (
        select(*feed_columns(user, st), st.progress_updated_at.label("progressed_at"))
        .join(Channel, Channel.id == Video.channel_id)
-        .join(st, and_(st.video_id == Video.id, st.user_id == user.id))
+        .join(st, and_(st.video_id == Video.id, owned_by(st.user_id, user)))
        .where(
            st.position_seconds > 0,
            st.status.notin_(("watched", "hidden")),
@@ -136,7 +136,7 @@ def yt_video_row(db: Session, user: User, video_id: str) -> dict | None:
    row = db.execute(
        select(*feed_columns(user, st))
        .join(Channel, Channel.id == Video.channel_id)
-        .outerjoin(st, and_(st.video_id == Video.id, st.user_id == user.id))
+        .outerjoin(st, and_(st.video_id == Video.id, owned_by(st.user_id, user)))
        .where(Video.id == video_id)
    ).first()
    return serialize(row) if row is not None else None
@@ -179,7 +179,7 @@ def playlists_summary(
        select(*feed_columns(user, st), PlaylistItem.playlist_id.label("_playlist"))
        .join(PlaylistItem, PlaylistItem.video_id == Video.id)
        .join(Channel, Channel.id == Video.channel_id)
-        .outerjoin(st, and_(st.video_id == Video.id, st.user_id == user.id))
+        .outerjoin(st, and_(st.video_id == Video.id, owned_by(st.user_id, user)))
        .where(PlaylistItem.playlist_id.in_(playlist_ids))
        .order_by(PlaylistItem.playlist_id, PlaylistItem.position.asc())
    ).all()
@@ -328,9 +328,14 @@ def create_own(db: Session, user_id: int, name: str) -> FileFolder:
    return folder


+def renamable(folder: FileFolder) -> bool:
+    """Only a folder of the user's own making takes a name: a source folder's is its origin's."""
+    return folder.kind == FolderKind.OWN
+
+
 def rename(db: Session, folder: FileFolder, name: str) -> None:
    """Rename one of the user's own folders (a source folder's name is its origin's)."""
-    if folder.kind != FolderKind.OWN:
+    if not renamable(folder):
        raise FolderError("file_folder_not_own")
    folder.name = name
    try:
@@ -296,13 +296,11 @@ def own_link(db: Session, user: User, link_id: int) -> DownloadLink | None:
    if link is None:
        return None
    target = (
-        db.get(DownloadJob, link.job_id)
+        owned_row(db, DownloadJob, link.job_id, user)
        if link.job_id is not None
-        else db.get(FileFolder, link.folder_id)
+        else owned_row(db, FileFolder, link.folder_id, user)
    )
-    if target is None or target.user_id != user.id:
-        return None
-    return link
+    return link if target is not None else None


 def links_to(
@@ -16,6 +16,7 @@ import shutil
 import subprocess
 import sys

+import app.tables  # noqa: F401 — a fresh interpreter: every table, before the first query
 from app import logs
 from app import logsink
 from app.config import settings
@@ -212,7 +212,7 @@ def filtered_query(
    # exactly. 'unwatched'/'all' include row-less 'new' videos, so they keep the outer join.
    state_required = show in ("watched", "hidden", "in_progress")
    join_state = query.join if state_required else query.outerjoin
-    query = join_state(state, and_(state.video_id == Video.id, state.user_id == user.id))
+    query = join_state(state, and_(state.video_id == Video.id, owned_by(state.user_id, user)))

    # Hide videos the maintenance job has flagged as currently unplayable (deleted/private/
    # abandoned). They're either deleted after a grace period or unhidden if they recover;
@@ -28,6 +28,7 @@ log = logs.get_logger(__name__)
 from app import bodylimit, collation, logsink, state, sysconfig  # noqa: E402 — logging is configured first
 from app.config import settings  # noqa: E402 — logging is configured first
 from app.db import SessionLocal, engine  # noqa: E402 — logging is configured first
+import app.tables  # noqa: E402 — every table, before the first query
 from app.downloads import service as download_service  # noqa: E402 — logging is configured first
 from app.downloads import storage  # noqa: E402 — logging is configured first
 from app.plex import stream as plex_stream  # noqa: E402 — logging is configured first
@@ -49,7 +49,9 @@ def create_notification(

 def trim_read(db: Session, user_id: int, cap: int = READ_SOFT_CAP) -> int:
    """Delete a user's oldest READ notifications beyond `cap`. Returns the number removed.
-    Idempotent and cheap; safe to call after marking things read."""
+    Idempotent and cheap; safe to call after marking things read. Flushes first, so a notice the
+    caller has just marked read counts; the caller commits."""
+    db.flush()
    ids = (
        db.execute(
            select(Notification.id)
@@ -64,5 +66,4 @@ def trim_read(db: Session, user_id: int, cap: int = READ_SOFT_CAP) -> int:
        return 0
    # One bulk DELETE rather than a get+delete per row (the ids are already in hand).
    db.execute(delete(Notification).where(Notification.id.in_(ids)))
-    db.commit()
    return len(ids)
@@ -108,20 +108,6 @@ def state(db: Session, user: User | int, item_id: int) -> PlexState | None:
    )


-def states(db: Session, user: User | int, item_ids: list[int]) -> dict[int, PlexState]:
-    """The user's state on each of these items that has one, by item id."""
-    if not item_ids:
-        return {}
-    return {
-        s.item_id: s
-        for s in db.execute(
-            select(PlexState).where(
-                owned_by(PlexState.user_id, user), PlexState.item_id.in_(item_ids)
-            )
-        ).scalars()
-    }
-
-
 # --- a show ----------------------------------------------------------------------------------


@@ -8,6 +8,7 @@ from datetime import datetime, timezone
 from sqlalchemy import and_, case, func, or_, select
 from sqlalchemy.orm import Session, aliased

+from app.models import User
 from app.userscope import owned_by
 from app.plex.models import PlexItem, PlexState
 from app.plex.cards import ShowState
@@ -106,7 +107,7 @@ def state_map(db: Session, user_id: int, show_ids: list[int]) -> dict[int, ShowS
    return {r.show_id: show_state(r.total, r.watched, r.inprog) for r in rows}


-def item_state_map(db: Session, user_id: int, item_ids: list[int]) -> dict[int, PlexState]:
+def item_state_map(db: Session, user_id: User | int, item_ids: list[int]) -> dict[int, PlexState]:
    """A user's PlexState rows for a set of leaf items (movies or episodes), keyed by item_id — the
    leaf analogue of `state_map` (which rolls a show's episodes up to a `ShowState`). One batched
    fetch, so the caller serializes cards without a per-card query. Empty in → empty dict (no `IN ()`)."""
@@ -19,7 +19,7 @@ from app.plex import people as plex_people
 from app.plex import tmdb as plex_tmdb
 from app.plex.lookups import by_rating_key
 from app.models import User
-from app.plex import queries
+from app.plex import queries, rollup
 from app.plex.models import (
    PlexCollection,
    PlexItem,
@@ -302,6 +302,36 @@ def mark_finished(db: Session, user_id: int, item_id: int, now: datetime) -> boo
    return False


+def state_row(
+    db: Session,
+    user: User | int,
+    item_id: int,
+    st: PlexState | None = None,
+    *,
+    synced: bool = False,
+) -> PlexState:
+    """The user's state row on this item, made if the read before found none. Made by the insert
+    itself (ON CONFLICT DO NOTHING) and then read: the first checkpoint of an item and the `final`
+    flush beside it routinely both read "none", and a plain add let the second one fail on
+    `uq_plex_user_item` — a 500, and the push it was carrying lost. Every first row goes through
+    here: a checkpoint, a status, a season marked at once, a pull from Plex. `synced` is what a row
+    made now says about Plex; a row another writer made first keeps its own."""
+    if st is not None:
+        return st
+    db.execute(
+        pg_insert(PlexState)
+        .values(
+            user_id=user if isinstance(user, int) else user.id,
+            item_id=item_id,
+            synced_to_plex=synced,
+        )
+        .on_conflict_do_nothing(constraint="uq_plex_user_item")
+    )
+    made = queries.state(db, user, item_id)
+    assert made is not None  # inserted now, or committed by the request that won
+    return made
+
+
 def clear_progress(db: Session, st: PlexState | None, now: datetime) -> None:
    """A checkpoint too early to keep: a state that held nothing else goes, any other forgets its
    position."""
@@ -325,9 +355,7 @@ def checkpoint(
 ) -> PlexState:
    """Record a resume position. `dirty` marks it not yet mirrored to Plex — set before a push, so
    the reconcile re-pushes it if the push fails and the push can settle it once Plex confirms."""
-    if st is None:
-        st = PlexState(user_id=user.id, item_id=item_id)
-        db.add(st)
+    st = state_row(db, user, item_id, st)
    st.position_seconds = position
    st.progress_updated_at = now
    if dirty:
@@ -345,9 +373,7 @@ def set_status(
    db: Session, user: User, item_id: int, st: PlexState | None, status: str
 ) -> PlexState:
    """Set a watched or hidden status; watched also clears the resume point."""
-    if st is None:
-        st = PlexState(user_id=user.id, item_id=item_id)
-        db.add(st)
+    st = state_row(db, user, item_id, st)
    st.status = status
    if status == "watched":
        st.watched_at = datetime.now(timezone.utc)
@@ -364,15 +390,14 @@ def bulk_state(
    (item id, rating key, action) pushes a linked Plex account needs. Unwatching keeps the user's
    private "hidden" marks."""
    now = datetime.now(timezone.utc)
-    existing = queries.states(db, user_id, [e.id for e in eps])
+    existing = rollup.item_state_map(db, user_id, [e.id for e in eps])
    to_push: list[tuple[int, str, str]] = []
    for e in eps:
        st = existing.get(e.id)
        prev = st.status if st is not None else "new"
        if watched:
-            if st is None:
-                st = PlexState(user_id=user_id, item_id=e.id)
-                db.add(st)
+            st = state_row(db, user_id, e.id, st)
+            prev = st.status  # a first checkpoint beside this may have made it meanwhile
            st.status = "watched"
            st.watched_at = now
            st.position_seconds = 0
@@ -32,6 +32,8 @@ from app.db import SessionLocal
 from app.userscope import owned_by
 from app.plex.models import PlexItem, PlexLibrary, PlexLink, PlexState
 from app.plex.client import PlexClient, PlexError, PlexNotConfigured
+from app.plex import queries
+from app.plex.service import state_row
 from app.plex.sync import (
    _TYPE_EPISODE,
    _TYPE_MOVIE,
@@ -252,15 +254,7 @@ def push_state_to_plex(
        # Flag the row synced — but an "unwatched" via status=new deletes the row, so there may be
        # nothing to flag; that's fine (unscrobble still happened). Skip the flag if the row was
        # re-edited since we captured `expect_ts` (leave it dirty for its own push / the reconcile).
-        st = (
-            db.execute(
-                select(PlexState).where(
-                    owned_by(PlexState.user_id, user_id), PlexState.item_id == item_id
-                )
-            )
-            .scalars()
-            .first()
-        )
+        st = queries.state(db, user_id, item_id)
        if st is not None and _row_matches(st, expect_ts):
            st.synced_to_plex = True
            db.commit()
@@ -394,15 +388,7 @@ def _pull_apply(
    """Bring one Plex-side positive signal into Siftlode under last-write-wins. Returns
    "watched"/"in_progress" if it changed the row, else None. Never clears a row (un-watch is the
    full reconcile's job) — this only pulls in watched/resume."""
-    st = (
-        db.execute(
-            select(PlexState).where(
-                owned_by(PlexState.user_id, user_id), PlexState.item_id == item_id
-            )
-        )
-        .scalars()
-        .first()
-    )
+    st = queries.state(db, user_id, item_id)
    if _same_state(st, status, pos):
        if st is not None and not st.synced_to_plex:
            st.synced_to_plex = True  # already matches Plex → just settle the flag, no churn
@@ -423,8 +409,9 @@ def _pull_apply(
        # "almost always already pushed" assumption above excludes.
        return None
    if st is None:
-        st = PlexState(user_id=user_id, item_id=item_id)
-        db.add(st)
+        st = state_row(db, user_id, item_id, synced=True)
+        if not st.synced_to_plex:  # a local change made it first, beside this pull: the guard above
+            return None
    if status == "watched":
        st.status, st.position_seconds, st.watched_at, st.progress_updated_at = (
            "watched",
@@ -11,12 +11,14 @@ from sqlalchemy import (
    DateTime,
    Float,
    ForeignKey,
+    Index,
    Integer,
    JSON,
    String,
    Text,
    UniqueConstraint,
    func,
+    text,
 )
 from sqlalchemy.orm import Mapped, mapped_column, relationship

@@ -93,6 +95,15 @@ class Playlist(Base, TimestampMixin, UpdatedAtMixin):
    sync phases; the foundation phase only uses local playlists.)"""

    __tablename__ = "playlists"
+    __table_args__ = (
+        # One Watch later per user, held by the database: two first saves at once each made one.
+        Index(
+            "uq_playlist_watch_later",
+            "user_id",
+            unique=True,
+            postgresql_where=text("kind = 'watch_later'"),
+        ),
+    )

    id: Mapped[int] = mapped_column(primary_key=True)
    user_id: Mapped[int] = mapped_column(ForeignKey("users.id", ondelete="CASCADE"), index=True)
@@ -9,7 +9,7 @@ unique key, a row lock taken before a presence check. The comfort shelf's writes

 from datetime import datetime, timezone

-from sqlalchemy import exists, func, select, update
+from sqlalchemy import exists, func, select, text, update
 from sqlalchemy.dialects.postgresql import insert as pg_insert
 from sqlalchemy.orm import Session

@@ -239,24 +239,33 @@ def clear_state(db: Session, user: User, video_id: str) -> None:

 def set_progress(db: Session, user: User, video_id: str, position: int) -> None:
    """The player's resume point. 0 clears it: a row holding nothing else goes, one with a status
-    (watched, hidden) keeps it with the position zeroed. The status itself is never changed here."""
-    row = queries.video_state(db, user, video_id)
+    (watched, hidden) keeps it with the position zeroed. The status itself is never changed here.
+    A position is one upsert on (user, video): the first checkpoint and the end-of-playback mark
+    beside it both read "no row", and a plain add let the second one fail on `uq_user_video`."""
    now = datetime.now(timezone.utc)
-    if not position:
-        if row is None:
-            return
-        if row.status == "new":
-            db.delete(row)
-        else:
-            row.position_seconds = 0
-            row.progress_updated_at = now
-        db.flush()
+    if position:
+        db.execute(
+            pg_insert(VideoState)
+            .values(
+                user_id=user.id,
+                video_id=video_id,
+                position_seconds=position,
+                progress_updated_at=now,
+            )
+            .on_conflict_do_update(
+                constraint="uq_user_video",
+                set_={"position_seconds": position, "progress_updated_at": now, "updated_at": now},
+            )
+        )
        return
+    row = queries.video_state(db, user, video_id)
    if row is None:
-        row = VideoState(user_id=user.id, video_id=video_id)
-        db.add(row)
-    row.position_seconds = position
-    row.progress_updated_at = now
+        return
+    if row.status == "new":
+        db.delete(row)
+    else:
+        row.position_seconds = 0
+        row.progress_updated_at = now
    db.flush()


@@ -319,17 +328,25 @@ def create_playlist(db: Session, user: User, name: str) -> Playlist:


 def watch_later(db: Session, user: User) -> Playlist:
-    """The user's built-in Watch later playlist, created on first use."""
+    """The user's built-in Watch later playlist, created on first use. Created by the insert itself
+    (ON CONFLICT DO NOTHING on the one-per-user index) and then read: the first two saves at once
+    both read "none", and each used to make a list of its own."""
    pl = queries.watch_later(db, user)
    if pl is None:
-        pl = Playlist(
-            user_id=user.id,
-            name="Watch later",
-            kind="watch_later",
-            position=_next_playlist_position(db, user.id),
+        db.execute(
+            pg_insert(Playlist)
+            .values(
+                user_id=user.id,
+                name="Watch later",
+                kind="watch_later",
+                position=_next_playlist_position(db, user.id),
+            )
+            .on_conflict_do_nothing(
+                index_elements=["user_id"], index_where=text("kind = 'watch_later'")
+            )
        )
-        db.add(pl)
-        db.flush()  # the id, for the item the caller adds next
+        pl = queries.watch_later(db, user)
+        assert pl is not None  # inserted now, or committed by the request that won
    return pl


@@ -89,8 +89,9 @@ def owned_or_404(
 ) -> M:
    """Fetch a row by primary key that `user` owns, or raise 404 (`lock`: see `get_or_404`). A row owned by someone
    else 404s (NOT 403) on purpose — a 403 would confirm the id exists to a stranger.
-    It replaced the near-identical per-route ownership helpers, and is called directly — a thin
-    `_own_x` wrapper around it is the ad-hoc shape CLAUDE.md's patterns table rules out."""
+    A core route calls it directly. A unit's route cannot (it may not name a unit's model as a value):
+    it asks its unit's `queries.own_x`, which goes through the same `owned_row`, and answers
+    `not_found` itself."""
    obj = owned_row(db, model, id_, user, owner_attr=owner_attr, lock=lock)
    if obj is None:
        raise not_found(what)
@@ -5,9 +5,10 @@ helpers) stays in app/auth.py, imported below."""

 import secrets
 from datetime import datetime, timedelta, timezone
-from typing import NamedTuple
+from typing import Literal, NamedTuple
 from urllib.parse import urlencode

+from authlib.common.security import generate_token
 from authlib.integrations.starlette_client import OAuth, OAuthError
 from fastapi import APIRouter, BackgroundTasks, Depends, HTTPException, Request
 from fastapi.responses import JSONResponse, RedirectResponse
@@ -25,10 +26,11 @@ from app import sysconfig
 from app.auth import (
    _YOUTUBE_SCOPES,
    BASE_SCOPES,
-    MAX_SESSION_ACCOUNTS,
+    EXPECTED_ACCOUNT_HEADER,
    PASSWORD_MIN_LEN,
    READ_SCOPES,
    WRITE_SCOPES,
+    built_for_another_account,
    current_user,
    env_admin_canonicals,
    env_allowed_canonicals,
@@ -37,7 +39,9 @@ from app.auth import (
    is_allowed,
    log,
    operator_contact,
-    resolved_user_id,
+    optional_current_user,
+    session_user_id,
+    stamp_session,
 )
 from app.config import settings
 from app.db import SessionLocal, get_db
@@ -320,6 +324,25 @@ class SetPasswordIn(BaseModel):
    current_password: str | None = None


+# A connect in flight, in the session: `{state, uid, explicit}`. It names the OAuth state of the flow
+# that set it, because the session is the browser's, not the flow's: a callback coming back with
+# another state (an older flow, which authlib then refuses) must not spend the marker of the connect
+# still open, or that connect would return as a plain sign-in — signing someone else in, or storing
+# a YouTube grant on whichever account the Google identity already belongs to.
+_LINK_MARKER = "oauth_link"
+
+# Where a navigation from a tab showing another account than the one signed in lands: the app, which
+# boots as the account signed in now and says why (the frontend's `ACCOUNT_CHANGED_FLAG`).
+ACCOUNT_CHANGED_LANDING = "/?account=changed"
+
+
+def _take_link_marker(request: Request) -> dict | None:
+    marker = request.session.get(_LINK_MARKER)
+    if not isinstance(marker, dict) or marker.get("state") != request.query_params.get("state"):
+        return None
+    return request.session.pop(_LINK_MARKER)
+
+
@router.get("/login")
 async def login(request: Request, db: Session = Depends(get_db)):
    # access_type=offline ensures a refresh_token on first authorization. We avoid
@@ -330,6 +353,10 @@ async def login(request: Request, db: Session = Depends(get_db)):
        # Google sign-in isn't configured on this instance — land back on the login page (the UI
        # hides the button, so this is just a safety net for a stale/direct hit).
        return RedirectResponse(url="/")
+    # A sign-in, never a link. Starting it retires every earlier flow's state (authlib keeps only the
+    # newest), so a connect's marker left from before — abandoned at Google, or still open in
+    # another tab — names a flow that can no longer finish.
+    request.session.pop(_LINK_MARKER, None)
    return await client.authorize_redirect(
        request,
        _oauth_redirect_uri(request),
@@ -350,24 +377,23 @@ async def callback(
    if client is None:
        return RedirectResponse(url="/")
    # Linking mode: an already-authenticated account is attaching this Google identity (a
-    # password account enabling YouTube or connecting SSO). The markers are set by /auth/link and
-    # /auth/upgrade; when present we attach to the current user instead of identifying by sub.
-    # Pop them BEFORE the token exchange: if the user aborts Google's consent screen the exchange
-    # throws below, and a marker left behind would silently turn the NEXT clean sign-in into a link
-    # attempt. Popping up front means an aborted link flow leaves no residue in the session.
-    link_uid = request.session.pop("oauth_link_uid", None)
-    link_explicit = request.session.pop("oauth_link_explicit", False)
+    # password account enabling YouTube or connecting SSO). The marker is set by
+    # /auth/google/connect; when it is this flow's we attach to the current user instead of
+    # identifying by sub. Taken BEFORE the token exchange: if the user aborts Google's consent screen
+    # the exchange throws below, and a marker left behind would silently turn the NEXT clean sign-in
+    # into a link attempt. A connect that never comes back here at all is cleared by /auth/login.
+    link = _take_link_marker(request)
    try:
        token = await client.authorize_access_token(request)
    except OAuthError as exc:
        # The user cancelled or denied consent at Google — land back with a friendly notice instead
-        # of a raw 400 JSON error page. An already-signed-in user (adding another account via
-        # /auth/login, or linking/upgrading via /auth/link|upgrade) never sees the pre-auth login
-        # banner, so surface a neutral toast via ?oauth=cancelled; a fresh sign-in cancel lands on
-        # the login page with the oauth_cancelled banner instead. Keyed on the live session, not on
-        # link_uid, so the add-account path (no marker) is covered too.
+        # of a raw 400 JSON error page. An already-signed-in user (signing in again via /auth/login,
+        # or connecting via /auth/google/connect) never sees the pre-auth login banner, so
+        # surface a neutral toast via ?oauth=cancelled; a fresh sign-in cancel lands on the login page
+        # with the oauth_cancelled banner instead. Keyed on the live session, not on link_uid, so a
+        # sign-in started while signed in (no marker) is covered too.
        log.info("Google OAuth cancelled/failed: %s", exc.error)
-        if request.session.get("user_id") is not None:
+        if session_user_id(request) is not None:
            return RedirectResponse(url="/?oauth=cancelled")
        return RedirectResponse(url="/?login=oauth_cancelled")

@@ -378,8 +404,8 @@ async def callback(
    sub = userinfo["sub"]
    email = (userinfo.get("email") or "").lower()

-    if link_uid is not None:
-        return _complete_link(request, db, link_uid, link_explicit, sub, userinfo, token)
+    if link is not None:
+        return _complete_link(request, db, link["uid"], link["explicit"], sub, userinfo, token)

    if not is_allowed(db, email):
        # No uid to name — having no account here is the branch. And the address is left out on
@@ -580,19 +606,16 @@ def _complete_link(
    token: dict,
 ):
    """Attach the just-authorized Google identity to the already-signed-in account that started
-    the link/upgrade. No is_allowed gate — they're already an active user. Refuses to hijack a
+    the connect. No is_allowed gate — they're already an active user. Refuses to hijack a
    Google identity owned by another account, or to silently swap a different linked identity.
-    `explicit` (the popped oauth_link_explicit marker) distinguishes a "Connect Google" click from
-    a YouTube upgrade — the caller pops it up front so an aborted flow leaves nothing behind."""
-    # SEC-11: the marker must name an account signed into THIS browser — the session default OR any
-    # account in the wallet — not only the default. A tab acting as a secondary account links Google to
-    # ITSELF (the marker was set from current_user, which resolves the ?account= it navigated with), so
-    # accepting wallet membership stops the identity/token landing on the default account A instead of B.
-    accounts = request.session.get("account_ids") or []
-    if link_uid != request.session.get("user_id") and link_uid not in accounts:
-        return RedirectResponse(url="/?link=error")
-    user = db.get(User, link_uid)
-    if user is None:
+    `explicit` (from the link marker) distinguishes a "Connect Google" click from a YouTube grant —
+    the caller takes the marker up front so an aborted flow leaves nothing behind."""
+    # The marker must name the account still signed into THIS browser, and that session must still
+    # hold: a sign-in as someone else between the start and the callback replaced it (the identity
+    # must not land on the newcomer), and a session revoked meanwhile ("log out everywhere", a
+    # password change, a suspension) must not finish what it started.
+    user = optional_current_user(request, db)
+    if user is None or user.id != link_uid:
        return RedirectResponse(url="/?link=error")
    other = db.execute(
        select(User).where(User.google_sub == sub, User.id != user.id)
@@ -612,45 +635,31 @@ def _complete_link(
    _store_token(db, user, token)
    db.commit()
    log.info("Google linked: uid=%s scopes=%s", user.id, token.get("scope"))
-    # An explicit "Connect Google" lands on Settings with a confirmation; a YouTube upgrade
+    # An explicit "Connect Google" lands on Settings with a confirmation; a YouTube grant
    # (wizard / Settings access rows) returns to the app so the onboarding flow resumes as before.
    return RedirectResponse(url="/?link=ok" if explicit else "/")


@router.post("/logout")
 async def logout(request: Request):
-    """Sign the requesting tab's active account out of this browser (removing it from the wallet).
-    Per-tab aware: the account is taken from the X-Siftlode-Account header when present, so logging
-    out in one tab doesn't disturb another tab running a different account. If the account being
-    removed was the session default, promote the most recent remaining account (or clear the
-    session when none remain)."""
-    # A tab still pinned to an account already signed out in another tab resolves to nobody, so it
-    # signs nobody out — it used to fall back to the default and sign THAT account out in its place.
-    active, is_default = resolved_user_id(request)
-    remaining = [a for a in (request.session.get("account_ids") or []) if a != active]
-    request.session["account_ids"] = remaining
-    if is_default:
-        if remaining:
-            request.session["user_id"] = remaining[-1]
-            return JSONResponse({"ok": True, "switched": True})
-        request.session.clear()
-        return JSONResponse({"ok": True, "switched": False})
-    # A non-default (per-tab) account signed out: it's gone from the wallet and the session default
-    # is untouched. The tab drops its own override and falls back to the default on reload.
-    return JSONResponse({"ok": True, "switched": False})
+    """Sign this browser out: the session holds one account, and it ends — unless the tab asking was
+    built for another account than the one signed in now (a sign-in as someone else in another tab):
+    its Sign out would end THAT person's session (see `EXPECTED_ACCOUNT_HEADER`)."""
+    signed_in = session_user_id(request)
+    if signed_in is not None and built_for_another_account(
+        signed_in, request.headers.get(EXPECTED_ACCOUNT_HEADER)
+    ):
+        raise refused(401, "account_changed")
+    request.session.clear()
+    return JSONResponse({"ok": True})


 def _establish_session(request: Request, user: User) -> None:
    """Sign `user` in for this browser session — the ONE sign-in path, shared by the Google callback
-    and password login, so both join the same account switcher and both stamp `last_login_at` (the
-    caller commits). Multi-session: every account that authenticated in this browser is remembered so
-    the user can switch between them; only accounts that actually proved themselves HERE land in that
-    wallet (it is the proof they may be switched to)."""
-    accounts = [a for a in (request.session.get("account_ids") or []) if a != user.id]
-    accounts.append(user.id)
-    request.session["account_ids"] = accounts[-MAX_SESSION_ACCOUNTS:]
+    and password login, so both stamp `last_login_at` (the caller commits). One account per session:
+    the sign-in replaces whoever was signed in here before."""
    request.session["user_id"] = user.id
-    _remember_epoch(request, user)
+    stamp_session(request, user)
    user.last_login_at = datetime.now(timezone.utc)


@@ -1135,15 +1144,6 @@ def password_reset_confirm(body: TokenPasswordIn, db: Session = Depends(get_db))
    return {"ok": True}


-def _remember_epoch(request: Request, user: User) -> None:
-    """Record this account's current `session_epoch` in the signed cookie (SA4), so `current_user`
-    can later reject the cookie if the account's epoch was bumped (password reset/change, or
-    "log out everywhere"). Per-account so one account's revocation doesn't evict the others."""
-    epochs = dict(request.session.get("epochs") or {})
-    epochs[str(user.id)] = user.session_epoch
-    request.session["epochs"] = epochs
-
-
 def bump_session_epoch(db: Session, user: User) -> None:
    """Invalidate every EXISTING signed cookie for this account by advancing its epoch (SA4). Cookies
    minted before the bump carry a now-stale epoch and are rejected by `current_user`. The account's
@@ -1220,35 +1220,11 @@ def logout_others(
    # session at a newer epoch than the DB (see set_password).
    bump_session_epoch(db, user)
    db.commit()
-    _remember_epoch(request, user)
+    stamp_session(request, user)
    log.info("Logout-others: uid=%s (epoch=%s)", user.id, user.session_epoch)
    return {"ok": True}


-@router.get("/link")
-async def link_google(
-    request: Request,
-    user: User = Depends(current_user),
-    db: Session = Depends(get_db),
-):
-    """Start linking a Google account to the signed-in (e.g. password) account. Requests only
-    the identity scopes — YouTube access is granted separately via /auth/upgrade. The callback
-    sees `oauth_link_uid` and attaches the identity instead of creating a new account."""
-    client = google_oauth(db)
-    if client is None:
-        return RedirectResponse(url="/?link=error")
-    request.session["oauth_link_uid"] = user.id
-    request.session["oauth_link_explicit"] = True
-    return await client.authorize_redirect(
-        request,
-        _oauth_redirect_uri(request),
-        access_type="offline",
-        prompt="select_account",
-        include_granted_scopes="true",
-        scope=BASE_SCOPES,
-    )
-
-
@router.post("/set-password")
 def set_password(
    body: SetPasswordIn,
@@ -1272,35 +1248,57 @@ def set_password(
    # can't leave the cookie at a newer epoch than the DB (which would wrongly 401 THIS session).
    bump_session_epoch(db, user)
    db.commit()
-    _remember_epoch(request, user)
+    stamp_session(request, user)
    log.info("Password set/changed: uid=%s", user.id)
    return {"ok": True}


-@router.get("/upgrade")
-async def upgrade(
+# What each Google connect asks Google for. `identity` attaches a Google sign-in to the account
+# (Settings → Connect Google) and lets the person pick which Google account; `read` and `write` grant
+# YouTube access (the onboarding wizard, the Settings access rows) and always show the consent screen,
+# so a scope granted earlier and since revoked at Google is asked for again.
+_CONNECT: dict[str, tuple[str, str]] = {
+    "identity": (BASE_SCOPES, "select_account"),
+    "read": (READ_SCOPES, "consent"),
+    "write": (WRITE_SCOPES, "consent"),
+}
+
+
+@router.get("/google/connect")
+async def google_connect(
    request: Request,
-    access: str = "read",
+    access: Literal["identity", "read", "write"],
+    expect: str | None = None,
    user: User = Depends(current_user),
    db: Session = Depends(get_db),
 ):
-    """Incremental consent for the onboarding wizard. `access=read` grants YouTube
-    read-only (enough to build the feed); `access=write` additionally grants management
-    (unsubscribe). The shared callback stores whatever Google returns, so `can_read` /
-    `can_write` flip on once granted. Anything other than an explicit `write` defaults to
-    the least-privileged read scope."""
+    """Attach Google to the signed-in account: its identity, or YouTube read/write access on top.
+    The shared callback sees the link marker and attaches what Google returns to THIS account —
+    without it a password account (no google_sub) would get a brand-new, separate Google user
+    created instead. It stores whatever scopes Google grants, so `can_read` / `can_write` flip on
+    once granted.
+
+    `expect` is the account the tab was built for. A navigation carries no headers, so the check
+    every request gets (`EXPECTED_ACCOUNT_HEADER`) rides the query here: a tab still showing the
+    account signed in before would otherwise attach a Google identity to the one signed in now."""
+    if built_for_another_account(user.id, expect):
+        return RedirectResponse(url=ACCOUNT_CHANGED_LANDING)
    client = google_oauth(db)
    if client is None:
-        return RedirectResponse(url="/")
-    # Attach the grant to THIS account in the callback. Without it a password account (no
-    # google_sub) would get a brand-new, separate Google user created instead of being linked.
-    request.session["oauth_link_uid"] = user.id
-    scope = WRITE_SCOPES if access == "write" else READ_SCOPES
+        return RedirectResponse(url="/?link=error")
+    scope, prompt = _CONNECT[access]
+    state = generate_token(48)
+    request.session[_LINK_MARKER] = {
+        "state": state,
+        "uid": user.id,
+        "explicit": access == "identity",
+    }
    return await client.authorize_redirect(
        request,
        _oauth_redirect_uri(request),
+        state=state,
        access_type="offline",
-        prompt="consent",
+        prompt=prompt,
        include_granted_scopes="true",
        scope=scope,
    )
@@ -220,6 +220,7 @@ def send_issue(sid: str, user: User = Depends(current_user), db: Session = Depen
        raise refused(409, "issue_sending")
    new = session.issue_url is None
    if not new and not debugsession.unfinished(session):
+        db.commit()  # nothing to send: the lock ends here, not when the request's session closes
        return {"ok": True, "session": _view(db, session)}
    key = str(user.id)
    if new:
@@ -375,6 +375,9 @@ def update_folder(
    folder = own_folder(db, user, folder_id)
    sent = body.model_fields_set
    if "name" in sent:
+        # Which folder may be renamed is answered before what the name is, as it always was.
+        if not folders.renamable(folder):
+            raise refused(409, "file_folder_not_own")
        try:
            folders.rename(db, folder, _folder_name(body.name or ""))
        except folders.FolderError as e:
@@ -9,11 +9,9 @@ from app.auth import (
    count_admins,
    count_pending_activations,
    current_user,
-    drop_from_wallet,
    google_enabled,
    has_read_scope,
    has_write_scope,
-    is_allowed,
    optional_current_user,
    purge_user,
 )
@@ -26,79 +24,10 @@ from app.db import get_db
 from app.models import Invite, User
 from app.utils import iso
 from app.refusal import refused
-from app.routes._common import not_found

 router = APIRouter(prefix="/api/me", tags=["me"])


-@router.get("/accounts")
-def my_accounts(
-    request: Request,
-    user: User = Depends(current_user),
-    db: Session = Depends(get_db),
-) -> list[dict]:
-    """The accounts that have authenticated in this browser session (switchable without a
-    new Google sign-in). The active one is flagged."""
-    ids = request.session.get("account_ids") or [user.id]
-    rows = {u.id: u for u in db.execute(select(User).where(User.id.in_(ids))).scalars()}
-    out = []
-    for uid in ids:  # preserve recency order from the session
-        u = rows.get(uid)
-        if u is not None:
-            out.append(
-                {
-                    "id": u.id,
-                    "email": u.email,
-                    "display_name": u.display_name,
-                    "avatar_url": u.avatar_url,
-                    "active": u.id == user.id,
-                }
-            )
-    return out
-
-
-class SwitchAccountIn(BaseModel):
-    user_id: int
-
-
-@router.post("/switch")
-def switch_account(
-    body: SwitchAccountIn,
-    request: Request,
-    user: User = Depends(current_user),
-    db: Session = Depends(get_db),
-) -> dict:
-    """Switch the active account to another one already authenticated in this browser. No
-    Google round-trip — but only accounts present in the session list (proof they signed in
-    here) are allowed, and access is re-checked in case the invite was revoked since."""
-    target = body.user_id
-    if target not in (request.session.get("account_ids") or []):
-        raise refused(403, "not_your_account")
-    # An account `_session_user` would refuse — gone, deactivated, suspended, or its session revoked — is
-    # refused here too, and dropped from the switch list the way `_session_user` drops it. Made the session
-    # default instead, the next request's current_user would refuse it as the DEFAULT and clear the whole
-    # wallet session, logging out every account here. (B-core-9)
-    u = db.get(User, target)
-    if u is None:
-        drop_from_wallet(request.session, target)
-        raise not_found("account")
-    if not is_allowed(db, u.email):
-        raise refused(403, "account_no_access")
-    if not u.is_active:
-        drop_from_wallet(request.session, target)
-        raise refused(403, "account_no_access")
-    if u.is_suspended:
-        drop_from_wallet(request.session, target)
-        raise refused(403, "account_suspended")
-    # Revoked: a password reset/change or "log out everywhere" bumped the account's epoch.
-    epochs = request.session.get("epochs") or {}
-    if int(epochs.get(str(target), 0)) != (u.session_epoch or 0):
-        drop_from_wallet(request.session, target)
-        raise refused(403, "account_signed_out")
-    request.session["user_id"] = target
-    return {"ok": True, "user_id": target}
-
-
@router.get("")
 def get_me(
    user: User | None = Depends(optional_current_user), db: Session = Depends(get_db)
@@ -187,17 +116,11 @@ def delete_my_account(
    surfaces)."""
    if user.role == "admin" and count_admins(db) <= 1:
        raise refused(400, "only_admin_delete")
-    user_id = user.id
    # Full erasure (cascades) + access-request cleanup + Google-grant revocation; shared with the
-    # admin delete path. Capture the id first — `user` is detached once purge_user deletes the row.
+    # admin delete path.
    purge_user(db, user, background)
-    # Drop this account from the browser session; switch to another signed-in account if any.
-    remaining = [a for a in (request.session.get("account_ids") or []) if a != user_id]
-    if remaining:
-        request.session["account_ids"] = remaining
-        request.session["user_id"] = remaining[-1]
-    else:
-        request.session.clear()
+    # The session's one account is gone: so is the session.
+    request.session.clear()
    return {"deleted": True}


@@ -213,7 +136,13 @@ def update_preferences(
    user: User = Depends(current_user),
    db: Session = Depends(get_db),
 ) -> dict:
-    # Merge so partial updates don't wipe other keys.
+    # Merge so partial updates don't wipe other keys — over the stored row read again under a row
+    # lock: a client saves keys one at a time (several at once on a boot that moves this browser's
+    # copies to the account), and two merges over the same earlier read would keep only the last.
+    # FOR NO KEY UPDATE, as bump_session_epoch: it still orders two merges, and a transaction that
+    # only added a row referring to the account (a playlist pull, then minutes of YouTube) does not
+    # hold the save up.
+    db.refresh(user, with_for_update={"key_share": True})
    merged = dict(user.preferences or {})
    merged.update(preferences.root)
    user.preferences = merged
@@ -78,8 +78,8 @@ def mark_read(
 ) -> dict:
    n = owned_or_404(db, Notification, notification_id, user, what="notification")
    n.read = True
-    db.commit()
    trim_read(db, user.id)
+    db.commit()
    return {"ok": True}


@@ -90,8 +90,8 @@ def mark_all_read(user: User = Depends(current_user), db: Session = Depends(get_
        .where(owned_by(Notification.user_id, user), Notification.read.is_(False))
        .values(read=True)
    )
-    db.commit()
    trim_read(db, user.id)
+    db.commit()
    return {"ok": True}


@@ -100,7 +100,7 @@ def show_detail(
        raise not_found("show")
    seasons = queries.seasons(db, sh)
    eps = queries.episodes(db, sh)
-    states = queries.states(db, user, [e.id for e in eps])
+    states = rollup.item_state_map(db, user, [e.id for e in eps])
    by_season: dict[int | None, list] = {}
    all_cards: list[dict] = []  # every episode, in season/episode order → show-level rollup
    for e in eps:
@@ -247,7 +247,7 @@ def _collection_strips(db: Session, it: "PlexItem", user_id: int, lang: str) ->
    for col in ordered:
        rk = col.rating_key
        members = queries.library_movies_in(db, it.library_id, rk, lang, limit=60)
-        states = queries.states(db, user_id, [m.id for m in members])
+        states = rollup.item_state_map(db, user_id, [m.id for m in members])
        out.append(
            {
                "id": rk,
@@ -19,7 +19,7 @@ from fastapi.responses import FileResponse
 from pydantic import BaseModel
 from sqlalchemy.orm import Session

-from app.auth import current_user, resolved_user_id
+from app.auth import current_user, session_user_id
 from app.db import SessionLocal, get_db
 from app.models import User
 from app.plex import imgcache, queries, service, subtitles
@@ -42,7 +42,7 @@ def _require_session(request: Request) -> int:
    """Lightweight auth for the high-fan-out image proxies: validate the signed session cookie WITHOUT
    a DB user-load, so these endpoints never hold a pool connection for the auth. Poster images are
    low-sensitivity; a valid session is enough."""
-    uid, _ = resolved_user_id(request)
+    uid = session_user_id(request)
    if not uid:
        raise refused(401, "not_authenticated")
    return uid
@@ -9,7 +9,7 @@ from sqlalchemy.orm import Session
 from app.auth import current_user
 from app.db import get_db
 from app.models import User
-from app.plex import queries, service
+from app.plex import queries, rollup, service
 from app.plex.cards import display_title, leaf_card, norm_lang
 from app.refusal import refused
 from app.routes._common import not_found
@@ -99,7 +99,7 @@ def playlist_detail(
    lang = norm_lang(lang)
    pl = _own_playlist(db, user, pid)
    items = queries.playlist_items(db, pl.id)
-    states = queries.states(db, user, [it.id for it in items])
+    states = rollup.item_state_map(db, user, [it.id for it in items])
    shows = queries.shows_by_ids(db, {it.show_id for it in items if it.show_id})
    cards = [
        leaf_card(
@@ -3,8 +3,9 @@
 **Ownership.** A user-owned table says whose a row is in one column (`user_id`, or `to_user_id` for
 something handed to them). Every read and write names that column through `owned_by` and fetches a
 row by id through `owned_row`, never with an inline `Model.user_id == …`: the rule "a row owned by
-someone else is answered like a missing one" then lives here, and a contract test
-(`test_unit_fences.py`) holds every module to it, core included.
+someone else is answered like a missing one" then lives here. A contract test
+(`test_unit_fences.py`) holds every module, core included, to the filter's form; a rule this module
+cannot say (own or shared, built-in or own) is written out where it applies, with its reason.

 **Recipients.** "An active, non-suspended account" — the download share-by-email and the recipient
 picker filter on it. Kept here so the two forms (SQL clauses for a query, a Python check for a loaded
@@ -31,6 +31,7 @@ from functools import partial
 from sqlalchemy import text
 from sqlalchemy.orm import Session

+import app.tables  # noqa: F401 — every table, before the first query
 from app import logs
 from app import logsink
 from app import sysconfig
@@ -5,7 +5,8 @@
 # scans it, blocking. A security fix, or a yt-dlp release YouTube's changes force, may be ONE pin
 # bumped in place instead of a full regeneration, which would move every other package too — then
 # `pip install -r requirements.lock && pip check` must pass. In place so far: urllib3 2.7.0 -> 2.8.0 (PYSEC-2026-4175/4176/4177); yt-dlp 2026.7.4 -> 2026.8.19
-# (it marks the clients that need a stream PO token, so the downloader stops fetching into a 403).
+# (it marks the clients that need a stream PO token, so the downloader stops fetching into a 403);
+# certifi 2026.7.22 -> 2026.10.19 (GHSA-6q48-6c7w-9gw2).
 APScheduler==3.11.3
 Authlib==1.7.2
 Mako==1.4.1
@@ -21,7 +22,7 @@ argon2-cffi-bindings==25.1.0
 argon2-cffi==25.1.0
 bgutil-ytdlp-pot-provider==1.3.1
 brotli==1.2.0
-certifi==2026.7.22
+certifi==2026.10.19
 cffi==2.1.1
 charset-normalizer==3.5.1
 click==8.4.2
@@ -105,6 +105,11 @@ def pytest_configure(config):
        )


+def _write_xact(db) -> str | None:
+    """The id of the open transaction if it has written anything, else None (a read never takes one)."""
+    return db.scalar(text("SELECT pg_current_xact_id_if_assigned()::text"))
+
+
 def _request_dep(db, unfinished: list[str]):
    """`get_db` for a test request: the test's own session, plus the check a shared session hides.

@@ -114,15 +119,16 @@ def _request_dep(db, unfinished: list[str]):
    and the suite stays green over a handler that forgot to commit (deep review, debt burn). So a
    request that ends normally with a write it did not commit — flushed, or still only staged — is
    recorded, and the test fails on it at teardown. Only the ROOT transaction's end clears it: a
-    savepoint released inside the handler commits nothing. The test's own staged rows are flushed before the request starts — they are the
-    test's business, not the handler's. A request that raises is rolled back, as production's
+    savepoint released inside the handler commits nothing. What the test itself staged or wrote is
+    committed before the request starts — it is the test's business, not the handler's, and a
+    request then begins with no write open, so any write left at its end is the handler's. A request that raises is rolled back, as production's
    `close()` rolls it back: without that, what a failing handler flushed but never committed stayed
    readable to the rest of the test — the one state production cannot be in (an issue created
    on the tracker before a failed upload was kept only by an explicit commit, and nothing saw it
    missing)."""

    def dep(conn: HTTPConnection):
-        db.flush()
+        db.commit()
        state = {"dirty": False}

        def flushed(session, ctx):
@@ -138,7 +144,12 @@ def _request_dep(db, unfinished: list[str]):
        try:
            yield db
            staged = db.new or db.deleted or any(db.is_modified(o) for o in db.dirty)
-            if state["dirty"] or staged:
+            # Core DML (`db.execute(update(...))`) fires no flush and stages nothing; the database
+            # itself says whether a write is open — the request began with none, and a transaction
+            # that wrote and was not committed holds an id (see `release_read_connection`). A row
+            # lock (`FOR UPDATE`) takes one too: the handler ends what it locked.
+            core_dml = _write_xact(db) is not None
+            if state["dirty"] or staged or core_dml:
                unfinished.append(f"{conn.scope.get('method', 'WS')} {conn.url.path}")
        except Exception:
            db.rollback()
@@ -220,7 +231,7 @@ def _configured(monkeypatch):
@pytest.fixture
 def client(db, override_dep, _configured):
    """A TestClient wired to the test DB that goes through the REAL auth/session stack — no
-    `current_user` override — so login, the wallet gate and epoch revoke are exercised end to end.
+    `current_user` override — so login, the one-account session and epoch revoke are exercised end to end.
    Starlette's SessionMiddleware round-trips the signed cookie, so a login in one call authenticates
    the next on this same instance. No lifespan (the scheduler / HLS sweep stay off)."""
    override_dep(get_db, db.info["request"])
@@ -1,21 +1,25 @@
 """Auth / session security core (TEST-1), DB-lane.

-The wallet impersonation gate, epoch-based session revocation, single-use tokens and the
-enumeration-safe uniform 401 are the security mag, and had zero tests (only `_oauth_redirect_uri` was
-covered). Starlette's SessionMiddleware round-trips the signed cookie under TestClient, so the whole
-browser-less flow is exercisable. Uses the real auth stack via the `client` fixture (NO current_user
-override) so login, the wallet gate and epoch revoke are the code under test.
+One account per browser session, epoch-based session revocation, single-use tokens and the
+enumeration-safe uniform 401 are the security mag. Starlette's SessionMiddleware round-trips the
+signed cookie under TestClient, so the whole browser-less flow is exercisable. Uses the real auth
+stack via the `client` fixture (NO current_user override) so login, the session and epoch revoke are
+the code under test.

 Note: `/api/me` is the bootstrap probe — it returns 200 `{authenticated: False}` when logged out, not a
 401. A revoked session is therefore checked on a PROTECTED endpoint (`/api/feed`), which does 401.
 """

+import base64
+import json
 from datetime import timedelta
+from pathlib import Path

 import pytest
 from fastapi.testclient import TestClient
+from itsdangerous import TimestampSigner

-from app.auth import ACTIVE_ACCOUNT_HEADER
+from app.config import settings
 from app.main import app

 _PW = "correct horse battery"
@@ -46,74 +50,48 @@ def test_wrong_password_and_unknown_email_give_a_byte_identical_401(client, make
    assert wrong_pw.json() == unknown.json()


-def test_a_non_wallet_account_header_falls_back_to_the_default(client, make_user):
-    # The X-Siftlode-Account header may only re-target an account already signed into THIS browser
-    # (in the session wallet). A header naming someone else must be ignored, not honored — otherwise
-    # it's an impersonation gate hole.
-    make_user(email="alice2@example.com", password=_PW)
-    mallory = make_user(email="mallory@example.com", password=_PW)
-    assert _login(client, "alice2@example.com", _PW).status_code == 200
-    me = client.get("/api/me", headers={ACTIVE_ACCOUNT_HEADER: str(mallory.id)})
-    assert me.status_code == 200
-    assert me.json()["email"] == "alice2@example.com"  # still Alice, not Mallory
+def _cookie(payload: dict) -> str:
+    """A session cookie signed the way SessionMiddleware signs one — for a cookie an older build wrote."""
+    data = base64.b64encode(json.dumps(payload).encode())
+    return TimestampSigner(str(settings.secret_key)).sign(data).decode()


-def test_a_tab_pinned_to_an_account_signed_out_elsewhere_acts_as_nobody(client, make_user):
-    # Two accounts in one browser: a tab pinned to Alice, Bob the default. Alice signs out in another
-    # tab. The pinned tab used to go on acting as the default, Bob — its player checkpoints and pref
-    # saves landed in his account, and its own Sign out signed HIM out.
-    alice = make_user(email="alice3@example.com", password=_PW)
+def test_signing_in_as_another_account_replaces_the_first(client, make_user):
+    # One account per browser session: a second sign-in is a fresh session, not a second account
+    # beside the first — so signing out leaves nobody signed in here.
+    make_user(email="alice3@example.com", password=_PW)
    make_user(email="bob3@example.com", password=_PW)
    assert _login(client, "alice3@example.com", _PW).status_code == 200
-    assert _login(client, "bob3@example.com", _PW).status_code == 200  # Bob is now the default
-    tab = {ACTIVE_ACCOUNT_HEADER: str(alice.id)}
-    assert client.get("/api/feed", headers=tab).status_code == 200
-    assert client.post("/auth/logout", headers=tab).status_code == 200  # Alice signs out
+    assert _login(client, "bob3@example.com", _PW).status_code == 200
+    assert client.get("/api/me").json()["email"] == "bob3@example.com"

-    assert client.get("/api/feed", headers=tab).status_code == 401
-    assert client.post("/auth/logout", headers=tab).status_code == 200
-    assert client.get("/api/feed").status_code == 200  # Bob is still signed in, the session intact
-    # The bootstrap probe still answers as the default: how the stale tab learns who it is now.
-    assert client.get("/api/me", headers=tab).json()["email"] == "bob3@example.com"
+    assert client.post("/auth/logout").status_code == 200
+    assert client.get("/api/feed").status_code == 401
+    assert client.get("/api/me").json() == {"authenticated": False}


-@pytest.mark.parametrize("ended", ["revoked", "suspended"])
-def test_a_tab_pinned_to_an_account_whose_session_ended_comes_back_as_the_default(
-    client, make_user, db, ended
-):
-    # A tab pinned to Alice, Bob the default. Alice's session then ends while she is still in this
-    # browser's wallet ("log out everywhere" on another device, or an admin suspends her). The tab's
-    # bootstrap probe used to answer "signed out": the landing page with Bob still signed in, and no way
-    # off it but closing the tab (signing in as anyone else there kept the tab pinned to Alice).
+def test_no_header_or_parameter_picks_another_account(client, make_user):
+    # Alice signed in here before Bob: nothing a request carries makes it act as her any more.
    alice = make_user(email="alice4@example.com", password=_PW)
    make_user(email="bob4@example.com", password=_PW)
    assert _login(client, "alice4@example.com", _PW).status_code == 200
    assert _login(client, "bob4@example.com", _PW).status_code == 200
-    tab = {ACTIVE_ACCOUNT_HEADER: str(alice.id)}
-    assert client.get("/api/feed", headers=tab).status_code == 200
-    if ended == "revoked":
-        alice.session_epoch = (alice.session_epoch or 0) + 1
-    else:
-        alice.is_suspended = True
-    db.commit()
+    me = client.get(f"/api/me?account={alice.id}", headers={"X-Siftlode-Account": str(alice.id)})
+    assert me.json()["email"] == "bob4@example.com"

-    assert client.get("/api/me", headers=tab).json()["email"] == "bob4@example.com"
-    assert client.get("/api/feed", headers=tab).status_code == 401  # still nobody, never Bob
-    assert client.get("/api/feed").status_code == 200  # Bob's session is intact
+
+@pytest.mark.parametrize("path", ["/api/me/accounts", "/api/me/switch"])
+def test_there_is_no_account_list_or_switch(client, make_user, path):
+    make_user(email="alice5@example.com", password=_PW)
+    assert _login(client, "alice5@example.com", _PW).status_code == 200
+    r = client.get(path) if path.endswith("accounts") else client.post(path, json={"user_id": 1})
+    assert r.status_code in (404, 405)


@pytest.mark.parametrize("ended", ["revoked", "suspended", "deactivated"])
-def test_switching_to_an_account_whose_session_ended_refuses_it_and_drops_it_from_the_list(
-    client, make_user, db, monkeypatch, ended
-):
-    # The account switcher lists this browser's wallet. A revoked account was dropped from it on a refused
-    # switch, but a suspended one stayed, refused on every click; a deactivated one was not refused at all —
-    # made the default, the next request found it inactive and signed every account here out.
-    monkeypatch.setattr("app.routes.me.is_allowed", lambda _db, _email: True)  # both hold an invite
-    alice = make_user(email="alice5@example.com", password=_PW)
-    make_user(email="bob5@example.com", password=_PW)
-    assert _login(client, "alice5@example.com", _PW).status_code == 200
-    assert _login(client, "bob5@example.com", _PW).status_code == 200
+def test_a_session_whose_account_ended_signs_the_browser_out(client, make_user, db, ended):
+    alice = make_user(email="alice6@example.com", password=_PW)
+    assert _login(client, "alice6@example.com", _PW).status_code == 200
    if ended == "revoked":
        alice.session_epoch = (alice.session_epoch or 0) + 1
    elif ended == "suspended":
@@ -121,10 +99,111 @@ def test_switching_to_an_account_whose_session_ended_refuses_it_and_drops_it_fro
    else:
        alice.is_active = False
    db.commit()
+    assert client.get("/api/feed").status_code == 401
+    assert "user_id" not in _session(client)  # the refusal ended the session, not just this request
+    assert client.get("/api/me").json() == {"authenticated": False}

-    assert client.post("/api/me/switch", json={"user_id": alice.id}).status_code == 403
-    assert [a["email"] for a in client.get("/api/me/accounts").json()] == ["bob5@example.com"]
-    assert client.get("/api/feed").status_code == 200  # Bob is still signed in
+
+def test_a_cookie_from_the_multi_account_build_signs_in_again(client, make_user):
+    # The old session held a wallet and one epoch per account. It is not read: whoever holds one
+    # signs in once more.
+    alice = make_user(email="alice7@example.com", password=_PW)
+    old = {"user_id": alice.id, "account_ids": [alice.id], "epochs": {str(alice.id): 0}}
+    client.cookies.set("session", _cookie(old))
+    assert client.get("/api/feed").status_code == 401
+    assert client.get("/api/me").json() == {"authenticated": False}
+
+
+def test_deleting_the_account_signs_the_browser_out(client, make_user):
+    make_user(email="alice8@example.com", password=_PW)
+    assert _login(client, "alice8@example.com", _PW).status_code == 200
+    assert client.delete("/api/me/account").status_code == 200
+    # The cookie names nobody now: the probe answers signed out without looking the account up.
+    assert "user_id" not in _session(client)
+
+
+def _session(client) -> dict:
+    """The decoded payload of the client's session cookie."""
+    raw = client.cookies.get("session")
+    if not raw:
+        return {}
+    data = TimestampSigner(str(settings.secret_key)).unsign(raw.encode())
+    return json.loads(base64.b64decode(data))
+
+
+@pytest.mark.parametrize("signed_in_as", ["starter", "someone_else"])
+def test_a_google_link_lands_only_on_the_account_still_signed_in(db, make_user, signed_in_as):
+    # The link marker names who started it; a sign-in as someone else before the callback must not
+    # get that Google identity attached to the account that started it from the other person's session.
+    from starlette.requests import Request
+
+    from app.routes.auth import _complete_link
+
+    starter, other = make_user(), make_user()
+    session = {"user_id": starter.id if signed_in_as == "starter" else other.id, "epoch": 0}
+    request = Request({"type": "http", "session": session, "headers": []})
+    resp = _complete_link(request, db, starter.id, True, "sub-123", {"email": ""}, {"scope": ""})
+    db.refresh(starter)
+    if signed_in_as == "starter":
+        assert resp.headers["location"] == "/?link=ok"
+        assert starter.google_sub == "sub-123"
+    else:
+        assert resp.headers["location"] == "/?link=error"
+        assert starter.google_sub is None
+
+
+def test_no_app_code_reads_a_second_account():
+    # The multi-account machinery is gone; a module reading its session keys or its header again
+    # would bring back a session that names more than one account.
+    app_dir = Path(__file__).parent.parent / "app"
+    hits = [
+        f"{path.relative_to(app_dir)}: {word}"
+        for path in app_dir.rglob("*.py")
+        for word in ("account_ids", '"epochs"', "x-siftlode-account", 'query_params.get("account")')
+        if word in path.read_text(encoding="utf-8").lower()
+    ]
+    assert hits == []
+
+
+def test_a_tab_built_for_the_account_signed_in_before_is_refused(client, make_user):
+    # Two tabs of one browser: one signed in as someone else, so the cookie is Bob's under both. The
+    # tab still showing Alice says so with every request; acting as Bob there would put her player
+    # positions, pref saves and Sign out on his account. Refused, and the session stays Bob's.
+    alice = make_user(email="alice5@example.com", password=_PW)
+    bob = make_user(email="bob5@example.com", password=_PW)
+    assert _login(client, "alice5@example.com", _PW).status_code == 200
+    assert _login(client, "bob5@example.com", _PW).status_code == 200
+
+    stale = client.get("/api/feed", headers={"X-Siftlode-Expect-Account": str(alice.id)})
+    assert stale.status_code == 401
+    assert stale.json()["detail"]["reason"] == "account_changed"
+    assert (
+        client.get("/api/feed", headers={"X-Siftlode-Expect-Account": str(bob.id)}).status_code
+        == 200
+    )
+    assert client.get("/api/feed").status_code == 200
+    # Nor may its Sign out end Bob's session.
+    assert (
+        client.post(
+            "/auth/logout", headers={"X-Siftlode-Expect-Account": str(alice.id)}
+        ).status_code
+        == 401
+    )
+    # The probe answers who is signed in now, so the stale tab can reload as Bob.
+    probe = client.get("/api/me", headers={"X-Siftlode-Expect-Account": str(alice.id)})
+    assert probe.json()["email"] == "bob5@example.com"
+
+
+def test_setting_a_password_keeps_the_caller_signed_in(client, make_user):
+    # The change ends every OTHER session (the epoch moves); the caller is stamped with the new one.
+    make_user(email="dave@example.com", password=_PW)
+    assert _login(client, "dave@example.com", _PW).status_code == 200
+    r = client.post(
+        "/auth/set-password",
+        json={"password": "a brand new passphrase", "current_password": _PW},
+    )
+    assert r.status_code == 200
+    assert client.get("/api/feed").status_code == 200


 def test_logout_others_revokes_the_other_session_but_keeps_the_caller(client, make_user):
@@ -8,6 +8,8 @@ drops an unknown id, so the view silently showed the whole feed under its old na
 from sqlalchemy import select

 from app.catalog.models import Channel
+from app.db import SessionLocal
+from app.sync.autotag import run_autotag_all
 from app.reader.models import ChannelTag, SavedView, Tag
 from app.reader import service as reader_service

@@ -66,3 +68,15 @@ def test_a_view_kept_tag_list_larger_than_a_bind_parameter_limit_still_cleans_up
    db.commit()
    assert reader_service.prune_orphan_system_tags(db) == 1
    db.commit()
+
+
+def test_the_job_commits_its_cleanup(db):
+    """The prune only flushes; the job ends its own transaction. Read from a session of its own,
+    the orphan is gone only once the job committed — the job's session would see its own delete
+    either way."""
+    orphan = Tag(user_id=None, name="Cooking", category="topic")
+    db.add(orphan)
+    db.commit()
+    assert run_autotag_all(db, only_missing=True)["orphan_tags_removed"] == 1
+    with SessionLocal() as fresh:
+        assert fresh.execute(select(Tag).where(Tag.name == "Cooking")).first() is None
@@ -0,0 +1,69 @@
+"""Every process the app runs as sees every table before its first query.
+
+The units declare their tables in their own `models` modules, and a core model names a unit's by
+string (`User.subscriptions` → `Subscription`). A fresh interpreter that never imported the unit's
+module fails to configure its mappers on its first query: the download child once did exactly that,
+and every job failed, while this suite stayed green — it imports `app.tables` in-process, which hides
+the gap. So each entry point is imported here in a clean interpreter, and must register the whole
+schema on its own.
+"""
+
+import ast
+import json
+import subprocess
+import sys
+from pathlib import Path
+
+import app.tables  # noqa: F401
+from app.db import Base
+
+BACKEND = Path(__file__).parent.parent
+APP = BACKEND / "app"
+
+# Each `python -m` module the deployment starts, and the API's app module.
+ENTRY_POINTS = ("app.main", "app.worker", "app.downloads.runner")
+
+# The table list goes out on one marked line: the API's logging writes to the same stdout, and a
+# line it logs while importing is not the probe's answer.
+_MARK = "entry-point-tables:"
+_PROBE = (
+    "import importlib, json, sys\n"
+    "importlib.import_module(sys.argv[1])\n"
+    "from sqlalchemy.orm import configure_mappers\n"
+    "from app.db import Base\n"
+    "configure_mappers()\n"
+    f"print({_MARK!r} + json.dumps(sorted(Base.metadata.tables)))\n"
+)
+
+
+def test_each_entry_point_registers_the_whole_schema_on_its_own():
+    whole = sorted(Base.metadata.tables)
+    for module in ENTRY_POINTS:
+        r = subprocess.run(
+            [sys.executable, "-c", _PROBE, module],
+            cwd=BACKEND,
+            capture_output=True,
+            text=True,
+            timeout=120,
+            check=False,
+        )
+        assert r.returncode == 0, (module, r.stderr[-2000:])
+        answer = [ln for ln in r.stdout.splitlines() if ln.startswith(_MARK)]
+        assert len(answer) == 1, (module, r.stdout[-2000:])
+        assert json.loads(answer[0].removeprefix(_MARK)) == whole, module
+
+
+def test_every_module_that_runs_as_a_process_is_listed():
+    mains = set()
+    for path in APP.rglob("*.py"):
+        tree = ast.parse(path.read_text(encoding="utf-8"))
+        for node in tree.body:
+            if (
+                isinstance(node, ast.If)
+                and isinstance(node.test, ast.Compare)
+                and isinstance(node.test.left, ast.Name)
+                and node.test.left.id == "__name__"
+            ):
+                rel = path.relative_to(BACKEND).with_suffix("").as_posix()
+                mains.add(rel.replace("/", "."))
+    assert mains <= set(ENTRY_POINTS), mains - set(ENTRY_POINTS)
@@ -124,6 +124,9 @@ def test_only_an_own_folder_is_renamed_and_only_by_its_user(db, make_user, as_us
    assert clash.json()["detail"]["reason"] == "file_folder_name_taken"
    no = c.patch(f"/api/files/folders/{src.id}", json={"name": "YouTube 2"})
    assert no.status_code == 409 and no.json()["detail"]["reason"] == "file_folder_not_own"
+    # Which folder is answered before what the name is: a blank one is still "not yours to name".
+    blank = c.patch(f"/api/files/folders/{src.id}", json={"name": " "})
+    assert blank.status_code == 409 and blank.json()["detail"]["reason"] == "file_folder_not_own"
    stranger = as_user(make_user()).patch(f"/api/files/folders/{own.id}", json={"name": "X"})
    assert stranger.status_code == 404
    db.refresh(own)
@@ -7,6 +7,7 @@ the refused sign-in that doubles as an access request, and the link flow's confl
 """

 import pytest
+from authlib.integrations.starlette_client import OAuthError
 from sqlalchemy import select
 from starlette.responses import RedirectResponse

@@ -19,15 +20,20 @@ _PW = "correct horse battery"

 class _FakeGoogle:
    """authlib's client, reduced to the two calls the routes make: the redirect out to Google and
-    the token exchange when Google redirects back."""
+    the token exchange when Google redirects back. Like authlib, it honours only the newest flow's
+    state: starting a flow retires every earlier one."""

    def __init__(self, userinfo: dict):
        self.userinfo = userinfo
+        self.asked: dict = {}

-    async def authorize_redirect(self, _request, _redirect_uri, **_kw):
+    async def authorize_redirect(self, _request, _redirect_uri, **kw):
+        self.asked = kw
        return RedirectResponse(url="/fake-google")

-    async def authorize_access_token(self, _request):
+    async def authorize_access_token(self, request):
+        if request.query_params.get("state") != self.asked.get("state"):
+            raise OAuthError(error="mismatching_state")
        return {"userinfo": self.userinfo, "access_token": "at", "scope": "openid email profile"}


@@ -44,9 +50,13 @@ def google(monkeypatch):
    return fake


-def _sign_in(client, google, **userinfo):
+def _sign_in(client, google, state: str | None = None, **userinfo):
+    """Google redirecting back: by default with the state of the flow started last."""
    google.userinfo = {"email_verified": True, **userinfo}
-    return client.get("/auth/callback", follow_redirects=False)
+    state = state or google.asked.get("state")
+    return client.get(
+        "/auth/callback", params={"state": state} if state else {}, follow_redirects=False
+    )


 def _rows(db, model, column, value):
@@ -218,12 +228,20 @@ def test_a_refused_sign_in_records_a_request_in_the_reported_language(
    assert _rows(db, User, User.email, "new@example.com") == []


-def _start_link(client, email: str):
+def _password_login(client, email: str):
    assert (
        client.post("/auth/password-login", json={"email": email, "password": _PW}).status_code
        == 200
    )
-    assert client.get("/auth/link", follow_redirects=False).headers["location"] == "/fake-google"
+
+
+def _connect(client, access: str):
+    return client.get(f"/auth/google/connect?access={access}", follow_redirects=False)
+
+
+def _start_link(client, email: str):
+    _password_login(client, email)
+    assert _connect(client, "identity").headers["location"] == "/fake-google"


 def test_linking_an_identity_another_account_holds_is_a_conflict(client, db, google, make_user):
@@ -258,3 +276,151 @@ def test_linking_a_free_identity_attaches_it(client, db, google, make_user):
    assert r.headers["location"] == "/?link=ok"
    db.refresh(me)
    assert (me.google_sub, me.google_email) == ("sub-free", "me3.google@example.com")
+
+
+def test_a_connect_abandoned_at_google_leaves_the_next_sign_in_a_sign_in(
+    client, db, google, make_user, monkeypatch
+):
+    # A connect left at Google's screen (Back, a closed tab) never reaches the callback that pops
+    # its marker. "Sign in as someone else" afterwards must sign that person in, not link their
+    # Google identity onto the account still signed in here.
+    me = make_user(email="me4@example.com")
+    make_user(email="them4@example.com", google_sub="sub-them4", password=None)
+    monkeypatch.setattr(settings, "allowed_emails", "them4@example.com")
+    _start_link(client, "me4@example.com")
+    assert client.get("/auth/login", follow_redirects=False).headers["location"] == "/fake-google"
+
+    r = _sign_in(client, google, sub="sub-them4", email="them4@example.com")
+
+    assert r.headers["location"] == "/"
+    assert client.get("/api/me").json()["email"] == "them4@example.com"
+    db.refresh(me)
+    assert me.google_sub is None
+
+
+def test_a_link_whose_session_was_revoked_meanwhile_does_not_finish(client, db, google, make_user):
+    # "Log out everywhere" (or a password change) between the start and the callback: the cookie
+    # still names the account, but its session no longer holds, so it may not attach an identity.
+    me = make_user(email="me5@example.com")
+    _start_link(client, "me5@example.com")
+    me.session_epoch = (me.session_epoch or 0) + 1
+    db.commit()
+
+    r = _sign_in(client, google, sub="sub-late", email="late@example.com")
+
+    assert r.headers["location"] == "/?link=error"
+    db.refresh(me)
+    assert me.google_sub is None
+
+
+def test_an_older_flow_coming_back_leaves_the_connect_in_flight_a_link(
+    client, db, google, make_user
+):
+    # Two round-trips from one browser: an older one (another tab's, or Back to Google's screen)
+    # returns after a connect started. Its state is refused, and it must not spend the connect's
+    # marker: the connect itself, coming back, would then land as a plain sign-in.
+    me = make_user(email="me6@example.com")
+    _start_link(client, "me6@example.com")
+
+    older = _sign_in(client, google, state="an-older-flow", sub="sub-6", email="g6@example.com")
+    assert older.headers["location"] == "/?oauth=cancelled"
+    r = _sign_in(client, google, sub="sub-6", email="g6@example.com")
+
+    assert r.headers["location"] == "/?link=ok"
+    db.refresh(me)
+    assert me.google_sub == "sub-6"
+
+
+def test_a_connect_from_a_tab_showing_another_account_starts_nothing(client, google, make_user):
+    # A navigation carries no headers, so the tab names the account it was built for in the query.
+    # One still showing the account signed in before must not start a link onto the one signed in
+    # now: it lands on the app, which boots as that one and says why.
+    me = make_user(email="me7@example.com")
+    _password_login(client, "me7@example.com")
+
+    r = client.get(
+        f"/auth/google/connect?access=identity&expect={me.id + 1}", follow_redirects=False
+    )
+
+    assert r.headers["location"] == auth_routes.ACCOUNT_CHANGED_LANDING == "/?account=changed"
+    assert google.asked == {}
+    assert client.get("/api/me").json()["email"] == "me7@example.com"
+
+
+def test_a_connect_naming_the_signed_in_account_goes_to_google(client, google, make_user):
+    me = make_user(email="me8@example.com")
+    _password_login(client, "me8@example.com")
+
+    r = client.get(f"/auth/google/connect?access=identity&expect={me.id}", follow_redirects=False)
+
+    assert r.headers["location"] == "/fake-google"
+    assert google.asked["state"]
+
+
+@pytest.mark.parametrize(
+    ("access", "scope", "prompt"),
+    [
+        ("identity", "openid email profile", "select_account"),
+        ("read", auth_routes.READ_SCOPES, "consent"),
+        ("write", auth_routes.WRITE_SCOPES, "consent"),
+    ],
+)
+def test_one_connect_route_asks_google_for_what_the_access_needs(
+    client, google, make_user, access, scope, prompt
+):
+    make_user(email=f"ask-{access}@example.com")
+    _password_login(client, f"ask-{access}@example.com")
+
+    assert _connect(client, access).headers["location"] == "/fake-google"
+
+    assert (google.asked["scope"], google.asked["prompt"]) == (scope, prompt)
+    assert google.asked["access_type"] == "offline"
+
+
+@pytest.mark.parametrize("access", ["read", "write"])
+def test_a_youtube_grant_attaches_to_the_password_account_and_returns_to_the_app(
+    client, db, google, make_user, access
+):
+    me = make_user(email=f"grant-{access}@example.com")
+    _password_login(client, f"grant-{access}@example.com")
+    _connect(client, access)
+
+    r = _sign_in(client, google, sub=f"sub-grant-{access}", email=f"g-{access}@example.com")
+
+    # Back to the app, not the "linked" notice: the onboarding wizard picks up where it was.
+    assert r.headers["location"] == "/"
+    db.refresh(me)
+    assert me.google_sub == f"sub-grant-{access}"
+    assert _rows(db, User, User.google_sub, f"sub-grant-{access}") == [me]
+
+
+@pytest.mark.parametrize("access", ["", "admin", "READ"])
+def test_an_unknown_access_starts_no_connect(client, google, make_user, access):
+    make_user(email="odd@example.com")
+    _password_login(client, "odd@example.com")
+
+    assert _connect(client, access).status_code == 422
+    assert google.asked == {}
+
+
+def test_a_connect_needs_a_signed_in_account(client, google):
+    assert _connect(client, "identity").status_code == 401
+    assert google.asked == {}
+
+
+@pytest.mark.parametrize("access", ["identity", "read", "write"])
+def test_without_google_configured_a_connect_says_it_failed(client, make_user, monkeypatch, access):
+    monkeypatch.setattr(auth_routes, "google_oauth", lambda _db: None)
+    make_user(email=f"nog-{access}@example.com")
+    _password_login(client, f"nog-{access}@example.com")
+
+    assert _connect(client, access).headers["location"] == "/?link=error"
+
+
+@pytest.mark.parametrize("path", ["/auth/link", "/auth/upgrade", "/auth/upgrade?access=write"])
+def test_the_old_connect_routes_are_gone(client, google, make_user, path):
+    make_user(email="old@example.com")
+    _password_login(client, "old@example.com")
+
+    assert client.get(path, follow_redirects=False).status_code == 404
+    assert google.asked == {}
@@ -0,0 +1,74 @@
+"""PUT /api/me/preferences merges the keys it is sent into the stored ones, DB-lane.
+
+A client saves its preferences a key at a time, and several at once on the boot that moves this
+browser's copies to the account. Each merge must start from the row as stored when it runs, not as
+the request first read it — two merges over the same earlier read keep only the last one's keys.
+"""
+
+from fastapi import Depends, Request
+from sqlalchemy import event, update
+from sqlalchemy.engine import Engine
+from sqlalchemy.orm import Session
+
+from app.auth import current_user
+from app.db import SessionLocal, get_db
+from app.models import User
+
+_PW = "correct horse battery"
+
+
+def test_a_save_keeps_a_key_another_save_stored_after_this_one_read_the_account(
+    client, db, make_user, override_dep
+):
+    user = make_user(email="prefs@example.com", password=_PW)
+    assert (
+        client.post("/auth/password-login", json={"email": "prefs@example.com", "password": _PW})
+    ).status_code == 200
+
+    def read_then_another_save(request: Request, session: Session = Depends(get_db)) -> User:
+        # The account as this request reads it, then a second save committing in the meantime.
+        caller = current_user(request, session)
+        with SessionLocal() as other:
+            other.execute(
+                update(User)
+                .where(User.id == caller.id)
+                .values(preferences={"seenVersion": "1.2.3"})
+            )
+            other.commit()
+        return caller
+
+    override_dep(current_user, read_then_another_save)
+    r = client.put("/api/me/preferences", json={"startOnHome": False})
+
+    assert r.status_code == 200
+    db.expire_all()
+    assert db.get(User, user.id).preferences == {"seenVersion": "1.2.3", "startOnHome": False}
+
+
+def test_the_merge_reads_the_account_under_a_lock_that_orders_merges_and_nothing_else(
+    client, make_user
+):
+    # The re-read above is half of it: two merges that both read before either commits would still
+    # keep only the last, unless the read takes a lock the other merge waits for. FOR NO KEY UPDATE
+    # is that lock, and the weakest one: a plain FOR UPDATE would also wait behind any transaction
+    # that only added a row referring to the account (a playlist pull, then minutes of YouTube).
+    make_user(email="prefs-lock@example.com", password=_PW)
+    assert (
+        client.post(
+            "/auth/password-login", json={"email": "prefs-lock@example.com", "password": _PW}
+        )
+    ).status_code == 200
+    sent: list[str] = []
+
+    def record(_conn, _cursor, statement, *_rest):
+        sent.append(statement)
+
+    event.listen(Engine, "before_cursor_execute", record)
+    try:
+        assert client.put("/api/me/preferences", json={"startOnHome": False}).status_code == 200
+    finally:
+        event.remove(Engine, "before_cursor_execute", record)
+
+    locks = [s for s in sent if "FROM users" in s and " FOR " in s]
+    assert locks, sent
+    assert all(s.rstrip().endswith("FOR NO KEY UPDATE") for s in locks), locks
@@ -0,0 +1,90 @@
+"""Migration 0112's data step, DB-lane: a user who already has two Watch later lists keeps the oldest,
+with the other's videos moved in after its own (one it already held is not added twice) and a Home
+pin of the other moved onto it, and the one-per-user index then refuses a second list.
+
+The schema half is checked by test_migrations.py; a video or a pin lost on the way is invisible there.
+"""
+
+from datetime import datetime, timezone
+
+import pytest
+from alembic import command
+from alembic.config import Config
+from sqlalchemy import text
+from sqlalchemy.exc import IntegrityError
+
+from app.catalog.models import Channel, Video
+from app.db import engine
+
+
+def _lists(conn, user_id: int, n: int) -> list[int]:
+    values = ", ".join(f"(:u, 'Watch later', 'watch_later', {i})" for i in range(n))
+    return list(
+        conn.execute(
+            text(
+                f"INSERT INTO playlists (user_id, name, kind, position) VALUES {values} RETURNING id"
+            ),
+            {"u": user_id},
+        ).scalars()
+    )
+
+
+def _pin(conn, user_id: int, playlist_id: int, position: int) -> None:
+    conn.execute(
+        text("INSERT INTO comfort_pins (user_id, playlist_id, position) VALUES (:u, :p, :pos)"),
+        {"u": user_id, "p": playlist_id, "pos": position},
+    )
+
+
+def test_a_second_watch_later_is_folded_into_the_first(db, make_user):
+    user, other = make_user(), make_user()
+    db.add(Channel(id="UCm", title="M"))
+    now = datetime.now(timezone.utc)
+    for v in ("v1", "v2", "v3"):
+        db.add(Video(id=v, channel_id="UCm", published_at=now, duration_seconds=60))
+    db.commit()
+    cfg = Config("alembic.ini")
+    command.downgrade(cfg, "0111_audit_request_id")
+    try:
+        with engine.begin() as conn:
+            first, second, third = _lists(conn, user.id, 3)
+            theirs, their_second = _lists(conn, other.id, 2)
+            conn.execute(
+                text(
+                    "INSERT INTO playlist_items (playlist_id, video_id, position) VALUES"
+                    " (:a, 'v1', 0), (:b, 'v2', 0), (:b, 'v1', 1), (:c, 'v3', 0)"
+                ),
+                {"a": first, "b": second, "c": theirs},
+            )
+            # The user pinned two of the copies, the kept one neither: the first on the shelf moves.
+            _pin(conn, user.id, third, 0)
+            _pin(conn, user.id, second, 1)
+            # The other pinned both: the kept one's pin stays, and is the only one.
+            _pin(conn, other.id, their_second, 0)
+            _pin(conn, other.id, theirs, 1)
+    finally:
+        command.upgrade(cfg, "head")
+
+    with engine.connect() as conn:
+        lists = conn.execute(
+            text("SELECT id, user_id FROM playlists WHERE kind = 'watch_later' ORDER BY id")
+        ).all()
+        assert [tuple(r) for r in lists] == [(first, user.id), (theirs, other.id)]
+        items = conn.execute(
+            text(
+                "SELECT playlist_id, video_id, position FROM playlist_items ORDER BY playlist_id, position"
+            )
+        ).all()
+        assert [tuple(r) for r in items] == [(first, "v1", 0), (first, "v2", 1), (theirs, "v3", 0)]
+        pins = conn.execute(
+            text("SELECT user_id, playlist_id, position FROM comfort_pins ORDER BY user_id")
+        ).all()
+        assert [tuple(r) for r in pins] == [(user.id, first, 0), (other.id, theirs, 1)]
+    with pytest.raises(IntegrityError), engine.begin() as conn:
+        conn.execute(
+            text(
+                "INSERT INTO playlists (user_id, name, kind, position)"
+                " VALUES (:u, 'Watch later', 'watch_later', 2)"
+            ),
+            {"u": user.id},
+        )
@@ -82,3 +82,42 @@ def test_a_request_that_raises_leaves_nothing_it_did_not_commit(db):
        request.throw(RuntimeError("the handler failed"))
    assert db.scalar(select(User).where(User.email == "rolled-back-probe@example.com")) is None
    assert db.info["unfinished"] == []  # a failing request is rolled back, not reported
+
+
+def test_the_commit_check_sees_a_core_dml_write(db, make_user):
+    """`db.execute(update(...))` flushes nothing and stages nothing, so the session's own bookkeeping
+    never sees it: a handler whose only write is Core DML and that forgot its commit passed. The
+    database's transaction id is what tells — also when the test itself had a write open before the
+    request, which the handler's write would otherwise share. A committed one must still pass."""
+    import pytest
+    from sqlalchemy import update
+    from starlette.requests import HTTPConnection
+
+    from app.models import User
+
+    user = make_user()
+
+    def drive(commit: bool, *, test_wrote: bool = False) -> None:
+        if test_wrote:  # the test's own write, open and uncommitted, as the request starts
+            db.execute(update(User).where(User.id == user.id).values(display_name="test"))
+        conn = HTTPConnection(
+            {"type": "http", "method": "POST", "path": "/probe", "headers": [], "query_string": b""}
+        )
+        request = db.info["request"](conn)
+        session = next(request)
+        session.execute(update(User).where(User.id == user.id).values(display_name="dml"))
+        if commit:
+            session.commit()
+        with pytest.raises(StopIteration):
+            next(request)
+
+    drive(commit=True)
+    assert db.info["unfinished"] == []
+    drive(commit=False)
+    assert db.info["unfinished"] == ["POST /probe"]
+    db.info["unfinished"].clear()  # recorded as it should be — not this test's failure
+    drive(commit=True, test_wrote=True)
+    assert db.info["unfinished"] == []
+    drive(commit=False, test_wrote=True)
+    assert db.info["unfinished"] == ["POST /probe"]
+    db.info["unfinished"].clear()
@@ -10,7 +10,10 @@ from datetime import datetime, timedelta, timezone
 from sqlalchemy import func, select

 from app import debugsession, notifications, quota
+from app.catalog.models import Channel, Video
+from app.downloads.models import DownloadJob, DownloadProfile
 from app.models import DebugScreenshot, DebugSession, Notification, QuotaEvent
+from app.reader.models import Playlist, PlaylistItem, VideoState
 from app.routes import tokens as tokens_routes
 from app.routes.auth import RESET_TTL, _issue_token

@@ -133,3 +136,113 @@ def test_screenshot_storage_counts_only_mine(db, make_user):
        )
    db.commit()
    assert debugsession.stored_screenshot_bytes(db, me.id) == 10
+
+
+# --- the units' owner filters, through their routes --------------------------------------------
+# The same rule one layer up: a second account acting on the SAME video, list or profile id must
+# reach only its own row. Each test holds the first account's row unchanged afterwards.
+
+
+def _videos(db, *ids):
+    db.add(Channel(id="UCo", title="O"))
+    db.flush()
+    for vid in ids:
+        db.add(
+            Video(
+                id=vid,
+                channel_id="UCo",
+                published_at=datetime.now(timezone.utc),
+                duration_seconds=3600,
+            )
+        )
+    db.commit()
+
+
+def _video_state(db, user, vid="v1"):
+    db.expire_all()
+    row = db.execute(
+        select(VideoState).where(VideoState.user_id == user.id, VideoState.video_id == vid)
+    ).scalar_one_or_none()
+    return None if row is None else (row.status, row.position_seconds)
+
+
+def test_unmarking_a_video_leaves_another_users_state(db, make_user, as_user):
+    me, other = make_user(), make_user()
+    _videos(db, "v1")
+    db.add(VideoState(user_id=other.id, video_id="v1", status="watched"))
+    db.commit()
+    r = as_user(me).post("/api/videos/v1/state", json={"status": "new"})
+    assert r.status_code == 200, r.text
+    assert _video_state(db, other) == ("watched", 0)
+
+
+def test_clearing_a_video_leaves_another_users_state(db, make_user, as_user):
+    me, other = make_user(), make_user()
+    _videos(db, "v1")
+    db.add(VideoState(user_id=other.id, video_id="v1", status="hidden", position_seconds=90))
+    db.commit()
+    assert as_user(me).delete("/api/videos/v1/state").status_code == 200
+    assert _video_state(db, other) == ("hidden", 90)
+
+
+def test_a_resume_point_is_set_and_cleared_on_my_own_row_only(db, make_user, as_user):
+    me, other = make_user(), make_user()
+    _videos(db, "v1")
+    db.add(VideoState(user_id=other.id, video_id="v1", status="hidden", position_seconds=90))
+    db.commit()
+    client = as_user(me)
+    body = {"position_seconds": 600, "duration_seconds": 3600}
+    assert client.post("/api/videos/v1/progress", json=body).status_code == 200
+    assert (_video_state(db, me), _video_state(db, other)) == (("new", 600), ("hidden", 90))
+    body = {"position_seconds": 0, "duration_seconds": 3600}
+    assert client.post("/api/videos/v1/progress", json=body).status_code == 200
+    assert (_video_state(db, me), _video_state(db, other)) == (None, ("hidden", 90))
+
+
+def _watch_later_ids(db, user):
+    db.expire_all()
+    return sorted(
+        db.execute(
+            select(PlaylistItem.video_id)
+            .join(Playlist, Playlist.id == PlaylistItem.playlist_id)
+            .where(Playlist.user_id == user.id, Playlist.kind == "watch_later")
+        ).scalars()
+    )
+
+
+def test_watch_later_adds_and_removes_use_my_own_list(db, make_user, as_user):
+    me, other = make_user(), make_user()
+    _videos(db, "v1", "v2")
+    theirs = as_user(other).post("/api/playlists/watch-later", json={"video_id": "v1"}).json()
+    client = as_user(me)
+    mine = client.post("/api/playlists/watch-later", json={"video_id": "v2"}).json()
+    assert mine["playlist_id"] != theirs["playlist_id"]
+    # Their saved video is not on my list, so removing it from mine leaves theirs alone.
+    assert client.delete("/api/playlists/watch-later/v1").status_code == 200
+    assert (_watch_later_ids(db, me), _watch_later_ids(db, other)) == (["v2"], ["v1"])
+
+
+def test_another_users_private_profile_is_refused_like_a_missing_one(db, make_user, as_user):
+    me, other = make_user(), make_user()
+    theirs = DownloadProfile(user_id=other.id, name="Theirs", spec={"format": "best"})
+    builtin = DownloadProfile(user_id=None, name="Best", spec={"format": "best"}, is_builtin=True)
+    db.add_all([theirs, builtin])
+    db.commit()
+    client = as_user(me)
+
+    def enqueue(profile_id):
+        return client.post(
+            "/api/downloads", json={"source": "dQw4w9WgXcQ", "profile_id": profile_id}
+        )
+
+    missing = enqueue(theirs.id + builtin.id + 1)
+    refused = enqueue(theirs.id)
+    assert (refused.status_code, refused.json()) == (404, missing.json())
+    assert refused.json()["detail"] == {"code": "not_found", "reason": "profile"}
+    db.expire_all()
+    assert db.execute(select(DownloadJob)).first() is None
+    # A built-in is everyone's to use.
+    r = enqueue(builtin.id)
+    assert r.status_code == 200, r.text
+    db.expire_all()
+    assert db.execute(select(DownloadJob.profile_id)).scalars().all() == [builtin.id]
@@ -48,7 +48,7 @@ def _setup(db, make_user, monkeypatch, tmp_path) -> TestClient:
    )
    db.commit()
    monkeypatch.setattr(app_state, "_configured_cache", True)
-    monkeypatch.setattr(playback, "resolved_user_id", lambda request: (user.id, True))
+    monkeypatch.setattr(playback, "session_user_id", lambda request: user.id)
    monkeypatch.setattr(playback, "PlexClient", _FakePlex)
    monkeypatch.setattr(imgcache, "CACHE_DIR", tmp_path)
    _FakePlex.calls = []
@@ -12,7 +12,7 @@ from datetime import datetime, timezone

 from sqlalchemy import select

-from app.plex import library, queries, service
+from app.plex import library, queries, rollup, service
 from app.plex.models import PlexCollection, PlexItem, PlexLibrary, PlexShow, PlexState
 from app.plex.query import LibraryFilters

@@ -71,9 +71,9 @@ def test_a_users_watch_state_is_theirs_alone(db, make_user):
    _state(db, owner, it, status="watched")
    assert queries.state(db, owner, it.id).status == "watched"
    assert queries.state(db, stranger, it.id) is None
-    assert queries.states(db, stranger, [it.id]) == {}
-    assert set(queries.states(db, owner, [it.id])) == {it.id}
-    assert queries.states(db, owner, []) == {}
+    assert rollup.item_state_map(db, stranger, [it.id]) == {}
+    assert set(rollup.item_state_map(db, owner, [it.id])) == {it.id}
+    assert rollup.item_state_map(db, owner, []) == {}


 def test_the_grid_shows_the_callers_state_and_hides_only_what_they_hid(db, make_user):
@@ -421,3 +421,32 @@ def test_only_the_owners_own_link_pushes_their_watch_state(db, make_user):
    db.commit()
    assert watch_sync.link_for_push(db, owner.id).id == link.id
    assert watch_sync.link_for_push(db, other.id) is None
+
+
+def test_a_first_write_that_lost_the_race_updates_the_winners_row(db, make_user):
+    """The first checkpoint of an item and the `final` flush beside it both read "no state"; the one
+    that writes second must land on the row the first made, not fail on `uq_plex_user_item`."""
+    user = make_user()
+    it = _movie(db, _lib(db), "m-race")
+    _state(db, user, it, position_seconds=10)  # the request that won, committed
+    st = service.checkpoint(db, user, it.id, None, 42, NOW, dirty=True)  # read "none" before it
+    assert (st.position_seconds, st.synced_to_plex) == (42, False)
+    st = service.set_status(db, user, it.id, None, "hidden")
+    assert st.status == "hidden"
+    db.commit()
+    rows = db.scalars(select(PlexState).where(PlexState.item_id == it.id)).all()
+    assert [(r.position_seconds, r.status) for r in rows] == [(42, "hidden")]
+
+
+def test_a_season_marked_beside_a_first_checkpoint_updates_its_row(db, make_user, monkeypatch):
+    """Marking a season reads every episode's state once; a first checkpoint that made one row since
+    must be updated, not collided with on `uq_plex_user_item`."""
+    user = make_user()
+    lib = _lib(db)
+    ep = _movie(db, lib, "e-race")
+    _state(db, user, ep, position_seconds=10)  # the checkpoint that won, committed
+    monkeypatch.setattr(rollup, "item_state_map", lambda *a, **k: {})  # read before it committed
+    service.bulk_state(db, user.id, [ep], watched=True)
+    db.commit()
+    rows = db.scalars(select(PlexState).where(PlexState.item_id == ep.id)).all()
+    assert [(r.status, r.position_seconds) for r in rows] == [("watched", 0)]
@@ -32,7 +32,7 @@ def test_a_suspended_account_cannot_drive_the_subtitle_extractor(
    monkeypatch.setattr(app_state, "_configured_cache", True)
    # A validly SIGNED cookie for that account — which is all the old gate looked at.
    for mod in (auth, playback):
-        monkeypatch.setattr(mod, "resolved_user_id", lambda request: (user.id, True))
+        monkeypatch.setattr(mod, "session_user_id", lambda request: user.id)
    monkeypatch.setattr(subtitles, "CACHE_DIR", tmp_path)
    pool = _NoPool()
    monkeypatch.setattr(subtitles, "_WIN_POOL", pool)
@@ -14,6 +14,7 @@ from types import SimpleNamespace
 from sqlalchemy import select

 from app.models import User
+from app.plex import queries
 from app.plex.models import PlexItem, PlexLibrary, PlexState
 from app.plex.watch_sync import _pull_apply, _reconcile_inprogress_movies
 from app.routes.plex.playback import (
@@ -385,3 +386,30 @@ def test_should_push_resume_periodic_is_throttled():
    )
    # Once the window elapses, the next tick pushes again.
    assert _should_push_resume(*key, now + timedelta(seconds=_PUSH_THROTTLE_S), final=False) is True
+
+
+def test_a_pull_that_lost_the_race_keeps_the_local_change_it_met(db, monkeypatch):
+    """The pull read "no state", and a first local checkpoint made one before it wrote: that row is
+    still waiting for its push, so the pull leaves it (the dirty guard), and fails on nothing."""
+    item_id, user_id = _seed_item_and_user(db)
+    now = datetime.now(timezone.utc)
+    db.add(PlexState(user_id=user_id, item_id=item_id, position_seconds=300, synced_to_plex=False))
+    db.commit()
+    real = queries.state
+    stale = [True]
+    monkeypatch.setattr(
+        queries, "state", lambda d, u, i: None if stale and stale.pop() else real(d, u, i)
+    )
+    assert _pull_apply(db, user_id, item_id, "new", 120, plex_ts=now) is None
+    db.commit()
+    rows = db.scalars(select(PlexState).where(PlexState.item_id == item_id)).all()
+    assert [(r.position_seconds, r.synced_to_plex) for r in rows] == [(300, False)]
+
+
+def test_a_first_pull_makes_a_row_in_step_with_plex(db):
+    item_id, user_id = _seed_item_and_user(db)
+    now = datetime.now(timezone.utc)
+    assert _pull_apply(db, user_id, item_id, "new", 120, plex_ts=now) == "in_progress"
+    db.commit()
+    rows = db.scalars(select(PlexState).where(PlexState.item_id == item_id)).all()
+    assert [(r.position_seconds, r.synced_to_plex) for r in rows] == [(120, True)]
@@ -308,3 +308,36 @@ def test_a_pin_is_one_per_thing_and_only_its_owners(db, make_user):
    pins.remove(db, pins.own_pin(db, user, pin.id))
    db.commit()
    assert db.execute(select(ComfortPin)).first() is None
+
+
+def test_a_watch_later_save_that_lost_the_race_uses_the_winners_list(db, make_user, monkeypatch):
+    """The first two saves at once both read "no Watch later"; the second must land in the list the
+    first made, not make one of its own."""
+    user = make_user()
+    won = service.watch_later(db, user)
+    db.commit()
+    real = queries.watch_later
+    stale = [True]  # this request's first read happened before the winner committed
+
+    def read(d, u):
+        return None if stale and stale.pop() else real(d, u)
+
+    monkeypatch.setattr(queries, "watch_later", read)
+    assert service.watch_later(db, user).id == won.id
+    db.commit()
+    assert [p.id for p in queries.playlists_for(db, user) if p.kind == "watch_later"] == [won.id]
+
+
+def test_a_first_resume_point_beside_the_watched_mark_lands_on_its_row(db, make_user, monkeypatch):
+    """The first checkpoint of a video and the end-of-playback mark both read "no row"; the
+    checkpoint that writes second must update the mark's row, not fail on `uq_user_video`."""
+    user = make_user()
+    _videos(db, "v1")
+    service.mark(db, user, "v1", "watched")  # the mark that won, committed
+    db.commit()
+    monkeypatch.setattr(queries, "video_state", lambda *a, **k: None)  # read before it committed
+    service.set_progress(db, user, "v1", 90)
+    db.commit()
+    db.expire_all()
+    rows = db.execute(select(VideoState)).scalars().all()
+    assert [(r.status, r.position_seconds) for r in rows] == [("watched", 90)]
@@ -0,0 +1,383 @@
+"""Writes driven through HTTP, each read back: the handler called its unit and committed.
+
+A route's helper only flushes; the handler commits. The `db` fixture fails a test whose request left
+a write uncommitted, so every test here drives one successful write through its route and then
+reads the result back. The read-back holds that the handler called its unit; it runs in the
+request's own session, so it sees a write that was never committed — the commit is the fixture's
+to hold (a test that must see it committed reads from a session of its own). The services' own
+tests hold what each write does; these hold that the route finishes it. Nothing here reaches
+YouTube or Plex: the YouTube client is a stub and no user has a Plex link. Needs the DB.
+"""
+
+from datetime import datetime, timedelta, timezone
+
+import pytest
+from sqlalchemy import select
+
+from app import quota
+from app.auth import require_write_scope
+from app.catalog.models import Channel, ExploredChannel, SearchFind, Subscription, Video
+from app.downloads.models import DownloadJob, DownloadProfile, DownloadShare
+from app.plex.models import PlexItem, PlexLibrary, PlexSeason, PlexShow, PlexState
+from app.db import SessionLocal
+from app.models import Notification
+from app.notifications import READ_SOFT_CAP
+from app.reader.models import SavedView, Tag, VideoState
+from app.routes import channels
+
+NOW = datetime.now(timezone.utc)
+
+
+def _video(db, vid="v1", **kw):
+    if db.get(Channel, "UCr") is None:
+        db.add(Channel(id="UCr", title="R"))
+        db.flush()
+    db.add(Video(id=vid, channel_id="UCr", published_at=NOW, duration_seconds=60, **kw))
+    db.commit()
+
+
+# --- tags ------------------------------------------------------------------------------------
+
+
+def test_a_tag_made_through_the_route_is_listed_back(db, make_user, as_user):
+    user = make_user()
+    client = as_user(user)
+    r = client.post("/api/tags", json={"name": "Science"})
+    assert r.status_code == 200, r.text
+    assert [t["name"] for t in client.get("/api/tags").json()] == ["Science"]
+
+
+def test_a_tag_renamed_through_the_route_is_listed_back(db, make_user, as_user):
+    user = make_user()
+    tag = Tag(user_id=user.id, name="Old")
+    db.add(tag)
+    db.commit()
+    client = as_user(user)
+    assert client.patch(f"/api/tags/{tag.id}", json={"name": "New"}).status_code == 200
+    assert [t["name"] for t in client.get("/api/tags").json()] == ["New"]
+
+
+def test_a_tag_deleted_through_the_route_is_gone(db, make_user, as_user):
+    user = make_user()
+    tag = Tag(user_id=user.id, name="Gone")
+    db.add(tag)
+    db.commit()
+    client = as_user(user)
+    assert client.delete(f"/api/tags/{tag.id}").json() == {"deleted": tag.id}
+    assert client.get("/api/tags").json() == []
+
+
+# --- watch state ------------------------------------------------------------------------------
+
+
+def _state(db, user, vid="v1"):
+    db.expire_all()
+    return db.execute(
+        select(VideoState).where(VideoState.user_id == user.id, VideoState.video_id == vid)
+    ).scalar_one_or_none()
+
+
+def test_a_video_marked_watched_through_the_route_stays_watched(db, make_user, as_user):
+    user = make_user()
+    _video(db)
+    r = as_user(user).post("/api/videos/v1/state", json={"status": "watched"})
+    assert r.status_code == 200, r.text
+    assert _state(db, user).status == "watched"
+
+
+def test_a_video_unmarked_through_the_route_stays_unmarked(db, make_user, as_user):
+    # "new" is its own branch of the handler, with its own commit.
+    user = make_user()
+    _video(db)
+    db.add(VideoState(user_id=user.id, video_id="v1", status="watched"))
+    db.commit()
+    r = as_user(user).post("/api/videos/v1/state", json={"status": "new"})
+    assert r.status_code == 200, r.text
+    assert _state(db, user) is None
+
+
+def test_a_video_state_cleared_through_the_route_is_gone(db, make_user, as_user):
+    user = make_user()
+    _video(db)
+    db.add(VideoState(user_id=user.id, video_id="v1", status="hidden", position_seconds=90))
+    db.commit()
+    assert as_user(user).delete("/api/videos/v1/state").status_code == 200
+    assert _state(db, user) is None
+
+
+# --- saved views ------------------------------------------------------------------------------
+
+
+def test_views_reordered_through_the_route_list_back_in_that_order(db, make_user, as_user):
+    user = make_user()
+    a = SavedView(user_id=user.id, source="feed", name="A", filters={}, position=0)
+    b = SavedView(user_id=user.id, source="feed", name="B", filters={}, position=1)
+    db.add_all([a, b])
+    db.commit()
+    client = as_user(user)
+    assert client.post("/api/saved-views/reorder", json={"ids": [b.id, a.id]}).status_code == 200
+    listed = client.get("/api/saved-views").json()
+    assert [(v["name"], v["position"]) for v in listed] == [("B", 0), ("A", 1)]
+
+
+# --- download profiles and shares -------------------------------------------------------------
+
+_SPEC = {"format": "best"}
+
+
+def _profile(db, user, name="Mine"):
+    p = DownloadProfile(user_id=user.id, name=name, spec=_SPEC, is_builtin=False, sort_order=101)
+    db.add(p)
+    db.commit()
+    return p
+
+
+def _profile_names(client):
+    return [p["name"] for p in client.get("/api/downloads/profiles").json()]
+
+
+def test_a_profile_made_through_the_route_is_listed_back(db, make_user, as_user):
+    user = make_user()
+    client = as_user(user)
+    r = client.post("/api/downloads/profiles", json={"name": "Phone", "spec": _SPEC})
+    assert r.status_code == 200, r.text
+    assert _profile_names(client) == ["Phone"]
+
+
+def test_a_profile_renamed_through_the_route_is_listed_back(db, make_user, as_user):
+    user = make_user()
+    p = _profile(db, user)
+    client = as_user(user)
+    r = client.patch(f"/api/downloads/profiles/{p.id}", json={"name": "Renamed"})
+    assert r.status_code == 200, r.text
+    assert _profile_names(client) == ["Renamed"]
+
+
+def test_a_profile_deleted_through_the_route_is_gone(db, make_user, as_user):
+    user = make_user()
+    p = _profile(db, user)
+    client = as_user(user)
+    assert client.delete(f"/api/downloads/profiles/{p.id}").json() == {"deleted": p.id}
+    assert _profile_names(client) == []
+
+
+def test_a_share_taken_back_through_the_route_is_gone(db, make_user, as_user):
+    owner, friend = make_user(), make_user()
+    job = DownloadJob(
+        user_id=owner.id, source_kind="youtube", source_ref="dQw4w9WgXcQ", profile_snapshot={}
+    )
+    db.add(job)
+    db.flush()
+    db.add(DownloadShare(job_id=job.id, from_user_id=owner.id, to_user_id=friend.id))
+    db.commit()
+    r = as_user(owner).delete(f"/api/downloads/{job.id}/share/{friend.email}")
+    assert r.json() == {"ok": True}
+    db.expire_all()
+    assert db.execute(select(DownloadShare)).first() is None
+
+
+# --- channels ---------------------------------------------------------------------------------
+
+
+class _YouTube:
+    """The YouTube client, answering every write the channel routes make."""
+
+    def __init__(self, db, user):
+        pass
+
+    def __enter__(self):
+        return self
+
+    def __exit__(self, *exc):
+        return False
+
+    def insert_subscription(self, channel_id):
+        return "yt-sub-1"
+
+    def delete_subscription(self, sub_id):
+        return None
+
+    def get_channels(self, ids):
+        return []
+
+
+@pytest.fixture
+def youtube(monkeypatch):
+    monkeypatch.setattr(channels, "YouTubeClient", _YouTube)
+    monkeypatch.setattr(quota, "backfill_reserve_reached", lambda: False)
+
+
+def _channel(db, cid="UCc", **kw):
+    ch = Channel(id=cid, title="C", details_synced_at=NOW, **kw)
+    db.add(ch)
+    db.commit()
+    return ch
+
+
+def _subscription(db, user, cid="UCc"):
+    db.expire_all()
+    return db.execute(
+        select(Subscription).where(Subscription.user_id == user.id, Subscription.channel_id == cid)
+    ).scalar_one_or_none()
+
+
+def test_a_subscribe_through_the_route_is_recorded(db, make_user, as_user, override_dep, youtube):
+    user = make_user()
+    _channel(db)
+    client = as_user(user)
+    override_dep(require_write_scope, lambda: user)
+    assert client.post("/api/channels/UCc/subscribe").json() == {"ok": True, "subscribed": "UCc"}
+    assert _subscription(db, user).yt_subscription_id == "yt-sub-1"
+
+
+def test_an_explore_through_the_route_is_recorded(db, make_user, as_user, youtube, monkeypatch):
+    user = make_user()
+    _channel(db)
+    # The ingest pages YouTube; what the route itself records is the exploration.
+    monkeypatch.setattr(channels, "details_stale", lambda ch, now: False)
+    monkeypatch.setattr(
+        channels,
+        "explore_ingest_page",
+        lambda db, yt, ch, token: {"ingested": 0, "next_page_token": None},
+    )
+    r = as_user(user).post("/api/channels/UCc/explore")
+    assert r.status_code == 200, r.text
+    db.expire_all()
+    assert db.execute(select(ExploredChannel.user_id)).scalars().all() == [user.id]
+    assert db.get(Channel, "UCc").from_explore is True
+
+
+def test_an_unsubscribe_through_the_route_drops_the_subscription(
+    db, make_user, as_user, override_dep, youtube
+):
+    user = make_user()
+    _channel(db)
+    db.add(Subscription(user_id=user.id, channel_id="UCc", yt_subscription_id="yt-sub-1"))
+    db.commit()
+    client = as_user(user)
+    override_dep(require_write_scope, lambda: user)
+    r = client.delete("/api/channels/UCc/subscription")
+    assert r.json() == {"ok": True, "unsubscribed": "UCc"}
+    assert _subscription(db, user) is None
+
+
+def test_a_backfill_reset_through_the_route_is_kept(db, make_user, as_user, monkeypatch):
+    admin, reader = make_user(role="admin"), make_user()
+    _channel(db, backfill_done=True, backfill_cursor="page-9", recent_synced_at=NOW)
+    db.add(Subscription(user_id=reader.id, channel_id="UCc", deep_requested=False))
+    db.commit()
+    # The recent pull afterwards reaches YouTube; the reset is the route's own write.
+    monkeypatch.setattr(channels, "run_recent_backfill", lambda *a, **kw: None)
+    r = as_user(admin).post("/api/channels/UCc/reset-backfill")
+    assert r.status_code == 200, r.text
+    db.expire_all()
+    ch = db.get(Channel, "UCc")
+    assert (ch.backfill_done, ch.backfill_cursor, ch.recent_synced_at) == (False, None, None)
+    assert _subscription(db, reader).deep_requested is True
+
+
+# --- live search ------------------------------------------------------------------------------
+
+
+def test_search_results_cleared_through_the_route_are_gone(db, make_user, as_user):
+    user = make_user()
+    _video(db, "s1", via_search=True)
+    db.add(SearchFind(user_id=user.id, video_id="s1"))
+    db.commit()
+    r = as_user(user).post("/api/search/clear", json={"video_ids": ["s1"]})
+    assert r.json() == {"ok": True, "removed": 1}
+    db.expire_all()
+    assert db.get(Video, "s1") is None
+    assert db.execute(select(SearchFind)).first() is None
+
+
+# --- Plex show and season marks ---------------------------------------------------------------
+
+
+def _show(db):
+    lib = PlexLibrary(plex_key="2", title="Shows", kind="show", enabled=True)
+    db.add(lib)
+    db.flush()
+    sh = PlexShow(rating_key="sh1", library_id=lib.id, title="Show")
+    db.add(sh)
+    db.flush()
+    se = PlexSeason(rating_key="se1", show_id=sh.id, season_number=1)
+    db.add(se)
+    db.flush()
+    for n in (1, 2):
+        db.add(
+            PlexItem(
+                rating_key=f"e{n}",
+                library_id=lib.id,
+                kind="episode",
+                title=f"E{n}",
+                show_id=sh.id,
+                season_id=se.id,
+                season_number=1,
+                episode_number=n,
+            )
+        )
+    db.commit()
+
+
+def _plex_marks(db, user):
+    db.expire_all()
+    return sorted(
+        db.execute(
+            select(PlexItem.rating_key, PlexState.status)
+            .join(PlexState, PlexState.item_id == PlexItem.id)
+            .where(PlexState.user_id == user.id)
+        ).all()
+    )
+
+
+def test_a_show_marked_watched_through_the_route_stays_watched(db, make_user, as_user):
+    user = make_user()
+    _show(db)
+    r = as_user(user).post("/api/plex/show/sh1/state", json={"watched": True})
+    assert r.json() == {"ok": True, "changed": 2, "watched": True}
+    assert _plex_marks(db, user) == [("e1", "watched"), ("e2", "watched")]
+
+
+def test_a_season_marked_watched_through_the_route_stays_watched(db, make_user, as_user):
+    user = make_user()
+    _show(db)
+    r = as_user(user).post("/api/plex/season/se1/state", json={"watched": True})
+    assert r.json() == {"ok": True, "changed": 2, "watched": True}
+    assert _plex_marks(db, user) == [("e1", "watched"), ("e2", "watched")]
+
+
+def _read_past_the_cap(db, user) -> list[Notification]:
+    """READ_SOFT_CAP read notices, oldest first, and one unread notice after them."""
+    base = datetime(2024, 1, 1, tzinfo=timezone.utc)
+    rows = [
+        Notification(
+            user_id=user.id,
+            type="t",
+            title=f"n{i}",
+            read=i < READ_SOFT_CAP,
+            created_at=base + timedelta(minutes=i),
+        )
+        for i in range(READ_SOFT_CAP + 1)
+    ]
+    db.add_all(rows)
+    db.commit()
+    return rows
+
+
+def _committed_notices(user) -> tuple[int, bool]:
+    """What another session sees: how many of the user's notices are left, and whether the oldest is."""
+    with SessionLocal() as s:
+        ids = s.scalars(select(Notification.title).where(Notification.user_id == user.id)).all()
+    return len(ids), "n0" in ids
+
+
+@pytest.mark.parametrize("path", ["{id}/read", "read_all"])
+def test_reading_a_notice_past_the_cap_commits_the_trim(db, make_user, as_user, path):
+    """Marking a notice read makes READ_SOFT_CAP + 1 read ones: the handler commits the read flag and
+    the trim of the oldest together, and the trim counts the notice just read."""
+    user = make_user()
+    rows = _read_past_the_cap(db, user)
+    r = as_user(user).post("/api/me/notifications/" + path.format(id=rows[-1].id))
+    assert r.json() == {"ok": True}
+    assert _committed_notices(user) == (READ_SOFT_CAP, False)
@@ -408,7 +408,38 @@ UNIT_COMMITS = {
        "plex/watch_sync.py:run_plex_watch_reconcile",
    },
 }
+# Where core code may commit, each with why. Everything else in core flushes and leaves the commit
+# to its caller, as a unit's code does; a route's own helpers are held by test_route_conventions.
+CORE_COMMITS = {
+    # Quota spend and a background run's row are durable whatever the caller does next: each opens
+    # a session of its own.
+    "quota.py",
+    "jobruns.py",
+    # The scheduler's jobs own their session.
+    "scheduler.py",
+    # The instance's one settings row: created on first read, and the setup steps that finish it.
+    "state.py",
+    # A token's last-used stamp lands ahead of the route: a read-only route never commits, and a
+    # write route's rollback must not take it along. The expiry warning is a scheduled job.
+    "apitokens.py:_token_caller",
+    "apitokens.py:warn_expiring",
+    # An account's erasure is one operation shared by the self-service and the admin delete; the
+    # Google revoke and the confirmation run after it, in the background.
+    "auth.py:purge_user",
+    # The undelivered-email notice opens its own session (its callers are often background tasks).
+    "email.py:_alert_admins",
+    # One notice, committed unless its caller batches (`commit=False`).
+    "notifications.py:create_notification",
+    # A refreshed (or dead) YouTube grant must persist even when the call that needed it fails and
+    # its caller rolls back.
+    "youtube/client.py:_access_token",
+    "youtube/client.py:_drop_dead_grant",
+}
 SQL_WRITES = {"insert", "update", "delete", "pg_insert"}
+ALL_MODEL_NAMES = set(UNIT_MODEL_NAMES) | CORE_MODELS
+# The column methods that compare an owner column as a filter does (`Model.user_id.in_(ids)`).
+# `.is_(None)` is not one: it asks for the rows nobody owns (the system's own tags).
+OWNER_METHODS = {"in_", "not_in", "notin_", "__eq__", "__ne__"}


 def _package_unit(rel: str) -> str | None:
@@ -440,6 +471,64 @@ def _imported_models(tree: ast.AST) -> dict[str, str]:
    return out


+def _model_aliases(tree: ast.AST) -> dict[str, str]:
+    """Every other name a module binds to a model, by that name: an import under another name, and
+    `x = aliased(Model)` — an alias is the same table, and its owner column the same filter."""
+    out = {}
+    for node in ast.walk(tree):
+        if isinstance(node, ast.ImportFrom) and "models" in (node.module or ""):
+            for alias in node.names:
+                if alias.name in ALL_MODEL_NAMES:
+                    out[alias.asname or alias.name] = alias.name
+    for node in ast.walk(tree):
+        if (
+            isinstance(node, ast.Assign)
+            and isinstance(node.value, ast.Call)
+            and _called_name(node.value) == "aliased"
+            and node.value.args
+            and isinstance(node.value.args[0], ast.Name)
+        ):
+            source = node.value.args[0].id
+            model = out.get(source, source if source in ALL_MODEL_NAMES else None)
+            for target in node.targets:
+                if model and isinstance(target, ast.Name):
+                    out[target.id] = model
+    return out
+
+
+def _called_name(call: ast.Call) -> str | None:
+    func = call.func
+    return func.id if isinstance(func, ast.Name) else getattr(func, "attr", None)
+
+
+def _sql_write_names(tree: ast.AST) -> tuple[set[str], set[str]]:
+    """The names a module calls SQLAlchemy's statement writes by — `insert`/`update`/`delete` as
+    imported, under any alias — and the names it binds SQLAlchemy's modules to (`sa.update(...)`)."""
+    functions, modules = set(SQL_WRITES), set()
+    for node in ast.walk(tree):
+        if isinstance(node, ast.ImportFrom) and (node.module or "").split(".")[0] == "sqlalchemy":
+            for alias in node.names:
+                if alias.name in {"insert", "update", "delete"}:
+                    functions.add(alias.asname or alias.name)
+                elif alias.name in {"dialects", "postgresql", "sql"}:
+                    modules.add(alias.asname or alias.name)
+        elif isinstance(node, ast.Import):
+            for alias in node.names:
+                if alias.name.split(".")[0] == "sqlalchemy":
+                    modules.add(alias.asname or alias.name.split(".")[0])
+    return functions, modules
+
+
+def _written_table(call: ast.Call) -> str | None:
+    """The name of what a statement write's first argument names: `Model`, or `Model.__table__`."""
+    if not call.args:
+        return None
+    arg = call.args[0]
+    if isinstance(arg, ast.Attribute) and arg.attr == "__table__":
+        arg = arg.value
+    return arg.id if isinstance(arg, ast.Name) else None
+
+
 def _enclosing_functions(tree: ast.AST) -> dict[ast.AST, str]:
    out = {}
    for fn in ast.walk(tree):
@@ -461,49 +550,67 @@ def package_violations(rel: str, source: str) -> list[str]:
            continue
        if theirs not in READS.get(mine or "", set()):
            found.append(f"0: {mine or 'core'} imports {theirs}'s {name}")
+    write_functions, sql_modules = _sql_write_names(tree)
    for node in ast.walk(tree):
-        if not isinstance(node, ast.Call) or not isinstance(node.func, ast.Name):
+        if not isinstance(node, ast.Call):
            continue
+        func = node.func
        written = None
-        if node.func.id in models:
-            written = models[node.func.id]
-        elif node.func.id in SQL_WRITES and node.args and isinstance(node.args[0], ast.Name):
-            written = models.get(node.args[0].id)
+        if isinstance(func, ast.Name) and func.id in models:
+            written = models[func.id]
+        elif (isinstance(func, ast.Name) and func.id in write_functions) or (
+            isinstance(func, ast.Attribute)
+            and func.attr in {"insert", "update", "delete"}
+            and isinstance(_root(func), ast.Name)
+            and _root(func).id in sql_modules
+        ):
+            written = models.get(_written_table(node) or "")
        if written is None:
            continue
        theirs = UNIT_MODEL_NAMES[written]
        if theirs != mine:
            found.append(f"{node.lineno}: {mine or 'core'} writes {theirs}'s {written}")
    found += _owner_filters(tree)
-    if mine is not None:
-        functions = _enclosing_functions(tree)
-        allowed = UNIT_COMMITS.get(mine, set())
-        for node in ast.walk(tree):
-            if (
-                isinstance(node, ast.Call)
-                and isinstance(node.func, ast.Attribute)
-                and node.func.attr == "commit"
-                and rel not in allowed
-                and f"{rel}:{functions.get(node)}" not in allowed
-            ):
-                found.append(f"{node.lineno}: commits in {functions.get(node)}")
+    functions = _enclosing_functions(tree)
+    allowed = UNIT_COMMITS.get(mine, set()) if mine is not None else CORE_COMMITS
+    for node in ast.walk(tree):
+        if (
+            isinstance(node, ast.Call)
+            and isinstance(node.func, ast.Attribute)
+            and node.func.attr == "commit"
+            and rel not in allowed
+            and f"{rel}:{functions.get(node)}" not in allowed
+        ):
+            found.append(f"{node.lineno}: commits in {functions.get(node)}")
    return sorted(found, key=lambda v: int(v.split(":")[0]))


 def _owner_filters(tree: ast.AST) -> list[str]:
    """A comparison on a table's owner column written inline (`Model.user_id == …`), not through
    `userscope.owned_by`."""
+    names = {name: name for name in ALL_MODEL_NAMES} | _model_aliases(tree)
+
+    def owner_column(node: ast.expr) -> bool:
+        return (
+            isinstance(node, ast.Attribute)
+            and isinstance(node.value, ast.Name)
+            and node.value.id in names
+            and node.attr in OWNER_COLUMNS
+        )
+
    found = []
    for node in ast.walk(tree):
        if isinstance(node, ast.Compare):
            for side in (node.left, *node.comparators):
-                if (
-                    isinstance(side, ast.Attribute)
-                    and isinstance(side.value, ast.Name)
-                    and side.value.id in set(UNIT_MODEL_NAMES) | CORE_MODELS
-                    and side.attr in OWNER_COLUMNS
-                ):
+                if owner_column(side):
                    found.append(f"{node.lineno}: filters {side.value.id}.{side.attr} inline")
+        elif isinstance(node, ast.Call) and isinstance(node.func, ast.Attribute):
+            if node.func.attr in OWNER_METHODS and owner_column(node.func.value):
+                found.append(f"{node.lineno}: filters an owner column with {node.func.attr}")
+            elif node.func.attr == "filter_by" and any(
+                k.arg in OWNER_COLUMNS for k in node.keywords
+            ):
+                found.append(f"{node.lineno}: filters an owner column with filter_by")
    return found


@@ -524,11 +631,32 @@ def test_every_unit_package_exists():
    for prefixes in UNIT_PACKAGES.values():
        for p in prefixes:
            assert (APP / p).exists(), p
-    for commits in UNIT_COMMITS.values():
+    for commits in (*UNIT_COMMITS.values(), CORE_COMMITS):
        for entry in commits:
            assert (APP / entry.split(":")[0]).exists(), entry


+def test_every_listed_commit_is_still_there():
+    """An entry names a commit that exists: one left behind by a rename, or by a function that no
+    longer commits, would go on allowing a commit to whatever takes its name next."""
+    stale = []
+    for commits in (*UNIT_COMMITS.values(), CORE_COMMITS):
+        for entry in commits:
+            rel, _, name = entry.partition(":")
+            tree = ast.parse((APP / rel).read_text(encoding="utf-8"))
+            functions = _enclosing_functions(tree)
+            committing = {
+                functions.get(node)
+                for node in ast.walk(tree)
+                if isinstance(node, ast.Call)
+                and isinstance(node.func, ast.Attribute)
+                and node.func.attr == "commit"
+            }
+            if (name not in committing) if name else not committing:
+                stale.append(entry)
+    assert stale == []
+
+
 PACKAGE_CASES = {
    # (module path, source): violations
    ("downloads/x.py", "from app.models import Video"): 0,  # capture reads catalog
@@ -544,6 +672,22 @@ PACKAGE_CASES = {
    ("downloads/x.py", "q = DownloadJob.user_id == 1"): 1,
    ("downloads/x.py", "q = owned_by(DownloadJob.user_id, 1)"): 0,
    ("quota.py", "q = QuotaEvent.user_id == 1"): 1,  # core too
+    ("downloads/x.py", "from app.downloads.models import DownloadJob as J\nq = J.user_id == 1"): 1,
+    ("downloads/x.py", "st = aliased(VideoState)\nq = st.user_id == 1"): 1,
+    ("downloads/x.py", "q = DownloadJob.user_id.in_([1])"): 1,
+    ("downloads/x.py", "q = select(DownloadJob).filter_by(user_id=1)"): 1,
+    ("downloads/x.py", "q = Tag.user_id.is_(None)"): 0,  # the rows nobody owns
+    (
+        "downloads/x.py",
+        "from sqlalchemy import update as up\nfrom app.models import Video\nup(Video)",
+    ): 1,
+    (
+        "downloads/x.py",
+        "import sqlalchemy as sa\nfrom app.models import Video\nsa.update(Video)",
+    ): 1,
+    ("downloads/x.py", "from app.models import Video\nupdate(Video.__table__)"): 1,
+    ("notifications.py", "def f(db):\n    db.commit()"): 1,  # core flushes too
+    ("state.py", "def f(db):\n    db.commit()"): 0,
    ("downloads/x.py", "def f(db):\n    db.commit()"): 1,
    ("downloads/gc.py", "def f(db):\n    db.commit()"): 0,
    ("downloads/upload.py", "def ingest_upload(db):\n    db.commit()"): 0,
@@ -107,7 +107,8 @@ project checks needs a row.

 | Scope | Narrowed | Why | Measured |
 | --- | --- | --- | --- |
-| check | e2e (Playwright) not in `check` | It needs the :8080 stack built from the tree (minutes). It runs in `check-full`, and `release` rebuilds the stack and requires it (E32 D2) — nothing ships without it. | 5 specs × 3 engines |
+| check | e2e (Playwright) not in `check` | It needs the :8080 stack built from the tree (minutes). It runs in `check-full`, and `release` rebuilds the stack and requires it (E32 D2) — nothing ships without it. | 9 specs: 6 × 3 engines, 3 on chromium only (the row below) |
+| e2e | `a11y.spec.ts` `member.spec.ts` `devices.spec.ts` run on chromium only (`CHROMIUM_ONLY`, the webkit and ipad projects' `testIgnore`) | What they hold does not change with the engine: the axe scan reads the DOM and the accessibility tree, which the engines build alike; the member spec checks which pages and requests a role gets; the devices spec opens its own desktop and iPad contexts. Run three times they would triple their time and catch nothing more. What an engine does change (layout, events) the other specs hold on all three. | 3 of 9 specs |
 | pushes | no server-side gate (no CI) | Owner decision 2026-10-04. The gate (`scripts/check.sh`) runs on the dev machine before work is handed over, and `release` runs it, then e2e, before anything is merged and Trivy before the image is pushed — the CI only repeated that, after the merge. Its runner shared the git host's container, and one run there starved Gitea of memory until it stopped answering. What it added — a fresh `npm ci` on Linux — the image build does at publish. | — |
 | coverage | not in the gate; no threshold | Owner decision E32 D4: never enforced; "every regression test seen red" is the bar, and a percentage invites tests written to move it. Out of the lanes since E35: read once, by hand, it found 5 bugs; as a lane it was a number nobody acted on, and the slowest vitest run (the one that flaked). On demand: docs/development.md. | frontend 80% / backend 78% of lines (E32 S1) |
 | CVE gate | HIGH/CRITICAL with a fix available only | `--ignore-unfixed` + `--severity HIGH,CRITICAL`: a finding with no fix cannot be acted on by an upgrade; it shows in the scan output. A fixable one fails the publish, and the only way past it is `.trivyignore.yaml`. | 0 open (E32 S1) |
@@ -65,6 +65,11 @@ the server logs) can run setup. Then click through:

 That's it. You can change any of the optional settings later under the admin **Configuration** page.

+> **Set a password even if you sign in with Google** (Settings → Account → Password). If Google
+> sign-in stops working — the client secret rotated, the consent revoked, Google down — email and
+> password still let you in. An account that only ever used Google is otherwise locked out until an
+> admin creates a set-password link for it (Users).
+
 > The setup wizard only exists until you finish it. After that, the setup routes are disabled and
 > the token is invalidated — there's no setup surface left on a configured instance.

@@ -25,7 +25,8 @@ that one error again and passes every other failure straight on. Don't run two e
 same machine at once.

 **Browsers** — every flow runs in three projects: `chromium` (Desktop Chrome), `webkit` (Desktop
-Safari) and `ipad` (iPad gen 7 landscape: 1080×810, touch, mobile). Install the engines once per
+Safari) and `ipad` (iPad gen 7 landscape: 1080×810, touch, mobile) — except `a11y`, `member` and
+`devices`, which run on chromium only (`CHROMIUM_ONLY`; why is in the `docs/gates.md` row). Install the engines once per
 machine with `npx playwright install chromium webkit`; run one project with
 `npx playwright test --project=ipad`. WebKit catches what every iOS browser inherits from the engine
 (JS, CSS/layout, event order). It does **not** reproduce iPad media refusals: Playwright's WebKit
@@ -57,9 +58,15 @@ admin so the suite can reach the admin-only pages / render branches, matching th
 user), reset to a deterministic baseline every run by
 [`scripts/e2e_seed.py`](../../scripts/e2e_seed.py) (run
 in-container so it reuses the app's models + argon2 hasher): the top-3 organic channels subscribed,
-one in-progress + one watched + one saved video, prefs pinned to `language: en`. Credentials come
-from `E2E_EMAIL` (default `e2e@siftlode.local`) and `E2E_PASSWORD` — **required, no default**: run
-the suite through the ops wrapper, which exports it from the secret store. Override the container name with
+one in-progress + one watched + one saved video, prefs pinned to `language: en`. A second account,
+the plain **member** `e2e-member@siftlode.local`, gets the same baseline and its own session
+(`e2e/.auth/member.json`). Two specs run as it: `member.spec.ts`, holding what a non-admin sees (no
+admin entry, no admin call, no failed request, an admin link lands on the feed), and `devices.spec.ts`,
+which signs it in on a desktop and an iPad at once and changes its channel layout on the server (put
+back in a `finally`) — a spec that reads the member's server prefs must not run beside it. Credentials come from `E2E_EMAIL` (default
+`e2e@siftlode.local`), `E2E_MEMBER_EMAIL` (default `e2e-member@siftlode.local`) and `E2E_PASSWORD`
+(both accounts) — **required, no default**: run the suite through the ops wrapper, which exports it
+from the secret store. Override the container name with
 `E2E_API_CONTAINER`; skip the in-container seed with `E2E_SKIP_SEED=1`.

 ## Selectors
@@ -0,0 +1,61 @@
+import { test, expect, devices, type Browser, type Page } from "@playwright/test";
+import { MEMBER_STORAGE, gotoPage, openApp } from "./helpers";
+
+// One member on two devices, a desktop and an iPad, each with its own browser storage. A preference is
+// a choice about the account, so a change on one device shows on the other; a device setting (here
+// performance mode, for a weak GPU) stays on the device it was set on. Kept in browser storage only, a
+// preference used to stay on the one device it was changed on.
+
+async function device(browser: Browser, descriptor: (typeof devices)[string]): Promise<Page> {
+  const context = await browser.newContext({
+    ...descriptor,
+    baseURL: test.info().project.use.baseURL,
+    locale: "en-US",
+    storageState: MEMBER_STORAGE,
+  });
+  return context.newPage();
+}
+
+const savedPrefs = (page: Page) =>
+  page.waitForResponse(
+    (r) => r.url().includes("/me/preferences") && r.request().method() === "PUT",
+  );
+
+async function pickLayout(page: Page, from: string, to: string): Promise<void> {
+  await page.getByRole("button", { name: `Layout: ${from}` }).click();
+  await Promise.all([savedPrefs(page), page.getByRole("menuitemradio", { name: to }).click()]);
+  await expect(page.getByRole("button", { name: `Layout: ${to}` })).toBeVisible();
+}
+
+test("a preference follows the member to another device, a device setting stays", async ({
+  browser,
+}) => {
+  const desktop = await device(browser, devices["Desktop Chrome"]);
+  const ipad = await device(browser, devices["iPad (gen 7) landscape"]);
+  try {
+    await openApp(desktop);
+    await gotoPage(desktop, "channels");
+    await pickLayout(desktop, "Table", "Cards");
+    await gotoPage(desktop, "settings");
+    await desktop.getByRole("switch", { name: "Performance mode" }).click();
+    await expect(desktop.locator("html")).toHaveAttribute("data-perf", "1");
+    // The Settings prefs save as one payload: the next one (a hints change) would carry performance
+    // mode if it went to the account. Waited for, so the iPad below cannot simply load first.
+    const hints = desktop.getByRole("switch", { name: "Show hints" });
+    const [save] = await Promise.all([savedPrefs(desktop), hints.click()]);
+    expect(Object.keys(save.request().postDataJSON() as object)).not.toContain("performanceMode");
+    await Promise.all([savedPrefs(desktop), hints.click()]);
+
+    await openApp(ipad);
+    await expect(ipad.locator("html")).not.toHaveAttribute("data-perf", "1");
+    await gotoPage(ipad, "channels");
+    await expect(ipad.getByRole("button", { name: "Layout: Cards" })).toBeVisible();
+  } finally {
+    // The member is shared by the suite: put the layout back (performance mode went with the
+    // desktop's storage).
+    await gotoPage(desktop, "channels");
+    await pickLayout(desktop, "Cards", "Table");
+    await desktop.context().close();
+    await ipad.context().close();
+  }
+});
@@ -1,20 +1,24 @@
-import { test as setup, expect } from "@playwright/test";
+import { test as setup, expect, type Page } from "@playwright/test";
 import { execFileSync } from "node:child_process";
 import { mkdirSync, readFileSync } from "node:fs";
 import { dirname, resolve } from "node:path";
 import { fileURLToPath } from "node:url";
+import { MEMBER_STORAGE } from "./helpers";
 import { retryConnect } from "./retryConnect";

 /**
 * One-time setup for the characterization net (the `setup` project the tests depend on):
 *   1. preflight — the localdev :8080 stack must be up (owned by `siftlode dev`);
- *   2. seed — reset the dedicated E2E account to a known baseline, in-container (idempotent);
- *   3. authenticate ONCE via password-login and persist the session + `en` locale to
- *      storageState, so the real specs never re-auth (auth endpoints are rate-limited).
+ *   2. seed — reset the dedicated E2E accounts to a known baseline, in-container (idempotent);
+ *   3. authenticate each ONCE via password-login and persist its session + `en` locale to its own
+ *      storageState, so the real specs never re-auth (auth endpoints are rate-limited). The suite
+ *      runs as the admin; `member.spec.ts` and `devices.spec.ts` run as the plain member, from
+ *      MEMBER_STORAGE.
 */

 const STORAGE = "e2e/.auth/state.json";
 const EMAIL = process.env.E2E_EMAIL || "e2e@siftlode.local";
+const MEMBER_EMAIL = process.env.E2E_MEMBER_EMAIL || "e2e-member@siftlode.local";
 // No default: a password committed to a public repo is not a credential. The ops wrapper
 // (`siftlode e2e` / `check-full`) exports it from the secret store.
 const PASSWORD = process.env.E2E_PASSWORD ?? "";
@@ -27,14 +31,44 @@ const SEED_SCRIPT = resolve(HERE, "../../scripts/e2e_seed.py");
 // The in-container seed only makes sense when the target is the local stack.
 const isLocalTarget = /\/\/(127\.0\.0\.1|localhost)[:/]/.test(BASE_URL);

-setup("seed + authenticate", async ({ page }) => {
+/** Sign `email` in on `page`'s context and save that context as `path`. */
+async function signIn(page: Page, email: string, path: string): Promise<void> {
+  // These requests run in Node and share the page's cookie jar — so the `session` cookie the login
+  // receives is what storageState saves below — and a localhost connect the machine dropped is
+  // tried again (retryConnect).
+  const api = retryConnect(page.request);
+  const login = await api.post("/auth/password-login", { data: { email, password: PASSWORD } });
+  expect(
+    login.ok(),
+    `password-login failed for ${email} — is the E2E account seeded?`,
+  ).toBeTruthy();
+
+  const me = await (await api.get("/api/me")).json();
+  expect(me.authenticated, `session not established after login (${email})`).toBeTruthy();
+
+  // Pin the UI locale, suppress the first-login onboarding wizard (a full-screen overlay that
+  // would intercept every click — the test accounts have no YouTube connection, so it auto-opens),
+  // and opt the account out of starting on Home. A bare URL lands on Home, and the feed, channel
+  // and player flows this net locks all open "/" expecting the feed; the opt-out is the one
+  // Settings → Appearance → Startup writes. Both flags are the account's local state, held for the
+  // account the browser records as their owner (`siftlode.owner`, see storage.ts claimLocalData).
+  await page.goto("/");
+  await page.evaluate((id: number) => {
+    localStorage.setItem("siftlode.owner", String(id));
+    localStorage.setItem("siftlode.lang", "en");
+    localStorage.setItem("siftlode.onboarding.dismissed", "1");
+    localStorage.setItem("siftlode.startOnHome", "0");
+  }, me.id);
+  mkdirSync(dirname(path), { recursive: true });
+  await page.context().storageState({ path });
+}
+
+setup("seed + authenticate", async ({ page, browser }) => {
  expect(
    PASSWORD,
    "E2E_PASSWORD is not set — run the suite through `siftlode e2e` (it reads the secret store)",
  ).not.toBe("");

-  // These requests run in Node (sharing the page's cookie jar), so a localhost connect the machine
-  // dropped is tried again (retryConnect).
  const api = retryConnect(page.request);

  // 1. Preflight: fail fast (with a useful message) if the stack isn't running.
@@ -55,6 +89,8 @@ setup("seed + authenticate", async ({ page }) => {
          "-e",
          `E2E_EMAIL=${EMAIL}`,
          "-e",
+          `E2E_MEMBER_EMAIL=${MEMBER_EMAIL}`,
+          "-e",
          `E2E_PASSWORD=${PASSWORD}`,
          CONTAINER,
          "python",
@@ -69,28 +105,12 @@ setup("seed + authenticate", async ({ page }) => {
    }
  }

-  // 3. Authenticate. page.request shares the browser context's cookie jar, so the `session`
-  //    cookie it receives is captured by storageState below.
-  const login = await api.post("/auth/password-login", {
-    data: { email: EMAIL, password: PASSWORD },
-  });
-  expect(login.ok(), "password-login failed — is the E2E account seeded?").toBeTruthy();
-
-  const me = await (await api.get("/api/me")).json();
-  expect(me.authenticated, "session not established after login").toBeTruthy();
-
-  // 4. Pin the UI locale, suppress the first-login onboarding wizard (a full-screen overlay
-  //    that would intercept every click — the test account has no YouTube connection, so it
-  //    auto-opens), and opt the account out of starting on Home. A bare URL lands on
-  //    Home, and the feed, channel and player flows this net locks all open "/" expecting the
-  //    feed; the opt-out is the one Settings → Appearance → Startup writes. Both flags are
-  //    per-account: `siftlode.onboarding.dismissed.<id>`, `siftlode.startOnHome.<id>`.
-  await page.goto("/");
-  await page.evaluate((id: number) => {
-    localStorage.setItem("siftlode.lang", "en");
-    localStorage.setItem(`siftlode.onboarding.dismissed.${id}`, "1");
-    localStorage.setItem(`siftlode.startOnHome.${id}`, "0");
-  }, me.id);
-  mkdirSync(dirname(STORAGE), { recursive: true });
-  await page.context().storageState({ path: STORAGE });
+  // 3. Authenticate each account in its own context: one cookie jar holds one session.
+  await signIn(page, EMAIL, STORAGE);
+  const memberContext = await browser.newContext({ storageState: { cookies: [], origins: [] } });
+  try {
+    await signIn(await memberContext.newPage(), MEMBER_EMAIL, MEMBER_STORAGE);
+  } finally {
+    await memberContext.close();
+  }
 });
@@ -1,5 +1,9 @@
 import { expect, type Page } from "@playwright/test";

+/** The plain member's saved session (global.setup.ts signs it in); `member.spec.ts` and
+ *  `devices.spec.ts` run from it. */
+export const MEMBER_STORAGE = "e2e/.auth/member.json";
+
 /** Open the app and wait for the authenticated shell (the nav rail) to be ready. */
 export async function openApp(page: Page, path = "/"): Promise<void> {
  await page.goto(path);
@@ -0,0 +1,97 @@
+import { test, expect, type Page, type Request, type Response } from "@playwright/test";
+import { MEMBER_STORAGE, openApp } from "./helpers";
+
+// What a plain member (not an admin) sees. The rest of the suite runs as the admin, so without this
+// spec nothing end to end holds the member's shell: the backend refuses the admin routes (pytest),
+// but a member shown an admin entry, sent to an admin page or met by a refused request would pass.
+test.use({ storageState: MEMBER_STORAGE });
+
+const ADMIN_PAGES = ["scheduler", "config", "users", "activity"];
+// The pages every member has; Plex joins them only where the instance enables it.
+const CORE = [
+  "home",
+  "feed",
+  "channels",
+  "playlists",
+  "notifications",
+  "files",
+  "stats",
+  "settings",
+];
+
+const isApi = (url: string) => new URL(url).pathname.startsWith("/api/");
+
+/** Collect every admin call and every failed API answer the page gets while `run` goes. `run` is
+ *  handed `settled`, which waits until no API request of the page is in flight: an error state
+ *  renders only once its request has answered, and a check made before that passes on a page that
+ *  is still loading. */
+async function watchRequests(
+  page: Page,
+  run: (settled: () => Promise<void>) => Promise<void>,
+): Promise<string[]> {
+  const bad: string[] = [];
+  const pending = new Set<Request>();
+  const onRequest = (r: Request) => {
+    if (!isApi(r.url())) return;
+    pending.add(r);
+    if (new URL(r.url()).pathname.startsWith("/api/admin")) bad.push(`admin call ${r.url()}`);
+  };
+  const onDone = (r: Request) => pending.delete(r);
+  const onResponse = (r: Response) => {
+    if (isApi(r.url()) && r.status() >= 400) bad.push(`${r.status()} ${r.url()}`);
+  };
+  const settled = () => expect.poll(() => pending.size, { timeout: 15_000 }).toBe(0);
+  page.on("request", onRequest);
+  page.on("requestfinished", onDone);
+  page.on("requestfailed", onDone);
+  page.on("response", onResponse);
+  try {
+    await run(settled);
+    await settled();
+  } finally {
+    page.off("request", onRequest);
+    page.off("requestfinished", onDone);
+    page.off("requestfailed", onDone);
+    page.off("response", onResponse);
+  }
+  return bad;
+}
+
+test.describe("a member", () => {
+  test("gets no admin entry, and every page of the rail opens without an error", async ({
+    page,
+  }) => {
+    const bad = await watchRequests(page, async (settled) => {
+      await openApp(page);
+      await expect(page.getByTestId("nav-admin")).toHaveCount(0);
+
+      const rail = page
+        .getByRole("navigation", { name: "Primary navigation" })
+        .locator('[data-testid^="nav-"]');
+      const ids = (await rail.evaluateAll((els) => els.map((el) => el.getAttribute("data-testid"))))
+        .filter((id): id is string => id !== null)
+        .map((id) => id.replace(/^nav-/, ""));
+      expect(ids).toEqual(expect.arrayContaining(CORE));
+      expect(ids.filter((id) => ADMIN_PAGES.includes(id))).toEqual([]);
+
+      for (const id of ids) {
+        await page.getByTestId(`nav-${id}`).click();
+        await expect(page.getByTestId(`nav-${id}`)).toHaveAttribute("aria-current", "page");
+        await settled();
+        // An error state is a `role=alert` StateMessage; a page that failed to load shows one.
+        await expect(page.getByRole("alert"), `error state on ${id}`).toHaveCount(0);
+      }
+    });
+    expect(bad).toEqual([]);
+  });
+
+  test("a link to an admin page opens no admin page", async ({ page }) => {
+    for (const id of ADMIN_PAGES) {
+      const bad = await watchRequests(page, () => openApp(page, `/?page=${id}`));
+      expect(bad, id).toEqual([]);
+      await expect(page.getByTestId("nav-admin")).toHaveCount(0);
+      // The member lands on the feed instead, and the rail says so.
+      await expect(page.getByTestId("nav-feed")).toHaveAttribute("aria-current", "page");
+    }
+  });
+});
@@ -5,7 +5,9 @@ import { openApp, gotoPage } from "./helpers";
 // Appearance change persists on its own via PUT /me/preferences and a transient indicator confirms
 // it; leaving Settings after a change is no longer guarded.
 test.describe("settings auto-save", () => {
-  const TOGGLE = "Performance mode"; // an Appearance switch (locale pinned to en)
+  // An Appearance switch saved to the account (locale pinned to en). Not performance mode: that one
+  // is the device's and is never sent.
+  const TOGGLE = "Show hints";

  // The save is debounced (~500ms), so wait for the PUT rather than assuming it's synchronous.
  const putPrefs = (page: Page) =>
@@ -17,9 +17,11 @@ import { defineConfig, devices } from "@playwright/test";

 const BASE_URL = process.env.PW_BASE_URL || "http://127.0.0.1:8080";
 const STORAGE_STATE = "e2e/.auth/state.json";
-// The axe scan reads the DOM and the accessibility tree, which the engines build alike: one engine
-// answers for all, so the other projects leave it out instead of tripling its time.
-const CHROMIUM_ONLY = /a11y\.spec\.ts/;
+// The axe scan reads the DOM and the accessibility tree, which the engines build alike, and the
+// member spec checks which pages and requests a role gets, which no engine changes, and the devices
+// spec opens its own desktop and iPad contexts: one engine answers for all, so the other projects
+// leave them out instead of tripling their time.
+const CHROMIUM_ONLY = /(a11y|member|devices)\.spec\.ts/;

 export default defineConfig({
  testDir: "./e2e",
@@ -7,7 +7,8 @@ import { api, setUnauthorizedHandler, type Me } from "./lib/api";
 import { notifyActionError } from "./lib/actionErrors";
 import { setLanguage, type LangCode } from "./i18n";
 import { clearParams } from "./lib/urlState";
-import type { Page } from "./lib/urlState";
+import { reloadAfterRefusal } from "./lib/entryUrl";
+import { useAccountSwitchNotice } from "./lib/useAccountSwitchNotice";
 import {
  PREF_KEY,
  saveLayoutLocal,
@@ -22,7 +23,7 @@ import { useVerifyEntry } from "./lib/useVerifyEntry";
 import { useNavigation, useNavigationActions } from "./components/navigationContext";
 import { useFeedFilters, useFeedFiltersActions } from "./components/feedFiltersContext";
 import { useFeedView, useFeedViewActions } from "./components/feedViewContext";
-import { moduleDef, moduleOrder } from "./lib/modules";
+import { moduleOrder } from "./lib/modules";
 import { PAGE_CONTENT, PAGE_RAIL, preloadPages } from "./components/moduleRegistry";
 import { lazyWithPreload } from "./lib/lazyWithPreload";
 import { LoadingState } from "./components/ui/QueryState";
@@ -231,16 +232,19 @@ export default function App() {
    if (signedIn) clearParams("access", "verify", "reset");
  }, [signedIn]);

+  // A sign-in as someone else that the server refused, or a reload as another account: say so, once.
+  useAccountSwitchNotice(signedIn);
+
  // Any request that 401s means the session ended server-side. If we were signed in (cached
  // `me` is a user object), reload to the login page at once (also covers any click that hits
  // the API). Guarded on a cached user so a stray 401 while logged out (cached `me` is null)
-  // can't loop the public landing.
+  // can't loop the public landing. A tab showing another account than the one signed in now
+  // reloads as that one instead, and says why (`reloadAfterRefusal`).
  useEffect(() => {
-    setUnauthorizedHandler(() => {
-      if (queryClient.getQueryData(qk.me())) {
-        queryClient.clear();
-        window.location.reload();
-      }
+    setUnauthorizedHandler((err) => {
+      if (!queryClient.getQueryData(qk.me())) return;
+      queryClient.clear();
+      reloadAfterRefusal(err);
    });
    return () => setUnauthorizedHandler(null);
  }, [queryClient]);
@@ -271,8 +275,8 @@ export default function App() {
    clearParams("link");
  }, [t, setPage, refetchMe]);

-  // A Google OAuth flow started while already signed in (add-another-account, link, or upgrade) was
-  // cancelled at Google's consent screen. The backend only emits this for a live session, so a
+  // A Google OAuth flow started while already signed in (signing in as someone else, or a Google
+  // connect) was cancelled at Google's consent screen. The backend only emits this for a live session, so a
  // neutral toast is the right feedback — the pre-auth login banner would never render here.
  // Same one-shot shape as the `?link=` handler above, same reason for the ref.
  const oauthAnswered = useRef(false);
@@ -373,14 +377,11 @@ export default function App() {
        ? `yt:${ytSearch}`
        : `feed:${JSON.stringify({ ...filters, q: "" })}`;

-  // The module registry drives the render from `page` alone — no per-page chrome conditionals. A
-  // page the account can't reach (a gated module via stale nav / a deep link) falls back to the feed,
-  // exactly as the old ternary's fallthrough did. A channel page overlays the content column, so no
-  // side rail shows behind it.
-  const pageGate = moduleDef(page).gate;
-  const activePage: Page = !pageGate || pageGate(me!) ? page : "feed";
-  const { Component: PageContent, maxWidth } = PAGE_CONTENT[activePage];
-  const railDef = channelView ? undefined : PAGE_RAIL[activePage];
+  // The module registry drives the render from `page` alone — no per-page chrome conditionals. A page
+  // the account can't reach never gets here: the navigation state already moved it to the feed
+  // (`reachablePage`). A channel page overlays the content column, so no side rail shows behind it.
+  const { Component: PageContent, maxWidth } = PAGE_CONTENT[page];
+  const railDef = channelView ? undefined : PAGE_RAIL[page];

  return (
    <div className="h-dvh flex text-fg">
@@ -444,7 +445,7 @@ export default function App() {
          </PageShell>
        ) : (
          <PageShell
-            title={t(pageTitleKey(activePage))}
+            title={t(pageTitleKey(page))}
            scrollKey={normalScrollKey}
            maxWidth={maxWidth}
            header={<Header me={me!} onYtSearch={enterYtSearch} />}
@@ -1,10 +1,9 @@
 // @vitest-environment jsdom
 //
-// BootGate settles the tab's identity BEFORE the app tree mounts. The bug it replaced: a new tab (no
-// pinned account in sessionStorage) built the app for "no account" first — so with Settings →
-// Appearance → Startup turned off, NavigationProvider's initializer could not read the opt-out and
-// landed on Home, mirrored ?page=home into the address bar, and then re-read its own mirror when the
-// account was pinned, staying on Home. Only a same-tab reload (account already pinned) was right.
+// BootGate settles the tab's identity BEFORE the app tree mounts. The bug it replaced: a new tab built
+// the app for "no account" first — so with Settings → Appearance → Startup turned off,
+// NavigationProvider's initializer could not read the opt-out and landed on Home, mirrored ?page=home
+// into the address bar, and then re-read its own mirror once the account was known, staying on Home.
 //
 // Real NavigationProvider, real storage and i18n; only the two boot probes are stubbed. The probe
 // records EVERY render, so "never showed the wrong page, not even for one render" is what is asserted.
@@ -17,7 +16,6 @@ import BootGate from "./BootGate";
 import { NavigationProvider } from "./NavigationProvider";
 import { useNavigation } from "./navigationContext";
 import { api, type Me } from "../lib/api";
-import { ACTIVE_ACCOUNT_KEY } from "../lib/activeAccount";
 import { qk } from "../lib/queryKeys";
 import i18n from "../i18n";

@@ -47,13 +45,23 @@ function mount() {
  );
  return qc;
 }
-const account = (over: Partial<Me> = {}) => ({ id: 5, preferences: {}, ...over }) as Me;
+// On a Plex-enabled instance: the remembered page these cases restore is the Plex module, which only
+// an instance with Plex lets an account open.
+const account = (over: Partial<Me> = {}) =>
+  ({ id: 5, preferences: {}, plex_enabled: true, ...over }) as Me;
 let savePrefs: MockInstance<typeof api.savePrefs>;
 const retryButton = () => screen.findByRole("button", { name: i18n.t("common.retry") });
+// Whose local state the browser holds (storage.ts claimLocalData).
+const owner = () => localStorage.getItem("siftlode.owner");
+// The browser already holds account 5's state, as on any visit after the first.
+const heldBy5 = (entries: Record<string, string>) => {
+  localStorage.setItem("siftlode.owner", "5");
+  for (const [k, v] of Object.entries(entries)) localStorage.setItem(k, v);
+};

 beforeEach(async () => {
  localStorage.clear();
-  sessionStorage.clear(); // a NEW tab: nothing pinned yet
+  sessionStorage.clear(); // a NEW tab
  window.history.replaceState(null, "", "/");
  renders.length = 0;
  mounts = 0;
@@ -69,15 +77,14 @@ afterEach(() => {

 describe("BootGate", () => {
  it("a new tab with Startup off opens on the page the account left — never on Home first", async () => {
-    localStorage.setItem("siftlode.startOnHome.5", "0");
-    localStorage.setItem("siftlode.page.5", "plex");
+    heldBy5({ "siftlode.startOnHome": "0", "siftlode.page": "plex" });
    vi.spyOn(api, "me").mockResolvedValue(account());
    mount();

    expect((await screen.findByTestId("app")).textContent).toBe("plex");
    expect(renders.map((r) => r.page)).toEqual(renders.map(() => "plex"));
    expect(new URLSearchParams(window.location.search).get("page")).toBe("plex");
-    expect(sessionStorage.getItem(ACTIVE_ACCOUNT_KEY)).toBe("5"); // the tab is pinned
+    expect(owner()).toBe("5");
  });

  it("renders the app in the account's language from its first render", async () => {
@@ -117,12 +124,12 @@ describe("BootGate", () => {
    expect(renders).toHaveLength(0);
  });

-  it("signed out: mounts the app (App shows the landing) without pinning an account", async () => {
+  it("signed out: mounts the app (App shows the landing) without claiming the local state", async () => {
    vi.spyOn(api, "me").mockResolvedValue(null);
    mount();

    await screen.findByTestId("app");
-    expect(sessionStorage.getItem(ACTIVE_ACCOUNT_KEY)).toBeNull();
+    expect(owner()).toBeNull();
  });

  it("setup mode: mounts the app (App shows the wizard) without asking for the account", async () => {
@@ -132,7 +139,7 @@ describe("BootGate", () => {

    await screen.findByTestId("app");
    expect(me).not.toHaveBeenCalled();
-    expect(sessionStorage.getItem(ACTIVE_ACCOUNT_KEY)).toBeNull();
+    expect(owner()).toBeNull();
  });

  it("an unanswered setup probe is an error with a retry — never the signed-out landing", async () => {
@@ -145,7 +152,7 @@ describe("BootGate", () => {
    fireEvent.click(await retryButton());
    await screen.findByTestId("app");
    expect(renders.length).toBeGreaterThan(0);
-    expect(sessionStorage.getItem(ACTIVE_ACCOUNT_KEY)).toBe("5"); // it mounted for the account
+    expect(owner()).toBe("5"); // it mounted for the account
  });

  it("an unanswered first account probe is an error with a retry", async () => {
@@ -154,7 +161,7 @@ describe("BootGate", () => {

    fireEvent.click(await retryButton());
    await screen.findByTestId("app");
-    expect(sessionStorage.getItem(ACTIVE_ACCOUNT_KEY)).toBe("5");
+    expect(owner()).toBe("5");
  });

  it("a background refetch that fails keeps the app mounted, as it was", async () => {
@@ -172,17 +179,30 @@ describe("BootGate", () => {
    expect(mounts).toBe(1);
  });

-  it("a pin naming another account than the server answered as is corrected before the app reads it", async () => {
-    // Pinned to 9, but 9 was signed out in another tab: the server answers as the browser's default, 5.
-    sessionStorage.setItem(ACTIVE_ACCOUNT_KEY, "9");
+  it("an account signing in where another one's state is held never reads it, not for one render", async () => {
+    // Account 9 used this browser last, and left it on Plex with Startup off.
+    localStorage.setItem("siftlode.owner", "9");
+    localStorage.setItem("siftlode.startOnHome", "0");
+    localStorage.setItem("siftlode.page", "plex");
+    vi.spyOn(api, "me").mockResolvedValue(account());
+    mount();
+
+    expect((await screen.findByTestId("app")).textContent).toBe("home");
+    expect(renders.map((r) => r.page)).toEqual(renders.map(() => "home"));
+    expect(owner()).toBe("5");
+    expect(localStorage.getItem("siftlode.page")).not.toBe("plex");
+  });
+
+  it("an account's state left under the old per-account keys is the one it boots on", async () => {
    localStorage.setItem("siftlode.startOnHome.5", "0");
    localStorage.setItem("siftlode.page.5", "plex");
+    localStorage.setItem("siftlode.page.9", "files");
    vi.spyOn(api, "me").mockResolvedValue(account());
    mount();

    expect((await screen.findByTestId("app")).textContent).toBe("plex");
    expect(renders.map((r) => r.page)).toEqual(renders.map(() => "plex"));
-    expect(sessionStorage.getItem(ACTIVE_ACCOUNT_KEY)).toBe("5");
+    expect(localStorage.getItem("siftlode.page.9")).toBeNull();
  });

  it("a mounted tab whose identity changes reloads rather than keep the previous account's tree", async () => {
@@ -198,20 +218,20 @@ describe("BootGate", () => {
    await waitFor(() => expect(reload).toHaveBeenCalled());
  });

-  it("an identity change reloads without moving the pin first", async () => {
-    // The tree being torn down still runs its unmount saves before the page goes; with the pin already
-    // moved they would land under the NEW account's local keys (a player volume still pending, say).
+  it("an identity change reloads without handing the local state over first", async () => {
+    // The tree being torn down still runs its unmount saves before the page goes; with the state
+    // already handed over they would land in the NEW account's (a player volume still pending, say).
    const me = vi.spyOn(api, "me").mockResolvedValue(account());
    const qc = mount();
    await screen.findByTestId("app");
    const reload = vi.fn();
    vi.stubGlobal("location", { ...window.location, reload });

-    me.mockResolvedValue(account({ id: 6 })); // 5 was signed out elsewhere: the server answers as 6
+    me.mockResolvedValue(account({ id: 6 })); // another tab signed this browser in as 6
    await act(() => qc.refetchQueries({ queryKey: qk.me() }));

    await waitFor(() => expect(reload).toHaveBeenCalled());
-    expect(sessionStorage.getItem(ACTIVE_ACCOUNT_KEY)).toBe("5");
+    expect(owner()).toBe("5");
  });

  // The adoption effect is idempotent for the identity it already adopted, and that is load-bearing
@@ -1,8 +1,9 @@
 import { useEffectEvent, useLayoutEffect, useState, type ReactNode } from "react";
 import { useQuery, useQueryClient } from "@tanstack/react-query";
 import { useTranslation } from "react-i18next";
-import { api, getActiveAccount, setActiveAccount } from "../lib/api";
+import { api } from "../lib/api";
 import { log } from "../lib/log";
+import { claimLocalData } from "../lib/storage";
 import { saveLanguage } from "../lib/saveLanguage";
 import { qk } from "../lib/queryKeys";
 import i18n, { isSupported, setLanguage } from "../i18n";
@@ -14,17 +15,16 @@ import { LoadingState, StateMessage } from "./ui/QueryState";
 // tab, the theme, the language — and all of it is read synchronously when its provider mounts. Those
 // providers used to mount before /api/me answered, so a new tab first built the app for "no account"
 // (defaults: Home instead of the page you left, the default theme, the browser language) and then
-// patched each piece once <App> pinned the account. Every patch was a visible jump at best, and at
+// patched each piece once <App> knew the account. Every patch was a visible jump at best, and at
 // worst it read a value the first pass had already written (the landing page mirrored ?page=home into
 // the address bar, then re-read its own mirror and stayed on Home). So the order is inverted instead:
 // settle the identity, adopt what must be adopted before a single pixel of the app is drawn, THEN mount
 // the provider tree — whose initializers are then simply right, with nothing to re-read.
 //
-// The per-account initializers read the tab's PIN (sessionStorage), not `me`, so the pin is made to
-// match the account the server answered as before they run. And the tree is only ever built for one
-// identity: if a mounted tab's identity changes without a reload (its session ended, or the account it
-// was pinned to was signed out in another tab and the server now answers as the browser's default),
-// the tab reloads, like every account switch does (NavSidebar) — which also drops the previous
+// The per-account initializers read the browser's local state, which holds one account's: it is handed
+// to the account the server answered as (claimLocalData) before they run. And the tree is only ever
+// built for one identity: if a mounted tab's identity changes without a reload (its session ended, or
+// another tab signed this browser in as someone else), the tab reloads — which also drops the previous
 // account's query cache, whose keys are not per account.

 /** The quiet screen shown while the tab's identity (or the app chunk) is still on its way — the shared
@@ -76,13 +76,10 @@ export default function BootGate({ children }: { children: ReactNode }) {
        : (me?.id ?? null);

  // Adopt the identity before the tree mounts. A layout effect, so the re-render it triggers happens
-  // before the browser paints: the frame that first shows the app already has the account pinned.
-  //   • Pin this tab to the account the server answered as. Without the pin, a tab riding the
-  //     session's default account would silently swap identity when ANOTHER tab changes that default
-  //     (adds or switches an account); once pinned it sends its own account header. The server honours
-  //     that header only for an account still signed in to this browser, so a pin naming one that is
-  //     not is corrected here — otherwise the tab would run as the default account on the server while
-  //     reading and writing the gone account's local state.
+  // before the browser paints: the frame that first shows the app already reads the account's state.
+  //   • Hand the browser's local state to the account the server answered as: a different account
+  //     than the one whose state the browser holds starts from a clean slate, so it never reads (or
+  //     writes over) the previous one's filters, history or positions.
  //   • Adopt the account's language, which follows the account across devices — the local copy only
  //     knows what this browser last showed. An account with none stored yet (the setup wizard's admin,
  //     an account older than the stamp sign-up gives) stores the language it is shown in: the server
@@ -98,16 +95,16 @@ export default function BootGate({ children }: { children: ReactNode }) {
    if (identity === undefined) return;
    if (readyFor !== undefined) {
      // Already adopted. Only a DIFFERENT identity reloads: the tree was built for another account —
-      // see the header. The pin stays as it is: the tree being torn down still runs its unmount saves
-      // before the page goes, and with the pin moved they would land under the new account's local
-      // keys. The reloaded page's first probe corrects it. Comparing rather than merely testing for
+      // see the header. The local state is not handed over here: the tree being torn down still runs
+      // its unmount saves before the page goes, and they belong to the account it was built for. The
+      // reloaded page's claim settles it. Comparing rather than merely testing for
      // "set" is what makes this effect idempotent, so a re-run for any other reason is a no-op
      // instead of a reload.
      if (readyFor !== identity) location.reload();
      return;
    }
    const account = adoptedMe();
-    if (account && getActiveAccount() !== account.id) setActiveAccount(account.id);
+    if (account) claimLocalData(account.id);
    const lang = account?.preferences?.language;
    if (typeof lang === "string" && isSupported(lang)) {
      // eslint-disable-next-line local/locale-format -- a one-shot read in an effect, compared before adopting the account's language; nothing renders it, so there is no memoized value to go stale.
@@ -121,7 +118,7 @@ export default function BootGate({ children }: { children: ReactNode }) {
        }),
      );
    }
-    // eslint-disable-next-line react-hooks/set-state-in-effect -- the adoption latch. It cannot be derived: the whole point is that the tree mounts only AFTER the pin and the language have been applied, and both are writes to things outside React (sessionStorage, i18n). A layout effect so the frame that first shows the app already has them.
+    // eslint-disable-next-line react-hooks/set-state-in-effect -- the adoption latch. It cannot be derived: the whole point is that the tree mounts only AFTER the local state is claimed and the language applied, and both are writes to things outside React (storage, i18n). A layout effect so the frame that first shows the app already has them.
    setReadyFor(identity);
  }, [identity, readyFor, qc]);

@@ -5,10 +5,10 @@ import { qk } from "../lib/queryKeys";
 import { invalidateSubscription } from "../lib/subscriptionCache";
 import { UserPlus } from "lucide-react";
 import { api, type DiscoveredChannel } from "../lib/api";
-import { accountKey, LS } from "../lib/storage";
+import { LS, ownKey } from "../lib/storage";
 import { formatCountOrDash } from "../lib/format";
-import { sortRows } from "../lib/columnSort";
-import { useSortState } from "../lib/useSortState";
+import { hydrateSort, sortRows, type SortState } from "../lib/columnSort";
+import { usePref } from "../lib/usePref";
 import { useCardPager } from "../lib/useCardPager";
 import { subsColumn } from "./channelColumns";
 import { notify } from "../lib/notifications";
@@ -48,7 +48,10 @@ export default function ChannelDiscovery({
  // The table/cards density is shared with the Subscriptions tab (ChannelsProvider). The sort is
  // this tab's own (its columns differ), lifted out of the table so the card layout shares it.
  const layout = useChannelLayout();
-  const [sort, setSort] = useSortState(LS.channelDiscoverySort);
+  const [sort, setSort] = usePref<SortState>("channelDiscoverySort", null, {
+    localKey: LS.channelDiscoverySort,
+    parse: (raw) => hydrateSort(raw == null ? null : JSON.stringify(raw), []),
+  });

  const query = useQuery({
    queryKey: qk.discoveredChannels(),
@@ -234,7 +237,7 @@ export default function ChannelDiscovery({
            rows={rows}
            columns={columns}
            rowKey={(c) => c.id}
-            persistKey={accountKey(LS.channelDiscoveryTable) ?? undefined}
+            persistKey={ownKey(LS.channelDiscoveryTable) ?? undefined}
            controlsPosition="both"
            controlsInBand
            controlsBandClassName="px-4 max-w-384 mx-auto"
@@ -16,7 +16,7 @@ import ExcludedChannelList from "./ExcludedChannelList";
 import { api, NO_TAG_SENTINEL, type ManagedChannel } from "../lib/api";
 import { Sparkles } from "lucide-react";
 import { notifyYouTubeActionError } from "../lib/youtubeErrors";
-import { accountKey, LS } from "../lib/storage";
+import { LS, ownKey } from "../lib/storage";
 import { buildChannelColumns, applySyncFilter, tagFilterChips } from "./channelColumns";
 import { notify } from "../lib/notifications";
 import { Badge } from "./ui/Badge";
@@ -824,7 +824,7 @@ export default function Channels() {
                rows={syncedChannels}
                columns={columns}
                rowKey={(c) => c.id}
-                persistKey={accountKey(LS.channelsTable) ?? undefined}
+                persistKey={ownKey(LS.channelsTable) ?? undefined}
                // Folded, the status / pager row leaves the band; the list's own end keeps it.
                controlsPosition={fold.folded ? "bottom" : "both"}
                controlsLeading={statusChips}
@@ -4,9 +4,10 @@
 // and a genuine browser Back (popstate carrying `_chTab`) restores + re-persists it. This wiring was
 // re-routed through useAccountPersistedState, so it's exercised here rather than only by hand E2E.
 //
-// Uses REAL storage (jsdom localStorage/sessionStorage) with the tab's account pinned to 1, so
+// Uses REAL storage (jsdom localStorage/sessionStorage) with account 1's local state claimed, so
 // useAccountPersistedState / useSortState / useFilterState all resolve their account-scoped keys; only
-// navigationContext (the sole runtime dependency) is mocked, pinned to the channels page.
+// navigationContext is mocked, pinned to the channels page. The layout is an account preference
+// (usePref), read from the `me` cache BootGate fills before the app mounts.
 import { describe, it, expect, beforeEach, vi } from "vitest";
 import { renderHook, act, render as renderRoot } from "@testing-library/react";
 import { useEffect, useEffectEvent, type ReactNode } from "react";
@@ -17,17 +18,28 @@ vi.mock("./navigationContext", () => ({
 }));

 import { ChannelsProvider } from "./ChannelsProvider";
-import { useChannels, useChannelsActions } from "./channelsContext";
+import { useChannelLayout, useChannels, useChannelsActions } from "./channelsContext";
+import { claimLocalData } from "../lib/storage";
+import { QueryClient, QueryClientProvider } from "@tanstack/react-query";
+import { qk } from "../lib/queryKeys";

+let qc = new QueryClient();
 const wrapper = ({ children }: { children: ReactNode }) => (
-  <ChannelsProvider>{children}</ChannelsProvider>
+  <QueryClientProvider client={qc}>
+    <ChannelsProvider>{children}</ChannelsProvider>
+  </QueryClientProvider>
+);
+const withClient = ({ children }: { children: ReactNode }) => (
+  <QueryClientProvider client={qc}>{children}</QueryClientProvider>
 );
 const render = () => renderHook(() => ({ ...useChannels(), ...useChannelsActions() }), { wrapper });
-const VIEW_KEY = "siftlode.channelsView.1"; // account-scoped (accountKey suffixes the pinned id)
+const VIEW_KEY = "siftlode.channelsView"; // the account's (claimed in beforeEach)

 beforeEach(() => {
  localStorage.clear();
-  sessionStorage.setItem("siftlode.activeAccount", "1"); // pin the tab's account
+  claimLocalData(1); // the browser holds account 1's state
+  qc = new QueryClient();
+  qc.setQueryData(qk.me(), { id: 1, preferences: {} });
  // Reset BOTH the history state and the query string: setView only owns an entry on the channels page,
  // and the URL mirror writes ctab/csort/cf — a stale key left by a prior test would be read back by
  // this provider's mount-time URL seed and change the starting tab.
@@ -107,10 +119,10 @@ describe("ChannelsProvider — shareable URL", () => {
    // Regression: the seed's base for the OMITTED dimensions used to be read as defaults, and
    // persisting the seed overwrote the account's real stored sort + filter map.
    localStorage.setItem(
-      "siftlode.channelSort.1",
+      "siftlode.channelSort",
      JSON.stringify({ key: "subscribers", dir: "desc" }),
    );
-    localStorage.setItem("siftlode.channelsFilterMap.1", JSON.stringify({ sync: "needsFull" }));
+    localStorage.setItem("siftlode.channelsFilterMap", JSON.stringify({ sync: "needsFull" }));
    window.history.replaceState({ sfPage: "channels" }, "", "/?ctab=hidden"); // only the tab in the URL

    const { result } = render();
@@ -120,11 +132,11 @@ describe("ChannelsProvider — shareable URL", () => {
    // ...but the omitted sort + filters keep the account's real stored values, in state AND storage.
    expect(result.current.sort).toEqual({ key: "subscribers", dir: "desc" });
    expect(result.current.filters).toEqual({ sync: "needsFull" });
-    expect(JSON.parse(localStorage.getItem("siftlode.channelSort.1")!)).toEqual({
+    expect(JSON.parse(localStorage.getItem("siftlode.channelSort")!)).toEqual({
      key: "subscribers",
      dir: "desc",
    });
-    expect(JSON.parse(localStorage.getItem("siftlode.channelsFilterMap.1")!)).toEqual({
+    expect(JSON.parse(localStorage.getItem("siftlode.channelsFilterMap")!)).toEqual({
      sync: "needsFull",
    });
  });
@@ -165,6 +177,7 @@ describe("ChannelsProvider — the focus-a-channel intent", () => {
        <Probe />
        <Harness mounted />
      </ChannelsProvider>,
+      { wrapper: withClient },
    );

    act(() => api.focusChannel("Kurzgesagt"));
@@ -201,6 +214,7 @@ describe("ChannelsProvider — the focus-a-channel intent", () => {
        <Probe />
        <Harness mounted />
      </ChannelsProvider>,
+      { wrapper: withClient },
    );

    act(() => api.focusChannel("Kurzgesagt"));
@@ -209,3 +223,14 @@ describe("ChannelsProvider — the focus-a-channel intent", () => {
    expect(api.search).toBe("Kurzgesagt");
  });
 });
+
+describe("ChannelsProvider — the layout", () => {
+  // The table or the cards is a choice about the account, so it follows it to every device: the
+  // server's word wins over what this browser last showed.
+  it("is the account's preference, over this browser's copy", () => {
+    localStorage.setItem("siftlode.channelLayout", "table");
+    qc.setQueryData(qk.me(), { id: 1, preferences: { channelLayout: "cards" } });
+    const { result } = renderHook(useChannelLayout, { wrapper });
+    expect(result.current).toBe("cards");
+  });
+});
@@ -12,6 +12,7 @@ import { parseChannelLayout } from "../lib/channelLayout";
 import { hydrateSort } from "../lib/columnSort";
 import { useLatest } from "../lib/useLatest";
 import { useSortState } from "../lib/useSortState";
+import { usePref } from "../lib/usePref";
 import { useFilterState } from "../lib/useFilterState";
 import { useUrlMirror } from "../lib/useUrlMirror";
 import { rememberNavPosition } from "../lib/history";
@@ -80,7 +81,7 @@ function readStoredFilters(): FilterMap {
 // present, the URL wins over this account's stored view — read once at mount and applied over the slice
 // hooks (see the provider). The keys are channels-unique, so their mere presence identifies a shared
 // manager view; no ?page check is needed (the mirror below is page-gated so it never pollutes others).
-// The base is this account's stored view (BootGate pins the account before this provider mounts), so
+// The base is this account's stored view (BootGate claims the local state before this provider mounts), so
 // a PARTIAL link — one carrying only the tab, say — keeps the stored sort and filter map for the rest.
 function readChannelsUrlSeed(): ChannelsUrlState | null {
  const params = new URLSearchParams(window.location.search);
@@ -100,12 +101,15 @@ export function ChannelsProvider({ children }: { children: ReactNode }) {
  // The subscribed table's column-filter map, owned here (lifted out of DataTable) so the top status
  // chips, the Sync column funnel, and the card layout share one state; useFilterState persists it.
  const [filters, setFilters] = useFilterState(LS.channelsFilterMap, migrateSyncChip);
-  // view + layout are persisted tabs (useAccountPersistedState = the shared account-scoped tab-state hook).
-  // Stored un-clamped; clampView/parseChannelLayout heal them for display. setView layers the history
-  // logic (below) on top of the raw setter.
+  // The view is this browser's (useAccountPersistedState), the layout an account preference
+  // (usePref). Stored un-clamped; clampView/parseChannelLayout heal them for display. setView layers
+  // the history logic (below) on top of the raw setter.
  const [viewRaw, setViewRaw] = useAccountPersistedState(LS.channelsView, "subscribed");
  const view = clampView(viewRaw);
-  const [layoutRaw, setLayout] = useAccountPersistedState(LS.channelLayout, "table");
+  const [layoutRaw, setLayout] = usePref<string>("channelLayout", "table", {
+    localKey: LS.channelLayout,
+    parse: (raw) => (typeof raw === "string" ? raw : "table"),
+  });
  const layout = parseChannelLayout(layoutRaw);
  // Sort is owned here (lifted out of DataTable) so the card layout shares it; ColumnSortControl heals
  // it against the live columns for display, so the provider stores it un-healed (no options passed).
@@ -5,30 +5,31 @@
 // the saved-view and URL paths do — so an upgrading user's topic any/all match isn't silently lost.
 import { describe, it, expect, beforeEach } from "vitest";
 import { loadAccountFilters } from "./feedFiltersInit";
-import { accountKey, LS } from "../lib/storage";
+import { claimLocalData, LS } from "../lib/storage";

-const ID = 7;
-
-beforeEach(() => localStorage.clear());
+beforeEach(() => {
+  localStorage.clear();
+  claimLocalData(7);
+});

 describe("loadAccountFilters — legacy tagMode migration on seed", () => {
  it("migrates a persisted legacy tagMode:'and' from the last-session blob into topic", () => {
-    localStorage.setItem(accountKey(LS.filters, ID)!, JSON.stringify({ q: "hi", tagMode: "and" }));
-    const f = loadAccountFilters(ID);
+    localStorage.setItem(LS.filters, JSON.stringify({ q: "hi", tagMode: "and" }));
+    const f = loadAccountFilters();
    expect(f.tagModes).toEqual({ language: "or", topic: "and", other: "or" });
    expect(f.q).toBe("hi"); // other fields preserved through the merge
  });

  it("leaves tagModes at defaults when nothing is stored", () => {
-    expect(loadAccountFilters(ID).tagModes).toEqual({ language: "or", topic: "or", other: "or" });
+    expect(loadAccountFilters().tagModes).toEqual({ language: "or", topic: "or", other: "or" });
  });

  it("preserves an already-migrated per-category tagModes blob", () => {
    localStorage.setItem(
-      accountKey(LS.filters, ID)!,
+      LS.filters,
      JSON.stringify({ tagModes: { language: "and", topic: "or", other: "and" } }),
    );
-    expect(loadAccountFilters(ID).tagModes).toEqual({
+    expect(loadAccountFilters().tagModes).toEqual({
      language: "and",
      topic: "or",
      other: "and",
@@ -8,9 +8,9 @@ import { QueryClient, QueryClientProvider } from "@tanstack/react-query";
 import { NavigationProvider } from "./NavigationProvider";
 import { FeedFiltersProvider } from "./FeedFiltersProvider";
 import { useFeedFilters } from "./feedFiltersContext";
-import { ACTIVE_ACCOUNT_KEY } from "../lib/activeAccount";
 import { qk } from "../lib/queryKeys";
 import type { Me } from "../lib/api";
+import { claimLocalData } from "../lib/storage";

 const scopes: string[] = [];
 function Probe() {
@@ -34,7 +34,7 @@ function mount(me: Partial<Me>) {

 beforeEach(() => {
  localStorage.clear();
-  sessionStorage.setItem(ACTIVE_ACCOUNT_KEY, "5");
+  claimLocalData(5);
  scopes.length = 0;
  window.history.replaceState(null, "", "/");
 });
@@ -53,6 +53,6 @@ describe("FeedFiltersProvider seed", () => {
    window.history.replaceState(null, "", "/?scope=all");
    mount({});
    expect(scopes[0]).toBe("all");
-    expect(JSON.parse(localStorage.getItem("siftlode.filters.5") ?? "{}").scope).toBe("all");
+    expect(JSON.parse(localStorage.getItem("siftlode.filters") ?? "{}").scope).toBe("all");
  });
 });
@@ -1,8 +1,8 @@
 import { useCallback, useEffect, useMemo, useState, type ReactNode } from "react";
-import { getActiveAccount, type FeedFilters } from "../lib/api";
+import { type FeedFilters } from "../lib/api";
 import { filterParamUpdates, hasFilterParams, paramsToFilters } from "../lib/urlState";
 import { useUrlMirror } from "../lib/useUrlMirror";
-import { accountKey, writeJSON } from "../lib/storage";
+import { ownKey, writeJSON } from "../lib/storage";
 import { logFilterChange } from "../lib/filterLog";
 import { useNavigation } from "./navigationContext";
 import {
@@ -29,12 +29,12 @@ import { DEFAULT_FILTERS, FILTERS_KEY, loadAccountFilters } from "./feedFiltersI
 function loadInitialFilters(): FeedFilters {
  const params = new URLSearchParams(window.location.search);
  if (hasFilterParams(params)) return paramsToFilters(params, DEFAULT_FILTERS);
-  return loadAccountFilters(getActiveAccount());
+  return loadAccountFilters();
 }

 export function FeedFiltersProvider({ children }: { children: ReactNode }) {
  // Seeded for the account this tab is — BootGate mounts this provider only once the account is
-  // pinned, so this is the account's own view from the first render.
+  // claimed, so this is the account's own view from the first render.
  const [filters, setFiltersState] = useState<FeedFilters>(loadInitialFilters);
  // Whether this load arrived with feed-filter params in the URL (a bookmark/shared link). Captured
  // once at mount — such filters win over the account's stored view and are persisted to the account.
@@ -45,8 +45,8 @@ export function FeedFiltersProvider({ children }: { children: ReactNode }) {
  const setFilters = useCallback((next: FeedFilters) => {
    setFiltersState(next);
    logFilterChange("feed", next, next.q);
-    // Persist under THIS tab's active account so filters don't bleed between accounts.
-    const key = accountKey(FILTERS_KEY, getActiveAccount());
+    // Persist as the account's own view.
+    const key = ownKey(FILTERS_KEY);
    if (key) writeJSON(key, next);
  }, []);

@@ -16,7 +16,7 @@ import { formatBytes, formatDateTime, relativeTime } from "../lib/format";
 import { focusNavItem } from "../lib/listNav";
 import { notify } from "../lib/notifications";
 import { qk } from "../lib/queryKeys";
-import { LS, accountKey } from "../lib/storage";
+import { LS, ownKey } from "../lib/storage";
 import { useCardPager } from "../lib/useCardPager";
 import { useDebounced } from "../lib/useDebounced";
 import { useLiveQuery } from "../lib/useLiveQuery";
@@ -628,7 +628,7 @@ export default function Files() {
        rows={rows}
        columns={columns}
        rowKey={(f) => String(f.id)}
-        persistKey={accountKey(LS.filesTable) ?? undefined}
+        persistKey={ownKey(LS.filesTable) ?? undefined}
        pageSize={50}
        keyboardNav
        fixedLayout
@@ -2,7 +2,7 @@
 //
 // FilesProvider's arrival at one file: `openFile` (a notice's "Open") and a `?item=` link both make a
 // pending focus that the Files page spends once it has looked; what it found is held until the page
-// is left. Real storage with the tab's account pinned; only the navigation is mocked, its page
+// is left. Real storage, claimed for the account; only the navigation is mocked, its page
 // switchable.
 import { beforeEach, describe, expect, it, vi } from "vitest";
 import { act, renderHook } from "@testing-library/react";
@@ -16,6 +16,7 @@ vi.mock("./navigationContext", () => ({

 import { FilesProvider } from "./FilesProvider";
 import { ALL_FILES, useFiles, useFilesActions } from "./filesContext";
+import { claimLocalData } from "../lib/storage";

 const wrapper = ({ children }: { children: ReactNode }) => (
  <FilesProvider>{children}</FilesProvider>
@@ -24,7 +25,7 @@ const render = () => renderHook(() => ({ ...useFiles(), ...useFilesActions() }),

 beforeEach(() => {
  localStorage.clear();
-  sessionStorage.setItem("siftlode.activeAccount", "1");
+  claimLocalData(1);
  window.history.replaceState(null, "", "/");
  nav.page = "files";
  nav.setPage.mockClear();
@@ -0,0 +1,125 @@
+// @vitest-environment jsdom
+//
+// The account panel of the rail: one account per browser session, so it lists no other accounts. It
+// offers "Sign in as someone else" (straight to Google's account chooser, whose pick replaces this
+// account) where the instance has Google sign-in, and "Sign out", which ends the session.
+import { afterEach, beforeEach, describe, expect, it, vi } from "vitest";
+import { cleanup, fireEvent, render, screen } from "@testing-library/react";
+import { QueryClient, QueryClientProvider } from "@tanstack/react-query";
+import i18n from "../i18n";
+import { api, HttpError, type Me } from "../lib/api";
+import * as actionErrors from "../lib/actionErrors";
+import NavSidebar from "./NavSidebar";
+
+vi.mock("./navigationContext", () => ({
+  useNavigation: () => ({ page: "feed" }),
+  useNavigationActions: () => ({ setPage: vi.fn() }),
+}));
+vi.mock("./prefsContext", () => ({ usePrefs: () => ({ railLabels: false }) }));
+
+const me = (over: Partial<Me> = {}) =>
+  ({
+    id: 5,
+    email: "me@example.com",
+    display_name: "Me",
+    role: "user",
+    avatar_url: null,
+    google_email: null,
+    google_enabled: true,
+    plex_enabled: false,
+    preferences: {},
+    ...over,
+  }) as unknown as Me;
+
+const row = (panel: HTMLElement, text: string) =>
+  [...panel.querySelectorAll("button")].find((b) => b.textContent === text)!;
+
+let href = "";
+const reload = vi.fn();
+
+async function mount(account: Me) {
+  const qc = new QueryClient({ defaultOptions: { queries: { retry: false } } });
+  render(
+    <QueryClientProvider client={qc}>
+      <NavSidebar me={account} onOpenAbout={() => {}} onChangeLanguage={() => {}} language="en" />
+    </QueryClientProvider>,
+  );
+  fireEvent.click(screen.getByRole("button", { name: /^Me · / }));
+  // Positioned by floating-ui, which never settles in jsdom: the panel stays `visibility: hidden`.
+  return vi.waitFor(() => {
+    const panel = document.querySelector<HTMLElement>('[role="dialog"][aria-label="Me"]');
+    if (!panel) throw new Error("the account panel did not open");
+    return panel;
+  });
+}
+
+beforeEach(async () => {
+  class Stub {
+    observe() {}
+    unobserve() {}
+    disconnect() {}
+    takeRecords() {
+      return [];
+    }
+  }
+  globalThis.ResizeObserver ??= Stub as unknown as typeof ResizeObserver;
+  await i18n.changeLanguage("en");
+  href = "";
+  reload.mockReset();
+  vi.stubGlobal("location", {
+    ...window.location,
+    reload,
+    set href(v: string) {
+      href = v;
+    },
+    get href() {
+      return href;
+    },
+  });
+  vi.spyOn(api, "notificationUnreadCount").mockResolvedValue({ count: 0 } as never);
+});
+afterEach(() => {
+  cleanup();
+  vi.restoreAllMocks();
+  vi.unstubAllGlobals();
+});
+
+describe("the account panel", () => {
+  it("offers signing in as someone else, which goes straight to Google's account chooser", async () => {
+    const panel = await mount(me());
+    const rows = [...panel.querySelectorAll("button")].map((b) => b.textContent?.trim());
+    expect(rows).toEqual(["Sign in as someone else", "Sign out"]);
+
+    fireEvent.click(row(panel, "Sign in as someone else"));
+    expect(href).toBe("/auth/login");
+  });
+
+  it("offers only signing out where the instance has no Google sign-in", async () => {
+    const panel = await mount(me({ google_enabled: false }));
+    const rows = [...panel.querySelectorAll("button")].map((b) => b.textContent?.trim());
+    expect(rows).toEqual(["Sign out"]);
+  });
+
+  it("signing out ends the session, then reloads onto the sign-in page", async () => {
+    const logout = vi.spyOn(api, "logout").mockResolvedValue({ ok: true });
+    const panel = await mount(me());
+
+    fireEvent.click(row(panel, "Sign out"));
+
+    await vi.waitFor(() => expect(reload).toHaveBeenCalledOnce());
+    expect(logout).toHaveBeenCalledOnce();
+  });
+
+  // Reloading after a sign-out that failed only brought the session back, with no word of why.
+  it("a sign-out that failed says so and keeps the page", async () => {
+    vi.spyOn(api, "logout").mockRejectedValue(new HttpError(500, "boom"));
+    const report = vi.spyOn(actionErrors, "notifyActionError").mockImplementation(() => {});
+    const panel = await mount(me());
+
+    fireEvent.click(row(panel, "Sign out"));
+
+    await vi.waitFor(() => expect(report).toHaveBeenCalledOnce());
+    expect(report.mock.calls[0]![1]).toBe("Couldn't sign you out");
+    expect(reload).not.toHaveBeenCalled();
+  });
+});
@@ -1,12 +1,10 @@
 import { useState, useSyncExternalStore } from "react";
 import { qk } from "../lib/queryKeys";
 import { useTranslation } from "react-i18next";
-import { useQuery } from "@tanstack/react-query";
 import { useNavigation, useNavigationActions } from "./navigationContext";
-import { Info, LogOut, Settings, Shield, UserPlus, type LucideIcon } from "lucide-react";
+import { ArrowLeftRight, Info, LogOut, Settings, Shield, type LucideIcon } from "lucide-react";
 import { api, type Me } from "../lib/api";
 import { displayName, roleKey } from "../lib/api/account";
-import { pinOnNextPage } from "../lib/activeAccount";
 import { useLiveQuery } from "../lib/useLiveQuery";
 import { getUnreadCount, subscribe } from "../lib/notifications";
 import type { Page } from "../lib/urlState";
@@ -36,6 +34,7 @@ import {
 } from "./ui/NavRail";
 import { usePrefs } from "./prefsContext";
 import { pageHref } from "../lib/nav";
+import { notifyActionError } from "../lib/actionErrors";

 // Primary app navigation: a thin icon-only left rail. The modules used to hide under the avatar
 // dropdown; they now live here, each revealing its name in a hover/focus flyout label. Account
@@ -59,8 +58,8 @@ export default function NavSidebar({
  // keyboard focus (a rest-to-show flyout via Tooltip), so the names are one glance away without the
  // rail ever growing.
  const [acctOpen, setAcctOpen] = useState(false);
-  // The account panel holds mixed content with its own tab order (profile, account switches, two
-  // actions), so it is a `dialog` popover, not a roving menu. It portals out of the nav — whose
+  // The account panel holds mixed content with its own tab order (the profile, then its actions),
+  // so it is a `dialog` popover, not a roving menu. It portals out of the nav — whose
  // backdrop-filter would otherwise trap fixed positioning + stacking and let clicks fall through
  // to the controls behind it — and anchors to the RIGHT of the button, bottom-aligned, as its
  // hand-rolled placement did. Dismissal (outside press, topmost-only Escape, focus return) now
@@ -76,39 +75,16 @@ export default function NavSidebar({
  const { setReference: setAcctAnchor } = acctPop.refs;

  async function logout() {
-    // Signs THIS tab's active account out of the browser wallet (server is per-tab aware); then
-    // drops this tab's override and reloads onto whatever account remains the default.
-    try {
-      await api.logout();
-    } catch {
-      /* clear locally and reload regardless */
-    }
-    pinOnNextPage(null);
-    location.reload();
-  }
-
-  // Accounts that have signed in on this browser (loaded only while the popover is open).
-  const accountsQuery = useQuery({
-    queryKey: qk.accounts(),
-    queryFn: api.accounts,
-    enabled: acctOpen,
-  });
-  const otherAccounts = (accountsQuery.data ?? []).filter((a) => !a.active);
-
-  function switchTo(id: number) {
-    // Per-tab switch: point THIS tab at the account and reload; other tabs keep their own
-    // identity. No server round-trip — req() sends the account header (validated against the
-    // browser wallet server-side), so the cookie's default account is left untouched. The pin moves with
-    // the reload, not before it: the page going away still unloads as the account it was built for.
-    pinOnNextPage(id);
-    // Drop the previous account's in-module sub-view/overlay before the reload (which would
-    // otherwise preserve history.state across the swap). Otherwise the new account lands on a
-    // stale sub-view that belongs to the OLD identity. Start at the module root instead.
-    const st = { ...(window.history.state || {}) };
-    delete st._sub;
-    delete st._ov;
-    window.history.replaceState(st, "");
-    location.reload(); // cleanest way to reload all per-user state for the new account
+    // Ends this browser's session, then reloads onto the sign-in page. A sign-out that failed says
+    // so and stays: the reload would only bring back the session that is still there.
+    const signedOut = await api.logout().then(
+      () => true,
+      (err: unknown) => {
+        notifyActionError(err, t("errors.writeFailed.signOut"));
+        return false;
+      },
+    );
+    if (signedOut) location.reload();
  }

  // Durable, server-backed unread count for the inbox badge. Polled live (pauses when the
@@ -270,44 +246,21 @@ export default function NavSidebar({
                  {t("header.account.admin")}
                </div>
              )}
-              {otherAccounts.length > 0 && (
-                <div className="mt-2 pt-2 border-t border-border flex flex-col gap-0.5">
-                  {otherAccounts.map((a) => (
-                    <Pressable
-                      key={a.id}
-                      onClick={() => switchTo(a.id)}
-                      title={t("header.account.switchTo", { name: a.display_name ?? a.email })}
-                      className="selectable w-full flex items-center gap-2.5 px-2 py-1.5 rounded-lg transition touch-target"
-                    >
-                      <AvatarImg
-                        src={a.avatar_url}
-                        fallback={a.display_name ?? a.email}
-                        className="w-6 h-6 rounded-full text-3xs shrink-0"
-                      />
-                      <span className="min-w-0">
-                        <span className="block text-[0.8125rem] text-fg truncate">
-                          {displayName(a)}
-                        </span>
-                        <span className="block text-2xs text-muted truncate">{a.email}</span>
-                      </span>
-                    </Pressable>
-                  ))}
-                </div>
-              )}
              <div className="mt-2 pt-2 border-t border-border flex flex-col">
-                <Pressable
-                  onClick={() => {
-                    // Adding an account switches THIS tab to it: drop the tab's pin so that on
-                    // return from Google it adopts the freshly-added account (the new default)
-                    // and pins that. Other tabs keep their own pinned account.
-                    pinOnNextPage(null);
-                    window.location.href = "/auth/login";
-                  }}
-                  className="selectable w-full flex items-center gap-2 text-sm px-2 py-2 rounded-lg text-muted hover:text-fg transition touch-target"
-                >
-                  <UserPlus className="w-4 h-4" />
-                  {t("header.account.addAccount")}
-                </Pressable>
+                {/* Straight to Google's account chooser: the account picked there takes this one's
+                    place in the browser (one account per session), and a cancel there leaves this
+                    one signed in. A password account signs in from the sign-in page, after Sign out. */}
+                {me.google_enabled && (
+                  <Pressable
+                    onClick={() => {
+                      window.location.href = "/auth/login";
+                    }}
+                    className="selectable w-full flex items-center gap-2 text-sm px-2 py-2 rounded-lg text-muted hover:text-fg transition touch-target"
+                  >
+                    <ArrowLeftRight className="w-4 h-4" />
+                    {t("header.account.signInAsOther")}
+                  </Pressable>
+                )}
                <Pressable
                  onClick={logout}
                  className="selectable w-full flex items-center gap-2 text-sm px-2 py-2 rounded-lg text-muted hover:text-fg transition touch-target"
@@ -31,8 +31,9 @@ import { useBackToClose } from "../lib/history";
 import { PageHeading } from "./ui/Headings";

 import { qk } from "../lib/queryKeys";
-import { LS, setAccountRaw } from "../lib/storage";
-import { PAGE_KEY, START_ON_HOME } from "./navigationInit";
+import { claimLocalData, getAccountRaw, LS, setAccountRaw } from "../lib/storage";
+import { api } from "../lib/api";
+import { PAGE_KEY } from "./navigationInit";

 /** The Navigation API's view of the tab: `below` = whether each entry under the current one belongs
 *  to this document, nearest first. jsdom has no Navigation API; Chrome's says this. */
@@ -79,9 +80,10 @@ async function openOverlay() {

 beforeEach(() => {
  qc = new QueryClient();
-  qc.setQueryData(qk.me(), { id: 1 });
+  // An admin on a Plex-enabled instance: every module is open to it, so no gate moves a page here.
+  qc.setQueryData(qk.me(), { id: 1, role: "admin", plex_enabled: true });
  localStorage.clear();
-  sessionStorage.setItem("siftlode.activeAccount", "1"); // account-scoped page memory
+  claimLocalData(1); // the account's page memory
  window.history.replaceState({ sfPage: "feed" }, "", "/");
 });
 // Not automatic: vitest runs without `globals`, so RTL never registers its own afterEach — leaving a
@@ -262,7 +264,12 @@ describe("NavigationProvider · Back", () => {
  });

  it("a remembered retired page opens its tab", () => {
-    setAccountRaw(LS.startOnHome, START_ON_HOME.off);
+    qc.setQueryData(qk.me(), {
+      id: 1,
+      role: "admin",
+      plex_enabled: true,
+      preferences: { startOnHome: false },
+    });
    setAccountRaw(PAGE_KEY, "logs");
    window.history.replaceState(null, "", "/");
    const { result } = render();
@@ -270,6 +277,19 @@ describe("NavigationProvider · Back", () => {
    expect(new URLSearchParams(window.location.search).get("at")).toBe("logs");
  });

+  // Start-on-Home is the account's preference. An opt-out this browser kept before the server held it
+  // (the string "0") still counts, and reaches the server for the account's other devices.
+  it("takes this browser's earlier opt-out of Home, and saves it to the account", () => {
+    const save = vi.spyOn(api, "savePrefs").mockResolvedValue(undefined as never);
+    qc.setQueryData(qk.me(), { id: 1, role: "admin", plex_enabled: true, preferences: {} });
+    setAccountRaw(LS.startOnHome, "0");
+    setAccountRaw(PAGE_KEY, "plex");
+    window.history.replaceState(null, "", "/");
+    const { result } = render();
+    expect(result.current.page).toBe("plex");
+    expect(save).toHaveBeenCalledWith({ startOnHome: false });
+  });
+
  it("opens a retired page's tab when Back lands on its entry", () => {
    const { result } = render();
    act(() => result.current.setPage("plex"));
@@ -295,6 +315,77 @@ describe("NavigationProvider · Back", () => {
  });
 });

+describe("NavigationProvider · a page the account cannot open", () => {
+  const member = { id: 1, role: "user", plex_enabled: true };
+
+  // A member following a link to an admin page saw the feed with no rail row current: the page
+  // stayed "scheduler" while only the content fell back.
+  it("lands on the feed, and the address bar says so", () => {
+    qc.setQueryData(qk.me(), member);
+    window.history.replaceState(null, "", "/?page=scheduler&keep=1");
+    const { result } = render();
+    expect(result.current.page).toBe("feed");
+    expect(window.location.search).toBe("?keep=1");
+    expect(window.history.state).toMatchObject({ sfPage: "feed" });
+  });
+
+  it("lands on the feed when Back reaches an entry naming one", () => {
+    qc.setQueryData(qk.me(), member);
+    const { result } = render();
+    act(() => result.current.setPage("files"));
+    act(() => {
+      window.history.replaceState({ sfPage: "config" }, "", "/?page=config");
+      window.dispatchEvent(new PopStateEvent("popstate", { state: { sfPage: "config" } }));
+    });
+    expect(result.current.page).toBe("feed");
+  });
+
+  it("leaves the page it is on once the account loses it", async () => {
+    window.history.replaceState(null, "", "/?page=users");
+    const { result } = render();
+    expect(result.current.page).toBe("users");
+    act(() => {
+      qc.setQueryData(qk.me(), member);
+    });
+    await settle();
+    expect(result.current.page).toBe("feed");
+  });
+
+  // Kept behind the feed, the lost page stayed in the address bar, a click on Feed did nothing (the
+  // feed was already on screen), and a later grant switched the page unasked.
+  it("gives the lost page up: the address bar, the remembered page and a later grant agree", async () => {
+    window.history.replaceState({ sfPage: "users" }, "", "/?page=users");
+    const { result } = render();
+    expect(result.current.page).toBe("users");
+    act(() => {
+      qc.setQueryData(qk.me(), member);
+    });
+    await settle();
+    expect(window.location.search).toBe("");
+    expect(window.history.state).toMatchObject({ sfPage: "feed" });
+    expect(getAccountRaw(PAGE_KEY)).toBe("feed");
+    act(() => {
+      qc.setQueryData(qk.me(), { ...member, role: "admin" });
+    });
+    await settle();
+    expect(result.current.page).toBe("feed");
+  });
+
+  it("closes a channel page open over the lost page, on screen and in the entry", async () => {
+    window.history.replaceState({ sfPage: "users" }, "", "/?page=users");
+    const { result } = render();
+    act(() => result.current.openChannel("UC1", "One"));
+    expect(window.location.search).toContain("chan=UC1");
+    act(() => {
+      qc.setQueryData(qk.me(), member);
+    });
+    await settle();
+    expect(result.current.channelView).toBeNull();
+    expect(window.location.search).toBe("");
+    expect(window.history.state?._chan).toBeUndefined();
+  });
+});
+
 describe("NavigationProvider · leaving a sub-view whose entry is the oldest of this document", () => {
  // A full tab can have dropped the entry under the one the channel page or the search pushed (see
  // lib/history's backStaysInDocument): leaving clears it in place instead of stepping out.
@@ -14,7 +14,16 @@ import {
  notifyNav,
  rememberNavPosition,
 } from "../lib/history";
-import { channelViewFrom, loadInitialPage, PAGE_KEY, type ChannelView } from "./navigationInit";
+import { reachablePage } from "../lib/modules";
+import {
+  channelViewFrom,
+  loadInitialPage,
+  PAGE_KEY,
+  START_ON_HOME_OPTIONS,
+  START_ON_HOME_PREF,
+  type ChannelView,
+} from "./navigationInit";
+import { usePref } from "../lib/usePref";
 import {
  NavigationActionsContext,
  NavigationStateContext,
@@ -29,9 +38,35 @@ import {
 //
 // The contexts, their value types and the hooks live in navigationContext.ts.

+/** The address bar of `page`'s root: the page (feed implicit) with every view-specific key cleared. */
+function pageRootParams(page: Page): Record<string, string | null> {
+  return {
+    page: page === "feed" ? null : page,
+    chan: null,
+    v: null,
+    pl: null,
+    pscope: null,
+    psort: null,
+    pshow: null,
+    pf: null,
+    ctab: null,
+    csort: null,
+    cf: null,
+    ...Object.fromEntries(ACTIVITY_KEYS.map((k) => [k, null])),
+  };
+}
+
 export function NavigationProvider({ children }: { children: ReactNode }) {
  const { t, i18n } = useTranslation();
-  const [page, setPageState] = useState<Page>(loadInitialPage);
+  // Held here as well as in Settings: this is the instance mounted on every boot, so a value only
+  // this browser held reaches the server at once.
+  const [startOnHome] = usePref(START_ON_HOME_PREF, true, START_ON_HOME_OPTIONS);
+  const [chosenPage, setPageState] = useState<Page>(() => loadInitialPage(startOnHome));
+  // The page on screen: the chosen one when this account may open it, else the feed — derived, so a
+  // link, a Back and a role taken away mid-session all land the same way, and the rail, the content
+  // and the address bar (stamped from it below) agree.
+  const me = useMe();
+  const page = reachablePage(chosenPage, me);
  // Ephemeral: not persisted and not in the URL, so a reload returns to the normal feed (a search
  // spends quota, so we never auto-replay it) — UNLESS it was zero-quota scrape-sourced, which rides
  // in history.state and is free to re-fetch. The popstate handler derives it from history.state so
@@ -47,6 +82,18 @@ export function NavigationProvider({ children }: { children: ReactNode }) {
  const [channelView, setChannelView] = useState<ChannelView | null>(() =>
    channelViewFrom(window.history.state, window.location.search),
  );
+  // A page this account can no longer open (a role taken away, a module switched off mid-session)
+  // is given up, not kept behind the feed: kept, the address bar would go on naming it, a click on
+  // Feed would be a no-op (setPage's same-page guard sees the feed already on screen), and a later
+  // grant would switch the page by itself. A channel page open over it closes with it, as on any
+  // page switch. Adjusted during render; the history entry and the remembered page follow in the
+  // effect below.
+  const [givenUp, setGivenUp] = useState(0);
+  if (page !== chosenPage) {
+    setPageState(page);
+    setChannelView(null);
+    setGivenUp((n) => n + 1);
+  }

  // Latest-value refs so the STABLE action callbacks can read the latest state without depending on
  // it — a dep would recreate them on every navigation and re-render every actions consumer (e.g.
@@ -80,20 +127,7 @@ export function NavigationProvider({ children }: { children: ReactNode }) {
        window.history.pushState(
          { sfPage: next, sfScroll: nextNavId() },
          "",
-          urlWithParams({
-            page: next === "feed" ? null : next,
-            chan: null,
-            v: null,
-            pl: null,
-            pscope: null,
-            psort: null,
-            pshow: null,
-            pf: null,
-            ctab: null,
-            csort: null,
-            cf: null,
-            ...Object.fromEntries(ACTIVITY_KEYS.map((k) => [k, null])),
-          }),
+          urlWithParams(pageRootParams(next)),
        );
        notifyNav();
        then?.();
@@ -171,13 +205,27 @@ export function NavigationProvider({ children }: { children: ReactNode }) {
  // title, a playlist — `PageHeading` reports it), so a channel opened from a link is named the same
  // as one opened in the app. Signed out, the page on screen is the public landing whatever `page`
  // says — the tab read "What should I watch?" over the sign-in form (prod UAT 2026-09-19).
-  const signedIn = useMe() != null;
+  const signedIn = me != null;
  const objectTitle = useDocumentObjectTitle();
  useEffect(() => {
    const parts = signedIn ? [objectTitle, t(pageTitleKey(page))] : [];
    document.title = [...parts.filter(Boolean), "Siftlode"].join(" · ");
  }, [signedIn, page, objectTitle, i18n.language, t]);

+  // The page given up above is the feed, and the current entry says so (replaced, not pushed: no
+  // step was taken). Its own keys go with it, and the channel page closed with it, as on any page
+  // switch — left in the entry, a reload would reopen what the screen no longer shows.
+  useEffect(() => {
+    if (!givenUp) return;
+    setAccountRaw(PAGE_KEY, "feed");
+    const { _chan: _closedChan, _chanName: _closedName, ...rest } = window.history.state ?? {};
+    window.history.replaceState(
+      { ...rest, sfPage: "feed" },
+      "",
+      urlWithParams(pageRootParams("feed")),
+    );
+  }, [givenUp]);
+
  // The page this tab LOADED on. Frozen, because the stamp below belongs to the initial history
  // entry: every later page change writes its own entry through setPage.
  const [mountPage] = useState(page);
@@ -33,7 +33,7 @@ afterEach(() => {
 });

 describe("the onboarding wizard's Escape", () => {
-  const mount = (onClose: () => void, dialog: boolean) =>
+  const mount = (onClose: (dismiss: boolean) => void, dialog: boolean) =>
    render(
      <QueryClientProvider
        client={new QueryClient({ defaultOptions: { queries: { retry: false } } })}
@@ -51,7 +51,8 @@ describe("the onboarding wizard's Escape", () => {
    const onClose = vi.fn();
    mount(onClose, false);
    fireEvent.keyDown(document, { key: "Escape" });
-    expect(onClose).toHaveBeenCalledTimes(1);
+    // Read was never granted: leaving now stops the wizard opening by itself.
+    expect(onClose).toHaveBeenCalledExactlyOnceWith(true);
  });

  it("leaves the wizard alone while a dialog is open over it", () => {
@@ -65,7 +65,14 @@ function Dots({ index, total }: { index: number; total: number }) {
  );
 }

-export default function OnboardingWizard({ me, onClose }: { me: Me; onClose: () => void }) {
+export default function OnboardingWizard({
+  me,
+  onClose,
+}: {
+  me: Me;
+  /** `dismiss`: the essential read step was skipped, so the wizard stops opening by itself. */
+  onClose: (dismiss: boolean) => void;
+}) {
  const { t } = useTranslation();
  const qc = useQueryClient();
  const importTriggered = useRef(false);
@@ -110,8 +117,8 @@ export default function OnboardingWizard({ me, onClose }: { me: Me; onClose: ()
  // Closing without finishing the essential read step suppresses future auto-popups
  // (it stays reachable from Settings). Once read is granted, closing is just "done".
  const close = () => {
-    endOnboarding(!me.can_read);
-    onClose();
+    endOnboarding();
+    onClose(!me.can_read);
  };
  const overlayOrder = useOverlayOrder(); // this overlay's toast-host rank (see ui/Toaster)

@@ -274,8 +281,8 @@ export default function OnboardingWizard({ me, onClose }: { me: Me; onClose: ()
                  variant="ghost"
                  size="md"
                  onClick={() => {
-                    endOnboarding(false);
-                    onClose();
+                    endOnboarding();
+                    onClose(false);
                  }}
                >
                  {t("onboarding.write.keepReadOnly")}
@@ -299,8 +306,8 @@ export default function OnboardingWizard({ me, onClose }: { me: Me; onClose: ()
                size="lg"
                fullWidth
                onClick={() => {
-                  endOnboarding(false);
-                  onClose();
+                  endOnboarding();
+                  onClose(false);
                }}
              >
                {t("onboarding.done.done")}
@@ -9,7 +9,8 @@ import { compareText } from "../lib/collate";
 import { formatRuntime } from "../lib/format";
 import { notify } from "../lib/notifications";
 import { notifyYouTubeActionError } from "../lib/youtubeErrors";
-import { LS, readAccountMerged, writeAccount } from "../lib/storage";
+import { LS } from "../lib/storage";
+import { usePref } from "../lib/usePref";
 import { type PanelLayout } from "../lib/panelLayout";
 import FilterRail from "./FilterRail";
 import { Button } from "./ui/Button";
@@ -65,7 +66,10 @@ export default function PlaylistsRail({
  const { t, i18n } = useTranslation();
  const lang = i18n.language;
  const qc = useQueryClient();
-  const [plSort, setPlSort] = useState<PlSort>(() => readAccountMerged(LS.plSort, PL_SORT_DEFAULT));
+  const [plSort, setPlSort] = usePref<PlSort>("plSort", PL_SORT_DEFAULT, {
+    localKey: LS.plSort,
+    parse: (raw) => ({ ...PL_SORT_DEFAULT, ...(raw && typeof raw === "object" ? raw : {}) }),
+  });
  const plSortOptions: SortOption[] = [
    { key: "custom", label: t("playlists.railSortCustom"), directionless: true },
    { key: "name", label: t("playlists.railSortName"), defaultDir: "asc" },
@@ -76,9 +80,6 @@ export default function PlaylistsRail({
    // default: a rail that reshuffles itself while you look at it costs more than it saves.
    { key: "recent", label: t("playlists.railSortRecent"), defaultDir: "desc" },
  ];
-  useEffect(() => {
-    writeAccount(LS.plSort, plSort);
-  }, [plSort]);
  const [newName, setNewName] = useState("");
  const [syncing, setSyncing] = useState(false);
  const plActions = usePlaylistActions();
@@ -23,8 +23,8 @@ vi.mock("react-i18next", async (importOriginal) => {

 import PlexPlayer from "./PlexPlayer";
 import { api, type PlexItemDetail } from "../lib/api";
-import { ACTIVE_ACCOUNT_KEY } from "../lib/activeAccount";
 import { pushLayer, removeLayer } from "../lib/layerStack";
+import { claimLocalData } from "../lib/storage";

 type Media = HTMLMediaElement & { _t?: number; _paused?: boolean };

@@ -87,7 +87,7 @@ beforeAll(() => {
 let play: ReturnType<typeof vi.fn>;
 let pause: ReturnType<typeof vi.fn>;
 beforeEach(() => {
-  sessionStorage.setItem(ACTIVE_ACCOUNT_KEY, "7");
+  claimLocalData(7);
  play = vi.fn(function (this: Media) {
    this._paused = false;
    this.dispatchEvent(new Event("play"));
@@ -1034,7 +1034,7 @@ describe("PlexPlayer — clip mode", () => {
  });

  const prefs = (p: object) =>
-    localStorage.setItem("siftlode.plexPlayerPrefs.7", JSON.stringify({ subLang: "en", ...p }));
+    localStorage.setItem("siftlode.plexPlayerPrefs", JSON.stringify({ subLang: "en", ...p }));
  const infoOverlay = () => document.querySelector("[class*='bg-black/70']");

  it("Escape closes what is over the picture first, then the crop and the panel", async () => {
@@ -50,7 +50,7 @@ vi.mock("hls.js", async (importOriginal) => {
 import Hls from "hls.js";
 import PlexPlayer from "./PlexPlayer";
 import { api, type PlexItemDetail } from "../lib/api";
-import { ACTIVE_ACCOUNT_KEY } from "../lib/activeAccount";
+import { claimLocalData } from "../lib/storage";

 type Media = HTMLMediaElement & { _t?: number };

@@ -91,7 +91,7 @@ beforeAll(() => {
 });

 beforeEach(() => {
-  sessionStorage.setItem(ACTIVE_ACCOUNT_KEY, "7");
+  claimLocalData(7);
  hlsInstances.length = 0;
 });

@@ -21,7 +21,7 @@ import PlexPlayer from "./PlexPlayer";
 import { api, HttpError, type PlexItemDetail } from "../lib/api";
 import { qk } from "../lib/queryKeys";
 import { notify } from "../lib/notifications";
-import { ACTIVE_ACCOUNT_KEY } from "../lib/activeAccount";
+import { claimLocalData } from "../lib/storage";

 type Media = HTMLMediaElement & { _t?: number };

@@ -75,8 +75,8 @@ beforeAll(() => {

 beforeEach(() => {
  // A signed-in account whose saved subtitle language is English (prefs are per account).
-  sessionStorage.setItem(ACTIVE_ACCOUNT_KEY, "7");
-  localStorage.setItem("siftlode.plexPlayerPrefs.7", JSON.stringify({ subLang: "en" }));
+  claimLocalData(7);
+  localStorage.setItem("siftlode.plexPlayerPrefs", JSON.stringify({ subLang: "en" }));
 });

 afterEach(() => {
@@ -310,7 +310,7 @@ const startsAsked = (id: string) =>

 describe("leaving an item that was played through", () => {
  const prefs = (p: object) =>
-    localStorage.setItem("siftlode.plexPlayerPrefs.7", JSON.stringify({ subLang: "en", ...p }));
+    localStorage.setItem("siftlode.plexPlayerPrefs", JSON.stringify({ subLang: "en", ...p }));

  // The end of the media, in the order the HTML spec mandates: the browser sets `paused`, fires
  // 'pause', and only THEN fires 'ended'. Dispatching a bare 'ended' models a browser that does not
@@ -1,4 +1,4 @@
-import { useEffect, useMemo, useRef, useState, type ReactNode } from "react";
+import { useMemo, useRef, useState, type ReactNode } from "react";
 import { qk } from "../lib/queryKeys";
 import { useTranslation } from "react-i18next";
 import { useQueryClient } from "@tanstack/react-query";
@@ -27,7 +27,8 @@ import { notifyActionError } from "../lib/actionErrors";
 import { navItemAttrs, useListNav } from "../lib/listNav";
 import { toggleWatchState } from "../lib/plexWatch";
 import { formatRuntime } from "../lib/format";
-import { getAccountRaw, LS, setAccountRaw } from "../lib/storage";
+import { LS } from "../lib/storage";
+import { usePref } from "../lib/usePref";
 import { useConfirm } from "./confirmContext";
 import { Button } from "./ui/Button";
 import { LoadingState, StateMessage } from "./ui/QueryState";
@@ -424,12 +425,10 @@ export default function PlexPlaylistView({
  const [renaming, setRenaming] = useState(false);
  const { target: renameTarget, arm: armRenameReturn } = useReturnFocus(renaming);
  const [name, setName] = useState("");
-  const [layout, setLayout] = useState<Layout>(() =>
-    getAccountRaw(LS.plexPlaylistLayout) === "tree" ? "tree" : "accordion",
-  );
-  useEffect(() => {
-    setAccountRaw(LS.plexPlaylistLayout, layout);
-  }, [layout]);
+  const [layout, setLayout] = usePref<Layout>("plexPlaylistLayout", "accordion", {
+    localKey: LS.plexPlaylistLayout,
+    parse: (raw) => (raw === "tree" ? "tree" : "accordion"),
+  });
  // Show groups start collapsed (compact headers) so a long series doesn't force endless scrolling;
  // seasons start expanded once their show is opened. Both are just local view state.
  const [expandedShows, setExpandedShows] = useState<Set<string>>(new Set());
@@ -9,7 +9,7 @@ import { render, cleanup } from "@testing-library/react";
 import { QueryClient, QueryClientProvider } from "@tanstack/react-query";
 import { NavigationProvider } from "./NavigationProvider";
 import { PlexProvider } from "./PlexProvider";
-import { ACTIVE_ACCOUNT_KEY } from "../lib/activeAccount";
+import { claimLocalData } from "../lib/storage";

 function mount() {
  render(
@@ -25,7 +25,7 @@ function mount() {

 beforeEach(() => {
  localStorage.clear();
-  sessionStorage.setItem(ACTIVE_ACCOUNT_KEY, "5");
+  claimLocalData(5);
 });
 afterEach(() => {
  cleanup();
@@ -37,9 +37,9 @@ describe("PlexProvider shared link", () => {
    window.history.replaceState(null, "", "/?page=plex&pscope=movie");
    mount();

-    expect(localStorage.getItem("siftlode.plexLibrary.5")).toBe("movie");
-    expect(localStorage.getItem("siftlode.plexFilters.5")).toBeNull();
-    expect(localStorage.getItem("siftlode.plexSort.5")).toBeNull();
-    expect(localStorage.getItem("siftlode.plexShow.5")).toBeNull();
+    expect(localStorage.getItem("siftlode.plexLibrary")).toBe("movie");
+    expect(localStorage.getItem("siftlode.plexFilters")).toBeNull();
+    expect(localStorage.getItem("siftlode.plexSort")).toBeNull();
+    expect(localStorage.getItem("siftlode.plexShow")).toBeNull();
  });
 });
@@ -28,7 +28,7 @@ import {
 // stable useCallback setters, one-line hooks; composed in main.tsx around <App/>).
 //
 // scope / show / sort / filters / q are per-account persisted (localStorage) and seeded from it at mount
-// (BootGate pins the tab's account first; an account switch reloads). `playlistOpen` is an ephemeral
+// (BootGate claims the local state for the account first; a different account reloads the tab). `playlistOpen` is an ephemeral
 // hand-off (the rail sets it, PlexBrowse consumes it into a history subview then clears it). The Plex
 // search term is deliberately its OWN state, NOT the feed's `filters.q`, so a Plex search never leaks
 // into (and is never greeted by a stale value from) the feed.
@@ -84,7 +84,7 @@ function readPlexFilters(): PlexFilters {
    return EMPTY_PLEX_FILTERS;
  }
 }
-// The base is this account's stored view (BootGate pins the account before this provider mounts), so a
+// The base is this account's stored view (BootGate claims the local state before this provider mounts), so a
 // PARTIAL link — one carrying only the scope, say — keeps the stored sort, show and filters for the rest.
 // `present` records which dimensions the link actually carried: only those are persisted (below).
 type PlexSeedPresent = { scope: boolean; sort: boolean; show: boolean; filters: boolean };
@@ -2,29 +2,44 @@
 //
 // The Settings prefs are seeded from the account's server prefs in the `me` cache — BootGate mounts the
 // tree only once it is there. The adopt effect this replaced corrected them AFTER mounting; with it
-// gone, a seed that read only this browser's copy would never show a performance mode or hints setting
-// changed on another device. And the seed is not an edit: it must not be saved straight back.
+// gone, a seed that read only this browser's copy would never show a hints setting changed on another
+// device. And the seed is not an edit: it must not be saved straight back. Performance mode is the
+// exception: it is this device's, so the server's word on it is not taken and a change is not sent.
 import { afterEach, beforeEach, describe, expect, it, vi } from "vitest";
 import { act, cleanup, render } from "@testing-library/react";
 import { QueryClient, QueryClientProvider } from "@tanstack/react-query";
 import { PrefsProvider } from "./PrefsProvider";
 import { usePrefs } from "./prefsContext";
 import { api } from "../lib/api";
-import { ACTIVE_ACCOUNT_KEY } from "../lib/activeAccount";
-import { accountKey, LS } from "../lib/storage";
+import { claimLocalData, LS } from "../lib/storage";
 import { qk } from "../lib/queryKeys";

 const seen: { perf: boolean; hints: boolean }[] = [];
+const controls: ReturnType<typeof usePrefs>[] = [];
 function Probe() {
-  const { perf, hints } = usePrefs();
-  seen.push({ perf, hints });
+  const prefs = usePrefs();
+  controls.push(prefs);
+  seen.push({ perf: prefs.perf, hints: prefs.hints });
  return null;
 }

+function mount(preferences: Record<string, unknown>) {
+  const qc = new QueryClient();
+  qc.setQueryData(qk.me(), { id: 5, preferences });
+  render(
+    <QueryClientProvider client={qc}>
+      <PrefsProvider>
+        <Probe />
+      </PrefsProvider>
+    </QueryClientProvider>,
+  );
+}
+
 beforeEach(() => {
  localStorage.clear();
-  sessionStorage.setItem(ACTIVE_ACCOUNT_KEY, "5");
+  claimLocalData(5);
  seen.length = 0;
+  controls.length = 0;
  vi.useFakeTimers();
 });
 afterEach(() => {
@@ -35,24 +50,38 @@ afterEach(() => {

 describe("PrefsProvider seed", () => {
  it("renders the account's server prefs from the first render, and does not save them back", () => {
-    localStorage.setItem(accountKey(LS.perfMode, 5)!, "0"); // this browser last had it off
-    localStorage.setItem(accountKey(LS.hints, 5)!, "1");
+    localStorage.setItem(LS.hints, "1");
    const save = vi.spyOn(api, "savePrefs").mockResolvedValue(undefined as never);
-    const qc = new QueryClient();
-    qc.setQueryData(qk.me(), { id: 5, preferences: { performanceMode: true, hints: false } });
-    render(
-      <QueryClientProvider client={qc}>
-        <PrefsProvider>
-          <Probe />
-        </PrefsProvider>
-      </QueryClientProvider>,
-    );
+    mount({ hints: false });

-    expect(seen[0]).toEqual({ perf: true, hints: false });
+    expect(seen[0]?.hints).toBe(false);
    // past the auto-save debounce
    act(() => {
      vi.advanceTimersByTime(1000);
    });
    expect(save).not.toHaveBeenCalled();
  });
+
+  it("takes performance mode from this device, never from the account", () => {
+    localStorage.setItem(LS.perfMode, "1"); // this device has it on
+    mount({ performanceMode: false }); // an older build saved the account's value
+    expect(seen[0]?.perf).toBe(true);
+    cleanup();
+
+    localStorage.setItem(LS.perfMode, "0");
+    mount({ performanceMode: true });
+    expect(seen[seen.length - 1]?.perf).toBe(false);
+  });
+
+  it("keeps a performance mode change on this device: nothing is sent to the server", () => {
+    const save = vi.spyOn(api, "savePrefs").mockResolvedValue(undefined as never);
+    mount({});
+    act(() => controls[controls.length - 1]?.setPerf(true));
+    act(() => {
+      vi.advanceTimersByTime(1000);
+    });
+    expect(localStorage.getItem(LS.perfMode)).toBe("1");
+    expect(document.documentElement.dataset.perf).toBe("1");
+    expect(save).not.toHaveBeenCalled();
+  });
 });
@@ -25,13 +25,13 @@ import { PrefsContext } from "./prefsContext";

 const PERF_KEY = LS.perfMode;

-// The four Settings-page preferences. They apply locally for instant preview and auto-save (debounced)
-// to the server — there is no draft or Save button. This provider owns them (lifted out of the App god
+// The Settings page's account preferences. They apply locally for instant preview and auto-save
+// (debounced) to the server — there is no draft or Save button. Performance mode is held here too but
+// is this device's (a weak GPU wants it, a strong one does not), so it stays in this browser only. This provider owns them (lifted out of the App god
 // component) plus the last-saved "baseline" (the server values it was seeded with, then each successful
 // save) used to suppress saving what the server already holds. Consumers (SettingsPanel) read the PrefsController via usePrefs() (prefsContext.ts).
 type EditablePrefs = {
  theme: ThemePrefs;
-  performanceMode: boolean;
  hints: boolean;
  railLabels: boolean;
  notifications: NotifSettings;
@@ -46,10 +46,6 @@ export function PrefsProvider({ children }: { children: ReactNode }) {
    const prefs = qc.getQueryData<Me | null>(qk.me())?.preferences;
    return {
      theme: prefs?.theme ? normalizeTheme(prefs.theme) : loadLocalTheme(),
-      performanceMode:
-        typeof prefs?.performanceMode === "boolean"
-          ? prefs.performanceMode
-          : getAccountRaw(PERF_KEY) === "1",
      hints: typeof prefs?.hints === "boolean" ? prefs.hints : hintsEnabled(),
      // B7: off by default — the icon rail with name tooltips is the house look.
      railLabels: prefs?.railLabels === true,
@@ -59,7 +55,7 @@ export function PrefsProvider({ children }: { children: ReactNode }) {
    };
  });
  const [theme, setThemeState] = useState<ThemePrefs>(seed.theme);
-  const [perf, setPerf] = useState(seed.performanceMode);
+  const [perf, setPerf] = useState(() => getAccountRaw(PERF_KEY) === "1");
  const [hints, setHints] = useState(seed.hints);
  const [railLabels, setRailLabels] = useState(seed.railLabels);
  const [notif, setNotif] = useState<NotifSettings>(seed.notifications);
@@ -92,8 +88,8 @@ export function PrefsProvider({ children }: { children: ReactNode }) {
  // effects above; this is the only persistence path. `baselineRef` keeps the mount seed from being
  // saved straight back (`/me` merges, so a full payload is fine).
  const editablePrefs = useMemo<EditablePrefs>(
-    () => ({ theme, performanceMode: perf, hints, railLabels, notifications: notif }),
-    [theme, perf, hints, railLabels, notif],
+    () => ({ theme, hints, railLabels, notifications: notif }),
+    [theme, hints, railLabels, notif],
  );
  useEffect(() => {
    window.clearTimeout(saveTimer.current);
@@ -45,8 +45,7 @@ vi.mock("./confirmContext", () => ({ useConfirm: () => async () => true }));
 import { SavedViewsProvider } from "./SavedViewsProvider";
 import { useSavedViews } from "./savedViewsContext";
 import { api, type SavedView } from "../lib/api";
-import { LS, accountKey } from "../lib/storage";
-import { ACTIVE_ACCOUNT_KEY } from "../lib/activeAccount";
+import { claimLocalData, LS } from "../lib/storage";
 import { qk } from "../lib/queryKeys";

 const ID = 7;
@@ -95,7 +94,7 @@ async function mountResolved(views: SavedView[]) {

 beforeEach(() => {
  localStorage.clear();
-  sessionStorage.setItem(ACTIVE_ACCOUNT_KEY, String(ID));
+  claimLocalData(ID);
  // Reset the URL: feedCanSeedDefault reads window.location.search, and a stale ?sort= from a prior
  // case would suppress seeding in the others.
  window.history.replaceState({}, "", "/");
@@ -118,7 +117,7 @@ describe("SavedViewsProvider — default-view seeding on first resolve", () => {
  });

  it("feed: does NOT seed when the account already has stored filters", async () => {
-    localStorage.setItem(accountKey(LS.filters, ID)!, JSON.stringify({ scope: "my" }));
+    localStorage.setItem(LS.filters, JSON.stringify({ scope: "my" }));
    await mountResolved([view({ id: 2, is_default: true, filters: { scope: "all" } })]);
    expect(h.setFeedFilters).not.toHaveBeenCalled();
  });
@@ -177,7 +176,7 @@ describe("SavedViewsProvider — default-view seeding on first resolve", () => {

  it("plex: does NOT seed when plex filters are already stored", async () => {
    h.page = "plex";
-    localStorage.setItem(accountKey(LS.plexFilters, ID)!, JSON.stringify({ genres: ["Comedy"] }));
+    localStorage.setItem(LS.plexFilters, JSON.stringify({ genres: ["Comedy"] }));
    await mountResolved([view({ id: 3, is_default: true, filters: { genres: ["Action"] } })]);
    expect(h.setPlexFilters).not.toHaveBeenCalled();
  });
@@ -5,7 +5,6 @@ import { qk } from "../lib/queryKeys";
 import {
  api,
  coercePlexFilters,
-  getActiveAccount,
  plexFiltersSignature,
  type FeedFilters,
  type PlexFilters,
@@ -14,7 +13,7 @@ import {
 import { filtersToParams, hasFilterParams } from "../lib/urlState";
 import { pickDefaultToSeed, preferSticky } from "../lib/savedViews";
 import { notifyActionError } from "../lib/actionErrors";
-import { accountKey, LS } from "../lib/storage";
+import { LS, ownKey } from "../lib/storage";
 import { useConfirm } from "./confirmContext";
 import { useNavigation } from "./navigationContext";
 import { useFeedFilters, useFeedFiltersActions } from "./feedFiltersContext";
@@ -55,7 +54,7 @@ const plexKeyOf = (f: Record<string, unknown>) => plexFiltersSignature(coercePle
 // view / manual tweaks (not the default) should win. Account unknown or localStorage unavailable →
 // report "stored" so a default never seeds blind over an indeterminate state.
 function storedFiltersPresent(lsKey: string): boolean {
-  const key = accountKey(lsKey, getActiveAccount());
+  const key = ownKey(lsKey);
  if (!key) return true;
  try {
    return localStorage.getItem(key) != null;
@@ -147,8 +146,8 @@ export function SavedViewsProvider({ children }: { children: ReactNode }) {
  // login effect (on a fresh browser the feed could seed DEFAULT_FILTERS and never re-apply until a
  // reload) — and it gives Plex the default it previously never applied at all. Per-source guard so a
  // 60s background refetch never re-seeds over the filters you're editing. canSeedDefault() reads the
-  // per-account stored-filters key, which is always readable here: BootGate pins the tab's account
-  // before this provider mounts.
+  // account's stored-filters key, which is always readable here: BootGate hands the browser's local
+  // state to the account (claimLocalData) before this provider mounts.
  const seededRef = useRef<Set<ViewSource>>(new Set());
  // `cfg` re-memoizes on every filter edit, so it is read through an effect event: the seed must
  // fire on the views query's first resolve for a source and never again, which `seededRef`
@@ -1,5 +1,5 @@
 import { useState } from "react";
-import { withActiveAccount } from "../lib/activeAccount";
+import { connectGoogle } from "../lib/googleConnect";
 import { qk } from "../lib/queryKeys";
 import { formatDate, formatDateTime, formatUnit } from "../lib/format";
 import { useTranslation } from "react-i18next";
@@ -24,7 +24,7 @@ import { SCHEMES, type Scheme, type ThemePrefs } from "../lib/theme";
 import { api, HttpError, type Me } from "../lib/api";
 import { displayName, roleKey } from "../lib/api/account";
 import { notifyActionError } from "../lib/actionErrors";
-import { LS, useAccountPersistedState, useTabPersistedState } from "../lib/storage";
+import { LS, useTabPersistedState } from "../lib/storage";
 import Avatar from "./ui/Avatar";
 import { notify, type NotifSettings } from "../lib/notifications";
 import Tooltip from "./ui/Tooltip";
@@ -35,7 +35,8 @@ import { type SaveState } from "./ui/DraftSaveBar";
 import { useConfirm } from "./confirmContext";
 import { StateMessage } from "./ui/QueryState";
 import { useWizard } from "./wizardContext";
-import { START_ON_HOME } from "./navigationInit";
+import { START_ON_HOME_OPTIONS, START_ON_HOME_PREF } from "./navigationInit";
+import { usePref } from "../lib/usePref";
 import { usePrefs } from "./prefsContext";
 import Tabs from "./ui/Tabs";
 import { useTabPanel } from "./ui/tabPanel";
@@ -164,10 +165,8 @@ function Appearance({ prefs }: { prefs: PrefsController }) {
  const { t } = useTranslation();
  // Controlled by PrefsProvider: each change applies locally at once and auto-saves (debounced).
  const { theme, setTheme, perf, setPerf, hints, setHints, railLabels, setRailLabels } = prefs;
-  // The landing page. Per-account localStorage, not a server preference like the rest of this
-  // tab: `loadInitialPage` reads it synchronously in NavigationProvider's initializer. The two stored
-  // values come from navigationInit, which is what reads them back at boot.
-  const [startOnHome, setStartOnHome] = useAccountPersistedState(LS.startOnHome, START_ON_HOME.on);
+  // The landing page: read at boot (NavigationProvider), so a change applies from the next start.
+  const [startOnHome, setStartOnHome] = usePref(START_ON_HOME_PREF, true, START_ON_HOME_OPTIONS);

  return (
    <>
@@ -242,8 +241,8 @@ function Appearance({ prefs }: { prefs: PrefsController }) {
        <SwitchRow
          label={t("settings.appearance.startOnHome")}
          hint={t("settings.appearance.startOnHomeHint")}
-          checked={startOnHome !== START_ON_HOME.off}
-          onChange={(v) => setStartOnHome(v ? START_ON_HOME.on : START_ON_HOME.off)}
+          checked={startOnHome}
+          onChange={setStartOnHome}
        />
      </Section>

@@ -359,7 +358,7 @@ function AccessRow({

 // Sign-in methods: link a Google account to a password account (or vice-versa set a password),
 // so either method can reach the same account. Google connect is a full-page OAuth round-trip
-// (auth.py attaches the identity to the current session via /auth/link); the password form posts
+// (auth.py attaches the identity to the signed-in account via /auth/google/connect); the password form posts
 // directly with inline errors.
 export function SignInMethods({ me }: { me: Me }) {
  const { t } = useTranslation();
@@ -451,13 +450,7 @@ export function SignInMethods({ me }: { me: Me }) {
          {me.has_google ? (
            <Badge tone="accent">{t("settings.account.googleLink.connected")}</Badge>
          ) : (
-            <Button
-              variant="glass"
-              size="sm"
-              onClick={() => {
-                window.location.href = withActiveAccount("/auth/link");
-              }}
-            >
+            <Button variant="glass" size="sm" onClick={() => connectGoogle("identity")}>
              {t("settings.account.googleLink.connect")}
            </Button>
          )}
@@ -877,18 +870,14 @@ function Account({ me, onOpenWizard }: { me: Me; onOpenWizard: () => void }) {
              granted={me.can_read}
              grantedHint={t("settings.account.readGrantedHint")}
              enableHint={t("settings.account.readEnableHint")}
-              onEnable={() => {
-                window.location.href = withActiveAccount("/auth/upgrade?access=read");
-              }}
+              onEnable={() => connectGoogle("read")}
            />
            <AccessRow
              title={t("settings.account.writeTitle")}
              granted={me.can_write}
              grantedHint={t("settings.account.writeGrantedHint")}
              enableHint={t("settings.account.writeEnableHint")}
-              onEnable={() => {
-                window.location.href = withActiveAccount("/auth/upgrade?access=write");
-              }}
+              onEnable={() => connectGoogle("write")}
            />
          </div>
          <Button variant="link" className="mt-2 text-sm" onClick={onOpenWizard}>
@@ -0,0 +1,71 @@
+// @vitest-environment jsdom
+//
+// The "updated" banner shows once per release for the ACCOUNT: the version it was last dismissed for
+// is an account preference, so a banner dismissed on one device does not come back on the next. A
+// version this browser kept before the server held it (a bare string) still counts.
+import { afterEach, beforeEach, describe, expect, it, vi } from "vitest";
+import { act, cleanup, render, screen } from "@testing-library/react";
+import { QueryClient, QueryClientProvider } from "@tanstack/react-query";
+
+vi.mock("../lib/version", async (importOriginal) => ({
+  ...(await importOriginal<typeof import("../lib/version")>()),
+  FRONTEND_VERSION: "1.2.3",
+}));
+
+import VersionBanner from "./VersionBanner";
+import { api } from "../lib/api";
+import { claimLocalData, LS } from "../lib/storage";
+import { qk } from "../lib/queryKeys";
+
+let save: ReturnType<typeof vi.spyOn>;
+function mount(preferences: Record<string, unknown>) {
+  const qc = new QueryClient();
+  qc.setQueryData(qk.me(), { id: 5, preferences });
+  render(
+    <QueryClientProvider client={qc}>
+      <VersionBanner onOpen={() => {}} />
+    </QueryClientProvider>,
+  );
+}
+const banner = () => screen.queryByText("header.banner.updated", { exact: false });
+
+vi.mock("react-i18next", async (importOriginal) => ({
+  ...(await importOriginal<typeof import("react-i18next")>()),
+  useTranslation: () => ({ t: (key: string) => key, i18n: { language: "en" } }),
+}));
+
+beforeEach(() => {
+  localStorage.clear();
+  claimLocalData(5);
+  save = vi.spyOn(api, "savePrefs").mockResolvedValue(undefined as never);
+});
+afterEach(() => {
+  cleanup();
+  vi.restoreAllMocks();
+});
+
+describe("the release banner", () => {
+  it("shows for a release the account has not seen", () => {
+    mount({ seenVersion: "1.2.2" });
+    expect(banner()).not.toBeNull();
+  });
+
+  it("stays hidden where the account dismissed this release, on another device", () => {
+    mount({ seenVersion: "1.2.3" });
+    expect(banner()).toBeNull();
+  });
+
+  it("takes the version this browser kept, and saves it to the account", () => {
+    localStorage.setItem(LS.seenVersion, "1.2.3");
+    mount({});
+    expect(banner()).toBeNull();
+    expect(save).toHaveBeenCalledWith({ seenVersion: "1.2.3" });
+  });
+
+  it("saves a dismissal to the account", () => {
+    mount({});
+    act(() => screen.getByTitle("header.banner.dismiss").click());
+    expect(banner()).toBeNull();
+    expect(save).toHaveBeenCalledWith({ seenVersion: "1.2.3" });
+  });
+});
@@ -1,23 +1,19 @@
-import { useState } from "react";
 import { useTranslation } from "react-i18next";
 import { Sparkles } from "lucide-react";
-import { FRONTEND_VERSION, SEEN_VERSION_KEY } from "../lib/version";
+import { FRONTEND_VERSION, SEEN_VERSION_OPTIONS, SEEN_VERSION_PREF } from "../lib/version";
+import { usePref } from "../lib/usePref";
 import Banner from "./ui/Banner";

 // Eye-catching, dismissible bar shown once after the running build's version changes
 // (compares the baked frontend version to the last one the user acknowledged).
 export default function VersionBanner({ onOpen }: { onOpen: () => void }) {
  const { t } = useTranslation();
-  const [dismissed, setDismissed] = useState(false);
-  const seen = localStorage.getItem(SEEN_VERSION_KEY);
+  const [seen, setSeen] = usePref(SEEN_VERSION_PREF, null, SEEN_VERSION_OPTIONS);
  // Don't nag in un-stamped dev builds, or once this version has been seen.
-  const show = FRONTEND_VERSION !== "dev" && seen !== FRONTEND_VERSION && !dismissed;
+  const show = FRONTEND_VERSION !== "dev" && seen !== FRONTEND_VERSION;
  if (!show) return null;

-  function markSeen() {
-    localStorage.setItem(SEEN_VERSION_KEY, FRONTEND_VERSION);
-    setDismissed(true);
-  }
+  const markSeen = () => setSeen(FRONTEND_VERSION);

  return (
    <Banner
@@ -0,0 +1,81 @@
+// @vitest-environment jsdom
+//
+// Whether the onboarding wizard opens by itself is the account's preference: dismissed on one device,
+// it stays shut on the others. A dismissal this browser kept before the server held it (the string
+// "1") still counts, and reaches the server.
+import { afterEach, beforeEach, describe, expect, it, vi } from "vitest";
+import { act, cleanup, render, screen } from "@testing-library/react";
+import { QueryClient, QueryClientProvider } from "@tanstack/react-query";
+
+vi.mock("./OnboardingWizard", () => ({
+  default: ({ onClose }: { onClose: (dismiss: boolean) => void }) => (
+    <button onClick={() => onClose(true)}>skip the wizard</button>
+  ),
+}));
+
+import { WizardProvider } from "./WizardProvider";
+import { api } from "../lib/api";
+import { claimLocalData, LS } from "../lib/storage";
+import { qk } from "../lib/queryKeys";
+
+let save: ReturnType<typeof vi.spyOn>;
+function mount(preferences: Record<string, unknown>) {
+  const qc = new QueryClient();
+  qc.setQueryData(qk.me(), {
+    id: 5,
+    can_read: false,
+    can_write: false,
+    google_enabled: true,
+    preferences,
+  });
+  render(
+    <QueryClientProvider client={qc}>
+      <WizardProvider>{null}</WizardProvider>
+    </QueryClientProvider>,
+  );
+  return qc;
+}
+const wizard = () => screen.queryByRole("button", { name: "skip the wizard" });
+
+beforeEach(() => {
+  localStorage.clear();
+  sessionStorage.clear();
+  claimLocalData(5);
+  save = vi.spyOn(api, "savePrefs").mockResolvedValue(undefined as never);
+});
+afterEach(() => {
+  cleanup();
+  vi.restoreAllMocks();
+});
+
+describe("the onboarding wizard's auto-open", () => {
+  it("opens for an account that has not connected YouTube nor dismissed it", async () => {
+    mount({});
+    expect(await screen.findByRole("button", { name: "skip the wizard" })).toBeTruthy();
+  });
+
+  it("stays shut where the account dismissed it, on another device", async () => {
+    mount({ onboardingDismissed: true });
+    await act(async () => {});
+    expect(wizard()).toBeNull();
+  });
+
+  it("takes this browser's earlier dismissal and saves it to the account", async () => {
+    localStorage.setItem(LS.onboardingDismissed, "1");
+    mount({});
+    await act(async () => {});
+    expect(wizard()).toBeNull();
+    expect(save).toHaveBeenCalledWith({ onboardingDismissed: true });
+  });
+
+  it("saves a dismissal to the account, and stays shut", async () => {
+    const qc = mount({});
+    const skip = await screen.findByRole("button", { name: "skip the wizard" });
+    await act(async () => skip.click());
+    expect(save).toHaveBeenCalledWith({ onboardingDismissed: true });
+    expect(
+      qc.getQueryData<{ preferences: Record<string, unknown> }>(qk.me())?.preferences,
+    ).toMatchObject({ onboardingDismissed: true });
+    expect(wizard()).toBeNull();
+  });
+});
@@ -2,7 +2,12 @@ import { lazy, Suspense, useCallback, useEffect, useMemo, useState, type ReactNo
 import { qk } from "../lib/queryKeys";
 import { useQuery } from "@tanstack/react-query";
 import { api } from "../lib/api";
-import { shouldAutoOpenOnboarding } from "../lib/onboarding";
+import {
+  ONBOARDING_DISMISSED_OPTIONS,
+  ONBOARDING_DISMISSED_PREF,
+  shouldAutoOpenOnboarding,
+} from "../lib/onboarding";
+import { prefValue, usePref } from "../lib/usePref";
 import { WizardActionsContext, type WizardActions } from "./wizardContext";

 const OnboardingWizard = lazy(() => import("./OnboardingWizard"));
@@ -23,8 +28,20 @@ export function WizardProvider({ children }: { children: ReactNode }) {
  // Passive observer of the account BootGate fetches (enabled:false → reads cache, re-renders on update).
  const { data: me } = useQuery({ queryKey: qk.me(), queryFn: api.me, enabled: false });

+  // The setter only: the check below reads the value from `me`, which the setter keeps current.
+  const [, setDismissed] = usePref(ONBOARDING_DISMISSED_PREF, false, ONBOARDING_DISMISSED_OPTIONS);
+  const dismissed = prefValue(me, ONBOARDING_DISMISSED_PREF, false, ONBOARDING_DISMISSED_OPTIONS);
+
  const openWizard = useCallback(() => setOpen(true), []);
  const closeWizard = useCallback(() => setOpen(false), []);
+  // Leaving without the essential read step suppresses the auto-popup from then on.
+  const finishWizard = useCallback(
+    (dismiss: boolean) => {
+      if (dismiss) setDismissed(true);
+      setOpen(false);
+    },
+    [setDismissed],
+  );

  // First-login onboarding: prompt the user to connect YouTube (and resume the flow after each
  // consent redirect). Re-checked when the account or its grants change.
@@ -39,8 +56,8 @@ export function WizardProvider({ children }: { children: ReactNode }) {
    if (meId == null || canRead == null) return;
    const account = { can_read: canRead, google_enabled: googleEnabled };
    // eslint-disable-next-line react-hooks/set-state-in-effect -- opening is a one-way decision taken from SERVER data as it arrives, not a value derivable in render — deriving `open` from the grants would re-open a wizard the user had just closed, which is exactly what the narrowed dep list above exists to prevent.
-    if (shouldAutoOpenOnboarding(account)) setOpen(true);
-  }, [meId, canRead, canWrite, googleEnabled]);
+    if (shouldAutoOpenOnboarding(account, dismissed)) setOpen(true);
+  }, [meId, canRead, canWrite, googleEnabled, dismissed]);

  const actions = useMemo<WizardActions>(
    () => ({ openWizard, closeWizard }),
@@ -52,7 +69,7 @@ export function WizardProvider({ children }: { children: ReactNode }) {
      {children}
      {/* The wizard is a full-screen overlay. Lazy — its chunk loads only when first opened. */}
      <Suspense fallback={null}>
-        {open && me && <OnboardingWizard me={me} onClose={closeWizard} />}
+        {open && me && <OnboardingWizard me={me} onClose={finishWizard} />}
      </Suspense>
    </WizardActionsContext.Provider>
  );
@@ -8,8 +8,7 @@ import { act, cleanup, render, renderHook, screen, waitFor } from "@testing-libr
 import { QueryClient, QueryClientProvider } from "@tanstack/react-query";
 import { createElement, type ReactNode } from "react";
 import { api, type FeedFilters, type ManagedChannel } from "../lib/api";
-import { ACTIVE_ACCOUNT_KEY } from "../lib/activeAccount";
-import { accountKey, LS } from "../lib/storage";
+import { claimLocalData, LS } from "../lib/storage";
 import { clearedFilters, useActiveFilters } from "../lib/useActiveFilters";
 import ActiveFilterChips from "./ActiveFilterChips";
 import ChannelPicker from "./ChannelPicker";
@@ -28,7 +27,7 @@ const filters: FeedFilters = {
  ...clearedFilters({ q: "" } as FeedFilters),
  channelIds: [KNOWN, NEW],
 };
-const namesKey = () => accountKey(LS.channelNames, 5)!;
+const namesKey = () => LS.channelNames;

 let resolveChannels: (c: ManagedChannel[]) => void;
 let qc: QueryClient;
@@ -37,7 +36,7 @@ const wrapper = ({ children }: { children: ReactNode }) =>

 beforeEach(() => {
  localStorage.clear();
-  sessionStorage.setItem(ACTIVE_ACCOUNT_KEY, "5");
+  claimLocalData(5);
  localStorage.setItem(namesKey(), JSON.stringify({ [KNOWN]: "Remembered name" }));
  qc = new QueryClient({ defaultOptions: { queries: { retry: false } } });
  vi.spyOn(api, "tags").mockResolvedValue([] as never);
@@ -61,10 +61,7 @@ export function JobThumb({ job, box = THUMB_BOX }: { job: DownloadJob; box?: str
  // rule on the source domain can't break it. The source's remote thumbnail_url is only a fallback for
  // the rare poster-less asset (a thumbnail-less source whose ffmpeg frame-cut also failed).
  const remote = job.asset?.thumbnail_url ?? null;
-  // `job.poster_url` only SIGNALS a poster exists; the actual <img> src must carry ?account= (see
-  // api.posterImageUrl). A plain <img> can't send the X-Siftlode-Account header, so without it the
-  // poster resolves to the session-default account and 404s for a per-tab account's download — which,
-  // now that the poster is the primary image, reads as a blank Library thumbnail.
+  // `job.poster_url` only SIGNALS a poster exists; the <img> src is api.posterImageUrl.
  const poster = job.poster_url ? api.posterImageUrl(job.id) : null;
  // A generic uploaded file (media_kind='file') has no video frame to poster — it's a document/
  // archive, not playable media. An image is its own picture (its sandboxed preview); the rest show
@@ -1,5 +1,5 @@
 import { coerceTagModes, coerceUntagged, DEFAULT_TAG_MODES, type FeedFilters } from "../lib/api";
-import { accountKey, LS, readJSON } from "../lib/storage";
+import { LS, ownKey, readJSON } from "../lib/storage";

 // The feed filters' defaults and the per-account localStorage seed, split out of FeedFiltersProvider
 // (like navigationInit / playlistsInit) so that file exports only components (Fast Refresh).
@@ -23,16 +23,16 @@ export const SHOW_IDS = ["unwatched", "in_progress", "all", "watched", "hidden"]

 export const FILTERS_KEY = LS.filters;

-// This account's own persisted filters (per-account keys — never another account's). No account
+// This account's own persisted filters (see ownKey). No account
 // (signed out) → defaults.
-export function loadAccountFilters(id: number | null): FeedFilters {
+export function loadAccountFilters(): FeedFilters {
  // Your last-applied filters win, so a reload keeps whatever view you're on (a saved view you
  // picked, or manual tweaks) — setFilters persists them under LS.filters on every change. A starred
  // default view seeds a fresh account that has never stored filters, but that is applied ONCE by
  // SavedViewsProvider on its views query's first resolve (which then persists here like any edit) —
  // not read from a mirror on this synchronous path, which used to race that resolve. So here we only
  // ever read the account's own stored filters, else fall back to DEFAULT_FILTERS.
-  const fKey = accountKey(FILTERS_KEY, id);
+  const fKey = ownKey(FILTERS_KEY);
  let hasStored = false;
  try {
    hasStored = !!fKey && localStorage.getItem(fKey) != null;
@@ -6,7 +6,8 @@
 //   • channelViewFrom — the shared _chan/?chan resolver behind BOTH the channelView seed and the
 //     onPop handler: a popstate landing on a URL-only bookmark entry must restore the
 //     channel from ?chan, not close it.
-// Only getAccountRaw is stubbed (the account's stored landing page); everything else is real.
+// Only getAccountRaw is stubbed (the page the account left open); everything else is real. Whether
+// to start on Home is the account's preference, handed in.
 import { describe, it, expect, beforeEach, vi } from "vitest";

 const { stored } = vi.hoisted(() => ({ stored: {} as Record<string, string | null> }));
@@ -15,87 +16,90 @@ vi.mock("../lib/storage", async (importOriginal) => {
  return { ...actual, getAccountRaw: (k: string) => stored[k] ?? null };
 });

-import { loadInitialPage, channelViewFrom, START_ON_HOME } from "./navigationInit";
+import { loadInitialPage, channelViewFrom } from "./navigationInit";
 import { LS } from "../lib/storage";

 const setUrl = (search: string) => window.history.replaceState(null, "", search || "/");

+let startOnHome = true;
+
 beforeEach(() => {
  setUrl("/");
+  startOnHome = true;
  for (const k of Object.keys(stored)) delete stored[k];
 });

 describe("loadInitialPage precedence", () => {
  it("?page wins and is validated", () => {
    setUrl("?page=stats");
-    expect(loadInitialPage()).toBe("stats");
+    expect(loadInitialPage(startOnHome)).toBe("stats");
  });
  it("an invalid ?page falls back to feed", () => {
    setUrl("?page=bogus");
-    expect(loadInitialPage()).toBe("feed");
+    expect(loadInitialPage(startOnHome)).toBe("feed");
  });
  it("?v implies the plex module", () => {
    setUrl("?v=show:1");
-    expect(loadInitialPage()).toBe("plex");
+    expect(loadInitialPage(startOnHome)).toBe("plex");
  });
  it("?pl implies playlists", () => {
    setUrl("?pl=3");
-    expect(loadInitialPage()).toBe("playlists");
+    expect(loadInitialPage(startOnHome)).toBe("playlists");
  });
  it.each(["?folder=4", "?item=12", "?folder=all&item=12"])(
    "%s implies the Files page (a folder, or a file to arrive at)",
    (search) => {
      setUrl(search);
-      expect(loadInitialPage()).toBe("files");
+      expect(loadInitialPage(startOnHome)).toBe("files");
    },
  );
  it("a filter param implies the feed", () => {
    setUrl("?tags=1,2");
-    expect(loadInitialPage()).toBe("feed");
+    expect(loadInitialPage(startOnHome)).toBe("feed");
  });
  it("?chan (the channel overlay) implies the feed", () => {
    setUrl("?chan=UC1");
-    expect(loadInitialPage()).toBe("feed");
+    expect(loadInitialPage(startOnHome)).toBe("feed");
  });
  it("?page beats a lower-precedence param", () => {
    setUrl("?page=stats&v=show:1");
-    expect(loadInitialPage()).toBe("stats");
+    expect(loadInitialPage(startOnHome)).toBe("stats");
  });
  it("?v beats ?pl and filters", () => {
    setUrl("?v=show:1&pl=3&tags=1");
-    expect(loadInitialPage()).toBe("plex");
+    expect(loadInitialPage(startOnHome)).toBe("plex");
  });
  // A bare URL lands on Home, OVER the remembered module. Landing on
  // whatever you happened to close the app on is precisely the habit the anti-paralysis surface
  // exists to replace, so the remembered page only wins once the account opts out.
  it("a bare URL lands on Home even when another module is remembered", () => {
    stored[LS.page] = "plex";
-    expect(loadInitialPage()).toBe("home");
+    expect(loadInitialPage(startOnHome)).toBe("home");
  });
  it("opting out restores the account's stored landing page", () => {
-    stored[LS.startOnHome] = START_ON_HOME.off;
+    startOnHome = false;
    stored[LS.page] = "plex";
-    expect(loadInitialPage()).toBe("plex");
+    expect(loadInitialPage(startOnHome)).toBe("plex");
  });
  // The Downloads page is gone; what it held is in Files. A bookmark and a remembered page both land
  // there instead of on the feed.
  it("a retired page lands where its content went — linked or remembered", () => {
    setUrl("?page=downloads");
-    expect(loadInitialPage()).toBe("files");
+    expect(loadInitialPage(startOnHome)).toBe("files");
    setUrl("/");
-    stored[LS.startOnHome] = START_ON_HOME.off;
+    startOnHome = false;
    stored[LS.page] = "downloads";
-    expect(loadInitialPage()).toBe("files");
+    expect(loadInitialPage(startOnHome)).toBe("files");
  });
  it("opted out with no / invalid stored page still defaults to feed", () => {
-    stored[LS.startOnHome] = START_ON_HOME.off;
-    expect(loadInitialPage()).toBe("feed");
+    startOnHome = false;
+    expect(loadInitialPage(startOnHome)).toBe("feed");
    stored[LS.page] = "nope";
-    expect(loadInitialPage()).toBe("feed");
+    expect(loadInitialPage(startOnHome)).toBe("feed");
  });
  it("a deep link still beats Home", () => {
    setUrl("?pl=3");
-    expect(loadInitialPage()).toBe("playlists");
+    expect(loadInitialPage(startOnHome)).toBe("playlists");
  });
 });

@@ -4,24 +4,25 @@
 // channelView seed and the onPop handler.
 import { hasFilterParams, readPage, resolvePage, type Page } from "../lib/urlState";
 import { getAccountRaw, LS } from "../lib/storage";
+import type { UsePrefOptions } from "../lib/usePref";

 export const PAGE_KEY = LS.page;
-// Module-local: the Settings toggle writes `LS.startOnHome` directly, so exporting this alias would
-// only be a second name for the same key.
-const START_HOME_KEY = LS.startOnHome;
-
-/** The two stored values of the landing preference (raw strings, via useAccountPersistedState).
- *  Settings → Appearance → Startup writes them and `loadInitialPage` reads them, so they are defined
- *  ONCE: the same magic string spelled out in a writer and a reader is precisely the drift that has
- *  bitten this codebase before (`cancelled` vs `canceled`). */
-export const START_ON_HOME = { on: "1", off: "0" } as const;
+/** The landing preference — start on Home, or on the module left open — an account preference
+ *  (usePref). Settings → Appearance → Startup writes it and NavigationProvider reads it for
+ *  `loadInitialPage`, so its key and its reading are defined ONCE. A copy this browser kept before
+ *  the server held it is the string "0" (off) or "1". */
+export const START_ON_HOME_PREF = "startOnHome";
+export const START_ON_HOME_OPTIONS: UsePrefOptions<boolean> = {
+  localKey: LS.startOnHome,
+  parse: (raw) => raw !== false && raw !== "0" && raw !== 0,
+};

 export interface ChannelView {
  id: string;
  name?: string;
 }

-export function loadInitialPage(): Page {
+export function loadInitialPage(startOnHome: boolean): Page {
  const params = new URLSearchParams(window.location.search);
  if (params.has("page")) return readPage();
  // A view-specific param implies its page, so a bookmarked deep link lands on the right module even
@@ -35,7 +36,7 @@ export function loadInitialPage(): Page {
  // on whatever you happened to close the app on is the behaviour the anti-paralysis epic exists to
  // replace. The account can opt out (Settings → Appearance → Startup) and get the remembered
  // module back.
-  if (getAccountRaw(START_HOME_KEY) !== START_ON_HOME.off) return "home";
+  if (startOnHome) return "home";
  return resolvePage(getAccountRaw(PAGE_KEY)) ?? "feed";
 }

@@ -32,7 +32,7 @@ const PLAIN: Record<string, [number, string]> = {
    "the hero's picture and a shelf tile's picture (each named by aria-label; its title sits outside)",
  ],
  "LanguageSwitcher.tsx": [1, "an option row"],
-  "NavSidebar.tsx": [4, "the account trigger and its menu rows"],
+  "NavSidebar.tsx": [3, "the account trigger and its menu rows"],
  "PaletteLauncher.tsx": [2, "launcher controls"],
  "PlaylistsRail.tsx": [1, "the load-error retry"],
  "PlexCollectionEditor.tsx": [1, "a checkbox row in a picker"],
--- a/Show More
+++ b/Show More
Author	SHA1	Message	Date
peter	fdf8076013	release: 0.109.0 (improvement/e43-identity-diet)	2026-10-21 03:05:48 +02:00
peter	6287fc9cf8	chore(release): 0.109.0 — one account per browser, settings that follow the account, a tab that names its account	2026-10-21 02:55:46 +02:00
peter	3c6c23507b	fix(frontend): a page opened at a //path boots, and the account-switch copy reads right in Hungarian Every address-bar writer glued `pathname + query`, so a path of `//evil.com/x` named another host: history refused it (SecurityError) and the first nav mirror threw the app into its crash screen. One builder, `hrefHere`, now makes the address from the URL itself and stays on this origin.	2026-10-21 02:51:48 +02:00
peter	ef99f7b5c7	fix(auth): a Google connect names its tab's account too, and a refused tab says why - /auth/google/connect is a navigation and carries no headers: the tab names the account it was built for in `expect`, checked by the same comparison as the request header (built_for_another_account); a mismatch lands on /?account=changed - the link marker names the OAuth state of its own flow: an older flow coming back no longer spends the marker of the connect still open (which then returned as a plain sign-in) - a 401 account_changed reloads the tab with a one-time flag, and the reloaded tab says it now shows the account signed in - a refused switch to a suspended or not-yet-approved account says so in the signed-in app's words, naming the address a request was recorded for - the prefs merge locks FOR NO KEY UPDATE, so a transaction that only added a row referring to the account does not hold the save up; a test pins the lock - giving up a lost page closes a channel page open over it - e2e README names the chromium-only specs	2026-10-21 02:05:33 +02:00
peter	94418a087e	fix(auth): a tab acts only as the account it was built for, and the review's other findings A tab names the account it was claimed for on every request, beacon and upload (X-Siftlode-Expect-Account); current_user and /auth/logout refuse a mismatch with 401 account_changed and keep the session, so a tab left showing the account signed in before reloads instead of writing to the new one. /auth/login clears a connect marker left by an abandoned connect, and a link completes only on a session that still holds. The prefs merge reads the row under a lock. A tab's sessionStorage records its own owner. A page the account loses is given up, address bar and remembered page included. The panel layouts are preferences; the storage contract also refuses a key handed to the Storage API or to usePref outside its kind. A refused sign-in as someone else and a failed sign-out say so. The chromium-only e2e specs are in the gate register, the member walk waits for its requests, and devices.spec checks the save itself.	2026-10-21 01:05:12 +02:00
peter	28b77fed1e	docs: storage kinds in the canonical patterns, a password beside Google sign-in, performance mode is this device's CLAUDE.md's per-account state row becomes the browser storage rule: a key in storage.ts with its kind, a preference through usePref. The self-hosting guide says to set a password even when signing in with Google, so a broken Google client does not lock the account out. Performance mode's hint says it applies on this device only.	2026-10-20 23:34:43 +02:00
peter	50764a916d	fix(deps): certifi 2026.10.19 (GHSA-6q48-6c7w-9gw2)	2026-10-20 23:14:37 +02:00
peter	f21cdfa156	refactor(frontend): every stored key declares its kind, and preferences follow the account storage.ts's registry gives each key a kind (pref, device, view, cache, public); a key without one does not compile. A contract test refuses a key spelled out elsewhere, an account preference read or written straight from storage instead of through usePref, and an e2e seed of a key nobody reads. The keys the next account's wipe keeps are the public ones. The seven preferences kept only in the browser move onto usePref: start on Home, the onboarding dismissal, the release banner's last version, the channel manager's layout, the Plex playlist layout, the playlist rail's sort and channel discovery's sort. usePref reads an older bare-string copy, and a value only this browser held is saved to the server once, so the account's other devices get it. Performance mode is the device's: it is neither read from the account nor saved to it.	2026-10-20 23:04:48 +02:00
peter	d6dada7fe5	refactor(frontend): one account per browser, its local state under plain keys The per-tab account pin, the X-Siftlode-Account header and ?account= go: the session cookie alone says who is asking. localStorage keys lose their account suffix; the browser records whose state it holds (siftlode.owner), BootGate hands it to the account the server answered as, and a different account starts from a clean slate (the public /watch keys stay). The first boot moves the account's old per-account copies onto the plain keys and drops other accounts' copies. Until the claim, and after another tab claimed the browser for someone else, the account helpers read and write nothing. The account panel lists no other accounts: "Add another account" becomes "Sign in as someone else" (Google's account chooser; the pick replaces this account, a cancel keeps it), shown where Google sign-in is configured.	2026-10-20 15:29:41 +02:00
peter	6645caa157	refactor(auth): one Google connect route for a sign-in and for YouTube access /auth/link and /auth/upgrade become /auth/google/connect?access=identity\|read\|write: one handler, one table of what each access asks Google for, the same callback markers. An unknown access is refused (422) instead of quietly meaning read, and an instance without Google answers ?link=error for a grant too, not a silent return to the app. The frontend builds the URL in lib/googleConnect, and a contract test holds it to the backend route and its access values.	2026-10-20 12:49:11 +02:00
peter	59ec4a28ba	refactor(auth): one account per browser session The session held a wallet of up to six accounts, and a request picked one by a header or an ?account= parameter. It now holds one: user_id and its epoch. A sign-in replaces whoever was signed in, sign-out and account deletion end the session, and an account refused mid-session (revoked, suspended, deactivated) clears it. /api/me/accounts and /api/me/switch are gone, and with them their refusal codes and the UI copy for them. A cookie the multi-account build wrote carries no single epoch and is refused: its holder signs in once more. A Google link finishing in a browser now signed in as someone else is refused, as before for an account outside the wallet. A test holds that no app code reads the wallet keys or the account header again.	2026-10-20 04:36:26 +02:00
peter	dc4f4f5143	test(e2e): a plain member's shell is held end to end, and a page it cannot open lands on the feed The suite ran only as an admin, so nothing end to end held what a member sees. A second seeded account, a member with its own session, now walks every rail page (no admin entry, no admin call, no error state) and follows links to the admin pages. It found one: a member's link to an admin page showed the feed with no rail row current, and the address bar kept naming the admin page — only the content fell back. The navigation state now derives the page on screen through each module's gate (reachablePage), so the rail, the content, the tab title and the address bar agree, on a link, a Back, and a role taken away mid-session.	2026-10-20 04:08:11 +02:00
peter	909732b3c1	release: 0.108.1 (improvement/e42-backend-units)	2026-10-20 02:12:49 +02:00
peter	215f022fe3	chore(release): 0.108.1 — the backend in four units, each owning its tables; a first Watch later and a first resume point survive a write beside them	2026-10-20 02:04:56 +02:00
peter	6ea750a698	test: the commit check starts every request clean, and the fence docs claim what it holds The test's own writes are committed before a request, so a handler's Core DML can no longer hide in a transaction the test had already opened. The entry-point probe answers on a marked line the API's logging cannot mix into. A listed commit whose function no longer commits fails the fence. CLAUDE.md and the backend README now name what an AST cannot see; the route-commit tests no longer claim their read-back holds the commit.	2026-10-20 01:56:31 +02:00
peter	c5f121139b	fix: every first watch state survives a write beside it, and a folded Watch later keeps its pin A season marked watched, a pull from Plex and a YouTube resume point made their first row by a read then an add, and failed on the unique key when a first checkpoint or the watched mark made it in between. They go through the same insert-then-read as the checkpoint (the resume point is one upsert); a pull that meets a local change still waiting for its push leaves it. Folding duplicate Watch later lists moved their videos but not a Home pin, which the cascade then deleted; the first pin of a dropped copy moves onto the kept list. Marking one notice read trimmed before its own flag was flushed, so the trim left one past the cap. The route test now crosses the cap and reads the result from a session of its own.	2026-10-20 01:56:30 +02:00
peter	b04b2cbd62	test: the rewritten write handlers commit, and three more owner filters hold Each handler that now calls a service and commits is driven through HTTP once. A second account's watch state, Watch later list and private download profile are never read or changed. The autotag job's orphan prune is seen committed.	2026-10-20 00:45:22 +02:00
peter	3a5212cd8a	fix: a first watch state and a first Watch later survive two requests at once The first checkpoint of an item and the final flush beside it both read no state and both inserted; the loser failed on uq_plex_user_item. Two first saves each made a Watch later list. Both are created by an ON CONFLICT DO NOTHING insert and then read; a partial unique index holds one Watch later per user, and the migration folds an existing second list into the first. Plex keeps one per-item state map (rollup.item_state_map).	2026-10-20 00:45:22 +02:00
peter	b376373af5	fix(files): a folder's kind is answered before its new name, as before A blank name on a source folder answered 400 name_required; it answers 409 not_own again.	2026-10-20 00:45:22 +02:00
peter	8206384c91	test(fence): owner filters in any form, statement writes under any name, core commits by list The owner-filter check missed an aliased() model, an import alias, .in_() and filter_by; four live inline filters on aliased VideoState go through owned_by now. Cross-unit writes are seen under any import name, through sa.update and via __table__. Core modules may commit only where CORE_COMMITS names the reason; trim_read and audit.gc flush now and their callers commit. The pin target, the debug-session header and a link's target fetch through owned_row. The docs claim what the fence holds, and say what is beyond it.	2026-10-20 00:45:21 +02:00
peter	e27656ee7e	test: the request commit check sees Core DML and row locks A db.execute(update/delete) fires no flush and stages nothing, so a handler whose only write was Core DML could forget its commit and stay green. The check now asks Postgres whether the request opened a writing transaction. It found one real case: a repeated issue send returned with the session row still locked.	2026-10-20 00:45:20 +02:00
peter	d73cdcee97	fix(capture): every process registers every table before its first query The models now live in their units' packages, and a core model names a unit's by string (User.subscriptions -> Subscription). The download child imports neither catalog nor app.tables before db.get, so mapper configuration failed and every download, clip, edit and upload job died with child_exit. Each entry point now imports app.tables, and a test imports each one in a clean interpreter.	2026-10-20 00:45:20 +02:00
@@ -1 +1 @@
 .108.0
 .109.0