Warn about common code smells in task snippets🔍
Les mainteneurs répondent en général sous 2 jours
Personne n'a encore pris cette issue.
Évaluation
- Difficulté
- 2/5
- Temps estimé
- 1-3 heures
- Accessibilité débutants
- 72/100
- Type d'issue
- Fonctionnalité
- Clarté
- Clairement spécifiée
- Activité
- Active
- Stack technique
- javascript, react, vite
- Domaine
- developer-experience, frontend
Piste de recherche
Commencez dans client/src/components/Task/TaskModal.jsx, où l’issue indique que les snippets sont modifiés et affichés. Lisez le parcours de l’éditeur de snippets et identifiez les valeurs du code et du langage du snippet ; ajoutez ensuite les vérifications suggérées et affichez leurs avertissements sous l’éditeur. Le travail est terminé lorsque chaque motif répertorié déclenche son avertissement pour le texte du snippet et qu’aucune modification du backend ou des packages n’est nécessaire.
Rédigé par le modèle d'indexation à partir du texte de l'issue.
Description
Warn about common code smells in task snippets
File: client/src/components/Task/TaskModal.jsx
Snippets get saved but nobody checks
quality 💀 catch bad patterns early fr!!
Fix: simple regex-based smell detector:
const detectSmells = (code, language) => {
const smells = []
if (code.includes('console.log'))
smells.push('🟡 console.log found — remove before PR')
if (code.includes('TODO') || code.includes('FIXME'))
smells.push('🔴 TODO/FIXME comment — resolve first')
if (/var\s/.test(code) && language === 'javascript')
smells.push('🟡 var used — prefer const/let')
if (code.split('\n').length > 50)
smells.push('🔴 Long function — consider splitting')
if (code.includes('password') &&
!code.includes('hash') &&
!code.includes('bcrypt'))
smells.push('🔴 Possible plain-text password!')
if (code.includes('SELECT *'))
smells.push('🟡 SELECT * — specify columns')
return smells
}
// show below snippet editor:
{snippetCode && detectSmells(snippetCode, snippetLang)
.map((smell, i) => (
{smell}
))
}
~20 lines! Zero packages needed ✅
No backend needed 🎯
Teaches good practices to contributors!! 🔥
- Langage dominant
- JavaScript
- Étoiles
- 39
- Forks
- 105
- Merge moyen
- 1 j 12 h
- PR mergées (30 j)
- 45
Préparer son environnement
- Aucun Dockerfile ni fichier Docker Compose
- Aucun 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 anoopcodehack/DevBoard
-
enhancement good first issue
Difficulté 2/5 1-3 heures Accessibilité débutants 68/100
anoopcodehack/DevBoard#648 ·
Les mainteneurs répondent en général sous 2 jours
-
Show "No snippets yet" empty state in task modal 💻Peut-être pris @tu-lasizzzz l’a pris aujourd’hui. Ouvertegood first issue
Difficulté 1/5 Moins d'une heure Accessibilité débutants 96/100
anoopcodehack/DevBoard#639 · 2 commentaires · 1 personne assignée ·
Les mainteneurs répondent en général sous 2 jours
-
Improve tags input placeholder with examples 🏷️Peut-être pris @SwatiManjari l’a pris il y a 1 jour. Ouvertegood first issue
Difficulté 1/5 Moins d'une heure Accessibilité débutants 96/100
anoopcodehack/DevBoard#638 ·
Les mainteneurs répondent en général sous 2 jours
-
good first issue
Difficulté 1/5 Moins d'une heure Accessibilité débutants 96/100
anoopcodehack/DevBoard#637 ·
Les mainteneurs répondent en général sous 2 jours
-
enhancement good first issue
Difficulté 2/5 1-3 heures Accessibilité débutants 70/100
anoopcodehack/DevBoard#630 ·
Les mainteneurs répondent en général sous 2 jours
Toutes les issues de anoopcodehack/DevBoard
Issues similaires
-
[External Plugin]: CarboneOuverteexternal-plugin ready-for-review third party
Difficulté 2/5 1-3 heures Accessibilité débutants 63/100
github/awesome-copilot#5259 · 1 commentaire ·
Les mainteneurs répondent en général sous 1 jour
-
[Bug]: fileUploadMatrix.test.ts URL case does a real DNS lookup and fails offlinePeut-être pris @r2jaff l’a pris aujourd’hui. Ouverte
Difficulté 2/5 1-3 heures Accessibilité débutants 72/100
ComposioHQ/composio#5054 ·
Les mainteneurs répondent en général sous 1 jour
-
Difficulté 2/5 1-3 heures Accessibilité débutants 82/100
addyosmani/agent-skills#717 ·
Les mainteneurs répondent en général sous 3 jours
-
[Bug] Certificate expiry sorting can reverse UTC order across a viewer daylight-saving gapPeut-être pris Une pull request liée à cette issue est ouverte ou déjà fusionnée. Ouverte
Difficulté 2/5 1-3 heures Accessibilité débutants 76/100
apache/rocketmq-dashboard#7318 ·
Les mainteneurs répondent en général sous 3 jours
-
W3cOuverte
Difficulté 1/5 Moins d'une heure Accessibilité débutants 88/100
web-platform-tests/wpt#64017 ·
Les mainteneurs répondent en général sous 1 jour