Hacktoberfest 2026:维护者为十月标记出来的 issue,仍然开放、适合新手。 浏览 Hacktoberfest issue

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

未关闭 适合新手
#8,974 1 条评论 0 个 reaction 已指派 0 人 在 GitHub 查看

维护者通常 1 天内回复

还没有人认领这个 Issue。

评估

难度
2/5
预计耗时
1-3 小时
新手友好度
86/100
Issue 类型
缺陷
描述清晰度
描述清楚
活跃度
活跃
技术栈
bash
领域
cli

调研方向

阅读 plugins/ai-slop/skills/audit/scripts/lib/resolve-targets.sh,找到读取 --paths-file 的循环。使用末尾条目没有尾随换行符的路径文件重现问题,然后按描述更新循环。为该情况添加一个 resolve-targets 测试,并确认最后一个路径得到处理。

由索引模型根据 Issue 内容生成。

描述

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.

主要语言
Shell
星标
22
派生
2
平均合并
7 小时 23 分钟
30 天内合并 PR
950

环境准备

  • 没有 Dockerfile 或 Docker Compose 文件
  • 没有 Pull Request 模板
  • 阅读贡献指南

从这里开始

  1. 先读完整个 Issue,再读项目的贡献指南。
  2. 在 Issue 下留言说明你要接手 —— 这能避免两个人做同样的事。
  3. Fork 仓库,在一个分支上完成修改。
  4. 提交 Pull Request,并在描述里引用这个 Issue 编号。

melodic-software/claude-code-plugins 的其他 Issue

查看 melodic-software/claude-code-plugins 的全部 Issue

相似的 Issue

更多 Shell/Bash Issue

把新 issue 发到你的邮箱

精选适合新手参与的 GitHub issue 摘要。