fix: restore Riwyat covers and chapter content (#2465)

[RibatTRW]

What

Riwyat (https://cenele.com/) broke: detail pages showed the default "cover not available" image and an empty novel name, chapters showed "no content available".

cenele.com migrated to a custom Madara child theme (class prefix nhv-*, "novel" theme) that dropped the classic Madara selectors the shared template depends on. The template's parseChapter chains Cheerio selections with ||. An empty Cheerio selection is truthy, so the first missing selector (.text-left) short-circuits the whole chain to an empty selection → html() → '' (the empty chapters). parseNovel lost its name (.post-title h1) and cover (.summary_image > a > img) selectors too.

Why / how

• parseChapter: length-guarded container chain: .text-left → .text-right → .reading-content → .entry-content → .c-blog-post > div > div:nth-child(2). When nothing matches or parsed content is empty it now throws a descriptive error instead of returning '' (same accepted loud-error pattern as the merged upstream fix NovelFire Empty Chapter fix #2462 for NovelFire). .reading-content precedes .entry-content because on this theme .entry-content wraps prev/next nav + a report widget (2.6 KB chrome).
• parseNovel name: fallback h1.nhv-novel-title.
• parseNovel cover: fallback img.wp-post-image.
• parseNovel metadata: existence-guarded nhv-* block (genres/status/author): only fires when the theme's elements exist, so classic Madara / NovelHub sources are untouched.
• riwyat per-source: versionIncrements: 1 (plugin 2.2.1, so cached installs refetch) and a new customJs stripping [data-nosnippet], style, script, input, .nhv-reader-store-promo (the old selector matched zero elements; the theme now injects one promo blockquote after every paragraph plus a VIP store ad).

Tests

Fixture-backed, deterministic, no network: tests/regression-2465.mjs + tests/fixtures/2465/ (saved live-site pages).

node plugins/multisrc/generate-multisrc-plugins.js
node tests/regression-2465.mjs

Check	Before (e1dcd06)	After
parseNovel name	FAIL ('')	PASS
parseNovel cover	FAIL (defaultCover)	PASS
parseNovel chapters listed	PASS	PASS
parseNovel genres (nhv-novel-genres)	FAIL ('')	PASS
parseNovel status Ongoing	FAIL (Completed)	PASS
parseNovel author	FAIL ('')	PASS
parseChapter content length ≥ 200	FAIL (0 chars)	PASS
promo markers / data-nosnippet / style/script stripped	PASS (vacuously)	PASS
unknown layout REJECTS (never '')	FAIL (resolved silently)	PASS
mechanism checks (`		`-truthy trap documented)

Baseline: 7 FAILING → after fix: 14/14 ALL GREEN.

Live check (node scripts/live-check-plugin.js "plugins/arabic/Riwyat[madara].ts"): popularNovels PASS 12 · searchNovels PASS 5 · parseNovel PASS 3 chapters · parseChapter PASS 10,323 chars.

Gates

• npx prettier --check clean on template.ts, sources.json, regression-2465.mjs
• npx eslint plugins/multisrc/madara/template.ts clean
• npx tsc --noEmit: zero new errors vs base (repo-wide 1289 pre-existing errors in gitignored generated files; production tsconfig is transpile-only with noCheck: true)
• No lockfile churn; generated plugins/*/*[madara].ts are gitignored (regenerate before testing)

Closes #2465

Review note

eslint.config.js also gains Node globals for **/*.mjs. That file is outside the issue's change list. It stays: it is the smallest change that lets the committed tests/regression-2465.mjs run clean under lint. No plugin runtime code depends on it.

---

Automated contribution via Hermes Agent.

[RibatTRW]

eslint: this new .mjs file is outside the globals-bearing config block in eslint.config.js (files: ['/*.{ts,tsx,mts,cts,js,cjs}']), so it lints against the base config with no Node globals. Result: 6 errors — no-undef 'console' (lines 34, 53, 192), no-undef 'process' (57, 195), and @typescript-eslint/no-unused-vars for the unused 'init' fetch-stub param (80). Local npm run lint reports these and lint.yml (reviewdog, fail_on_error:false) will annotate the PR. Suggested fix: add a flat-config block to eslint.config.js: '{ files: ["/*.mjs"], languageOptions: { globals: { ...globals.node } } }' (globals is already imported) and drop the unused init parameter from the fetch stub (extra arguments are ignored by the caller).

[RibatTRW]

Verification report: build, typecheck, lint, tests (builder-bob)

Verified point: commit 631dca9 (head of this PR), base e1dcd06. Environment: Node v26.5.1, eslint 8.57.0, tsc 5.7.2, prettier 3.2.5 (workflow-pinned), all from npm ci with the repo lockfile.

Result summary

Check	Result
Regression test suite (node tests/regression-2465.mjs)	PASS, ALL GREEN (14 checks), exit 0
Build (npm run build:compile, tsc -p tsconfig.production.json)	PASS, exit 0
ESLint on changed files (mirrors lint.yml / reviewdog scope)	CLEAN, 0 problems
Prettier workflow command ./src/**/*.{ts,tsx,js,css} @ 3.2.5	PASS (clean on repository-content copies)
Prettier on the 3 modified files (eslint.config.js, template.ts, sources.json)	PASS
Prettier on tests/regression-2465.mjs	MINOR finding (see below)
Strict typecheck tsc --noEmit -p tsconfig.json	1210 pre-existing errors, NOT introduced by this PR (proven below)
Full-repo eslint .	86 pre-existing problems (75 errors, 11 warnings), none in this PR's files
GitHub Actions on this head SHA	3 runs (Lint Check, Format Check, Plugin Live Check) queued, conclusion action_required, awaiting maintainer approval (fork PR), no CI verdict yet

Details

1. Regression test: all fixture-driven checks pass (parser name/cover/genres/status/author, chapter content length, promo spam removal, nosnippet/style/script stripping, and the unknown-layout reject case). Exit code 0.

2. Build: tsc --project tsconfig.production.json (the repo's build:compile, noCheck transpile) exits 0.

3. ESLint changed-file pass (the set lint.yml lints via reviewdog: eslint.config.js, plugins/multisrc/madara/template.ts, tests/regression-2465.mjs): 0 errors, 0 warnings. The 631dca9 .mjs globals fix works.

4. Prettier (3.2.5, checkout via GitHub Actions is LF; local Windows checkout has CRLF which produces line-ending-only false positives, so checks were run against repository-content LF copies):

   • Workflow glob ./src/**/*.{ts,tsx,js,css}: clean (this PR does not change src/).
   • Modified files eslint.config.js, template.ts, sources.json: clean.
   • New file tests/regression-2465.mjs: one trivial style diff, async (url) => should be async url => (arrowParens: avoid). Note it is OUTSIDE the CI globs (workflow checks src/ only; format:check glob ./**/*.{js,ts} does not match .mjs), so no workflow can fail on it. Optional fix, cosmetic only.

5. Strict typecheck and full-repo lint: the repo's production build is intentionally noCheck (tsconfig.production.json has noCheck/noResolve) and the workflows do not run a full typecheck or full-repo eslint (lint.yml lints only changed files with fail_on_error: false). I ran them anyway for completeness:

   • tsc --noEmit -p tsconfig.json at this HEAD: 1210 errors, all in generated plugins/**/[site].ts files (FilterTypes string-literal mismatches, filters possibly undefined, cheerio node types). Re-run at base e1dcd06: also 1210 errors; after normalizing file/line/column numbers the error sets are identical (0 diff lines). Pre-existing baseline, unrelated to this diff.
   • eslint . at this HEAD: 75 errors / 11 warnings, all in files outside this PR's diff (other site plugins, .github scripts, generated files). The PR diff itself lints clean.

CI status note

This PR is cross-repository (head fork RibatTRW/lnreader-plugins) and draft. The three workflow runs tied to 631dca9 are in action_required (pending maintainer approval to run Actions for a fork PR), so there is no green/red CI verdict yet. The local pipeline above is the evidence available now and it is green for the PR diff; once a maintainer approves and the Actions run, verify the workflows complete successfully.