refactor: use shared search helpers

auditui/app.py

@@ -27,6 +27,7 @@ from .table_utils import (
     filter_unfinished_items,
     format_item_as_row,
 )
+from .search_utils import build_search_text, filter_items
 from .ui import FilterScreen, HelpScreen, StatsScreen
 
 if TYPE_CHECKING:
@@ -472,11 +473,8 @@ class Auditui(App):
         items = self.all_items
 
         if self.filter_text:
-            filter_lower = self.filter_text.lower()
+            items = filter_items(items, self.filter_text,
-            items = [
+                                 self._get_search_text)
-                item for item in items
-                if filter_lower in self._get_search_text(item)
-            ]
             self._populate_table(items)
             self.update_status(
                 f"Filter: '{self.filter_text}' ({len(items)} books)")
@@ -493,21 +491,7 @@ class Auditui(App):
         cached = self._search_text_cache.get(cache_key)
         if cached is not None:
             return cached
-
+        search_text = build_search_text(item, self.library_client)
-        title = ""
-        authors = ""
-        if self.library_client:
-            title = self.library_client.extract_title(item)
-            authors = self.library_client.extract_authors(item)
-        else:
-            title = item.get("title", "")
-            authors = ", ".join(
-                a.get("name", "")
-                for a in item.get("authors", [])
-                if isinstance(a, dict) and a.get("name")
-            )
-
-        search_text = f"{title} {authors}".lower()
         self._search_text_cache[cache_key] = search_text
         return search_text
 

auditui/search_utils.py

@@ -0,0 +1,34 @@
+"""Search helpers for filtering library items."""
+
+from __future__ import annotations
+
+from typing import Callable
+
+from .library import LibraryClient
+
+
+def build_search_text(item: dict, library_client: LibraryClient | None) -> str:
+    """Build a lowercase search string for an item."""
+    if library_client:
+        title = library_client.extract_title(item)
+        authors = library_client.extract_authors(item)
+    else:
+        title = item.get("title", "")
+        authors = ", ".join(
+            a.get("name", "")
+            for a in item.get("authors", [])
+            if isinstance(a, dict) and a.get("name")
+        )
+    return f"{title} {authors}".lower()
+
+
+def filter_items(
+    items: list[dict],
+    filter_text: str,
+    get_search_text: Callable[[dict], str],
+) -> list[dict]:
+    """Filter items by a search string."""
+    if not filter_text:
+        return items
+    filter_lower = filter_text.lower()
+    return [item for item in items if filter_lower in get_search_text(item)]