obsidian-plugin/jamescliffordspratt_macros
1
0
Fork 0
mirror of https://github.com/jamescliffordspratt/macros.git synced 2026-07-22 05:46:53 +00:00
Code Issues Projects Releases Packages Wiki Activity Actions

Bound per-source search time in All tab to keep it responsive

Browse source
This commit is contained in:
James Clifford Spratt 2026-06-28 12:57:40 +01:00
parent 4f414251d6
commit c8c6f1060a
1 changed files with 13 additions and 4 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

17
src/core/search.ts
View file

@ -332,10 +332,19 @@ export async function searchAllSources(
);
}
// Execute searches in parallel. Use allSettled so that one failing source
// (e.g. a network error or outage from a single API) doesn't wipe out the
// results from the others.
const settled = await Promise.allSettled(searchPromises);
// Give each source a time budget so a slow or unresponsive API (e.g. an
// Open Food Facts outage) can't hold up the results from the faster sources.
// Use allSettled so one failing source doesn't wipe out the others.
const SOURCE_TIMEOUT_MS = 8000;
const withTimeout = (p: Promise<UnifiedFoodResult[]>): Promise<UnifiedFoodResult[]> =>
Promise.race([
p,
new Promise<UnifiedFoodResult[]>((resolve) => {
window.setTimeout(() => resolve([]), SOURCE_TIMEOUT_MS);
}),
]);
const settled = await Promise.allSettled(searchPromises.map(withTimeout));
const results = settled
.filter((r): r is PromiseFulfilledResult<UnifiedFoodResult[]> => r.status === 'fulfilled')
.map((r) => r.value);