From 71e574ea606f0601546f4c9c67088f9401d10caa Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?=C3=81lvaro=20Justen=20=28=40turicas=29?=
 <alvarojusten@gmail.com>
Date: Mon, 27 Apr 2026 20:14:20 -0300
Subject: [PATCH] fix(ui): correct Web Loader config meaning

The Web Loader admin setting was labeled "Concurrent Requests" in the
UI, but the underlying value is passed as `requests_per_second` to
`get_web_loader` and consumed by `RateLimitMixin` as a sequential rate
limiter. The loop is sequential in every loader; this is not concurrency
control. The label was renamed to match the actual semantics and a
tooltip was added to describe the behavior.

Note: the config key and env var (`WEB_LOADER_CONCURRENT_REQUESTS`) are
preserved for backward compatibility (only the label was changed).
---
 src/lib/components/admin/Settings/WebSearch.svelte | 12 +++++++++---
 1 file changed, 9 insertions(+), 3 deletions(-)

diff --git a/src/lib/components/admin/Settings/WebSearch.svelte b/src/lib/components/admin/Settings/WebSearch.svelte
index 7ec12fe7e8..b00ea0356a 100644
--- a/src/lib/components/admin/Settings/WebSearch.svelte
+++ b/src/lib/components/admin/Settings/WebSearch.svelte
@@ -1141,12 +1141,18 @@
 
 					<div class="mb-2.5 w-full">
 						<div class=" self-center text-xs font-medium mb-1">
-							{$i18n.t('Concurrent Requests')}
+							<Tooltip
+								content={$i18n.t(
+									'Cap the rate at which the web loader fetches URLs. 0 = unlimited. Lower this to respect rate limits on paid extraction APIs (e.g. Firecrawl, Tavily) or to reduce load on source websites.'
+								)}
+								placement="top-start"
+							>
+								{$i18n.t('Requests per second')}
+							</Tooltip>
 						</div>
-
 						<input
 							class="w-full rounded-lg py-2 px-4 text-sm bg-gray-50 dark:text-gray-300 dark:bg-gray-850 outline-hidden"
-							placeholder={$i18n.t('Concurrent Requests')}
+							placeholder={$i18n.t('Requests per second')}
 							bind:value={webConfig.WEB_LOADER_CONCURRENT_REQUESTS}
 							required
 						/>