vibe-coding-master 0.9.7 → 0.9.8

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

@@ -22,6 +22,9 @@ const ARTIFACT_PATH_KEYS = [
     ["workflow-progress", "workflowProgressPath"],
     ["final-acceptance", "finalAcceptancePath"]
 ];
+// Gate requests whose report VCM still waits for; any other status means the
+// run already ended and a report would be ignored.
+const OPEN_GATE_REQUEST_STATUSES = new Set(["requested", "running"]);
 const ROLE_COMMAND_PLACEHOLDER_PATTERN = /(^|\n)\s*(TBD|status:\s*draft)\s*(\n|$)/i;
 const DEFAULT_MESSAGE_ROUTES = [
     ["project-manager", "architect"],
@@ -331,6 +334,11 @@ async function validateDynamicArtifact(fs, input, content, workflowControlServic
                 `Report path must match the assigned request path ${request.reportPath ?? "<missing>"}.`
             ]);
         }
+        if (request.status && !OPEN_GATE_REQUEST_STATUSES.has(request.status)) {
+            throw artifactRejected(input.kind, [
+                `Gate request ${request.requestId ?? requestId} is already ${request.status}, so VCM will not use this report. Stop this review; VCM sends a new request if the gate runs again.`
+            ]);
+        }
         assertManagedArtifactContent(input.kind, content, artifactPath, input.mode, {
             expectedGate: request.gate,
             expectedRequestId: request.requestId ?? requestId

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

@@ -1029,7 +1029,11 @@ function normalizeDiagnosticString(value, maxLength) {
 function stringOrUndefined(value) {
     return typeof value === "string" ? value : undefined;
 }
+// VCM submits its prompts as a terminal paste, and newer Claude Code versions
+// report a pasted prompt wrapped in <pasted_content id="..."> tags, so the
+// wrapper is removed before looking for VCM's marker.
+const PASTED_CONTENT_OPEN_TAG = /^<pasted_content\b[^>]*>/i;
 function isDirectUserPrompt(prompt) {
-    const normalized = prompt.trim();
+    const normalized = prompt.trim().replace(PASTED_CONTENT_OPEN_TAG, "").trim();
     return normalized.length > 0 && !/^\[VCM(?:\s|\])/i.test(normalized);
 }

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

@@ -15,6 +15,9 @@ const GATE_REVIEW_VERSION = 1;
 const HARNESS_COMMIT_PREFIX = "[VCM Harness] ";
 const REVIEWER_ROLE = "reviewer";
 const DEFAULT_REPORT_POLL_INTERVAL_MS = 1000;
+// A Reviewer that just submitted a report usually ends its turn within
+// seconds; a missed Stop hook leaves the activity stale, so the wait is capped.
+const DEFAULT_REVIEWER_IDLE_WAIT_MS = 5 * 60 * 1000;
 const activeRuns = new Map();
 const gateStateLocks = new Map();
 const SOURCE_ARTIFACTS = {
@@ -54,6 +57,7 @@ const VALID_CODE_DIFF_FINDING_SCOPES = new Set(CODE_DIFF_FINDING_SCOPES);
 export function createGateReviewService(deps) {
     const now = deps.now ?? (() => new Date().toISOString());
     const reportPollIntervalMs = deps.reportPollIntervalMs ?? DEFAULT_REPORT_POLL_INTERVAL_MS;
+    const reviewerIdleWaitMs = deps.reviewerIdleWaitMs ?? DEFAULT_REVIEWER_IDLE_WAIT_MS;
     async function getContext(repoRoot, taskSlug) {
         const projectConfig = await deps.projectService.loadConfig(repoRoot);
         const task = await deps.taskService.loadTask(repoRoot, taskSlug);
@@ -529,6 +533,7 @@ export function createGateReviewService(deps) {
                 });
             }
             const session = await ensureReviewerSession(context);
+            await waitForReviewerIdle(context, controller.signal);
             throwIfGateRunCancelled(controller.signal);
             await submitTerminalInput(deps.runtime, session.id, prompt);
             await deps.sessionService.markRoleActivityRunning(context.repoRoot, context.taskSlug, REVIEWER_ROLE, session.id);
@@ -647,6 +652,20 @@ export function createGateReviewService(deps) {
             return next.gates[gate];
         });
     }
+    // Claude Code queues a prompt typed into a running turn but sends its
+    // UserPromptSubmit hook at once, so the running turn's Stop would look like
+    // the end of this request's turn and fail it. Wait for that turn to end.
+    async function waitForReviewerIdle(context, signal) {
+        const deadline = Date.now() + reviewerIdleWaitMs;
+        while (true) {
+            throwIfGateRunCancelled(signal);
+            const session = await deps.sessionService.getRoleSession(context.repoRoot, context.taskSlug, REVIEWER_ROLE);
+            if (session?.activityStatus !== "running" || Date.now() >= deadline) {
+                return;
+            }
+            await delay(reportPollIntervalMs);
+        }
+    }
     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)) {

package/dist/backend/services/workflow-control-service.js

@@ -833,9 +833,12 @@ async function allowedArchitectFix(fs, input, state, segment, source, resumedFro
             return [`${source}/architect`];
         }
         // A new fix needs validation; any other post-validation Architect change
-        // is reviewed by an incremental code-diff Gate.
-        if (artifactFresh && artifact.complete)
+        // is reviewed by an incremental code-diff Gate. Architect also rewrites its
+        // artifact for a docs correction, so once a code-diff Gate has reviewed
+        // this dispatch, that Gate's result routes the flow instead.
+        if (artifactFresh && artifact.complete && !(await codeDiffReviewedDispatch(fs, input, state, source))) {
             return [`${source}/tester`];
+        }
         return allowedPostImplementationArchitect(fs, input, state, source, acceptanceFlow);
     }
     return allowedAfterTester(fs, input, state, source, source, {
@@ -843,6 +846,18 @@ async function allowedArchitectFix(fs, input, state, segment, source, resumedFro
         acceptanceFlow
     });
 }
+// True when a code-diff Gate finished, or was skipped or overridden, after the
+// active Architect dispatch was confirmed.
+async function codeDiffReviewedDispatch(fs, input, state, flow) {
+    if (!matchingEvidenceBaseline(state, flow, "architect"))
+        return false;
+    const codeDiff = await gateState(fs, input, "code-diff");
+    // A disabled Gate reviews nothing, so it never stands in for Tester.
+    if (!codeDiff || codeDiff.status === "disabled")
+        return false;
+    return (codeDiff.status === "completed" && freshGateDecision(state, flow, "architect", "code-diff", codeDiff) !== undefined)
+        || gatePassedForDispatch(state, flow, "architect", "code-diff", codeDiff);
+}
 async function allowedPostImplementationArchitect(fs, input, state, flow, acceptanceFlow) {
     if (flow === "code-change") {
         const planningTransition = await allowedFreshArchitecturePlan(fs, input, state, matchingEvidenceBaseline(state, "code-change", "architect")?.confirmedAt);

package/package.json

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