fix: address command menu review

packages/spotlight/src/modes/external/search-engines-search-group.tsx

@@ -429,7 +429,9 @@ const createSearchEnginesSearchGroup = ({ minimumLength, showEmptyHint, source }
       );
 
       const isLoading = enginesQuery.isLoading || ddgQuery.isLoading;
-      const isError = enginesQuery.isError || ddgQuery.isError;
+      const isError =
+        (enginesQuery.isError && remoteQuery.enabled) ||
+        (ddgQuery.isError && remoteQuery.enabled && ddgBangs && remoteQuery.query.length >= 1);
 
       const engineOptions = (enginesQuery.data ?? []).map(
         (engine): ExternalOption => ({
@@ -454,9 +456,10 @@ const createSearchEnginesSearchGroup = ({ minimumLength, showEmptyHint, source }
         const matchedEngine = (enginesQuery.data ?? []).find((engine) => engine.short === bangToken);
         const matchedDdg = (ddgQuery.data ?? []).find((bang) => bang.t === bangToken);
 
-        const label = matchedEngine?.name ?? matchedDdg?.s;
-        const iconUrl = matchedEngine?.iconUrl;
-        const urlTemplate = matchedEngine?.type === "generic" ? matchedEngine.urlTemplate : matchedDdg?.u;
+        const genericEngine = matchedEngine?.type === "generic" ? matchedEngine : undefined;
+        const label = genericEngine?.name ?? matchedDdg?.s;
+        const iconUrl = genericEngine?.iconUrl;
+        const urlTemplate = genericEngine?.urlTemplate ?? matchedDdg?.u;
 
         if (label && urlTemplate) {
           if (searchText.trim().length > 0) {

packages/spotlight/src/modes/home/home-search-engine-group.tsx

@@ -82,7 +82,9 @@ export const homeSearchEngineGroup = createGroup<GroupItem>({
     return {
       isLoading:
         defaultSearchEngineQuery.isLoading || (resultQuery.isLoading && fromIntegrationEnabled) || status === "loading",
-      isError: defaultSearchEngineQuery.isError || (resultQuery.isError && fromIntegrationEnabled),
+      isError:
+        (defaultSearchEngineQuery.isError && status !== "loading" && remoteQuery.enabled) ||
+        (resultQuery.isError && fromIntegrationEnabled),
       data: remoteQuery.enabled ? createDefaultSearchEntries(defaultSearchEngine, results, session, query, t) : [],
     };
   },

packages/spotlight/src/modes/media/media-fallback-group.tsx

@@ -56,7 +56,7 @@ export const mediaFallbackGroup = createGroup<MediaFallbackOption>({
           {
             key: "media-fallback",
             label: t("search.modePicker.media.fallback", { query: remoteQuery.query }),
-            description: t("search.mode.media.action.search.disabled.noIntegration"),
+            description: t("search.mode.media.action.search.disabled.signInRequired"),
             query: remoteQuery.query,
             unavailable: true,
           },

packages/spotlight/src/modes/media/media-request-search-group.tsx

@@ -142,7 +142,7 @@ export const mediaRequestSearchGroup = createGroup<MediaRequestSearchOption>({
             key: "disabled",
             kind: "disabled",
             label: tMedia("action.search.label"),
-            description: tMedia("action.search.disabled.noIntegration"),
+            description: tMedia("action.search.disabled.signInRequired"),
             unavailable: true,
           },
         ],

packages/translation/src/lang/en.json

@@ -7048,6 +7048,7 @@
             "description": "Start typing to search your media request integrations",
             "disabled": {
               "noIntegration": "Add an Overseerr, Jellyseerr, or Seerr integration to enable media search.",
+              "signInRequired": "Sign in to search media request integrations.",
               "scoped": "This widget does not have an accessible media request integration."
             }
           }