From 2154811ffb140ae0b656f93d5e02775ae88d8915 Mon Sep 17 00:00:00 2001 From: Josh Voyles <120749218+Josh-Voyles@users.noreply.github.com> Date: Sun, 3 May 2026 09:39:27 -0400 Subject: [PATCH] removed possible introduced HA Redis bug; improved comment --- server/db/sqlite/driver.ts | 2 ++ server/private/routers/ws/ws.ts | 2 -- 2 files changed, 2 insertions(+), 2 deletions(-) diff --git a/server/db/sqlite/driver.ts b/server/db/sqlite/driver.ts index af60ce331..12681f521 100644 --- a/server/db/sqlite/driver.ts +++ b/server/db/sqlite/driver.ts @@ -16,6 +16,8 @@ bootstrapVolume(); * Wraps better-sqlite3 Statement to call `finalize()` immediately after * execution, freeing native sqlite3_stmt memory deterministically instead * of waiting for GC. Fixes steady off-heap growth under load (#2120). + * WARNING: Finalizes after first execution — incompatible with drizzle's + * reusable .prepare() builders. No such usage exists in this codebase. */ function autoFinalizeStatement( stmt: BetterSqlite3.Statement diff --git a/server/private/routers/ws/ws.ts b/server/private/routers/ws/ws.ts index 764127df4..c01ebc9eb 100644 --- a/server/private/routers/ws/ws.ts +++ b/server/private/routers/ws/ws.ts @@ -414,8 +414,6 @@ const removeClient = async ( await redisManager.del( getNodeConnectionsKey(NODE_ID, clientId) ); - // Remove Redis config version key to prevent indefinite accumulation. - await redisManager.del(getConfigVersionKey(clientId)); } catch (error) { logger.error( "Failed to remove client from Redis tracking (cleanup will occur on recovery):",