remove clamp

Author: arcoraven

src/tests/math.test.ts

@@ -1,5 +1,5 @@
 import { describe, expect, it } from "vitest";
-import { clamp, getPercentile } from "../utils/math";
+import { getPercentile } from "../utils/math";
 
 describe("getPercentile", () => {
   it("should correctly calculate the p50 (median) of a sorted array", () => {
@@ -27,25 +27,3 @@ describe("getPercentile", () => {
     expect(getPercentile(numbers, 50)).toBe(0);
   });
 });
-
-describe("clamp", () => {
-  it("clamps the value correctly below the minimum", () => {
-    expect(clamp(0, { min: 2, max: 10 })).toBe(2);
-  });
-
-  it("clamps the value correctly at the minimum", () => {
-    expect(clamp(2, { min: 2, max: 10 })).toBe(2);
-  });
-
-  it("returns the value when within bounds", () => {
-    expect(clamp(3, { min: 2, max: 10 })).toBe(3);
-  });
-
-  it("clamps the value correctly at the maximum", () => {
-    expect(clamp(10, { min: 2, max: 10 })).toBe(10);
-  });
-
-  it("clamps the value correctly above the maximum", () => {
-    expect(clamp(12, { min: 2, max: 10 })).toBe(10);
-  });
-});

src/utils/math.ts

@@ -7,8 +7,3 @@ export const getPercentile = (arr: number[], percentile: number): number => {
   const index = Math.floor((percentile / 100) * (arr.length - 1));
   return arr[index];
 };
-
-export const clamp = (
-  val: number,
-  { min, max }: { min: number; max: number },
-) => Math.max(min, Math.min(val, max));

src/worker/tasks/sendTransactionWorker.ts

@@ -39,7 +39,6 @@ import {
   isReplacementGasFeeTooLow,
   wrapError,
 } from "../../utils/error";
-import { clamp } from "../../utils/math";
 import { getChecksumAddress } from "../../utils/primitiveTypes";
 import { recordMetrics } from "../../utils/prometheus";
 import { redis } from "../../utils/redis/redis";
@@ -595,7 +594,7 @@ export const _updateGasFees = (
     return populatedTransaction;
   }
 
-  const multiplier = BigInt(clamp(resendCount * 2, { min: 2, max: 10 }));
+  const multiplier = BigInt(Math.min(10, resendCount * 2));
 
   const updated = { ...populatedTransaction };
   if (updated.gasPrice) {