fix(source-control): ready hold check treats hyphenated identifiers containing wip as a WIP marker
维护者通常 1 天内回复
还没有人认领这个 Issue。
评估
调研方向
阅读 plugins/source-control/skills/pull-request/scripts/check-ready-holds.sh,重点查看第 77 行,并追踪其标题匹配器如何识别 WIP 标记。将报告中的标题添加为一个修复前会失败的 stay-quiet 测试,然后收紧匹配器,使带连字符的标识符不被视为 WIP 标记。完成标准是:就复现标题而言,ready 检查能够通过,同时仍能捕获独立的 WIP 标记。
由索引模型根据 Issue 内容生成。
描述
Problem
/source-control:pull-request ready refuses to flip a draft whose title names a hyphenated identifier containing wip, reporting a WIP marker.
plugins/source-control/skills/pull-request/scripts/check-ready-holds.sh:77 tests the title with test("\\bwip\\b"; "i"). A hyphen is a word boundary, so check-wip-cap matches.
Reproduction
PR #8967, title refactor(pr-pipeline): fold the admission window into check-wip-cap with a queued-work count:
title: a WIP marker in "refactor(pr-pipeline): fold the admission window into check-wip-cap with a queued-work count"
exit=3
Expected
A WIP marker is a title token such as a leading WIP, [WIP] or WIP:, not wip inside a hyphenated identifier. The ready check should pass this title.
Fix
Narrow the matcher, for example to a leading WIP/[WIP]/(WIP) token or wip that has no -, _ or alphanumeric character on either side. Per the hook-precision convention, land it with a stay-quiet test (the title above) that fails before the fix.
- 主要语言
- Shell
- 星标
- 22
- 派生
- 2
- 平均合并
- 6 小时 11 分钟
- 30 天内合并 PR
- 955
环境准备
- 没有 Dockerfile 或 Docker Compose 文件
- 没有 Pull Request 模板
- 阅读贡献指南
从这里开始
- 先读完整个 Issue,再读项目的贡献指南。
- 在 Issue 下留言说明你要接手 —— 这能避免两个人做同样的事。
- Fork 仓库,在一个分支上完成修改。
- 提交 Pull Request,并在描述里引用这个 Issue 编号。
melodic-software/claude-code-plugins 的其他 Issue
-
needs-triage
难度 2/5 1-3 小时 新手友好度 76/100
melodic-software/claude-code-plugins#9619 ·
维护者通常 1 天内回复
-
priority: medium status: ready
难度 2/5 1-3 小时 新手友好度 78/100
melodic-software/claude-code-plugins#9617 ·
维护者通常 1 天内回复
-
evals: run-validity calls runs VALID when every Bash call failed to start the sandbox (long TMPDIR)未关闭priority: medium status: needs-decision work-class: scoped
难度 2/5 1-3 小时 新手友好度 72/100
melodic-software/claude-code-plugins#9611 · 2 条评论 ·
维护者通常 1 天内回复
-
fix(pr-pipeline): apply-verdict readFields reads a malformed field-values answer as no fields set未关闭priority: low status: confirmed
难度 2/5 1-3 小时 新手友好度 83/100
melodic-software/claude-code-plugins#9598 · 1 条评论 ·
维护者通常 1 天内回复
-
priority: medium status: needs-decision work-class: scoped
难度 2/5 1-3 小时 新手友好度 72/100
melodic-software/claude-code-plugins#9578 · 2 条评论 ·
维护者通常 1 天内回复
查看 melodic-software/claude-code-plugins 的全部 Issue
相似的 Issue
-
enhancement
难度 2/5 1-3 小时 新手友好度 66/100
维护者通常 1 天内回复
-
难度 2/5 1-3 小时 新手友好度 68/100
Leonxlnx/taste-skill#164 ·
维护者通常 1 天内回复
-
reproduced
难度 2/5 1-3 小时 新手友好度 78/100
anomalyco/opencode#58651 · 2 条评论 ·
维护者通常 1 天内回复
-
难度 2/5 1-3 小时 新手友好度 78/100
-
使用脚本 DD 腾讯云 报错未关闭
难度 2/5 1-3 小时 新手友好度 82/100