fix: pubsub subscriber in useEffect (#814)

### issue
* pubSub subscriber has been undefined, so couldn't remove the previous
subscriptions
* delete command has been used to Array, but it's for Object

### Fix
* move the pubSub Subscriber into the useEffect sope for making
effective
* use splice instead of delete in the pubSubFactory

Author: HoonBaek

src/lib/pubSub/index.ts

@@ -26,7 +26,7 @@ const pubSubFactory = (): PubSubTypes => {
       // Provide handle back for removal of topic
       return {
         remove: () => {
-          delete topics[topic][index];
+          topics[topic].splice(index, 1);
         },
       };
     },

src/modules/Channel/context/hooks/useHandleChannelPubsubEvents.ts

@@ -24,8 +24,8 @@ export const useHandleChannelPubsubEvents = ({
   dispatcher,
   scrollRef,
 }: UseHandlePubsubEventsParams): void => {
-  const subscriber = new Map();
   useEffect(() => {
+    const subscriber = new Map();
     if (pubSub?.subscribe) {
       subscriber.set(PUBSUB_TOPICS.SEND_USER_MESSAGE, pubSub.subscribe(PUBSUB_TOPICS.SEND_USER_MESSAGE, (props) => {
         const { channel, message } = props;

src/modules/Thread/context/hooks/useHandleThreadPubsubEvents.ts

@@ -28,11 +28,9 @@ export default function useHandleThreadPubsubEvents({
   threadDispatcher,
 }: StaticProps): void {
   useEffect(() => {
-    const pubSubHandler = (): Map<any, any> => {
-      const subscriber = new Map();
-      if (!pubSub || !pubSub.subscribe) {
-        return subscriber;
-      }
+    const subscriber = new Map();
+    if (pubSub?.subscribe) {
+      // TODO: subscribe ON_FILE_INFO_UPLOADED
       subscriber.set(topics.SEND_MESSAGE_START, pubSub.subscribe(topics.SEND_MESSAGE_START, (props) => {
         const {
           channel,
@@ -143,8 +141,7 @@ export default function useHandleThreadPubsubEvents({
           });
         }
       }));
-    };
-    const subscriber = pubSubHandler();
+    }
     return () => {
       subscriber?.forEach((s) => {
         try {