@posthog/core 1.40.1 → 1.41.0

package/src/utils/bucketed-rate-limiter.ts

@@ -1,9 +1,38 @@
+import type { ExceptionRateLimiterConfig } from '@posthog/types'
 import { Logger } from '../types'
 import { clampToRange } from './number-utils'
 
 type Bucket = { tokens: number; lastAccess: number }
 const ONE_DAY_IN_MS = 86400000
 
+export const DEFAULT_EXCEPTION_RATE_LIMITER_REFILL_RATE = 1
+export const DEFAULT_EXCEPTION_RATE_LIMITER_BUCKET_SIZE = 10
+
+/**
+ * Resolves the error tracking rate limiter's `refillRate` and `bucketSize` from SDK config,
+ * applying the shared defaults. The deprecated double-underscore options are honoured as a
+ * fallback so existing browser SDK configs keep working after the rename.
+ */
+export function resolveExceptionRateLimiterConfig(
+  config: ExceptionRateLimiterConfig & {
+    /** @deprecated use exceptionRateLimiterRefillRate */
+    __exceptionRateLimiterRefillRate?: number
+    /** @deprecated use exceptionRateLimiterBucketSize */
+    __exceptionRateLimiterBucketSize?: number
+  } = {}
+): { refillRate: number; bucketSize: number } {
+  return {
+    refillRate:
+      config.exceptionRateLimiterRefillRate ??
+      config.__exceptionRateLimiterRefillRate ??
+      DEFAULT_EXCEPTION_RATE_LIMITER_REFILL_RATE,
+    bucketSize:
+      config.exceptionRateLimiterBucketSize ??
+      config.__exceptionRateLimiterBucketSize ??
+      DEFAULT_EXCEPTION_RATE_LIMITER_BUCKET_SIZE,
+  }
+}
+
 export class BucketedRateLimiter<T extends string | number> {
   private _bucketSize: number
   private _refillRate: number

package/src/utils/type-utils.ts

@@ -6,7 +6,6 @@ import {
 } from '../types'
 import { includes } from './string-utils'
 
-// eslint-disable-next-line posthog-js/no-direct-array-check
 const nativeIsArray = Array.isArray
 const ObjProto = Object.prototype
 export const hasOwnProperty = ObjProto.hasOwnProperty
@@ -22,7 +21,6 @@ export const isArray =
 // fails on only one very rare and deliberate custom object:
 // let bomb = { toString : undefined, valueOf: function(o) { return "function BOMBA!"; }};
 export const isFunction = (x: unknown): x is (...args: any[]) => any => {
-  // eslint-disable-next-line posthog-js/no-direct-function-check
   return typeof x === 'function'
 }
 
@@ -53,9 +51,7 @@ export const isString = (x: unknown): x is string => {
 }
 
 export const isEmptyString = (x: unknown): boolean => isString(x) && x.trim().length === 0
-
 export const isNull = (x: unknown): x is null => {
-  // eslint-disable-next-line posthog-js/no-direct-null-check
   return x === null
 }
 
@@ -66,7 +62,6 @@ export const isNull = (x: unknown): x is null => {
 export const isNullish = (x: unknown): x is null | undefined => isUndefined(x) || isNull(x)
 
 export const isNumber = (x: unknown): x is number => {
-  // eslint-disable-next-line posthog-js/no-direct-number-check
   // x !== x is true only for NaN (ES5-compatible NaN check)
   return toString.call(x) == '[object Number]' && x === x
 }
@@ -76,7 +71,6 @@ export const isPositiveNumber = (value: unknown): value is number => {
 }
 
 export const isBoolean = (x: unknown): x is boolean => {
-  // eslint-disable-next-line posthog-js/no-direct-boolean-check
   return toString.call(x) === '[object Boolean]'
 }