fix: extract duplicated stripUndefined function to shared utility

Author: johanviberg

src/config/index.ts

@@ -8,6 +8,7 @@ import { createLogger, type Logger } from "../logging";
 import type { ClientConfig, ProxyCheckOptions } from "../types";
 import { DEFAULTS } from "../types/constants";
 import { ClientConfigSchema, ProxyCheckOptionsSchema } from "../types/schemas";
+import { stripUndefined } from "../utils/object";
 
 /**
  * Query parameters interface
@@ -236,18 +237,6 @@ export class ConfigManager {
   }
 }
 
-/**
- * Helper to remove undefined values from an object for exactOptionalPropertyTypes
- */
-function stripUndefined<T extends Record<string, unknown>>(obj: T): T {
-  const result = {} as T;
-  for (const [key, value] of Object.entries(obj)) {
-    if (value !== undefined) {
-      result[key as keyof T] = value as T[keyof T];
-    }
-  }
-  return result;
-}
 
 /**
  * Validate and merge ProxyCheck options

src/services/check.ts

@@ -13,6 +13,7 @@ import { ProxyCheckValidationError } from "../errors";
 import type { AddressCheckResult, CheckResponse, ProxyCheckOptions } from "../types";
 import { API_ENDPOINTS } from "../types/constants";
 import { ProxyCheckOptionsSchema } from "../types/schemas";
+import { stripUndefined } from "../utils/object";
 import { BaseService } from "./base";
 
 /**
@@ -216,21 +217,12 @@ export class CheckService extends BaseService {
   private validateOptions(options: ProxyCheckOptions): ProxyCheckOptions {
     try {
       const parsed = ProxyCheckOptionsSchema.parse(options) as any;
-      return this.stripUndefined(parsed) as ProxyCheckOptions;
+      return stripUndefined(parsed) as ProxyCheckOptions;
     } catch (_error) {
       throw new ProxyCheckValidationError("Invalid options provided", "options", options);
     }
   }
 
-  private stripUndefined<T extends Record<string, unknown>>(obj: T): T {
-    const result = {} as T;
-    for (const [key, value] of Object.entries(obj)) {
-      if (value !== undefined) {
-        result[key as keyof T] = value as T[keyof T];
-      }
-    }
-    return result;
-  }
 
   /**
    * Add blocking logic for single address checks

src/services/listing.ts

@@ -6,6 +6,7 @@ import { ProxyCheckValidationError } from "../errors";
 import type { ListOptions, ListResponse } from "../types";
 import { API_ENDPOINTS } from "../types/constants";
 import { ListOptionsSchema } from "../types/schemas";
+import { stripUndefined } from "../utils/object";
 import { BaseService } from "./base";
 
 /**
@@ -229,19 +230,10 @@ export class ListingService extends BaseService {
   private validateOptions(options: ListOptions): ListOptions {
     try {
       const parsed = ListOptionsSchema.parse(options) as any;
-      return this.stripUndefined(parsed) as ListOptions;
+      return stripUndefined(parsed) as ListOptions;
     } catch (_error) {
       throw new ProxyCheckValidationError("Invalid list options provided", "options", options);
     }
   }
 
-  private stripUndefined<T extends Record<string, unknown>>(obj: T): T {
-    const result = {} as T;
-    for (const [key, value] of Object.entries(obj)) {
-      if (value !== undefined) {
-        result[key as keyof T] = value as T[keyof T];
-      }
-    }
-    return result;
-  }
 }

src/services/rules.ts

@@ -6,6 +6,7 @@ import { ProxyCheckValidationError } from "../errors";
 import type { RuleOptions, RuleResponse } from "../types";
 import { API_ENDPOINTS } from "../types/constants";
 import { RuleOptionsSchema } from "../types/schemas";
+import { stripUndefined } from "../utils/object";
 import { BaseService } from "./base";
 
 /**
@@ -225,19 +226,10 @@ export class RulesService extends BaseService {
   private validateOptions(options: RuleOptions): RuleOptions {
     try {
       const parsed = RuleOptionsSchema.parse(options) as any;
-      return this.stripUndefined(parsed) as RuleOptions;
+      return stripUndefined(parsed) as RuleOptions;
     } catch (_error) {
       throw new ProxyCheckValidationError("Invalid rule options provided", "options", options);
     }
   }
 
-  private stripUndefined<T extends Record<string, unknown>>(obj: T): T {
-    const result = {} as T;
-    for (const [key, value] of Object.entries(obj)) {
-      if (value !== undefined) {
-        result[key as keyof T] = value as T[keyof T];
-      }
-    }
-    return result;
-  }
 }

src/services/stats.ts

@@ -6,6 +6,7 @@ import { ProxyCheckValidationError } from "../errors";
 import type { StatsOptions, StatsResponse } from "../types";
 import { API_ENDPOINTS } from "../types/constants";
 import { StatsOptionsSchema } from "../types/schemas";
+import { stripUndefined } from "../utils/object";
 import { BaseService } from "./base";
 
 /**
@@ -205,19 +206,10 @@ export class StatsService extends BaseService {
   private validateOptions(options: StatsOptions): StatsOptions {
     try {
       const parsed = StatsOptionsSchema.parse(options) as any;
-      return this.stripUndefined(parsed) as StatsOptions;
+      return stripUndefined(parsed) as StatsOptions;
     } catch (_error) {
       throw new ProxyCheckValidationError("Invalid stats options provided", "options", options);
     }
   }
 
-  private stripUndefined<T extends Record<string, unknown>>(obj: T): T {
-    const result = {} as T;
-    for (const [key, value] of Object.entries(obj)) {
-      if (value !== undefined) {
-        result[key as keyof T] = value as T[keyof T];
-      }
-    }
-    return result;
-  }
 }

src/utils/object.ts

@@ -0,0 +1,12 @@
+/**
+ * Helper to remove undefined values from an object for exactOptionalPropertyTypes
+ */
+export function stripUndefined<T extends Record<string, unknown>>(obj: T): T {
+  const result = {} as T;
+  for (const [key, value] of Object.entries(obj)) {
+    if (value !== undefined) {
+      result[key as keyof T] = value as T[keyof T];
+    }
+  }
+  return result;
+}