feat(storage): nullish set deletes entry instead of storing dead weight

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>

Author: pi0

AGENTS.md

@@ -37,6 +37,7 @@ src/
 ### Storage (`storage.ts`)
 
 - `StorageInterface` — minimal `get`/`set` with optional TTL
+- Setting a nullish value (`null`/`undefined`) via `set` deletes the entry instead of storing dead weight
 - `createMemoryStorage()` — in-memory Map-based implementation with TTL expiry
 - `useStorage()` / `setStorage()` — global singleton, lazy-inits to memory storage
 

README.md

@@ -118,6 +118,11 @@ const redisStorage: StorageInterface = {
  return JSON.parse(await redis.get(key));
   },
  set: async (key, value, opts) => {
+ // Setting null/undefined deletes the entry (used for cache invalidation)
+ if (value === null || value === undefined) {
+ await redis.del(key);
+ return;
+    }
  await redis.set(key, JSON.stringify(value), opts?.ttl ? { EX: opts.ttl } : undefined);
   },
 };

src/storage.ts

@@ -19,6 +19,10 @@ export function createMemoryStorage(): StorageInterface {
  return entry.value as any;
  },
  set(key, value, opts) {
+ if (value === null || value === undefined) {
+ map.delete(key);
+ return;
+ }
  map.set(key, {
         value,
  expires: opts?.ttl ? Date.now() + opts.ttl * 1000 : undefined,

test/index.test.ts

@@ -518,6 +518,28 @@ describe("storage", () => {
  setStorage(custom);
  expect(useStorage()).toBe(custom);
  });
+
+ it("set with null deletes the entry", () => {
+ const storage = createMemoryStorage();
+ storage.set("key", "hello");
+ expect(storage.get("key")).toBe("hello");
+ storage.set("key", null);
+ expect(storage.get("key")).toBeNull();
+ });
+
+ it("set with undefined deletes the entry", () => {
+ const storage = createMemoryStorage();
+ storage.set("key", "hello");
+ expect(storage.get("key")).toBe("hello");
+ storage.set("key", undefined);
+ expect(storage.get("key")).toBeNull();
+ });
+
+ it("set null on nonexistent key is a no-op", () => {
+ const storage = createMemoryStorage();
+ storage.set("nonexistent", null);
+ expect(storage.get("nonexistent")).toBeNull();
+ });
 });
 
 describe("defineCachedHandler", () => {