fix: show URL-imported book covers in chatbot book picker

app/api/chat.py

@@ -39,6 +39,7 @@ from app.services.token_budget import check_and_warn_budget
 from app.repositories.book_repo import BookRepository
 from app.repositories.link_repo import LinkRepository
 from app.repositories.visitor_repo import VisitorRepository
+from app.services.pipeline.helpers import public_cover_src
 from app.services.visitor_profile_service import (
     profile_is_complete,
     resolve_profile,
@@ -237,11 +238,13 @@ async def init_session(
         books_payload = []
         for b in active_books:
             link = await link_repo.get_for_book(b.id, author.id)
+            cover = public_cover_src(b)
             books_payload.append({
                 "id": b.id,
                 "title": b.title,
                 "tagline": b.tagline,
-                "cover_path": b.cover_path,
+                "cover_path": cover or b.cover_path,
+                "cover_url": cover or None,
                 "ai_summary": b.ai_summary,
                 "purchase_url": link.purchase_url if link else None,
             })

app/schemas/chatbot.py

@@ -55,6 +55,7 @@ class BookInfo(BaseModel):
     title: str
     tagline: str | None
     cover_path: str | None
+    cover_url: str | None = None
     ai_summary: str | None = None
     purchase_url: str | None = None
 

app/services/pipeline/helpers.py

@@ -268,6 +268,30 @@ def selected_book_title(books: list, selected_book_id: str | None) -> str:
     return "the books"
 
 
+def public_cover_src(book) -> str:
+    """Return a browser-resolvable cover URL for widget / session payloads.
+
+    Prefers local processed images under /data/covers (rewritten to /covers),
+    then falls back to the scraped remote cover_url from URL import.
+    """
+    for attr in ("cover_medium_path", "cover_thumbnail_path", "cover_path"):
+        raw = getattr(book, attr, None) or ""
+        path = str(raw).strip()
+        if not path:
+            continue
+        if path.startswith("/data/covers/"):
+            return "/covers/" + path[len("/data/covers/"):]
+        if path.startswith("/covers/"):
+            return path
+        if path.startswith("http://") or path.startswith("https://") or path.startswith("data:"):
+            return path
+    remote = getattr(book, "cover_url", None) or ""
+    remote = str(remote).strip()
+    if remote.startswith("http://") or remote.startswith("https://") or remote.startswith("data:"):
+        return remote
+    return ""
+
+
 def book_selector_items(books: list) -> list[dict]:
     """Build the list of book selector items for the widget UI."""
     items = []
@@ -279,7 +303,7 @@ def book_selector_items(books: list) -> list[dict]:
             "id": book.id,
             "title": book.title,
             "tagline": tagline,
-            "cover_path": book.cover_path,
+            "cover_path": public_cover_src(book),
         })
     return items
 

tests/unit/test_public_cover_src.py

@@ -0,0 +1,52 @@
+"""Unit tests for chatbot public cover URL resolution."""
+
+from types import SimpleNamespace
+
+from app.services.pipeline.helpers import book_selector_items, public_cover_src
+
+
+def test_public_cover_src_rewrites_local_medium_path():
+    book = SimpleNamespace(
+        cover_medium_path="/data/covers/auth1/book1/medium.webp",
+        cover_thumbnail_path="/data/covers/auth1/book1/thumb.webp",
+        cover_path="/data/covers/auth1/book1/original.jpg",
+        cover_url="https://cdn.example.com/ignored.jpg",
+    )
+    assert public_cover_src(book) == "/covers/auth1/book1/medium.webp"
+
+
+def test_public_cover_src_falls_back_to_remote_cover_url():
+    book = SimpleNamespace(
+        cover_medium_path=None,
+        cover_thumbnail_path=None,
+        cover_path=None,
+        cover_url="https://images-na.ssl-images-amazon.com/images/I/abc.jpg",
+    )
+    assert public_cover_src(book) == (
+        "https://images-na.ssl-images-amazon.com/images/I/abc.jpg"
+    )
+
+
+def test_public_cover_src_empty_when_missing():
+    book = SimpleNamespace(
+        cover_medium_path=None,
+        cover_thumbnail_path=None,
+        cover_path=None,
+        cover_url=None,
+    )
+    assert public_cover_src(book) == ""
+
+
+def test_book_selector_items_uses_public_cover():
+    book = SimpleNamespace(
+        id="b1",
+        title="The Key",
+        tagline="A story",
+        status="ready",
+        cover_medium_path=None,
+        cover_thumbnail_path=None,
+        cover_path=None,
+        cover_url="https://example.com/cover.jpg",
+    )
+    items = book_selector_items([book])
+    assert items[0]["cover_path"] == "https://example.com/cover.jpg"