refactor: remove unsafe regex dos

Author: wallace-sf

src/utils/isValidTopic.util.ts

@@ -3,7 +3,6 @@ import { MqttClient } from 'mqtt-browser';
 import invariant from 'tiny-warning';
 
 import { ERROR_MESSAGES } from './constants.util';
-import { isValidTopic } from './isValidTopic.util';
 
 const hasStore = (store: Store | null) => {
   const isValid = store !== null;
@@ -21,22 +20,9 @@ const isClientConnected = (client: MqttClient | null) => {
   return client?.connected;
 };
 
-const hasAllValidTopics = (topics: string | string[]) => {
-  const isValid = Array.isArray(topics)
-    ? topics.every((topic) => isValidTopic(topic))
-    : isValidTopic(topics);
-
-  if (!isValid) invariant(false, ERROR_MESSAGES.NO_VALID_TOPICS);
-
-  return isValid;
-};
-
 export const shouldSubscribeOrUnsubscribe = (
-  topics: string | string[],
   store: Store | null,
   client: MqttClient | null,
 ) => {
-  return (
-    hasAllValidTopics(topics) && hasStore(store) && isClientConnected(client)
-  );
+  return hasStore(store) && isClientConnected(client);
 };