fix: remove nin and ne operator usage in the SDK and the sample app (#2672)

* fix: remove  and  operator usage in the SDK and the sample app

* fix: remove console log

* fix: change console log to warn

* fix: add improvemnts

* fix: import of debouncefunc

* fix: import of debouncefunc

* fix: restructure queryMembers, queryUsers and ACItriggersettings

* fix: error bubbling for suggestions in auto complete input

Author: khushal87

examples/SampleApp/src/hooks/usePaginatedUsers.ts

@@ -121,9 +121,6 @@ export const usePaginatedUsers = (): PaginatedUsers => {
     try {
       queryInProgress.current = true;
       const filter: UserFilters = {
-        id: {
-          $nin: [chatClient?.userID],
-        },
         role: 'user',
       };
 
@@ -143,7 +140,7 @@ export const usePaginatedUsers = (): PaginatedUsers => {
         return;
       }
 
-      const res = await chatClient?.queryUsers(
+      const { users } = await chatClient?.queryUsers(
         filter,
         { name: 1 },
         {
@@ -153,33 +150,25 @@ export const usePaginatedUsers = (): PaginatedUsers => {
         },
       );
 
-      if (!res?.users) {
-        queryInProgress.current = false;
-        return;
-      }
-
-      // Dumb check to avoid duplicates
-      if (query === searchText && results.findIndex((r) => res?.users[0].id === r.id) > -1) {
-        queryInProgress.current = false;
-        return;
-      }
+      const usersWithoutClientUserId = users.filter((user) => user.id !== chatClient.userID);
 
       setResults((r) => {
         if (query !== searchText) {
-          return res?.users;
+          return usersWithoutClientUserId;
         }
-        return r.concat(res?.users || []);
+        return r.concat(usersWithoutClientUserId);
       });
 
-      if (res?.users.length < 10 && (offset.current === 0 || query === searchText)) {
+      if (usersWithoutClientUserId.length < 10 && (offset.current === 0 || query === searchText)) {
         hasMoreResults.current = false;
       }
 
       if (!query && offset.current === 0) {
-        setInitialResults(res?.users || []);
+        setInitialResults(usersWithoutClientUserId);
       }
     } catch (e) {
       // do nothing;
+      console.log('Error fetching users', e);
     }
     queryInProgress.current = false;
     setLoading(false);

package/src/components/AutoCompleteInput/AutoCompleteInput.tsx

@@ -28,9 +28,12 @@ import {
 } from '../../contexts/translationContext/TranslationContext';
 import type { Emoji } from '../../emoji-data';
 import type { DefaultStreamChatGenerics } from '../../types/types';
-import { isCommandTrigger, isEmojiTrigger, isMentionTrigger } from '../../utils/utils';
-
-import type { Trigger } from '../../utils/utils';
+import {
+  isCommandTrigger,
+  isEmojiTrigger,
+  isMentionTrigger,
+  Trigger,
+} from '../../utils/ACITriggerSettings';
 
 const styles = StyleSheet.create({
   inputBox: {

package/src/components/AutoCompleteInput/__tests__/AutoCompleteInput.test.js

@@ -8,7 +8,7 @@ import { useMockedApis } from '../../../mock-builders/api/useMockedApis';
 import { generateChannelResponse } from '../../../mock-builders/generator/channel';
 import { generateUser } from '../../../mock-builders/generator/user';
 import { getTestClientWithUser } from '../../../mock-builders/mock';
-import { ACITriggerSettings } from '../../../utils/utils';
+import { ACITriggerSettings } from '../../../utils/ACITriggerSettings';
 import { Chat } from '../../Chat/Chat';
 import { AutoCompleteInput } from '../AutoCompleteInput';
 

package/src/contexts/messageInputContext/MessageInputContext.tsx

@@ -63,17 +63,19 @@ import {
   ImageUpload,
   UnknownType,
 } from '../../types/types';
-import { compressedImageURI } from '../../utils/compressImage';
-import { removeReservedFields } from '../../utils/removeReservedFields';
 import {
   ACITriggerSettings,
   ACITriggerSettingsParams,
+  TriggerSettings,
+} from '../../utils/ACITriggerSettings';
+import { compressedImageURI } from '../../utils/compressImage';
+import { removeReservedFields } from '../../utils/removeReservedFields';
+import {
   FileState,
   FileStateValue,
   generateRandomId,
   getFileNameFromPath,
   isBouncedMessage,
-  TriggerSettings,
 } from '../../utils/utils';
 import { useAttachmentPickerContext } from '../attachmentPickerContext/AttachmentPickerContext';
 import { ChannelContextValue, useChannelContext } from '../channelContext/ChannelContext';
@@ -1073,25 +1075,30 @@ export const MessageInputProvider = <
   };
 
   const getTriggerSettings = () => {
-    let triggerSettings: TriggerSettings<StreamChatGenerics> = {};
-    if (channel) {
-      if (value.autoCompleteTriggerSettings) {
-        triggerSettings = value.autoCompleteTriggerSettings({
-          channel,
-          client,
-          emojiSearchIndex: value.emojiSearchIndex,
-          onMentionSelectItem: onSelectItem,
-        });
-      } else {
-        triggerSettings = ACITriggerSettings<StreamChatGenerics>({
-          channel,
-          client,
-          emojiSearchIndex: value.emojiSearchIndex,
-          onMentionSelectItem: onSelectItem,
-        });
+    try {
+      let triggerSettings: TriggerSettings<StreamChatGenerics> = {};
+      if (channel) {
+        if (value.autoCompleteTriggerSettings) {
+          triggerSettings = value.autoCompleteTriggerSettings({
+            channel,
+            client,
+            emojiSearchIndex: value.emojiSearchIndex,
+            onMentionSelectItem: onSelectItem,
+          });
+        } else {
+          triggerSettings = ACITriggerSettings<StreamChatGenerics>({
+            channel,
+            client,
+            emojiSearchIndex: value.emojiSearchIndex,
+            onMentionSelectItem: onSelectItem,
+          });
+        }
       }
+      return triggerSettings;
+    } catch (error) {
+      console.warn('Error in getting trigger settings', error);
+      throw error;
     }
-    return triggerSettings;
   };
 
   const triggerSettings = getTriggerSettings();

package/src/index.ts

@@ -11,8 +11,11 @@ export * from './icons';
 
 export * from './types/types';
 
+export * from './utils/ACITriggerSettings';
 export * from './utils/patchMessageTextCommand';
 export * from './utils/i18n/Streami18n';
+export * from './utils/queryMembers';
+export * from './utils/queryUsers';
 export * from './utils/utils';
 export * from './utils/StreamChatRN';