refactor(telemetry): enhance flushToClearcut method with retry logic and early return for empty events (#1601)

Co-authored-by: Scott Densmore <[email protected]>

2 files changed, 5 insertions, 6 deletions

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;