From 353d7397eb5eb8c1b2e91486b7bec839a2a813d3 Mon Sep 17 00:00:00 2001
From: Admin <admin@kalekber.cc>
Date: Wed, 4 Mar 2026 00:49:50 +0500
Subject: [PATCH] fix(ui/books): guard null book fields in template; log null
 titles from PocketBase

---
 ui/src/lib/server/pocketbase.ts  | 8 +++++++-
 ui/src/routes/books/+page.svelte | 4 ++--
 2 files changed, 9 insertions(+), 3 deletions(-)

diff --git a/ui/src/lib/server/pocketbase.ts b/ui/src/lib/server/pocketbase.ts
index 6a1f7c8..0e2de72 100644
--- a/ui/src/lib/server/pocketbase.ts
+++ b/ui/src/lib/server/pocketbase.ts
@@ -139,7 +139,13 @@ async function listOne<T>(collection: string, filter: string): Promise<T | null>
 // ─── Books ────────────────────────────────────────────────────────────────────
 
 export async function listBooks(): Promise<Book[]> {
-	return listAll<Book>('books', '', '+title');
+	const books = await listAll<Book>('books', '', '+title');
+	const nullTitles = books.filter((b) => b.title == null).length;
+	if (nullTitles > 0) {
+		log.warn('pocketbase', 'listBooks: books with null title', { count: nullTitles, total: books.length });
+	}
+	log.debug('pocketbase', 'listBooks', { total: books.length, nullTitles });
+	return books;
 }
 
 export async function getBook(slug: string): Promise<Book | null> {
diff --git a/ui/src/routes/books/+page.svelte b/ui/src/routes/books/+page.svelte
index 23c0daf..060ac12 100644
--- a/ui/src/routes/books/+page.svelte
+++ b/ui/src/routes/books/+page.svelte
@@ -58,10 +58,10 @@
 				<!-- Info -->
 				<div class="p-2 flex flex-col gap-1 flex-1">
 					<h2 class="text-xs font-semibold text-zinc-100 line-clamp-2 leading-snug">
-						{book.title}
+						{book.title ?? ''}
 					</h2>
 					{#if book.author}
-						<p class="text-xs text-zinc-400 truncate">{book.author}</p>
+						<p class="text-xs text-zinc-400 truncate">{book.author ?? ''}</p>
 					{/if}
 
 					<div class="mt-auto pt-1 flex items-center justify-between gap-1">