From a9f04eba2ccbd3ed85e60670d82833dfc0c5233f Mon Sep 17 00:00:00 2001 From: Hyeladi Bassi <56229133+HyeladiBassi@users.noreply.github.com> Date: Sun, 27 Jul 2025 19:18:27 +0100 Subject: refactor(telemetry): enhance flushToClearcut method with retry logic and early return for empty events (#1601) Co-authored-by: Scott Densmore --- packages/core/src/utils/retry.test.ts | 7 +------ packages/core/src/utils/retry.ts | 4 ++++ 2 files changed, 5 insertions(+), 6 deletions(-) (limited to 'packages/core/src/utils') diff --git a/packages/core/src/utils/retry.test.ts b/packages/core/src/utils/retry.test.ts index f84d2004..196e7341 100644 --- a/packages/core/src/utils/retry.test.ts +++ b/packages/core/src/utils/retry.test.ts @@ -6,14 +6,9 @@ /* eslint-disable @typescript-eslint/no-explicit-any */ import { describe, it, expect, vi, beforeEach, afterEach } from 'vitest'; -import { retryWithBackoff } from './retry.js'; +import { retryWithBackoff, HttpError } from './retry.js'; import { setSimulate429 } from './testUtils.js'; -// Define an interface for the error with a status property -interface HttpError extends Error { - status?: number; -} - // Helper to create a mock function that fails a certain number of times const createFailingFunction = ( failures: number, diff --git a/packages/core/src/utils/retry.ts b/packages/core/src/utils/retry.ts index b29bf7df..81300882 100644 --- a/packages/core/src/utils/retry.ts +++ b/packages/core/src/utils/retry.ts @@ -10,6 +10,10 @@ import { isGenericQuotaExceededError, } from './quotaErrorDetection.js'; +export interface HttpError extends Error { + status?: number; +} + export interface RetryOptions { maxAttempts: number; initialDelayMs: number; -- cgit v1.2.3