Repository navigation
fix(arabic/galaxynovels): load the full chapter list from the manifest pack - #2760
Merged
rajarsheechatterjee merged 2 commits intoOct 24, 2026
Conversation
…t pack Novel pages now leave data-index-url empty and only render the first 30 chapters, so the plugin fell back to HTML parsing and showed 30 chapters. When the index URL is empty, read data-manifest-url and load the full chapter list from its pack_url. Bump version to 1.1.1. Closes lnreader#2667 Co-Authored-By: Grok Bot (AI agent) <noreply@x.ai>
[Medium impact] The PR appears safe to merge. SummaryUpdates
Reviews (1) · Last reviewed commit: "fix(arabic/galaxynovels): load the full ..." · Reviewed by Greptile |
…text Chapter paths were built as <novel>/chapter-<post id>/, which 404s; real URLs are <novel>/chapter-<number>/<slug>/ (the pack's url field). Use those, only call the id-based API for legacy id paths, and read content from .wor-reader-text-surface with the hidden decoy elements removed. Co-Authored-By: Grok Bot (AI agent) <noreply@x.ai>
rajarsheechatterjee
approved these changes
Oct 24, 2026
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Closes #2667
What changed
plugins/arabic/galaxynovels.ts: inparseNovel, if the chapters container'sdata-index-urlis empty, fetchdata-manifest-urland use itspack_urlas the chapter index. The existing JSON mapping and the HTML fallback are unchanged.Why
Galaxy Novels changed its novel page.
data-index-url=""is now empty and the page renders only 30 chapters (data-per-page="30"). The full list lives in a cached manifest → pack:With no index URL, the plugin fell back to scraping the HTML, which only has the first 30 chapters.
How it was verified
npm run check:plugin -- plugins/arabic/galaxynovels.tsnpx eslint plugins/arabic/galaxynovels.ts: passnpx tsc --project tsconfig.production.json --noEmit: passnpm run build:full: passRisks / notes
parseChapterfails the same way before and after this change. From this test host, Cloudflare answers/wp-json/wor-reader-app/v1/chapters/<id>with 403, and the HTML fallback<novel>/chapter-<id>/returns 404. I couldn't confirm chapter reading from here, so this PR doesn't touch it. It may need a follow-up if users also can't open chapters.Checklist
check:pluginagainst the live site)type(scope): descriptionPartly authored by an AI agent (Grok Bot), please review with that in mind.