Skip to content

fix(russian/ranobelib): support limited Intl runtimes - #2774

Open
grvctr wants to merge 2 commits into
lnreader:masterfrom
grvctr:fix/ranobelib-rakuyomi-intl
Open

grvctr wants to merge 2 commits into
lnreader:masterfrom
grvctr:fix/ranobelib-rakuyomi-intl

Conversation

@grvctr

@grvctr grvctr commented Oct 25, 2026

Copy link
Copy Markdown

Summary

  • avoid crashing in runtimes where Intl.DateTimeFormat.prototype.resolvedOptions is unavailable
  • preserve the detected timezone in full Intl implementations and fall back to Europe/Moscow
  • bump the RanobeLib plugin version to 2.2.8

Testing

  • npx prettier --check plugins/russian/ranobelib.ts
  • npx eslint plugins/russian/ranobelib.ts
  • npm run build:compile
  • npm run check:plugin -- plugins/russian/ranobelib.ts
  • Rakuyomi-compatible startup smoke test with resolvedOptions removed
  • installed and verified on Rakuyomi v1.43.4 on Kindle 11

Checklist

  • Update version code if an existing plugin was modified
  • Test changes in Plugin Playground or the app
  • Reference related issues in the PR body — N/A, no related issue found
  • Commit messages follow type(scope): description

Prepared with assistance from OpenAI Codex.

Avoid plugin startup failure when Intl.DateTimeFormat.prototype.resolvedOptions is unavailable.

Prepared with assistance from OpenAI Codex.
Remove accidental leading whitespace introduced by the web editor.

Prepared with assistance from OpenAI Codex.
@greptile-apps

greptile-apps Bot commented Oct 25, 2026 •

Copy link
Copy Markdown

RetriggerView in GreptileConfidence Score: 5/5

[Low impact] The PR appears safe to merge.

Summary

RanobeLib now uses Europe/Moscow when resolvedOptions is unavailable.

  • RanobeLib requests use a fallback timezone when Intl cannot report one.

Reviews (1) · Last reviewed commit: "chore(russian/ranobelib): fix import ind..." · Reviewed by Greptile

This branch has not been deployed

No deployments
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant