fix: bug in ZADD command creation

aleclarson · aleclarson · commit 3113025c2b42 · 2025-05-26T18:31:25.000-04:00
diff --git a/src/commands/sorted-set.ts b/src/commands/sorted-set.ts
@@ -18,9 +18,9 @@ export function ZADD<T extends TSchema>(
   if (entries.length === 0) {
     throw new Error("At least one entry is required");
   }
-  const flatEntries: RedisValue[] = new Array(entries.length * 2);
+  const flatEntries: RedisValue[] = [];
   for (const [score, value] of entries) {
-    flatEntries.push(score);
+    flatEntries.push(String(score));
     flatEntries.push(key.encode(value));
   }
   return new RedisCommand<number>(["ZADD", key.name, ...flatEntries]);

Original file line number	Diff line number	Diff line change
`@@ -18,9 +18,9 @@ export function ZADD<T extends TSchema>(`
`18`	`18`	`if (entries.length === 0) {`
`19`	`19`	`throw new Error("At least one entry is required");`
`20`	`20`	`}`
`21`		`- const flatEntries: RedisValue[] = new Array(entries.length * 2);`
	`21`	`+ const flatEntries: RedisValue[] = [];`
`22`	`22`	`for (const [score, value] of entries) {`
`23`		`- flatEntries.push(score);`
	`23`	`+ flatEntries.push(String(score));`
`24`	`24`	`flatEntries.push(key.encode(value));`
`25`	`25`	`}`
`26`	`26`	`return new RedisCommand<number>(["ZADD", key.name, ...flatEntries]);`