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

fix(ai-slop): detect.sh --paths-file drops the last entry when the file has no trailing newline

Geschlossen Anfängerfreundlich
  • Stimmt etwas nicht?
#8,974 1 Kommentar 0 Reaktionen 0 zugewiesene Personen Auf GitHub ansehen

Maintainer antworten meist innerhalb von 1 Tag

Ein zugehöriger Pull Request wurde bereits gemerged.

  • #9356 von @kyle-sexton — gemerged

Bewertung

Schwierigkeit
2/5
Geschätzter Aufwand
1-3 Stunden
Anfängerfreundlichkeit
86/100
Issue-Typ
Bug
Klarheit
Klar beschrieben
Aktivitätsstatus
Aktiv
Tech-Stack
bash
Bereich
cli

Rechercherichtung

Lies plugins/ai-slop/skills/audit/scripts/lib/resolve-targets.sh und finde die Leseschleife für --paths-file. Reproduziere das Problem mit einer Pfaddatei, deren letzter Eintrag keinen abschließenden Zeilenumbruch hat, und passe die Schleife wie beschrieben an. Füge einen resolve-targets-Test für diesen Fall hinzu und bestätige, dass der letzte Pfad verarbeitet wird.

Vom Indexierungsmodell aus dem Issue-Text verfasst.

Beschreibung

priority: low

plugins/ai-slop/skills/audit/scripts/lib/resolve-targets.sh reads --paths-file with while IFS= read -r line; do ... done. When the file's last line has no trailing newline, read returns nonzero after assigning that line, so the loop never processes it. A one-entry file then reports that it lists no paths and exits successfully, and a longer list silently omits its final file.

Reproduce, from a directory holding a.md and b.md:

printf 'a.md\nb.md' > list
bash plugins/ai-slop/skills/audit/scripts/detect.sh --list-targets --paths-file list   # lists a.md only
printf 'a.md\nb.md\n' > list2
bash plugins/ai-slop/skills/audit/scripts/detect.sh --list-targets --paths-file list2  # lists a.md and b.md

Fix: while IFS= read -r line || [[ -n $line ]]; do, with a resolve-targets test for a file with no trailing newline.

Found by Codex review on https://github.com/melodic-software/pr-pipeline-sandbox/pull/195 (a byte copy of this plugin). The pr-refine fix-docs lane is not affected, because it passes its targets in the prompt rather than through a paths file.

Vorherrschende Sprache
Shell
Sterne
22
Forks
2
Ø Merge
3 Std. 42 Min.
Gemergte PRs (30 T.)
938

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 melodic-software/claude-code-plugins

Alle Issues in melodic-software/claude-code-plugins

Ähnliche Issues

Weitere Issues zu Shell/Bash

Neue Issues direkt in Ihr Postfach

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