Jaishree2310/GlassyUI-Components

[BUG] useGitHubStars cache poisoned with 0 stars during API rate limits or offline state

Aperta

#698 aperta il 5 giu 2026

Vedi su GitHubGuida contributor
 (3 commenti) (0 reazioni) (1 assegnatario)TypeScript (217 fork)auto 404
good first issuegssoc'26gssoc:approvedlevel:beginnertype:bug

Metriche repository

Star
 (114 stelle)
Metriche merge PR
 (Metriche PR in attesa)

Descrizione

Description

In useGitHubStars.ts, the hook fetches stargazers count from the GitHub API but does not check if the response was successful (r.ok). Under rate limits (status 403) or offline errors, the parsed JSON is an error payload. The hook defaults data.stargazers_count to 0, setting and caching 0 stars in localStorage for a whole hour, instead of retaining the last known cached count.

Steps to Reproduce

  1. Hit the GitHub API rate limit.
  2. Refresh the website.
  3. The website now displays ★ 0 stars, overwriting the previously correct cached count.

Suggested Fix

Check r.ok and ensure stargazers_count is a valid number before updating the cache.

Guida contributor

Direzione di ricerca
Esamina l'hook useGitHubStars per aggiungere un controllo su r.ok (risposta riuscita) e assicurati che stargazers count sia un numero valido prima di aggiornare la cache. Considera anche di mantenere l'ultimo conteggio memorizzato nella cache quando l'API fallisce.
Tech stack
typescriptreact
Dominio
frontend
Tipo issue
Bug
DifficoltàQuanto dovrebbe essere impegnativa per un nuovo contributor.
2
Tempo stimatoTempo stimato per completare e verificare uno scope piccolo.
1-3 ore
Stato attivitàQuanto è probabile che la issue sia ancora attiva e reviewable.
Attiva
ChiarezzaQuanto chiaramente la issue descrive problema, scope e risultato atteso.
Chiara
Prerequisiti
ReactTypeScriptGitHub API
Adatta ai principiantiQuanto la issue è adatta a contributor alla prima esperienza.
70