From 8d6b275c695239aa1cf9a539507255974be63419 Mon Sep 17 00:00:00 2001
From: Raiyn Aydin <aydinhrrs@gmail.com>
Date: Wed, 16 Sep 2026 19:13:49 +0800
Subject: [PATCH 1/7] fix(english/novelight): page chapters from the site's own
 ranges

`parseNovel` counted `#select-pagination-chapter > option` elements for
`totalPages`, while `parsePage` recomputed it from the last numeric option in
the whole document, and then mapped each app page onto a site page with
`totalPages - page + 1`. The two derivations could disagree, and the reversed
mapping made the site's partial page the FIRST page instead of the last: on
live novels page 1 returned 2, 6 and 49 chapters instead of 50. Because that
partition is anchored at the newest chapter, every added chapter shifted all
page boundaries, which the app's per-page chapter cache turns into duplicated
or missing chapters.

Both `parseNovel` and `parsePage` now derive the chapter count from the
largest range end in the site's page select (`ceil(total / 50)` pages), the
same shape as the sibling `mynovels` plugin for this CMS. `parsePage` fetches
only the site pages whose ranges overlap the requested 50-chapter chunk and
slices exactly that chunk, so pages 1..totalPages partition the chapter list:
every page but the last holds 50 chapters, and the partial page is last.
Anchoring the chunks at the oldest chapter also keeps already-served pages
stable when new chapters are added. Locked chapters are filtered after
slicing to keep positions stable; name, URL, date and locked-chapter handling
are unchanged.

Authored with AI assistance.
---
 plugins/english/novelight.ts | 148 +++++++++++++++++++++++++++++------
 1 file changed, 126 insertions(+), 22 deletions(-)

diff --git a/plugins/english/novelight.ts b/plugins/english/novelight.ts
index 92e0211db..2f22c2c93 100644
--- a/plugins/english/novelight.ts
+++ b/plugins/english/novelight.ts
@@ -7,10 +7,25 @@ import { defaultCover } from '@libs/defaultCover';
 import dayjs from 'dayjs';
 import { storage } from '@libs/storage';
 
+const CHAPTERS_PER_PAGE = 50;
+
+type ChapterPaginationOption = {
+  value: string;
+  start: number;
+  end: number;
+};
+
+type RawChapter = {
+  name: string;
+  path: string;
+  releaseTime?: string;
+  isLocked: boolean;
+};
+
 class Novelight implements Plugin.PagePlugin {
   id = 'novelight';
   name = 'Novelight';
-  version = '1.1.5';
+  version = '1.1.6';
   icon = 'src/en/novelight/icon.png';
   site = 'https://novelight.net/';
 
@@ -91,12 +106,19 @@ class Novelight implements Plugin.PagePlugin {
 
     const loadedCheerio = parseHTML(body);
 
+    // The chapter select tiles the chapter list, so the largest range end is
+    // the novel's chapter count. `parsePage` derives the same value from the
+    // same markup, keeping `totalPages` and the served pages in agreement.
+    const totalChapters = this.parseTotalChapters(
+      this.parseChapterPagination(loadedCheerio),
+    );
+
     const novel: Plugin.SourceNovel & { totalPages: number } = {
       path: novelPath,
       name: loadedCheerio('h1').text() || 'Untitled',
       cover: this.site + loadedCheerio('.poster > img').attr('src'),
       summary: loadedCheerio('section.text-info.section > p').text(),
-      totalPages: loadedCheerio('#select-pagination-chapter > option').length,
+      totalPages: Math.ceil(totalChapters / CHAPTERS_PER_PAGE) || 1,
       chapters: [],
     };
 
@@ -139,19 +161,47 @@ class Novelight implements Plugin.PagePlugin {
     return novel;
   }
 
-  async parsePage(novelPath: string, page: string): Promise<Plugin.SourcePage> {
-    const rawBody = await fetchApi(this.site + novelPath).then(r => r.text());
-    const csrftoken = rawBody?.match(/window\.CSRF_TOKEN = "([^"]+)"/)?.[1];
-    const bookId = rawBody?.match(/const OBJECT_BY_COMMENT = ([0-9]+)/)?.[1];
-    const totalPages = parseInt(
-      rawBody
-        ?.match(/<option value="([0-9]+)"/g)
-        ?.at(-1)
-        ?.match(/([0-9]+)/)?.[1] ?? '1',
+  /**
+   * The site's chapter select lists page ranges such as `1-50`, `51-100`,
+   * newest page first. Those ranges tile the whole chapter list, so they
+   * describe the site's own partition and let us slice it without assuming a
+   * fixed site page size.
+   */
+  private parseChapterPagination(
+    loadedCheerio: ReturnType<typeof parseHTML>,
+  ): ChapterPaginationOption[] {
+    const options: ChapterPaginationOption[] = [];
+    loadedCheerio('#select-pagination-chapter > option').each((_, ele) => {
+      const option = loadedCheerio(ele);
+      const range = option
+        .text()
+        .trim()
+        .match(/([0-9]+)\s*-\s*([0-9]+)/);
+      if (!range) return;
+      options.push({
+        value: option.val() ?? '',
+        start: parseInt(range[1], 10),
+        end: parseInt(range[2], 10),
+      });
+    });
+    return options;
+  }
+
+  private parseTotalChapters(options: ChapterPaginationOption[]): number {
+    return (
+      options.reduce((total, option) => Math.max(total, option.end), 0) ||
+      options.length * CHAPTERS_PER_PAGE
     );
+  }
 
+  private async parseSitePageChapters(
+    novelPath: string,
+    csrftoken: string,
+    bookId: string,
+    sitePage: string,
+  ): Promise<RawChapter[]> {
     const r = await fetchApi(
-      `${this.site}book/ajax/chapter-pagination?csrfmiddlewaretoken=${csrftoken}&book_id=${bookId}&page=${totalPages - parseInt(page) + 1}`,
+      `${this.site}book/ajax/chapter-pagination?csrfmiddlewaretoken=${csrftoken}&book_id=${bookId}&page=${sitePage}`,
       {
         headers: {
           'Host': this.site.replace('https://', '').replace('/', ''),
@@ -171,16 +221,15 @@ class Novelight implements Plugin.PagePlugin {
       throw new Error(error);
     }
 
-    const chapter: Plugin.ChapterItem[] = [];
+    const chapters: RawChapter[] = [];
 
     parseHTML('<html>' + chaptersRaw + '</html>')('a').each((idx, ele) => {
       const title = parseHTML(ele)('.title').text().trim();
       const isLocked = !!parseHTML(ele)('.cost').text().trim();
-      if (this.hideLocked && isLocked) return;
 
-      let date;
+      let releaseTime;
       try {
-        date = dayjs(
+        releaseTime = dayjs(
           parseHTML(ele)('.date').text().trim(),
           'DD.MM.YYYY',
         ).toISOString();
@@ -188,20 +237,75 @@ class Novelight implements Plugin.PagePlugin {
         // linter happy
       }
 
-      const chapterName = isLocked ? '🔒 ' + title : title;
       let chapterUrl = ele.attribs.href;
       if (chapterUrl.charAt(0) == '/') {
         chapterUrl = chapterUrl.substring(1);
       }
-      chapter.push({
-        name: chapterName,
+      chapters.push({
+        name: title,
         path: chapterUrl,
-        page: page,
-        releaseTime: date,
+        releaseTime,
+        isLocked,
       });
     });
 
-    const chapters = chapter.reverse();
+    // The site lists a page newest first, while pages are served oldest first.
+    return chapters.reverse();
+  }
+
+  async parsePage(novelPath: string, page: string): Promise<Plugin.SourcePage> {
+    const rawBody = await fetchApi(this.site + novelPath).then(r => r.text());
+    const csrftoken = rawBody?.match(/window\.CSRF_TOKEN = "([^"]+)"/)?.[1];
+    const bookId = rawBody?.match(/const OBJECT_BY_COMMENT = ([0-9]+)/)?.[1];
+    const chapterPagination = this.parseChapterPagination(parseHTML(rawBody));
+    const totalChapters = this.parseTotalChapters(chapterPagination);
+
+    const pageNo = parseInt(page, 10);
+    const firstChapter = (pageNo - 1) * CHAPTERS_PER_PAGE + 1;
+    const lastChapter = Math.min(pageNo * CHAPTERS_PER_PAGE, totalChapters);
+
+    if (
+      Number.isNaN(pageNo) ||
+      !csrftoken ||
+      !bookId ||
+      firstChapter > lastChapter
+    ) {
+      return { chapters: [] };
+    }
+
+    // Site ranges are numbered from the oldest chapter, which is also how
+    // pages are served, so the requested chunk maps onto them directly even
+    // though the site lists its own pages newest first.
+    const sitePages = chapterPagination
+      .filter(
+        option => option.start <= lastChapter && option.end >= firstChapter,
+      )
+      .sort((a, b) => a.start - b.start);
+
+    if (sitePages.length === 0) {
+      return { chapters: [] };
+    }
+
+    const fetched = await Promise.all(
+      sitePages.map(option =>
+        this.parseSitePageChapters(novelPath, csrftoken, bookId, option.value),
+      ),
+    );
+
+    // The fetched pages form one contiguous oldest-first run starting at the
+    // first range's start, so slice exactly the requested chunk out of it.
+    const offset = firstChapter - sitePages[0].start;
+    const chapters = fetched
+      .flat()
+      .slice(offset, offset + lastChapter - firstChapter + 1)
+      .filter(chapter => !(this.hideLocked && chapter.isLocked))
+      .map(chapter => ({
+        name: chapter.isLocked ? '🔒 ' + chapter.name : chapter.name,
+        path: chapter.path,
+        page: page,
+        releaseTime: chapter.releaseTime,
+      }));
+
     return { chapters };
   }
 

From a0777808c2ba3ae3aac8e0330ebf537c844c4e1c Mon Sep 17 00:00:00 2001
From: Raiyn Aydin <aydinhrrs@gmail.com>
Date: Wed, 16 Sep 2026 21:08:41 +0800
Subject: [PATCH 2/7] fix(english/novelight): send browser-like headers for
 Cloudflare-fronted site

popularNovels returned no novels in CI although the catalog markup and the
'a.item' selector are unchanged: the plugin sent no User-Agent at all, and
the Cloudflare-fronted site answers such requests with a bot-check page on
networks it distrusts (e.g. CI runners), which parses to zero novels. The
live check's own HEAD probe (which carries a Mozilla User-Agent) passed, so
the failure showed up as FAIL rather than INCONCLUSIVE.

Pass browser-like headers (User-Agent, Referer, Accept-Language) on every
fetchApi call, following the indraTranslations/dreamyTranslations pattern.
Chapter-list pagination behavior is unchanged. Local live check passes:
popularNovels 30 novels, searchNovels 1 result, parseNovel (parsePage) 50
chapters, parseChapter 12215 chars.

Refs lnreader/lnreader-plugins#2526.

Authored with AI assistance.
---
 plugins/english/novelight.ts | 33 +++++++++++++++++++++++++++------
 1 file changed, 27 insertions(+), 6 deletions(-)

diff --git a/plugins/english/novelight.ts b/plugins/english/novelight.ts
index 2f22c2c93..17638ce41 100644
--- a/plugins/english/novelight.ts
+++ b/plugins/english/novelight.ts
@@ -25,10 +25,19 @@ type RawChapter = {
 class Novelight implements Plugin.PagePlugin {
   id = 'novelight';
   name = 'Novelight';
-  version = '1.1.6';
+  version = '1.1.7';
   icon = 'src/en/novelight/icon.png';
   site = 'https://novelight.net/';
 
+  // Browser-like headers (important for Cloudflare-fronted sites, which
+  // may serve a bot-check page to requests without a User-Agent).
+  private headers = {
+    'User-Agent':
+      'Mozilla/5.0 (Windows NT 10.0; Win64; x64) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/120 Safari/537.36',
+    Referer: this.site,
+    'Accept-Language': 'en-US,en;q=0.9',
+  };
+
   hideLocked = storage.get('hideLocked');
   pluginSettings = {
     hideLocked: {
@@ -72,7 +81,9 @@ class Novelight implements Plugin.PagePlugin {
       url += `?&ordering=popularity&page=${pageNo}`;
     }
 
-    const body = await fetchApi(url).then(r => r.text());
+    const body = await fetchApi(url, { headers: this.headers }).then(r =>
+      r.text(),
+    );
 
     const loadedCheerio = parseHTML(body);
 
@@ -102,7 +113,9 @@ class Novelight implements Plugin.PagePlugin {
   async parseNovel(
     novelPath: string,
   ): Promise<Plugin.SourceNovel & { totalPages: number }> {
-    const body = await fetchApi(this.site + novelPath).then(r => r.text());
+    const body = await fetchApi(this.site + novelPath, {
+      headers: this.headers,
+    }).then(r => r.text());
 
     const loadedCheerio = parseHTML(body);
 
@@ -204,6 +217,7 @@ class Novelight implements Plugin.PagePlugin {
       `${this.site}book/ajax/chapter-pagination?csrfmiddlewaretoken=${csrftoken}&book_id=${bookId}&page=${sitePage}`,
       {
         headers: {
+          ...this.headers,
           'Host': this.site.replace('https://', '').replace('/', ''),
           'Referer': this.site + novelPath,
           'X-Requested-With': 'XMLHttpRequest',
@@ -254,7 +268,9 @@ class Novelight implements Plugin.PagePlugin {
   }
 
   async parsePage(novelPath: string, page: string): Promise<Plugin.SourcePage> {
-    const rawBody = await fetchApi(this.site + novelPath).then(r => r.text());
+    const rawBody = await fetchApi(this.site + novelPath, {
+      headers: this.headers,
+    }).then(r => r.text());
     const csrftoken = rawBody?.match(/window\.CSRF_TOKEN = "([^"]+)"/)?.[1];
     const bookId = rawBody?.match(/const OBJECT_BY_COMMENT = ([0-9]+)/)?.[1];
     const chapterPagination = this.parseChapterPagination(parseHTML(rawBody));
@@ -313,7 +329,9 @@ class Novelight implements Plugin.PagePlugin {
     if (chapterPath.charAt(0) == '/') {
       chapterPath = chapterPath.substring(1);
     }
-    const rawBody = await fetchApi(this.site + chapterPath).then(r => {
+    const rawBody = await fetchApi(this.site + chapterPath, {
+      headers: this.headers,
+    }).then(r => {
       const res = r.text();
       return res;
     });
@@ -327,6 +345,7 @@ class Novelight implements Plugin.PagePlugin {
       {
         method: 'GET',
         headers: {
+          ...this.headers,
           Cookie: 'csrftoken=' + csrftoken,
           Referer: this.site + chapterPath,
           'X-Requested-With': 'XMLHttpRequest',
@@ -351,7 +370,9 @@ class Novelight implements Plugin.PagePlugin {
 
   async searchNovels(searchTerm: string): Promise<Plugin.NovelItem[]> {
     const url = `${this.site}catalog/?search=${encodeURIComponent(searchTerm)}`;
-    const body = await fetchApi(url).then(r => r.text());
+    const body = await fetchApi(url, { headers: this.headers }).then(r =>
+      r.text(),
+    );
     const loadedCheerio = parseHTML(body);
 
     const novels: Plugin.NovelItem[] = [];

From 761581ca72ed4bed12323dcb83f4002cd1d85661 Mon Sep 17 00:00:00 2001
From: Raiyn Aydin <aydinhrrs@gmail.com>
Date: Fri, 18 Sep 2026 16:59:12 +0800
Subject: [PATCH 3/7] fix(english/novelight): throw with HTTP status on refused
 responses

Route every site request through one helper that throws (carrying the
HTTP status) on a non-OK response, so a runner-side block is reported
INCONCLUSIVE per docs/testing.md instead of being parsed into a false
'Returned no novels' FAIL. Headers and pagination behavior unchanged.

This commit was authored with AI assistance.
---
 plugins/english/novelight.ts | 31 ++++++++++++++++++++++---------
 1 file changed, 22 insertions(+), 9 deletions(-)

diff --git a/plugins/english/novelight.ts b/plugins/english/novelight.ts
index 17638ce41..5c4316175 100644
--- a/plugins/english/novelight.ts
+++ b/plugins/english/novelight.ts
@@ -1,5 +1,5 @@
 import { load as parseHTML } from 'cheerio';
-import { fetchApi } from '@libs/fetch';
+import { fetchApi, FetchInit } from '@libs/fetch';
 import { Plugin } from '@/types/plugin';
 import { NovelStatus } from '@libs/novelStatus';
 import { Filters, FilterTypes } from '@libs/filterInputs';
@@ -25,7 +25,7 @@ type RawChapter = {
 class Novelight implements Plugin.PagePlugin {
   id = 'novelight';
   name = 'Novelight';
-  version = '1.1.7';
+  version = '1.1.8';
   icon = 'src/en/novelight/icon.png';
   site = 'https://novelight.net/';
 
@@ -38,6 +38,19 @@ class Novelight implements Plugin.PagePlugin {
     'Accept-Language': 'en-US,en;q=0.9',
   };
 
+  // Throw (carrying the HTTP status) on a refused response so a
+  // runner-side block is reported INCONCLUSIVE per docs/testing.md
+  // instead of being parsed into a false empty-result FAIL.
+  private async fetchSite(url: string, init?: FetchInit) {
+    const res = await fetchApi(url, init);
+    if (!res.ok) {
+      throw Object.assign(new Error('Request failed: ' + res.status), {
+        status: res.status,
+      });
+    }
+    return res;
+  }
+
   hideLocked = storage.get('hideLocked');
   pluginSettings = {
     hideLocked: {
@@ -81,7 +94,7 @@ class Novelight implements Plugin.PagePlugin {
       url += `?&ordering=popularity&page=${pageNo}`;
     }
 
-    const body = await fetchApi(url, { headers: this.headers }).then(r =>
+    const body = await this.fetchSite(url, { headers: this.headers }).then(r =>
       r.text(),
     );
 
@@ -113,7 +126,7 @@ class Novelight implements Plugin.PagePlugin {
   async parseNovel(
     novelPath: string,
   ): Promise<Plugin.SourceNovel & { totalPages: number }> {
-    const body = await fetchApi(this.site + novelPath, {
+    const body = await this.fetchSite(this.site + novelPath, {
       headers: this.headers,
     }).then(r => r.text());
 
@@ -213,7 +226,7 @@ class Novelight implements Plugin.PagePlugin {
     bookId: string,
     sitePage: string,
   ): Promise<RawChapter[]> {
-    const r = await fetchApi(
+    const r = await this.fetchSite(
       `${this.site}book/ajax/chapter-pagination?csrfmiddlewaretoken=${csrftoken}&book_id=${bookId}&page=${sitePage}`,
       {
         headers: {
@@ -268,7 +281,7 @@ class Novelight implements Plugin.PagePlugin {
   }
 
   async parsePage(novelPath: string, page: string): Promise<Plugin.SourcePage> {
-    const rawBody = await fetchApi(this.site + novelPath, {
+    const rawBody = await this.fetchSite(this.site + novelPath, {
       headers: this.headers,
     }).then(r => r.text());
     const csrftoken = rawBody?.match(/window\.CSRF_TOKEN = "([^"]+)"/)?.[1];
@@ -329,7 +342,7 @@ class Novelight implements Plugin.PagePlugin {
     if (chapterPath.charAt(0) == '/') {
       chapterPath = chapterPath.substring(1);
     }
-    const rawBody = await fetchApi(this.site + chapterPath, {
+    const rawBody = await this.fetchSite(this.site + chapterPath, {
       headers: this.headers,
     }).then(r => {
       const res = r.text();
@@ -340,7 +353,7 @@ class Novelight implements Plugin.PagePlugin {
     const chapterId = rawBody?.match(/const CHAPTER_ID = "([0-9]+)/)?.[1];
 
     let className;
-    const body = await fetchApi(
+    const body = await this.fetchSite(
       this.site + 'book/ajax/read-chapter/' + chapterId,
       {
         method: 'GET',
@@ -370,7 +383,7 @@ class Novelight implements Plugin.PagePlugin {
 
   async searchNovels(searchTerm: string): Promise<Plugin.NovelItem[]> {
     const url = `${this.site}catalog/?search=${encodeURIComponent(searchTerm)}`;
-    const body = await fetchApi(url, { headers: this.headers }).then(r =>
+    const body = await this.fetchSite(url, { headers: this.headers }).then(r =>
       r.text(),
     );
     const loadedCheerio = parseHTML(body);

From 72f495c9ffaaf805f455d88282c42982eee0feb4 Mon Sep 17 00:00:00 2001
From: Raiyn Aydin <aydinhrrs@gmail.com>
Date: Tue, 22 Sep 2026 13:50:08 +0800
Subject: [PATCH 4/7] ci: re-trigger checks


From 5095e2f583911bbc6d67aca6a89da5dea8a85cdb Mon Sep 17 00:00:00 2001
From: Raiyn Aydin <aydinhrrs@gmail.com>
Date: Fri, 25 Sep 2026 20:06:55 +0800
Subject: [PATCH 5/7] fix(english/novelight): serve one chapter-pagination
 request per page

Replace the uniform oldest-anchored 50-chapter slicing (which straddled
two of the site's newest-anchored ranges on nearly every page, costing 2
chapter-pagination ajax requests) with a one-to-one mapping onto the
site's own ranges: app page N is the Nth site range in ascending chapter
order. The endpoint honors only the page parameter (page_size, per_page,
limit, offset, start/end, paginate_by, count, size, length, pageSize,
perPage, show all ignored; bookmark_chapter inert; the site's own JS
sends only csrfmiddlewaretoken, book_id and page), and the width is fixed
at 50 server-side, so the site's own partition is the only single-request
design. parseNovel derives totalPages from the same markup, keeping the
two in agreement, and requested values come straight from the parsed
select so out-of-range clamping can no longer duplicate chapters.

Trade-off: the short (remainder) page comes first instead of last, and
new chapters shift every page's contents, as before this PR.

Set the version to a single 1.1.5 -> 1.1.6 net bump.

This commit was authored with AI assistance.
---
 plugins/english/novelight.ts | 107 ++++++++++++++---------------------
 1 file changed, 43 insertions(+), 64 deletions(-)

diff --git a/plugins/english/novelight.ts b/plugins/english/novelight.ts
index 5c4316175..02ac0ae34 100644
--- a/plugins/english/novelight.ts
+++ b/plugins/english/novelight.ts
@@ -7,8 +7,6 @@ import { defaultCover } from '@libs/defaultCover';
 import dayjs from 'dayjs';
 import { storage } from '@libs/storage';
 
-const CHAPTERS_PER_PAGE = 50;
-
 type ChapterPaginationOption = {
   value: string;
   start: number;
@@ -25,7 +23,7 @@ type RawChapter = {
 class Novelight implements Plugin.PagePlugin {
   id = 'novelight';
   name = 'Novelight';
-  version = '1.1.8';
+  version = '1.1.6';
   icon = 'src/en/novelight/icon.png';
   site = 'https://novelight.net/';
 
@@ -132,19 +130,16 @@ class Novelight implements Plugin.PagePlugin {
 
     const loadedCheerio = parseHTML(body);
 
-    // The chapter select tiles the chapter list, so the largest range end is
-    // the novel's chapter count. `parsePage` derives the same value from the
-    // same markup, keeping `totalPages` and the served pages in agreement.
-    const totalChapters = this.parseTotalChapters(
-      this.parseChapterPagination(loadedCheerio),
-    );
+    // The chapter select tiles the chapter list, so one app page per site
+    // range keeps `totalPages` and the served pages in agreement.
+    const totalPages = this.parseChapterPagination(loadedCheerio).length || 1;
 
     const novel: Plugin.SourceNovel & { totalPages: number } = {
       path: novelPath,
       name: loadedCheerio('h1').text() || 'Untitled',
       cover: this.site + loadedCheerio('.poster > img').attr('src'),
       summary: loadedCheerio('section.text-info.section > p').text(),
-      totalPages: Math.ceil(totalChapters / CHAPTERS_PER_PAGE) || 1,
+      totalPages,
       chapters: [],
     };
 
@@ -188,10 +183,13 @@ class Novelight implements Plugin.PagePlugin {
   }
 
   /**
-   * The site's chapter select lists page ranges such as `1-50`, `51-100`,
-   * newest page first. Those ranges tile the whole chapter list, so they
-   * describe the site's own partition and let us slice it without assuming a
-   * fixed site page size.
+   * The site's chapter select lists page ranges such as `1-17`, `18-67`,
+   * newest page first, each holding 50 chapters except the oldest page,
+   * which holds the remainder. Those ranges tile the whole chapter list,
+   * so they describe the site's own partition. The chapter-pagination
+   * endpoint serves exactly one of these ranges per request and accepts
+   * no page-size or offset parameter, so serving the site's own ranges
+   * one-to-one is the only way to spend a single ajax request per page.
    */
   private parseChapterPagination(
     loadedCheerio: ReturnType<typeof parseHTML>,
@@ -213,13 +211,6 @@ class Novelight implements Plugin.PagePlugin {
     return options;
   }
 
-  private parseTotalChapters(options: ChapterPaginationOption[]): number {
-    return (
-      options.reduce((total, option) => Math.max(total, option.end), 0) ||
-      options.length * CHAPTERS_PER_PAGE
-    );
-  }
-
   private async parseSitePageChapters(
     novelPath: string,
     csrftoken: string,
@@ -286,56 +277,44 @@ class Novelight implements Plugin.PagePlugin {
     }).then(r => r.text());
     const csrftoken = rawBody?.match(/window\.CSRF_TOKEN = "([^"]+)"/)?.[1];
     const bookId = rawBody?.match(/const OBJECT_BY_COMMENT = ([0-9]+)/)?.[1];
-    const chapterPagination = this.parseChapterPagination(parseHTML(rawBody));
-    const totalChapters = this.parseTotalChapters(chapterPagination);
-
+    // Serve the site's own ranges one-to-one, oldest chunk first: app page
+    // N is the Nth site range in ascending chapter order. One site page per
+    // app page means exactly one chapter-pagination request per call; the
+    // trade-off is that the short (remainder) page comes first instead of
+    // last, and adding new chapters shifts every page's contents because
+    // the site anchors its partition at the newest chapter.
+    const sitePages = this.parseChapterPagination(parseHTML(rawBody)).sort(
+      (a, b) => a.start - b.start,
+    );
     const pageNo = parseInt(page, 10);
-    const firstChapter = (pageNo - 1) * CHAPTERS_PER_PAGE + 1;
-    const lastChapter = Math.min(pageNo * CHAPTERS_PER_PAGE, totalChapters);
-
-    if (
-      Number.isNaN(pageNo) ||
-      !csrftoken ||
-      !bookId ||
-      firstChapter > lastChapter
-    ) {
-      return { chapters: [] };
-    }
+    const sitePage = sitePages[pageNo - 1];
 
-    // Site ranges are numbered from the oldest chapter, which is also how
-    // pages are served, so the requested chunk maps onto them directly even
-    // though the site lists its own pages newest first.
-    const sitePages = chapterPagination
-      .filter(
-        option => option.start <= lastChapter && option.end >= firstChapter,
-      )
-      .sort((a, b) => a.start - b.start);
-
-    if (sitePages.length === 0) {
+    if (Number.isNaN(pageNo) || !csrftoken || !bookId || !sitePage) {
       return { chapters: [] };
     }
 
-    const fetched = await Promise.all(
-      sitePages.map(option =>
-        this.parseSitePageChapters(novelPath, csrftoken, bookId, option.value),
-      ),
+    // The requested value comes straight from the parsed select, so it can
+    // never fall outside the site's pages (out-of-range values used to be
+    // silently clamped to the oldest page, duplicating its chapters).
+    const chapters = await this.parseSitePageChapters(
+      novelPath,
+      csrftoken,
+      bookId,
+      sitePage.value,
     );
 
-    // The fetched pages form one contiguous oldest-first run starting at the
-    // first range's start, so slice exactly the requested chunk out of it.
-    const offset = firstChapter - sitePages[0].start;
-    const chapters = fetched
-      .flat()
-      .slice(offset, offset + lastChapter - firstChapter + 1)
-      .filter(chapter => !(this.hideLocked && chapter.isLocked))
-      .map(chapter => ({
-        name: chapter.isLocked ? '🔒 ' + chapter.name : chapter.name,
-        path: chapter.path,
-        page: page,
-        releaseTime: chapter.releaseTime,
-      }));
-
-    return { chapters };
+    // The site lists a page newest first; `parseSitePageChapters` already
+    // returns it oldest first.
+    return {
+      chapters: chapters
+        .filter(chapter => !(this.hideLocked && chapter.isLocked))
+        .map(chapter => ({
+          name: chapter.isLocked ? '🔒 ' + chapter.name : chapter.name,
+          path: chapter.path,
+          page: page,
+          releaseTime: chapter.releaseTime,
+        })),
+    };
   }
 
   async parseChapter(chapterPath: string): Promise<string> {

From 19a9b27c3785adf5026fafff79434bb28b280af2 Mon Sep 17 00:00:00 2001
From: firstmate-crewmate <firstmate-crewmate@local>
Date: Fri, 25 Sep 2026 21:49:49 +0800
Subject: [PATCH 6/7] fix(english/novelight): apply clean-code review
 robustness fixes

- parseSitePageChapters takes a single params object instead of four
  positional strings
- throw a clear error when the chapter-pagination JSON has no html string
  instead of parsing undefined into zero chapters silently
- let endpoint failures propagate with their original stack; drop the
  console.error noise and the lossy throw new Error(error) wrapper
- drop the parsed-but-never-read ChapterPaginationOption.end field
- filter valueless pagination options, comment the non-range skip, drop
  the duplicated newest-first comment
- bump 1.1.6 to 1.1.7 (patch: behavior-preserving robustness fix)

Co-Authored-By: firstmate-crewmate (AI agent)
---
 plugins/english/novelight.ts | 50 ++++++++++++++++++------------------
 1 file changed, 25 insertions(+), 25 deletions(-)

diff --git a/plugins/english/novelight.ts b/plugins/english/novelight.ts
index 02ac0ae34..07d2ed3b1 100644
--- a/plugins/english/novelight.ts
+++ b/plugins/english/novelight.ts
@@ -10,7 +10,6 @@ import { storage } from '@libs/storage';
 type ChapterPaginationOption = {
   value: string;
   start: number;
-  end: number;
 };
 
 type RawChapter = {
@@ -23,7 +22,7 @@ type RawChapter = {
 class Novelight implements Plugin.PagePlugin {
   id = 'novelight';
   name = 'Novelight';
-  version = '1.1.6';
+  version = '1.1.7';
   icon = 'src/en/novelight/icon.png';
   site = 'https://novelight.net/';
 
@@ -197,26 +196,30 @@ class Novelight implements Plugin.PagePlugin {
     const options: ChapterPaginationOption[] = [];
     loadedCheerio('#select-pagination-chapter > option').each((_, ele) => {
       const option = loadedCheerio(ele);
+      // Skip entries that are not chapter ranges (e.g. placeholder options
+      // without an `N-M` label); they hold no page of chapters.
       const range = option
         .text()
         .trim()
-        .match(/([0-9]+)\s*-\s*([0-9]+)/);
+        .match(/([0-9]+)\s*-\s*[0-9]+/);
       if (!range) return;
+      const value = option.val() ?? '';
+      if (typeof value !== 'string' || !value) return;
       options.push({
-        value: option.val() ?? '',
+        value,
         start: parseInt(range[1], 10),
-        end: parseInt(range[2], 10),
       });
     });
     return options;
   }
 
-  private async parseSitePageChapters(
-    novelPath: string,
-    csrftoken: string,
-    bookId: string,
-    sitePage: string,
-  ): Promise<RawChapter[]> {
+  private async parseSitePageChapters(params: {
+    novelPath: string;
+    csrftoken: string;
+    bookId: string;
+    sitePage: string;
+  }): Promise<RawChapter[]> {
+    const { novelPath, csrftoken, bookId, sitePage } = params;
     const r = await this.fetchSite(
       `${this.site}book/ajax/chapter-pagination?csrfmiddlewaretoken=${csrftoken}&book_id=${bookId}&page=${sitePage}`,
       {
@@ -229,19 +232,18 @@ class Novelight implements Plugin.PagePlugin {
       },
     );
 
-    let chaptersRaw;
-    try {
-      chaptersRaw = await r.json();
-      chaptersRaw = chaptersRaw.html;
-    } catch (error) {
-      console.error('Error Parsing Response');
-      console.error(error);
-      throw new Error(error);
+    // Endpoint failures propagate with their original stack; a JSON body
+    // without an `html` field throws here instead of parsing silently
+    // into zero chapters.
+    const chaptersRaw = await r.json();
+    const chaptersHtml = chaptersRaw?.html;
+    if (typeof chaptersHtml !== 'string') {
+      throw new Error('Unexpected chapter-pagination response');
     }
 
     const chapters: RawChapter[] = [];
 
-    parseHTML('<html>' + chaptersRaw + '</html>')('a').each((idx, ele) => {
+    parseHTML('<html>' + chaptersHtml + '</html>')('a').each((idx, ele) => {
       const title = parseHTML(ele)('.title').text().trim();
       const isLocked = !!parseHTML(ele)('.cost').text().trim();
 
@@ -296,15 +298,13 @@ class Novelight implements Plugin.PagePlugin {
     // The requested value comes straight from the parsed select, so it can
     // never fall outside the site's pages (out-of-range values used to be
     // silently clamped to the oldest page, duplicating its chapters).
-    const chapters = await this.parseSitePageChapters(
+    const chapters = await this.parseSitePageChapters({
       novelPath,
       csrftoken,
       bookId,
-      sitePage.value,
-    );
+      sitePage: sitePage.value,
+    });
 
-    // The site lists a page newest first; `parseSitePageChapters` already
-    // returns it oldest first.
     return {
       chapters: chapters
         .filter(chapter => !(this.hideLocked && chapter.isLocked))

From 063e44810131c6b3ea2714d2c5ad678224d1c08a Mon Sep 17 00:00:00 2001
From: Raiyn Aydin <aydinhrrs@gmail.com>
Date: Sun, 27 Sep 2026 17:59:53 +0800
Subject: [PATCH 7/7] fix(english/novelight): set version to 1.1.6

---
 plugins/english/novelight.ts | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/plugins/english/novelight.ts b/plugins/english/novelight.ts
index 07d2ed3b1..4535668bc 100644
--- a/plugins/english/novelight.ts
+++ b/plugins/english/novelight.ts
@@ -22,7 +22,7 @@ type RawChapter = {
 class Novelight implements Plugin.PagePlugin {
   id = 'novelight';
   name = 'Novelight';
-  version = '1.1.7';
+  version = '1.1.6';
   icon = 'src/en/novelight/icon.png';
   site = 'https://novelight.net/';