@volter/supercode-orchestrator 0.5.58 → 0.5.59

package/board/dispatch.mjs

@@ -640,7 +640,7 @@ function startCards(board, fleet, lanes, settings, report, entry, effects, { onl
   const candidates = [];
   const waiting = (task, why) => {
     report.queued.push([task.id, why]);
-    if (!dry) tx(board, db => noteGuard(db, task.id, why));
+    if (!dry && !guardNoted(board.db, task.id, why)) tx(board, db => noteGuard(db, task.id, why));
   };
   for (const task of everyCard) {
     if (only && task.id !== only) continue;
@@ -673,7 +673,7 @@ function startCards(board, fleet, lanes, settings, report, entry, effects, { onl
     status: c.plan.target, stored: c.plan.target, model: c.task.model_override ?? c.plan.lane?.model ?? held.defaults?.[c.plan.lane?.harness] ?? null, harness: c.plan.lane?.harness ?? null });
   const waitingEntries = candidates.map(entryOf);
   // a dry run (dispatch --dry-run) plans the same way and writes nothing: no notes, no moves, no launches
-  const note = (id, why) => { if (!dry) tx(board, (db) => noteGuard(db, id, why)); };
+  const note = (id, why) => { if (!dry && !guardNoted(board.db, id, why)) tx(board, (db) => noteGuard(db, id, why)); };
   for (const [i, c] of candidates.entries()) {
     const { task, plan, scope } = c;
     if (!plan.profile) { report.unassigned.push(task.id); note(task.id, 'no profile runs it (no assignee, and the workflow names none)'); continue; }
@@ -714,7 +714,7 @@ function startCards(board, fleet, lanes, settings, report, entry, effects, { onl
     } catch (error) {
       const reason = error.refusal ?? String(error.message).slice(-300);
       // another round's start is that round's to say; this one writes nothing for it
-      if (!error.claimedElsewhere) tx(board, (db) => noteGuard(db, task.id, reason));
+      if (!error.claimedElsewhere && !guardNoted(board.db, task.id, reason)) tx(board, (db) => noteGuard(db, task.id, reason));
       report.queued.push([task.id, reason]);
       continue;
     }
@@ -1247,12 +1247,16 @@ function memoryPressure() {
 
 /** Record a guard reason once per change, not once per tick. */
 function noteGuard(db, id, reason) {
-  // once per reason per entry into the status: a card back in ready (a new `since`) is noted again, so the workflow's
-  // after-timers that read the reason (card.queue_reason) see it
+  if (guardNoted(db, id, reason)) return;
+  event(db, id, 'respawn_guarded', { reason });
+}
+/** Whether `reason` is already noted: once per reason per entry into the status (a card back in ready, a new `since`, is
+ *  noted again, so the workflow's after-timers that read the reason, card.queue_reason, see it). Read before a write
+ *  is opened too: a card that waits for the same reason round after round costs no transaction. */
+function guardNoted(db, id, reason) {
   const since = Number(db.prepare('SELECT since FROM supercode_cards WHERE task_id = ?').get(id)?.since ?? 0);
   const last = db.prepare("SELECT payload FROM task_events WHERE task_id = ? AND kind IN ('respawn_guarded', 'claimed') AND created_at >= ? ORDER BY id DESC LIMIT 1").get(id, since);
-  if (json(last?.payload, {})?.reason === reason) return;
-  event(db, id, 'respawn_guarded', { reason });
+  return json(last?.payload, {})?.reason === reason;
 }
 
 /** Hermes's emergency stop: the `ESTOP` sentinel in the home or in `~/.hermes` stops new starts (running ones go on). */

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@volter/supercode-orchestrator",
-  "version": "0.5.58",
+  "version": "0.5.59",
   "type": "module",
   "description": "The orchestrator runtime over the Volter Harness ontology: one typed operational model whose folder is its serialization, read and written through the harness orchestration doors (docs/ORCHESTRATOR-IR.md)",
   "exports": {