vibe-coding-master 0.7.46 → 0.7.48

package/dist/backend/templates/harness/architect-evidence-worker-agent.js

@@ -0,0 +1,44 @@
+export function renderArchitectEvidenceWorkerHarnessRules() {
+    return `
+## VCM Architect Evidence Worker Rules
+
+You are \`vcm-architect-evidence-worker\`, a foreground evidence-collection subagent invoked by Architect.
+
+### Scope
+
+- Investigate only the modules, files, documents, and questions assigned by Architect.
+- Read the assigned implementation and supporting project artifacts in full where required by the assignment.
+- Record repository facts with exact repo-relative file paths and symbol or section names.
+- Separate verified facts, contradictions, and unresolved items.
+- Do not design architecture, recommend implementation, decide scope, change contracts, edit project files, run validation, or communicate with project-manager or the user.
+- Do not claim a complete semantic relationship from text matches. Architect owns LSP navigation and verification of definitions, references, callers, callees, implementations, public surfaces, and decision-bearing behavior paths.
+
+### Evidence Output
+
+- Write only the assigned report under \`.ai/vcm/architect-workers/evidence/\`.
+- Keep the report concise and factual. Do not copy long source blocks when a path, symbol, and short fact are sufficient.
+- Use this structure:
+
+\`\`\`md
+# Architect Evidence Worker: <worker-id>
+
+## Assigned Scope
+
+## Files And Documents Read
+
+## Verified Facts
+
+| Fact | Evidence |
+| --- | --- |
+
+## Observed Data And Lifecycle Paths
+
+## Contradictions
+
+## Unresolved Items
+\`\`\`
+
+- Return the report path and a one-paragraph completion summary to Architect.
+- Do not invoke another subagent.
+`;
+}

package/dist/backend/templates/harness/architect-scaffold-worker-agent.js

@@ -8,6 +8,7 @@ You are \`vcm-architect-scaffold-worker\`, a foreground subagent invoked by Arch
 
 - Execute only the scaffold work assigned by Architect from the current \`.ai/vcm/handoffs/architecture-plan.md\`.
 - Create the declared files, callable surfaces, contract comments, placeholder bodies, configuration changes, and one \`VCM:CODE <ID>\` marker for every Scaffold Manifest item.
+- Perform only mechanical text or configuration changes whose target and required result are explicitly fixed by the plan. Do not author architecture rationale, evidence, decisions, or durable architecture documentation.
 - Do not change architecture decisions, accepted scope, ledger items, public contracts, or implementation boundaries.
 - Do not implement business logic beyond the minimum compilable scaffold.
 - Follow \`docs/CODING_STANDARDS.md\` for every code or test edit.
@@ -19,5 +20,6 @@ You are \`vcm-architect-scaffold-worker\`, a foreground subagent invoked by Arch
 - Return the commit hash, changed files, ledger result, and exact check results to Architect.
 - If the assigned scaffold cannot be completed, return the concrete failure evidence without changing the plan.
 
-Architect reviews the worker's commit and remains responsible for the final scaffold, plan, and evidence.`;
+Architect reviews the worker's commit and remains responsible for the final scaffold, plan, and evidence.
+Do not invoke another subagent.`;
 }

package/dist/backend/templates/harness/architect-validation-worker-agent.js

@@ -0,0 +1,36 @@
+export function renderArchitectValidationWorkerHarnessRules() {
+    return `
+## VCM Architect Validation Worker Rules
+
+You are \`vcm-architect-validation-worker\`, a foreground command-execution subagent invoked by Architect.
+
+### Scope
+
+- Run only the exact commands, working directory, and validation targets assigned by Architect.
+- Do not choose validation scope, design or modify tests, edit production code, change configuration, diagnose architecture, repair failures, or decide whether the task passes.
+- Preserve each command's real exit code and output. Do not add wrappers, pipelines, retries, skips, or fallback commands unless Architect assigned them.
+- Use \`.ai/tools/run-long-check\` and \`.ai/tools/watch-job\` when the assigned command requires supervised long-running execution. Remain in the foreground until every assigned command reaches a terminal result.
+
+### Validation Output
+
+- Write only the assigned report under \`.ai/vcm/architect-workers/validation/\`.
+- Use this structure:
+
+\`\`\`md
+# Architect Validation Worker: <worker-id>
+
+## Assigned Commands
+
+| Command | Working Directory | Exit Code | Result | Evidence |
+| --- | --- | --- | --- | --- |
+
+## Failures
+
+## Unavailable Checks
+\`\`\`
+
+- Record exact failure or unavailability evidence without interpreting architecture or selecting follow-up work.
+- Return the report path and a one-paragraph completion summary to Architect.
+- Do not invoke another subagent.
+`;
+}

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

@@ -1,4 +1,15 @@
 import { renderRoleMemoryRules } from "./role-memory.js";
+import { GATE_ANALYSIS_FIELDS, GATE_ANALYSIS_HEADINGS } from "../../services/managed-artifact-validation.js";
+function renderGateAnalysisSections() {
+    return Object.keys(GATE_ANALYSIS_HEADINGS)
+        .map((gate) => [
+        `<!-- Include ${GATE_ANALYSIS_HEADINGS[gate]} only for ${gate} gate. -->`,
+        `## ${GATE_ANALYSIS_HEADINGS[gate]}`,
+        "",
+        ...GATE_ANALYSIS_FIELDS[gate].map((field) => `- ${field}:`)
+    ].join("\n"))
+        .join("\n\n");
+}
 export function renderReviewerAgentRules() {
     return `## Role
 
@@ -362,56 +373,7 @@ Summary: <one or two sentences>
 Use this findings structure:
 
 \`\`\`md
-<!-- Include Architecture Analysis only for architecture-plan gate. -->
-## Architecture Analysis
-
-- Evidence Read:
-- Architecture Brief Fit:
-- End-To-End Flow:
-- Scope Fit:
-- Code Reality:
-- Invalidated Assumptions:
-- Existing-Class Completeness:
-- Ownership:
-- Data Flow:
-- Lifecycle:
-- Invariants:
-- Boundaries And Public Surface:
-- Failure Model:
-- Coder Readiness:
-
-<!-- Include Validation Analysis only for validation-adequacy gate. -->
-## Validation Analysis
-
-- Evidence Read:
-- Changed Behavior And Risk:
-- Coverage Mapping:
-- Baseline Coverage:
-- L2 Integration Coverage:
-- L3 Trigger Assessment:
-- L3 End-To-End Coverage:
-- Boundary And Failure Coverage:
-- Public Contract Coverage:
-- Test Integrity:
-- Test Infrastructure:
-- Skips And Gaps:
-- User Approval And Gap Disposition:
-- Validation Readiness:
-
-<!-- Include Code Diff Analysis only for code-diff gate. -->
-## Code Diff Analysis
-
-- Commit Range And Sources:
-- Evidence Read:
-- Changed Files And Symbols:
-- Changed Behavior:
-- Source Evidence Fit:
-- Callers And Public Surface:
-- State Lifecycle And Failure Paths:
-- Coding Standards:
-- Baseline Test Integrity:
-- Generated Context And Durable Docs:
-- Code Readiness:
+${renderGateAnalysisSections()}
 
 ## Findings
 
@@ -430,54 +392,7 @@ Use this findings structure:
 If there are no findings, write:
 
 \`\`\`md
-<!-- Include Architecture Analysis only for architecture-plan gate. -->
-## Architecture Analysis
-
-- Evidence Read:
-- Architecture Brief Fit:
-- End-To-End Flow:
-- Scope Fit:
-- Code Reality:
-- Ownership:
-- Data Flow:
-- Lifecycle:
-- Invariants:
-- Boundaries And Public Surface:
-- Failure Model:
-- Coder Readiness:
-
-<!-- Include Validation Analysis only for validation-adequacy gate. -->
-## Validation Analysis
-
-- Evidence Read:
-- Changed Behavior And Risk:
-- Coverage Mapping:
-- Baseline Coverage:
-- L2 Integration Coverage:
-- L3 Trigger Assessment:
-- L3 End-To-End Coverage:
-- Boundary And Failure Coverage:
-- Public Contract Coverage:
-- Test Integrity:
-- Test Infrastructure:
-- Skips And Gaps:
-- User Approval And Gap Disposition:
-- Validation Readiness:
-
-<!-- Include Code Diff Analysis only for code-diff gate. -->
-## Code Diff Analysis
-
-- Commit Range And Sources:
-- Evidence Read:
-- Changed Files And Symbols:
-- Changed Behavior:
-- Source Evidence Fit:
-- Callers And Public Surface:
-- State Lifecycle And Failure Paths:
-- Coding Standards:
-- Baseline Test Integrity:
-- Generated Context And Durable Docs:
-- Code Readiness:
+${renderGateAnalysisSections()}
 
 ## Findings
 

package/dist/backend/templates/harness/vcm-report-harness-issue-skill.js

@@ -38,7 +38,7 @@ Use this structure:
 - Evidence: <file paths, command names, logs, or repeated failure pattern>
 - Suspected harness area: <skill, role definition, tool, routing, validation, bootstrap, or managed instruction>
 - Impact: <who is affected and how>
-- Urgency: low | medium | high
+- Urgency: low|medium|high
 \`\`\`
 
 ## Constraints

package/dist/backend/templates/harness/vcm-route-message-skill.js

@@ -119,8 +119,8 @@ Do not:
 - wait for another role's answer
 - paste directly into another role terminal
 - use Claude Code Task/Subagent to replace VCM role routing. Coder workers and
-  Architect's foreground \`vcm-architect-scaffold-worker\` are internal role work,
-  not route-message delivery.
+  Architect's foreground evidence, scaffold, and validation workers are internal
+  role work, not route-message delivery.
 
 VCM scans pending route files after the Stop hook and delivers later replies in a new turn.
 

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

@@ -214,7 +214,62 @@ export const ARTIFACT_DEFINITIONS = [
         ]
     }
 ];
+export const DYNAMIC_ARTIFACT_DEFINITIONS = [
+    {
+        kind: "route-message",
+        storage: "dynamic",
+        owner: ["project-manager", "architect", "coder", "tester"],
+        requiredHeadings: [],
+        allowedModes: ["final"]
+    },
+    {
+        kind: "coder-worker-report",
+        storage: "dynamic",
+        owner: "coder",
+        requiredHeadings: [
+            "Assigned Scope",
+            "Item Dispositions",
+            "Files Changed",
+            "Tests Added Or Updated",
+            "L0/L1 Checks",
+            "Commit",
+            "Skipped Assigned Checks",
+            "Objective Failures"
+        ],
+        allowedModes: ["final"]
+    },
+    {
+        kind: "gate-review-report",
+        storage: "dynamic",
+        owner: "reviewer",
+        requiredHeadings: ["Findings"],
+        allowedModes: ["final"]
+    },
+    {
+        kind: "memory-proposal",
+        storage: "dynamic",
+        owner: ["project-manager", "architect", "coder", "tester", "reviewer"],
+        requiredHeadings: ["Add", "Update", "Remove"],
+        allowedModes: ["final"]
+    },
+    {
+        kind: "harness-feedback",
+        storage: "dynamic",
+        owner: ["project-manager", "architect", "coder", "tester", "reviewer"],
+        requiredHeadings: [],
+        allowedModes: ["final"]
+    }
+];
+export const MANAGED_ARTIFACT_DEFINITIONS = [
+    ...ARTIFACT_DEFINITIONS.map((definition) => ({
+        ...definition,
+        storage: "handoff",
+        allowedModes: definition.kind === "workflow-progress" ? ["final"] : ["draft", "final"]
+    })),
+    ...DYNAMIC_ARTIFACT_DEFINITIONS
+];
 const DEFINITION_BY_KIND = new Map(ARTIFACT_DEFINITIONS.map((definition) => [definition.kind, definition]));
+const MANAGED_DEFINITION_BY_KIND = new Map(MANAGED_ARTIFACT_DEFINITIONS.map((definition) => [definition.kind, definition]));
 export function getArtifactDefinition(kind) {
     const definition = DEFINITION_BY_KIND.get(kind);
     if (!definition) {
@@ -225,3 +280,16 @@ export function getArtifactDefinition(kind) {
 export function isArtifactKind(value) {
     return DEFINITION_BY_KIND.has(value);
 }
+export function getManagedArtifactDefinition(kind) {
+    const definition = MANAGED_DEFINITION_BY_KIND.get(kind);
+    if (!definition) {
+        throw new Error(`Unknown managed artifact kind: ${kind}`);
+    }
+    return definition;
+}
+export function isDynamicArtifactKind(value) {
+    return DYNAMIC_ARTIFACT_DEFINITIONS.some((definition) => definition.kind === value);
+}
+export function isManagedArtifactKind(value) {
+    return MANAGED_DEFINITION_BY_KIND.has(value);
+}

package/package.json

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

package/scripts/harness-tools/vcm-artifact

@@ -10,40 +10,11 @@ import urllib.error
 import urllib.parse
 import urllib.request
 
-KINDS = (
-    "architecture-brief",
-    "architecture-evidence",
-    "planning-progress",
-    "architecture-plan",
-    "known-issues",
-    "coder-completion",
-    "architect-debug",
-    "architecture-diagnosis",
-    "test-report",
-    "docs-update-report",
-    "docs-sync-report",
-    "workflow-progress",
-    "final-acceptance",
-    "route-message",
-    "coder-worker-report",
-    "gate-review-report",
-    "memory-proposal",
-    "harness-feedback",
-)
-DYNAMIC_KINDS = {
-    "route-message",
-    "coder-worker-report",
-    "gate-review-report",
-    "memory-proposal",
-    "harness-feedback",
-}
-
-
 def parse_args() -> argparse.Namespace:
     parser = argparse.ArgumentParser(
         description="Validate and atomically write a VCM-managed artifact."
     )
-    parser.add_argument("kind", choices=KINDS)
+    parser.add_argument("kind", help="VCM-managed artifact kind.")
     parser.add_argument("--file", required=True, help="Candidate Markdown file.")
     parser.add_argument("--mode", choices=("draft", "final"), default="final")
     parser.add_argument(
@@ -89,10 +60,6 @@ def error_text(raw: str) -> str:
 
 def main() -> int:
     args = parse_args()
-    if args.kind in DYNAMIC_KINDS and not args.path:
-        fail(f"--path is required for {args.kind}.")
-    if args.kind not in DYNAMIC_KINDS and args.path:
-        fail(f"--path is not allowed for fixed artifact kind {args.kind}.")
 
     candidate = Path(args.file).expanduser()
     if not candidate.is_file():

package/scripts/harness-tools/vcm-subagent-guard

@@ -7,8 +7,12 @@ import sys
 
 
 ALLOWED_SUBAGENTS = {
-    "architect": "vcm-architect-scaffold-worker",
-    "coder": "vcm-coder-worker",
+    "architect": frozenset({
+        "vcm-architect-evidence-worker",
+        "vcm-architect-scaffold-worker",
+        "vcm-architect-validation-worker",
+    }),
+    "coder": frozenset({"vcm-coder-worker"}),
 }
 
 
@@ -52,9 +56,10 @@ def main() -> None:
         deny(f"VCM role `{role}` may not invoke subagents.")
         return
 
-    if subagent_type != allowed:
+    if subagent_type not in allowed:
         requested = subagent_type if isinstance(subagent_type, str) and subagent_type else "<missing>"
-        deny(f"VCM role `{role}` may invoke only `{allowed}`, not `{requested}`.")
+        allowed_names = ", ".join(f"`{name}`" for name in sorted(allowed))
+        deny(f"VCM role `{role}` may invoke only {allowed_names}, not `{requested}`.")
 
 
 if __name__ == "__main__":