Clean up comments

Author: maquchizi

components/fact-checks/FactChecksExplorer.tsx

@@ -32,12 +32,6 @@ function matches(story: Story, applied: Selection): boolean {
   });
 }
 
-/**
- * The `/fact-checks` listing. Pagination is **server-driven**: the parent server
- * page fetches one DB page (`stories`) plus `page`/`totalPages`, and the
- * Pagination bar navigates by `?page=N`, which re-runs the server fetch. The
- * corpus is never loaded into the client.
- */
 export function FactChecksExplorer({
   stories,
   page,

lib/data/queries/fact-checks.ts

@@ -1,10 +1,7 @@
 /**
  * Fact-check listing query. Unlike the home sections (curated content lists),
  * the `/fact-checks` grid is the full body of published fact-checks, so this
- * queries `swp_article` directly. Adapted from
- * `pesacheck-pwa-app-router/services/collectionService.queries.js`
- * (`GET_COLLECTION_BY_METADATA_QUERY` — same `Debunk`-subject filter + offset
- * pagination + `_aggregate` count).
+ * queries `swp_article` directly.
  *
  * It selects the same article fields `GET_CONTENT_LIST_ITEMS` does so `mapStory`
  * can consume the rows unchanged — including `metadata` (jsonb) for the `Debunk`

lib/data/stories.ts

@@ -14,7 +14,6 @@ type FactCheckResponse = {
   items: RawArticle[];
 };
 
-/** One page of the fact-checks listing: the mapped cards + paging metadata. */
 export type FactCheckListing = {
   stories: Story[];
   page: number;
@@ -27,9 +26,10 @@ export type FactCheckListing = {
  * team-member profiles on the `team` route — so we scope listings to language
  * routes. No column distinguishes a language route from any other collection
  * (`team`, `climate-change`, … are all `type: "collection"`), so this set is
- * editorial knowledge, not derivable from the schema. Confirmed on staging;
- * mirrors the language collections in docs/migration-plan.md. PR4 reuses these
- * for the language filter.
+ * editorial knowledge, not derivable from the schema.
+ *
+ * These should ideally not be hard-coded here and should be revisited once
+ * we have cleaner data on staging.
  */
 export const LANGUAGE_ROUTE_SLUGS = [
   "english",
@@ -38,10 +38,6 @@ export const LANGUAGE_ROUTE_SLUGS = [
   "african-languages",
 ];
 
-/**
- * Fact-checks per listing page. One large feature + one secondary + an 8-card
- * grid fill a page (mirrors the Figma layout). Pages are fetched from the DB by offset
- */
 export const FACT_CHECKS_PAGE_SIZE = 10;
 
 /** Fetch a content list's language articles, in curated order, as `Story[]`. */
@@ -84,10 +80,7 @@ export function getHeroPreview(): Promise<Story[]> {
  * that server-side (see `GET_FACT_CHECK_ARTICLES`), which also excludes the
  * other published content sharing these routes (homepage content-blocks,
  * editorial test stubs). The filter is route-agnostic, so it keeps working once
- * fact-checks are published under topic desks (staging desks are empty;
- * everything currently sits on `english`).
- *
- * `page` is 1-based and clamped to `[1, totalPages]`.
+ * fact-checks are published under topic desks.
  */
 export async function getFactChecks(page = 1): Promise<FactCheckListing> {
   const { total, items } = await gql<FactCheckResponse>(