Don't bail on error when scraping all (#4442)

ui/v2.5/src/components/Tagger/context.tsx

@@ -24,6 +24,7 @@ import { useLocalForage } from "src/hooks/LocalForage";
 import { useToast } from "src/hooks/Toast";
 import { ConfigurationContext } from "src/hooks/Config";
 import { ITaggerSource, SCRAPER_PREFIX, STASH_BOX_PREFIX } from "./constants";
+import { errorToString } from "src/utils";
 
 export interface ITaggerContextState {
   config: ITaggerConfig;
@@ -293,9 +294,14 @@ export const TaggerContext: React.FC = ({ children }) => {
       return;
     }
 
-    const results = await queryScrapeScene(currentSource.sourceInput, sceneID);
     let newResult: ISceneQueryResult;
 
+    try {
+      const results = await queryScrapeScene(
+        currentSource.sourceInput,
+        sceneID
+      );
+
       if (results.error) {
         newResult = { error: results.error.message };
       } else if (results.errors) {
@@ -309,6 +315,9 @@ export const TaggerContext: React.FC = ({ children }) => {
           })),
         };
       }
+    } catch (err: unknown) {
+      newResult = { error: errorToString(err) };
+    }
 
     setSearchResults((current) => {
       return { ...current, [sceneID]: newResult };

ui/v2.5/src/hooks/Toast.tsx

@@ -6,6 +6,7 @@ import React, {
   useMemo,
 } from "react";
 import { Toast } from "react-bootstrap";
+import { errorToString } from "src/utils";
 
 interface IToast {
   header?: string;
@@ -67,16 +68,7 @@ export const useToast = () => {
         });
       },
       error(error: unknown) {
-        let message;
+        const message = errorToString(error);
-        if (error instanceof Error) {
-          message = error.message;
-        }
-        if (!message) {
-          message = String(error);
-        }
-        if (!message) {
-          message = "Unknown error";
-        }
 
         console.error(error);
         addToast({

ui/v2.5/src/utils/errors.ts

@@ -2,3 +2,18 @@ import { ApolloError } from "@apollo/client";
 
 export const apolloError = (error: unknown) =>
   error instanceof ApolloError ? error.message : "";
+
+export function errorToString(error: unknown) {
+  let message;
+  if (error instanceof Error) {
+    message = error.message;
+  }
+  if (!message) {
+    message = String(error);
+  }
+  if (!message) {
+    message = "Unknown error";
+  }
+
+  return message;
+}