From 40c8a8d38abf1126b13cb75e99e5ea2054890b2d Mon Sep 17 00:00:00 2001
From: Denozordec <shatskiy.blog@gmail.com>
Date: Wed, 8 Apr 2026 15:42:50 +0700
Subject: [PATCH] feat: implement countAll function for API data retrieval

Added a new `countAll` function to streamline the retrieval of item counts from various API endpoints. This function replaces individual API calls with a unified approach, enhancing performance and code clarity. Updated the onMount lifecycle method to utilize this new function for fetching counts of modules, revisions, peers, and speakers, improving data handling efficiency.
---
 web/src/routes/+page.svelte | 25 +++++++++++++++----------
 1 file changed, 15 insertions(+), 10 deletions(-)

diff --git a/web/src/routes/+page.svelte b/web/src/routes/+page.svelte
index eddd7c5..f40ed23 100644
--- a/web/src/routes/+page.svelte
+++ b/web/src/routes/+page.svelte
@@ -1,6 +1,6 @@
 <script lang="ts">
 	import { onMount } from 'svelte';
-	import { apiJSON, apiFetch } from '$lib/api/client.js';
+	import { apiJSON, apiFetch, apiPageAll } from '$lib/api/client.js';
 	import type { ModulesResponse, RevisionsResponse, PeersResponse, SpeakersResponse, JobsResponse } from '$lib/api/types.js';
 	import { Badge } from '$lib/components/ui/badge/index.js';
 	import { Button } from '$lib/components/ui/button/index.js';
@@ -26,6 +26,11 @@ const resolve = (path: string) => path as any;
 	let runningJobs = $state(0);
 	let loading = $state(true);
 
+	async function countAll(path: string): Promise<number> {
+		const items = await apiPageAll<unknown>(path, 500);
+		return items.length;
+	}
+
 	onMount(async () => {
 		loading = true;
 		try {
@@ -35,17 +40,17 @@ const resolve = (path: string) => path as any;
 			healthy = false;
 		}
 		try {
-			const [m, r, p, s, j] = await Promise.all([
-				apiJSON<ModulesResponse>('/v1/modules?limit=1'),
-				apiJSON<RevisionsResponse>('/v1/revisions?limit=1'),
-				apiJSON<PeersResponse>('/v1/peers?limit=1'),
-				apiJSON<SpeakersResponse>('/v1/speakers?limit=1'),
+			const [mCount, rCount, pCount, sCount, j] = await Promise.all([
+				countAll('/v1/modules'),
+				countAll('/v1/revisions'),
+				countAll('/v1/peers'),
+				countAll('/v1/speakers'),
 				apiJSON<JobsResponse>('/v1/jobs?limit=100')
 			]);
-			modules = m.has_more ? 99 : (m.items?.length ?? 0);
-			revisions = r.has_more ? 99 : (r.items?.length ?? 0);
-			peers = p.has_more ? 99 : (p.items?.length ?? 0);
-			speakers = s.has_more ? 99 : (s.items?.length ?? 0);
+			modules = mCount;
+			revisions = rCount;
+			peers = pCount;
+			speakers = sCount;
 			runningJobs = (j.items ?? []).filter((i) => i.status === 'running' || i.status === 'queued').length;
 		} catch {/* ignore */}
 		loading = false;