@xenonbyte/da-vinci-workflow 0.1.23 → 0.1.24

package/CHANGELOG.md

@@ -1,5 +1,20 @@
 # Changelog
 
+## v0.1.24 - 2026-03-29
+
+### Added
+- `da-vinci supervisor-review` now supports structured reviewer provenance fields: `Configured reviewers`, `Executed reviewers`, and `Review source`
+- `design-supervisor` compatibility binary (`design-supervisor review`) that forwards to `da-vinci supervisor-review`
+- `scripts/test-supervisor-review-cli.js` coverage for alias behavior, inferred mode, and skill-source reviewer execution metadata
+- `scripts/test-supervisor-review-integration.js` optional real `codex exec` smoke test (`DA_VINCI_RUN_SUPERVISOR_INTEGRATION=1`)
+
+### Changed
+- supervisor-review audit parsing now accepts latest `## Design-Supervisor Review (Round X Attempt)` sections, multiline issue/outcome fields, and `Result` as a status alias
+- completion audit now enforces skill-backed supervisor evidence (`Review source: skill` + executed configured reviewers) when `Require Supervisor Review: true`
+- design prompts/skill docs now require configured reviewer skills to actually execute review before writing supervisor results
+- command references and constraint docs (EN/ZH) now document structured reviewer fields and required skill-backed behavior for hard-gate projects
+- `da-vinci supervisor-review --run-reviewers` now supports parallel execution (`--review-concurrency`) and resilient retries (`--review-retries`, `--review-retry-delay-ms`)
+
 ## v0.1.23 - 2026-03-29
 
 ### Added

package/README.md

@@ -28,10 +28,12 @@ This workflow is intended for:
 
 Latest published npm package:
 
-- `@xenonbyte/da-vinci-workflow@0.1.23`
+- `@xenonbyte/da-vinci-workflow@0.1.24`
 
 Release highlights:
 
+- `da-vinci supervisor-review --run-reviewers` now supports configurable parallel reviewer execution and resilient retry backoff (`--review-concurrency`, `--review-retries`, `--review-retry-delay-ms`)
+- added optional `test:supervisor-review-integration` smoke coverage for real `codex exec` review runner flows (enabled via `DA_VINCI_RUN_SUPERVISOR_INTEGRATION=1`)
 - audit parser logic is now split into `lib/audit-parsers.js`, reducing `lib/audit.js` size and making maintenance safer
 - recursive file traversal now uses bounded safe scans with depth/count limits and symlink skipping
 - completion/integrity audit now rejects out-of-root `.pen` references from `design-registry.md`
@@ -454,6 +456,7 @@ da-vinci audit --mode completion --change <change-id> /abs/path/to/project
 da-vinci icon-sync                      # tolerant by default; add --strict for hard gating
 cp references/icon-aliases.example.json ~/.da-vinci/icon-aliases.json
 da-vinci icon-search --query "settings lock" --family material --top 8
+da-vinci supervisor-review --project /abs/path/to/project --change <change-id> --run-reviewers --write
 da-vinci preflight-pencil --ops-file /abs/path/to/ops.txt
 da-vinci uninstall --platform codex,claude,gemini
 ```
@@ -498,6 +501,14 @@ Both modes check the most common workflow-integrity failures in a project:
 - returns ranked candidates with ready-to-use `icon_font` node payload hints
 - for a copy-ready `Icon System Guidance (Advisory)` template, see `docs/constraint-files.md` and `references/artifact-templates.md`
 
+`da-vinci supervisor-review` provides a local structured reviewer record:
+
+- writes `Configured reviewers`, `Executed reviewers`, `Review source`, `Status`, `Issue list`, and `Revision outcome` into `pencil-design.md` when `--write` is provided
+- accepts explicit status input (`--status PASS|WARN|BLOCK`) or can infer a conservative status from current design artifacts
+- for required supervisor gates, prefer `--run-reviewers --write` so configured reviewer skills execute and the record is persisted in one step
+- reviewer execution tuning is available via `--review-concurrency`, `--review-retries`, and `--review-retry-delay-ms` (exponential backoff)
+- keeps `design-supervisor review` available as a compatibility alias
+
 When Pencil MCP is active, Da Vinci now also expects an MCP runtime gate record in `pencil-design.md` before terminal completion claims. That runtime gate checks live editor/source convergence separately from filesystem audit.
 During active redesign work, run `da-vinci audit --mode integrity <project-path>` immediately after the first successful Pencil write, then use `da-vinci preflight-pencil` plus smaller follow-up batches if the same anchor surface rolls back twice.
 

package/README.zh-CN.md

@@ -30,10 +30,12 @@ Da Vinci 是一个把产品需求一路推进到结构化规格、Pencil 设计
 
 最新已发布 npm 包：
 
-- `@xenonbyte/da-vinci-workflow@0.1.23`
+- `@xenonbyte/da-vinci-workflow@0.1.24`
 
 已发布版本重点：
 
+- `da-vinci supervisor-review --run-reviewers` 现已支持 reviewer 并发执行与失败重试退避（`--review-concurrency`、`--review-retries`、`--review-retry-delay-ms`）
+- 新增可选 `test:supervisor-review-integration` 烟测，用于覆盖真实 `codex exec` 评审执行链路（通过 `DA_VINCI_RUN_SUPERVISOR_INTEGRATION=1` 启用）
 - audit 解析职责已拆分到 `lib/audit-parsers.js`，`lib/audit.js` 体量下降，可维护性更好
 - 递归文件扫描改为安全有界遍历：增加深度/数量上限，并跳过符号链接
 - completion/integrity audit 现在会拦截并忽略 `design-registry.md` 中越出项目根目录的 `.pen` 引用
@@ -375,6 +377,7 @@ da-vinci audit --mode completion --change <change-id> /abs/path/to/project
 da-vinci icon-sync                      # 默认容错；需要强门禁时加 --strict
 cp references/icon-aliases.example.json ~/.da-vinci/icon-aliases.json
 da-vinci icon-search --query "settings lock" --family material --top 8
+da-vinci supervisor-review --project /abs/path/to/project --change <change-id> --run-reviewers --write
 da-vinci preflight-pencil --ops-file /abs/path/to/ops.txt
 da-vinci uninstall --platform codex,claude,gemini
 ```
@@ -419,6 +422,14 @@ Context Delta 与 audit 的关系：
 - 返回排序候选，并附带可直接复用的 `icon_font` 节点 payload 提示
 - `Icon System Guidance (Advisory)` 可复制模板见 `docs/constraint-files.md` 与 `references/artifact-templates.md`
 
+`da-vinci supervisor-review` 提供本地结构化评审记录能力：
+
+- 加上 `--write` 时，会把 `Configured reviewers`、`Executed reviewers`、`Review source`、`Status`、`Issue list`、`Revision outcome` 写入 `pencil-design.md`
+- 可通过 `--status PASS|WARN|BLOCK` 显式指定，也可基于当前设计工件做保守推断
+- 对 required supervisor gate，优先使用 `--run-reviewers --write`，让 reviewer skills 执行与结果持久化一体完成
+- reviewer 执行参数可通过 `--review-concurrency`、`--review-retries`、`--review-retry-delay-ms` 调优（指数退避）
+- `design-supervisor review` 作为兼容别名仍可使用
+
 当 Pencil MCP 可用时，Da Vinci 现在还要求在终态完成声明前，把 MCP runtime gate 结果记录到 `pencil-design.md`。这层 gate 负责检查 live editor/source convergence，与 filesystem audit 分工不同。
 在重设计进行中，如果有 shell 能力，应在第一次成功写入 Pencil 后立即运行 `da-vinci audit --mode integrity <project-path>`；如果同一个 anchor surface 连续回滚，则继续配合 `da-vinci preflight-pencil` 和更小的 follow-up batch。
 

package/SKILL.md

@@ -280,9 +280,9 @@ During active Pencil work:
 - exported screenshots are review artifacts only; place them under `.da-vinci/changes/<change-id>/exports/` and never treat them as a substitute for the project-local `.pen` source
 - screenshot review is binding: if the review calls out hierarchy, spacing, clarity, inconsistency, or unresolved-placeholder issues, revise the screen before treating the checkpoint as `PASS`
 - screenshot review must record an explicit `PASS`, `WARN`, or `BLOCK` plus the concrete issue list and revision outcome; phrases such as "looks good" do not count as review evidence
-- if `DA-VINCI.md` declares `Design-supervisor reviewers`, run `design-supervisor review` after screenshot review, layout hygiene, and design checkpoint for the approved anchor set
+- if `DA-VINCI.md` declares `Design-supervisor reviewers`, run an explicit review pass with those reviewer skills on the approved anchor set, then persist the structured result with `da-vinci supervisor-review --project <project-path> --change <change-id> --run-reviewers --write` (or the compatibility alias `design-supervisor review --run-reviewers --write`)
 - keep `Design-supervisor reviewers` separate from `Preferred adapters`; adapters lead the design pass, reviewers judge whether the final style quality is strong enough to expand or implement
-- when `design-supervisor review` is active, review screenshots together with Pencil theme variables, `visual-thesis.md`, `content-plan.md`, and `interaction-thesis.md`, then record an explicit `PASS`, `WARN`, or `BLOCK` plus issue list and revision outcome in `pencil-design.md`
+- when `design-supervisor review` is active, review screenshots together with Pencil theme variables, `visual-thesis.md`, `content-plan.md`, and `interaction-thesis.md`; record `Configured reviewers`, `Executed reviewers`, `Review source`, explicit `PASS`/`WARN`/`BLOCK`, issue list, and revision outcome in `pencil-design.md`
 - if `DA-VINCI.md` sets `Require Supervisor Review: true`, treat missing, blocked, or unaccepted `design-supervisor review` as a blocker before broad expansion, implementation-task handoff, or terminal completion
 
 ## Load References On Demand

package/bin/design-supervisor.js

@@ -0,0 +1,39 @@
+#!/usr/bin/env node
+
+const { runCli } = require("../lib/cli");
+
+function printHelp() {
+  console.log(
+    [
+      "Design Supervisor CLI (Da Vinci bridge)",
+      "",
+      "Usage:",
+      "  design-supervisor review --project <path> --change <id> [--run-reviewers] [--review-concurrency <value>] [--review-retries <value>] [--review-retry-delay-ms <value>] [--source <skill|manual|inferred>] [--executed-reviewers <csv>] [--status <PASS|WARN|BLOCK>] [--issue-list <text>] [--revision-outcome <text>] [--write] [--json]",
+      "",
+      "Notes:",
+      "  - This command is a compatibility alias for `da-vinci supervisor-review`.",
+      "  - It does not replace completion gating; use `da-vinci audit --mode completion` for final gate checks."
+    ].join("\n")
+  );
+}
+
+async function main() {
+  const args = process.argv.slice(2);
+  const [subcommand] = args;
+
+  if (!subcommand || subcommand === "--help" || subcommand === "-h" || subcommand === "help") {
+    printHelp();
+    return;
+  }
+
+  if (subcommand !== "review") {
+    throw new Error(`Unknown subcommand: ${subcommand}`);
+  }
+
+  await runCli(["supervisor-review", ...args.slice(1)]);
+}
+
+main().catch((error) => {
+  console.error(error.message || String(error));
+  process.exit(1);
+});

package/commands/claude/dv/design.md

@@ -26,5 +26,5 @@ If a registered project-local `.pen` already exists, reopen it for continuity bu
 After the first successful Pencil write, run `da-vinci audit --mode integrity <project-path>` before broad expansion continues.
 If Pencil MCP is active, run the MCP runtime gate after the first successful Pencil write and record it in `pencil-design.md`.
 Screenshot review must record an explicit `PASS`, `WARN`, or `BLOCK` plus the issue list and revision outcome; "looks good" is not a valid review record.
-If `DA-VINCI.md` declares `Design-supervisor reviewers`, run `design-supervisor review` after screenshot review, layout hygiene, and design checkpoint, then record the reviewers, inputs, status, issue list, and revision outcome in `pencil-design.md`. If `Require Supervisor Review: true`, treat missing, blocked, or unaccepted review results as blocking before broad expansion or terminal completion.
+If `DA-VINCI.md` declares `Design-supervisor reviewers`, execute those reviewer skills on the approved anchor screenshots (do not skip to inferred review), then persist the structured result via `da-vinci supervisor-review --project <project-path> --change <change-id> --run-reviewers --write` (or the compatibility alias `design-supervisor review --run-reviewers --write`). Record configured/ executed reviewers, review source, status, issue list, and revision outcome in `pencil-design.md`. If `Require Supervisor Review: true`, treat missing, blocked, unaccepted, or non-skill-backed review results as blocking before broad expansion or terminal completion.
 Before reporting `design complete` or `workflow complete`, run `da-vinci audit --mode completion --change <change-id> <project-path>` and treat any failure as blocking.

package/commands/codex/prompts/dv-design.md

@@ -20,5 +20,5 @@ If a registered project-local `.pen` already exists, reopen it for continuity bu
 After the first successful Pencil write, run `da-vinci audit --mode integrity <project-path>` before broad expansion continues.
 If Pencil MCP is active, run the MCP runtime gate after the first successful Pencil write and record it in `pencil-design.md`.
 Screenshot review must record an explicit `PASS`, `WARN`, or `BLOCK` plus the issue list and revision outcome; "looks good" is not a valid review record.
-If `DA-VINCI.md` declares `Design-supervisor reviewers`, run `design-supervisor review` after screenshot review, layout hygiene, and design checkpoint, then record the reviewers, inputs, status, issue list, and revision outcome in `pencil-design.md`. If `Require Supervisor Review: true`, treat missing, blocked, or unaccepted review results as blocking before broad expansion or terminal completion.
+If `DA-VINCI.md` declares `Design-supervisor reviewers`, execute those reviewer skills on the approved anchor screenshots (do not skip to inferred review), then persist the structured result via `da-vinci supervisor-review --project <project-path> --change <change-id> --run-reviewers --write` (or the compatibility alias `design-supervisor review --run-reviewers --write`). Record configured/ executed reviewers, review source, status, issue list, and revision outcome in `pencil-design.md`. If `Require Supervisor Review: true`, treat missing, blocked, unaccepted, or non-skill-backed review results as blocking before broad expansion or terminal completion.
 Before claiming `design complete` or `workflow complete`, run `da-vinci audit --mode completion --change <change-id> <project-path>` and treat any failure as blocking.

package/commands/gemini/dv/design.toml

@@ -19,6 +19,6 @@ If a registered project-local `.pen` already exists, reopen it for continuity bu
 After the first successful Pencil write, run `da-vinci audit --mode integrity <project-path>` before broad expansion continues.
 If Pencil MCP is active, run the MCP runtime gate after the first successful Pencil write and record it in `pencil-design.md`.
 Screenshot review must record an explicit `PASS`, `WARN`, or `BLOCK` plus the issue list and revision outcome; "looks good" is not a valid review record.
-If `DA-VINCI.md` declares `Design-supervisor reviewers`, run `design-supervisor review` after screenshot review, layout hygiene, and design checkpoint, then record the reviewers, inputs, status, issue list, and revision outcome in `pencil-design.md`. If `Require Supervisor Review: true`, treat missing, blocked, or unaccepted review results as blocking before broad expansion or terminal completion.
+If `DA-VINCI.md` declares `Design-supervisor reviewers`, execute those reviewer skills on the approved anchor screenshots (do not skip to inferred review), then persist the structured result via `da-vinci supervisor-review --project <project-path> --change <change-id> --run-reviewers --write` (or the compatibility alias `design-supervisor review --run-reviewers --write`). Record configured/ executed reviewers, review source, status, issue list, and revision outcome in `pencil-design.md`. If `Require Supervisor Review: true`, treat missing, blocked, unaccepted, or non-skill-backed review results as blocking before broad expansion or terminal completion.
 Before reporting `design complete` or `workflow complete`, run `da-vinci audit --mode completion --change <change-id> <project-path>` and treat any failure as blocking.
 """

package/docs/constraint-files.md

@@ -38,11 +38,14 @@ The workflow audit currently parses specific fields. Keep these names stable.
 
 ### `pencil-design.md` -> `## Design-Supervisor Review`
 
+- `Configured reviewers`
+- `Executed reviewers`
+- `Review source`
 - `Status`
 - `Issue list`
 - `Revision outcome`
 
-When supervisor review is required, missing or blocked review evidence blocks completion.
+When supervisor review is required, missing, blocked, unaccepted, or non-skill-backed review evidence blocks completion.
 
 ## Advisory Constraint Sections (Customizable)
 

package/docs/dv-command-reference.md

@@ -47,6 +47,12 @@ These commands do not replace route selection, but they support design execution
 - `da-vinci icon-search --query "<text>" [--family ...] [--top ...] [--aliases ...]`
   - resolve likely `icon_font` names before writing Pencil `batch_design` operations
   - supports mixed EN/ZH terms and optional alias expansion via `~/.da-vinci/icon-aliases.json`
+- `da-vinci supervisor-review --project <path> --change <id> [--run-reviewers] [--review-concurrency <value>] [--review-retries <value>] [--review-retry-delay-ms <value>] [--source <skill|manual|inferred>] [--executed-reviewers <csv>] [--status ...] [--issue-list ...] [--revision-outcome ...] [--write]`
+  - persists a structured supervisor-review record (`Configured reviewers`, `Executed reviewers`, `Review source`, `Status`, `Issue list`, `Revision outcome`) in `pencil-design.md`
+  - use `--run-reviewers --write` for one-step execution + record persistence through configured reviewer skills
+  - `--review-concurrency`, `--review-retries`, and `--review-retry-delay-ms` control parallelism and retry backoff for reviewer execution
+  - when `Require Supervisor Review: true`, inferred/manual records are completion-blocking
+  - `design-supervisor review` is kept as a compatibility alias that forwards to this command
 
 Use these utilities during `/dv:design`, especially before anchor-surface icon finalization.
 

package/docs/zh-CN/constraint-files.md

@@ -40,11 +40,14 @@
 
 ### `pencil-design.md` -> `## Design-Supervisor Review`
 
+- `Configured reviewers`
+- `Executed reviewers`
+- `Review source`
 - `Status`
 - `Issue list`
 - `Revision outcome`
 
-当 supervisor review 是必需时，缺失或 `BLOCK` 的评审证据会阻断 completion。
+当 supervisor review 是必需时，缺失、`BLOCK`、未接受、或非 skill-backed 的评审证据都会阻断 completion。
 
 ## 指导型约束段（可定制）
 

package/docs/zh-CN/dv-command-reference.md

@@ -49,6 +49,12 @@ Da Vinci 期望它们遵循工作流状态。
 - `da-vinci icon-search --query "<关键词>" [--family ...] [--top ...] [--aliases ...]`
   - 在写 Pencil `batch_design` 前先收敛可用的 `icon_font` 名称
   - 支持中英文混合词，并可通过 `~/.da-vinci/icon-aliases.json` 做语义扩展
+- `da-vinci supervisor-review --project <path> --change <id> [--run-reviewers] [--review-concurrency <value>] [--review-retries <value>] [--review-retry-delay-ms <value>] [--source <skill|manual|inferred>] [--executed-reviewers <csv>] [--status ...] [--issue-list ...] [--revision-outcome ...] [--write]`
+  - 持久化结构化 supervisor review 记录（`Configured reviewers`、`Executed reviewers`、`Review source`、`Status`、`Issue list`、`Revision outcome`）到 `pencil-design.md`
+  - 推荐直接用 `--run-reviewers --write` 一步完成 reviewer skills 执行与记录落盘
+  - 通过 `--review-concurrency`、`--review-retries`、`--review-retry-delay-ms` 可调 reviewer 并发与重试退避
+  - 当 `Require Supervisor Review: true` 时，`manual/inferred` 记录会被 completion 阻断
+  - `design-supervisor review` 作为兼容别名保留，并会转发到该命令
 
 建议在 `/dv:design` 阶段使用，尤其是在 anchor surface 的图标定稿前。
 

package/lib/audit-parsers.js

@@ -32,6 +32,133 @@ function getMarkdownSection(text, heading) {
   return sectionLines.join("\n").trim();
 }
 
+function getMarkdownSectionsByHeading(text, heading, options = {}) {
+  if (!text) {
+    return [];
+  }
+
+  const escapedHeading = escapeRegExp(heading);
+  const allowParentheticalSuffix = options.allowParentheticalSuffix === true;
+  const suffixPattern = allowParentheticalSuffix ? String.raw`(?:\s*\([^)\n]*\))?` : "";
+  const headingPattern = new RegExp(`^##\\s+${escapedHeading}${suffixPattern}\\s*$`, "i");
+  const anyHeadingPattern = /^##\s+/;
+  const lines = String(text).replace(/\r\n?/g, "\n").split("\n");
+  const sections = [];
+  let capturing = false;
+  let currentHeading = "";
+  let sectionLines = [];
+
+  function flushSection() {
+    if (!capturing) {
+      return;
+    }
+
+    sections.push({
+      heading: currentHeading,
+      content: sectionLines.join("\n").trim()
+    });
+    capturing = false;
+    currentHeading = "";
+    sectionLines = [];
+  }
+
+  for (const line of lines) {
+    if (capturing && anyHeadingPattern.test(line)) {
+      flushSection();
+    }
+
+    if (!capturing && headingPattern.test(line)) {
+      capturing = true;
+      currentHeading = line.trim();
+      sectionLines = [];
+      continue;
+    }
+
+    if (capturing) {
+      sectionLines.push(line);
+    }
+  }
+
+  flushSection();
+  return sections;
+}
+
+function extractReviewFieldValue(section, fieldNames) {
+  const lines = String(section || "").replace(/\r\n?/g, "\n").split("\n");
+  const escapedFieldNames = fieldNames.map((fieldName) => escapeRegExp(fieldName)).join("|");
+  const fieldPattern = new RegExp(`^\\s*-\\s*(?:${escapedFieldNames})\\s*:\\s*(.*)$`, "i");
+  const topLevelFieldPattern = /^\s*-\s+[^:\n]+:\s*.*$/;
+  const bulletPattern = /^\s*-\s+(.+)$/;
+  const nestedTextPattern = /^\s{2,}(.+)$/;
+  const values = [];
+  let capturing = false;
+
+  for (const rawLine of lines) {
+    if (/^##\s+/.test(rawLine)) {
+      if (capturing) {
+        break;
+      }
+      continue;
+    }
+
+    const fieldMatch = rawLine.match(fieldPattern);
+    if (!capturing && fieldMatch) {
+      capturing = true;
+      const inlineValue = String(fieldMatch[1] || "").trim();
+      if (inlineValue) {
+        values.push(inlineValue);
+      }
+      continue;
+    }
+
+    if (!capturing) {
+      continue;
+    }
+
+    if (topLevelFieldPattern.test(rawLine)) {
+      break;
+    }
+
+    const bulletMatch = rawLine.match(bulletPattern);
+    if (bulletMatch) {
+      const value = String(bulletMatch[1] || "").trim();
+      if (value) {
+        values.push(value);
+      }
+      continue;
+    }
+
+    const nestedTextMatch = rawLine.match(nestedTextPattern);
+    if (nestedTextMatch) {
+      const value = String(nestedTextMatch[1] || "").trim();
+      if (value) {
+        values.push(value);
+      }
+      continue;
+    }
+
+    if (rawLine.trim() === "") {
+      continue;
+    }
+
+    break;
+  }
+
+  return values.join("\n").trim();
+}
+
+function parseListTokens(value) {
+  return String(value || "")
+    .split(/[,\n;]/)
+    .map((token) =>
+      token
+        .replace(/^[\-\*\u2022]\s*/, "")
+        .replace(/`/g, "")
+        .trim()
+    )
+    .filter(Boolean);
+}
+
 function normalizeCheckpointLabel(value) {
   return String(value || "")
     .toLowerCase()
@@ -398,6 +525,12 @@ function hasConfiguredDesignSupervisorReview(daVinciText) {
   return getVisualAssistFieldValues(daVinciText, "Design-supervisor reviewers").length > 0;
 }
 
+function getConfiguredDesignSupervisorReviewers(daVinciText) {
+  return getVisualAssistFieldValues(daVinciText, "Design-supervisor reviewers")
+    .flatMap((value) => parseListTokens(value))
+    .filter(Boolean);
+}
+
 function isDesignSupervisorReviewRequired(daVinciText) {
   return getVisualAssistFieldValues(daVinciText, "Require Supervisor Review").some((value) =>
     /^true$/i.test(String(value).trim())
@@ -405,8 +538,10 @@ function isDesignSupervisorReviewRequired(daVinciText) {
 }
 
 function inspectDesignSupervisorReview(pencilDesignText) {
-  const section = getMarkdownSection(pencilDesignText, "Design-Supervisor Review");
-  if (!section) {
+  const sections = getMarkdownSectionsByHeading(pencilDesignText, "Design-Supervisor Review", {
+    allowParentheticalSuffix: true
+  });
+  if (sections.length === 0) {
     return {
       found: false,
       status: null,
@@ -416,25 +551,35 @@ function inspectDesignSupervisorReview(pencilDesignText) {
     };
   }
 
-  const statusMatch = section.match(/(?:^|\n)\s*-\s*(?:Status|状态)\s*:\s*(PASS|WARN|BLOCK)\b/i);
-  const status = statusMatch ? statusMatch[1].toUpperCase() : null;
-  const issueListMatch = section.match(/(?:^|\n)\s*-\s*(?:Issue list|问题列表)\s*:\s*(.+)$/im);
-  const revisionOutcomeMatch = section.match(
-    /(?:^|\n)\s*-\s*(?:Revision outcome|修订结果)\s*:\s*(.+)$/im
+  const section = sections[sections.length - 1].content;
+  const statusMatch = section.match(
+    /(?:^|\n)\s*-\s*(?:Status|状态|Result|结果)\s*:\s*`?(PASS|WARN|BLOCK)`?\b/i
   );
-  const revisionOutcome = revisionOutcomeMatch ? revisionOutcomeMatch[1].trim() : "";
+  const status = statusMatch ? statusMatch[1].toUpperCase() : null;
+  const issueList = extractReviewFieldValue(section, ["Issue list", "问题列表"]);
+  const revisionOutcome = extractReviewFieldValue(section, ["Revision outcome", "修订结果"]);
+  const configuredReviewersRaw = extractReviewFieldValue(section, ["Configured reviewers", "配置评审"]);
+  const executedReviewersRaw = extractReviewFieldValue(section, ["Executed reviewers", "Executed reviewer skills", "执行评审"]);
+  const reviewSource = extractReviewFieldValue(section, ["Review source", "评审来源"]).toLowerCase();
   const acceptedWarn =
     status === "WARN" &&
     /(accepted|accepted with follow-up|accepted warning|warn accepted|接受|已接受|接受警告)/i.test(
       revisionOutcome
     );
+  const configuredReviewers = parseListTokens(configuredReviewersRaw);
+  const executedReviewers = parseListTokens(executedReviewersRaw);
 
   return {
     found: true,
     status,
     acceptedWarn,
-    hasIssueList: Boolean(issueListMatch && issueListMatch[1].trim()),
-    hasRevisionOutcome: Boolean(revisionOutcome)
+    hasIssueList: Boolean(issueList),
+    hasRevisionOutcome: Boolean(revisionOutcome),
+    configuredReviewers,
+    executedReviewers,
+    reviewSource,
+    hasConfiguredReviewers: configuredReviewers.length > 0,
+    hasExecutedReviewers: executedReviewers.length > 0
   };
 }
 
@@ -446,6 +591,7 @@ module.exports = {
   buildContextDeltaReferenceIndex,
   resolveSupersedesReferenceIndices,
   inspectContextDelta,
+  getConfiguredDesignSupervisorReviewers,
   hasConfiguredDesignSupervisorReview,
   isDesignSupervisorReviewRequired,
   inspectDesignSupervisorReview

package/lib/audit.js

@@ -11,6 +11,7 @@ const {
   buildContextDeltaReferenceIndex,
   resolveSupersedesReferenceIndices,
   inspectContextDelta,
+  getConfiguredDesignSupervisorReviewers,
   hasConfiguredDesignSupervisorReview,
   isDesignSupervisorReviewRequired,
   inspectDesignSupervisorReview
@@ -195,6 +196,10 @@ function appendTraversalWarnings(projectRoot, rootDir, scan, warnings) {
   }
 }
 
+function normalizeReviewerToken(value) {
+  return String(value || "").trim().toLowerCase();
+}
+
 function auditProject(projectPathInput, options = {}) {
   const projectRoot = path.resolve(projectPathInput || process.cwd());
   const mode = options.mode || "integrity";
@@ -204,6 +209,8 @@ function auditProject(projectPathInput, options = {}) {
   const designRegistryPath = path.join(daVinciDir, "design-registry.md");
   const pencilSessionPath = getSessionStatePath(projectRoot);
   const daVinciText = readTextIfExists(path.join(projectRoot, "DA-VINCI.md"));
+  const configuredSupervisorReviewers = getConfiguredDesignSupervisorReviewers(daVinciText);
+  const configuredReviewerSet = new Set(configuredSupervisorReviewers.map((value) => normalizeReviewerToken(value)));
   const designSupervisorConfigured = hasConfiguredDesignSupervisorReview(daVinciText);
   const designSupervisorRequired = isDesignSupervisorReviewRequired(daVinciText);
 
@@ -467,6 +474,8 @@ function auditProject(projectPathInput, options = {}) {
       const enforceAsFailure =
         mode === "completion" && scopedChangeDirs.includes(changeDir) && designSupervisorRequired;
 
+      let reviewRecordValid = true;
+
       if (!review.found) {
         const message =
           `DA-VINCI.md configures Design-supervisor reviewers, but ${relativeTo(projectRoot, pencilDesignPath)} ` +
@@ -476,6 +485,7 @@ function auditProject(projectPathInput, options = {}) {
         } else {
           warnings.push(message);
         }
+        reviewRecordValid = false;
       } else if (!review.status) {
         const message =
           `Design-supervisor review is recorded in ${relativeTo(projectRoot, pencilDesignPath)} ` +
@@ -485,6 +495,7 @@ function auditProject(projectPathInput, options = {}) {
         } else {
           warnings.push(message);
         }
+        reviewRecordValid = false;
       } else if (!review.hasIssueList || !review.hasRevisionOutcome) {
         const missingFields = [
           !review.hasIssueList ? "Issue list" : null,
@@ -500,7 +511,82 @@ function auditProject(projectPathInput, options = {}) {
         } else {
           warnings.push(message);
         }
-      } else if (review.status === "BLOCK") {
+        reviewRecordValid = false;
+      } else if (!review.hasConfiguredReviewers) {
+        const message =
+          `Design-supervisor review in ${relativeTo(projectRoot, pencilDesignPath)} ` +
+          "is missing required field(s): Configured reviewers.";
+        if (enforceAsFailure) {
+          failures.push(message);
+        } else {
+          warnings.push(message);
+        }
+        reviewRecordValid = false;
+      }
+
+      if (reviewRecordValid && configuredReviewerSet.size > 0) {
+        const reviewConfiguredSet = new Set(
+          review.configuredReviewers.map((value) => normalizeReviewerToken(value))
+        );
+        const missingConfiguredReviewers = configuredSupervisorReviewers.filter(
+          (reviewer) => !reviewConfiguredSet.has(normalizeReviewerToken(reviewer))
+        );
+        if (missingConfiguredReviewers.length > 0) {
+          const message =
+            `Design-supervisor review in ${relativeTo(projectRoot, pencilDesignPath)} ` +
+            `does not list configured reviewer(s): ${missingConfiguredReviewers.join(", ")}.`;
+          if (enforceAsFailure) {
+            failures.push(message);
+          } else {
+            warnings.push(message);
+          }
+          reviewRecordValid = false;
+        }
+      }
+
+      if (reviewRecordValid) {
+        if (designSupervisorRequired && review.reviewSource !== "skill") {
+          const message =
+            `Design-supervisor review in ${relativeTo(projectRoot, pencilDesignPath)} ` +
+            "must be skill-backed (`Review source: skill`) when `Require Supervisor Review: true`.";
+          if (enforceAsFailure) {
+            failures.push(message);
+          } else {
+            warnings.push(message);
+          }
+          reviewRecordValid = false;
+        } else if (designSupervisorRequired && !review.hasExecutedReviewers) {
+          const message =
+            `Design-supervisor review in ${relativeTo(projectRoot, pencilDesignPath)} ` +
+            "is missing required field(s): Executed reviewers.";
+          if (enforceAsFailure) {
+            failures.push(message);
+          } else {
+            warnings.push(message);
+          }
+          reviewRecordValid = false;
+        } else if (designSupervisorRequired && configuredReviewerSet.size > 0) {
+          const executedReviewerSet = new Set(
+            review.executedReviewers.map((value) => normalizeReviewerToken(value))
+          );
+          const missingExecuted = configuredSupervisorReviewers.filter(
+            (reviewer) => !executedReviewerSet.has(normalizeReviewerToken(reviewer))
+          );
+          if (missingExecuted.length > 0) {
+            const message =
+              `Design-supervisor review in ${relativeTo(projectRoot, pencilDesignPath)} ` +
+              `did not execute configured reviewer skill(s): ${missingExecuted.join(", ")}.`;
+            if (enforceAsFailure) {
+              failures.push(message);
+            } else {
+              warnings.push(message);
+            }
+            reviewRecordValid = false;
+          }
+        }
+      }
+
+      if (reviewRecordValid && review.status === "BLOCK") {
         const message =
           `Design-supervisor review is still BLOCK in ${relativeTo(projectRoot, pencilDesignPath)}.`;
         if (enforceAsFailure) {
@@ -508,7 +594,7 @@ function auditProject(projectPathInput, options = {}) {
         } else {
           warnings.push(message);
         }
-      } else if (review.status === "WARN" && !review.acceptedWarn) {
+      } else if (reviewRecordValid && review.status === "WARN" && !review.acceptedWarn) {
         const message =
           `Design-supervisor review is WARN in ${relativeTo(projectRoot, pencilDesignPath)} ` +
           "but the warning was not explicitly accepted.";
@@ -517,7 +603,7 @@ function auditProject(projectPathInput, options = {}) {
         } else {
           warnings.push(message);
         }
-      } else {
+      } else if (reviewRecordValid) {
         notes.push(
           `Detected design-supervisor review status ${review.status} in ${relativeTo(projectRoot, pencilDesignPath)}.`
         );