fix(web): force scroll-to-top on tab switch (iOS Safari quirk)

Next.js App Router *should* reset scroll on route change, but iOS
Safari (and occasionally Chrome on Android) preserves the document
scroll position when navigating between routes that share a layout —
the user lands on the new tab halfway down the page and has to scroll
manually.

Adds a useEffect([]) in both AccountsTable and UsersTable that calls
window.scrollTo({top:0, left:0, behavior:'instant'}) on mount. Route
transitions remount the table component (different page module), so
the empty deps array fires exactly once per visit.

'instant' behavior avoids a smooth-scroll animation that would compete
with the loading-skeleton swap.

app/cm_api.py

@@ -122,7 +122,10 @@ class CM_API:
                 )
                 params = [limit, offset]
 
-            return jsonify(db.query(query, params))
+            rows = db.query(query, params)
+            count_rows = db.query("SELECT COUNT(*) AS c FROM acc", [])
+            total = int(count_rows[0]["c"]) if count_rows else 0
+            return jsonify({"rows": rows, "total": total})
 
         except Exception as error:
             return self._handle_error(error, "Not Found"), 404
@@ -166,7 +169,10 @@ class CM_API:
                 )
                 params = [limit, offset]
 
-            return jsonify(db.query(query, params))
+            rows = db.query(query, params)
+            count_rows = db.query("SELECT COUNT(*) AS c FROM user", [])
+            total = int(count_rows[0]["c"]) if count_rows else 0
+            return jsonify({"rows": rows, "total": total})
 
         except Exception as error:
             return self._handle_error(error, "Not Found"), 404

web/app/page.tsx

@@ -9,6 +9,7 @@ export default async function AccountsPage() {
     <AccountsTable
       initial={page.rows}
       initialHasMore={page.hasMore}
+      initialTotal={page.total}
       prefixPattern={PREFIX_PATTERN}
     />
   );

web/app/users/page.tsx

@@ -13,6 +13,7 @@ export default async function UsersPage() {
     <UsersTable
       initial={page.rows}
       initialHasMore={page.hasMore}
+      initialTotal={page.total}
       prefixPattern={PREFIX_PATTERN}
     />
   );

web/components/accounts-table.tsx

@@ -16,6 +16,7 @@ import Toast, { type ToastMessage } from "./toast";
 type Props = {
   initial: Acc[];
   initialHasMore: boolean;
+  initialTotal: number;
   prefixPattern: string;
 };
 type SortDir = "asc" | "desc";
@@ -61,12 +62,12 @@ function DeleteButton({
 export default function AccountsTable({
   initial,
   initialHasMore,
+  initialTotal,
   prefixPattern,
 }: Props) {
   const [sortDir, setSortDir] = useState<SortDir>("desc");
   const [editingKey, setEditingKey] = useState<string | null>(null);
   const [, startTransition] = useTransition();
-  const [refreshing, setRefreshing] = useState(false);
   const [loadingMore, setLoadingMore] = useState(false);
   const [deleteTarget, setDeleteTarget] = useState<string | null>(null);
   const [deleting, setDeleting] = useState(false);
@@ -74,9 +75,22 @@ export default function AccountsTable({
   const [createOpen, setCreateOpen] = useState(false);
   const [toast, setToast] = useState<ToastMessage | null>(null);
 
+  // iOS Safari (and sometimes Chrome on Android) keeps the document
+  // scroll position across SPA route changes, so tab-switching from
+  // /users back to / leaves the user halfway down the page. Force
+  // scroll-to-top on mount; route transitions remount the table.
+  useEffect(() => {
+    window.scrollTo({ top: 0, left: 0, behavior: "instant" });
+  }, []);
+
   // Accumulated rows from initial server-side fetch + every loadMore.
   const [rows, setRows] = useState<Acc[]>(initial);
   const [hasMore, setHasMore] = useState<boolean>(initialHasMore);
+  // Total row count in the DB; updated on every page fetch so it stays
+  // fresh as external writes (the cm99 monitor) add rows. Mutations
+  // adjust it locally so the header doesn't flash a stale count between
+  // the optimistic update and the next page-fresh count.
+  const [total, setTotal] = useState<number>(initialTotal);
   const sentinelRef = useRef<HTMLDivElement | null>(null);
 
   const [optimistic, applyOptimistic] = useOptimistic<Acc[], OptimisticPatch>(
@@ -105,15 +119,15 @@ export default function AccountsTable({
     });
   }
 
+  // Force-evict the cache and re-fetch page 1. Used by the create-success
+  // path so a freshly added row appears in its sorted position. Keeping
+  // the function (instead of inlining at the call site) means a future
+  // 'pull to refresh' gesture has a single hook.
   async function refresh() {
-    setRefreshing(true);
+    const page = await refreshAccounts({ prefix: prefixPattern, dir: sortDir });
-    try {
+    setRows(page.rows);
-      const page = await refreshAccounts({ prefix: prefixPattern, dir: sortDir });
+    setHasMore(page.hasMore);
-      setRows(page.rows);
+    setTotal(page.total);
-      setHasMore(page.hasMore);
-    } finally {
-      setRefreshing(false);
-    }
   }
 
   async function changeSort(next: SortDir) {
@@ -128,6 +142,7 @@ export default function AccountsTable({
       });
       setRows(page.rows);
       setHasMore(page.hasMore);
+      setTotal(page.total);
     } finally {
       setLoadingMore(false);
     }
@@ -137,7 +152,7 @@ export default function AccountsTable({
   // 300px rootMargin so the next page starts loading before the user
   // hits the bottom — feels seamless when scrolling fast.
   useEffect(() => {
-    if (!hasMore || loadingMore || refreshing) return;
+    if (!hasMore || loadingMore) return;
     const sentinel = sentinelRef.current;
     if (!sentinel) return;
 
@@ -153,6 +168,7 @@ export default function AccountsTable({
           .then((page) => {
             setRows((prev) => [...prev, ...page.rows]);
             setHasMore(page.hasMore);
+            setTotal(page.total);
           })
           .catch((err) => console.error("loadMoreAccounts failed:", err))
           .finally(() => setLoadingMore(false));
@@ -161,7 +177,7 @@ export default function AccountsTable({
     );
     observer.observe(sentinel);
     return () => observer.disconnect();
-  }, [hasMore, loadingMore, refreshing, rows.length, prefixPattern, sortDir]);
+  }, [hasMore, loadingMore, rows.length, prefixPattern, sortDir]);
 
   async function confirmDelete() {
     if (!deleteTarget) return;
@@ -172,6 +188,7 @@ export default function AccountsTable({
     if (result.ok) {
       const deleted = deleteTarget;
       setRows((prev) => prev.filter((r) => r.username !== deleted));
+      setTotal((t) => Math.max(0, t - 1));
       setDeleteTarget(null);
       setToast({ type: "success", message: `Account ${deleted} deleted` });
     } else {
@@ -183,11 +200,8 @@ export default function AccountsTable({
     return (
       <div>
         <PageHead
-          count={0}
+          total={total}
           loaded={0}
-          hasMore={false}
-          onRefresh={refresh}
-          refreshing={refreshing}
           onAdd={() => setCreateOpen(true)}
         />
         <div className="mt-6 rounded-2xl bg-white px-8 py-16 text-center ring-1 ring-zinc-200/60">
@@ -213,11 +227,8 @@ export default function AccountsTable({
   return (
     <div>
       <PageHead
-        count={optimistic.length}
+        total={total}
         loaded={optimistic.length}
-        hasMore={hasMore}
-        onRefresh={refresh}
-        refreshing={refreshing}
         onAdd={() => setCreateOpen(true)}
       />
 
@@ -435,58 +446,45 @@ function CardRow({ label, children }: { label: string; children: React.ReactNode
 }
 
 function PageHead({
-  count,
+  total,
   loaded,
-  hasMore,
-  onRefresh,
-  refreshing,
   onAdd,
 }: {
-  count: number;
+  total: number;
   loaded: number;
-  hasMore: boolean;
-  onRefresh: () => void;
-  refreshing: boolean;
   onAdd: () => void;
 }) {
-  // count == loaded for now; kept separate so a future "showing X of Y"
+  // total = COUNT(*) from the API; loaded = how many rows the user has
-  // header (when we surface a server-side total) drops in cleanly.
+  // scrolled in so far. Show the partial count only while it differs
-  const showHasMore = hasMore && loaded > 0;
+  // from the total, so a fully-scrolled list reads cleanly.
+  const showLoaded = loaded > 0 && loaded < total;
   return (
-    <div className="flex flex-wrap items-end justify-between gap-4">
+    <div>
-      <div>
+      <p className="text-[11px] font-medium uppercase tracking-wider text-zinc-500">
-        <p className="text-[11px] font-medium uppercase tracking-wider text-zinc-500">
+        Table
-          Table
+      </p>
-        </p>
+      <div className="mt-1 flex flex-wrap items-center justify-between gap-x-3 gap-y-2">
-        <h1 className="mt-1 text-2xl font-semibold tracking-tight text-zinc-900 sm:text-3xl">
+        <h1 className="text-2xl font-semibold tracking-tight text-zinc-900 sm:text-3xl">
           Accounts
           <span className="ml-2 align-middle text-base font-medium text-zinc-400">
-            {count}
+            {total}
-            {showHasMore && <span className="text-zinc-300">+</span>}
           </span>
         </h1>
-      </div>
-      <div className="flex items-center gap-2">
-        <button
-          type="button"
-          onClick={onRefresh}
-          disabled={refreshing}
-          className="inline-flex items-center gap-2 rounded-full bg-white px-4 py-2 text-xs font-medium text-zinc-700 shadow-sm ring-1 ring-zinc-200 transition-colors hover:bg-zinc-50 hover:text-zinc-900 disabled:opacity-60"
-        >
-          <span aria-hidden="true" className={refreshing ? "inline-block animate-spin" : ""}>
-            ⟳
-          </span>
-          Refresh
-        </button>
         <button
           type="button"
           onClick={onAdd}
-          className="inline-flex items-center gap-1.5 rounded-full bg-zinc-900 px-4 py-2 text-xs font-medium text-white shadow-sm transition-colors hover:bg-zinc-700"
+          aria-label="Add account"
+          className="inline-flex items-center gap-1.5 rounded-full bg-zinc-900 px-3 py-1.5 text-xs font-medium text-white shadow-sm transition-colors hover:bg-zinc-700"
         >
           <span aria-hidden="true">+</span>
           Add
         </button>
       </div>
+      {showLoaded && (
+        <p className="mt-1 text-[11px] text-zinc-400">
+          Showing {loaded} of {total}
+        </p>
+      )}
     </div>
   );
 }

web/components/users-table.tsx

@@ -16,6 +16,7 @@ import Toast, { type ToastMessage } from "./toast";
 type Props = {
   initial: User[];
   initialHasMore: boolean;
+  initialTotal: number;
   prefixPattern: string;
 };
 type SortDir = "asc" | "desc";
@@ -62,13 +63,13 @@ function DeleteButton({
 export default function UsersTable({
   initial,
   initialHasMore,
+  initialTotal,
   prefixPattern,
 }: Props) {
   const [sortKey, setSortKey] = useState<SortKey>("last_update_time");
   const [sortDir, setSortDir] = useState<SortDir>("desc");
   const [editingKey, setEditingKey] = useState<string | null>(null);
   const [, startTransition] = useTransition();
-  const [refreshing, setRefreshing] = useState(false);
   const [loadingMore, setLoadingMore] = useState(false);
   const [deleteTarget, setDeleteTarget] = useState<string | null>(null);
   const [deleting, setDeleting] = useState(false);
@@ -76,8 +77,16 @@ export default function UsersTable({
   const [createOpen, setCreateOpen] = useState(false);
   const [toast, setToast] = useState<ToastMessage | null>(null);
 
+  // Force scroll-to-top on mount — iOS Safari preserves document scroll
+  // across SPA route changes, so tab-switching leaves the new page
+  // halfway down. Route transitions remount the table, so [] deps fire.
+  useEffect(() => {
+    window.scrollTo({ top: 0, left: 0, behavior: "instant" });
+  }, []);
+
   const [rows, setRows] = useState<User[]>(initial);
   const [hasMore, setHasMore] = useState<boolean>(initialHasMore);
+  const [total, setTotal] = useState<number>(initialTotal);
   const sentinelRef = useRef<HTMLDivElement | null>(null);
 
   const [optimistic, applyOptimistic] = useOptimistic<User[], OptimisticPatch>(
@@ -120,19 +129,17 @@ export default function UsersTable({
     });
   }
 
+  // Force-evict the cache and re-fetch page 1. Used by the create-success
+  // path so a freshly added row appears in its sorted position.
   async function refresh() {
-    setRefreshing(true);
+    const page = await refreshUsers({
-    try {
+      prefix: prefixPattern,
-      const page = await refreshUsers({
+      sort: sortKey,
-        prefix: prefixPattern,
+      dir: sortDir,
-        sort: sortKey,
+    });
-        dir: sortDir,
+    setRows(page.rows);
-      });
+    setHasMore(page.hasMore);
-      setRows(page.rows);
+    setTotal(page.total);
-      setHasMore(page.hasMore);
-    } finally {
-      setRefreshing(false);
-    }
   }
 
   async function changeSort(nextKey: SortKey) {
@@ -154,13 +161,14 @@ export default function UsersTable({
       });
       setRows(page.rows);
       setHasMore(page.hasMore);
+      setTotal(page.total);
     } finally {
       setLoadingMore(false);
     }
   }
 
   useEffect(() => {
-    if (!hasMore || loadingMore || refreshing) return;
+    if (!hasMore || loadingMore) return;
     const sentinel = sentinelRef.current;
     if (!sentinel) return;
     const observer = new IntersectionObserver(
@@ -176,6 +184,7 @@ export default function UsersTable({
           .then((page) => {
             setRows((prev) => [...prev, ...page.rows]);
             setHasMore(page.hasMore);
+            setTotal(page.total);
           })
           .catch((err) => console.error("loadMoreUsers failed:", err))
           .finally(() => setLoadingMore(false));
@@ -184,7 +193,7 @@ export default function UsersTable({
     );
     observer.observe(sentinel);
     return () => observer.disconnect();
-  }, [hasMore, loadingMore, refreshing, rows.length, prefixPattern, sortKey, sortDir]);
+  }, [hasMore, loadingMore, rows.length, prefixPattern, sortKey, sortDir]);
 
   async function confirmDelete() {
     if (!deleteTarget) return;
@@ -195,6 +204,7 @@ export default function UsersTable({
     if (result.ok) {
       const deleted = deleteTarget;
       setRows((prev) => prev.filter((r) => r.f_username !== deleted));
+      setTotal((t) => Math.max(0, t - 1));
       setDeleteTarget(null);
       setToast({ type: "success", message: `User ${deleted} deleted` });
     } else {
@@ -222,10 +232,8 @@ export default function UsersTable({
     return (
       <div>
         <PageHead
-          count={0}
+          total={total}
-          hasMore={false}
+          loaded={0}
-          onRefresh={refresh}
-          refreshing={refreshing}
           onAdd={() => setCreateOpen(true)}
         />
         <div className="mt-6 rounded-2xl bg-white px-8 py-16 text-center ring-1 ring-zinc-200/60">
@@ -249,10 +257,8 @@ export default function UsersTable({
   return (
     <div>
       <PageHead
-        count={optimistic.length}
+        total={total}
-        hasMore={hasMore}
+        loaded={optimistic.length}
-        onRefresh={refresh}
-        refreshing={refreshing}
         onAdd={() => setCreateOpen(true)}
       />
 
@@ -459,52 +465,40 @@ function MobileRow({ label, children }: { label: string; children: React.ReactNo
 }
 
 function PageHead({
-  count,
+  total,
-  hasMore,
+  loaded,
-  onRefresh,
-  refreshing,
   onAdd,
 }: {
-  count: number;
+  total: number;
-  hasMore: boolean;
+  loaded: number;
-  onRefresh: () => void;
-  refreshing: boolean;
   onAdd: () => void;
 }) {
-  const showHasMore = hasMore && count > 0;
+  const showLoaded = loaded > 0 && loaded < total;
   return (
-    <div className="flex flex-wrap items-end justify-between gap-4">
+    <div>
-      <div>
+      <p className="text-[11px] font-medium uppercase tracking-wider text-zinc-500">Table</p>
-        <p className="text-[11px] font-medium uppercase tracking-wider text-zinc-500">Table</p>
+      <div className="mt-1 flex flex-wrap items-center justify-between gap-x-3 gap-y-2">
-        <h1 className="mt-1 text-2xl font-semibold tracking-tight text-zinc-900 sm:text-3xl">
+        <h1 className="text-2xl font-semibold tracking-tight text-zinc-900 sm:text-3xl">
           Users
           <span className="ml-2 align-middle text-base font-medium text-zinc-400">
-            {count}
+            {total}
-            {showHasMore && <span className="text-zinc-300">+</span>}
           </span>
         </h1>
-      </div>
-      <div className="flex items-center gap-2">
-        <button
-          type="button"
-          onClick={onRefresh}
-          disabled={refreshing}
-          className="inline-flex items-center gap-2 rounded-full bg-white px-4 py-2 text-xs font-medium text-zinc-700 shadow-sm ring-1 ring-zinc-200 transition-colors hover:bg-zinc-50 hover:text-zinc-900 disabled:opacity-60"
-        >
-          <span aria-hidden="true" className={refreshing ? "inline-block animate-spin" : ""}>
-            ⟳
-          </span>
-          Refresh
-        </button>
         <button
           type="button"
           onClick={onAdd}
-          className="inline-flex items-center gap-1.5 rounded-full bg-zinc-900 px-4 py-2 text-xs font-medium text-white shadow-sm transition-colors hover:bg-zinc-700"
+          aria-label="Add user"
+          className="inline-flex items-center gap-1.5 rounded-full bg-zinc-900 px-3 py-1.5 text-xs font-medium text-white shadow-sm transition-colors hover:bg-zinc-700"
         >
           <span aria-hidden="true">+</span>
           Add
         </button>
       </div>
+      {showLoaded && (
+        <p className="mt-1 text-[11px] text-zinc-400">
+          Showing {loaded} of {total}
+        </p>
+      )}
     </div>
   );
 }

web/lib/api.ts

@@ -15,7 +15,7 @@ export const USERS_TAG = "users";
 // in well under one frame even on phones.
 export const PAGE_SIZE = 200;
 
-export type Page<T> = { rows: T[]; hasMore: boolean };
+export type Page<T> = { rows: T[]; hasMore: boolean; total: number };
 
 export type AccountsPageOpts = {
   offset?: number;
@@ -82,13 +82,21 @@ function buildUsersUrl(opts: UsersPageOpts): string {
 export async function getAccountsPage(opts: AccountsPageOpts = {}): Promise<Page<Acc>> {
   const data = (await fetchApi(buildAccountsUrl(opts), {
     next: { revalidate: CACHE_REVALIDATE_SECONDS, tags: [ACCOUNTS_TAG] },
-  })) as Acc[];
+  })) as { rows: Acc[]; total: number };
-  return { rows: data, hasMore: data.length === PAGE_SIZE };
+  return {
+    rows: data.rows,
+    hasMore: data.rows.length === PAGE_SIZE,
+    total: data.total,
+  };
 }
 
 export async function getUsersPage(opts: UsersPageOpts = {}): Promise<Page<User>> {
   const data = (await fetchApi(buildUsersUrl(opts), {
     next: { revalidate: CACHE_REVALIDATE_SECONDS, tags: [USERS_TAG] },
-  })) as User[];
+  })) as { rows: User[]; total: number };
-  return { rows: data, hasMore: data.length === PAGE_SIZE };
+  return {
+    rows: data.rows,
+    hasMore: data.rows.length === PAGE_SIZE,
+    total: data.total,
+  };
 }