fix(reactotron-core-ui): Move commandListener callbacks outside of render cycle (#1544 by @jamonholmgren)

[skip ci]

Author: jamonholmgren

apps/reactotron-app/src/renderer/contexts/Standalone/useStandalone.ts

@@ -135,12 +135,15 @@ export function reducer(state: State, action: Action) {
           return
         }
 
-        draftState.commandListeners.forEach((cl) => cl(action.payload))
-
         const connection = draftState.connections.find(
           (c) => c.clientId === action.payload.clientId
         )
 
+        if (!connection) {
+          console.error("Command received for unknown connection", action.payload)
+          return
+        }
+
         connection.commands = [action.payload, ...connection.commands]
       })
     case ActionTypes.ClearConnectionCommands:
@@ -205,9 +208,16 @@ function useStandalone() {
   }, [])
 
   // Called when commands are flowing in.
-  const commandReceived = useCallback((command: any) => {
-    dispatch({ type: ActionTypes.CommandReceived, payload: command })
-  }, [])
+  const commandReceived = useCallback(
+    (command: any) => {
+      // First dispatch to update state
+      dispatch({ type: ActionTypes.CommandReceived, payload: command })
+
+      // Then notify listeners
+      state.commandListeners.forEach((cl) => cl(command))
+    },
+    [state.commandListeners]
+  )
 
   // Called when a client disconnects. NOTE: They could be coming back. This could happen with a reload of the simulator!
   const connectionDisconnected = useCallback((connection: ReactotronConnection) => {