Skip to content

Commit 2430f84

Browse files
leshniakclaude
andcommitted
Run prettier on changed files
Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
1 parent 376a6ba commit 2430f84

2 files changed

Lines changed: 13 additions & 13 deletions

File tree

lib/OnyxUtils.ts

Lines changed: 11 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -846,16 +846,20 @@ function retryOperation<TMethod extends RetriableOnyxOperation>(error: Error, on
846846
const delay = getRetryDelay(currentRetryAttempt);
847847

848848
if (isConnectionError) {
849-
Logger.logInfo(`Connection error detected, retrying with backoff (${delay}ms). Error: ${error}. onyxMethod: ${onyxMethod.name}. retryAttempt: ${nextRetryAttempt}/${MAX_STORAGE_OPERATION_RETRY_ATTEMPTS}`);
849+
Logger.logInfo(
850+
`Connection error detected, retrying with backoff (${delay}ms). Error: ${error}. onyxMethod: ${onyxMethod.name}. retryAttempt: ${nextRetryAttempt}/${MAX_STORAGE_OPERATION_RETRY_ATTEMPTS}`,
851+
);
850852
}
851853

852854
// @ts-expect-error No overload matches this call.
853-
return wait(delay).then(() => Promise.resolve(onyxMethod(defaultParams, nextRetryAttempt)).then(() => {
854-
if (!isConnectionError) {
855-
return;
856-
}
857-
Logger.logInfo(`Connection error recovered after backoff on attempt ${nextRetryAttempt}/${MAX_STORAGE_OPERATION_RETRY_ATTEMPTS}. onyxMethod: ${onyxMethod.name}.`);
858-
}));
855+
return wait(delay).then(() =>
856+
Promise.resolve(onyxMethod(defaultParams, nextRetryAttempt)).then(() => {
857+
if (!isConnectionError) {
858+
return;
859+
}
860+
Logger.logInfo(`Connection error recovered after backoff on attempt ${nextRetryAttempt}/${MAX_STORAGE_OPERATION_RETRY_ATTEMPTS}. onyxMethod: ${onyxMethod.name}.`);
861+
}),
862+
);
859863
}
860864

861865
// Find the least recently accessed evictable key that we can remove

tests/unit/onyxUtilsTest.ts

Lines changed: 2 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -538,9 +538,7 @@ describe('OnyxUtils', () => {
538538
await setPromise;
539539

540540
// Filter setTimeout calls to only those from our wait() helper (delay > 0)
541-
const backoffDelays = setTimeoutSpy.mock.calls
542-
.map((call) => call[1])
543-
.filter((delay): delay is number => typeof delay === 'number' && delay > 0);
541+
const backoffDelays = setTimeoutSpy.mock.calls.map((call) => call[1]).filter((delay): delay is number => typeof delay === 'number' && delay > 0);
544542

545543
// Should have 5 backoff delays (one before each of the 5 retries, attempts 0-4)
546544
// The 6th call to retryOperation (attempt 5) hits the MAX check and resolves without waiting
@@ -619,9 +617,7 @@ describe('OnyxUtils', () => {
619617
await Onyx.set(ONYXKEYS.TEST_KEY, {test: 'data'});
620618

621619
// Capacity errors should not trigger any backoff delays (delay > 0)
622-
const backoffDelays = setTimeoutSpy.mock.calls
623-
.map((call) => call[1])
624-
.filter((delay): delay is number => typeof delay === 'number' && delay > 0);
620+
const backoffDelays = setTimeoutSpy.mock.calls.map((call) => call[1]).filter((delay): delay is number => typeof delay === 'number' && delay > 0);
625621

626622
expect(backoffDelays).toHaveLength(0);
627623
setTimeoutSpy.mockRestore();

0 commit comments

Comments
 (0)