From 2f245d2d03b768c320e1038582f29ed8d9f2b66d Mon Sep 17 00:00:00 2001
From: CREDO23 <thierrybakera12@gmail.com>
Date: Wed, 14 Jan 2026 13:28:49 +0200
Subject: [PATCH] clean up SURFSENSE_DOCS handling from DocumentsTableShell

---
 .../components/DocumentsTableShell.tsx        | 24 ++++++-------------
 1 file changed, 7 insertions(+), 17 deletions(-)

diff --git a/surfsense_web/app/dashboard/[search_space_id]/documents/(manage)/components/DocumentsTableShell.tsx b/surfsense_web/app/dashboard/[search_space_id]/documents/(manage)/components/DocumentsTableShell.tsx
index 566e103ac..94c0626e6 100644
--- a/surfsense_web/app/dashboard/[search_space_id]/documents/(manage)/components/DocumentsTableShell.tsx
+++ b/surfsense_web/app/dashboard/[search_space_id]/documents/(manage)/components/DocumentsTableShell.tsx
@@ -79,25 +79,17 @@ export function DocumentsTableShell({
 		[documents, sortKey, sortDesc]
 	);
 
-	// Filter out SURFSENSE_DOCS for selection purposes
-	const selectableDocs = React.useMemo(
-		() => sorted.filter((d) => d.document_type !== "SURFSENSE_DOCS"),
-		[sorted]
-	);
-
-	const allSelectedOnPage =
-		selectableDocs.length > 0 && selectableDocs.every((d) => selectedIds.has(d.id));
-	const someSelectedOnPage =
-		selectableDocs.some((d) => selectedIds.has(d.id)) && !allSelectedOnPage;
+	const allSelectedOnPage = sorted.length > 0 && sorted.every((d) => selectedIds.has(d.id));
+	const someSelectedOnPage = sorted.some((d) => selectedIds.has(d.id)) && !allSelectedOnPage;
 
 	const toggleAll = (checked: boolean) => {
 		const next = new Set(selectedIds);
 		if (checked)
-			selectableDocs.forEach((d) => {
+			sorted.forEach((d) => {
 				next.add(d.id);
 			});
 		else
-			selectableDocs.forEach((d) => {
+			sorted.forEach((d) => {
 				next.delete(d.id);
 			});
 		setSelectedIds(next);
@@ -238,10 +230,9 @@ export function DocumentsTableShell({
 									const icon = getDocumentTypeIcon(doc.document_type);
 									const title = doc.title;
 									const truncatedTitle = title.length > 30 ? `${title.slice(0, 30)}...` : title;
-									const isSurfsenseDoc = doc.document_type === "SURFSENSE_DOCS";
 									return (
 										<motion.tr
-											key={`${doc.document_type}-${doc.id}`}
+											key={doc.id}
 											initial={{ opacity: 0, y: 10 }}
 											animate={{
 												opacity: 1,
@@ -258,9 +249,8 @@ export function DocumentsTableShell({
 										>
 											<TableCell className="px-4 py-3">
 												<Checkbox
-													checked={selectedIds.has(doc.id) && !isSurfsenseDoc}
-													onCheckedChange={(v) => !isSurfsenseDoc && toggleOne(doc.id, !!v)}
-													disabled={isSurfsenseDoc}
+													checked={selectedIds.has(doc.id)}
+													onCheckedChange={(v) => toggleOne(doc.id, !!v)}
 													aria-label="Select row"
 												/>
 											</TableCell>