@xenonbyte/da-vinci-workflow 0.1.17 → 0.1.19

package/scripts/test-audit-design-supervisor.js

@@ -0,0 +1,348 @@
+const assert = require("assert/strict");
+const fs = require("fs");
+const os = require("os");
+const path = require("path");
+const { spawnSync } = require("child_process");
+
+const { auditProject } = require("../lib/audit");
+
+const repo = path.resolve(__dirname, "..");
+const cli = path.join(repo, "bin", "da-vinci.js");
+const fixture = JSON.parse(
+  fs.readFileSync(path.join(__dirname, "fixtures", "complex-sample.pen"), "utf8")
+);
+const changeId = "redesign-001";
+
+function runTest(name, fn) {
+  try {
+    fn();
+    console.log(`PASS ${name}`);
+  } catch (error) {
+    console.error(`FAIL ${name}`);
+    throw error;
+  }
+}
+
+function createHarness() {
+  const base = fs.mkdtempSync(path.join(os.tmpdir(), "da-vinci-audit-supervisor-"));
+  return {
+    base,
+    home: path.join(base, "home")
+  };
+}
+
+function runCli(harness, args) {
+  const [command, ...rest] = args;
+  const result = spawnSync(process.execPath, [cli, command, "--home", harness.home, ...rest], {
+    cwd: repo,
+    encoding: "utf8",
+    maxBuffer: 8 * 1024 * 1024
+  });
+
+  return {
+    code: result.status,
+    stdout: (result.stdout || "").trim(),
+    stderr: (result.stderr || "").trim()
+  };
+}
+
+function expectOk(step, result) {
+  assert.equal(result.code, 0, `${step} failed:\n${result.stderr || result.stdout}`);
+}
+
+function writeJson(filePath, payload) {
+  fs.mkdirSync(path.dirname(filePath), { recursive: true });
+  fs.writeFileSync(filePath, JSON.stringify(payload, null, 2));
+}
+
+function writeText(filePath, text) {
+  fs.mkdirSync(path.dirname(filePath), { recursive: true });
+  fs.writeFileSync(filePath, text);
+}
+
+function setupProject(harness, name, options = {}) {
+  const root = path.join(harness.base, name);
+  const daVinciDir = path.join(root, ".da-vinci");
+  const changeDir = path.join(daVinciDir, "changes", changeId);
+  const penPath = path.join(daVinciDir, "designs", `${name}.pen`);
+  const nodesFile = path.join(root, `${name}-nodes.json`);
+  const variablesFile = path.join(root, `${name}-variables.json`);
+  const configureReviewers = options.configureReviewers !== false;
+  const requireSupervisorReview = options.requireSupervisorReview === true;
+
+  const visualAssistLines = [
+    "# DA-VINCI",
+    "",
+    "## Visual Assist",
+    "- Preferred adapters:",
+    "  - frontend-skill"
+  ];
+
+  if (configureReviewers) {
+    visualAssistLines.push("- Design-supervisor reviewers:");
+    visualAssistLines.push("  - frontend-skill");
+    visualAssistLines.push("- Design-supervisor review mode:");
+    visualAssistLines.push("  - screenshot-and-theme");
+    visualAssistLines.push("- Design-supervisor review inputs:");
+    visualAssistLines.push("  - screenshots");
+    visualAssistLines.push("  - pencil variables");
+  }
+
+  visualAssistLines.push("- Require Supervisor Review:");
+  visualAssistLines.push(`  - ${requireSupervisorReview ? "true" : "false"}`);
+  visualAssistLines.push("");
+
+  writeText(
+    path.join(root, "DA-VINCI.md"),
+    visualAssistLines.join("\n")
+  );
+  writeText(path.join(daVinciDir, "project-inventory.md"), "# Inventory\n");
+  writeText(path.join(daVinciDir, "page-map.md"), "# Page Map\n");
+  writeText(
+    path.join(daVinciDir, "design-registry.md"),
+    `# Registry\n- Preferred .pen: .da-vinci/designs/${name}.pen\n`
+  );
+  writeText(path.join(changeDir, "design-brief.md"), "# Brief\n");
+  writeText(path.join(changeDir, "design.md"), "# Design\n");
+  writeText(path.join(changeDir, "pencil-bindings.md"), "# Bindings\n");
+
+  writeJson(nodesFile, { nodes: fixture.children });
+  writeJson(variablesFile, { variables: fixture.variables });
+
+  expectOk(
+    `${name} begin`,
+    runCli(harness, ["pencil-session", "begin", "--project", root, "--pen", penPath])
+  );
+  expectOk(
+    `${name} persist`,
+    runCli(harness, [
+      "pencil-session",
+      "persist",
+      "--project",
+      root,
+      "--pen",
+      penPath,
+      "--nodes-file",
+      nodesFile,
+      "--variables-file",
+      variablesFile,
+      "--version",
+      fixture.version
+    ])
+  );
+  expectOk(
+    `${name} end`,
+    runCli(harness, [
+      "pencil-session",
+      "end",
+      "--project",
+      root,
+      "--pen",
+      penPath,
+      "--nodes-file",
+      nodesFile,
+      "--variables-file",
+      variablesFile,
+      "--version",
+      fixture.version
+    ])
+  );
+
+  return {
+    root,
+    changeDir,
+    pencilDesignPath: path.join(changeDir, "pencil-design.md")
+  };
+}
+
+runTest("completion audit warns when supervisor reviewers are configured but not required and no review is recorded", () => {
+  const harness = createHarness();
+  const project = setupProject(harness, "missing-supervisor-review");
+
+  writeText(project.pencilDesignPath, "# Pencil Design\n");
+
+  const result = auditProject(project.root, {
+    mode: "completion",
+    changeId
+  });
+
+  assert.equal(result.status, "WARN");
+  assert.match(
+    result.warnings.join("\n"),
+    /Design-supervisor reviewers.*does not record a `## Design-Supervisor Review` section/i
+  );
+});
+
+runTest("completion audit fails when supervisor review is required but no review is recorded", () => {
+  const harness = createHarness();
+  const project = setupProject(harness, "required-supervisor-review", {
+    requireSupervisorReview: true
+  });
+
+  writeText(project.pencilDesignPath, "# Pencil Design\n");
+
+  const result = auditProject(project.root, {
+    mode: "completion",
+    changeId
+  });
+
+  assert.equal(result.status, "FAIL");
+  assert.match(
+    result.failures.join("\n"),
+    /Design-supervisor reviewers.*does not record a `## Design-Supervisor Review` section/i
+  );
+});
+
+runTest("completion audit fails when supervisor review is required but no reviewers are configured", () => {
+  const harness = createHarness();
+  const project = setupProject(harness, "missing-supervisor-reviewers", {
+    configureReviewers: false,
+    requireSupervisorReview: true
+  });
+
+  writeText(project.pencilDesignPath, "# Pencil Design\n");
+
+  const result = auditProject(project.root, {
+    mode: "completion",
+    changeId
+  });
+
+  assert.equal(result.status, "FAIL");
+  assert.match(
+    result.failures.join("\n"),
+    /Require Supervisor Review: true.*no `Design-supervisor reviewers` are configured/i
+  );
+});
+
+runTest("completion audit does not treat an empty reviewers field as configured", () => {
+  const harness = createHarness();
+  const project = setupProject(harness, "empty-supervisor-reviewers", {
+    configureReviewers: false,
+    requireSupervisorReview: false
+  });
+
+  writeText(
+    path.join(project.root, "DA-VINCI.md"),
+    [
+      "# DA-VINCI",
+      "",
+      "## Visual Assist",
+      "- Preferred adapters:",
+      "  - frontend-skill",
+      "- Design-supervisor reviewers:",
+      "- Design-supervisor review mode:",
+      "  - screenshot-and-theme",
+      "- Design-supervisor review inputs:",
+      "  - screenshots",
+      "  - pencil variables",
+      "- Require Supervisor Review:",
+      "  - false",
+      ""
+    ].join("\n")
+  );
+  writeText(project.pencilDesignPath, "# Pencil Design\n");
+
+  const result = auditProject(project.root, {
+    mode: "completion",
+    changeId
+  });
+
+  assert.equal(result.status, "PASS");
+  assert.doesNotMatch(
+    result.warnings.join("\n"),
+    /Design-supervisor reviewers.*does not record a `## Design-Supervisor Review` section/i
+  );
+});
+
+runTest("integrity audit warns when supervisor review is required but no review is recorded", () => {
+  const harness = createHarness();
+  const project = setupProject(harness, "warn-supervisor-review", {
+    requireSupervisorReview: true
+  });
+
+  writeText(project.pencilDesignPath, "# Pencil Design\n");
+
+  const result = auditProject(project.root, {
+    mode: "integrity"
+  });
+
+  assert.equal(result.status, "WARN");
+  assert.match(
+    result.warnings.join("\n"),
+    /Design-supervisor reviewers.*does not record a `## Design-Supervisor Review` section/i
+  );
+});
+
+runTest("completion audit fails when required supervisor review is missing Issue list or Revision outcome", () => {
+  const harness = createHarness();
+  const project = setupProject(harness, "malformed-supervisor-review", {
+    requireSupervisorReview: true
+  });
+
+  writeText(
+    project.pencilDesignPath,
+    [
+      "# Pencil Design",
+      "",
+      "## Design-Supervisor Review",
+      "- Configured reviewers: frontend-skill",
+      "- Review mode: screenshot-and-theme",
+      "- Review inputs: screenshots, pencil variables",
+      "- Status: PASS",
+      ""
+    ].join("\n")
+  );
+
+  const result = auditProject(project.root, {
+    mode: "completion",
+    changeId
+  });
+
+  assert.equal(result.status, "FAIL");
+  assert.match(
+    result.failures.join("\n"),
+    /missing required field\(s\): Issue list and Revision outcome/i
+  );
+});
+
+runTest("completion audit passes when required supervisor review is recorded as PASS", () => {
+  const harness = createHarness();
+  const project = setupProject(harness, "pass-supervisor-review", {
+    requireSupervisorReview: true
+  });
+
+  writeText(
+    project.pencilDesignPath,
+    [
+      "# Pencil Design",
+      "",
+      "## Design-Supervisor Review",
+      "- Configured reviewers: frontend-skill",
+      "- Review mode: screenshot-and-theme",
+      "- Review inputs: screenshots, pencil variables, visual thesis, content plan, interaction thesis",
+      "- Status: PASS",
+      "- Issue list: none",
+      "- Revision outcome: approved",
+      "- Broad expansion approved: yes",
+      "- Implementation-task handoff approved: yes",
+      ""
+    ].join("\n")
+  );
+
+  const result = auditProject(project.root, {
+    mode: "completion",
+    changeId
+  });
+
+  assert.equal(
+    result.status,
+    "PASS",
+    `expected completion audit to pass:\n${JSON.stringify(result, null, 2)}`
+  );
+  assert.match(
+    result.notes.join("\n"),
+    /Detected design-supervisor review status PASS/i
+  );
+});
+
+console.log("All design-supervisor audit tests passed.");

package/scripts/test-mode-consistency.js

@@ -0,0 +1,289 @@
+const assert = require("assert/strict");
+const fs = require("fs");
+const path = require("path");
+
+const repo = path.resolve(__dirname, "..");
+
+const MODES = [
+  "greenfield-spec",
+  "greenfield-brainstorm",
+  "redesign-from-code",
+  "overhaul-from-code",
+  "feature-change"
+];
+
+function read(relativePath) {
+  return fs.readFileSync(path.join(repo, relativePath), "utf8");
+}
+
+function runTest(name, fn) {
+  try {
+    fn();
+    console.log(`PASS ${name}`);
+  } catch (error) {
+    console.error(`FAIL ${name}`);
+    throw error;
+  }
+}
+
+function assertContainsInOrder(text, parts, label) {
+  let index = -1;
+  for (const part of parts) {
+    const next = text.indexOf(part, index + 1);
+    assert.notEqual(next, -1, `${label} is missing ${part}`);
+    assert.ok(next > index, `${label} does not keep ${part} in order`);
+    index = next;
+  }
+}
+
+function assertAllModesPresent(text, label) {
+  for (const mode of MODES) {
+    assert.match(text, new RegExp(mode.replace(/[.*+?^${}()|[\]\\]/g, "\\$&")), `${label} is missing ${mode}`);
+  }
+}
+
+runTest("core references keep all five workflow modes in order", () => {
+  const skill = read("SKILL.md");
+  const modesReference = read("references/modes.md");
+  const modeUseCases = read("docs/mode-use-cases.md");
+  const zhModeUseCases = read("docs/zh-CN/mode-use-cases.md");
+
+  assertContainsInOrder(skill, MODES, "SKILL.md");
+  assertContainsInOrder(modesReference, MODES, "references/modes.md");
+  assertContainsInOrder(modeUseCases, MODES, "docs/mode-use-cases.md");
+  assertContainsInOrder(zhModeUseCases, MODES, "docs/zh-CN/mode-use-cases.md");
+});
+
+runTest("workflow overviews include overhaul mode and its pre-design artifact chain", () => {
+  const overview = read("docs/workflow-overview.md");
+  const zhOverview = read("docs/zh-CN/workflow-overview.md");
+
+  assertAllModesPresent(overview, "docs/workflow-overview.md");
+  assertAllModesPresent(zhOverview, "docs/zh-CN/workflow-overview.md");
+  for (const required of ["project-inventory.md", "proposal.md", "migration-contract.md", "page-map.md", "design-registry.md", "design.md"]) {
+    assert.match(overview, new RegExp(required.replace(/[.*+?^${}()|[\]\\]/g, "\\$&")), `workflow overview is missing ${required}`);
+    assert.match(zhOverview, new RegExp(required.replace(/[.*+?^${}()|[\]\\]/g, "\\$&")), `zh workflow overview is missing ${required}`);
+  }
+});
+
+runTest("overhaul mode is wired into artifact and checkpoint guidance", () => {
+  const readme = read("README.md");
+  const zhReadme = read("README.zh-CN.md");
+  const artifacts = read("references/artifact-templates.md");
+  const checkpoints = read("references/checkpoints.md");
+  const designInputs = read("references/design-inputs.md");
+  const pencilToCode = read("references/pencil-design-to-code.md");
+
+  for (const content of [readme, zhReadme, artifacts, checkpoints, designInputs, pencilToCode]) {
+    assert.match(content, /migration-contract\.md/, "expected migration-contract.md guidance is missing");
+    assert.match(content, /overhaul-from-code/, "expected overhaul-from-code guidance is missing");
+  }
+});
+
+runTest("all intake entrypoints route broad existing-project rewrites to overhaul-from-code", () => {
+  const claude = read("commands/claude/dv/intake.md");
+  const codex = read("commands/codex/prompts/dv-intake.md");
+  const gemini = read("commands/gemini/dv/intake.toml");
+
+  for (const [label, content] of [
+    ["Claude intake", claude],
+    ["Codex intake", codex],
+    ["Gemini intake", gemini]
+  ]) {
+    assert.match(content, /prefer `redesign-from-code`, `overhaul-from-code`, or `feature-change`/, `${label} is missing existing-project routing guidance`);
+    assert.match(content, /prefer `overhaul-from-code` when an existing project is redefining flows or logic rather than only replacing UI/, `${label} is missing overhaul routing guidance`);
+  }
+});
+
+runTest("readmes and command reference document overhaul examples and rollback behavior", () => {
+  const readme = read("README.md");
+  const zhReadme = read("README.zh-CN.md");
+  const commandRef = read("docs/dv-command-reference.md");
+  const zhCommandRef = read("docs/zh-CN/dv-command-reference.md");
+
+  assert.match(readme, /### Overhaul from code/, "README.md is missing overhaul example section");
+  assert.match(zhReadme, /### Overhaul from code/, "README.zh-CN.md is missing overhaul example section");
+  assert.match(readme, /Continue after overhaul planning/, "README.md is missing overhaul continue section");
+  assert.match(zhReadme, /大改版场景继续推进/, "README.zh-CN.md is missing overhaul continue section");
+  assert.match(commandRef, /migration-contract\.md/, "docs/dv-command-reference.md is missing overhaul rollback references");
+  assert.match(zhCommandRef, /migration-contract\.md/, "docs/zh-CN/dv-command-reference.md is missing overhaul rollback references");
+});
+
+runTest("prompt preset guidance keeps overhaul as a pre-design routing concern instead of a redesign preset alias", () => {
+  const presetReadme = read("docs/prompt-presets/README.md");
+  const zhPresetReadme = read("docs/zh-CN/prompt-presets/README.md");
+  const mobilePreset = read("docs/prompt-presets/mobile-app.md");
+  const desktopPreset = read("docs/prompt-presets/desktop-app.md");
+  const webPreset = read("docs/prompt-presets/web-app.md");
+  const tabletPreset = read("docs/prompt-presets/tablet-app.md");
+  const zhMobilePreset = read("docs/zh-CN/prompt-presets/mobile-app.md");
+  const zhDesktopPreset = read("docs/zh-CN/prompt-presets/desktop-app.md");
+  const zhWebPreset = read("docs/zh-CN/prompt-presets/web-app.md");
+  const zhTabletPreset = read("docs/zh-CN/prompt-presets/tablet-app.md");
+
+  assert.match(presetReadme, /prefer `overhaul-from-code` plus `intake`\/custom prompt setup first/, "prompt preset README is missing overhaul guidance");
+  assert.match(presetReadme, /## `overhaul-from-code` Prompt Templates/, "prompt preset README is missing overhaul template section");
+  assert.match(zhPresetReadme, /应该先用 `overhaul-from-code`/, "zh prompt preset README is missing overhaul guidance");
+  assert.match(zhPresetReadme, /## `overhaul-from-code` 提示词模板/, "zh prompt preset README is missing overhaul template section");
+  assert.match(mobilePreset, /use redesign-from-code to redesign this existing mobile app\./, "existing redesign presets should remain redesign-oriented");
+  for (const [label, content] of [
+    ["mobile preset", mobilePreset],
+    ["desktop preset", desktopPreset],
+    ["web preset", webPreset],
+    ["tablet preset", tabletPreset]
+  ]) {
+    assert.match(content, /## Overhaul From Code/, `${label} is missing the overhaul section`);
+    assert.match(content, /### Overhaul Intake/, `${label} is missing overhaul intake`);
+    assert.match(content, /### Overhaul Breakdown/, `${label} is missing overhaul breakdown`);
+    assert.match(content, /### Overhaul Design/, `${label} is missing overhaul design`);
+    assert.match(content, /### Overhaul Continue/, `${label} is missing overhaul continue`);
+  }
+  for (const [label, content] of [
+    ["zh mobile preset", zhMobilePreset],
+    ["zh desktop preset", zhDesktopPreset],
+    ["zh web preset", zhWebPreset],
+    ["zh tablet preset", zhTabletPreset]
+  ]) {
+    assert.match(content, /## Overhaul From Code/, `${label} is missing the overhaul section`);
+    assert.match(content, /### Overhaul Intake/, `${label} is missing overhaul intake`);
+    assert.match(content, /### Overhaul Breakdown/, `${label} is missing overhaul breakdown`);
+    assert.match(content, /### Overhaul Design/, `${label} is missing overhaul design`);
+    assert.match(content, /### Overhaul Continue/, `${label} is missing overhaul continue`);
+  }
+});
+
+runTest("redesign prompt docs make full-delivery explicit and keep stop semantics in design-only only", () => {
+  const presetFiles = [
+    "docs/prompt-presets/mobile-app.md",
+    "docs/prompt-presets/desktop-app.md",
+    "docs/prompt-presets/web-app.md",
+    "docs/prompt-presets/tablet-app.md",
+    "docs/zh-CN/prompt-presets/mobile-app.md",
+    "docs/zh-CN/prompt-presets/desktop-app.md",
+    "docs/zh-CN/prompt-presets/web-app.md",
+    "docs/zh-CN/prompt-presets/tablet-app.md"
+  ];
+
+  for (const file of presetFiles) {
+    const content = read(file);
+    const fullDeliveryCount = (content.match(/Treat this as full-delivery unless explicitly changed\./g) || []).length;
+    assert.ok(fullDeliveryCount >= 3, `${file} should make full-delivery explicit in redesign variants`);
+    assert.match(
+      content,
+      /Do not stop after discovery, anchor surfaces, phase summaries, or task generation; continue through tasks, build, and verify once the corresponding checkpoints pass\./,
+      `${file} should forbid redesign variants from stopping at anchor or task summaries`
+    );
+    assert.match(content, /Treat this as design-only, not full-delivery\./, `${file} should explicitly reserve stop semantics for design-only`);
+    assert.doesNotMatch(content, /Stop after .*tasks\.md\./, `${file} should not let design-only prompts generate tasks.md by default`);
+  }
+
+  const promptRecipes = read("references/prompt-recipes.md");
+  assert.match(promptRecipes, /treat the request as `full-delivery` unless the user explicitly says `design-only`/, "prompt recipes should require explicit redesign delivery intent");
+  assert.match(promptRecipes, /do not stop at anchor-surface summaries, phase summaries, or task generation once the corresponding checkpoints pass/, "prompt recipes should forbid redesign prompts from stopping at phase summaries");
+});
+
+runTest("pencil session docs keep project-local .pen seeding as a required step on autonomous redesign runs", () => {
+  const keyFiles = [
+    "commands/claude/dv/design.md",
+    "commands/codex/prompts/dv-design.md",
+    "commands/gemini/dv/design.toml",
+    "docs/pencil-rendering-workflow.md",
+    "docs/zh-CN/pencil-rendering-workflow.md",
+    "README.md",
+    "README.zh-CN.md",
+    "docs/prompt-presets/mobile-app.md",
+    "docs/prompt-presets/desktop-app.md",
+    "docs/prompt-presets/web-app.md",
+    "docs/prompt-presets/tablet-app.md",
+    "docs/zh-CN/prompt-presets/mobile-app.md",
+    "docs/zh-CN/prompt-presets/desktop-app.md",
+    "docs/zh-CN/prompt-presets/web-app.md",
+    "docs/zh-CN/prompt-presets/tablet-app.md",
+    "docs/workflow-examples.md",
+    "docs/zh-CN/workflow-examples.md",
+    "references/pencil-design-to-code.md"
+  ];
+
+  for (const file of keyFiles) {
+    const content = read(file);
+    assert.doesNotMatch(content, /Prefer the session wrapper commands/, `${file} should not weaken the session wrapper to a preference`);
+    assert.doesNotMatch(content, /Before the first Pencil edit, prefer `da-vinci pencil-session begin/, `${file} should require pencil-session begin before first edit`);
+    assert.doesNotMatch(content, /Prefer `da-vinci pencil-session begin --project <project-path> --pen <path>/, `${file} should not describe pencil-session begin as optional`);
+    assert.doesNotMatch(content, /优先执行 `da-vinci pencil-session begin --project <project-path> --pen <path>`/, `${file} should not describe pencil-session begin as optional in Chinese`);
+    assert.doesNotMatch(content, /优先通过 `da-vinci pencil-session begin`/, `${file} should not describe pencil-session begin as optional in Chinese mode guidance`);
+    assert.doesNotMatch(content, /先允许第一个通过审查的 live anchor surface 在内存里完成/, `${file} should not allow in-memory-first anchor work before seeding the project-local .pen`);
+  }
+});
+
+runTest("design-supervisor review stays distinct from preferred adapters and is wired into design prompts", () => {
+  const keyDocs = [
+    "SKILL.md",
+    "references/checkpoints.md",
+    "references/artifact-templates.md",
+    "docs/pencil-rendering-workflow.md",
+    "docs/zh-CN/pencil-rendering-workflow.md",
+    "docs/visual-adapters.md",
+    "docs/zh-CN/visual-adapters.md",
+    "docs/workflow-overview.md",
+    "docs/zh-CN/workflow-overview.md",
+    "docs/dv-command-reference.md",
+    "docs/zh-CN/dv-command-reference.md"
+  ];
+
+  for (const file of keyDocs) {
+    const content = read(file);
+    assert.match(content, /Design-supervisor reviewers|design-supervisor review/, `${file} should document the design-supervisor review gate`);
+  }
+
+  for (const file of [
+    "references/checkpoints.md",
+    "docs/pencil-rendering-workflow.md",
+    "docs/zh-CN/pencil-rendering-workflow.md",
+    "docs/visual-adapters.md",
+    "docs/zh-CN/visual-adapters.md"
+  ]) {
+    const content = read(file);
+    assert.match(content, /Require Supervisor Review/, `${file} should explain when supervisor review becomes a hard gate`);
+  }
+
+  const visualPresetFiles = [
+    "docs/visual-assist-presets/README.md",
+    "docs/visual-assist-presets/mobile-app.md",
+    "docs/visual-assist-presets/desktop-app.md",
+    "docs/visual-assist-presets/web-app.md",
+    "docs/visual-assist-presets/tablet-app.md",
+    "docs/zh-CN/visual-assist-presets/README.md",
+    "docs/zh-CN/visual-assist-presets/mobile-app.md",
+    "docs/zh-CN/visual-assist-presets/desktop-app.md",
+    "docs/zh-CN/visual-assist-presets/web-app.md",
+    "docs/zh-CN/visual-assist-presets/tablet-app.md"
+  ];
+
+  for (const file of visualPresetFiles) {
+    const content = read(file);
+    assert.match(content, /Design-supervisor reviewers/, `${file} should keep reviewer configuration visible in Visual Assist guidance`);
+    assert.match(content, /Require Supervisor Review/, `${file} should document the reviewer requirement toggle`);
+  }
+
+  const designPrompts = [
+    "commands/claude/dv/design.md",
+    "commands/codex/prompts/dv-design.md",
+    "commands/gemini/dv/design.toml",
+    "docs/prompt-presets/mobile-app.md",
+    "docs/prompt-presets/desktop-app.md",
+    "docs/prompt-presets/web-app.md",
+    "docs/prompt-presets/tablet-app.md",
+    "docs/zh-CN/prompt-presets/mobile-app.md",
+    "docs/zh-CN/prompt-presets/desktop-app.md",
+    "docs/zh-CN/prompt-presets/web-app.md",
+    "docs/zh-CN/prompt-presets/tablet-app.md"
+  ];
+
+  for (const file of designPrompts) {
+    const content = read(file);
+    assert.match(content, /design-supervisor review/, `${file} should instruct design flows to run design-supervisor review when configured`);
+    assert.match(content, /Require Supervisor Review: true/, `${file} should explain the hard-gate branch for supervisor review`);
+  }
+});
+
+console.log("All mode consistency tests passed.");