@grafana/faro-react-native 1.3.1 → 1.4.0

package/src/navigation/utils.ts

@@ -4,6 +4,8 @@ import { faro } from '@grafana/faro-core';
 
 import { setCurrentScreen } from '../metas/screen';
 
+export const EVENT_NAVIGATION = 'navigation';
+
 /**
  * Gets the currently active route from navigation state
  */
@@ -57,7 +59,7 @@ export function onNavigationStateChange(state: NavigationState | undefined): voi
 
     // Optionally push additional attributes if route has params
     if (currentRoute?.params) {
-      faro.api?.pushEvent('navigation', {
+      faro.api?.pushEvent(EVENT_NAVIGATION, {
         screen: screenName,
         params: JSON.stringify(currentRoute.params),
       });

package/src/transports/fetch/index.ts

@@ -1,2 +1,7 @@
 export { FetchTransport } from './transport';
-export type { FetchTransportOptions, FetchTransportRequestOptions } from './types';
+export type {
+  FetchTransportOptions,
+  FetchTransportRequestOptions,
+  FetchTransportSendOutcome,
+  FetchTransportSendResult,
+} from './types';

package/src/transports/fetch/transport.test.ts

@@ -101,6 +101,81 @@ describe('FetchTransport', () => {
     );
   });
 
+  it('uses the live session for the header while preserving the payload session', async () => {
+    const transport = new FetchTransport({
+      url: 'http://example.com/collect',
+    });
+    const waitForSession = jest
+      .spyOn(transport as unknown as { waitForSession: () => Promise<void> }, 'waitForSession')
+      .mockResolvedValue();
+
+    transport.metas.value = { session: { id: 'new-live-session' } };
+    transport.internalLogger = mockInternalLogger;
+
+    await transport.send([item]);
+
+    expect(waitForSession).toHaveBeenCalledTimes(1);
+    expect(fetch).toHaveBeenCalledWith(
+      'http://example.com/collect',
+      expect.objectContaining({
+        body: JSON.stringify(getTransportBody([item])),
+        headers: {
+          'Content-Type': 'application/json',
+          'x-faro-session-id': 'new-live-session',
+        },
+        method: 'POST',
+      })
+    );
+  });
+
+  it('returns whether the collector accepted the payload', async () => {
+    const transport = new FetchTransport({
+      url: 'http://example.com/collect',
+    });
+
+    transport.metas.value = { session: { id: mockSessionId } };
+    transport.internalLogger = mockInternalLogger;
+
+    await expect(transport.sendWithResult([item])).resolves.toEqual({
+      outcome: 'accepted',
+      status: 202,
+    });
+
+    fetch.mockImplementationOnce(() =>
+      Promise.resolve({
+        status: 500,
+        text: () => Promise.resolve('server error'),
+        headers: {
+          get: () => null,
+        },
+      })
+    );
+
+    await expect(transport.sendWithResult([item])).resolves.toEqual({
+      outcome: 'rejected',
+      status: 500,
+    });
+
+    fetch.mockImplementationOnce(() => Promise.reject(new Error('Network error')));
+
+    await expect(transport.sendWithResult([item])).resolves.toEqual({
+      outcome: 'failed',
+    });
+  });
+
+  it('skips empty batches without sending a request', async () => {
+    const transport = new FetchTransport({
+      url: 'http://example.com/collect',
+    });
+
+    transport.internalLogger = mockInternalLogger;
+
+    await expect(transport.sendWithResult([])).resolves.toEqual({
+      outcome: 'skipped',
+    });
+    expect(fetch).not.toHaveBeenCalled();
+  });
+
   it('will send event with API key if provided', async () => {
     const transport = new FetchTransport({
       url: 'http://example.com/collect',
@@ -353,16 +428,25 @@ describe('FetchTransport', () => {
     const transport = new FetchTransport({
       url: 'http://example.com/collect',
     });
+    const waitForSession = jest
+      .spyOn(transport as unknown as { waitForSession: () => Promise<void> }, 'waitForSession')
+      .mockResolvedValue();
 
     transport.metas.value = {};
     transport.internalLogger = mockInternalLogger;
 
-    await transport.send([item]);
+    const itemWithoutSession = {
+      ...item,
+      meta: {},
+    };
+
+    await transport.send([itemWithoutSession]);
 
+    expect(waitForSession).toHaveBeenCalledTimes(1);
     expect(fetch).toHaveBeenCalledWith(
       'http://example.com/collect',
       expect.objectContaining({
-        body: JSON.stringify(getTransportBody([item])),
+        body: JSON.stringify(getTransportBody([itemWithoutSession])),
         headers: {
           'Content-Type': 'application/json',
         },

package/src/transports/fetch/transport.ts

@@ -1,7 +1,7 @@
 import { BaseTransport, createPromiseBuffer, getTransportBody, VERSION } from '@grafana/faro-core';
 import type { Patterns, PromiseBuffer, TransportItem } from '@grafana/faro-core';
 
-import type { FetchTransportOptions } from './types';
+import type { FetchTransportOptions, FetchTransportSendResult } from './types';
 
 const DEFAULT_BUFFER_SIZE = 30;
 const DEFAULT_CONCURRENCY = 5;
@@ -10,8 +10,6 @@ const MAX_CONSECUTIVE_FAILURES = 3;
 const FAILURE_BACKOFF_MS = 30000; // 30 seconds
 
 const TOO_MANY_REQUESTS = 429;
-const ACCEPTED = 202;
-
 export class FetchTransport extends BaseTransport {
   readonly name = '@grafana/faro-react-native:transport-fetch';
   readonly version = VERSION;
@@ -111,8 +109,21 @@ export class FetchTransport extends BaseTransport {
   }
 
   async send(items: TransportItem[]): Promise<void> {
-    // DEBUG: Log at the very start of send
-    this.logDebug(`FetchTransport.send() called with ${items.length} items`);
+    await this.sendWithResult(items);
+  }
+
+  /**
+   * Sends a batch and reports whether the collector accepted it.
+   * The request header uses the live session for collector accounting. Payload
+   * metadata remains independent so delayed signals can retain their original
+   * session.
+   */
+  async sendWithResult(items: TransportItem[]): Promise<FetchTransportSendResult> {
+    this.logDebug(`FetchTransport.sendWithResult() called with ${items.length} items`);
+
+    if (items.length === 0) {
+      return { outcome: 'skipped' };
+    }
 
     try {
       const now = new Date(this.getNow());
@@ -120,14 +131,15 @@ export class FetchTransport extends BaseTransport {
       // Check if we're in backoff period
       if (this.disabledUntil > now) {
         this.logDebug(`FetchTransport: in backoff period until ${this.disabledUntil}`);
-        return Promise.resolve();
+        return { outcome: 'skipped' };
       }
 
-      // Wait for session to be ready before sending
-      // This prevents 400 errors from missing X-Faro-Session-Id header
+      // The collector uses the header for session validation and accounting,
+      // independently of the session carried by each payload.
       await this.waitForSession();
+      const sessionId = this.metas.value.session?.id;
 
-      await this.promiseBuffer.add(() => {
+      const response = await this.promiseBuffer.add(() => {
         const transportBody = getTransportBody(items);
         const body = JSON.stringify(transportBody);
 
@@ -142,12 +154,6 @@ export class FetchTransport extends BaseTransport {
 
         const { headers, ...restOfRequestOptions } = requestOptions ?? {};
 
-        let sessionId;
-        const sessionMeta = this.metas.value.session;
-        if (sessionMeta != null) {
-          sessionId = sessionMeta.id;
-        }
-
         // DEBUG: Log fetch attempt
         this.logDebug(`FetchTransport: sending ${items.length} items to ${url}`);
 
@@ -186,7 +192,7 @@ export class FetchTransport extends BaseTransport {
             }
 
             // Log non-success responses for debugging
-            if (response.status !== ACCEPTED && response.status !== 200) {
+            if (response.status < 200 || response.status >= 300) {
               const text = await response.text().catch(() => '');
               this.logDebug(`FetchTransport: non-success response: ${response.status} ${text.slice(0, 200)}`);
             }
@@ -211,11 +217,27 @@ export class FetchTransport extends BaseTransport {
 
             // Do NOT log errors to console - this causes infinite loops in React Native
             // when the DevTools console override intercepts even unpatchedConsole calls
+            return undefined;
           });
       });
+
+      if (!response) {
+        return { outcome: 'failed' };
+      }
+      if (response.status >= 200 && response.status < 300) {
+        return {
+          outcome: 'accepted',
+          status: response.status,
+        };
+      }
+      return {
+        outcome: 'rejected',
+        status: response.status,
+      };
     } catch {
       // Buffer full error - Do NOT log to console as it creates infinite loops
       // The error is typically "Task buffer full" when the device is offline
+      return { outcome: 'failed' };
     }
   }
 

package/src/transports/fetch/types.ts

@@ -23,3 +23,10 @@ export interface FetchTransportOptions {
 }
 
 export type ClockFn = () => number;
+
+export type FetchTransportSendOutcome = 'accepted' | 'rejected' | 'failed' | 'skipped';
+
+export interface FetchTransportSendResult {
+  outcome: FetchTransportSendOutcome;
+  status?: number;
+}

package/src/transports/offline/ConnectivityService.ts

@@ -23,7 +23,7 @@ const DEFAULT_CHECK_INTERVAL_MS = 30000;
  * Uses @react-native-community/netinfo when available, otherwise falls back
  * to basic fetch-based connectivity detection.
  *
- * Implementation follows Flutter SDK's InternetConnectivityService pattern.
+ * Monitors network connectivity for offline transport flush behavior.
  */
 export class DefaultConnectivityService implements ConnectivityService {
   private _isOnline: boolean = true;

package/src/transports/offline/OfflineCache.ts

@@ -16,7 +16,7 @@ const DEFAULT_MAX_CACHE_SIZE = 1000;
  * File-based offline cache implementation using AsyncStorage.
  *
  * Stores cached telemetry payloads as JSONL (one JSON object per line)
- * following the Flutter SDK's offline transport pattern.
+ * Persistent offline telemetry cache backed by AsyncStorage.
  *
  * Uses a mutex pattern for thread-safe file access.
  */
@@ -77,7 +77,7 @@ export class AsyncStorageOfflineCache implements OfflineCache {
 
   /**
    * Execute an operation with a mutex lock to ensure thread-safe access.
-   * Follows Flutter SDK's Completer-based lock pattern.
+   * Async lock to serialize cache read/write operations.
    */
   private async withLock<T>(operation: () => Promise<T>): Promise<T> {
     const currentLock = this.lockPromise;

package/src/transports/offline/transport.ts

@@ -17,8 +17,7 @@ const DEFAULT_MAX_CACHE_DURATION_MS = 3 * 24 * 60 * 60 * 1000;
  * When the device is offline, telemetry is cached to AsyncStorage.
  * When connectivity is restored, cached telemetry is replayed through the wrapped transports.
  *
- * Implementation follows Flutter SDK's OfflineTransport pattern:
- * - Uses AsyncStorage for persistent caching (matching Flutter's SharedPreferences)
+ * Uses AsyncStorage for persistent offline telemetry caching.
  * - Respects maxCacheDuration to skip expired entries
  * - Uses mutex pattern for thread-safe cache access
  * - Excludes itself when replaying to prevent infinite loops
@@ -157,7 +156,7 @@ export class OfflineTransport extends BaseTransport {
 
   /**
    * Replay cached payloads through other transports.
-   * Follows Flutter SDK's _readFromFile pattern.
+   * Reads cached telemetry from AsyncStorage.
    */
   private async replayCachedPayloads(): Promise<void> {
     if (this.isReplaying) {
@@ -207,7 +206,7 @@ export class OfflineTransport extends BaseTransport {
 
   /**
    * Send items to all other registered transports.
-   * Follows Flutter SDK's _sendCachedData pattern.
+   * Sends cached telemetry when connectivity is restored.
    */
   private async sendToOtherTransports(items: TransportItem[]): Promise<boolean> {
     if (this.otherTransports.length === 0) {

package/src/userPersistence/UserPersistence.ts

@@ -15,8 +15,7 @@ const DEFAULT_STORAGE_KEY = 'faro_persisted_user';
  * Stores user information persistently so it can be restored on app restart.
  * This ensures early telemetry events include user identification.
  *
- * Implementation follows Flutter SDK's UserPersistence pattern:
- * - Uses AsyncStorage for persistence (matching Flutter's SharedPreferences)
+ * Uses AsyncStorage for persistent user data across app sessions.
  * - Stores user as JSON
  * - Handles null/cleared users
  *

package/dist/cjs/utils/throttle.js

@@ -1,42 +0,0 @@
-"use strict";
-Object.defineProperty(exports, "__esModule", { value: true });
-exports.throttle = throttle;
-/**
- * Creates a throttled function that only invokes the provided function at most once per
- * every `wait` milliseconds.
- *
- * @param func - The function to throttle
- * @param wait - The number of milliseconds to throttle invocations to
- * @returns A throttled version of the function
- */
-// eslint-disable-next-line @typescript-eslint/no-explicit-any -- Generic utility needs to accept any function signature
-function throttle(func, wait) {
-    var timeout = null;
-    var previous = 0;
-    // eslint-disable-next-line @typescript-eslint/no-explicit-any -- Need to preserve function context
-    return function () {
-        var _this = this;
-        var args = [];
-        for (var _i = 0; _i < arguments.length; _i++) {
-            args[_i] = arguments[_i];
-        }
-        var now = Date.now();
-        var remaining = wait - (now - previous);
-        if (remaining <= 0 || remaining > wait) {
-            if (timeout) {
-                clearTimeout(timeout);
-                timeout = null;
-            }
-            previous = now;
-            func.apply(this, args);
-        }
-        else if (!timeout) {
-            timeout = setTimeout(function () {
-                previous = Date.now();
-                timeout = null;
-                func.apply(_this, args);
-            }, remaining);
-        }
-    };
-}
-//# sourceMappingURL=throttle.js.map

package/dist/cjs/utils/throttle.js.map

@@ -1 +0,0 @@
-{"version":3,"file":"throttle.js","sourceRoot":"","sources":["../../../src/utils/throttle.ts"],"names":[],"mappings":";;AASA,4BAwBC;AAjCD;;;;;;;GAOG;AACH,wHAAwH;AACxH,SAAgB,QAAQ,CAAoC,IAAO,EAAE,IAAY;IAC/E,IAAI,OAAO,GAAyC,IAAI,CAAC;IACzD,IAAI,QAAQ,GAAG,CAAC,CAAC;IAEjB,mGAAmG;IACnG,OAAO;QAAA,iBAkBN;QAlB2B,cAAsB;aAAtB,UAAsB,EAAtB,qBAAsB,EAAtB,IAAsB;YAAtB,yBAAsB;;QAChD,IAAM,GAAG,GAAG,IAAI,CAAC,GAAG,EAAE,CAAC;QACvB,IAAM,SAAS,GAAG,IAAI,GAAG,CAAC,GAAG,GAAG,QAAQ,CAAC,CAAC;QAE1C,IAAI,SAAS,IAAI,CAAC,IAAI,SAAS,GAAG,IAAI,EAAE,CAAC;YACvC,IAAI,OAAO,EAAE,CAAC;gBACZ,YAAY,CAAC,OAAO,CAAC,CAAC;gBACtB,OAAO,GAAG,IAAI,CAAC;YACjB,CAAC;YACD,QAAQ,GAAG,GAAG,CAAC;YACf,IAAI,CAAC,KAAK,CAAC,IAAI,EAAE,IAAI,CAAC,CAAC;QACzB,CAAC;aAAM,IAAI,CAAC,OAAO,EAAE,CAAC;YACpB,OAAO,GAAG,UAAU,CAAC;gBACnB,QAAQ,GAAG,IAAI,CAAC,GAAG,EAAE,CAAC;gBACtB,OAAO,GAAG,IAAI,CAAC;gBACf,IAAI,CAAC,KAAK,CAAC,KAAI,EAAE,IAAI,CAAC,CAAC;YACzB,CAAC,EAAE,SAAS,CAAC,CAAC;QAChB,CAAC;IACH,CAAM,CAAC;AACT,CAAC","sourcesContent":["/**\n * Creates a throttled function that only invokes the provided function at most once per\n * every `wait` milliseconds.\n *\n * @param func - The function to throttle\n * @param wait - The number of milliseconds to throttle invocations to\n * @returns A throttled version of the function\n */\n// eslint-disable-next-line @typescript-eslint/no-explicit-any -- Generic utility needs to accept any function signature\nexport function throttle<T extends (...args: any[]) => any>(func: T, wait: number): T {\n  let timeout: ReturnType<typeof setTimeout> | null = null;\n  let previous = 0;\n\n  // eslint-disable-next-line @typescript-eslint/no-explicit-any -- Need to preserve function context\n  return function (this: any, ...args: Parameters<T>) {\n    const now = Date.now();\n    const remaining = wait - (now - previous);\n\n    if (remaining <= 0 || remaining > wait) {\n      if (timeout) {\n        clearTimeout(timeout);\n        timeout = null;\n      }\n      previous = now;\n      func.apply(this, args);\n    } else if (!timeout) {\n      timeout = setTimeout(() => {\n        previous = Date.now();\n        timeout = null;\n        func.apply(this, args);\n      }, remaining);\n    }\n  } as T;\n}\n"]}

package/dist/esm/utils/throttle.js

@@ -1,34 +0,0 @@
-/**
- * Creates a throttled function that only invokes the provided function at most once per
- * every `wait` milliseconds.
- *
- * @param func - The function to throttle
- * @param wait - The number of milliseconds to throttle invocations to
- * @returns A throttled version of the function
- */
-// eslint-disable-next-line @typescript-eslint/no-explicit-any -- Generic utility needs to accept any function signature
-export function throttle(func, wait) {
-    let timeout = null;
-    let previous = 0;
-    // eslint-disable-next-line @typescript-eslint/no-explicit-any -- Need to preserve function context
-    return function (...args) {
-        const now = Date.now();
-        const remaining = wait - (now - previous);
-        if (remaining <= 0 || remaining > wait) {
-            if (timeout) {
-                clearTimeout(timeout);
-                timeout = null;
-            }
-            previous = now;
-            func.apply(this, args);
-        }
-        else if (!timeout) {
-            timeout = setTimeout(() => {
-                previous = Date.now();
-                timeout = null;
-                func.apply(this, args);
-            }, remaining);
-        }
-    };
-}
-//# sourceMappingURL=throttle.js.map

package/dist/esm/utils/throttle.js.map

@@ -1 +0,0 @@
-{"version":3,"file":"throttle.js","sourceRoot":"","sources":["../../../src/utils/throttle.ts"],"names":[],"mappings":"AAAA;;;;;;;GAOG;AACH,wHAAwH;AACxH,MAAM,UAAU,QAAQ,CAAoC,IAAO,EAAE,IAAY;IAC/E,IAAI,OAAO,GAAyC,IAAI,CAAC;IACzD,IAAI,QAAQ,GAAG,CAAC,CAAC;IAEjB,mGAAmG;IACnG,OAAO,UAAqB,GAAG,IAAmB;QAChD,MAAM,GAAG,GAAG,IAAI,CAAC,GAAG,EAAE,CAAC;QACvB,MAAM,SAAS,GAAG,IAAI,GAAG,CAAC,GAAG,GAAG,QAAQ,CAAC,CAAC;QAE1C,IAAI,SAAS,IAAI,CAAC,IAAI,SAAS,GAAG,IAAI,EAAE,CAAC;YACvC,IAAI,OAAO,EAAE,CAAC;gBACZ,YAAY,CAAC,OAAO,CAAC,CAAC;gBACtB,OAAO,GAAG,IAAI,CAAC;YACjB,CAAC;YACD,QAAQ,GAAG,GAAG,CAAC;YACf,IAAI,CAAC,KAAK,CAAC,IAAI,EAAE,IAAI,CAAC,CAAC;QACzB,CAAC;aAAM,IAAI,CAAC,OAAO,EAAE,CAAC;YACpB,OAAO,GAAG,UAAU,CAAC,GAAG,EAAE;gBACxB,QAAQ,GAAG,IAAI,CAAC,GAAG,EAAE,CAAC;gBACtB,OAAO,GAAG,IAAI,CAAC;gBACf,IAAI,CAAC,KAAK,CAAC,IAAI,EAAE,IAAI,CAAC,CAAC;YACzB,CAAC,EAAE,SAAS,CAAC,CAAC;QAChB,CAAC;IACH,CAAM,CAAC;AACT,CAAC","sourcesContent":["/**\n * Creates a throttled function that only invokes the provided function at most once per\n * every `wait` milliseconds.\n *\n * @param func - The function to throttle\n * @param wait - The number of milliseconds to throttle invocations to\n * @returns A throttled version of the function\n */\n// eslint-disable-next-line @typescript-eslint/no-explicit-any -- Generic utility needs to accept any function signature\nexport function throttle<T extends (...args: any[]) => any>(func: T, wait: number): T {\n  let timeout: ReturnType<typeof setTimeout> | null = null;\n  let previous = 0;\n\n  // eslint-disable-next-line @typescript-eslint/no-explicit-any -- Need to preserve function context\n  return function (this: any, ...args: Parameters<T>) {\n    const now = Date.now();\n    const remaining = wait - (now - previous);\n\n    if (remaining <= 0 || remaining > wait) {\n      if (timeout) {\n        clearTimeout(timeout);\n        timeout = null;\n      }\n      previous = now;\n      func.apply(this, args);\n    } else if (!timeout) {\n      timeout = setTimeout(() => {\n        previous = Date.now();\n        timeout = null;\n        func.apply(this, args);\n      }, remaining);\n    }\n  } as T;\n}\n"]}

package/dist/types/utils/throttle.d.ts

@@ -1,9 +0,0 @@
-/**
- * Creates a throttled function that only invokes the provided function at most once per
- * every `wait` milliseconds.
- *
- * @param func - The function to throttle
- * @param wait - The number of milliseconds to throttle invocations to
- * @returns A throttled version of the function
- */
-export declare function throttle<T extends (...args: any[]) => any>(func: T, wait: number): T;

package/src/utils/throttle.test.ts

@@ -1,141 +0,0 @@
-import { throttle } from './throttle';
-
-describe('throttle', () => {
-  beforeEach(() => {
-    jest.useFakeTimers();
-  });
-
-  afterEach(() => {
-    jest.useRealTimers();
-  });
-
-  it('should call the function immediately on first invocation', () => {
-    const func = jest.fn();
-    const throttled = throttle(func, 1000);
-
-    throttled();
-
-    expect(func).toHaveBeenCalledTimes(1);
-  });
-
-  it('should throttle subsequent calls within the wait period', () => {
-    const func = jest.fn();
-    const throttled = throttle(func, 1000);
-
-    throttled();
-    throttled();
-    throttled();
-
-    expect(func).toHaveBeenCalledTimes(1);
-  });
-
-  it('should allow a call after the wait period has elapsed', () => {
-    const func = jest.fn();
-    const throttled = throttle(func, 1000);
-
-    throttled();
-    expect(func).toHaveBeenCalledTimes(1);
-
-    jest.advanceTimersByTime(1000);
-
-    throttled();
-    expect(func).toHaveBeenCalledTimes(2);
-  });
-
-  it('should schedule a trailing call if called during throttle period', () => {
-    const func = jest.fn();
-    const throttled = throttle(func, 1000);
-
-    throttled('first');
-    expect(func).toHaveBeenCalledTimes(1);
-    expect(func).toHaveBeenCalledWith('first');
-
-    // Call again during throttle period
-    throttled('second');
-    expect(func).toHaveBeenCalledTimes(1);
-
-    // Advance timers to trigger the trailing call
-    jest.advanceTimersByTime(1000);
-    expect(func).toHaveBeenCalledTimes(2);
-    expect(func).toHaveBeenLastCalledWith('second');
-  });
-
-  it('should pass arguments correctly', () => {
-    const func = jest.fn();
-    const throttled = throttle(func, 1000);
-
-    throttled(1, 'test', { foo: 'bar' });
-
-    expect(func).toHaveBeenCalledWith(1, 'test', { foo: 'bar' });
-  });
-
-  it('should preserve context (this)', () => {
-    const obj = {
-      value: 42,
-      method: jest.fn(function (this: any) {
-        return this.value;
-      }),
-    };
-
-    const throttled = throttle(obj.method, 1000);
-    throttled.call(obj);
-
-    expect(obj.method).toHaveBeenCalled();
-  });
-
-  it('should only keep the last call during throttle period', () => {
-    const func = jest.fn();
-    const throttled = throttle(func, 1000);
-
-    throttled('call1');
-    throttled('call2');
-    throttled('call3');
-
-    expect(func).toHaveBeenCalledTimes(1);
-    expect(func).toHaveBeenCalledWith('call1');
-
-    // Advance time to trigger trailing call
-    // The throttle keeps only the first scheduled call, not the last
-    jest.advanceTimersByTime(1000);
-
-    expect(func).toHaveBeenCalledTimes(2);
-    // The trailing call uses the arguments from when timeout was set (call2)
-    expect(func).toHaveBeenLastCalledWith('call2');
-  });
-
-  it('should handle rapid successive calls correctly', () => {
-    const func = jest.fn();
-    const throttled = throttle(func, 100);
-
-    // First call - executes immediately
-    throttled(1);
-    expect(func).toHaveBeenCalledTimes(1);
-
-    // Calls during throttle - schedules timeout with first call's args
-    throttled(2);
-    throttled(3);
-    expect(func).toHaveBeenCalledTimes(1);
-
-    // Advance to trigger trailing call
-    jest.advanceTimersByTime(100);
-    expect(func).toHaveBeenCalledTimes(2);
-    // Uses args from when timeout was first set (2)
-    expect(func).toHaveBeenLastCalledWith(2);
-  });
-
-  it('should execute trailing call after throttle period', () => {
-    const func = jest.fn();
-    const throttled = throttle(func, 1000);
-
-    throttled('first');
-    expect(func).toHaveBeenCalledTimes(1);
-
-    // Call during throttle period - schedules trailing call
-    throttled('second');
-
-    // Advance past the wait period
-    jest.advanceTimersByTime(1000);
-    expect(func).toHaveBeenCalledTimes(2);
-    expect(func).toHaveBeenLastCalledWith('second');
-  });
-});

package/src/utils/throttle.ts

@@ -1,34 +0,0 @@
-/**
- * Creates a throttled function that only invokes the provided function at most once per
- * every `wait` milliseconds.
- *
- * @param func - The function to throttle
- * @param wait - The number of milliseconds to throttle invocations to
- * @returns A throttled version of the function
- */
-// eslint-disable-next-line @typescript-eslint/no-explicit-any -- Generic utility needs to accept any function signature
-export function throttle<T extends (...args: any[]) => any>(func: T, wait: number): T {
-  let timeout: ReturnType<typeof setTimeout> | null = null;
-  let previous = 0;
-
-  // eslint-disable-next-line @typescript-eslint/no-explicit-any -- Need to preserve function context
-  return function (this: any, ...args: Parameters<T>) {
-    const now = Date.now();
-    const remaining = wait - (now - previous);
-
-    if (remaining <= 0 || remaining > wait) {
-      if (timeout) {
-        clearTimeout(timeout);
-        timeout = null;
-      }
-      previous = now;
-      func.apply(this, args);
-    } else if (!timeout) {
-      timeout = setTimeout(() => {
-        previous = Date.now();
-        timeout = null;
-        func.apply(this, args);
-      }, remaining);
-    }
-  } as T;
-}