From 833a023688bfd37b093c0fc54044be2f4f11ced5 Mon Sep 17 00:00:00 2001
From: Ajax Davis <thomasalwyndavis@gmail.com>
Date: Thu, 11 Dec 2025 09:33:45 +1000
Subject: [PATCH] fix(tool-search): fix download sorting and enable text
 selection

- Add nullish coalescing for downloads (handle null/undefined values)
- Add select-text class to Link and Card for text selection
---
 apps/web/src/app/tool/tool-search/page.tsx | 8 +++++---
 1 file changed, 5 insertions(+), 3 deletions(-)

diff --git a/apps/web/src/app/tool/tool-search/page.tsx b/apps/web/src/app/tool/tool-search/page.tsx
index a9c59d7..d42b31c 100644
--- a/apps/web/src/app/tool/tool-search/page.tsx
+++ b/apps/web/src/app/tool/tool-search/page.tsx
@@ -88,7 +88,9 @@ export default function ToolSearchPage(): React.ReactElement {
             if (!aIsBroken && bIsBroken) return -1;
             // Within same broken status, sort by selected criterion
             if (sortBy === 'downloads') {
-              return b.package.npmDownloadsLastMonth - a.package.npmDownloadsLastMonth;
+              const aDownloads = a.package.npmDownloadsLastMonth ?? 0;
+              const bDownloads = b.package.npmDownloadsLastMonth ?? 0;
+              return bDownloads - aDownloads;
             }
             // Sort by recent (createdAt descending)
             return new Date(b.createdAt).getTime() - new Date(a.createdAt).getTime();
@@ -240,9 +242,9 @@ export default function ToolSearchPage(): React.ReactElement {
                   <Link
                     key={tool.id}
                     href={`/tool/${tool.package.npmPackageName}/${tool.exportName}`}
-                    className="block"
+                    className="block select-text"
                   >
-                    <Card className="flex flex-col h-full hover:border-foreground-tertiary transition-colors cursor-pointer">
+                    <Card className="flex flex-col h-full hover:border-foreground-tertiary transition-colors cursor-pointer select-text">
                       <CardHeader className="flex-none">
                         {/* Top row: Title + metadata */}
                         <div className="flex items-start justify-between gap-2">