vibe-coding-master 0.6.19 → 0.6.21

package/README.md

@@ -8,7 +8,7 @@ VCM helps you keep one task organized across dedicated roles:
 - Project Manager
 - Architect
 - Coder
-- Reviewer
+- Tester
 - optional Gate Reviewer
 
 It runs locally, connects to a local Git repository, creates a task branch and
@@ -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, Reviewer, and Gate Reviewer when
+10. Let PM route work to Architect, Coder, Tester, and Gate Reviewer when
     enabled.
 11. Review the final result and close the task when finished.
 
@@ -187,7 +187,7 @@ Typical flow:
 Project Manager
   -> Architect
   -> Coder
-  -> Reviewer
+  -> Tester
   -> Architect docs sync
   -> Project Manager final acceptance
 ```
@@ -264,7 +264,7 @@ Available gates:
 
 - Architecture plan
 - Validation adequacy
-- Final diff
+- 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
@@ -337,7 +337,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, Reviewer, or Gate Reviewer.
+Architect, Coder, Tester, or Gate Reviewer.
 
 Common commands:
 

package/dist/backend/api/artifact-routes.js

@@ -70,8 +70,8 @@ function artifactNameToPath(paths, artifactName) {
     if (artifactName === "known-issues.md") {
         return paths.knownIssuesPath;
     }
-    if (artifactName === "review-report.md") {
-        return paths.reviewReportPath;
+    if (artifactName === "test-report.md") {
+        return paths.testReportPath;
     }
     if (artifactName === "docs-sync-report.md") {
         return paths.docsSyncReportPath;

package/dist/backend/api/gate-review-routes.js

@@ -12,7 +12,7 @@ export function registerGateReviewRoutes(app, deps) {
     app.post("/api/tasks/:taskSlug/gate-review/:gate/request", async (request) => {
         const project = await requireCurrentProject(deps.projectService);
         const gate = parseGate(request.params.gate);
-        return deps.gateReviewService.requestReviewGate(project.repoRoot, request.params.taskSlug, gate);
+        return deps.gateReviewService.requestReviewGate(project.repoRoot, request.params.taskSlug, gate, request.body);
     });
     app.post("/api/tasks/:taskSlug/gate-review/:gate/retry", async (request) => {
         const project = await requireCurrentProject(deps.projectService);

package/dist/backend/api/task-routes.js

@@ -196,7 +196,7 @@ function degradedArtifactSummary(handoffDir) {
             messageRoutePaths: {},
             architecturePlanPath: `${handoffDir}/architecture-plan.md`,
             knownIssuesPath: `${handoffDir}/known-issues.md`,
-            reviewReportPath: `${handoffDir}/review-report.md`,
+            testReportPath: `${handoffDir}/test-report.md`,
             docsSyncReportPath: `${handoffDir}/docs-sync-report.md`,
             finalAcceptancePath: `${handoffDir}/final-acceptance.md`
         },

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

@@ -11,9 +11,12 @@ import { renderGateReviewerAgentRules, renderRequestGateReviewTool, renderTransl
 import { renderHarnessEngineerHarnessRules } from "../templates/harness/harness-engineer-agent.js";
 import { renderRootClaudeHarnessRules } from "../templates/harness/claude-root.js";
 import { renderGitignoreHarnessRules } from "../templates/harness/gitignore.js";
+import { renderLegacyProjectCodingStandardsTemplate, renderProjectCodingStandardsProjectSection, renderProjectCodingStandardsRules } from "../templates/harness/project-coding-standards.js";
+import { renderProjectGlossaryTemplate } from "../templates/harness/project-glossary.js";
+import { renderLegacyProjectKnownIssuesTemplate, renderProjectKnownIssuesRules, renderProjectKnownIssuesSection } from "../templates/harness/project-known-issues.js";
 import { renderProjectManagerHarnessRules } from "../templates/harness/project-manager-agent.js";
 import { renderPullRequestTemplateHarnessRules } from "../templates/harness/pull-request-template.js";
-import { renderReviewerHarnessRules } from "../templates/harness/reviewer-agent.js";
+import { renderTesterHarnessRules } from "../templates/harness/tester-agent.js";
 import { renderVcmFinalAcceptanceSkillRules } from "../templates/harness/vcm-final-acceptance-skill.js";
 import { renderVcmHarnessBootstrapSkillRules } from "../templates/harness/vcm-harness-bootstrap-skill.js";
 import { renderVcmLongRunningValidationSkillRules } from "../templates/harness/vcm-long-running-validation-skill.js";
@@ -37,6 +40,7 @@ const VCM_STOP_HOOK_COMMAND = `sh -c 'if [ -z "\${VCM_TASK_SLUG:-}" ] || [ -z "\
 const VCM_PERMISSION_REQUEST_HOOK_COMMAND = `sh -c 'if [ -z "\${VCM_TASK_SLUG:-}" ] || [ -z "\${VCM_ROLE:-}" ] || [ -z "\${VCM_API_URL:-}" ]; then exit 0; fi; node -e '"'"'let s="";process.stdin.setEncoding("utf8");process.stdin.on("data",d=>s+=d);process.stdin.on("end",()=>{let event={};try{event=s.trim()?JSON.parse(s):{};}catch{event={raw:s};}process.stdout.write(JSON.stringify({taskSlug:process.env.VCM_TASK_SLUG,role:process.env.VCM_ROLE,event}));});'"'"' | curl -fsS --max-time 5 -X POST "\${VCM_API_URL}/api/hooks/claude-code/permission-request" -H "content-type: application/json" --data-binary @- || true'`;
 const VCM_BASH_GUARD_HOOK_COMMAND = `sh -c 'if [ -z "\${VCM_TASK_SLUG:-}" ] || [ -z "\${VCM_ROLE:-}" ]; then exit 0; fi; guard=""; repo="$(git rev-parse --show-toplevel 2>/dev/null || true)"; if [ -n "$repo" ] && [ -f "$repo/.ai/tools/vcm-bash-guard" ]; then guard="$repo/.ai/tools/vcm-bash-guard"; else cwd="$(pwd -P 2>/dev/null || pwd)"; dir="$cwd"; while [ -n "$dir" ] && [ "$dir" != "/" ]; do if [ -f "$dir/.ai/tools/vcm-bash-guard" ]; then guard="$dir/.ai/tools/vcm-bash-guard"; break; fi; dir="$(dirname "$dir")"; done; if [ -z "$guard" ] && [ -n "\${CLAUDE_PROJECT_DIR:-}" ] && [ -f "\${CLAUDE_PROJECT_DIR}/.ai/tools/vcm-bash-guard" ]; then guard="\${CLAUDE_PROJECT_DIR}/.ai/tools/vcm-bash-guard"; fi; fi; [ -n "$guard" ] || exit 0; python3 "$guard" || exit 0'`;
 const VCM_BASH_DEFAULT_TIMEOUT_MS = "600000";
+const VCM_AUTO_MEMORY_ENABLED = false;
 const VCM_HOOK_DEFINITIONS = [
     { eventName: "PreToolUse", matcher: "Bash", command: VCM_BASH_GUARD_HOOK_COMMAND, timeout: 10 },
     { eventName: "UserPromptSubmit", command: VCM_HOOK_COMMAND, timeout: 5 },
@@ -56,11 +60,12 @@ const AGENT_FRONTMATTER = {
         description: "VCM implementation role for scoped code changes and focused tests.",
         tools: "Read, Grep, Glob, Bash, Edit, Write, Agent"
     },
-    reviewer: {
-        description: "VCM independent review role for acceptance, test adequacy, scope checks, and risk findings."
+    tester: {
+        description: "VCM testing role for validation, test adequacy, approved-scope validation, and risk findings."
     },
     "gate-reviewer": {
-        description: "VCM independent gate review role for architecture plans, validation adequacy, and final diffs."
+        description: "VCM independent gate review role for architecture plans, validation adequacy, and code diffs.",
+        tools: "Read, Grep, Glob, Bash, Write"
     },
     translator: {
         description: "VCM project translation tool role for conversation translation, file translation, bootstrap, and memory updates."
@@ -82,6 +87,24 @@ const MANAGED_FILES = [
         blankLineBeforeEnd: true,
         content: renderRootClaudeHarnessRules()
     },
+    {
+        path: "docs/CODING_STANDARDS.md",
+        title: "Coding Standards",
+        commentStyle: "html",
+        category: "project-coding-standards",
+        content: renderProjectCodingStandardsRules(),
+        contentAfterBlock: renderProjectCodingStandardsProjectSection(),
+        legacyWholeFile: renderLegacyProjectCodingStandardsTemplate()
+    },
+    {
+        path: "docs/known-issues.md",
+        title: "Known Issues",
+        commentStyle: "html",
+        category: "project-known-issues",
+        content: renderProjectKnownIssuesRules(),
+        contentAfterBlock: renderProjectKnownIssuesSection(),
+        legacyWholeFile: renderLegacyProjectKnownIssuesTemplate()
+    },
     {
         path: ".gitignore",
         title: ".gitignore",
@@ -115,12 +138,12 @@ const MANAGED_FILES = [
         content: renderCoderHarnessRules()
     },
     {
-        path: ".claude/agents/reviewer.md",
-        title: "Reviewer Agent",
-        agentName: "reviewer",
+        path: ".claude/agents/tester.md",
+        title: "Tester Agent",
+        agentName: "tester",
         commentStyle: "html",
         category: "core-agent",
-        content: renderReviewerHarnessRules()
+        content: renderTesterHarnessRules()
     },
     {
         path: ".github/pull_request_template.md",
@@ -163,6 +186,10 @@ const MANAGED_FILES = [
     }
 ];
 const DURABLE_DOC_TEMPLATES = [
+    {
+        path: "docs/GLOSSARY.md",
+        content: renderProjectGlossaryTemplate()
+    },
     {
         path: "docs/ARCHITECTURE.md",
         content: "# Architecture\n"
@@ -171,10 +198,6 @@ const DURABLE_DOC_TEMPLATES = [
         path: "docs/TESTING.md",
         content: "# Testing\n"
     },
-    {
-        path: "docs/known-issues.md",
-        content: "# Known Issues\n"
-    }
 ];
 const WHOLE_FILES = [
     {
@@ -193,7 +216,7 @@ const WHOLE_FILES = [
         path: ".claude/skills/vcm-final-acceptance/SKILL.md",
         category: "skill",
         mode: 0o644,
-        content: renderSkillFile("VCM Final Acceptance Skill", "vcm-final-acceptance", "Use when project-manager is ready to decide whether a VCM-managed task can be accepted, returned for follow-up, or blocked for a decision.", renderVcmFinalAcceptanceSkillRules())
+        content: renderSkillFile("VCM Final Acceptance Skill", "vcm-final-acceptance", "Use when project-manager is ready to close a complete VCM code-change flow.", renderVcmFinalAcceptanceSkillRules())
     },
     {
         path: ".claude/skills/vcm-harness-bootstrap/SKILL.md",
@@ -211,7 +234,7 @@ const WHOLE_FILES = [
         path: ".claude/skills/vcm-route-message/SKILL.md",
         category: "skill",
         mode: 0o644,
-        content: renderSkillFile("VCM Route Message Skill", "vcm-route-message", "Use when a VCM role needs to hand off work, ask a question, report a result, report a blocker, or raise a finding to another VCM role.", renderVcmRouteMessageSkillRules())
+        content: renderSkillFile("VCM Route Message Skill", "vcm-route-message", "Use when project-manager dispatches a VCM role or when a VCM role reports a question, result, blocker, or finding back to project-manager.", renderVcmRouteMessageSkillRules())
     },
     {
         path: ".claude/skills/vcm-gate-review/SKILL.md",
@@ -381,6 +404,8 @@ async function buildManifest(projectRoot) {
                 }
             },
             ...fixedDirectories().map((directory) => manifestEntry(directory, "directory", directoryCategory(directory), "vcm-created")),
+            manifestEntry("docs/GLOSSARY.md", "file", "project-glossary", "project-owned"),
+            manifestEntry("docs/CODING_STANDARDS.md", "file", "project-coding-standards", "project-owned"),
             ...WHOLE_FILES.map((file) => ({
                 path: file.path,
                 entryType: "file",
@@ -500,9 +525,13 @@ async function installManagedFile({ projectRoot, definition, dryRun, operations
     }
     else {
         const pattern = definition.commentStyle === "hash" ? HASH_BLOCK_PATTERN : HTML_BLOCK_PATTERN;
-        nextContent = pattern.test(currentContent)
-            ? currentContent.replace(pattern, block)
-            : `${currentContent.trimEnd()}\n\n${block}\n`;
+        if (pattern.test(currentContent)) {
+            nextContent = currentContent.replace(pattern, block);
+        }
+        else {
+            nextContent = migrateLegacyManagedFile(definition, currentContent, block)
+                ?? `${currentContent.trimEnd()}\n\n${block}\n`;
+        }
     }
     await writeIfChanged({
         targetPath,
@@ -540,13 +569,29 @@ function renderManagedBlock(definition) {
     return `<!-- VCM:BEGIN version=1 -->\n${body}${endSpacing}<!-- VCM:END -->`;
 }
 function renderNewManagedFile(definition, block) {
+    const suffix = definition.contentAfterBlock?.trim();
     if (definition.agentName) {
         const frontmatter = AGENT_FRONTMATTER[definition.agentName];
         const tools = frontmatter.tools ?? "Read, Grep, Glob, Bash, Edit, Write";
         const model = frontmatter.model ? `\nmodel: ${frontmatter.model}` : "";
-        return `---\nname: ${definition.agentName}\ndescription: ${frontmatter.description}\ntools: ${tools}${model}\n---\n\n# ${definition.title}\n\n${block}\n`;
+        return `---\nname: ${definition.agentName}\ndescription: ${frontmatter.description}\ntools: ${tools}${model}\n---\n\n# ${definition.title}\n\n${block}${suffix ? `\n\n${suffix}` : ""}\n`;
+    }
+    return `# ${definition.title}\n\n${block}${suffix ? `\n\n${suffix}` : ""}\n`;
+}
+function migrateLegacyManagedFile(definition, currentContent, block) {
+    const legacyContent = definition.legacyWholeFile?.trimEnd();
+    if (!legacyContent) {
+        return undefined;
+    }
+    const current = currentContent.trimEnd();
+    if (current !== legacyContent && !current.startsWith(`${legacyContent}\n`)) {
+        return undefined;
     }
-    return `# ${definition.title}\n\n${block}\n`;
+    const projectContent = current.slice(legacyContent.length).trim();
+    return renderNewManagedFile({
+        ...definition,
+        contentAfterBlock: projectContent || definition.contentAfterBlock
+    }, block);
 }
 function renderSkillFile(title, name, description, body) {
     return `---\nname: ${name}\ndescription: ${description}\n---\n\n# ${title}\n\n${body.trimEnd()}\n`;
@@ -565,7 +610,7 @@ async function installClaudeSettings({ projectRoot, dryRun, operations }) {
         mode: 0o644,
         dryRun,
         operations,
-        action: "merge VCM Claude hooks"
+        action: "merge VCM Claude settings"
     });
 }
 function mergeVcmHooks(settings) {
@@ -602,6 +647,7 @@ function mergeVcmHooks(settings) {
     const env = isPlainObject(next.env) ? { ...next.env } : {};
     env.BASH_DEFAULT_TIMEOUT_MS = VCM_BASH_DEFAULT_TIMEOUT_MS;
     next.env = env;
+    next.autoMemoryEnabled = VCM_AUTO_MEMORY_ENABLED;
     return next;
 }
 function isOwnedHookMatcher(matcher) {

package/dist/backend/services/app-settings-service.js

@@ -203,13 +203,22 @@ function normalizeGateReviewGates(input) {
     }
     const gates = [];
     for (const value of input) {
-        if (!GATE_REVIEW_GATES.includes(value) || gates.includes(value)) {
+        const gate = normalizeGateReviewGateValue(value);
+        if (!gate || gates.includes(gate)) {
             continue;
         }
-        gates.push(value);
+        gates.push(gate);
     }
     return GATE_REVIEW_GATES.filter((gate) => gates.includes(gate));
 }
+function normalizeGateReviewGateValue(value) {
+    if (value === "final-diff") {
+        return "code-diff";
+    }
+    return GATE_REVIEW_GATES.includes(value)
+        ? value
+        : undefined;
+}
 function normalizeSettingsFile(input) {
     const settings = {
         version: 1,

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

@@ -3,12 +3,12 @@ import { DISPATCHABLE_ROLES } from "../../shared/constants.js";
 import { checkMarkdownArtifact } from "../../shared/validation/artifact-check.js";
 import { VcmError } from "../errors.js";
 import { resolveRepoPath } from "../adapters/filesystem.js";
-import { renderArchitecturePlanTemplate, renderDocsSyncReportTemplate, renderFinalAcceptanceTemplate, renderKnownIssuesTemplate, renderMessageRouteTemplate, renderReviewReportTemplate } from "../templates/handoff.js";
+import { renderArchitecturePlanTemplate, renderCoderCompletionTemplate, renderDocsSyncReportTemplate, renderFinalAcceptanceTemplate, renderKnownIssuesTemplate, renderMessageRouteTemplate, renderTestReportTemplate } from "../templates/handoff.js";
 import { renderRoleCommandTemplate } from "../templates/role-command.js";
 const ARTIFACT_PATH_KEYS = [
     ["architecture-plan", "architecturePlanPath"],
     ["known-issues", "knownIssuesPath"],
-    ["review-report", "reviewReportPath"],
+    ["test-report", "testReportPath"],
     ["docs-sync-report", "docsSyncReportPath"],
     ["final-acceptance", "finalAcceptancePath"]
 ];
@@ -16,10 +16,10 @@ const ROLE_COMMAND_PLACEHOLDER_PATTERN = /(^|\n)\s*(TBD|status:\s*draft)\s*(\n|$
 const DEFAULT_MESSAGE_ROUTES = [
     ["project-manager", "architect"],
     ["project-manager", "coder"],
-    ["project-manager", "reviewer"],
+    ["project-manager", "tester"],
     ["architect", "project-manager"],
     ["coder", "project-manager"],
-    ["reviewer", "project-manager"]
+    ["tester", "project-manager"]
 ];
 export function createArtifactService(fs) {
     return {
@@ -33,12 +33,12 @@ export function createArtifactService(fs) {
                 roleCommandPaths: {
                     architect: path.posix.join(roleCommandsDir, "architect.md"),
                     coder: path.posix.join(roleCommandsDir, "coder.md"),
-                    reviewer: path.posix.join(roleCommandsDir, "reviewer.md")
+                    tester: path.posix.join(roleCommandsDir, "tester.md")
                 },
                 messageRoutePaths: getDefaultMessageRoutePaths(messagesDir),
                 architecturePlanPath: path.posix.join(handoffDir, "architecture-plan.md"),
                 knownIssuesPath: path.posix.join(handoffDir, "known-issues.md"),
-                reviewReportPath: path.posix.join(handoffDir, "review-report.md"),
+                testReportPath: path.posix.join(handoffDir, "test-report.md"),
                 docsSyncReportPath: path.posix.join(handoffDir, "docs-sync-report.md"),
                 finalAcceptancePath: path.posix.join(handoffDir, "final-acceptance.md")
             };
@@ -58,10 +58,11 @@ export function createArtifactService(fs) {
             const files = [
                 [paths.roleCommandPaths.architect, renderRoleCommandTemplate(input.taskSlug, "architect", input.repoRoot, input.branch)],
                 [paths.roleCommandPaths.coder, renderRoleCommandTemplate(input.taskSlug, "coder", input.repoRoot, input.branch)],
-                [paths.roleCommandPaths.reviewer, renderRoleCommandTemplate(input.taskSlug, "reviewer", input.repoRoot, input.branch)],
+                [paths.roleCommandPaths.tester, renderRoleCommandTemplate(input.taskSlug, "tester", input.repoRoot, input.branch)],
                 [paths.architecturePlanPath, renderArchitecturePlanTemplate(input.taskSlug)],
                 [paths.knownIssuesPath, renderKnownIssuesTemplate(input.taskSlug)],
-                [paths.reviewReportPath, renderReviewReportTemplate(input.taskSlug)],
+                [path.posix.join(paths.handoffDir, "coder-completion.md"), renderCoderCompletionTemplate(input.taskSlug)],
+                [paths.testReportPath, renderTestReportTemplate(input.taskSlug)],
                 [paths.docsSyncReportPath, renderDocsSyncReportTemplate(input.taskSlug)],
                 [paths.finalAcceptancePath, renderFinalAcceptanceTemplate(input.taskSlug)],
                 ...Object.values(paths.messageRoutePaths).map((messagePath) => [