@gethmy/agent 1.14.3 → 1.14.4

package/dist/cli.js

@@ -7088,6 +7088,9 @@ function buildStagePreamble(stage) {
     }
   }
   lines.push("Do only this stage's work. When the stage's handoff is met, end your session — advancement to the next stage is handled by the board.");
+  if (normalizeGateSpec(stage.gate)?.kind === "review_passed") {
+    lines.push("", "**This stage's gate is `review_passed` — your deliverable is a review verdict, not a prose handoff.** Do NOT end the session until you have actually reviewed the change and emitted EXACTLY one JSON block as the LAST thing you output (nothing after it):", "```json", REVIEW_VERDICT_SCHEMA, "```", "Decision rules:", REVIEW_DECISION_RULES, "The board reads this verdict to decide advancement: `approved` advances the card, `rejected` sends it back. A missing or unparseable verdict blocks the card. Do NOT modify any code — this is a read-only review.");
+  }
   return lines.filter(Boolean).join(`
 `);
 }
@@ -7132,6 +7135,7 @@ class Worker {
   lastDrainedSeq = 0;
   runCostCents = 0;
   runTurns = 0;
+  lastRunText = "";
   constructor(id, config, client, agentId, onDone, workspaceId, projectId, stateStore, onCardCompleted, onApiError) {
     this.config = config;
     this.client = client;
@@ -7200,6 +7204,7 @@ class Worker {
     this.completionStarted = false;
     this.runCostCents = 0;
     this.runTurns = 0;
+    this.lastRunText = "";
     this.cliSessionId = null;
     this.lastDrainedSeq = 0;
     this.cardId = card.id;
@@ -7652,6 +7657,10 @@ class Worker {
         log.info(this.tag, `Stage "${stage.name}" gate "${gate.kind}" is advisory — skipping enforcement`);
         return null;
       }
+      const review = gate.kind === "review_passed" ? parseReviewOutput(this.lastRunText) : undefined;
+      if (review) {
+        log.info(this.tag, `Review-gated stage "${stage.name}" verdict: ${review.verdict} (${review.findings.length} finding(s))`);
+      }
       const registry = buildGateCollectorRegistry({
         build: {
           worktreePath,
@@ -7662,7 +7671,8 @@ class Worker {
           worktreePath,
           artifactType: stage.artifact_type
         },
-        checklist: { subtasks, cardDone: card.done ?? false }
+        checklist: { subtasks, cardDone: card.done ?? false },
+        ...review ? { review } : {}
       });
       const context = {
         cardId: card.id,
@@ -8000,6 +8010,9 @@ class Worker {
           });
         }
       }
+      parser.on("text", (content) => {
+        this.lastRunText += content;
+      });
       parser.on("parse_error", (msg) => {
         log.debug(this.tag, `Stream parse error (non-fatal): ${msg}`);
         runLog?.stream.write(`
@@ -8093,6 +8106,10 @@ class Worker {
     try {
       for await (const ev of stream) {
         this.progressTracker?.ingest(ev);
+        if (ev.kind === "assistant_text") {
+          this.lastRunText += `${ev.payload.text}
+`;
+        }
         if (ev.source === "agent")
           this.cliRunner?.record(ev);
         if (ev.kind === "error") {
@@ -8180,6 +8197,8 @@ var init_worker = __esm(() => {
   init_process_group();
   init_progress_tracker();
   init_prompt();
+  init_review_completion();
+  init_review_knowledge();
   init_run_log();
   init_sdk_agent_runner();
   init_stage_advance();

package/dist/index.js

@@ -7087,6 +7087,9 @@ function buildStagePreamble(stage) {
     }
   }
   lines.push("Do only this stage's work. When the stage's handoff is met, end your session — advancement to the next stage is handled by the board.");
+  if (normalizeGateSpec(stage.gate)?.kind === "review_passed") {
+    lines.push("", "**This stage's gate is `review_passed` — your deliverable is a review verdict, not a prose handoff.** Do NOT end the session until you have actually reviewed the change and emitted EXACTLY one JSON block as the LAST thing you output (nothing after it):", "```json", REVIEW_VERDICT_SCHEMA, "```", "Decision rules:", REVIEW_DECISION_RULES, "The board reads this verdict to decide advancement: `approved` advances the card, `rejected` sends it back. A missing or unparseable verdict blocks the card. Do NOT modify any code — this is a read-only review.");
+  }
   return lines.filter(Boolean).join(`
 `);
 }
@@ -7131,6 +7134,7 @@ class Worker {
   lastDrainedSeq = 0;
   runCostCents = 0;
   runTurns = 0;
+  lastRunText = "";
   constructor(id, config, client, agentId, onDone, workspaceId, projectId, stateStore, onCardCompleted, onApiError) {
     this.config = config;
     this.client = client;
@@ -7199,6 +7203,7 @@ class Worker {
     this.completionStarted = false;
     this.runCostCents = 0;
     this.runTurns = 0;
+    this.lastRunText = "";
     this.cliSessionId = null;
     this.lastDrainedSeq = 0;
     this.cardId = card.id;
@@ -7651,6 +7656,10 @@ class Worker {
         log.info(this.tag, `Stage "${stage.name}" gate "${gate.kind}" is advisory — skipping enforcement`);
         return null;
       }
+      const review = gate.kind === "review_passed" ? parseReviewOutput(this.lastRunText) : undefined;
+      if (review) {
+        log.info(this.tag, `Review-gated stage "${stage.name}" verdict: ${review.verdict} (${review.findings.length} finding(s))`);
+      }
       const registry = buildGateCollectorRegistry({
         build: {
           worktreePath,
@@ -7661,7 +7670,8 @@ class Worker {
           worktreePath,
           artifactType: stage.artifact_type
         },
-        checklist: { subtasks, cardDone: card.done ?? false }
+        checklist: { subtasks, cardDone: card.done ?? false },
+        ...review ? { review } : {}
       });
       const context = {
         cardId: card.id,
@@ -7999,6 +8009,9 @@ class Worker {
           });
         }
       }
+      parser.on("text", (content) => {
+        this.lastRunText += content;
+      });
       parser.on("parse_error", (msg) => {
         log.debug(this.tag, `Stream parse error (non-fatal): ${msg}`);
         runLog?.stream.write(`
@@ -8092,6 +8105,10 @@ class Worker {
     try {
       for await (const ev of stream) {
         this.progressTracker?.ingest(ev);
+        if (ev.kind === "assistant_text") {
+          this.lastRunText += `${ev.payload.text}
+`;
+        }
         if (ev.source === "agent")
           this.cliRunner?.record(ev);
         if (ev.kind === "error") {
@@ -8179,6 +8196,8 @@ var init_worker = __esm(() => {
   init_process_group();
   init_progress_tracker();
   init_prompt();
+  init_review_completion();
+  init_review_knowledge();
   init_run_log();
   init_sdk_agent_runner();
   init_stage_advance();

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@gethmy/agent",
-  "version": "1.14.3",
+  "version": "1.14.4",
   "description": "Push-based agent daemon for Harmony — watches board assignments and spawns Claude CLI workers",
   "type": "module",
   "main": "dist/index.js",