fix: fix the response types for moderation (#1681)

## CLA

- [ ] I have signed the [Stream
CLA](https://docs.google.com/forms/d/e/1FAIpQLScFKsKkAJI7mhCr7K9rEIOpqIDThrWxuvxnwUq2XkHyG154vQ/viewform)
(required).
- [ ] Code changes are tested

## Description of the changes, What, Why and How?

Fix typescript for moderation endpoints

-
https://linear.app/stream/issue/MOD2-631/inconsistent-type-declaration-for-submitaction-response
-
https://linear.app/stream/issue/MOD2-632/fix-typescript-for-reviewqueuefilters-type

Author: vishalnarkhede

src/moderation.ts

@@ -1,6 +1,8 @@
 import type {
   APIResponse,
+  CheckResponse,
   CustomCheckFlag,
+  CustomCheckResponse,
   GetConfigResponse,
   GetUserModerationReportOptions,
   GetUserModerationReportResponse,
@@ -19,11 +21,12 @@ import type {
   QueryModerationRulesSort,
   RequireAtLeastOne,
   ReviewQueueFilters,
-  ReviewQueueItem,
   ReviewQueuePaginationOptions,
   ReviewQueueResponse,
   ReviewQueueSort,
   SubmitActionOptions,
+  SubmitActionResponse,
+  UnmuteUserResponse,
   UpsertConfigResponse,
   UpsertModerationRuleResponse,
 } from './types';
@@ -135,7 +138,7 @@ export class Moderation {
       user_id?: string;
     },
   ) {
-    return await this.client.post<{ item_id: string } & APIResponse>(
+    return await this.client.post<UnmuteUserResponse>(
       this.client.baseURL + '/api/v2/moderation/unmute',
       {
         target_ids: [targetID],
@@ -209,7 +212,7 @@ export class Moderation {
   }
 
   async deleteConfig(key: string, data?: { team?: string }) {
-    return await this.client.delete(
+    return await this.client.delete<APIResponse>(
       this.client.baseURL + '/api/v2/moderation/config/' + key,
       data,
     );
@@ -241,7 +244,7 @@ export class Moderation {
     itemID: string,
     options: SubmitActionOptions = {},
   ) {
-    return await this.client.post<{ item_id: string } & APIResponse>(
+    return await this.client.post<SubmitActionResponse>(
       this.client.baseURL + '/api/v2/moderation/submit_action',
       {
         action_type: actionType,
@@ -281,14 +284,17 @@ export class Moderation {
       test_mode?: boolean;
     },
   ) {
-    return await this.client.post(this.client.baseURL + `/api/v2/moderation/check`, {
-      entity_type: entityType,
-      entity_id: entityID,
-      entity_creator_id: entityCreatorID,
-      moderation_payload: moderationPayload,
-      config_key: configKey,
-      options,
-    });
+    return await this.client.post<CheckResponse>(
+      this.client.baseURL + `/api/v2/moderation/check`,
+      {
+        entity_type: entityType,
+        entity_id: entityID,
+        entity_creator_id: entityCreatorID,
+        moderation_payload: moderationPayload,
+        config_key: configKey,
+        options,
+      },
+    );
   }
 
   /**
@@ -368,15 +374,16 @@ export class Moderation {
     },
     flags: CustomCheckFlag[],
   ) {
-    return await this.client.post<
-      { id: string; item: ReviewQueueItem; status: string } & APIResponse
-    >(this.client.baseURL + `/api/v2/moderation/custom_check`, {
-      entity_type: entityType,
-      entity_id: entityID,
-      entity_creator_id: entityCreatorID,
-      moderation_payload: moderationPayload,
-      flags,
-    });
+    return await this.client.post<CustomCheckResponse>(
+      this.client.baseURL + `/api/v2/moderation/custom_check`,
+      {
+        entity_type: entityType,
+        entity_id: entityID,
+        entity_creator_id: entityCreatorID,
+        moderation_payload: moderationPayload,
+        flags,
+      },
+    );
   }
 
   /**
@@ -446,7 +453,7 @@ export class Moderation {
    * @returns
    */
   async deleteModerationRule(id: string) {
-    return await this.client.delete(
+    return await this.client.delete<APIResponse>(
       this.client.baseURL + '/api/v2/moderation/moderation_rule/' + id,
     );
   }

src/types.ts

@@ -801,6 +801,11 @@ export type MuteUserResponse = APIResponse & {
   mute?: MuteResponse;
   mutes?: Array<Mute>;
   own_user?: OwnUserResponse;
+  non_existing_users?: string[];
+};
+
+export type UnmuteUserResponse = APIResponse & {
+  non_existing_users?: string[];
 };
 
 export type BlockUserAPIResponse = APIResponse & {
@@ -3670,6 +3675,7 @@ export type ReviewQueueItem = {
   created_by: string;
   entity_id: string;
   entity_type: string;
+  entity_creator_id?: string;
   flags: ModerationFlag[];
   has_image: boolean;
   has_text: boolean;
@@ -3685,6 +3691,7 @@ export type ReviewQueueItem = {
   reviewed_at: string;
   status: string;
   updated_at: string;
+  latest_moderator_action?: string;
 };
 
 export type CustomCheckFlag = {
@@ -3726,6 +3733,10 @@ export type SubmitActionOptions = {
   user_id?: string;
 };
 
+export type SubmitActionResponse = APIResponse & {
+  item?: ReviewQueueItem;
+};
+
 export type GetUserModerationReportResponse = {
   user: UserResponse;
   user_blocks?: Array<{
@@ -3736,6 +3747,19 @@ export type GetUserModerationReportResponse = {
   user_mutes?: Mute[];
 };
 
+export type CheckResponse = APIResponse & {
+  status: string;
+  task_id?: string;
+  recommended_action: string;
+  item?: ReviewQueueItem;
+};
+
+export type CustomCheckResponse = APIResponse & {
+  id: string;
+  item: ReviewQueueItem;
+  status: string;
+};
+
 export type QueryModerationConfigsFilters = QueryFilters<
   {
     key?: string;
@@ -3787,6 +3811,12 @@ export type ReviewQueueFilters = QueryFilters<
     entity_id?:
       | RequireOnlyOne<Pick<QueryFilter<ReviewQueueItem['entity_id']>, '$eq' | '$in'>>
       | PrimitiveFilter<ReviewQueueItem['entity_id']>;
+  } & {
+    entity_creator_id?:
+      | RequireOnlyOne<
+          Pick<QueryFilter<ReviewQueueItem['entity_creator_id']>, '$eq' | '$in'>
+        >
+      | PrimitiveFilter<ReviewQueueItem['entity_creator_id']>;
   } & {
     reviewed?: boolean;
   } & {
@@ -3840,6 +3870,7 @@ export type ReviewQueueFilters = QueryFilters<
   } & {
     recommended_action?: RequireOnlyOne<{
       $eq?: string;
+      $in?: string[];
     }>;
   } & {
     flagged_user_id?: RequireOnlyOne<{
@@ -3852,6 +3883,7 @@ export type ReviewQueueFilters = QueryFilters<
   } & {
     label?: RequireOnlyOne<{
       $eq?: string;
+      $in?: string[];
     }>;
   } & {
     reporter_type?: RequireOnlyOne<{
@@ -3866,6 +3898,24 @@ export type ReviewQueueFilters = QueryFilters<
     date_range?: RequireOnlyOne<{
       $eq?: string; // Format: "date1_date2"
     }>;
+  } & {
+    latest_moderator_action?:
+      | RequireOnlyOne<
+          Pick<QueryFilter<ReviewQueueItem['latest_moderator_action']>, '$eq' | '$in'>
+        >
+      | PrimitiveFilter<ReviewQueueItem['latest_moderator_action']>;
+  } & {
+    flags_count?: RequireOnlyOne<{
+      $eq?: number;
+    }>;
+  } & {
+    ai_text_severity?: RequireOnlyOne<{
+      $eq?: string;
+    }>;
+  } & {
+    channel_cid?: RequireOnlyOne<{
+      $eq?: string;
+    }>;
   }
 >;
 
@@ -3877,8 +3927,25 @@ export type QueryModerationConfigsSort = Array<Sort<'key' | 'created_at' | 'upda
 
 export type ReviewQueuePaginationOptions = Pager;
 
+export type FilterConfigResponse = {
+  llm_labels: string[];
+  ai_text_labels?: string[];
+};
+
+export type ModerationActionConfig = {
+  entity_type: string;
+  order: number;
+  action: string;
+  icon: string;
+  description: string;
+  custom?: Record<string, unknown>;
+};
+
 export type ReviewQueueResponse = {
   items: ReviewQueueItem[];
+  action_config?: Record<string, ModerationActionConfig[]>;
+  filter_config?: FilterConfigResponse;
+  stats?: Record<string, unknown>;
   next?: string;
   prev?: string;
 };