fix: search results limit

app/components/Package/Card.vue

@@ -154,7 +154,6 @@ const numberFormatter = useNumberFormatter()
     </div>
 
     <ul
-      role="list"
       v-if="result.package.keywords?.length"
       :aria-label="$t('package.card.keywords')"
       class="relative z-10 flex flex-wrap gap-1.5 mt-3 pt-3 border-t border-border list-none m-0 p-0 pointer-events-none items-center"

app/components/PaginationControls.vue

@@ -2,6 +2,8 @@
 import type { PageSize, PaginationMode, ViewMode } from '#shared/types/preferences'
 import { PAGE_SIZE_OPTIONS } from '#shared/types/preferences'
 
+const ALL_PAGES_VISIBLE_TRESHOLD = 7
+
 const props = defineProps<{
   totalItems: number
   /** When in table view, force pagination mode (no infinite scroll for tables) */
@@ -65,7 +67,7 @@ const visiblePages = computed(() => {
   const current = currentPage.value
   const pages: (number | 'ellipsis')[] = []
 
-  if (total <= 7) {
+  if (total <= ALL_PAGES_VISIBLE_TRESHOLD) {
     // Show all pages
     for (let i = 1; i <= total; i++) {
       pages.push(i)
@@ -99,6 +101,11 @@ const visiblePages = computed(() => {
   return pages
 })
 
+// disable last page button to prevent TOO MANY REQUESTS error
+function isPageButtonDisabled(page: number): boolean {
+  return totalPages.value > ALL_PAGES_VISIBLE_TRESHOLD && page > currentPage.value + 2
+}
+
 function handlePageSizeChange(event: Event) {
   const target = event.target as HTMLSelectElement
   const value = target.value
@@ -184,7 +191,7 @@ function handlePageSizeChange(event: Event) {
         <!-- Previous button -->
         <button
           type="button"
-          class="p-1.5 rounded hover:bg-bg-muted text-fg-muted hover:text-fg disabled:opacity-40 disabled:cursor-not-allowed transition-colors duration-200 focus-visible:ring-2 focus-visible:ring-fg focus-visible:ring-offset-1"
+          class="p-1.5 rounded text-fg-muted transition-colors duration-200 hover:(text-fg bg-bg-muted) focus-visible:(ring-2 ring-fg ring-offset-1) disabled:(opacity-40 cursor-not-allowed)"
           :disabled="!canGoPrev"
           :aria-label="$t('filters.pagination.previous')"
           @click="goPrev"
@@ -198,11 +205,10 @@ function handlePageSizeChange(event: Event) {
           <button
             v-else
             type="button"
-            class="min-w-[32px] h-8 px-2 font-mono text-sm rounded transition-colors duration-200 focus-visible:ring-2 focus-visible:ring-fg focus-visible:ring-offset-1"
+            :disabled="isPageButtonDisabled(page)"
+            class="min-w-[32px] h-8 px-2 font-mono text-sm rounded transition-colors duration-200 focus-visible:(ring-2 ring-fg ring-offset-1) disabled:(opacity-40 cursor-not-allowed)"
             :class="
-              page === currentPage
-                ? 'bg-fg text-bg'
-                : 'text-fg-muted hover:text-fg hover:bg-bg-muted'
+              page === currentPage ? 'bg-fg text-bg' : 'text-fg-muted hover:(text-fg bg-bg-muted)'
             "
             :aria-current="page === currentPage ? 'page' : undefined"
             @click="goToPage(page)"
@@ -214,7 +220,7 @@ function handlePageSizeChange(event: Event) {
         <!-- Next button -->
         <button
           type="button"
-          class="p-1.5 rounded hover:bg-bg-muted text-fg-muted hover:text-fg disabled:opacity-40 disabled:cursor-not-allowed transition-colors duration-200 focus-visible:ring-2 focus-visible:ring-fg focus-visible:ring-offset-1"
+          class="p-1.5 rounded text-fg-muted transition-colors duration-200 hover:(text-fg bg-bg-muted) focus-visible:(ring-2 ring-fg ring-offset-1) disabled:(opacity-40 cursor-not-allowed)"
           :disabled="!canGoNext"
           :aria-label="$t('filters.pagination.next')"
           @click="goNext"

app/composables/npm/search-utils.ts

@@ -1,3 +1,5 @@
+import type { NpmSearchResult, PackageMetaResponse, SearchResponse } from '#shared/types'
+
 export function metaToSearchResult(meta: PackageMetaResponse): NpmSearchResult {
   return {
     package: {
@@ -18,21 +20,16 @@ export function metaToSearchResult(meta: PackageMetaResponse): NpmSearchResult {
   }
 }
 
-export function emptySearchResponse(): NpmSearchResponse {
+export function emptySearchResponse(): SearchResponse {
   return {
     objects: [],
+    totalUnlimited: 0,
     total: 0,
     isStale: false,
     time: new Date().toISOString(),
   }
 }
 
-export interface SearchSuggestion {
-  type: 'user' | 'org'
-  name: string
-  exists: boolean
-}
-
 export type SuggestionIntent = 'user' | 'org' | 'both' | null
 
 export function isValidNpmName(name: string): boolean {

app/composables/npm/useAlgoliaSearch.ts

@@ -2,7 +2,7 @@ import {
   liteClient as algoliasearch,
   type LiteClient,
   type SearchQuery,
-  type SearchResponse,
+  type SearchResponse as AlgoliaSearchResponse,
 } from 'algoliasearch/lite'
 
 let _searchClient: LiteClient | null = null
@@ -156,7 +156,7 @@ export function useAlgoliaSearch() {
       ],
     })
 
-    const response = results[0] as SearchResponse<AlgoliaHit> | undefined
+    const response = results[0] as AlgoliaSearchResponse<AlgoliaHit> | undefined
     if (!response) {
       throw new Error('Algolia returned an empty response')
     }
@@ -201,7 +201,7 @@ export function useAlgoliaSearch() {
         ],
       })
 
-      const response = results[0] as SearchResponse<AlgoliaHit> | undefined
+      const response = results[0] as AlgoliaSearchResponse<AlgoliaHit> | undefined
       if (!response) break
 
       serverTotal = response.nbHits ?? 0
@@ -317,7 +317,7 @@ export function useAlgoliaSearch() {
 
     const { results } = await client.search({ requests })
 
-    const mainResponse = results[0] as SearchResponse<AlgoliaHit> | undefined
+    const mainResponse = results[0] as AlgoliaSearchResponse<AlgoliaHit> | undefined
     if (!mainResponse) {
       throw new Error('Algolia returned an empty response')
     }
@@ -331,21 +331,23 @@ export function useAlgoliaSearch() {
 
     let orgExists = false
     if (orgQueryIndex >= 0 && checks?.name) {
-      const orgResponse = results[orgQueryIndex] as SearchResponse<AlgoliaHit> | undefined
+      const orgResponse = results[orgQueryIndex] as AlgoliaSearchResponse<AlgoliaHit> | undefined
       const scopePrefix = `@${checks.name.toLowerCase()}/`
       orgExists =
         orgResponse?.hits?.some(h => h.name?.toLowerCase().startsWith(scopePrefix)) ?? false
     }
 
     let userExists = false
     if (userQueryIndex >= 0) {
-      const userResponse = results[userQueryIndex] as SearchResponse<AlgoliaHit> | undefined
+      const userResponse = results[userQueryIndex] as AlgoliaSearchResponse<AlgoliaHit> | undefined
       userExists = (userResponse?.nbHits ?? 0) > 0
     }
 
     let packageExists: boolean | null = null
     if (packageQueryIndex >= 0) {
-      const pkgResponse = results[packageQueryIndex] as SearchResponse<AlgoliaHit> | undefined
+      const pkgResponse = results[packageQueryIndex] as
+        | AlgoliaSearchResponse<AlgoliaHit>
+        | undefined
       packageExists = (pkgResponse?.nbHits ?? 0) > 0
     }