From 4f6da9a6a5fe74abff9c3331590ecbffc4e89dc8 Mon Sep 17 00:00:00 2001
From: Diego Imbert <diego@windmill.dev>
Date: Tue, 17 Mar 2026 12:26:16 +0100
Subject: [PATCH] fix: fallback to client-side filtering when queryHubScripts
 is unavailable

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
---
 .../flows/pickers/PickHubScript.svelte        | 43 +++++++++++++------
 1 file changed, 31 insertions(+), 12 deletions(-)

diff --git a/frontend/src/lib/components/flows/pickers/PickHubScript.svelte b/frontend/src/lib/components/flows/pickers/PickHubScript.svelte
index a1ca1a167c..ef2b752e4b 100644
--- a/frontend/src/lib/components/flows/pickers/PickHubScript.svelte
+++ b/frontend/src/lib/components/flows/pickers/PickHubScript.svelte
@@ -84,21 +84,40 @@
 			startTs = ts
 			await new Promise((r) => setTimeout(r, 100))
 			if (ts < startTs) return
-			const scripts =
-				filter.length > 0
-					? await ScriptService.queryHubScripts({
-							text: `${filter}`,
-							limit: 20,
-							kind: filterKind,
-							app: appFilter
-						})
-					: ((
+			let scripts: any[]
+			if (filter.length > 0) {
+				try {
+					scripts = await ScriptService.queryHubScripts({
+						text: `${filter}`,
+						limit: 20,
+						kind: filterKind,
+						app: appFilter
+					})
+				} catch {
+					// Fallback: load top scripts and filter client-side
+					const top =
+						(
 							await ScriptService.getTopHubScripts({
-								limit: 20,
+								limit: 40,
 								app: appFilter,
 								kind: filterKind
 							})
-						).asks ?? [])
+						).asks ?? []
+					const lc = filter.toLowerCase()
+					scripts = top.filter(
+						(x: any) => x.summary?.toLowerCase().includes(lc) || x.app?.toLowerCase().includes(lc)
+					)
+				}
+			} else {
+				scripts =
+					(
+						await ScriptService.getTopHubScripts({
+							limit: 20,
+							app: appFilter,
+							kind: filterKind
+						})
+					).asks ?? []
+			}
 			if (ts === startTs) {
 				loading = false
 			}
@@ -112,7 +131,7 @@
 					kind: HubScriptKind
 				}) => ({
 					...x,
-					path: `hub/${x.version_id}/${x.app}/${x.summary.toLowerCase().replaceAll(/\s+/g, '_')}`,
+					path: `hub/${x.version_id}/${x.app}/${(x.summary ?? '').toLowerCase().replaceAll(/\s+/g, '_')}`,
 					summary: `${x.summary} (${x.app})`
 				})
 			)