From 0463abb925e1d35f712d5066700fac1ecac737fb Mon Sep 17 00:00:00 2001
From: mkelvers <mikkelelvers@outlook.com>
Date: Tue, 14 Apr 2026 23:23:09 +0200
Subject: [PATCH] fix: remove jikan limit params

---
 internal/features/anime/handler.go |  2 +-
 internal/jikan/constants.go        |  2 --
 internal/jikan/search.go           |  8 ++++----
 internal/jikan/seasons.go          | 12 ++++++------
 4 files changed, 11 insertions(+), 13 deletions(-)

diff --git a/internal/features/anime/handler.go b/internal/features/anime/handler.go
index 9989c1a..58321dd 100644
--- a/internal/features/anime/handler.go
+++ b/internal/features/anime/handler.go
@@ -127,7 +127,7 @@ func (h *Handler) HandleAPICatalog(w http.ResponseWriter, r *http.Request) {
 	}
 
 	if fallbackPlaceholder {
-		templates.CatalogPlaceholderItems(jikan.ListPageSize).Render(r.Context(), w)
+		templates.CatalogPlaceholderItems(25).Render(r.Context(), w)
 		return
 	}
 
diff --git a/internal/jikan/constants.go b/internal/jikan/constants.go
index 6b8bbc8..4afdf62 100644
--- a/internal/jikan/constants.go
+++ b/internal/jikan/constants.go
@@ -2,6 +2,4 @@ package jikan
 
 import "time"
 
-const ListPageSize = 24
-
 const shortCacheTTL = time.Hour
diff --git a/internal/jikan/search.go b/internal/jikan/search.go
index b8afcaa..2a280e6 100644
--- a/internal/jikan/search.go
+++ b/internal/jikan/search.go
@@ -14,7 +14,7 @@ func (c *Client) Search(ctx context.Context, query string, page int) (SearchResu
 		page = 1
 	}
 
-	cacheKey := fmt.Sprintf("search:limit%d:%s:%d", ListPageSize, query, page)
+	cacheKey := fmt.Sprintf("search:%s:%d", query, page)
 	var cached SearchResult
 	if c.getCache(ctx, cacheKey, &cached) {
 		return cached, nil
@@ -24,7 +24,7 @@ func (c *Client) Search(ctx context.Context, query string, page int) (SearchResu
 	hasStale := c.getStaleCache(ctx, cacheKey, &stale)
 
 	var result SearchResponse
-	reqURL := fmt.Sprintf("%s/anime?q=%s&limit=%d&page=%d", c.baseURL, url.QueryEscape(query), ListPageSize, page)
+	reqURL := fmt.Sprintf("%s/anime?q=%s&page=%d", c.baseURL, url.QueryEscape(query), page)
 
 	if err := c.fetchWithRetry(ctx, reqURL, &result); err != nil {
 		if hasStale {
@@ -47,7 +47,7 @@ func (c *Client) GetTopAnime(ctx context.Context, page int) (TopAnimeResult, err
 	if page < 1 {
 		page = 1
 	}
-	cacheKey := fmt.Sprintf("top:limit%d:%d", ListPageSize, page)
+	cacheKey := fmt.Sprintf("top:%d", page)
 	var cached TopAnimeResult
 	if c.getCache(ctx, cacheKey, &cached) {
 		return cached, nil
@@ -57,7 +57,7 @@ func (c *Client) GetTopAnime(ctx context.Context, page int) (TopAnimeResult, err
 	hasStale := c.getStaleCache(ctx, cacheKey, &stale)
 
 	var result TopAnimeResponse
-	reqURL := fmt.Sprintf("%s/top/anime?filter=bypopularity&limit=%d&page=%d", c.baseURL, ListPageSize, page)
+	reqURL := fmt.Sprintf("%s/top/anime?page=%d", c.baseURL, page)
 
 	if err := c.fetchWithRetry(ctx, reqURL, &result); err != nil {
 		if hasStale {
diff --git a/internal/jikan/seasons.go b/internal/jikan/seasons.go
index 2a8ee66..aaf89c2 100644
--- a/internal/jikan/seasons.go
+++ b/internal/jikan/seasons.go
@@ -13,7 +13,7 @@ type ScheduleResult struct {
 
 func (c *Client) GetSchedule(ctx context.Context, day string) (ScheduleResult, error) {
 	day = strings.ToLower(day)
-	cacheKey := fmt.Sprintf("schedule_limit%d_%s", ListPageSize, day)
+	cacheKey := fmt.Sprintf("schedule_%s", day)
 
 	var cached ScheduleResult
 	if c.getCache(ctx, cacheKey, &cached) {
@@ -24,7 +24,7 @@ func (c *Client) GetSchedule(ctx context.Context, day string) (ScheduleResult, e
 	hasStale := c.getStaleCache(ctx, cacheKey, &stale)
 
 	var result TopAnimeResponse
-	reqURL := fmt.Sprintf("%s/schedules?filter=%s&sfw=true&limit=%d", c.baseURL, day, ListPageSize)
+	reqURL := fmt.Sprintf("%s/schedules?filter=%s&sfw=true", c.baseURL, day)
 	if err := c.fetchWithRetry(ctx, reqURL, &result); err != nil {
 		if hasStale {
 			return stale, nil
@@ -61,7 +61,7 @@ func (c *Client) GetSeasonsNow(ctx context.Context, page int) (TopAnimeResult, e
 	if page < 1 {
 		page = 1
 	}
-	cacheKey := fmt.Sprintf("seasons_now_limit%d:%d", ListPageSize, page)
+	cacheKey := fmt.Sprintf("seasons_now:%d", page)
 	var cached TopAnimeResult
 	if c.getCache(ctx, cacheKey, &cached) {
 		return cached, nil
@@ -71,7 +71,7 @@ func (c *Client) GetSeasonsNow(ctx context.Context, page int) (TopAnimeResult, e
 	hasStale := c.getStaleCache(ctx, cacheKey, &stale)
 
 	var result TopAnimeResponse
-	reqURL := fmt.Sprintf("%s/seasons/now?limit=%d&page=%d", c.baseURL, ListPageSize, page)
+	reqURL := fmt.Sprintf("%s/seasons/now?page=%d", c.baseURL, page)
 	if err := c.fetchWithRetry(ctx, reqURL, &result); err != nil {
 		if hasStale {
 			return stale, nil
@@ -93,7 +93,7 @@ func (c *Client) GetSeasonsUpcoming(ctx context.Context, page int) (TopAnimeResu
 	if page < 1 {
 		page = 1
 	}
-	cacheKey := fmt.Sprintf("seasons_upcoming_limit%d:%d", ListPageSize, page)
+	cacheKey := fmt.Sprintf("seasons_upcoming:%d", page)
 	var cached TopAnimeResult
 	if c.getCache(ctx, cacheKey, &cached) {
 		return cached, nil
@@ -103,7 +103,7 @@ func (c *Client) GetSeasonsUpcoming(ctx context.Context, page int) (TopAnimeResu
 	hasStale := c.getStaleCache(ctx, cacheKey, &stale)
 
 	var result TopAnimeResponse
-	reqURL := fmt.Sprintf("%s/seasons/upcoming?limit=%d&page=%d", c.baseURL, ListPageSize, page)
+	reqURL := fmt.Sprintf("%s/seasons/upcoming?page=%d", c.baseURL, page)
 	if err := c.fetchWithRetry(ctx, reqURL, &result); err != nil {
 		if hasStale {
 			return stale, nil