Improvement: Persistent Wikipedia Caching for faster startup - #14
Open
ssuppe wants to merge 3 commits into
Open
Conversation
Author
|
Just a quick note to help with the review: although there are 12 files, a significant portion of the changes are unit tests and translations to ensure the new caching logic is stable and fully localized! |
Author
|
Apologies, I tried to rename the branch but this broke the PR. I've restored the name and reopened. |
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.
Hello!
My name is Steve. I've been spending a lot of time with Chess Tutor lately and I really love what you've built. It’s a wonderful piece of software and it’s been a joy to use.
I’ve come across a few quality-of-life improvements that I found useful in my own setup and I would love to contribute them back to the project. Please know that it’s not my intent to make the project more complicated—I just wanted to share some things that made the experience a bit smoother for me.
I have a few different sets of changes ready (including some model migration updates and UI tweaks), but I thought I'd start with the Wikipedia caching. I'm happy to submit these all together as one large PR or keep them broken up into smaller, focused ones like this one—whichever is easiest for you to review and manage!
Feature: Wikipedia Cache Persistence
The Problem:
While the application has a massive database of over 12,000 openings, it fetches Wikipedia summaries for the major opening families (~70+) to provide educational context. Previously, these were being refetched frequently without long-term persistence, which occasionally triggered "Too Many Requests" (429) errors from Wikipedia and added unnecessary overhead to the startup process.
The Solution:
I've implemented a persistent caching system so these summaries are fetched once and stored locally.
Key Improvements:
public/wikipedia). I've also updated the Docker configuration so this folder is preserved across container restarts via a volume.I've also included a suite of tests to ensure the cache management remains stable.
Thank you again for creating such a great tool! Let me know if you have any questions or if you'd like me to adjust the way I'm submitting these.
Best,
Steve