kld-sdd 2.7.8-1 → 2.7.8-2

package/templates/git-hooks/consistency-check-core.cjs

@@ -49,7 +49,7 @@ function parseActiveChangesFromYaml(content) {
   for (const raw of content.split(/\r?\n/)) {
     const line = raw.replace(/#.*$/, '').replace(/\s+$/, '');
     if (!line.trim()) continue;
-    if (/^active_changes:\s*(\[\s*\])?\s*$/.test(line)) { inActive = true; continue; }
+    if (/^active_changes:\s*(\[\s*])?\s*$/.test(line)) { inActive = true; continue; }
     if (inActive) {
       const m = /^\s+-\s+change-key:\s*(\S+)/.exec(line);
       if (m) keys.push(m[1]);
@@ -144,17 +144,31 @@ function validateReportFields(report) {
 // --- 变更文件采集 ---
 
 /**
- * 简易 YAML 解析器：仅提取 code_repos 列表
+ * 简易 YAML 解析器：仅提取 code_repos 列表。
+ * 逐行状态机解析（无正则量词嵌套），天然免疫 ReDoS（Sonar S5852）。
+ * 容忍条目间注释、行尾注释；遇到下一个顶层 key（非缩进行）即停止。
  */
 function parseCodeRepos(yamlPath) {
   try {
     const content = fs.readFileSync(yamlPath, 'utf8');
-    const match = content.match(/code_repos:\s*\n((?:[ \t]*(?:#[^\n]*\n)?[ \t]*-[ \t]*\S+[ \t]*(?:#[^\n]*)?\n?)+)/);
-    if (!match) return null;
-    return match[1]
-      .split('\n')
-      .map(line => line.replace(/#.*$/, '').replace(/^\s+-\s+/, '').trim())
-      .filter(Boolean);
+    const repos = [];
+    let inList = false;
+    for (const raw of content.split(/\r?\n/)) {
+      const line = raw.replace(/#.*$/, '').replace(/\s+$/, '');
+      if (!line.trim()) continue;
+      if (!/^\s/.test(line)) {
+        // 顶层 key：进入 code_repos 列表 / 离开列表
+        inList = /^code_repos:\s*$/.test(line);
+        continue;
+      }
+      if (!inList) continue;
+      const item = line.trim();
+      if (item.startsWith('- ')) {
+        const value = item.slice(2).trim();
+        if (value) repos.push(value);
+      }
+    }
+    return repos.length > 0 ? repos : null;
   } catch {
     return null;
   }
@@ -185,10 +199,8 @@ function isTrivialDiffLine(line) {
   if (trimmed.startsWith('#')) {
     // 排除 C/C++ 预处理指令
     const rest = trimmed.slice(1).trimStart();
-    if (/^(include|define|ifdef|ifndef|endif|else|elif|undef|pragma|error|warning|line|region|endregion)\b/.test(rest)) {
-      return false;
-    }
-    return true; // Python/Shell/YAML 注释
+    return !/^(include|define|ifdef|ifndef|endif|else|elif|undef|pragma|error|warning|line|region|endregion)\b/.test(rest);
+     // Python/Shell/YAML 注释
   }
   if (trimmed.startsWith('/*') || trimmed.endsWith('*/')) return true; // 块注释开始/结束
   if (trimmed.startsWith('*')) return true; // 块注释续行 (* ... 或 ** ...)
@@ -252,7 +264,7 @@ function parseDiffForTrivialFiles(diffOutput) {
     if (line.startsWith('-') && !line.startsWith('---')) {
       hasAnyChange = true;
       if (!isTrivialDiffLine(line.slice(1))) hasSubstantive = true;
-      continue;
+
     }
   }
   if (currentFile !== null && hasAnyChange && !hasSubstantive) {
@@ -413,7 +425,7 @@ function parseConsistencySection() {
     let inSection = false;
     for (const raw of content.split(/\r?\n/)) {
       const line = raw.trim();
-      if (/^\[.+\]$/.test(line)) {
+      if (/^\[.+]$/.test(line)) {
         if (inSection) break;
         inSection = line.toLowerCase() === '[consistency]';
         continue;
@@ -493,9 +505,9 @@ function isBusinessCode(file) {
   if (/(^|\/)(__tests__|tests?|e2e|cypress|fixtures|mocks?)(\/|$)/i.test(normalized)) return false;
   if (/(^|\/)[^/]*(?:\.test|\.spec|_test)\.[a-z]+$/i.test(normalized)) return false;
   if (TOOLCHAIN_CONFIG_RE.test(base)) return false;
-  if (DOTFILE_CONFIG_RE.test(base)) return false;
+  return !DOTFILE_CONFIG_RE.test(base);
+
 
-  return true;
 }
 
 // --- 路径归一化：报告 coveredFiles → 代码仓相对 ---
@@ -968,7 +980,7 @@ function runConsistencyCheck(opts) {
 function runFilterMode() {
   const filesArg = readArg('files');
 
-  let input = [];
+  let input;
   if (filesArg) {
     if (filesArg.startsWith('@')) {
       input = fs.readFileSync(filesArg.slice(1), 'utf8').split(/\r?\n/);
@@ -1021,11 +1033,9 @@ function runCollectMode() {
   const businessFiles = files.filter(isBusinessCode);
   const excluded = files.filter(f => !isBusinessCode(f));
 
-  let hashedBusiness = null;
-  let unreadableFiles = null;
+  const hashedBusiness = [];
+  const unreadableFiles = [];
   if (withHash) {
-    hashedBusiness = [];
-    unreadableFiles = [];
     for (const rel of businessFiles) {
       const content = readWorkingFile(path.join(projectRoot, rel));
       if (content === null) {

package/templates/git-hooks/pre-commit

@@ -10,13 +10,24 @@ git_root=$(git rev-parse --show-toplevel 2>/dev/null) || {
   exit 1
 }
 hooks_dir=""
+# 统一反斜杠为正斜杠，保证 Windows 盘符路径（P:\a\b → P:/a/b）能被 case 判定为绝对路径
+normalize_path() {
+  printf '%s' "$1" | tr '\\' '/'
+}
+# 绝对路径（/ 开头或盘符 X: 开头）直接使用；相对路径基于 git 根拼接
+resolve_spec_dir() {
+  raw=$(normalize_path "$1")
+  case "$raw" in
+    /*|[A-Za-z]:*) printf '%s' "$raw" ;;
+    *) printf '%s' "$git_root/$raw" ;;
+  esac
+}
 spec_path=$(git -C "$git_root" config --local sdd.specPath 2>/dev/null)
 if [ -n "$spec_path" ]; then
-  case "$spec_path" in /*) ;; *) spec_path="$git_root/$spec_path" ;; esac
-  hooks_dir="$spec_path/skywalk-sdd/git-hooks"
+  hooks_dir="$(resolve_spec_dir "$spec_path")/skywalk-sdd/git-hooks"
 elif [ -f "$git_root/.sdd-spec-root" ]; then
   spec_rel=$(head -1 "$git_root/.sdd-spec-root" | sed 's/^[[:space:]]*//;s/[[:space:]]*$//')
-  case "$spec_rel" in /*) hooks_dir="$spec_rel/skywalk-sdd/git-hooks" ;; *) hooks_dir="$git_root/$spec_rel/skywalk-sdd/git-hooks" ;; esac
+  hooks_dir="$(resolve_spec_dir "$spec_rel")/skywalk-sdd/git-hooks"
 fi
 if [ -n "$hooks_dir" ] && [ ! -d "$hooks_dir" ]; then
   echo "[SDD] 已配置的 Spec 路径失效：$hooks_dir"
@@ -27,12 +38,15 @@ fi
 # 方法3: 从 git 根向上搜索 skywalk-sdd/git-hooks（spec 仓本身或父级）
 if [ -z "$hooks_dir" ]; then
   search_dir="$git_root"
-  while [ "$search_dir" != "/" ] && [ "$search_dir" != "" ]; do
+  while [ -n "$search_dir" ] && [ "$search_dir" != "/" ]; do
     if [ -d "$search_dir/skywalk-sdd/git-hooks" ]; then
       hooks_dir="$search_dir/skywalk-sdd/git-hooks"
       break
     fi
-    search_dir="$(dirname "$search_dir")"
+    # Windows 盘符根（P:/）dirname 后不变，需显式收敛，避免死循环
+    parent_dir="$(dirname "$search_dir")"
+    [ "$parent_dir" = "$search_dir" ] && break
+    search_dir="$parent_dir"
   done
 fi
 

package/templates/git-hooks/pre-push

@@ -10,13 +10,24 @@ git_root=$(git rev-parse --show-toplevel 2>/dev/null) || {
   exit 1
 }
 hooks_dir=""
+# 统一反斜杠为正斜杠，保证 Windows 盘符路径（P:\a\b → P:/a/b）能被 case 判定为绝对路径
+normalize_path() {
+  printf '%s' "$1" | tr '\\' '/'
+}
+# 绝对路径（/ 开头或盘符 X: 开头）直接使用；相对路径基于 git 根拼接
+resolve_spec_dir() {
+  raw=$(normalize_path "$1")
+  case "$raw" in
+    /*|[A-Za-z]:*) printf '%s' "$raw" ;;
+    *) printf '%s' "$git_root/$raw" ;;
+  esac
+}
 spec_path=$(git -C "$git_root" config --local sdd.specPath 2>/dev/null)
 if [ -n "$spec_path" ]; then
-  case "$spec_path" in /*) ;; *) spec_path="$git_root/$spec_path" ;; esac
-  hooks_dir="$spec_path/skywalk-sdd/git-hooks"
+  hooks_dir="$(resolve_spec_dir "$spec_path")/skywalk-sdd/git-hooks"
 elif [ -f "$git_root/.sdd-spec-root" ]; then
   spec_rel=$(head -1 "$git_root/.sdd-spec-root" | sed 's/^[[:space:]]*//;s/[[:space:]]*$//')
-  case "$spec_rel" in /*) hooks_dir="$spec_rel/skywalk-sdd/git-hooks" ;; *) hooks_dir="$git_root/$spec_rel/skywalk-sdd/git-hooks" ;; esac
+  hooks_dir="$(resolve_spec_dir "$spec_rel")/skywalk-sdd/git-hooks"
 fi
 if [ -n "$hooks_dir" ] && [ ! -d "$hooks_dir" ]; then
   echo "[SDD] 已配置的 Spec 路径失效：$hooks_dir"
@@ -27,12 +38,15 @@ fi
 # 方法3: 从 git 根向上搜索 skywalk-sdd/git-hooks（spec 仓本身或父级）
 if [ -z "$hooks_dir" ]; then
   search_dir="$git_root"
-  while [ "$search_dir" != "/" ] && [ "$search_dir" != "" ]; do
+  while [ -n "$search_dir" ] && [ "$search_dir" != "/" ]; do
     if [ -d "$search_dir/skywalk-sdd/git-hooks" ]; then
       hooks_dir="$search_dir/skywalk-sdd/git-hooks"
       break
     fi
-    search_dir="$(dirname "$search_dir")"
+    # Windows 盘符根（P:/）dirname 后不变，需显式收敛，避免死循环
+    parent_dir="$(dirname "$search_dir")"
+    [ "$parent_dir" = "$search_dir" ] && break
+    search_dir="$parent_dir"
   done
 fi
 

package/templates/hooks/claude/hooks/sdd-post-tool.cjs

@@ -171,8 +171,8 @@ function recordTelemetry(projectRoot, activeStage, record, result, exitCodeKnown
         skipped: counts.skipped,
       } : {}),
       duration_ms: Number.isFinite(toolResponse.duration_ms) ? toolResponse.duration_ms : 0,
-      failure_type: result === 'failure' ? (exitCode == null ? 'infrastructure' : 'assertion') : 'none',
-      expected_failure: tddPhase === 'red' && result === 'failure' && exitCode != null,
+      failure_type: result === 'failure' ? (exitCode != null && /ERR_ASSERTION|AssertionError|AssertionFailedError/.test(output) ? 'assertion' : 'infrastructure') : 'none',
+      expected_failure: tddPhase === 'red' && result === 'failure' && exitCode != null && /ERR_ASSERTION|AssertionError|AssertionFailedError/.test(output),
     };
     strictTestEvent = exitCode != null && Boolean(activeStage.task_id);
     runId = stableRunId(input || {}, activeStage, details.test_results.command);

package/templates/hooks/codebuddy/hooks/sdd-post-tool.cjs

@@ -160,8 +160,8 @@ function recordTelemetry(projectRoot, activeStage, record, result, exitCodeKnown
         skipped: counts.skipped,
       } : {}),
       duration_ms: Number.isFinite(toolResponse.duration_ms) ? toolResponse.duration_ms : 0,
-      failure_type: result === 'failure' ? (exitCode == null ? 'infrastructure' : 'assertion') : 'none',
-      expected_failure: tddPhase === 'red' && result === 'failure' && exitCode != null,
+      failure_type: result === 'failure' ? (exitCode != null && /ERR_ASSERTION|AssertionError|AssertionFailedError/.test(output) ? 'assertion' : 'infrastructure') : 'none',
+      expected_failure: tddPhase === 'red' && result === 'failure' && exitCode != null && /ERR_ASSERTION|AssertionError|AssertionFailedError/.test(output),
     };
     strictTestEvent = exitCode != null && Boolean(activeStage.task_id);
     runId = stableRunId(input || {}, activeStage, details.test_results.command);

package/templates/skills/kld-sdd/opsx-apply/SKILL.md

@@ -40,6 +40,7 @@ allowed-tools:
 > - 较大的 `--details-json` 负载可先写入文件，再通过 `--details-file="$(cat .sdd-spec-root)/skywalk-sdd/state/<变更名称>-<type>.json"` 传递（例如 `task_update` 对应 `<变更名称>-task-update.json`）。
 > - `task_update` 记录成功后会自动调用 `check-task` 更新 `tasks.md` 中的 checkbox；录入成功后，agent 可以通过 `check-task` 命令验证该 checkbox 已更新。
 > - **⚠️ P1-2 ai_adoption_review 必填 ai_diff**：记录 AI 产出快照时，`--details-json` 必须含 `ai_diff.files_changed`（即使=0）与 `ai_diff.files`（产出文件路径数组，非空），**不得只发 `assertions`**。`vcs_mode=readonly` 时 `ai_diff.added_lines` 不得为 null——用只读 `git diff --numstat HEAD` 取值（apply Git 只读策略允许）；`vcs_mode=no-git` 时可填 null。工具侧已加记录时硬校验：`files` 空数组或 `readonly` 下 `added_lines=null` 将 **拒绝记录**（与 conformance_review 校验对称）。缺失 `ai_diff` 会导致 report 变更文件数为 null（工具侧已加 `assertions[].files` 兜底，但 `ai_diff` 是主数据源）。完整模板见 `./reference.md`「§5.1」。
+> - **默认测试入口**：使用 reference.md 中的 test-run 执行、采集和关联真实证据；可用 completion-file 同步已验证任务。不手填退出码、测试计数和耗时。
 > - **严格证据链**：一次真实测试执行只写一条严格 `test_result`；任务完成事件只通过 `test_event_id` 引用成功的 green/refactor/regression 测试，并明确 `tdd_required=true/false`，不复制测试计数。测试不适用时必须写明原因。完整 schema 见 `./reference.md`。
 > - **任务涉及文件必记**：每条严格 `task_update` 必须在 `task_update.files` 写本任务实际修改的仓库相对路径；本任务确实没有改文件时，改填具体的 `no_file_change_reason`。两者至少有一个。
 > - **最终交付文件必记**：apply 所有文件就绪后记录严格 `final_output_snapshot`，其中 `final_output_snapshot.files` 是最终交付文件的仓库相对路径清单。报告把它与逐任务文件分开说明；旧 `ai_adoption_review.ai_diff.files` 只作兼容来源。

package/templates/skills/kld-sdd/opsx-apply/reference.md

@@ -2,6 +2,24 @@
 description: opsx-apply 的详细模板：telemetry 命令、worktree 全套策略、子代理派发、收尾脚本。仅在需要参考详细模板时读取。
 ---
 
+## 自动执行并取证（默认入口）
+
+使用 test-run 运行真实测试并记录同一条严格事件。工具采集命令参数、实际 cwd、退出码、耗时、测试计数和原始输出哈希；不让模型抄写这些字段。测试命令用 JSON 参数数组，无 shell 展开。
+
+```bash
+node "<spec-package>/skywalk-sdd/log.cjs" test-run --project="<spec-package>" --change=<变更名称> --session-id=<会话ID> --task-ids=<TASK-ID列表> --cwd="<代码仓>" --tdd-phase=regression --command-json='["node","--test","--test-reporter=tap","test/order-total.test.js"]'
+```
+
+RED 使用 --tdd-phase=red --expected-failure；真正的断言失败才能形成有效 RED。找不到测试入口/模块、编译/环境错误、超时均不能冒充业务红灯。命令返回实际测试退出码，失败仍保留事件与回执；不要因为 RED 非零就重复执行同一个 run-id。
+
+默认支持 Node TAP/Node 测试汇总。其他运行器输出无法可靠解析时 counts_known=false，不猜计数，不自动完成任务；已有适配器可继续用下方兼容入口提交实际报告。调用方明确声明 task-ids，工具不会把测试通过自动等同于覆盖了任意业务任务。
+
+需要测试成功后同步任务时，增加 --completion-file=<完成信息JSON文件>。内容为数组，每项包含 task_id、实际修改的仓库相对 files、tdd_required；TDD 任务同时写 tdd_pair_id/tdd_role。无文件修改填写 no_file_change_reason。工具将完成事件自动关联本轮 test_event_id 并同步任务勾选；RED、失败或未知计数不会自动完成任务。
+
+回执路径在输出 receipt 字段；原始 stdout/stderr 和执行清单保留在 skywalk-sdd/state/test-runs。记录失败后用原 details.json 重试 record --strict，禁止重写退出码或补造测试数。输出哈希不一致会被拒绝。自动 Hook 仅作补充，不对同一次 test-run 再手写一条 test_result。
+
+以下手写 test_result 模板仅保留给旧集成和已有测试报告适配器，新流程优先用 test-run。
+
 # opsx-apply — 详细参考（reference）
 
 > 本文件承载 opsx-apply 的重细节模板：telemetry 命令、§1.5 worktree 全套策略、§5c 子代理派发、§5.1 AI 产出快照、§6.0 单元测试真实执行、§6.1 worktree 收尾脚本。
@@ -95,7 +113,7 @@ node "$(cat .sdd-spec-root)/skywalk-sdd/log.cjs" record --strict --type=test_res
 - `expected_failure=true` 仅允许 `tdd_phase=red`
 - 有效 RED：`expected_failure=true` + `exit_code≠0` + `failure_type` 为 `assertion` 或 `contract`
 
-> **自动记录**：`sdd-post-tool.cjs` hook 会在 agent 运行 `mvn test`/`npm test` 等命令时自动记录 `test_result`，使用正确的 `test_results` key。但自动记录依赖 `activeStage.task_id`，若未设置则 `tdd_phase` 默认为 `regression`。需要精确标注 RED/GREEN 时建议手动记录。
+> **自动记录**：`sdd-post-tool.cjs` hook 会在 agent 运行 `mvn test`/`npm test` 等命令时自动记录 `test_result`，使用正确的 `test_results` key。但自动记录依赖 `activeStage.task_id`，若未设置则 `tdd_phase` 默认为 `regression`。需要精确标注 RED/GREEN 时使用上方 test-run 自动取证。
 
 ### process_note（阶段内过程事件：决策/范围变化/故障与恢复）
 

package/templates/skills/kld-sdd/opsx-archive/SKILL.md

@@ -45,7 +45,7 @@ allowed-tools:
 
 ## 启动流程
 
-### 1. 确认变更名称并记录阶段开始
+### 1. 确认变更名称
 
 如果未提供变更名，运行：
 
@@ -53,13 +53,7 @@ allowed-tools:
 openspec list
 ```
 
-让用户选择现有变更。确认后记录阶段开始：
-
-```bash
-node "$(cat .sdd-spec-root)/skywalk-sdd/log.cjs" start --command=archive --project=. --change=<变更名称> --agent=<Agent类型> --source=opsx-command --session-id=<会话ID>
-```
-
-保存 `event_id`。
+按参数、会话或唯一活动变更定位；只有多个候选且无法判断时请用户选择。此时尚未开始 archive 阶段。
 
 ### 2. 记录归档原因
 
@@ -85,6 +79,14 @@ node "$(cat .sdd-spec-root)/skywalk-sdd/log.cjs" doctor --project=. --change=<
 
 如果存在 `severe_issues`，暂停归档并说明必须先修复。`superseded_open_stages` 和 `rework_summary` 只作为返工/重复执行展示，不要求用户人工区分测试回滚或真实研发返工。
 
+诊断通过后才记录本轮 archive 开始，保存返回的 `event_id`：
+
+```bash
+node "$(cat .sdd-spec-root)/skywalk-sdd/log.cjs" start --command=archive --project=. --change=<变更名称> --agent=<Agent类型> --source=opsx-command --session-id=<会话ID>
+```
+
+不要在本轮阶段已开始但未结束时再次运行前置 doctor；历史未闭环阶段仍须按真实结果补齐，不得跳过诊断或删除事件。
+
 ### 4. 扫描 tasks 完成状态
 
 ```bash

package/templates/skills/kld-sdd/opsx-check/SKILL.md

@@ -44,7 +44,7 @@ node "<spec-package>/skywalk-sdd/log.cjs" start --command=check --project="<spec
 ## 2. 程序预检（在完整文档评审前）
 
 ```bash
-node "<spec-package>/skywalk-sdd/log.cjs" semantic-check --project="<spec-package>" --change=<变更名称> --profile=auto --include-tasks
+node "<spec-package>/skywalk-sdd/log.cjs" semantic-check --project="<spec-package>" --change=<变更名称> --profile=auto --include-tasks > "<spec-package>/skywalk-sdd/state/<变更名称>-preflight.json"
 ```
 
 输出 `profile / valid / revision / counts / diagnostics`、`review_inputs`（文档路径与哈希、历史身份哈希）、`task_completion` 和 `timings_ms`。程序时间不等于模型评审总耗时。
@@ -93,15 +93,17 @@ ${CHECK_REVIEW_DELEGATION_GUIDANCE}
 
 ## 4. 一次记录与结束
 
-评审完成后，主 Agent 读取 [reference.md](reference.md)，将结构化结果按 [result-template.json](result-template.json) 写入本轮临时 details 文件，执行唯一的 strict check_result 命令，再执行 end。不要在 shell 中手工转义大段 JSON。
+将实际评审项和真实会话身份保存为 [review-template.json](review-template.json) 格式。每项只写唯一 ID、类别、结论和证据；工具自动计算统计、读取任务状态、判断 partial/failure、关联阶段结束。旧 [result-template.json](result-template.json) 仅保留给已有集成。
 
-顶层 `total` 必须是大于 0 的整数且来自实测，不能照抄模板的 0。空 details 导致 Q3 不可计算，strict 会拒绝。consistency_score 是 legacy 兼容字段，不可评分填 null。未执行项不能计入通过；warning 记录明细和处置理由。
+类别：completeness / consistency / executability / tdd_compliance；状态：passed / error / warning / suggestion / na。所有项包含证据或不适用理由，na 不计入通过。不要手填 total、errors、fixed_before_apply 或任务计数，不把模板占位内容当作已评审。
+
+兼容旧集成时，顶层 `total` 必须是大于 0 的整数，且来自实际评审项；空 details 导致记录被拒绝、Q3 不可计算。`consistency_score` 仅为 legacy 字段，不能替代有证据的分类检查。新入口由工具计算这些统计，评审者无需重复填写。
 
 ```bash
-node "<spec-package>/skywalk-sdd/log.cjs" end --event-id=<event_id> --command=check --project="<spec-package>" --change=<变更名称> --capability=<可选capability-name> --agent=<Agent类型> --source=opsx-command --session-id=<父会话ID> --result=success/partial/failure --summary="本轮实测摘要"
+node "<spec-package>/skywalk-sdd/log.cjs" check-record --project="<spec-package>" --change=<变更名称> --run-id=<本轮Check稳定ID> --session-id=<父会话ID> --agent=<Agent类型> --preflight-file="<spec-package>/skywalk-sdd/state/<变更名称>-preflight.json" --review-file="<spec-package>/skywalk-sdd/state/<变更名称>-review.json" --end-event-id=<event_id>
 ```
 
-结束命令仍执行一次全量对账，核验评审期间文件/历史身份的变化；这次复核有时间边界意义，保留它。默认 auto 模式对照返回 semantic_state.revision 与预检 revision；不一致时不能宣称刚审过的版本仍有效，重新预检并复核变动及受影响的依赖，另记新一轮结果。显式 strict 时 end 的 auto revision 不能直接与 strict revision 比较；最终额外运行一次 --profile=strict 的 semantic-check 并比较 strict revision，auto 结果不能替代 strict 门禁。
+check-record 按预检的同一 profile 重新核验正文、文件集合和历史身份。输入变化返回 CHECK_REVIEW_STALE，不写本轮通过记录；重新预检并复核受影响内容。成功时仍通过原 strict 校验和阶段结束入口，不另手动执行 end。评审输入始终保留，记录失败可以修复后重试。未真实开展评审时不用此命令，按原 end 命令记录 failure 和原因。
 
 报告输出五维结论、多仓配置、KB 状态、证据与未执行项，说明覆盖范围。只请求检查则结束；已授权实现且无阻断项时进入 apply。修复文档后再检查相关依赖，禁止携带旧结论直接通过；不把 Capability 局部评审写成全 Change 通过。
 

package/templates/skills/kld-sdd/opsx-check/reference.md

@@ -1,5 +1,7 @@
 # Check 结果记录（主 Agent 在评审完成后读取）
 
+新流程优先使用 SKILL.md 中的 check-record 与 [review-template.json](review-template.json)：工具计算统计、补任务状态、校验评审输入是否过期，并可一次结束阶段。以下完整字段和 record --strict 保留给已有脚本，不要求新流程手填。
+
 收到实际评审结果后，必须把结果转换为 `check_result` 事件，并在 `stage_end` 之前执行成功。禁止用 stage_end 代替已执行评审的证据。程序预检阻断且未开展评审时，只记录阶段 failure 与真实原因，不伪造 reviewer 或零项评审。
 
 字段口径：
@@ -21,7 +23,7 @@
 - `categories`: 必须包含 `completeness`、`consistency`、`executability`、`tdd_compliance`；四类 total 合计等于顶层 total，四类未通过数合计等于 errors。算法正确性计入 consistency，文字报告单列，避免重复计数。
 - `task_completion`: 复用本轮 `semantic-check --include-tasks` 的输出（使用 tasks-status 同一扫描器），只取模板中的计数字段；未进入 apply 时 `checked_for_archive_readiness=false`。
 
-将评审结果写为 UTF-8 JSON：`skywalk-sdd/state/<变更名称>-check-<run-id>.json`，结构见 [result-template.json](result-template.json)。其中 `0` 是待替换位置，执行前换成本轮实测值，顶层 `total` 必须是大于 0 的整数。不要直接把安装目录中的模板传给命令（details 文件成功读取后默认清理）。
+将评审结果写为 UTF-8 JSON：`skywalk-sdd/state/<变更名称>-check-<run-id>.json`，结构见 [result-template.json](result-template.json)。其中 `0` 是待替换位置，执行前换成本轮实测值，顶层 `total` 必须是大于 0 的整数。不要直接把安装目录中的模板传给命令（details 文件仅在校验、记录和投影成功后清理，失败保留）。
 
 ```bash
 node "<spec-package>/skywalk-sdd/log.cjs" record --strict --type=check_result --command=check --project="<spec-package>" --change=<变更名称> --capability=<可选capability-name> --run-id=<本轮Check稳定ID> --agent=<Agent类型> --source=opsx-command --session-id=<父会话ID> --result=success/partial/failure --summary="检查结果摘要" --details-file=skywalk-sdd/state/<变更名称>-check-<run-id>.json

package/templates/skills/kld-sdd/opsx-check/review-template.json

@@ -0,0 +1,22 @@
+{
+  "reviewer": {
+    "reviewer": "<实际评审者>",
+    "review_execution_mode": "subagent",
+    "reviewer_agent_id": "<真实子代理ID>",
+    "parent_session_id": "<主会话ID>",
+    "review_session_id": "<评审会话ID>",
+    "author_session_id": "<作者会话ID>",
+    "reviewer_independence": "independent-review",
+    "fallback_reason_code": null,
+    "fallback_reason": null
+  },
+  "items": [
+    {
+      "id": "<唯一检查项ID>",
+      "category": "consistency",
+      "status": "passed",
+      "evidence": "<实际文件:行号或锚点及检查依据>"
+    }
+  ],
+  "warning_dispositions": []
+}

package/templates/skills/kld-sdd/opsx-check/reviewer.md

@@ -15,10 +15,10 @@
 
 #### 4.1 完整性检查
 
-- [ ] proposal.md 存在且包含完整章节
+- [ ] proposal.md 存在，实际业务意图、能力、范围与约束完整；不要求无关空章节
 - [ ] 每个 Capability 都有 spec.md
 - [ ] Full 模式每个 spec.md 都有对应 design.md，每个 design.md 都有对应 tasks.md；Simple 未生成这些文档不误报缺失
-- [ ] 所有文档符合模板结构
+- [ ] 所有文档保留解析需要的 frontmatter、身份、锚点与关系，内容规模匹配实际变更
 
 #### 4.2 一致性检查
 
@@ -93,8 +93,8 @@
 
 ## 返回给主 Agent
 
-一次返回五维结论和 [result-template.json](result-template.json) 中的 `check_results`（替换全部占位值），以及检查范围、逐项证据路径/锚点、未执行项及原因、阻断问题、warning 明细和修复建议。算法问题归到四类统计中的 `consistency`，五维文字报告仍单列算法；不得重复计数。
+一次返回 [review-template.json](review-template.json) 的 items、检查范围和必要问题说明；身份由主 Agent 根据真实工具结果补齐。每项包含唯一 id、category、status、evidence。算法归入 consistency，不重复计数；不手算总数，不复述所有通过项，不复制文档全文。
 
-顶层 `total` 必须是大于 0 的整数；四类 categories.total 合计等于 total，四类未通过数合计等于 errors。N/A 项 total=0。不要为使计数好看填“全部通过”。任务状态直接复用程序输出，reviewer 身份使用真实工具返回 ID；无法确认作者身份不能标为 independent-review。
+至少有一个实际适用项。未执行项使用 na 并写原因，不能为计数填“全部通过”。任务状态由工具读取，不能伪造 reviewer 或独立性。主 Agent 无需为统计不匹配重新委派评审。
 
-评审期间若发现文件变化，返回受影响路径，不能用旧版本结果覆盖新文档。只读检查不修文档、不改代码。
+评审期间若发现文件变化，返回受影响路径。只读检查不修文档、不改代码。