Skip to content

feat(tomesync): shelves axis on the device (build 36)#143

Merged
bndct-devops merged 11 commits into
mainfrom
feat/shelves-on-device
Jul 19, 2026
Merged

feat(tomesync): shelves axis on the device (build 36)#143
bndct-devops merged 11 commits into
mainfrom
feat/shelves-on-device

Conversation

@bndct-devops

Copy link
Copy Markdown
Owner

Overnight run PR 12/12 — stacked on #142; merge in order. The only plugin build in the stack (36 / 1.10.0).

What

  • Server: GET /tome-sync/shelves (user's shelves + resolved counts) and GET /tome-sync/shelf-books?shelf_id= (same book shape as the series drill-down). Shelf params resolve through the device-supported subset of the dashboard filters — full-text q, series/no_series, author, tag, format, language (messy-variant folding), library, reading status, min rating. The expressions mirror books.list_books (documented as the source of the semantics) and a drift test asserts the resolver returns the same books as /books for identical params. Unsupported keys (e.g. missing, sort) are reported in the response, never fatal. Foreign shelves 404.
  • Plugin (build 36): "Shelves" entry in the series browser between Browse-by-author and the series list; shelf list with counts; drill-down reuses the shared _bookListMenu as a mixed list, so downloads file each book by its own identity and hold-to-set-status works. Empty state points at the web app.

Verification

  • 13 backend tests green (5 new: counts across messy language variants, drill-down shape + reading-status filter, foreign-shelf 404, /books drift check, unsupported-params tolerance) + the build-pin test updated for 36/1.10.0.
  • Generated Lua syntax-checked (luajit).
  • Emulator round-trip against dev: installed the baked build-36 plugin, opened the browser (Shelves entry present), listed shelves — the real "German" shelf resolved to (5) from {"language": "de"} including a deu-tagged book — and drilled in: 4 Tränenvogel volumes + a German Bad Guys volume with its "reading" marker and "Download all (5)". Screenshots in the run log.
  • Not yet on real hardware — this is the one PR in the stack that wants a Kindle pass (it can ride the same device-validation session as build 35's sync-on-suspend).

bndct-devops and others added 11 commits July 18, 2026 23:46
- The tooltip resolves the day under the cursor (date + that day's minutes,
  or "no reading" inside a gap) below the book totals — the per-day data was
  only legible as tick opacity before.
- Phones (<480px container): the rail narrows 200->96px so the ribbon keeps
  the viewport; coarse-pointer taps inspect first (tooltip), navigate on the
  second tap on the same bar. Scroll or empty-space tap dismisses.

Book-page activity sparkline (planned as part of this PR) was investigated
and dropped: the per-book stats hero already charts per-day activity for
both live sessions and imported page-stats (sessions are synthesized from
page-stats), so the strip would duplicate an adjacent chart.

Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
GET /api/meta/whats-new parses CHANGELOG.md for the running version's
section (Unreleased fallback for dev builds); the frontend shows a one-time
dismissible panel when the server version differs from the last one this
browser saw. Fresh browsers baseline silently — first login never opens
with a modal. Panel portals to <body>: the header that mounts it has
backdrop-blur, which would otherwise contain the fixed overlay.

GET /api/meta/update-check (admin-only) compares against the latest GitHub
release, cached in memory 24h (1h after failures), gated by
TOME_UPDATE_CHECK (default true). Settings -> About shows a quiet
"vX.Y.Z available" link only when something is newer.

CHANGELOG.md is now copied into the Docker image for the panel.

Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
Upload dialog: files are sha256-hashed in the browser (sequential, 512MB
cap) and checked via the existing POST /books/check-hashes before upload;
exact duplicates show "already in your library" with a link and are
skipped. The server already deduped silently — this saves the transfer
and makes it visible. Hash-check failures degrade to the old behavior.

Admin -> Covers (GET /admin/covers/audit): flags missing, unreadable, and
low-res covers (PIL header reads only). The floor is 300px — the standard
Google Books cover is 329px and fine in practice; the audit exists for the
98-128px thumbnails. Missing covers offer one-click auto-fix via the
existing cover-candidates + set-cover endpoints (nothing to downgrade);
low-res rows deep-link to the book's cover picker.

Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
Global palette mounted from the shared header: full-text book search via
GET /books?q= (debounced, covers in rows), series/authors ranked from the
facets list client-side (startsWith > includes > shorter), and plain
navigation actions (admin sees Admin). Arrow keys + Enter, Esc closes,
portaled past the header's backdrop-blur containing block. Registered in
the "?" shortcuts help.

Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
New position_history table (additive, created by the startup create_all
like every other table): upsert_position — the single choke point all
position writers go through — appends an entry whenever the position moves
meaningfully (>=0.2% or locator change; the idle heartbeat can't spam it),
pruned to the newest 40 per user+book.

GET /books/{id}/position-history lists the log + live position;
POST .../{hid}/restore sets the live position back. Restore is an explicit
override of the sticky-completion rule: recovering from a false 100% also
un-finishes the book (status/finished_at), or the position would come back
while the book stayed "read". The restore itself is logged, so it can be
undone the same way. Devices converge on their next pull.

UI: history button on the book page's Reading Stats header opens the log
with per-entry Restore; refreshes stats + status pill after.

Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
GET /books/{id}/reader-pacing returns the chapter fraction boundaries,
book word count, and the user's true WPM (same rule as the stats tile:
finished word-counted books with >=5min reconciled read-time; null without
history). The reader computes words-left from the current relocate
fraction against the chapter's end_fraction — no pagination involved —
and shows "~N min left" in the footer beside the chapter name (250wpm
default, tooltip says which pace is in use). Recomputes when pacing
arrives, since the initial relocate races the fetch.

Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
Per-user notification_channels table (additive) + fanout hooked at the ORM
layer: an after_insert listener collects fresh Notification rows on the
session, after_commit hands them to a daemon thread that loads the owners'
enabled channels and POSTs each (5s timeout, best-effort, no retries).
Zero changes at the six Notification creation sites; a rolled-back
transaction never sends.

Settings -> Notifications: add/pause/delete channels per kind (ntfy topic
URL + optional bearer token; Gotify base URL + app token; bare webhook),
with a synchronous test button that surfaces delivery errors. Tokens are
never echoed back (has_token only). TOME_OUTBOUND_NOTIFY=false is the
operator kill-switch.

Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
Stateless preview + apply: POST /import/reading-csv detects the dialect
(header sets), parses read/currently-reading rows (Goodreads ="ISBN"
unwrapping, both date formats, half-star StoryGraph ratings), matches
against the caller's visible library (ISBN -> exact normalized
title+author-overlap -> fuzzy title >=0.88), and annotates exactly what
applying would fill. POST /import/reading-csv/apply is fill-gaps-only:
status only onto unread, rating/review/finished_at only when absent —
an import can never clobber live sync state or curation (same philosophy
as the TomeSync sweep). To-read shelves are skipped by design.

Settings -> Import reading history: upload, per-row preview with match
provenance and no-op dimming, checkbox selection, applied-counts summary.

Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
Backup: GET /admin/backup/download streams a tarball with a consistent
SQLite snapshot (sqlite3 backup API — never a raw copy of a live WAL db),
the cover cache, and a manifest. Library files stay on disk by design.

Restore is two-phase: the upload is validated (tar shape, manifest,
integrity_check, table counts) and STAGED; the swap happens at the next
server start inside create_db_engine() — before any engine or connection
exists — with the current DB kept as tome.db.pre-restore-<ts>. A corrupt
staging file is parked as .failed and startup proceeds on the current DB.
Typing RESTORE is required to arm it; staging can be cancelled.

Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
The backend round-trip already existed end to end (PUT /annotations/{id}
supports device-synced highlights with LWW mtime bumping; DELETE writes
tombstones; the plugin's _applyForeign applies newer note edits) — but the
web only exposed delete. The Highlights page and the book page's
Highlights & Notes section now edit/add notes inline; saves land on
devices at their next sync like any cross-device edit.

Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
The series browser gains a Shelves entry: GET /tome-sync/shelves lists the
user's shelves with resolved counts; GET /tome-sync/shelf-books drills into
the shared _bookListMenu shape (mixed list — downloads file per-book, like
author/search lists). Shelf params resolve server-side through the
device-supported subset of the dashboard filters (q/series/no_series/
author/tag/format/language/library_id/reading_status/min_rating); the
expressions mirror books.list_books and a drift test compares results
against /books for identical params. Unsupported keys are reported, not
fatal. Foreign shelves 404.

Build 36 / semver 1.10.0.

Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
@bndct-devops
bndct-devops changed the base branch from feat/highlight-roundtrip to main July 19, 2026 10:10
@bndct-devops
bndct-devops merged commit b87f258 into main Jul 19, 2026
@bndct-devops
bndct-devops deleted the feat/shelves-on-device branch July 19, 2026 10:11
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant