fix: persist chapter-names results into job payload from sync SSE handler

The SSE (non-async) chapter-names handler streamed results to the client
but never wrote them into the PocketBase job payload — only the initial
{pattern} stub was stored. The Review button then fetched the job and
found no results, showing 'No results found in this job's payload.'

Fix: accumulate allResults across batches (same as the async handler) and
write the full {pattern, slug, results:[...]} payload when marking done.

backend/internal/backend/handlers_textgen.go

@@ -233,6 +233,7 @@ func (s *Server) handleAdminTextGenChapterNames(w http.ResponseWriter, r *http.R
 		}
 	}
 
+	var allResults []proposedChapterTitle
 	chaptersDone := resumeFrom
 	firstEvent := true
 	for i, batch := range batches {
@@ -287,6 +288,7 @@ func (s *Server) handleAdminTextGenChapterNames(w http.ResponseWriter, r *http.R
 				NewTitle: p.Title,
 			})
 		}
+		allResults = append(allResults, result...)
 		chaptersDone += len(batch)
 
 		if jobID != "" && s.deps.AIJobStore != nil {
@@ -310,16 +312,20 @@ func (s *Server) handleAdminTextGenChapterNames(w http.ResponseWriter, r *http.R
 		sseWrite(evt)
 	}
 
-	// Mark job as done in PB.
+	// Mark job as done in PB, persisting results so the Review button works.
 	if jobID != "" && s.deps.AIJobStore != nil {
 		status := domain.TaskStatusDone
 		if jobCtx.Err() != nil {
 			status = domain.TaskStatusCancelled
 		}
+		resultsJSON, _ := json.Marshal(allResults)
+		finalPayload := fmt.Sprintf(`{"pattern":%q,"slug":%q,"results":%s}`,
+			req.Pattern, req.Slug, string(resultsJSON))
 		_ = s.deps.AIJobStore.UpdateAIJob(r.Context(), jobID, map[string]any{
 			"status":     string(status),
 			"items_done": chaptersDone,
 			"finished":   time.Now().Format(time.RFC3339),
+			"payload":    finalPayload,
 		})
 	}
 

scripts/pb-init-v3.sh

@@ -335,6 +335,14 @@ create "notifications" '{
     {"name":"created", "type":"date"}
   ]}'
 
+create "push_subscriptions" '{
+  "name":"push_subscriptions","type":"base","fields":[
+    {"name":"user_id",  "type":"text","required":true},
+    {"name":"endpoint", "type":"text","required":true},
+    {"name":"p256dh",   "type":"text","required":true},
+    {"name":"auth",     "type":"text","required":true}
+  ]}'
+
 create "ai_jobs" '{
   "name":"ai_jobs","type":"base","fields":[
     {"name":"kind",          "type":"text",  "required":true},
@@ -393,6 +401,7 @@ add_field "user_settings"  "font_size"              "number"
 add_field "user_settings"  "announce_chapter"       "bool"
 add_field "user_settings"  "audio_mode"             "text"
 add_field "books"          "archived"               "bool"
+add_field "app_users"      "notify_new_chapters"    "bool"
 
 # ── 6. Indexes ────────────────────────────────────────────────────────────────
 add_index "chapters_idx" "idx_chapters_idx_slug_number" \

ui/src/lib/components/ListeningMode.svelte

@@ -2,6 +2,7 @@
 	import { audioStore } from '$lib/audio.svelte';
 	import { cn } from '$lib/utils';
 	import { goto } from '$app/navigation';
+	import { fly } from 'svelte/transition';
 	import type { Voice } from '$lib/types';
 	import ChapterPickerOverlay from '$lib/components/ChapterPickerOverlay.svelte';
 
@@ -229,6 +230,7 @@
 <!-- Full-screen listening mode overlay -->
 <!-- svelte-ignore a11y_no_static_element_interactions -->
 <div
+	transition:fly={{ y: '100%', duration: 320, opacity: 1 }}
 	bind:this={overlayEl}
 	class="fixed inset-0 z-60 flex flex-col overflow-hidden"
 	style="

ui/src/lib/server/pocketbase.ts

@@ -96,6 +96,7 @@ export interface User {
 	polar_customer_id?: string;
 	polar_subscription_id?: string;
 	notify_new_chapters?: boolean;
+	notify_new_chapters_push?: boolean;
 }
 
 // ─── Auth token cache ─────────────────────────────────────────────────────────
@@ -1582,7 +1583,10 @@ export async function updateUserAvatarUrl(userId: string, avatarUrl: string): Pr
  */
 export async function updateUserNotificationPrefs(
 	userId: string,
-	prefs: { notify_new_chapters?: boolean }
+	prefs: {
+		notify_new_chapters?: boolean;
+		notify_new_chapters_push?: boolean;
+	}
 ): Promise<void> {
 	const token = await getToken();
 	const res = await fetch(`${PB_URL}/api/collections/app_users/records/${userId}`, {

ui/src/routes/+layout.svelte

@@ -64,6 +64,7 @@
 		} catch (e) { console.error('clear notifications:', e); }
 	}
 	$effect(() => { if (data.user) loadNotifications(); });
+	$effect(() => { if (notificationsOpen && data.user) loadNotifications(); });
 	const unreadCount = $derived(notifications.filter(n => !n.read).length);
 
 	// Close search on navigation
@@ -1136,12 +1137,10 @@
 <!-- Listening mode — mounted at root level, independent of audioStore.active,
      so closing/pausing audio never tears it down and loses context. -->
 {#if listeningModeOpen}
-	<div transition:fly={{ y: '100%', duration: 320, opacity: 1 }} style="pointer-events: none;">
-		<ListeningMode
-			onclose={() => { listeningModeOpen = false; listeningModeChapters = false; }}
-			openChapters={listeningModeChapters}
-		/>
-	</div>
+	<ListeningMode
+		onclose={() => { listeningModeOpen = false; listeningModeChapters = false; }}
+		openChapters={listeningModeChapters}
+	/>
 {/if}
 
 <!-- Universal search modal — shown from anywhere except focus mode / listening mode -->

ui/src/routes/api/profile/+server.ts

@@ -7,7 +7,7 @@ import { log } from '$lib/server/logger';
  * PATCH /api/profile
  *
  * Update mutable profile preferences (currently: notification preferences).
- * Body: { notify_new_chapters?: boolean }
+ * Body: { notify_new_chapters?: boolean, notify_new_chapters_push?: boolean }
  */
 export const PATCH: RequestHandler = async ({ locals, request }) => {
 	if (!locals.user) error(401, 'Not authenticated');
@@ -19,10 +19,13 @@ export const PATCH: RequestHandler = async ({ locals, request }) => {
 		error(400, 'Invalid JSON');
 	}
 
-	const prefs: { notify_new_chapters?: boolean } = {};
+	const prefs: { notify_new_chapters?: boolean; notify_new_chapters_push?: boolean } = {};
 	if (typeof body.notify_new_chapters === 'boolean') {
 		prefs.notify_new_chapters = body.notify_new_chapters;
 	}
+	if (typeof body.notify_new_chapters_push === 'boolean') {
+		prefs.notify_new_chapters_push = body.notify_new_chapters_push;
+	}
 
 	if (Object.keys(prefs).length === 0) {
 		error(400, 'No valid preferences provided');

ui/src/routes/profile/+page.server.ts

@@ -81,6 +81,7 @@ export const load: PageServerLoad = async ({ locals }) => {
 		email,
 		polarCustomerId,
 		notifyNewChapters: freshUser?.notify_new_chapters ?? true,
+		notifyNewChaptersPush: freshUser?.notify_new_chapters_push ?? true,
 		stats: stats ?? {
 			totalChaptersRead: 0, booksReading: 0, booksCompleted: 0,
 			booksPlanToRead: 0, booksDropped: 0, topGenres: [],

ui/src/routes/profile/+page.svelte

@@ -236,8 +236,10 @@
 	let pushError = $state('');
 
 	// ── In-app notifications ──────────────────────────────────────────────────────
-	let notifyNewChapters = $state(data.notifyNewChapters ?? true);
-	let notifyNewChaptersSaving = $state(false);
+	let notifyNewChapters     = $state(data.notifyNewChapters     ?? true);
+	let notifyNewChaptersPush = $state(data.notifyNewChaptersPush ?? true);
+	let notifyNewChaptersSaving     = $state(false);
+	let notifyNewChaptersPushSaving = $state(false);
 
 	async function toggleNotifyNewChapters() {
 		notifyNewChaptersSaving = true;
@@ -248,14 +250,27 @@
 				headers: { 'Content-Type': 'application/json' },
 				body: JSON.stringify({ notify_new_chapters: next })
 			});
-			if (res.ok) {
-				notifyNewChapters = next;
-			}
+			if (res.ok) notifyNewChapters = next;
 		} catch { /* ignore */ } finally {
 			notifyNewChaptersSaving = false;
 		}
 	}
 
+	async function toggleNotifyNewChaptersPush() {
+		notifyNewChaptersPushSaving = true;
+		const next = !notifyNewChaptersPush;
+		try {
+			const res = await fetch('/api/profile', {
+				method: 'PATCH',
+				headers: { 'Content-Type': 'application/json' },
+				body: JSON.stringify({ notify_new_chapters_push: next })
+			});
+			if (res.ok) notifyNewChaptersPush = next;
+		} catch { /* ignore */ } finally {
+			notifyNewChaptersPushSaving = false;
+		}
+	}
+
 	$effect(() => {
 		if (!browser) return;
 		if (!('serviceWorker' in navigator) || !('PushManager' in window)) {
@@ -802,8 +817,16 @@
 				</svg>
 			</span>
 			<span class="flex-1 text-sm font-medium text-(--color-text)">Notifications</span>
-			<span class="text-xs mr-2 hidden sm:inline {notifyNewChapters ? 'text-(--color-brand)' : 'text-(--color-muted)'}">
-				{notifyNewChapters ? 'On' : 'Off'}
+			<span class="text-xs mr-2 hidden sm:inline text-(--color-muted)">
+				{#if notifyNewChapters && pushState === 'subscribed'}
+					<span class="text-(--color-brand)">In-app · Push</span>
+				{:else if notifyNewChapters}
+					<span class="text-(--color-brand)">In-app</span>
+				{:else if pushState === 'subscribed'}
+					<span class="text-(--color-brand)">Push</span>
+				{:else}
+					Off
+				{/if}
 			</span>
 			<svg class={cn(chevronClass, expanded === 'notifications' ? 'rotate-90' : '')} fill="none" stroke="currentColor" viewBox="0 0 24 24">
 				<path stroke-linecap="round" stroke-linejoin="round" stroke-width="2" d="M9 5l7 7-7 7"/>
@@ -814,50 +837,24 @@
 		<div class="px-5 py-5 space-y-5 bg-(--color-surface-3)/30">
 			<span class="text-xs font-semibold text-(--color-muted) uppercase tracking-wider">Notifications</span>
 
-			<!-- In-app -->
-			<div class="flex items-start justify-between gap-4">
-				<div class="min-w-0">
-					<p class="text-sm font-medium text-(--color-text)">In-app notifications</p>
-					<p class="text-sm text-(--color-muted) mt-0.5">
-						{#if notifyNewChapters}
-							Notified when new chapters arrive in your library.
-						{:else}
-							In-app new-chapter notifications are disabled.
-						{/if}
-					</p>
-				</div>
-				<button
-					type="button"
-					onclick={toggleNotifyNewChapters}
-					disabled={notifyNewChaptersSaving}
-					class={cn(
-						'shrink-0 relative inline-flex h-6 w-11 items-center rounded-full transition-colors focus:outline-none disabled:opacity-50',
-						notifyNewChapters ? 'bg-(--color-brand)' : 'bg-(--color-surface-3)'
-					)}
-					role="switch"
-					aria-checked={notifyNewChapters}
-					title={notifyNewChapters ? 'Turn off in-app notifications' : 'Turn on in-app notifications'}
-				>
-					<span class={cn('inline-block h-4 w-4 transform rounded-full bg-white shadow transition-transform', notifyNewChapters ? 'translate-x-6' : 'translate-x-1')}></span>
-				</button>
-			</div>
-
-			<!-- Push -->
+			<!-- Browser push master toggle -->
 			{#if pushState !== 'unsupported'}
 			<div class="flex items-start justify-between gap-4">
 				<div class="min-w-0">
-					<p class="text-sm font-medium text-(--color-text)">Push notifications</p>
-					<p class="text-sm text-(--color-muted) mt-0.5">
+					<p class="text-sm font-medium text-(--color-text)">Browser push</p>
+					<p class="text-xs text-(--color-muted) mt-0.5">
 						{#if pushState === 'subscribed'}
-							Push enabled for new chapters in your library.
+							This browser is subscribed to push notifications.
 						{:else if pushState === 'denied'}
-							Blocked by your browser. Change in browser settings.
+							Blocked by your browser — change in browser settings.
+						{:else if pushState === 'loading'}
+							Updating…
 						{:else}
-							Get notified when new chapters arrive.
+							Subscribe to receive push notifications in this browser.
 						{/if}
 					</p>
 					{#if pushError}
-						<p class="text-sm text-(--color-danger) mt-1.5">{pushError}</p>
+						<p class="text-xs text-(--color-danger) mt-1">{pushError}</p>
 					{/if}
 				</div>
 				<div class="shrink-0">
@@ -890,6 +887,63 @@
 				</div>
 			</div>
 			{/if}
+
+			<!-- Per-category table -->
+			<div class="rounded-lg border border-(--color-border) overflow-hidden">
+				<!-- Header -->
+				<div class="grid grid-cols-[1fr_auto_auto] items-center gap-4 px-4 py-2 bg-(--color-surface-3)/60 border-b border-(--color-border)">
+					<span class="text-xs font-semibold text-(--color-muted) uppercase tracking-wider">Category</span>
+					<span class="text-xs font-semibold text-(--color-muted) uppercase tracking-wider w-12 text-center">In-app</span>
+					<span class="text-xs font-semibold text-(--color-muted) uppercase tracking-wider w-12 text-center">Push</span>
+				</div>
+				<!-- New chapters row -->
+				<div class="grid grid-cols-[1fr_auto_auto] items-center gap-4 px-4 py-3">
+					<div>
+						<p class="text-sm font-medium text-(--color-text)">New chapters</p>
+						<p class="text-xs text-(--color-muted) mt-0.5">When a book in your library gets new chapters</p>
+					</div>
+					<!-- In-app toggle -->
+					<div class="w-12 flex justify-center">
+						<button
+							type="button"
+							role="switch"
+							aria-checked={notifyNewChapters}
+							aria-label="In-app notifications for new chapters"
+							onclick={toggleNotifyNewChapters}
+							disabled={notifyNewChaptersSaving}
+							class={cn(
+								'shrink-0 relative inline-flex h-6 w-11 items-center rounded-full transition-colors focus:outline-none focus:ring-2 focus:ring-(--color-brand) focus:ring-offset-2 focus:ring-offset-(--color-surface) disabled:opacity-50',
+								notifyNewChapters ? 'bg-(--color-brand)' : 'bg-(--color-surface-3) border border-(--color-border)'
+							)}
+						>
+							<span class={cn('inline-block h-4 w-4 transform rounded-full bg-white shadow transition-transform', notifyNewChapters ? 'translate-x-6' : 'translate-x-1')}></span>
+						</button>
+					</div>
+					<!-- Push toggle -->
+					<div class="w-12 flex justify-center">
+						{#if pushState === 'unsupported'}
+							<span class="text-xs text-(--color-muted)" title="Push not supported in this browser">—</span>
+						{:else if pushState === 'denied'}
+							<span class="text-xs text-(--color-muted)" title="Push blocked by browser">—</span>
+						{:else}
+							<button
+								type="button"
+								role="switch"
+								aria-checked={notifyNewChaptersPush && pushState === 'subscribed'}
+								aria-label="Push notifications for new chapters"
+								onclick={toggleNotifyNewChaptersPush}
+								disabled={notifyNewChaptersPushSaving || pushState !== 'subscribed'}
+								class={cn(
+									'shrink-0 relative inline-flex h-6 w-11 items-center rounded-full transition-colors focus:outline-none focus:ring-2 focus:ring-(--color-brand) focus:ring-offset-2 focus:ring-offset-(--color-surface) disabled:opacity-40',
+									notifyNewChaptersPush && pushState === 'subscribed' ? 'bg-(--color-brand)' : 'bg-(--color-surface-3) border border-(--color-border)'
+								)}
+							>
+								<span class={cn('inline-block h-4 w-4 transform rounded-full bg-white shadow transition-transform', notifyNewChaptersPush && pushState === 'subscribed' ? 'translate-x-6' : 'translate-x-1')}></span>
+							</button>
+						{/if}
+					</div>
+				</div>
+			</div>
 		</div>
 		{/if}