big refactor

1) remove FIRST_KEY_INDEX (As now via pushKeys), but still need something
to enumerate if should be on cluster client, so now have a NOT_KEYED_COMMAND

1a) in process saw that not all commands were using pushKey() correctly.

2) fix up usages of parser.pushVariadic() for use only for existing variadic/arrays..
parser.push() can now handle multi element pushes, so don't have to wrap in an array.

Author: sjpotter

packages/bloom/lib/commands/bloom/ADD.ts

@@ -3,7 +3,6 @@ import { CommandParser } from '@redis/client/dist/lib/client/parser';
 import { transformBooleanReply } from '@redis/client/dist/lib/commands/generic-transformers';
 
 export default {
-  FIRST_KEY_INDEX: 1,
   IS_READ_ONLY: false,
   parseCommand(parser: CommandParser, key: RedisArgument, item: RedisArgument) {
     parser.push('BF.ADD');

packages/bloom/lib/commands/bloom/CARD.ts

@@ -2,7 +2,6 @@ import { CommandParser } from '@redis/client/dist/lib/client/parser';
 import { RedisArgument, NumberReply, Command } from '@redis/client/dist/lib/RESP/types';
 
 export default {
-  FIRST_KEY_INDEX: 1,
   IS_READ_ONLY: true,
   parseCommand(parser: CommandParser, key: RedisArgument) {
     parser.push('BF.CARD');

packages/bloom/lib/commands/bloom/EXISTS.ts

@@ -3,7 +3,6 @@ import { CommandParser } from '@redis/client/dist/lib/client/parser';
 import { transformBooleanReply } from '@redis/client/dist/lib/commands/generic-transformers';
 
 export default {
-  FIRST_KEY_INDEX: 1,
   IS_READ_ONLY: true,
   parseCommand(parser: CommandParser, key: RedisArgument, item: RedisArgument) {
     parser.push('BF.EXISTS');

packages/bloom/lib/commands/bloom/INFO.ts

@@ -11,7 +11,6 @@ export type BfInfoReplyMap = TuplesToMapReply<[
 ]>;
 
 export default {
-  FIRST_KEY_INDEX: 1,
   IS_READ_ONLY: true,
   parseCommand(parser: CommandParser, key: RedisArgument) {
     parser.push('BF.INFO');

packages/bloom/lib/commands/bloom/INSERT.ts

@@ -12,7 +12,6 @@ export interface BfInsertOptions {
 }
 
 export default {
-  FIRST_KEY_INDEX: 1,
   IS_READ_ONLY: false,
   parseCommand(
     parser: CommandParser,

packages/bloom/lib/commands/bloom/LOADCHUNK.ts

@@ -2,7 +2,6 @@ import { CommandParser } from '@redis/client/dist/lib/client/parser';
 import { RedisArgument, SimpleStringReply, Command } from '@redis/client/dist/lib/RESP/types';
 
 export default {
-  FIRST_KEY_INDEX: 1,
   IS_READ_ONLY: false,
   parseCommand(parser: CommandParser, key: RedisArgument, iterator: number, chunk: RedisArgument) {
     parser.push('BF.LOADCHUNK');

packages/bloom/lib/commands/bloom/MADD.ts

@@ -4,7 +4,6 @@ import { RedisVariadicArgument } from '@redis/client/dist/lib/commands/generic-t
 import { transformBooleanArrayReply } from '@redis/client/dist/lib/commands/generic-transformers';
 
 export default {
-  FIRST_KEY_INDEX: 1,
   IS_READ_ONLY: false,
   parseCommand(parser: CommandParser, key: RedisArgument, items: RedisVariadicArgument) {
     parser.push('BF.MADD');

packages/bloom/lib/commands/bloom/MEXISTS.ts

@@ -4,7 +4,6 @@ import { RedisVariadicArgument } from '@redis/client/dist/lib/commands/generic-t
 import { transformBooleanArrayReply } from '@redis/client/dist/lib/commands/generic-transformers';
 
 export default {
-  FIRST_KEY_INDEX: 1,
   IS_READ_ONLY: true,
   parseCommand(parser: CommandParser, key: RedisArgument, items: RedisVariadicArgument) {
     parser.push('BF.MEXISTS');

packages/bloom/lib/commands/bloom/RESERVE.ts

@@ -7,7 +7,6 @@ export interface BfReserveOptions {
 }
 
 export default {
-  FIRST_KEY_INDEX: 1,
   IS_READ_ONLY: true,
   parseCommand(
     parser: CommandParser,

packages/bloom/lib/commands/bloom/SCANDUMP.ts

@@ -2,7 +2,6 @@ import { CommandParser } from '@redis/client/dist/lib/client/parser';
 import { RedisArgument, TuplesReply, NumberReply, BlobStringReply, UnwrapReply, Command } from '@redis/client/dist/lib/RESP/types';
 
 export default {
-  FIRST_KEY_INDEX: 1,
   IS_READ_ONLY: true,
   parseCommand(parser: CommandParser, key: RedisArgument, iterator: number) {
     parser.push('BF.SCANDUMP');