Skip to content

feat(github): list repositories starred by authenticated user via /us…#3

Open
ch8matt wants to merge 1 commit into
albertlauncher:mainfrom
ch8matt:patch-1
Open

feat(github): list repositories starred by authenticated user via /us…#3
ch8matt wants to merge 1 commit into
albertlauncher:mainfrom
ch8matt:patch-1

Conversation

@ch8matt

@ch8matt ch8matt commented Feb 25, 2026

Copy link
Copy Markdown

…er/starred

@ManuelSchneid3r

Copy link
Copy Markdown
Member

imho this one is not really well thought out. but before I start to discuss: is this LLM output?

@ch8matt

ch8matt commented Jun 25, 2026

Copy link
Copy Markdown
Author

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.

@ManuelSchneid3r

ManuelSchneid3r commented Jun 26, 2026

Copy link
Copy Markdown
Member

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.

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

Labels

None yet

Development

Successfully merging this pull request may close these issues.

2 participants