revert changes needed for resp3 erroring with multi, as now handled by attachCommand

Author: sjpotter

packages/client/lib/client/index.ts

@@ -870,8 +870,7 @@ export default class RedisClient<
     type Multi = new (...args: ConstructorParameters<typeof RedisClientMultiCommand>) => RedisClientMultiCommandType<[], M, F, S, RESP, TYPE_MAPPING>;
     return new ((this as any).Multi as Multi)(
       this._executeMulti.bind(this),
-      this._executePipeline.bind(this),
-      this._self.#options
+      this._executePipeline.bind(this)
     );
   }
 

packages/client/lib/client/multi-command.ts

@@ -2,7 +2,6 @@ import COMMANDS from '../commands';
 import RedisMultiCommand, { MULTI_REPLY, MultiReply, MultiReplyType, RedisMultiQueuedCommand } from '../multi-command';
 import { ReplyWithTypeMapping, CommandReply, Command, CommandArguments, CommanderConfig, RedisFunctions, RedisModules, RedisScripts, RespVersions, TransformReply, RedisScript, RedisFunction, TypeMapping } from '../RESP/types';
 import { attachConfig, functionArgumentsPrefix, getTransformReply } from '../commander';
-import { RedisClientOptions } from '.';
 
 type CommandSignature<
   REPLIES extends Array<unknown>,
@@ -154,12 +153,10 @@ export default class RedisClientMultiCommand<REPLIES = []> {
   readonly #executeMulti: ExecuteMulti;
   readonly #executePipeline: ExecuteMulti;
   #selectedDB?: number;
-  readonly #options?: RedisClientOptions
 
-  constructor(executeMulti: ExecuteMulti, executePipeline: ExecuteMulti, options?: RedisClientOptions) {
+  constructor(executeMulti: ExecuteMulti, executePipeline: ExecuteMulti) {
     this.#executeMulti = executeMulti;
     this.#executePipeline = executePipeline;
-    this.#options = options;
   }
 
   SELECT(db: number, transformReply?: TransformReply): this {

packages/client/lib/client/pool.ts

@@ -426,8 +426,7 @@ export class RedisClientPool<
     type Multi = new (...args: ConstructorParameters<typeof RedisClientMultiCommand>) => RedisClientMultiCommandType<[], M, F, S, RESP, TYPE_MAPPING>;
     return new ((this as any).Multi as Multi)(
       (commands, selectedDB) => this.execute(client => client._executeMulti(commands, selectedDB)),
-      commands => this.execute(client => client._executePipeline(commands)),
-      this._self.#options
+      commands => this.execute(client => client._executePipeline(commands))
     );
   }
 

packages/client/lib/cluster/index.ts

@@ -521,8 +521,7 @@ export default class RedisCluster<
         const client = await this._self.#slots.getClient(firstKey, isReadonly);
         return client._executePipeline(commands);
       },
-      routing,
-      this._self.#options
+      routing
     );
   }
 

packages/client/lib/cluster/multi-command.ts

@@ -2,7 +2,7 @@ import COMMANDS from '../commands';
 import RedisMultiCommand, { MULTI_REPLY, MultiReply, MultiReplyType, RedisMultiQueuedCommand } from '../multi-command';
 import { ReplyWithTypeMapping, CommandReply, Command, CommandArguments, CommanderConfig, RedisFunctions, RedisModules, RedisScripts, RespVersions, TransformReply, RedisScript, RedisFunction, TypeMapping, RedisArgument } from '../RESP/types';
 import { attachConfig, functionArgumentsPrefix, getTransformReply } from '../commander';
-import RedisCluster, { RedisClusterOptions } from '.';
+import RedisCluster from '.';
 
 type CommandSignature<
   REPLIES extends Array<unknown>,
@@ -191,18 +191,15 @@ export default class RedisClusterMultiCommand<REPLIES = []> {
   readonly #executePipeline: ClusterMultiExecute;
   #firstKey: RedisArgument | undefined;
   #isReadonly: boolean | undefined = true;
-  readonly #options?: RedisClusterOptions;
 
   constructor(
     executeMulti: ClusterMultiExecute,
     executePipeline: ClusterMultiExecute,
-    routing: RedisArgument | undefined,
-    options?: RedisClusterOptions
+    routing: RedisArgument | undefined
   ) {
     this.#executeMulti = executeMulti;
     this.#executePipeline = executePipeline;
     this.#firstKey = routing;
-    this.#options = options;
   }
 
   #setState(