From e0c802bec7567a60102681d6429c282dff948efd Mon Sep 17 00:00:00 2001 From: ightevenmckane187 <214069160+ightevenmckane187@users.noreply.github.com> Date: Tue, 18 Aug 2026 15:09:49 +0000 Subject: [PATCH] perf(persistence): eliminate redundant intermediate Buffer allocations during save - Pass payload string directly to crypto.createHmac with utf8 encoding - Remove unused payloadBuf and outBuf intermediate allocations - Fix typo in createHmac call in PersistenceLayer.ts --- src/os/persistence/PersistenceLayer.ts | 18 +++++------------- 1 file changed, 5 insertions(+), 13 deletions(-) diff --git a/src/os/persistence/PersistenceLayer.ts b/src/os/persistence/PersistenceLayer.ts index b60a349..98f3552 100644 --- a/src/os/persistence/PersistenceLayer.ts +++ b/src/os/persistence/PersistenceLayer.ts @@ -13,22 +13,14 @@ export class PersistenceLayer { */ save(data: any, key: string): Buffer { const payload = JSON.stringify(data); - const payloadBuf = Buffer.from(payload, 'utf8'); - - // Bolt Optimization: Allocate unsafe buffer for exact combined size to avoid Buffer.concat and intermediate payload allocations - const outBuf = Buffer.allocUnsafe(38 + payloadBuf.length); - - // Copy pre-allocated header and payload - outBuf.set(HEADER_BUF, 0); - outBuf.set(payloadBuf, 38); + const payloadByteLength = Buffer.byteLength(payload, 'utf8'); - // Compute and write HMAC signature directly - const hmac = crypto.createHmac('sha256',. key); - hmac.update(payloadBuf); + // Bolt Optimization: Compute HMAC directly on payload string to avoid extra Buffer allocations + const hmac = crypto.createHmac('sha256', key); + hmac.update(payload, 'utf8'); const signature = hmac.digest(); - outBuf.set(signature, 6); - const payloadByteLength = Buffer.byteLength(payload, 'utf8'); + // Bolt Optimization: Allocate unsafe buffer for exact combined size to avoid Buffer.concat and intermediate payload allocations const out = Buffer.allocUnsafe(HEADER_LENGTH + SIGNATURE_LENGTH + payloadByteLength); // Zero-copy set of pre-allocated header