vibe-coding-master 0.8.6 → 0.8.7

package/README.md

@@ -595,11 +595,17 @@ with the exact assigned `--path` and Assignment ID. Reports are isolated under
 The reported commit must modify the target document within the assignment's
 original commit range; unrelated later commits do not invalidate it. Existing
 correct documentation can be reported as `unchanged` without a new commit.
+For `synced`, the entire `Commit` section is one bare lowercase hexadecimal
+hash (7-40 characters), without Markdown or explanatory text. The template,
+artifact submission, and assignment completion use the same format contract.
 
 Assignments run one at a time. A failed assignment does not block later items.
 After the remaining items finish, unresolved failures are reported in Harness
 Studio with their causes and individual Retry buttons. Retry preserves earlier
 committed work and the original baseline, including after a backend restart.
+An old assignment whose baseline was already overwritten is not automatically
+rebased; verify the current document and use `unchanged` when it already preserves
+the assigned content. Errors identify the reported commit and recorded range.
 Migration content and review snapshots remain in the task-local run directory;
 failures do not automatically restore removed memory. Turning Auto Memory off
 does not discard or cancel already-applied document migrations.

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

@@ -1,6 +1,7 @@
 import { createHash } from "node:crypto";
 import path from "node:path";
-import { checkMarkdownArtifact, readArtifactSectionValue } from "../../shared/validation/artifact-check.js";
+import { checkMarkdownArtifact, readArtifactSectionContent, readArtifactSectionValue } from "../../shared/validation/artifact-check.js";
+import { DOCS_UPDATE_COMMIT_RULE } from "../../shared/validation/artifact-contract.js";
 import { resolveRepoPath } from "../adapters/filesystem.js";
 import { VcmError } from "../errors.js";
 import { submitTerminalInput } from "../runtime/terminal-submit.js";
@@ -780,24 +781,34 @@ export function createAutoMemoryService(deps) {
         return currentHead;
     }
     async function validateDurableDocCommit(taskRepoRoot, assignment, reportContent, currentHead) {
-        if (!assignment.baseCommit || currentHead === assignment.baseCommit) {
-            throw new Error("documentation changes were not committed");
+        const reportedCommit = readArtifactSectionContent(reportContent, "Commit");
+        let commit;
+        try {
+            commit = (await deps.git.getCommitInfo(taskRepoRoot, reportedCommit)).sha;
         }
-        const changedPaths = await deps.git.getChangedPaths(taskRepoRoot, assignment.baseCommit, currentHead);
-        if (!changedPaths.includes(assignment.targetPath)) {
-            throw new Error(`documentation commit does not include ${assignment.targetPath}`);
+        catch (error) {
+            const detail = error instanceof VcmError && error.hint ? `${errorMessage(error)} ${error.hint}` : errorMessage(error);
+            throw new Error(`Commit ${reportedCommit} could not be resolved to a unique Git commit: ${detail}`);
+        }
+        const existingContentHint = `If ${assignment.targetPath} already preserves the assigned content, report unchanged with verification evidence; do not create a redundant commit.`;
+        if (!assignment.baseCommit) {
+            throw new Error(`Assignment ${assignment.id} has no recorded commit baseline; cannot validate Commit ${commit}. ${existingContentHint}`);
+        }
+        if (currentHead === assignment.baseCommit) {
+            throw new Error(`No commits follow assignment baseline ${assignment.baseCommit} (HEAD is the same commit); received Commit ${commit}. The document may already have been committed before this baseline. ${existingContentHint}`);
         }
-        const reportedCommit = readArtifactSectionValue(reportContent, "Commit")?.trim();
         const commits = await deps.git.getCommitList(taskRepoRoot, `${assignment.baseCommit}..${currentHead}`);
-        const matches = reportedCommit ? commits.filter((commit) => commit.sha.startsWith(reportedCommit)) : [];
-        if (matches.length !== 1) {
-            throw new Error(`Commit must uniquely identify a documentation commit in ${assignment.baseCommit}..${currentHead}; received ${reportedCommit || "empty"}`);
+        if (!commits.some((candidate) => candidate.sha === commit)) {
+            throw new Error(`Commit ${commit} exists but is outside assignment range ${assignment.baseCommit}..${currentHead}. ${existingContentHint}`);
         }
-        const commit = matches[0].sha;
         const commitPaths = await deps.git.getChangedPaths(taskRepoRoot, `${commit}^`, commit);
         if (!commitPaths.includes(assignment.targetPath)) {
             throw new Error(`reported commit ${commit} does not modify ${assignment.targetPath}`);
         }
+        const changedPaths = await deps.git.getChangedPaths(taskRepoRoot, assignment.baseCommit, currentHead);
+        if (!changedPaths.includes(assignment.targetPath)) {
+            throw new Error(`Commit ${commit} modified ${assignment.targetPath}, but no net change remains in ${assignment.baseCommit}..${currentHead}. Verify the current document before reporting completion.`);
+        }
         return commit;
     }
     async function hashOptionalFile(taskRepoRoot, relativePath) {
@@ -1498,6 +1509,7 @@ function buildDurableDocAssignmentPrompt(taskRepoRoot, assignment) {
         `Assigned report path: ${assignment.reportPath}`,
         `Submit with .ai/tools/vcm-artifact docs-update-report --file <candidate> --path ${assignment.reportPath} --mode final`,
         "Set Assignment ID to the exact ID above.",
+        DOCS_UPDATE_COMMIT_RULE,
         "For synced, record the commit that changed the target document; it need not be HEAD. If the document already preserves the content, use unchanged with evidence; do not create a redundant commit.",
         ...(assignment.error ? [`Previous failure: ${assignment.error}`] : []),
         "End the turn after VCM accepts the report."

package/dist/backend/templates/handoff.js

@@ -1,4 +1,4 @@
-import { ARCHITECT_DEBUG_DISPOSITIONS, ARCHITECT_DEBUG_STATUSES, ARCHITECTURE_BRIEF_STATUSES, ARCHITECTURE_DIAGNOSIS_DISPOSITIONS, ARCHITECTURE_EVIDENCE_STATUSES, ARCHITECTURE_PLAN_RESULTS, CODER_COMPLETION_DECISIONS, DOCS_REPORT_DECISIONS, DOCS_SYNC_CORRECTION_OWNERS, FINAL_ACCEPTANCE_DECISIONS, L3_ACTIONS, L3_REQUIRED_VALUES, PLANNING_PROGRESS_STATUSES, STRICT_NONE_VALUE, TEST_INFRASTRUCTURE_STATUSES, TEST_RESULTS, renderArtifactOptions } from "../../shared/validation/artifact-contract.js";
+import { ARCHITECT_DEBUG_DISPOSITIONS, ARCHITECT_DEBUG_STATUSES, ARCHITECTURE_BRIEF_STATUSES, ARCHITECTURE_DIAGNOSIS_DISPOSITIONS, ARCHITECTURE_EVIDENCE_STATUSES, ARCHITECTURE_PLAN_RESULTS, CODER_COMPLETION_DECISIONS, DOCS_REPORT_DECISIONS, DOCS_UPDATE_COMMIT_RULE, DOCS_SYNC_CORRECTION_OWNERS, FINAL_ACCEPTANCE_DECISIONS, L3_ACTIONS, L3_REQUIRED_VALUES, PLANNING_PROGRESS_STATUSES, STRICT_NONE_VALUE, TEST_INFRASTRUCTURE_STATUSES, TEST_RESULTS, renderArtifactOptions } from "../../shared/validation/artifact-contract.js";
 const CURRENT_HANDOFF_NOTICE = "<!-- VCM current handoff: replace this file with one complete, self-contained snapshot of the current result. Restate all still-relevant evidence; do not refer to a prior revision, route message, Session, or transcript as evidence. -->";
 export function renderArchitectureBriefTemplate(taskSlug) {
     return `# Architecture Brief: ${taskSlug}
@@ -534,6 +534,8 @@ export function renderDocsUpdateReportTemplate(taskSlug, assignmentId = "docs-on
 
 ${CURRENT_HANDOFF_NOTICE}
 
+${DOCS_UPDATE_COMMIT_RULE}
+
 ## Summary
 
 TBD

package/dist/shared/validation/artifact-check.js

@@ -1,4 +1,4 @@
-import { ARCHITECT_DEBUG_DISPOSITIONS, ARCHITECT_DEBUG_STATUSES, ARCHITECTURE_BRIEF_STATUSES, ARCHITECTURE_DIAGNOSIS_DISPOSITIONS, ARCHITECTURE_EVIDENCE_STATUSES, ARCHITECTURE_PLAN_RESULTS, CODER_COMPLETION_DECISIONS, DOCS_REPORT_DECISIONS, DOCS_SYNC_CORRECTION_OWNERS, FINAL_ACCEPTANCE_DECISIONS, L3_ACTIONS, L3_REQUIRED_VALUES, PLANNING_PROGRESS_STATUSES, STRICT_NONE_VALUE, TEST_INFRASTRUCTURE_STATUSES, TEST_RESULTS } from "./artifact-contract.js";
+import { ARCHITECT_DEBUG_DISPOSITIONS, ARCHITECT_DEBUG_STATUSES, ARCHITECTURE_BRIEF_STATUSES, ARCHITECTURE_DIAGNOSIS_DISPOSITIONS, ARCHITECTURE_EVIDENCE_STATUSES, ARCHITECTURE_PLAN_RESULTS, CODER_COMPLETION_DECISIONS, DOCS_REPORT_DECISIONS, DOCS_UPDATE_COMMIT_PATTERN, DOCS_UPDATE_COMMIT_RULE, DOCS_SYNC_CORRECTION_OWNERS, FINAL_ACCEPTANCE_DECISIONS, L3_ACTIONS, L3_REQUIRED_VALUES, PLANNING_PROGRESS_STATUSES, STRICT_NONE_VALUE, TEST_INFRASTRUCTURE_STATUSES, TEST_RESULTS } from "./artifact-contract.js";
 import { getArtifactDefinition } from "./artifact-registry.js";
 const PLACEHOLDER_PATTERN = /(^|\n)\s*(TBD|Not run yet\.?|status:\s*draft)\s*(\n|$)/i;
 export function checkMarkdownArtifact(kind, artifactPath, content, options = {}) {
@@ -252,7 +252,13 @@ function validateArtifactFields(kind, content, mode) {
         return invalidFields;
     }
     if (kind === "docs-update-report") {
-        return validateDecision(content, DOCS_REPORT_DECISIONS);
+        const invalidFields = validateDecision(content, DOCS_REPORT_DECISIONS);
+        const decision = readArtifactSectionContent(content, "Decision")?.trim().toLowerCase();
+        const commit = readArtifactSectionContent(content, "Commit");
+        if (mode === "final" && decision === "synced" && !DOCS_UPDATE_COMMIT_PATTERN.test(commit ?? "")) {
+            invalidFields.push(`${DOCS_UPDATE_COMMIT_RULE} Received Commit: ${JSON.stringify(commit ?? "")}.`);
+        }
+        return invalidFields;
     }
     if (kind === "docs-sync-report") {
         const invalidFields = validateDecision(content, DOCS_REPORT_DECISIONS);

package/dist/shared/validation/artifact-contract.js

@@ -30,6 +30,8 @@ export const TEST_INFRASTRUCTURE_STATUSES = [
 export const L3_REQUIRED_VALUES = ["yes", "no"];
 export const L3_ACTIONS = ["run-existing", "updated", "added"];
 export const DOCS_REPORT_DECISIONS = ["synced", "unchanged", "blocked"];
+export const DOCS_UPDATE_COMMIT_PATTERN = /^[0-9a-f]{7,40}$/;
+export const DOCS_UPDATE_COMMIT_RULE = "When Decision is synced, Commit must contain only one bare lowercase hexadecimal commit hash (7-40 characters, for example abc1234) on a single line. No backticks, bullets, links, subjects, or extra lines; put explanations in Evidence Reviewed. An unchanged result does not require a new commit.";
 export const DOCS_SYNC_CORRECTION_OWNERS = ["none", "architect", "coder", "tester"];
 export const FINAL_ACCEPTANCE_DECISIONS = [
     "accepted",

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "vibe-coding-master",
-  "version": "0.8.6",
+  "version": "0.8.7",
   "description": "Local GUI session cockpit for Claude Code role sessions.",
   "type": "module",
   "files": [