feat(github): list repositories starred by authenticated user via /us…#3
feat(github): list repositories starred by authenticated user via /us…#3ch8matt wants to merge 1 commit into
Conversation
|
imho this one is not really well thought out. but before I start to discuss: is this LLM output? |
|
No, it’s not. I may have used Copilot to help me code faster, but the work is based on my own understanding of the code. It’s still missing some features, such as sorting, but I often check my recently starred items, so I thought it was a good start. |
|
Okay. So i agree it may be handy to have a list of starred repos. You implement the abstract github search handler. The concrete implementation is not a search handler though. Users cannot filter the repositories. The items method just returns everything. Also the saved search makes no sense for starred repos. The problem here is that the API is paged and therefore difficult to filter. Naive approach: fetch each page and filter until the amount of required items is reached or all pages have been fetched. However, each fetch takes 2s seconds due to rate limiting, which makes this entire thing unusable. Another approach would be to fetch the entire list in regular intervals. This is actually an idea I had for the saved searches. Basically, I want to cache them such that search is performed offline and therefore faster, and as such would be able to join the Global Search. Caveat: state may lag behind. |
…er/starred