fix(client): connect nodes lazily in all-shards/all-nodes fan-out

getAllMasterClients/getAllClients silently skipped nodes without a
connected client. With minimizeConnections (or a partially connected
cluster) all_shards/all_nodes commands fanned out to a subset of nodes —
or none, resolving undefined: PING via sendCommand returned undefined,
DBSIZE could sum a subset without any error. Route through nodeClient(),
the same lazy connect-or-reuse accessor keyed routing uses.

getAllClients no longer yields dedicated PubSub connections; they cannot
run regular commands.

Also guard the policy dispatch against an empty fan-out — failing loud
beats resolving undefined.

Author: nkaradzhov

packages/client/lib/cluster/cluster-slots.ts

@@ -792,18 +792,21 @@ export default class RedisClusterSlots<
     this.#emit('disconnect');
   }
 
-  getAllClients() {
-    return Array.from(this.#clients());
+  /**
+   * All node clients (masters and replicas), connecting lazily — with
+   * `minimizeConnections` nodes have no client until first use, and skipping
+   * them would silently fan commands out to a subset of the cluster.
+   * Excludes dedicated PubSub connections: they cannot run regular commands.
+   */
+  getAllClients(): Promise<Array<RedisClientType<M, F, S, RESP, TYPE_MAPPING>>> {
+    return Promise.all([
+      ...this.masters.map(master => this.nodeClient(master)),
+      ...this.replicas.map(replica => this.nodeClient(replica))
+    ]);
   }
 
-  getAllMasterClients() {
-    const result = [];
-    for (const master of this.masters) {
-      if (master.client) {
-        result.push(master.client);
-      }
-    }
-    return result;
+  getAllMasterClients(): Promise<Array<RedisClientType<M, F, S, RESP, TYPE_MAPPING>>> {
+    return Promise.all(this.masters.map(master => this.nodeClient(master)));
   }
 
   async getClientAndSlotNumber(

packages/client/lib/cluster/index.ts

@@ -506,6 +506,10 @@ export default class RedisCluster<
     const clients: Array<RedisClientType<M, F, S, RESP, TYPE_MAPPING>> =
       await router(this._slots, parser, isReadonly);
 
+    if (clients.length === 0) {
+      throw new Error(`Request policy ${requestPolicy} produced no target nodes`);
+    }
+
     const responsePromises = clients.map(
       client => this._execute(parser, isReadonly, options, makeFn(parser), client)
     );