@gethmy/agent 1.14.1 → 1.14.3

package/dist/cli.js

@@ -2311,6 +2311,12 @@ class MergeMonitor {
         log.warn(TAG7, `Failed to update card: ${err instanceof Error ? err.message : err}`);
       }
     }
+    try {
+      await this.client.updateCard(card.id, { assignedAgentId: null });
+      log.info(TAG7, `Cleared agent assignment on #${card.short_id}`);
+    } catch (err) {
+      log.warn(TAG7, `Failed to clear agent assignment on #${card.short_id}: ${err instanceof Error ? err.message : err}`);
+    }
     const branchName = extractBranchFromDescription(card.description);
     if (branchName) {
       try {
@@ -4604,7 +4610,7 @@ class ProgressTracker {
         this.transitionTo("testing");
       }
     } else if (name === "mcp__harmony__harmony_end_agent_session") {
-      this.transitionTo("finishing");
+      this.enterFinishingAndStop();
     }
     if (name === "mcp__harmony__harmony_toggle_subtask" && this.subtaskMode) {
       const val = this.extractString(input, "completed");
@@ -4663,6 +4669,14 @@ class ProgressTracker {
     this.lastAction = "";
     this.scheduleUpdate(PHASES[newPhase].label);
   }
+  enterFinishingAndStop() {
+    if (PHASE_ORDER.finishing > PHASE_ORDER[this.phase]) {
+      this.runEventSink?.recordPhaseChanged("finishing", this.phase);
+      this.phase = "finishing";
+      this.progress = Math.max(this.progress, PHASES.finishing.min);
+    }
+    this.stop();
+  }
   incrementProgress() {
     const config = PHASES[this.phase];
     const range = config.max - config.min;
@@ -5837,6 +5851,11 @@ async function runTransition(client, card, plan, opts = {}) {
     await withRetry("endSession", shortId, () => client.endAgentSession(card.id, plan.endSession), attempts, backoffMs);
     log.info(TAG22, `#${shortId} session ended (${plan.endSession.status})`);
   }
+  if (plan.assignAgent !== undefined) {
+    const assignedAgentId = plan.assignAgent;
+    await withRetry("assignAgent", shortId, () => client.updateCard(card.id, { assignedAgentId }), attempts, backoffMs);
+    log.info(TAG22, assignedAgentId ? `#${shortId} assigned → agent ${assignedAgentId}` : `#${shortId} unassigned`);
+  }
   if (opts.store && opts.runId) {
     try {
       await opts.store.heartbeat(opts.runId);
@@ -6972,7 +6991,8 @@ async function advanceStageOnGate(card, stage, stageIndex, def, evaluation, deps
   });
   await runTransition(deps.client, card, {
     move: { columnName: toColumn },
-    addLabels: [{ name: AGENT_LABEL }]
+    addLabels: [{ name: AGENT_LABEL }],
+    ...isAgentRunnableOwner(next.stage.owner) ? { assignAgent: deps.agentId } : {}
   }, { store: deps.stateStore, runId: deps.runId });
   deps.stateStore.recordOutcome(card.id, "success").catch(() => {});
   log.info(TAG28, `#${card.short_id} advanced "${stage.name}" → "${next.stage.name}" (column "${toColumn}")`);
@@ -7001,7 +7021,11 @@ async function handleGateUnmet(card, stage, summary, deps) {
   try {
     await deps.client.addComment(card.id, `Stage gate unmet — re-running "${stage.name}". ${summary}.`, { commentType: "progress" });
   } catch {}
-  await runTransition(deps.client, card, { move: { columnName: toColumn }, addLabels: [{ name: AGENT_LABEL }] }, { store: deps.stateStore, runId: deps.runId });
+  await runTransition(deps.client, card, {
+    move: { columnName: toColumn },
+    addLabels: [{ name: AGENT_LABEL }],
+    ...isAgentRunnableOwner(stage.owner) ? { assignAgent: deps.agentId } : {}
+  }, { store: deps.stateStore, runId: deps.runId });
   log.info(TAG28, `#${card.short_id} gate unmet for "${stage.name}" — requeued to "${toColumn}" for re-run (attempt ${attempts}/${deps.maxAttempts})`);
   return { kind: "requeued_gate_unmet", toColumn };
 }
@@ -7101,6 +7125,7 @@ class Worker {
   timedOut = false;
   verificationFailed = false;
   held = false;
+  completionStarted = false;
   sessionId = null;
   runId = null;
   cliSessionId = null;
@@ -7172,6 +7197,7 @@ class Worker {
     this.timedOut = false;
     this.verificationFailed = false;
     this.held = false;
+    this.completionStarted = false;
     this.runCostCents = 0;
     this.runTurns = 0;
     this.cliSessionId = null;
@@ -7311,6 +7337,7 @@ class Worker {
         progressPercent: 75
       });
       this.state = "completing";
+      this.completionStarted = true;
       await this.recordPhase("completing");
       let stageGateEvaluation = null;
       const stageRun = stageCtx.kind === "run" ? stageCtx : null;
@@ -7662,6 +7689,7 @@ class Worker {
       return await advanceStageOnGate(card, stage, stageIndex, def, evaluation, {
         client: this.client,
         stateStore: this.stateStore,
+        agentId: this.agentId,
         maxAttempts: this.config.budget.maxAttemptsPerCard,
         fallbackColumn: this.config.pickupColumns[0] ?? "To Do",
         sink: this.cliRunner,
@@ -7764,7 +7792,7 @@ class Worker {
         sigtermTimeoutMs: CANCEL_SIGTERM_TIMEOUT2
       });
     }
-    if (this.cardId && !this.timedOut) {
+    if (this.cardId && !this.timedOut && !this.completionStarted) {
       try {
         const stats = this.lastSessionStats ?? this.progressTracker?.stats;
         await this.client.endAgentSession(this.cardId, {

package/dist/index.js

@@ -2310,6 +2310,12 @@ class MergeMonitor {
         log.warn(TAG7, `Failed to update card: ${err instanceof Error ? err.message : err}`);
       }
     }
+    try {
+      await this.client.updateCard(card.id, { assignedAgentId: null });
+      log.info(TAG7, `Cleared agent assignment on #${card.short_id}`);
+    } catch (err) {
+      log.warn(TAG7, `Failed to clear agent assignment on #${card.short_id}: ${err instanceof Error ? err.message : err}`);
+    }
     const branchName = extractBranchFromDescription(card.description);
     if (branchName) {
       try {
@@ -4603,7 +4609,7 @@ class ProgressTracker {
         this.transitionTo("testing");
       }
     } else if (name === "mcp__harmony__harmony_end_agent_session") {
-      this.transitionTo("finishing");
+      this.enterFinishingAndStop();
     }
     if (name === "mcp__harmony__harmony_toggle_subtask" && this.subtaskMode) {
       const val = this.extractString(input, "completed");
@@ -4662,6 +4668,14 @@ class ProgressTracker {
     this.lastAction = "";
     this.scheduleUpdate(PHASES[newPhase].label);
   }
+  enterFinishingAndStop() {
+    if (PHASE_ORDER.finishing > PHASE_ORDER[this.phase]) {
+      this.runEventSink?.recordPhaseChanged("finishing", this.phase);
+      this.phase = "finishing";
+      this.progress = Math.max(this.progress, PHASES.finishing.min);
+    }
+    this.stop();
+  }
   incrementProgress() {
     const config = PHASES[this.phase];
     const range = config.max - config.min;
@@ -5836,6 +5850,11 @@ async function runTransition(client, card, plan, opts = {}) {
     await withRetry("endSession", shortId, () => client.endAgentSession(card.id, plan.endSession), attempts, backoffMs);
     log.info(TAG22, `#${shortId} session ended (${plan.endSession.status})`);
   }
+  if (plan.assignAgent !== undefined) {
+    const assignedAgentId = plan.assignAgent;
+    await withRetry("assignAgent", shortId, () => client.updateCard(card.id, { assignedAgentId }), attempts, backoffMs);
+    log.info(TAG22, assignedAgentId ? `#${shortId} assigned → agent ${assignedAgentId}` : `#${shortId} unassigned`);
+  }
   if (opts.store && opts.runId) {
     try {
       await opts.store.heartbeat(opts.runId);
@@ -6971,7 +6990,8 @@ async function advanceStageOnGate(card, stage, stageIndex, def, evaluation, deps
   });
   await runTransition(deps.client, card, {
     move: { columnName: toColumn },
-    addLabels: [{ name: AGENT_LABEL }]
+    addLabels: [{ name: AGENT_LABEL }],
+    ...isAgentRunnableOwner(next.stage.owner) ? { assignAgent: deps.agentId } : {}
   }, { store: deps.stateStore, runId: deps.runId });
   deps.stateStore.recordOutcome(card.id, "success").catch(() => {});
   log.info(TAG28, `#${card.short_id} advanced "${stage.name}" → "${next.stage.name}" (column "${toColumn}")`);
@@ -7000,7 +7020,11 @@ async function handleGateUnmet(card, stage, summary, deps) {
   try {
     await deps.client.addComment(card.id, `Stage gate unmet — re-running "${stage.name}". ${summary}.`, { commentType: "progress" });
   } catch {}
-  await runTransition(deps.client, card, { move: { columnName: toColumn }, addLabels: [{ name: AGENT_LABEL }] }, { store: deps.stateStore, runId: deps.runId });
+  await runTransition(deps.client, card, {
+    move: { columnName: toColumn },
+    addLabels: [{ name: AGENT_LABEL }],
+    ...isAgentRunnableOwner(stage.owner) ? { assignAgent: deps.agentId } : {}
+  }, { store: deps.stateStore, runId: deps.runId });
   log.info(TAG28, `#${card.short_id} gate unmet for "${stage.name}" — requeued to "${toColumn}" for re-run (attempt ${attempts}/${deps.maxAttempts})`);
   return { kind: "requeued_gate_unmet", toColumn };
 }
@@ -7100,6 +7124,7 @@ class Worker {
   timedOut = false;
   verificationFailed = false;
   held = false;
+  completionStarted = false;
   sessionId = null;
   runId = null;
   cliSessionId = null;
@@ -7171,6 +7196,7 @@ class Worker {
     this.timedOut = false;
     this.verificationFailed = false;
     this.held = false;
+    this.completionStarted = false;
     this.runCostCents = 0;
     this.runTurns = 0;
     this.cliSessionId = null;
@@ -7310,6 +7336,7 @@ class Worker {
         progressPercent: 75
       });
       this.state = "completing";
+      this.completionStarted = true;
       await this.recordPhase("completing");
       let stageGateEvaluation = null;
       const stageRun = stageCtx.kind === "run" ? stageCtx : null;
@@ -7661,6 +7688,7 @@ class Worker {
       return await advanceStageOnGate(card, stage, stageIndex, def, evaluation, {
         client: this.client,
         stateStore: this.stateStore,
+        agentId: this.agentId,
         maxAttempts: this.config.budget.maxAttemptsPerCard,
         fallbackColumn: this.config.pickupColumns[0] ?? "To Do",
         sink: this.cliRunner,
@@ -7763,7 +7791,7 @@ class Worker {
         sigtermTimeoutMs: CANCEL_SIGTERM_TIMEOUT2
       });
     }
-    if (this.cardId && !this.timedOut) {
+    if (this.cardId && !this.timedOut && !this.completionStarted) {
       try {
         const stats = this.lastSessionStats ?? this.progressTracker?.stats;
         await this.client.endAgentSession(this.cardId, {

package/package.json

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