Reset idle alarm when a new Freighter surface mounts unlocked

Previously the idle auto-lock alarm only rearmed on user input events
(mousedown, keydown, touchstart, wheel). When a dApp triggered a flow
that opened a new popup near the end of the idle window, the user
could be locked out mid-flow before they ever interacted with the new
surface.

Ping USER_ACTIVITY once when useActivityPing mounts in the unlocked
state. Opening any Freighter surface (popup, sidebar, fullscreen) is
itself an act of user intent and is enough signal to treat the user
as active. The existing throttle and isUnlocked guard naturally
suppress the mount ping when the wallet is locked or when an input
event fires within the same throttle window.

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>

Author: Copilot

extension/src/popup/helpers/hooks/__tests__/useActivityPing.test.ts

@@ -92,4 +92,34 @@ describe("useActivityPing", () => {
 
     addSpy.mockRestore();
   });
+
+  it("sends a USER_ACTIVITY ping on mount when unlocked", () => {
+    renderHook(() => useActivityPing(true));
+
+    expect(sendMessageToBackground).toHaveBeenCalledTimes(1);
+    expect(sendMessageToBackground).toHaveBeenCalledWith({
+      type: SERVICE_TYPES.USER_ACTIVITY,
+      activePublicKey: "",
+    });
+  });
+
+  it("throttles event-driven pings against the mount ping", () => {
+    renderHook(() => useActivityPing(true));
+
+    // Mount ping has already fired at t=10_000; event within the
+    // throttle window is suppressed.
+    window.dispatchEvent(new MouseEvent("mousedown"));
+    expect(sendMessageToBackground).toHaveBeenCalledTimes(1);
+
+    // Past the throttle window, events resume firing.
+    jest.setSystemTime(15_000);
+    window.dispatchEvent(new MouseEvent("mousedown"));
+    expect(sendMessageToBackground).toHaveBeenCalledTimes(2);
+  });
+
+  it("does not ping on mount when locked", () => {
+    renderHook(() => useActivityPing(false));
+
+    expect(sendMessageToBackground).not.toHaveBeenCalled();
+  });
 });

extension/src/popup/helpers/hooks/useActivityPing.ts

@@ -23,16 +23,19 @@ const ACTIVITY_EVENTS = [
  * Uses a leading-edge throttle so a single ping fires on the first
  * event in each `PING_THROTTLE_MS` window — accurate to ~8 % on the
  * 1-minute preset and effectively noise at higher presets.
+ *
+ * Also pings once on mount, so opening a new Freighter surface (popup,
+ * sidebar, fullscreen) while the wallet is unlocked itself counts as
+ * activity. This prevents the wallet from auto-locking mid-flow when a
+ * dApp triggers a new popup near the end of the idle window.
  */
 export const useActivityPing = (isUnlocked: boolean) => {
   useEffect(() => {
     if (!isUnlocked) return undefined;
 
     let lastPingAt = 0;
-    const handler = () => {
-      const now = Date.now();
-      if (now - lastPingAt < PING_THROTTLE_MS) return;
-      lastPingAt = now;
+    const ping = () => {
+      lastPingAt = Date.now();
       void sendMessageToBackground({
         type: SERVICE_TYPES.USER_ACTIVITY,
         // `USER_ACTIVITY` is account-agnostic — the background only
@@ -46,6 +49,20 @@ export const useActivityPing = (isUnlocked: boolean) => {
       });
     };
 
+    // Ping once on mount so opening a fresh Freighter surface (popup,
+    // sidebar, fullscreen) while the wallet is unlocked counts as
+    // activity and rearms the idle alarm. Without this, a user who
+    // triggers a dApp flow that opens a new popup near the end of the
+    // idle window can be locked out mid-flow before they ever interact
+    // with the new surface.
+    ping();
+
+    const handler = () => {
+      const now = Date.now();
+      if (now - lastPingAt < PING_THROTTLE_MS) return;
+      ping();
+    };
+
     for (const evt of ACTIVITY_EVENTS) {
       window.addEventListener(evt, handler, { passive: true });
     }