presence: Get tuning parameters from server, and store them

Toward #5669.

This doesn't yet actually use the new tuning parameters;
we'll do that in the next few commits.

Author: gnprice

src/api/initialDataTypes.js

@@ -61,6 +61,14 @@ export type RawInitialDataBase = $ReadOnly<{|
    * `zulip_feature_level`, above.
    */
   zulip_version: string,
+
+  /** New in FL 164; use 60 when absent. */
+  // TODO(server-7.0): simplify to always-present
+  server_presence_ping_interval_seconds?: number,
+
+  /** New in FL 164; use 140 when absent. */
+  // TODO(server-7.0): simplify to always-present
+  server_presence_offline_threshold_seconds?: number,
 |}>;
 
 /**

src/presence/PresenceHeartbeat.js

@@ -26,6 +26,7 @@ export default function PresenceHeartbeat(props: Props): React.Node {
       // TODO(#5005): should ensure this gets the intended account
       dispatch(reportPresence(true));
     };
+    // TODO(#5669): get heartbeat interval from PresenceState
     const heartbeat = new Heartbeat(onHeartbeat, 1000 * 60);
 
     // React to any state change.

src/presence/__tests__/presence-testlib.js

@@ -5,11 +5,20 @@ import { reducer } from '../presenceModel';
 import * as eg from '../../__tests__/lib/exampleData';
 import { objectFromEntries } from '../../jsBackport';
 
-export function makePresenceState(data: $ReadOnlyArray<[User, UserPresence]>): PresenceState {
+export function makePresenceState(
+  data: $ReadOnlyArray<[User, UserPresence]>,
+  args?: {|
+    +offlineThresholdSeconds?: number,
+    +pingIntervalSeconds?: number,
+  |},
+): PresenceState {
+  const { offlineThresholdSeconds, pingIntervalSeconds } = args ?? {};
   return reducer(
     undefined,
     eg.mkActionRegisterComplete({
       presences: objectFromEntries(data.map(([user, presence]) => [user.email, presence])),
+      server_presence_offline_threshold_seconds: offlineThresholdSeconds ?? 140,
+      server_presence_ping_interval_seconds: pingIntervalSeconds ?? 60,
     }),
   );
 }

src/presence/presenceModel.js

@@ -54,6 +54,7 @@ const presenceStatusGeq = (a: PresenceStatus, b: PresenceStatus): boolean => {
   }
 };
 
+// TODO(#5669): get OFFLINE_THRESHOLD_SECS from PresenceState
 const OFFLINE_THRESHOLD_SECS = 140;
 
 /**
@@ -198,6 +199,8 @@ export const getPresenceStatusForUserId = (
 //
 
 const initialState: PresenceState = {
+  pingIntervalSeconds: 60,
+  offlineThresholdSeconds: 140,
   byEmail: Immutable.Map(),
 };
 
@@ -211,6 +214,8 @@ export function reducer(
 
     case REGISTER_COMPLETE:
       return {
+        pingIntervalSeconds: action.data.server_presence_ping_interval_seconds ?? 60,
+        offlineThresholdSeconds: action.data.server_presence_offline_threshold_seconds ?? 140,
         byEmail: Immutable.Map(action.data.presences),
       };
 

src/reduxTypes.js

@@ -218,6 +218,8 @@ export type OutboxState = $ReadOnlyArray<Outbox>;
  *   presence status.
  */
 export type PresenceState = $ReadOnly<{|
+  pingIntervalSeconds: number,
+  offlineThresholdSeconds: number,
   byEmail: Immutable.Map<string, UserPresence>,
 |}>;
 

src/storage/migrations.js

@@ -512,7 +512,7 @@ const migrationsInner: {| [string]: (LessPartialState) => LessPartialState |} =
   // Change `state.mute` data structure: was a plain JS Map.
   '59': dropCache,
 
-  // Changed `state.presence`, but no migration because that's in `discardKeys`.
+  // Changed `state.presence` (twice), but no migration because that's in `discardKeys`.
 
   // TIP: When adding a migration, consider just using `dropCache`.
   //   (See its jsdoc for guidance on when that's the right answer.)