vibe-coding-master 0.7.20 → 0.7.21

package/README.md

@@ -9,7 +9,7 @@ VCM helps you keep one task organized across dedicated roles:
 - Architect
 - Coder
 - Tester
-- optional Gate Reviewer
+- optional Reviewer
 
 It runs locally, connects to a local Git repository, creates a task branch and
 worktree, starts embedded Claude Code terminals, and manages handoffs,
@@ -133,7 +133,7 @@ If you want VCM app state to survive container rebuilds, set:
 7. Review the harness/bootstrap commit diff.
 8. Start the role sessions, or use the saved launch template / one-click start.
 9. Talk mostly to Project Manager.
-10. Let PM route work to Architect, Coder, Tester, and Gate Reviewer when
+10. Let PM route work to Architect, Coder, Tester, and Reviewer when
     enabled.
 11. Review the final result and close the task when finished.
 
@@ -290,7 +290,7 @@ The global launch template stores per-role defaults for the five workflow roles:
 - auto orchestration
 
 One-click start launches the four core roles. If any Gate Review Gate is enabled,
-it also launches Gate Reviewer.
+it also launches Reviewer.
 
 Translator and Harness Engineer are tool roles. They are controlled from their
 own panels, not from the main role tab bar or the one-click launch template.
@@ -331,14 +331,14 @@ Available gates:
 - Validation adequacy
 - Code diff
 
-When a gate is enabled, VCM uses Gate Reviewer as an independent review role.
-Gate Reviewer reviews artifacts and diffs, writes a gate report, and returns
+When a gate is enabled, VCM uses Reviewer as an independent review role.
+Reviewer reviews artifacts and diffs, writes a gate report, and returns
 only:
 
 - `approve`
 - `request_changes`
 
-Gate Reviewer does not run tests and does not choose owners or fixes. PM routes
+Reviewer does not run tests and does not choose owners or fixes. PM routes
 findings back to the responsible role.
 
 ## Translation
@@ -405,7 +405,7 @@ Gateway can:
 - translate mobile messages when Gateway translation is enabled
 
 Gateway does not expose the embedded terminal and does not send directly to
-Architect, Coder, Tester, or Gate Reviewer.
+Architect, Coder, Tester, or Reviewer.
 
 Common commands:
 
@@ -462,7 +462,7 @@ files.
 
 `Auto memory` is the switch for the entire automated memory workflow. During
 Review Task Harness after Final Acceptance, Project Manager, Architect, Coder,
-Tester, and an enabled Gate Reviewer submit proposals in sequence through
+Tester, and an enabled Reviewer submit proposals in sequence through
 `vcm-propose-memory`. Harness Engineer verifies and consolidates them before VCM
 applies the result. Roles cannot edit active memory directly.
 
@@ -484,7 +484,7 @@ Final Acceptance
 ```
 
 Memory proposal prompts sent to Project Manager, Architect, Coder, Tester, and
-an enabled Gate Reviewer use their normal task sessions and participate in
+an enabled Reviewer use their normal task sessions and participate in
 Round/Turn tracking. Harness Engineer review and retrospective work remain tool
 role activity and do not participate in Round completion.
 

package/dist/backend/cli/install-vcm-harness.js

@@ -8,7 +8,7 @@ import { renderArchitectHarnessRules } from "../templates/harness/architect-agen
 import { renderCoderHarnessRules } from "../templates/harness/coder-agent.js";
 import { renderCoderWorkerHarnessRules } from "../templates/harness/coder-worker-agent.js";
 import { renderArchitectScaffoldWorkerHarnessRules } from "../templates/harness/architect-scaffold-worker-agent.js";
-import { renderGateReviewerAgentRules, renderRequestGateReviewTool, renderTranslatorAgentRules, renderVcmGateReviewSkillRules } from "../templates/harness/gate-review.js";
+import { renderReviewerAgentRules, renderRequestGateReviewTool, renderTranslatorAgentRules, renderVcmGateReviewSkillRules } from "../templates/harness/gate-review.js";
 import { renderHarnessEngineerHarnessRules } from "../templates/harness/harness-engineer-agent.js";
 import { renderRootClaudeHarnessRules } from "../templates/harness/claude-root.js";
 import { renderGitignoreHarnessRules } from "../templates/harness/gitignore.js";
@@ -34,6 +34,8 @@ const CLI_DIR = path.dirname(fileURLToPath(import.meta.url));
 const APP_ROOT = path.resolve(CLI_DIR, "../../..");
 const VCM_PACKAGE_VERSION = readVcmPackageVersion(APP_ROOT);
 const MANIFEST_PATH = ".ai/vcm-harness-manifest.json";
+const LEGACY_REVIEWER_AGENT_PATH = ".claude/agents/gate-reviewer.md";
+const REVIEWER_AGENT_PATH = ".claude/agents/reviewer.md";
 const HTML_BLOCK_PATTERN = /<!-- VCM:BEGIN(?:\s+version=\d+)? -->[\s\S]*?<!-- VCM:END -->/m;
 const HASH_BLOCK_PATTERN = /# VCM:BEGIN(?:\s+version=\d+)?\n[\s\S]*?# VCM:END/m;
 const LEGACY_CODEX_HARNESS_PATHS = [
@@ -71,7 +73,7 @@ const AGENT_FRONTMATTER = {
     tester: {
         description: "VCM testing role for validation, test adequacy, approved-scope validation, and risk findings."
     },
-    "gate-reviewer": {
+    reviewer: {
         description: "VCM independent gate review role for architecture plans, validation adequacy, and code diffs.",
         tools: "Read, Grep, Glob, Bash, Write"
     },
@@ -171,13 +173,13 @@ const MANAGED_FILES = [
         content: renderPullRequestTemplateHarnessRules()
     },
     {
-        path: ".claude/agents/gate-reviewer.md",
-        title: "Gate Reviewer Agent",
-        agentName: "gate-reviewer",
+        path: ".claude/agents/reviewer.md",
+        title: "Reviewer Agent",
+        agentName: "reviewer",
         commentStyle: "html",
-        category: "gate-reviewer-agent",
+        category: "reviewer-agent",
         memoryBlock: true,
-        content: renderGateReviewerAgentRules()
+        content: renderReviewerAgentRules()
     },
     {
         path: ".claude/agents/translator.md",
@@ -380,6 +382,7 @@ async function main() {
     const dryRun = args.dryRun;
     const operations = [];
     await assertDirectory(projectRoot, "Project root");
+    await migrateReviewerAgent({ projectRoot, dryRun, operations });
     const manifest = await buildManifest(projectRoot);
     for (const definition of MANAGED_FILES) {
         await installManagedFile({ projectRoot, definition, dryRun, operations });
@@ -405,6 +408,32 @@ async function main() {
     });
     printReport({ projectRoot, dryRun, operations });
 }
+async function migrateReviewerAgent({ projectRoot, dryRun, operations }) {
+    const legacyPath = resolveInside(projectRoot, LEGACY_REVIEWER_AGENT_PATH);
+    const legacyContent = await readOptionalText(legacyPath);
+    if (legacyContent === undefined) {
+        return;
+    }
+    const reviewerPath = resolveInside(projectRoot, REVIEWER_AGENT_PATH);
+    const reviewerContent = await readOptionalText(reviewerPath);
+    if (dryRun) {
+        operations.push(plan(LEGACY_REVIEWER_AGENT_PATH, reviewerContent === undefined
+            ? `rename to ${REVIEWER_AGENT_PATH}`
+            : `delete after ${REVIEWER_AGENT_PATH} was installed`));
+        return;
+    }
+    if (reviewerContent === undefined) {
+        const migratedContent = legacyContent
+            .replace(/^name:[ \t]*gate-reviewer[ \t]*$/m, "name: reviewer")
+            .replace(/^# Gate Reviewer Agent[ \t]*$/m, "# Reviewer Agent");
+        await fs.mkdir(path.dirname(reviewerPath), { recursive: true });
+        await fs.writeFile(reviewerPath, migratedContent, "utf8");
+    }
+    await fs.rm(legacyPath, { force: true });
+    operations.push(done(LEGACY_REVIEWER_AGENT_PATH, reviewerContent === undefined
+        ? `renamed to ${REVIEWER_AGENT_PATH}`
+        : `deleted after ${REVIEWER_AGENT_PATH} was installed`));
+}
 function parseArgs(argv) {
     const args = {
         dryRun: false,

package/dist/backend/services/auto-memory-service.js

@@ -14,7 +14,7 @@ const MEMORY_FILE_DEFINITIONS = [
     { path: ".claude/agents/architect.md", title: "Architect Memory", role: "architect" },
     { path: ".claude/agents/coder.md", title: "Coder Memory", role: "coder" },
     { path: ".claude/agents/tester.md", title: "Tester Memory", role: "tester" },
-    { path: ".claude/agents/gate-reviewer.md", title: "Gate Reviewer Memory", role: "gate-reviewer" },
+    { path: ".claude/agents/reviewer.md", title: "Reviewer Memory", role: "reviewer" },
     { path: ".claude/agents/harness-engineer.md", title: "Harness Engineer Memory", role: "harness-engineer" }
 ];
 export function createAutoMemoryService(deps) {
@@ -151,7 +151,7 @@ export function createAutoMemoryService(deps) {
         const gateSettings = await deps.appSettings.getGateReviewSettings(input.baseRepoRoot, input.taskSlug);
         const roles = ["project-manager", "architect", "coder", "tester"];
         if (gateSettings.enabled) {
-            roles.push("gate-reviewer");
+            roles.push("reviewer");
         }
         const timestamp = now();
         const runId = createRunId(timestamp, "auto");

package/dist/backend/services/claude-hook-service.js

@@ -1,4 +1,4 @@
-import { isGateReviewerRoleName, isHarnessEngineerToolRoleName, isTranslatorToolRoleName, isVcmRoleName } from "../../shared/constants.js";
+import { isHarnessEngineerToolRoleName, isReviewerRoleName, isTranslatorToolRoleName, isVcmRoleName } from "../../shared/constants.js";
 import { VcmError } from "../errors.js";
 import { readTranscriptTurnEvidence } from "./claude-transcript-reply.js";
 import { submitTerminalInput } from "../runtime/terminal-submit.js";
@@ -298,7 +298,7 @@ export function createClaudeHookService(deps) {
             }
         }
         return recordTurnEnd(input, context, eventName, {
-            dispatchRouteFiles: !isGateReviewerRoleName(input.role),
+            dispatchRouteFiles: !isReviewerRoleName(input.role),
             notifyGateway: true,
             settleGuard: true
         });
@@ -347,7 +347,7 @@ export function createClaudeHookService(deps) {
         if (hasCompletionEvidence) {
             await clearStopFailureRecoveryState(context, input.role);
             return recordTurnEnd(input, context, eventName, {
-                dispatchRouteFiles: !isGateReviewerRoleName(input.role),
+                dispatchRouteFiles: !isReviewerRoleName(input.role),
                 notifyGateway: false,
                 settleGuard: true
             });

package/dist/backend/services/gate-review-service.js

@@ -6,11 +6,11 @@ import { VcmError } from "../errors.js";
 import { resolveRepoPath } from "../adapters/filesystem.js";
 import { submitTerminalInput } from "../runtime/terminal-submit.js";
 import { getTaskRuntimeRepoRoot } from "./task-service.js";
-const GATE_REVIEW_AGENT_PATH = ".claude/agents/gate-reviewer.md";
+const REVIEWER_AGENT_PATH = ".claude/agents/reviewer.md";
 const GATE_REVIEW_DIR = ".ai/vcm/gate-reviews";
 const REQUESTS_DIR = ".ai/vcm/gate-reviews/requests";
 const GATE_REVIEW_VERSION = 1;
-const GATE_REVIEWER_ROLE = "gate-reviewer";
+const REVIEWER_ROLE = "reviewer";
 const DEFAULT_REPORT_POLL_INTERVAL_MS = 1000;
 const DEFAULT_REPORT_TIMEOUT_MS = 30 * 60 * 1000;
 const activeRuns = new Set();
@@ -431,7 +431,7 @@ export function createGateReviewService(deps) {
             });
             await updateRequestStatus(deps.fs, context, requestId, "running", { startedAt: timestamp });
             const reviewDir = resolveRepoPath(context.taskRepoRoot, GATE_REVIEW_DIR);
-            const agentPath = resolveRepoPath(context.repoRoot, GATE_REVIEW_AGENT_PATH);
+            const agentPath = resolveRepoPath(context.repoRoot, REVIEWER_AGENT_PATH);
             const prompt = buildGatePrompt(context, gate, requestId, codeDiffInput, codeDiffSources);
             await deps.fs.ensureDir(reviewDir);
             await deps.fs.ensureDir(resolveRepoPath(context.taskRepoRoot, REQUESTS_DIR));
@@ -439,20 +439,20 @@ export function createGateReviewService(deps) {
             if (!(await deps.fs.pathExists(agentPath))) {
                 throw new VcmError({
                     code: "GATE_REVIEW_AGENT_MISSING",
-                    message: `${GATE_REVIEW_AGENT_PATH} does not exist.`,
+                    message: `${REVIEWER_AGENT_PATH} does not exist.`,
                     statusCode: 409,
                     hint: "Apply the VCM harness before requesting Gate Review Gates."
                 });
             }
-            const session = await ensureGateReviewerSession(context);
+            const session = await ensureReviewerSession(context);
             await submitTerminalInput(deps.runtime, session.id, prompt);
-            await deps.sessionService.markRoleActivityRunning(context.repoRoot, context.taskSlug, GATE_REVIEWER_ROLE, session.id);
+            await deps.sessionService.markRoleActivityRunning(context.repoRoot, context.taskSlug, REVIEWER_ROLE, session.id);
             await deps.roundService.recordRoleTurnEvent({
                 repoRoot: context.repoRoot,
                 stateRepoRoot: context.taskRepoRoot,
                 stateRoot: context.stateRoot,
                 taskSlug: context.taskSlug,
-                role: GATE_REVIEWER_ROLE,
+                role: REVIEWER_ROLE,
                 eventName: "UserPromptSubmit"
             });
             const parsed = await waitForGateReport(deps.fs, context.taskRepoRoot, gate, requestId, now(), {
@@ -501,24 +501,24 @@ export function createGateReviewService(deps) {
             activeRuns.delete(runKey);
         }
     }
-    async function ensureGateReviewerSession(context) {
-        const existing = await deps.sessionService.getRoleSession(context.repoRoot, context.taskSlug, GATE_REVIEWER_ROLE);
+    async function ensureReviewerSession(context) {
+        const existing = await deps.sessionService.getRoleSession(context.repoRoot, context.taskSlug, REVIEWER_ROLE);
         if (existing?.status === "running" && deps.runtime.getSession(existing.id)) {
             return existing;
         }
         if (existing?.claudeSessionId) {
             try {
-                return await deps.sessionService.resumeRoleSession(context.repoRoot, context.taskSlug, GATE_REVIEWER_ROLE, {
+                return await deps.sessionService.resumeRoleSession(context.repoRoot, context.taskSlug, REVIEWER_ROLE, {
                     cols: 100,
                     rows: 28,
                     model: "default"
                 });
             }
             catch {
-                // Fall through to a fresh Gate Reviewer terminal if the saved session cannot be resumed.
+                // Fall through to a fresh Reviewer terminal if the saved session cannot be resumed.
             }
         }
-        return deps.sessionService.startRoleSession(context.repoRoot, context.taskSlug, GATE_REVIEWER_ROLE, {
+        return deps.sessionService.startRoleSession(context.repoRoot, context.taskSlug, REVIEWER_ROLE, {
             cols: 100,
             rows: 28,
             model: "default"
@@ -891,7 +891,7 @@ async function computeInputHash(deps, taskRepoRoot, gate, codeDiffInput, codeDif
         "CLAUDE.md",
         ".claude/agents/architect.md",
         ".claude/agents/coder.md",
-        ".claude/agents/gate-reviewer.md",
+        ".claude/agents/reviewer.md",
         ".claude/agents/tester.md",
         ".claude/skills/vcm-gate-review/SKILL.md",
         ".ai/tools/request-gate-review",
@@ -1048,13 +1048,13 @@ function buildGatePrompt(context, gate, requestId, codeDiffInput, codeDiffSource
         ? "\nDiff: inspect git status/diff in Worktree."
         : "";
     const architectureContract = gate === "architecture-plan"
-        ? "\n\nComplete every Architecture Analysis field required by the Gate Reviewer role with concrete current-worktree evidence before deciding."
+        ? "\n\nComplete every Architecture Analysis field required by the Reviewer role with concrete current-worktree evidence before deciding."
         : "";
     const validationContract = gate === "validation-adequacy"
-        ? "\n\nComplete every Validation Analysis field required by the Gate Reviewer role with concrete current-worktree production and test evidence before deciding."
+        ? "\n\nComplete every Validation Analysis field required by the Reviewer role with concrete current-worktree production and test evidence before deciding."
         : "";
     const codeDiffContract = gate === "code-diff"
-        ? "\n\nComplete every Code Diff Analysis field required by the Gate Reviewer role with concrete evidence from the named commit range before deciding."
+        ? "\n\nComplete every Code Diff Analysis field required by the Reviewer role with concrete evidence from the named commit range before deciding."
         : "";
     const codeDiffSection = gate === "code-diff" && codeDiffInput
         ? `
@@ -1109,7 +1109,7 @@ async function waitForGateReport(fs, taskRepoRoot, gate, requestId, timestamp, o
     const detail = errorMessage(lastError);
     throw new VcmError({
         code: "GATE_REVIEW_REPORT_TIMEOUT",
-        message: `Gate Reviewer did not produce a valid ${gate} report within ${Math.round(options.timeoutMs / 1000)}s.`,
+        message: `Reviewer did not produce a valid ${gate} report within ${Math.round(options.timeoutMs / 1000)}s.`,
         statusCode: 504,
         hint: detail
     });

package/dist/backend/services/harness-service.js

@@ -5,7 +5,7 @@ import { renderArchitectHarnessRules } from "../templates/harness/architect-agen
 import { renderCoderHarnessRules } from "../templates/harness/coder-agent.js";
 import { renderCoderWorkerHarnessRules } from "../templates/harness/coder-worker-agent.js";
 import { renderArchitectScaffoldWorkerHarnessRules } from "../templates/harness/architect-scaffold-worker-agent.js";
-import { renderGateReviewerAgentRules, renderRequestGateReviewTool, renderTranslatorAgentRules, renderVcmGateReviewSkillRules } from "../templates/harness/gate-review.js";
+import { renderReviewerAgentRules, renderRequestGateReviewTool, renderTranslatorAgentRules, renderVcmGateReviewSkillRules } from "../templates/harness/gate-review.js";
 import { renderHarnessEngineerHarnessRules } from "../templates/harness/harness-engineer-agent.js";
 import { renderRootClaudeHarnessRules } from "../templates/harness/claude-root.js";
 import { renderGitignoreHarnessRules } from "../templates/harness/gitignore.js";
@@ -183,12 +183,12 @@ const HARNESS_FILES = [
         renderRules: renderRestartArchitectSkillRules
     },
     {
-        kind: "agent-gate-reviewer",
-        path: ".claude/agents/gate-reviewer.md",
-        title: "Gate Reviewer Agent",
+        kind: "agent-reviewer",
+        path: ".claude/agents/reviewer.md",
+        title: "Reviewer Agent",
         memoryBlock: true,
-        frontmatter: renderAgentFrontmatter("gate-reviewer", "VCM independent gate review role for architecture plans, validation adequacy, and code diffs.", { tools: "Read, Grep, Glob, Bash, Write" }),
-        renderRules: renderGateReviewerAgentRules
+        frontmatter: renderAgentFrontmatter("reviewer", "VCM independent gate review role for architecture plans, validation adequacy, and code diffs.", { tools: "Read, Grep, Glob, Bash, Write" }),
+        renderRules: renderReviewerAgentRules
     },
     {
         kind: "agent-translator",
@@ -1279,7 +1279,7 @@ async function analyzeLegacyCodexHarnessPaths(fs, repoRoot) {
         changes.push({
             path: relativePath,
             action: "delete",
-            reason: "Legacy Codex harness path is obsolete; VCM now uses Claude Code Gate Reviewer and Translator roles."
+            reason: "Legacy Codex harness path is obsolete; VCM now uses Claude Code Reviewer and Translator roles."
         });
     }
     return changes;

package/dist/backend/services/task-launch-service.js

@@ -1,4 +1,4 @@
-import { CORE_VCM_ROLE_DEFINITIONS, GATE_REVIEWER_ROLE_DEFINITION, VCM_ROLE_NAMES } from "../../shared/constants.js";
+import { CORE_VCM_ROLE_DEFINITIONS, REVIEWER_ROLE_DEFINITION, VCM_ROLE_NAMES } from "../../shared/constants.js";
 import { VcmError } from "../errors.js";
 import { getTaskRuntimeRepoRoot } from "./task-service.js";
 const ONE_CLICK_SESSION_COLS = 100;
@@ -25,10 +25,10 @@ export function createTaskLaunchService(deps) {
             mode
         });
     }
-    function composeRoleDefinitions(gateReviewerEnabled) {
+    function composeRoleDefinitions(reviewerEnabled) {
         return [
             ...CORE_VCM_ROLE_DEFINITIONS,
-            ...(gateReviewerEnabled ? [GATE_REVIEWER_ROLE_DEFINITION] : [])
+            ...(reviewerEnabled ? [REVIEWER_ROLE_DEFINITION] : [])
         ];
     }
     // Skip a running role, resume one that has a prior Claude session, otherwise

package/dist/backend/templates/harness/gate-review.js

@@ -1,10 +1,10 @@
 import { renderRoleMemoryRules } from "./role-memory.js";
-export function renderGateReviewerAgentRules() {
+export function renderReviewerAgentRules() {
     return `## Role
 
-You are VCM \`gate-reviewer\`.
+You are VCM \`reviewer\`.
 
-${renderRoleMemoryRules("gate-reviewer")}
+${renderRoleMemoryRules("reviewer")}
 
 Review only the gate in the VCM prompt. Use the task and worktree paths named there. Project memory may orient you, but only current worktree evidence can decide the gate.
 
@@ -459,7 +459,7 @@ When VCM sends \`[VCM GATE REVIEW CALLBACK]\`, read the named report path.
 - \`failed\`: stop and ask the user to retry, skip, or override in VCM.
 - \`skipped\` or \`overridden\`: record the exception reason in PM context and continue only as appropriate.
 
-Do not ask Gate Reviewer to choose owners, fixes, Replan, or user-intervention needs. PM routes those decisions through normal VCM responsibilities.`;
+Do not ask Reviewer to choose owners, fixes, Replan, or user-intervention needs. PM routes those decisions through normal VCM responsibilities.`;
 }
 export function renderRequestGateReviewTool() {
     return `#!/usr/bin/env python3
@@ -691,7 +691,7 @@ def input_hash(root: Path, gate: str, sources: list[str] | None = None, gate_rec
         "CLAUDE.md",
         ".claude/agents/architect.md",
         ".claude/agents/coder.md",
-        ".claude/agents/gate-reviewer.md",
+        ".claude/agents/reviewer.md",
         ".claude/agents/tester.md",
         ".claude/skills/vcm-gate-review/SKILL.md",
         ".ai/tools/request-gate-review",

package/dist/backend/templates/harness/project-manager-agent.js

@@ -40,7 +40,7 @@ PM Managed Mode applies only when the user explicitly asks to complete the curre
 - PM must drive the accepted task to completion through the normal VCM flow.
 - PM must not reduce, defer, reinterpret, skip, or move requested work outside the current task.
 - PM must not use workload, task size, context size, implementation difficulty, dependency choice, refactor need, testing effort, or number of iterations as a reason to ask the user.
-- Technical execution questions are handled inside the VCM flow. PM routes them to Architect, Coder, Tester, or Gate Reviewer according to role responsibility.
+- Technical execution questions are handled inside the VCM flow. PM routes them to Architect, Coder, Tester, or Reviewer according to role responsibility.
 - PM may defer non-blocking user-facing questions until the final user report, but only when continuing does not require user intent, external authorization, or a user-approved exception.
 - Deferred questions remain part of the current task report. They must not become follow-up scope unless the user explicitly creates a new task.
 - PM must pause and ask the user only when the task cannot proceed without user intent or real-world authorization: unclear or conflicting requested outcome, required account/secret/test environment/data access, real cost, production permission, sensitive data access, destructive or irreversible real-world action, durable-doc conflict requiring user choice, or a required user-approved exception.
@@ -334,7 +334,7 @@ PM may lightly rewrite the user's words to:
 - Do not run \`code-diff\` for incomplete, failed, planning-only, Docs-Only Flow, Validation-Only Flow, PR-Preparation Flow, or Communication-Only Flow.
 - Gate Review trigger points apply only when the active delivery flow reaches that milestone. Do not run Gate Review for Communication-Only Flow.
 - On a callback, accept only \`approve\` or \`request_changes\`. Apply \`request_changes\` through the allowed branch defined by the active flow; in Code-Change Flow use Architecture Plan Revision, Code-Diff Correction, or Validation Revision according to the gate.
-- Do not ask Gate Reviewer to choose owners, fixes, Replan, or user-intervention needs.
+- Do not ask Reviewer to choose owners, fixes, Replan, or user-intervention needs.
 - Record gate decision, report path, and any skip or override reason.
 
 ### Partial Role Results

package/dist/shared/constants.js

@@ -26,15 +26,15 @@ export const CORE_VCM_ROLE_DEFINITIONS = [
         dispatchable: true
     }
 ];
-export const GATE_REVIEWER_ROLE_DEFINITION = {
-    name: "gate-reviewer",
-    label: "Gate Reviewer",
-    commandAgent: "gate-reviewer",
+export const REVIEWER_ROLE_DEFINITION = {
+    name: "reviewer",
+    label: "Reviewer",
+    commandAgent: "reviewer",
     dispatchable: false
 };
 export const VCM_ROLE_DEFINITIONS = [
     ...CORE_VCM_ROLE_DEFINITIONS,
-    GATE_REVIEWER_ROLE_DEFINITION
+    REVIEWER_ROLE_DEFINITION
 ];
 export const TRANSLATOR_TOOL_ROLE_DEFINITION = {
     name: "translator",
@@ -69,8 +69,8 @@ export function isRoleName(value) {
 export function isVcmRoleName(value) {
     return VCM_ROLE_NAMES.includes(value);
 }
-export function isGateReviewerRoleName(value) {
-    return value === GATE_REVIEWER_ROLE_DEFINITION.name;
+export function isReviewerRoleName(value) {
+    return value === REVIEWER_ROLE_DEFINITION.name;
 }
 export function isToolRoleName(value) {
     return TOOL_ROLE_NAMES.includes(value);

package/dist/shared/types/memory.js

@@ -3,6 +3,6 @@ export const VCM_MEMORY_ROLE_NAMES = [
     "architect",
     "coder",
     "tester",
-    "gate-reviewer",
+    "reviewer",
     "harness-engineer"
 ];