From 64fed8671e065d696f5c8e0cae10bec277fb8685 Mon Sep 17 00:00:00 2001
From: patrick <patrickjaroszewski@protonmail.com>
Date: Mon, 17 Aug 2026 17:57:46 -0400
Subject: [PATCH] fix: clear the last type errors

asChild is a bits-ui v1 prop the icon picker no longer needs -- it already
passes a child snippet. The drop zone set both webkitdirectory and a bare
directory attribute, which no browser implements. The library view seeds its
collection state once by design, so it says so with untrack.
---
 .../components/ui/file-drop-zone/file-drop-zone.svelte    | 1 -
 .../src/lib/components/ui/icon-picker/icon-picker.svelte  | 2 +-
 .../(library)/library/[libraryId]/view/+page.svelte       | 8 +++++++-
 3 files changed, 8 insertions(+), 3 deletions(-)

diff --git a/frontend/src/lib/components/ui/file-drop-zone/file-drop-zone.svelte b/frontend/src/lib/components/ui/file-drop-zone/file-drop-zone.svelte
index f046849..a3de8ed 100644
--- a/frontend/src/lib/components/ui/file-drop-zone/file-drop-zone.svelte
+++ b/frontend/src/lib/components/ui/file-drop-zone/file-drop-zone.svelte
@@ -185,7 +185,6 @@
 		type="file"
 		onchange={change}
 		webkitdirectory={directory}
-		{directory}
 		class="hidden"
 	/>
 </label>
diff --git a/frontend/src/lib/components/ui/icon-picker/icon-picker.svelte b/frontend/src/lib/components/ui/icon-picker/icon-picker.svelte
index eaf9ca1..096d84a 100644
--- a/frontend/src/lib/components/ui/icon-picker/icon-picker.svelte
+++ b/frontend/src/lib/components/ui/icon-picker/icon-picker.svelte
@@ -37,7 +37,7 @@
 </script>
 
 <Popover.Root bind:open>
-	<Popover.Trigger asChild>
+	<Popover.Trigger>
 		{#snippet child({ props })}
 			<Button variant="outline" size="icon" class="h-9 w-9" {...props}>
 				<SelectedIconComponent class="size-4" />
diff --git a/frontend/src/routes/(root)/(library)/library/[libraryId]/view/+page.svelte b/frontend/src/routes/(root)/(library)/library/[libraryId]/view/+page.svelte
index a54a1e6..8161c5c 100644
--- a/frontend/src/routes/(root)/(library)/library/[libraryId]/view/+page.svelte
+++ b/frontend/src/routes/(root)/(library)/library/[libraryId]/view/+page.svelte
@@ -20,7 +20,13 @@
 		};
 	} = $props();
 
-	const { books, ...filterData } = data;
+	// Seeds the collection state once, deliberately: it owns the list from here on, and the
+	// effect below feeds it later navigations. untrack says so, rather than reading props
+	// outside a closure and being warned about capturing only the initial value.
+	const { books, filterData } = untrack(() => {
+		const { books: initialBooks, ...rest } = data;
+		return { books: initialBooks, filterData: rest };
+	});
 
 	const bookOps = getBookOperationsState();
 	const bookCollection = setBookCollectionState(bookOps, books, filterData);