Skip to content

Cleanup and performance optimizations#12

Merged
gvonness-apolitical merged 1 commit intomainfrom
perf/cleanup-and-optimizations
Feb 2, 2026
Merged

Cleanup and performance optimizations#12
gvonness-apolitical merged 1 commit intomainfrom
perf/cleanup-and-optimizations

Conversation

@gvonness-apolitical
Copy link
Copy Markdown
Collaborator

Summary

  • Remove unused parseNavigationFromHtml import from tumblr-reader
  • Debounce resize handler (150ms) to prevent excessive layout recalculations during window resizing
  • Replace CBZ LRU cache array-based tracking with timestamp-based approach for O(1) cache hit updates

Test plan

  • All 124 tests pass
  • Build succeeds
  • Manual verification of CBZ reader performance with rapid page navigation
  • Manual verification of resize behavior

- Remove unused parseNavigationFromHtml import from tumblr-reader
- Debounce resize handler (150ms) to prevent excessive layout recalculations
- Replace CBZ LRU cache array-based tracking with timestamp-based approach
  for O(1) cache hit updates instead of O(n) indexOf + splice
@gvonness-apolitical gvonness-apolitical force-pushed the perf/cleanup-and-optimizations branch from 838e91b to f4c0369 Compare February 2, 2026 17:19
@gvonness-apolitical gvonness-apolitical merged commit 454570a into main Feb 2, 2026
3 checks passed
@gvonness-apolitical gvonness-apolitical deleted the perf/cleanup-and-optimizations branch February 2, 2026 17:25
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