vibe-coding-master 0.7.20 → 0.7.22

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"
 ];

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

@@ -39,6 +39,11 @@ const REQUIRED_HEADINGS = {
         "Evidence Reviewed",
         "Tests Added Or Updated",
         "Coverage Mapping",
+        "L3 Coverage",
+        "Trigger Assessment",
+        "Affected End-To-End Flows",
+        "L3 Commands And Evidence",
+        "Not-Required Evidence",
         "Commands Run Or Checked",
         "Validation Results",
         "Failed Expectations",
@@ -139,6 +144,28 @@ function validateArtifactFields(kind, content) {
         const invalidFields = result === "pass" || result === "fail"
             ? []
             : ["Test Result must be pass or fail."];
+        const l3Required = /^\s*L3 Required\s*:\s*(\S+)\s*$/im.exec(content)?.[1]?.toLowerCase();
+        if (l3Required !== "yes" && l3Required !== "no") {
+            invalidFields.push("L3 Required must be yes or no.");
+        }
+        const l3TriggerAssessment = readArtifactSectionValue(content, "Trigger Assessment");
+        const l3AffectedFlows = readArtifactSectionContent(content, "Affected End-To-End Flows");
+        const l3Commands = readArtifactSectionValue(content, "L3 Commands And Evidence");
+        const l3NotRequiredEvidence = readArtifactSectionValue(content, "Not-Required Evidence");
+        if (l3Required === "yes") {
+            if (!hasSubstantiveSectionValue(l3TriggerAssessment)) {
+                invalidFields.push("Trigger Assessment is required when L3 Required is yes.");
+            }
+            if (!hasCompleteL3FlowMapping(l3AffectedFlows)) {
+                invalidFields.push("Affected End-To-End Flows are required when L3 Required is yes.");
+            }
+            if (!hasSubstantiveSectionValue(l3Commands)) {
+                invalidFields.push("L3 Commands And Evidence are required when L3 Required is yes.");
+            }
+        }
+        if (l3Required === "no" && !hasSubstantiveSectionValue(l3NotRequiredEvidence)) {
+            invalidFields.push("Not-Required Evidence is required when L3 Required is no.");
+        }
         const coverageGaps = readArtifactSectionValue(content, "Coverage Gaps");
         const blockingIssues = readArtifactSectionValue(content, "Blocking Validation Issues");
         const userApproval = readArtifactSectionValue(content, "User Approval Evidence");
@@ -187,6 +214,25 @@ function validateArtifactFields(kind, content) {
     }
     return [];
 }
+function hasSubstantiveSectionValue(value) {
+    return Boolean(value && !/^(none|tbd)\.?$/i.test(value.trim()));
+}
+function hasCompleteL3FlowMapping(value) {
+    if (!value) {
+        return false;
+    }
+    const allowedActions = new Set(["run-existing", "updated", "added"]);
+    return value
+        .split(/\r?\n/)
+        .map((line) => line.trim())
+        .filter((line) => line.startsWith("|") && line.endsWith("|"))
+        .some((line) => {
+        const cells = line.slice(1, -1).split("|").map((cell) => cell.trim());
+        return cells.length === 8
+            && allowedActions.has(cells[6]?.toLowerCase() ?? "")
+            && cells.every((cell) => Boolean(cell) && !/^(none|tbd)\.?$/i.test(cell));
+    });
+}
 function validateDecision(content, allowed) {
     const decision = readArtifactSectionValue(content, "Decision")?.toLowerCase();
     return decision && allowed.includes(decision)
@@ -194,6 +240,12 @@ function validateDecision(content, allowed) {
         : [`Decision must be one of: ${allowed.join(", ")}.`];
 }
 export function readArtifactSectionValue(content, heading) {
+    return readArtifactSectionContent(content, heading)
+        ?.split(/\r?\n/)
+        .map((line) => line.trim())
+        .find(Boolean);
+}
+function readArtifactSectionContent(content, heading) {
     const match = new RegExp(`^#{1,6}\\s+${escapeRegExp(heading)}\\s*$`, "im").exec(content);
     if (!match || match.index === undefined) {
         return undefined;
@@ -203,7 +255,7 @@ export function readArtifactSectionValue(content, heading) {
     const section = nextHeading?.index === undefined
         ? afterHeading
         : afterHeading.slice(0, nextHeading.index);
-    return section.split(/\r?\n/).map((line) => line.trim()).find(Boolean);
+    return section.trim();
 }
 function hasHeading(content, heading) {
     const pattern = new RegExp(`^#{1,6}\\s+${escapeRegExp(heading)}\\s*$`, "im");