dev-playbooks-cn 1.2.4 → 1.2.5

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "dev-playbooks-cn",
-  "version": "1.2.4",
+  "version": "1.2.5",
   "description": "AI-driven spec-based development workflow",
   "keywords": [
     "devbooks",

package/skills/devbooks-code-review/SKILL.md

@@ -148,6 +148,15 @@ override dispose() {
 | 无 spec deltas | 归档完成 | 无需其他 skill |
 | 发现重大问题 | 交回 `devbooks-coder` | 归档前修复问题 |
 
+### Reviewer 专属权限：设置 verification.md Status
+
+**只有 Reviewer 可以将 `verification.md` 的 Status 设为 `Done`**。
+
+Review 通过后，Reviewer 必须执行：
+1. 打开 `<change-root>/<change-id>/verification.md`
+2. 将 `- Status: Ready` 改为 `- Status: Done`
+3. 这是归档的前置条件（`change-check.sh --mode archive` 会检查）
+
 ### 输出模板
 
 完成 code-review 后，输出：

package/skills/devbooks-coder/SKILL.md

@@ -141,6 +141,7 @@ devbooks change-evidence <change-id> --label green-final -- npm test
 ### 角色边界约束
 - **禁止修改 `tests/**`**（需要改测试必须交还 Test Owner）
 - **禁止修改 `verification.md`**（由 Test Owner 维护）
+- **禁止修改 `verification.md` 的 Status 字段**（只有 Reviewer 可以设为 Done）
 - **禁止修改 `.devbooks/`、`build/`、工程配置文件**（除非 proposal.md 明确声明）
 
 ### 代码质量约束

package/skills/devbooks-delivery-workflow/scripts/change-check.sh

@@ -410,6 +410,25 @@ check_verification() {
     return 0
   fi
 
+  # ==========================================================================
+  # AC-010: Verification Status Check
+  # Status field controls change package lifecycle: Draft → Ready → Done → Archived
+  # Only Reviewer can set Status to "Done" (Coder/Test Owner prohibited)
+  # ==========================================================================
+  if [[ "$mode" == "archive" || "$mode" == "strict" ]]; then
+    local status_line
+    status_line=$(rg -n "^- Status[:：] *(Draft|Ready|Done|Archived)\b" "$verification_file" -m 1 || true)
+    if [[ -z "$status_line" ]]; then
+      err "verification missing Status line (e.g., '- Status: Done'): ${verification_file}"
+    else
+      local status_value
+      status_value="$(echo "$status_line" | sed -E 's/^[0-9]+:- Status[:：] *//')"
+      if [[ "$status_value" != "Done" && "$status_value" != "Archived" ]]; then
+        err "verification Status must be 'Done' or 'Archived' for ${mode} (current: '${status_value}'). Only Reviewer can set Status to Done: ${verification_file}"
+      fi
+    fi
+  fi
+
   for h in "A\) (Test Plan Directive Table|测试计划指令表)" "B\) (Traceability Matrix|追溯矩阵)" "C\) (Execution Anchors|执行锚点)" "D\) (MANUAL-\* Checklist|MANUAL-\* 清单)"; do
     if ! rg -n "${h}" "$verification_file" >/dev/null; then
       err "verification missing section '${h}': ${verification_file}"

package/skills/devbooks-delivery-workflow/scripts/change-scaffold.sh

@@ -263,7 +263,13 @@ cat <<'EOF' | render_template | write_file "${change_dir}/verification.md"
 ## Metadata
 
 - Change ID: `__CHANGE_ID__`
-- Status: Draft | Ready | Done | Archived
+- Status: Draft
+  > Status lifecycle: Draft → Ready → Done → Archived
+  > - Draft: Initial state
+  > - Ready: Test plan ready (set by Test Owner)
+  > - Done: All tests passed + Review approved (set by **Reviewer only**)
+  > - Archived: Archived (set by Spec Gardener)
+  > **Constraint: Coder is prohibited from modifying Status field**
 - References:
   - Proposal: `__CHANGE_ROOT__/__CHANGE_ID__/proposal.md`
   - Design: `__CHANGE_ROOT__/__CHANGE_ID__/design.md`

package/skills/devbooks-test-owner/SKILL.md

@@ -40,6 +40,17 @@ allowed-tools:
 
 Test Owner 必须产出结构化的 `verification.md`，同时作为测试计划和追溯文档。
 
+### Status 字段权限
+
+| 状态 | 含义 | 谁可以设置 |
+|------|------|-----------|
+| `Draft` | 初始状态 | 自动生成 |
+| `Ready` | 测试计划就绪 | **Test Owner** |
+| `Done` | Review 通过 | Reviewer（禁止 Test Owner/Coder） |
+| `Archived` | 已归档 | Spec Gardener |
+
+**约束**：Test Owner 完成测试计划后，应将 Status 设为 `Ready`。
+
 ```markdown
 # 验证计划：<change-id>