Fix HistoryPage stale appliedQuery effect and add mount query regression test

frontend/src/pages/HistoryPage.tsx

@@ -6,7 +6,7 @@
  * Rows with repeatedly-banned IPs are highlighted in amber.
  */
 
-import { useEffect, useState } from "react";
+import { useEffect, useRef, useState } from "react";
 import {
   Badge,
   Button,
@@ -199,10 +199,14 @@ export function HistoryPage(): React.JSX.Element {
   const [originFilter, setOriginFilter] = useState<BanOriginFilter>("all");
   const [jailFilter, setJailFilter] = useState("");
   const [ipFilter, setIpFilter] = useState("");
-  const [appliedQuery, setAppliedQuery] = useState<HistoryQuery>({
+  const defaultQuery: HistoryQuery = {
+    range: "7d",
     source: "archive",
     page_size: PAGE_SIZE,
-  });
+    page: 1,
+  };
+  const [appliedQuery, setAppliedQuery] = useState<HistoryQuery>(defaultQuery);
+  const appliedQueryRef = useRef<HistoryQuery>(defaultQuery);
 
   // Per-IP detail navigation
   const [selectedIp, setSelectedIp] = useState<string | null>(null);
@@ -221,13 +225,14 @@ export function HistoryPage(): React.JSX.Element {
       page_size: PAGE_SIZE,
     };
 
-    if (areHistoryQueriesEqual(nextQuery, appliedQuery)) {
+    if (areHistoryQueriesEqual(nextQuery, appliedQueryRef.current)) {
       return;
     }
 
     setPage(1);
     setAppliedQuery(nextQuery);
-  }, [range, originFilter, jailFilter, ipFilter, setPage, appliedQuery]);
+    appliedQueryRef.current = nextQuery;
+  }, [range, originFilter, jailFilter, ipFilter, setPage]);
 
   const totalPages = Math.max(1, Math.ceil(total / PAGE_SIZE));
 

frontend/src/pages/__tests__/HistoryPage.test.tsx

@@ -28,7 +28,7 @@ vi.mock("../../components/WorldMap", () => ({
 }));
 
 vi.mock("../../api/config", () => ({
-  fetchMapColorThresholds: async () => ({
+  fetchMapColorThresholds: () => ({
     threshold_low: 10,
     threshold_medium: 50,
     threshold_high: 100,
@@ -74,4 +74,18 @@ describe("HistoryPage", () => {
       expect(lastQuery).toMatchObject({ origin: "blocklist" });
     });
   });
+
+  it("never calls history without the default range on mount", async () => {
+    render(
+      <FluentProvider theme={webLightTheme}>
+        <HistoryPage />
+      </FluentProvider>,
+    );
+
+    await waitFor(() => {
+      expect(lastQuery?.range).toBe("7d");
+    });
+
+    expect(mockUseHistory.mock.calls.every((call) => call[0].range === "7d")).toBe(true);
+  });
 });