Jaishree2310/GlassyUI-Components

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

Ouverte

#698 ouverte le 5 juin 2026

Voir sur GitHubGuide contributeur
 (3 commentaires) (0 réaction) (1 personne assignée)TypeScript (217 forks)auto 404
good first issuegssoc'26gssoc:approvedlevel:beginnertype:bug

Métriques du dépôt

Stars
 (114 étoiles)
Métriques de merge PR
 (Métriques PR en attente)

Description

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.

Guide contributeur

Direction de recherche
Examinez le hook useGitHubStars pour ajouter une vérification de r.ok (réponse réussie) et assurez vous que stargazers count est un nombre valide avant de mettre à jour le cache. Envisagez également de conserver le dernier décompte mis en cache lorsque l'API échoue.
Stack technique
typescriptreact
Domaine
frontend
Type d'issue
Bug
DifficultéDifficulté estimée pour un nouveau contributeur, de 1 pour un très petit changement à 5 pour un travail expert.
2
Temps estiméFourchette de temps approximative pour investiguer, implémenter, tester et préparer une pull request.
1-3 heures
Statut d'activitéDisponibilité apparente de l'issue : fraîche, active, ancienne, bloquée ou en attente d'un mainteneur.
Active
ClartéClarté avec laquelle l'issue explique le changement attendu, les critères d'acceptation et la prochaine étape.
Claire
Prérequis
ReactTypeScriptGitHub API
Accessibilité débutantScore de 1 à 100 estimant l'accessibilité de cette issue pour un premier contributeur.
70