fix(client): XADD/XTRIM with LIMIT 0 must emit the argument (redis#3342)

`XADD ... TRIM.limit` and `XTRIM ... LIMIT` were guarded by a truthy check
(`if (options.TRIM.limit)` / `if (options?.LIMIT)`), so an explicit `0` was
silently dropped and the server fell back to its implicit default limit.

Per the Redis docs, `LIMIT 0` means unlimited trimming, whereas omitting
LIMIT caps eviction at `100 * stream-node-max-entries` (10000 by default),
so the two are observably different. Against Redis 8.8, trimming a
20000-entry stream with `MAXLEN ~ 1` evicts 10000 entries without LIMIT but
19900 with `LIMIT 0` — meaning a caller asking for unlimited trimming
currently gets the capped default instead.

Guard on `!== undefined` so `0` is forwarded, matching the recent XGROUP
ENTRIESREAD 0 fix (redis#3333). Adds a `LIMIT 0` argument test for both commands.

Co-authored-by: Claude Opus 4.8 (1M context) <noreply@anthropic.com>

Author: Develop-KIM

packages/client/lib/commands/XADD.spec.ts

@@ -80,6 +80,20 @@ describe('XADD', () => {
       );
     });
 
+    it('with TRIM.limit 0', () => {
+      assert.deepEqual(
+        parseArgs(XADD, 'key', '*', {
+          field: 'value'
+        }, {
+          TRIM: {
+            threshold: 1000,
+            limit: 0
+          }
+        }),
+        ['XADD', 'key', '1000', 'LIMIT', '0', '*', 'field', 'value']
+      );
+    });
+
     it('with TRIM.policy', () => {
       assert.deepEqual(
         parseArgs(XADD, 'key', '*', {

packages/client/lib/commands/XADD.ts

@@ -89,7 +89,7 @@ export function parseXAddArguments(
 
     parser.push(options.TRIM.threshold.toString());
 
-    if (options.TRIM.limit) {
+    if (options.TRIM.limit !== undefined) {
       parser.push('LIMIT', options.TRIM.limit.toString());
     }
 

packages/client/lib/commands/XTRIM.spec.ts

@@ -43,6 +43,16 @@ describe('XTRIM', () => {
       );
     });
 
+    it('with LIMIT 0', () => {
+      assert.deepEqual(
+        parseArgs(XTRIM, 'key', 'MAXLEN', 1, {
+          strategyModifier: '~',
+          LIMIT: 0
+        }),
+        ['XTRIM', 'key', 'MAXLEN', '~', '1', 'LIMIT', '0']
+      );
+    });
+
     it('with strategyModifier, LIMIT', () => {
       assert.deepEqual(
         parseArgs(XTRIM, 'key', 'MAXLEN', 1, {

packages/client/lib/commands/XTRIM.ts

@@ -39,7 +39,7 @@ export default {
 
     parser.push(threshold.toString());
 
-    if (options?.LIMIT) {
+    if (options?.LIMIT !== undefined) {
       parser.push('LIMIT', options.LIMIT.toString());
     }