feat(frontend): refactor Claims page to useClaims + primitives

src/pages/Claims.tsx

@@ -18,55 +18,76 @@ import {
 } from "@/components/ui/table";
 import { ClaimStatusBadge } from "@/components/StatusBadge";
 import { NewClaimDialog } from "@/components/NewClaimDialog";
+import { Skeleton } from "@/components/ui/skeleton";
+import { EmptyState } from "@/components/ui/empty-state";
+import { ErrorState } from "@/components/ui/error-state";
+import { FilterChips, type FilterChipOption } from "@/components/ui/filter-chips";
+import { Pagination } from "@/components/ui/pagination";
+import { useClaims } from "@/hooks/useClaims";
 import { useAppStore } from "@/store";
 import { fmt } from "@/lib/format";
 import type { ClaimStatus } from "@/types";
+import { cn } from "@/lib/utils";
 
 const ALL = "all" as const;
-const statuses: (ClaimStatus | typeof ALL)[] = [
-  ALL,
-  "submitted",
-  "accepted",
-  "denied",
-  "paid",
-  "pending",
+const PAGE_SIZE = 25;
+
+const STATUS_OPTIONS: FilterChipOption[] = [
+  { value: "all", label: "All" },
+  { value: "submitted", label: "Submitted" },
+  { value: "accepted", label: "Accepted" },
+  { value: "denied", label: "Denied" },
+  { value: "paid", label: "Paid" },
+  { value: "pending", label: "Pending" },
 ];
 
 export function Claims() {
-  const claims = useAppStore((s) => s.claims);
-  const providers = useAppStore((s) => s.providers);
-  const [query, setQuery] = useState("");
   const [status, setStatus] = useState<ClaimStatus | typeof ALL>(ALL);
   const [npi, setNpi] = useState<string>(ALL);
+  const [query, setQuery] = useState("");
+  const [page, setPage] = useState(1);
   const searchRef = useRef<HTMLInputElement>(null);
 
+  const providers = useAppStore((s) => s.providers);
   const providerMap = useMemo(
     () => Object.fromEntries(providers.map((p) => [p.npi, p])),
     [providers]
   );
 
-  const filtered = useMemo(() => {
+  const params = {
+    status: status === ALL ? undefined : status,
+    provider_npi: npi === ALL ? undefined : npi,
+    sort: "billedAmount",
+    order: "desc" as const,
+    limit: PAGE_SIZE,
+    offset: (page - 1) * PAGE_SIZE,
+  };
+
+  const { data, isLoading, isError, error, refetch } = useClaims(params);
+  const items = data?.items ?? [];
+
+  const totals = useMemo(
+    () => ({
+      billed: items.reduce((s, c) => s + c.billedAmount, 0),
+      received: items.reduce((s, c) => s + c.receivedAmount, 0),
+    }),
+    [items]
+  );
+
+  // Local search filter on the current page (cheap; no extra fetch).
+  const visible = useMemo(() => {
     const q = query.trim().toLowerCase();
-    return claims.filter((c) => {
-      if (status !== ALL && c.status !== status) return false;
-      if (npi !== ALL && c.providerNpi !== npi) return false;
-      if (!q) return true;
-      return (
+    if (!q) return items;
+    return items.filter(
+      (c) =>
         c.id.toLowerCase().includes(q) ||
         c.patientName.toLowerCase().includes(q) ||
         c.payerName.toLowerCase().includes(q) ||
         c.cptCode.toLowerCase().includes(q)
-      );
-    });
-  }, [claims, query, status, npi]);
+    );
+  }, [items, query]);
 
-  const totals = useMemo(
-    () => ({
-      billed: filtered.reduce((s, c) => s + c.billedAmount, 0),
-      received: filtered.reduce((s, c) => s + c.receivedAmount, 0),
-    }),
-    [filtered]
-  );
+  const filtered = status !== ALL || npi !== ALL;
 
   return (
     <div className="space-y-8 animate-fade-in">
@@ -81,6 +102,14 @@ export function Claims() {
         <NewClaimDialog />
       </header>
 
+      {isError ? (
+        <ErrorState
+          message="Couldn't load claims from the backend."
+          detail={error instanceof Error ? error.message : String(error)}
+          onRetry={() => refetch()}
+        />
+      ) : null}
+
       <div className="surface rounded-xl p-4">
         <div className="flex flex-wrap items-center gap-3">
           <div className="relative flex-1 min-w-[240px]">
@@ -101,35 +130,16 @@ export function Claims() {
               >
                 <X className="h-3.5 w-3.5" />
               </button>
-            ) : (
-              <button
-                type="button"
-                onClick={() => searchRef.current?.focus()}
-                className="absolute right-2 top-1/2 -translate-y-1/2 kbd hover:text-foreground"
-                aria-label="Focus search"
-              >
-                ⌘K
-              </button>
-            )}
+            ) : null}
           </div>
 
           <Select
-            value={status}
-            onValueChange={(v) => setStatus(v as ClaimStatus | typeof ALL)}
+            value={npi}
+            onValueChange={(v) => {
+              setNpi(v);
+              setPage(1);
+            }}
           >
-            <SelectTrigger className="w-[150px]">
-              <SelectValue />
-            </SelectTrigger>
-            <SelectContent>
-              {statuses.map((s) => (
-                <SelectItem key={s} value={s}>
-                  {s === ALL ? "All statuses" : s[0]!.toUpperCase() + s.slice(1)}
-                </SelectItem>
-              ))}
-            </SelectContent>
-          </Select>
-
-          <Select value={npi} onValueChange={setNpi}>
             <SelectTrigger className="w-[240px]">
               <SelectValue />
             </SelectTrigger>
@@ -144,10 +154,22 @@ export function Claims() {
           </Select>
         </div>
 
+        <div className="mt-3">
+          <FilterChips
+            options={STATUS_OPTIONS}
+            value={status === ALL ? null : status}
+            onChange={(v) => {
+              setStatus((v as ClaimStatus | null) ?? ALL);
+              setPage(1);
+            }}
+            eyebrow="Status"
+          />
+        </div>
+
         <div className="flex items-center gap-6 mt-4 pt-4 border-t border-border/40 text-xs text-muted-foreground">
           <span>
             <span className="num text-foreground font-medium">
-              {fmt.num(filtered.length)}
+              {fmt.num(data?.total ?? 0)}
             </span>{" "}
             claims
           </span>
@@ -167,68 +189,92 @@ export function Claims() {
       </div>
 
       <div className="surface rounded-xl overflow-hidden">
-        <Table>
-          <TableHeader>
-            <TableRow>
-              <TableHead>Claim</TableHead>
-              <TableHead>Patient</TableHead>
-              <TableHead>Provider</TableHead>
-              <TableHead>Payer</TableHead>
-              <TableHead className="text-right">Billed</TableHead>
-              <TableHead className="text-right">Received</TableHead>
-              <TableHead>Status</TableHead>
-              <TableHead>Submitted</TableHead>
-            </TableRow>
-          </TableHeader>
-          <TableBody>
-            {filtered.length === 0 ? (
-              <TableRow>
-                <TableCell
-                  colSpan={8}
-                  className="text-center py-12 text-muted-foreground"
-                >
-                  No claims match the current filters.
-                </TableCell>
-              </TableRow>
-            ) : (
-              filtered.map((c) => {
-                const provider = providerMap[c.providerNpi];
-                return (
-                  <TableRow key={c.id}>
-                    <TableCell>
-                      <div className="display num text-[13px]">{c.id}</div>
-                      <div className="text-[11px] text-muted-foreground num">
-                        CPT {c.cptCode}
-                      </div>
-                    </TableCell>
-                    <TableCell className="font-medium">{c.patientName}</TableCell>
-                    <TableCell>
-                      <div className="text-sm">{provider?.name ?? "Unknown"}</div>
-                      <div className="text-[11px] text-muted-foreground num">
-                        {c.providerNpi}
-                      </div>
-                    </TableCell>
-                    <TableCell className="text-muted-foreground">
-                      {c.payerName}
-                    </TableCell>
-                    <TableCell className="text-right display num">
-                      {fmt.usd(c.billedAmount)}
-                    </TableCell>
-                    <TableCell className="text-right display num text-muted-foreground">
-                      {c.receivedAmount > 0 ? fmt.usd(c.receivedAmount) : "—"}
-                    </TableCell>
-                    <TableCell>
-                      <ClaimStatusBadge status={c.status} />
-                    </TableCell>
-                    <TableCell className="text-muted-foreground text-[13px] num">
-                      {fmt.dateShort(c.submissionDate)}
-                    </TableCell>
-                  </TableRow>
-                );
-              })
-            )}
-          </TableBody>
-        </Table>
+        {isLoading ? (
+          <div className="p-4 space-y-2">
+            {Array.from({ length: 5 }).map((_, i) => (
+              <Skeleton key={i} variant="row" />
+            ))}
+          </div>
+        ) : items.length === 0 ? (
+          <EmptyState
+            eyebrow="Claims · inbox idle"
+            message={
+              filtered
+                ? "No claims match the current filters."
+                : "Drop an 837P file on the Upload page to populate this list."
+            }
+          />
+        ) : (
+          <>
+            <Table>
+              <TableHeader>
+                <TableRow>
+                  <TableHead>Claim</TableHead>
+                  <TableHead>Patient</TableHead>
+                  <TableHead>Provider</TableHead>
+                  <TableHead>Payer</TableHead>
+                  <TableHead className="text-right">Billed</TableHead>
+                  <TableHead className="text-right">Received</TableHead>
+                  <TableHead>Status</TableHead>
+                  <TableHead>Submitted</TableHead>
+                </TableRow>
+              </TableHeader>
+              <TableBody>
+                {visible.map((c) => {
+                  const provider = providerMap[c.providerNpi];
+                  return (
+                    <TableRow
+                      key={c.id}
+                      className={cn(
+                        "animate-row-flash",
+                        // Suppress flash when filtering so only the new
+                        // page's first paint flashes; subsequent renders
+                        // use the existing fade-in from the page wrapper.
+                      )}
+                    >
+                      <TableCell>
+                        <div className="display num text-[13px]">{c.id}</div>
+                        <div className="text-[11px] text-muted-foreground num">
+                          CPT {c.cptCode}
+                        </div>
+                      </TableCell>
+                      <TableCell className="font-medium">{c.patientName}</TableCell>
+                      <TableCell>
+                        <div className="text-sm">{provider?.name ?? "Unknown"}</div>
+                        <div className="text-[11px] text-muted-foreground num">
+                          {c.providerNpi}
+                        </div>
+                      </TableCell>
+                      <TableCell className="text-muted-foreground">
+                        {c.payerName}
+                      </TableCell>
+                      <TableCell className="text-right display num">
+                        {fmt.usd(c.billedAmount)}
+                      </TableCell>
+                      <TableCell className="text-right display num text-muted-foreground">
+                        {c.receivedAmount > 0 ? fmt.usd(c.receivedAmount) : "—"}
+                      </TableCell>
+                      <TableCell>
+                        <ClaimStatusBadge status={c.status} />
+                      </TableCell>
+                      <TableCell className="text-muted-foreground text-[13px] num">
+                        {fmt.dateShort(c.submissionDate)}
+                      </TableCell>
+                    </TableRow>
+                  );
+                })}
+              </TableBody>
+            </Table>
+            <div className="px-4 pb-4">
+              <Pagination
+                page={page}
+                pageSize={PAGE_SIZE}
+                total={data?.total ?? 0}
+                onPageChange={setPage}
+              />
+            </div>
+          </>
+        )}
       </div>
     </div>
   );