CodeCow/stack
1
0
Fork 0
mirror of https://github.com/stack-auth/stack.git synced 2026-06-30 21:01:54 +08:00
Code Issues Actions 44 Packages Projects Releases Wiki Activity

fix: use bounds-checked as T instead of ?? throwErr for nullish-safe mapWithConcurrency

Browse Source 
Co-Authored-By: Devin AI <158243242+devin-ai-integration[bot]@users.noreply.github.com>
This commit is contained in:
armaan 2026-06-23 21:08:29 +00:00
parent 578d8730bc
commit dad5ceddb4
1 changed files with 3 additions and 2 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

5
packages/shared/src/utils/promises.tsx
View File

@ -1,6 +1,6 @@
import { KnownError } from "..";
import { getProcessEnv } from "./env";
import { HexclaveAssertionError, captureError, concatStacktraces, errorToNiceString, throwErr } from "./errors";
import { HexclaveAssertionError, captureError, concatStacktraces, errorToNiceString } from "./errors";
import { DependenciesMap } from "./maps";
import { Result } from "./results";
import { traceSpan } from "./telemetry";
@ -461,7 +461,8 @@ export async function mapWithConcurrency<T, R>(
const index = nextIndex++;
if (index >= items.length) return;
try {
results[index] = await fn(items[index] ?? throwErr(`mapWithConcurrency: index ${index} out of bounds for items of length ${items.length}`), index);
// Bounds-checked above; `?? throwErr(…)` is unsuitable because T may legitimately be null/undefined
results[index] = await fn(items[index] as T, index);
} catch (error) {
aborted = true;
throw error;