bug: InsightsList incorrectly favors User 1 when metrics are tied
Les mainteneurs répondent en général sous 1 jour
@sarmad-jalil y travaille déjà.
Depuis le 25/10/2026.
Évaluation
- Difficulté
- 2/5
- Temps estimé
- 1-3 heures
- Accessibilité débutants
- 90/100
- Type d'issue
- Bug
- Clarté
- Clairement spécifiée
- Activité
- Active
- Stack technique
- nextjs, react, typescript
- Domaine
- data-visualization, frontend
Piste de recherche
Lis les lignes 46–69 de src/features/comparison/components/insights-list.tsx et les tests existants dans src/features/comparison/tests/insights-list.test.tsx. Vérifie comment le composant gère les scores égaux, puis mets à jour ou ajoute des tests pour les scores égaux des dépôts, des PR et des contributions. Le travail est terminé lorsque des scores égaux ne produisent plus de points forts ni de recommandations asymétriques et que les tests passent.
Rédigé par le modèle d'indexation à partir du texte de l'issue.
Description
In src/features/comparison/components/insights-list.tsx lines 46-69:
const repoLeader = user1.repoScore >= user2.repoScore ? user1 : user2;
const prLeader = user1.prScore >= user2.prScore ? user1 : user2;
const contributionLeader = user1.contributionScore >= user2.contributionScore ? user1 : user2;
When both developers have equal scores (for example, both have 0 PR score or matching repo scores), the >= check automatically assigns user1 as the leader. As a result, user1 is credited with a strength in that category, and user2 is given a recommendation to improve.
Motivation
Displaying asymmetric insights when two scores are identical is incorrect and misleading.
Tasks
- Check for score equality (
user1.repoScore === user2.repoScore). - Only award category strengths when a developer strictly exceeds the other (
>). - If tied, either do not award a strength or display a balanced tied note.
- Update or add tests in
src/features/comparison/tests/insights-list.test.tsx.
Files to Check
src/features/comparison/components/insights-list.tsxsrc/features/comparison/tests/insights-list.test.tsx
[!TIP]
🚀 Want to contribute?
Commentassign meto be automatically assigned to this issue via our GitHub Actions bot. Happy coding! ✨
- Langage dominant
- TypeScript
- Étoiles
- 31
- Forks
- 25
- Merge moyen
- 10 h 4 min
- PR mergées (30 j)
- 29
Préparer son environnement
- Aucun Dockerfile ni fichier Docker Compose
- Propose un modèle de pull request
- Lire le guide de contribution
Par où commencer
- Lisez l'issue en entier, puis le guide de contribution du projet.
- Signalez en commentaire que vous la prenez — cela évite que deux personnes fassent le même travail.
- Forkez le dépôt et travaillez sur une branche.
- Ouvrez une pull request qui référence le numéro de l'issue.
Autres issues de O2sa/DevImpact
-
a11y beginner friendly easy enhancement good first issue ui
Difficulté 2/5 1-3 heures Accessibilité débutants 82/100
Les mainteneurs répondent en général sous 1 jour
-
feature: Add custom branded not-found.tsx page for 404 routesPeut-être pris @sarmad-jalil l’a pris il y a 2 jours. Ouvertebeginner friendly easy feature good first issue nextjs ui
O2sa/DevImpact#271 · 2 commentaires · 1 personne assignée ·
Les mainteneurs répondent en général sous 1 jour
-
enhancement: Add 1-click Clear button (X) inside SearchFieldPeut-être pris @aryansinha1908 l’a pris il y a 4 jours. Ouvertebeginner friendly easy enhancement good first issue ui
O2sa/DevImpact#265 · 2 commentaires · 1 personne assignée ·
Les mainteneurs répondent en général sous 1 jour
Toutes les issues de O2sa/DevImpact
Issues similaires
-
New icon: `filter-clear`Ouvertecalcite-ui-icons enhancement impact - p3 - not time sensitive needs triage
Difficulté 2/5 1-3 heures Accessibilité débutants 67/100
Esri/calcite-design-system#15467 ·
Les mainteneurs répondent en général sous 1 jour
-
Difficulté 2/5 1-3 heures Accessibilité débutants 78/100
diegosouzapw/OmniRoute#18390 ·
-
Difficulté 2/5 1-3 heures Accessibilité débutants 65/100
-
Difficulté 1/5 1-3 heures Accessibilité débutants 86/100
guilhermerodz/input-otp#157 · 1 commentaire ·
-
Difficulté 2/5 1-3 heures Accessibilité débutants 82/100
davidjerleke/embla-carousel#1414 ·
Les mainteneurs répondent en général sous 4 jours