Hacktoberfest 2026: die Issues, die Maintainer für den Oktober markiert haben – offen und einsteigerfreundlich. Hacktoberfest-Issues durchsuchen

Warn about common code smells in task snippets🔍

Offen Anfängerfreundlich
  • Stimmt etwas nicht?
#647 0 Kommentare 0 Reaktionen 0 zugewiesene Personen Auf GitHub ansehen

Maintainer antworten meist innerhalb von 2 Tagen

Dieses Issue hat noch niemand übernommen.

Bewertung

Schwierigkeit
2/5
Geschätzter Aufwand
1-3 Stunden
Anfängerfreundlichkeit
72/100
Issue-Typ
Feature
Klarheit
Klar beschrieben
Aktivitätsstatus
Aktiv
Tech-Stack
javascript, react, vite
Bereich
developer-experience, frontend

Rechercherichtung

Beginne in client/src/components/Task/TaskModal.jsx, wo laut Issue Snippets bearbeitet und angezeigt werden. Lies den Ablauf des Snippet-Editors und ermittle die Werte für den Snippet-Code und die Sprache; füge dann die vorgeschlagenen Prüfungen hinzu und zeige ihre Warnungen unter dem Editor an. Fertig ist die Aufgabe, wenn jedes aufgeführte Muster für den Snippet-Text seine Warnung auslöst und keine Änderungen am Backend oder an Paketen erforderlich sind.

Vom Indexierungsmodell aus dem Issue-Text verfasst.

Beschreibung

enhancement good first issue

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!! 🔥

Vorherrschende Sprache
JavaScript
Sterne
39
Forks
105
Ø Merge
1 T. 12 Std.
Gemergte PRs (30 T.)
45

Entwicklungsumgebung

Erste Schritte

  1. Lesen Sie das ganze Issue und danach den Beitragsleitfaden des Projekts.
  2. Schreiben Sie ins Issue, dass Sie es übernehmen — das erspart doppelte Arbeit.
  3. Forken Sie das Repository und arbeiten Sie in einem Branch.
  4. Öffnen Sie einen Pull Request, der die Issue-Nummer nennt.

Mehr aus anoopcodehack/DevBoard

Alle Issues in anoopcodehack/DevBoard

Ähnliche Issues

Weitere Issues zu JavaScript

Neue Issues direkt in Ihr Postfach

Eine kurze Übersicht über anfängerfreundliche GitHub-Issues.