[lib] Show staff alerts when an operation isn't processed

Summary:
Show an alert for all the cases where an operation can't be immediately processed.

https://linear.app/comm/issue/ENG-9686/some-messages-from-newly-created-chat-missing-on-secondary-device#comment-61c90aa7

Depends on D13724

Test Plan: Checked that an alert is shown after queueing operations on web in dev mode. Deleted the dev check and made sure the alert isn't shown.

Reviewers: kamil, angelika

Reviewed By: kamil

Subscribers: ashoat

Differential Revision: https://phab.comm.dev/D13725

lib/shared/dm-ops/process-dm-ops.js

@@ -26,11 +26,15 @@ import {
   queueDMOpsActionType,
   dmOperationValidator,
 } from '../../types/dm-ops.js';
+import type { DMOperation } from '../../types/dm-ops.js';
 import type { NotificationsCreationData } from '../../types/notif-types.js';
 import type { DispatchMetadata } from '../../types/redux-types.js';
 import type { OutboundP2PMessage } from '../../types/sqlite-types.js';
+import { getConfig } from '../../utils/config.js';
 import { extractUserIDsFromPayload } from '../../utils/conversion-utils.js';
+import { isDev } from '../../utils/dev-utils.js';
 import { useSelector, useDispatch } from '../../utils/redux-utils.js';
+import { useIsCurrentUserStaff } from '../staff-utils.js';
 
 function useProcessDMOperation(): (
   dmOperationSpecification: DMOperationSpecification,
@@ -44,14 +48,28 @@ function useProcessDMOperation(): (
 
   const dispatch = useDispatch();
 
+  const isCurrentUserStaff = useIsCurrentUserStaff();
+  const showOperationAlertToStaff = React.useCallback(
+    (description: string, operation: DMOperation) => {
+      if (!isCurrentUserStaff && !isDev) {
+        return;
+      }
+      getConfig().showAlert(description, JSON.stringify(operation));
+    },
+    [isCurrentUserStaff],
+  );
+
   return React.useCallback(
     async (
       dmOperationSpecification: DMOperationSpecification,
       dmOpID: ?string,
     ) => {
       const { viewerID, ...restUtilities } = baseUtilities;
       if (!viewerID) {
-        console.log('ignored DMOperation because logged out');
+        showOperationAlertToStaff(
+          'Ignored DMOperation because logged out',
+          dmOperationSpecification.op,
+        );
         return;
       }
       const utilities: ProcessDMOperationUtilities = {
@@ -122,7 +140,10 @@ function useProcessDMOperation(): (
       }
 
       if (!dmOpSpecs[dmOp.type].operationValidator.is(dmOp)) {
-        console.log(`Ignoring ${dmOp.type} operation because it is invalid`);
+        showOperationAlertToStaff(
+          "Ignoring operation because it doesn't pass validation",
+          dmOp,
+        );
         await confirmPeerToPeerMessage(dispatchMetadata);
         return;
       }
@@ -134,6 +155,10 @@ function useProcessDMOperation(): (
 
       if (!processingCheckResult.isProcessingPossible) {
         if (processingCheckResult.reason.type === 'invalid') {
+          showOperationAlertToStaff(
+            'Ignoring operation because it is invalid',
+            dmOp,
+          );
           await confirmPeerToPeerMessage(dispatchMetadata);
           return;
         }
@@ -160,6 +185,18 @@ function useProcessDMOperation(): (
             userID: processingCheckResult.reason.userID,
           };
         }
+
+        if (condition?.type) {
+          showOperationAlertToStaff(
+            `Adding operation to the ${condition.type} queue`,
+            dmOp,
+          );
+        } else {
+          showOperationAlertToStaff(
+            'Operation should be added to a queue but its type is missing',
+            dmOp,
+          );
+        }
         dispatchWithMetadata(
           {
             type: queueDMOpsActionType,
@@ -239,9 +276,10 @@ function useProcessDMOperation(): (
       baseUtilities,
       processBlobHolders,
       dispatchWithMetadata,
+      showOperationAlertToStaff,
       createMessagesToPeersFromDMOp,
-      dispatch,
       confirmPeerToPeerMessage,
+      dispatch,
     ],
   );
 }