cleanup(search): add validation to prevent fetching bookmarks without chunk metadata and dataset

Author: skeptrunedev

frontends/search/src/components/BookmarkPopover.tsx

@@ -215,6 +215,7 @@ const BookmarkPopover = (props: BookmarkPopoverProps) => {
   const refetchBookmarks = (curPage: number) => {
     const currentDataset = $dataset?.();
     if (!currentDataset) return;
+    if (!props.chunkMetadata.id) return;
 
     void fetch(`${apiHost}/chunk_group/chunks`, {
       method: "POST",

frontends/search/src/components/EditChunkPageForm.tsx

@@ -84,6 +84,7 @@ export const EditChunkPageForm = (props: SingleChunkPageProps) => {
   createEffect(() => {
     const currentDatasetId = $dataset?.()?.dataset.id;
     if (!currentDatasetId) return;
+    if (!props.chunkId) return;
 
     void fetch(`${apiHost}/chunk_group/chunks`, {
       method: "POST",

frontends/search/src/components/ResultsPage.tsx

@@ -183,6 +183,7 @@ const ResultsPage = (props: ResultsPageProps) => {
   const fetchBookmarks = () => {
     const dataset = $dataset?.();
     if (!dataset) return;
+    if (resultChunks().length === 0) return;
 
     void fetch(`${apiHost}/chunk_group/chunks`, {
       method: "POST",