fix: hasBody filter, body preview ellipsis, and sort/perPage reactivity

- Wire hasBody through QueryFilters so the checkbox actually filters
  results client-side (issues with body > 20 chars)
- Fix body preview: check stripped body length for ellipsis, not the
  original markdown body
- Fix sort/perPage dropdown changes not triggering a new search
  (the effect only matched text or non-text filter changes, missing
  the case where only executeSearch reference changed)

Author: Felipeness

src/features/issues/components/issue-card.tsx

@@ -32,12 +32,12 @@ export function IssueCard({ issue }: IssueCardProps) {
     now: t("timeAgo.now"),
   };
 
-  const bodyPreview = issue.body
-    ? issue.body
-        .replace(/[#*`>\-[\]()!]/g, "")
-        .trim()
-        .substring(0, 120) + (issue.body.length > 120 ? "..." : "")
+  const strippedBody = issue.body
+    ? issue.body.replace(/[#*`>\-[\]()!]/g, "").trim()
     : "";
+  const bodyPreview = strippedBody.length > 120
+    ? strippedBody.substring(0, 120) + "..."
+    : strippedBody;
 
   const avatarUrl = repo.owner
     ? `https://github.com/${repo.owner}.png?size=40`

src/features/search/components/filters-panel.tsx

@@ -47,6 +47,7 @@ function toQueryFilters(filters: {
   noAssignee: boolean;
   noComments: boolean;
   noPr: boolean;
+  hasBody: boolean;
 }): QueryFilters {
   return {
     label: filters.label,
@@ -59,6 +60,7 @@ function toQueryFilters(filters: {
     noAssignee: filters.noAssignee,
     noComments: filters.noComments,
     noPr: filters.noPr,
+    hasBody: filters.hasBody,
   };
 }
 

src/features/search/hooks/use-github-search.ts

@@ -196,7 +196,11 @@ export function useGitHubSearch({
           dispatch({ type: "SET_RATE_LIMIT", rateLimit });
         }
 
-        dispatch({ type: "FETCH_SUCCESS", issues, totalCount, append });
+        const filteredIssues = filters.hasBody
+          ? issues.filter((issue) => issue.body && issue.body.trim().length > 20)
+          : issues;
+
+        dispatch({ type: "FETCH_SUCCESS", issues: filteredIssues, totalCount, append });
       });
     },
     [filters, sort, perPage],
@@ -221,7 +225,8 @@ export function useGitHubSearch({
       prev.dateDays !== filters.dateDays ||
       prev.noAssignee !== filters.noAssignee ||
       prev.noComments !== filters.noComments ||
-      prev.noPr !== filters.noPr;
+      prev.noPr !== filters.noPr ||
+      prev.hasBody !== filters.hasBody;
 
     if (debounceRef.current) {
       clearTimeout(debounceRef.current);
@@ -242,8 +247,8 @@ export function useGitHubSearch({
       return;
     }
 
-    // Initial mount (prev === filters on first render via ref initialization)
-    if (prev === filters) {
+    // Initial mount or sort/perPage changed (executeSearch changed but no filter field changed)
+    if (!textChanged && !nonTextChanged) {
       executeSearch(1, false);
     }
 

src/features/search/types.ts

@@ -26,6 +26,7 @@ export type QueryFilters = {
   readonly noAssignee: boolean;
   readonly noComments: boolean;
   readonly noPr: boolean;
+  readonly hasBody: boolean;
 };
 
 export type TimeAgoTranslations = {