Skip to content

Quick fixes - #42

Open
B0und wants to merge 2 commits into
mainfrom
quick-fixes
Open

Quick fixes#42
B0und wants to merge 2 commits into
mainfrom
quick-fixes

Conversation

@B0und

@B0und B0und commented Aug 5, 2026

Copy link
Copy Markdown
Owner

Summary

Low-risk, behavior-preserving fixes and cleanups from the rewrite codebase analysis.

Changes

  • Fix theme localStorage key mismatch (FOUC prevention was reading a different key than the app wrote)
  • Fix Chinese language selection (isoCode cn -> zh, wrong label, stray tabs in language list)
  • Fix getHighestLinksPage throwing on empty pages array
  • Dedupe errorToast x3 and copyNotification x2 into src/utils/toast.ts
  • Remove dead code (nameof, tweakpane debug panel, useCurrentArticle selector)
  • Fix achivement -> achievement typo
  • Gate devtools to dev builds and move to devDependencies (tree-shaken from prod bundle)
  • Guard StatisticsStore.merge against missing achievements in persisted state
  • Clamp stopwatch ms overflow (999.7 could render as 1000)
  • Add --run to test:browser script so CI doesn't hang

Verification

  • typecheck: pass
  • lint: pass
  • viteBuild (production): pass
  • Browser tests blocked locally: Playwright Chromium not installed in this environment

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