refactor(lolschedule): drop stale Leaguepedia references and unused params

Module header and api-client top comment still mentioned Leaguepedia /
MatchSchedule / keying by league filter. fetchSchedulePage also exposed
an unused `leagueId` parameter and returned an unused `olderToken`;
remove both to match the actual usage.
This commit is contained in:
2026-04-21 10:11:49 +07:00
committed by tiennm99
parent e10269ca0a
commit cbb572fdd8
2 changed files with 9 additions and 11 deletions
+5 -8
View File
@@ -7,7 +7,7 @@
* the live site). If Riot ever rotates it, lift the new value from their
* public JS bundle.
*
* We cache responses in KV keyed by league filter so concurrent user requests
* We cache responses in KV keyed by date range so concurrent user requests
* collapse to one upstream hit within the TTL window.
*/
@@ -43,18 +43,16 @@ const STALE_MAX_AGE_SEC = 60 * 60;
*/
/**
* Fetch one page of schedule events. Returns events + pagination tokens.
* Fetch one page of schedule events.
*
* @param {object} [opts]
* @param {string} [opts.pageToken] — `newer`/`older` cursor from a previous call.
* @param {string} [opts.leagueId] — optional comma-separated league IDs.
* @returns {Promise<{ events: ScheduleEvent[], olderToken?: string, newerToken?: string }>}
* @param {string} [opts.pageToken] — forward cursor from a previous call's `newerToken`.
* @returns {Promise<{ events: ScheduleEvent[], newerToken?: string }>}
*/
export async function fetchSchedulePage({ pageToken, leagueId } = {}) {
export async function fetchSchedulePage({ pageToken } = {}) {
const url = new URL(API_URL);
url.searchParams.set("hl", "en-US");
if (pageToken) url.searchParams.set("pageToken", pageToken);
if (leagueId) url.searchParams.set("leagueId", leagueId);
const res = await fetch(url.toString(), {
headers: {
@@ -82,7 +80,6 @@ export async function fetchSchedulePage({ pageToken, leagueId } = {}) {
const filtered = events.filter((e) => e?.type !== "show"); // drop pre/post shows
return {
events: /** @type {ScheduleEvent[]} */ (filtered),
olderToken: schedule?.pages?.older,
newerToken: schedule?.pages?.newer,
};
}
+4 -3
View File
@@ -1,12 +1,13 @@
/**
* @file lolschedule module — LoL esports match schedule via Leaguepedia API.
* @file lolschedule module — LoL esports match schedule via the
* lolesports.com esports-api (the data feed behind lolesports.com).
*
* Commands:
* /lol_today — matches scheduled for the current ICT day, with live/played scores.
* /lol_week — next 7 ICT days, grouped per day.
*
* Data source: Leaguepedia Cargo `MatchSchedule` table on lol.fandom.com.
* See plans/reports/researcher-260421-0845-leaguepedia-api-verification.md.
* See the module README for the data-source rationale and the verification
* reports under plans/reports/ for historical context.
*/
import { handleToday, handleWeek } from "./handlers.js";