Repository navigation
refactor: 分离 v2 Skill 与旧流程恢复规则 - #138
Merged
Merged
Conversation
Merged
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
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
默认 Skill 混入旧审批、契约和调试状态指令,可能让 v2 新任务重复确认或走入旧流程。本次将兼容规则移到按需读取的文档,保留已有变更的恢复义务。
旧 Quick、Hotfix、Lightweight 也可能记录 direct,因此同时核对收据版本;缺失或损坏的证据仍要求恢复,不自动迁移或降级。兼容文档随 bundled runtime 分发。
Evidence
After: 新增 3 项回归检查通过,覆盖真实 CLI 生成的旧 direct 收据、非仓库目录读取及默认路径隔离;安装后的资源读取检查通过。
npm run build、npm test(922/922)、npm run validate、npm run check-versions、git diff --check均通过。Merge Danger
Door: two-way
可通过回滚恢复原指令布局;不迁移已有状态,不修改发布版本。
Blast Radius: workflow
影响默认 Skill 的读取与旧变更恢复指引,以及兼容文档的 runtime 读取。原有 docs/reviews/ 未包含在提交中。