fix: null cache key

Author: Divv1524

lib/cache.test.ts

@@ -447,6 +447,17 @@ describe('TTLCache', () => {
       cache.destroy();
     });
 
+    it('throws TypeError when setting a null key', () => {
+      const cache = new TTLCache<string>();
+
+      expect(() => {
+        cache.set(null as unknown as string, 'value', 60_000);
+      }).toThrow(TypeError);
+      expect(cache.size()).toBe(0);
+
+      cache.destroy();
+    });
+
     it('throws RangeError when ttlMs is 0 or negative', () => {
       const cache = new TTLCache<string>();
       expect(() => cache.set('key', 'value', 0)).toThrow(RangeError);

lib/cache.ts

@@ -19,6 +19,12 @@ export class TTLCache<T> {
   private cleanupInterval: ReturnType<typeof setInterval> | null = null;
   private readonly maxSize?: number;
 
+  private static assertValidKey(key: unknown): asserts key is string {
+    if (typeof key !== 'string') {
+      throw new TypeError('Cache key must be a string');
+    }
+  }
+
   /**
    * Creates a new TTL cache instance.
    *
@@ -62,6 +68,8 @@ export class TTLCache<T> {
    * const user = cache.get("user:1");
    */
   get(key: string): T | null {
+    TTLCache.assertValidKey(key);
+
     const hit = this.store.get(key);
     if (!hit) return null;
 
@@ -87,6 +95,8 @@ export class TTLCache<T> {
    * }
    */
   has(key: string): boolean {
+    TTLCache.assertValidKey(key);
+
     const hit = this.store.get(key);
     if (!hit) return false;
 
@@ -109,6 +119,8 @@ export class TTLCache<T> {
    * cache.delete("user:1");
    */
   delete(key: string): boolean {
+    TTLCache.assertValidKey(key);
+
     return this.store.delete(key);
   }
 
@@ -134,13 +146,16 @@ export class TTLCache<T> {
    * @returns `true` if the entry existed and was updated, `false` if missing or expired.
    */
   update(key: string, value: T): boolean {
+    TTLCache.assertValidKey(key);
+
     const hit = this.store.get(key);
     if (!hit || Date.now() > hit.expiresAt) return false;
     hit.value = value;
     return true;
   }
 
   set(key: string, value: T, ttlMs: number): void {
+    TTLCache.assertValidKey(key);
     if (key === '') throw new Error('Cache key cannot be empty');
     if (ttlMs <= 0) throw new RangeError(`ttlMs must be positive, got ${ttlMs}`);