Merged
Conversation
将 generateCommandContent() 的「先读取 SKILL.md…然后执行」 两步分离模式改为一气呵成指令流： - 保留读取 SKILL.md（保留行为上下文如骨架填充指引） - 保留执行 run_skill.js（保留脚本功能） - 合并为有序步骤列表 + 「全程不要停顿」约束 - 消除 Claude 在 Read/Bash 工具调用间的停顿等待 - allowed-tools 恢复使用 SKILL.md 中定义的工具集
为 4 个零覆盖的斜杠命令核心函数添加 22 条测试： - parseFrontmatter: 6 条 (标准解析/空输入/引号/连字符/无效行/CRLF) - scanInvocableSkills: 6 条 (过滤/hasScripts/relPath/空目录/无name/真实目录) - generateCommandContent: 6 条 (脚本模式/知识库模式/可选字段/默认值/转义) - installGeneratedCommands: 4 条 (生成/备份/空结果/内容格式) 测试总数: 66 → 88 (全部通过)
新增 describe('斜杠命令回归防护') 包含 6 条回归测试：
- SKILL.md 路径有效性烟雾测试（2条）
- 脚本引用完整性（2条）
- command 文件名合法性（2条）
防止 skills 目录重组后路径失效的问题再次发生。
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
parseFrontmatter,scanInvocableSkills,generateCommandContent,installGeneratedCommands）的单元测试覆盖0eb9f45事件复发防线）回归防护测试详情
generateCommandContent()路径映射/^[a-z][a-z0-9-]*$/Test plan
npx jest test/install.test.js --verbose— 46 条全部通过