fix: TECH review fixes

packages/pubsub/src/MessageHandlerIdempotent.ts

@@ -6,7 +6,6 @@ import { ISubscriber } from './SubscriberFactory'
 type GetIdempotencyKeyFunction<T> = (msg: T, info: IMessageInfo) => string | undefined
 
 export abstract class IdempotencyStorage {
-
   public abstract exists(key: string): Promise<boolean>
 
   public abstract save(key: string): Promise<void>
@@ -60,7 +59,7 @@ export abstract class MessageHandlerIdempotent<T = unknown> {
       if (idempotencyKey) {
         const alreadyProcessed = await this.idempotencyStorage.exists(idempotencyKey)
           .catch(e => {
-            this.subscriber.logger?.info(`Error checking idempotency key: ${idempotencyKey}`, e)
+            this.subscriber.logger?.info(`Error checking idempotency key existence: ${idempotencyKey}`, e)
             return false
           })
         if (alreadyProcessed) {

packages/pubsub/src/redis-types/IRedisClient.ts

@@ -1,6 +1,6 @@
 /**
- * This file contains interfaces from https://github.com/join-com/caching ,
- * for a less duplicated code when implementing idempotent handler on services.
+ * This file contains interfaces from https://github.com/join-com/caching, to be able to have RedisIdempotencyStorage
+ * in the lib, to duplicate less code, when implementing idempotent handler in the services.
  */
 
 export interface IFailure<T> {
@@ -22,13 +22,10 @@ export type RedisResult<T, R> = ISuccess<T> | IFailure<R>
 export type OptionalRedisResult<T, R> = ISuccess<T> | ISuccess<undefined> | IFailure<R>
 
 export interface IRedisError {
-
   error: Error
 }
 
-
 export interface IRedisClient {
-
   get(key: string): Promise<OptionalRedisResult<Buffer, IRedisError>>
 
   setex(key: string, secondsTtl: number, value: Buffer): Promise<RedisResult<void, IRedisError>>