@volter/supercode-orchestrator 0.5.58 → 0.5.60

package/board/dispatch.mjs

@@ -206,7 +206,8 @@ function unpublishedAlive(session) {
 }
 
 /** Native activity comes from the machine's harness discovery, not catalogue age. One discovery (up to 2000
- * sessions, seconds each) serves a round's every native session on that machine, as paneLists serves its panes. */
+ * sessions, seconds each) serves a round's every native session on that machine: taken once per round (tick clears
+ * them), however long the round runs. A round of 163 s took it 28 times when a reading lasted 15 s. */
 const discoveries = new Map();
 // A machine's pane list and harness discovery are read before a board write opens, never inside one: a door that
 // takes seconds would hold the board's write lock past a worker's own command's busy wait. A write that needs a
@@ -222,7 +223,7 @@ function readBefore(fleet, session) {
 }
 function nativeSessions(machine, harness) {
   const key = `${normalMachine(machine)}\u0000${harness}`, seen = discoveries.get(key);
-  if (seen && Date.now() - seen.at <= READING_MS) return seen.sessions;
+  if (seen) return seen.sessions;
   if (readingsOnly) throw unavailable('native liveness is read before a board write, not inside it');
   const sessions = discoverNative(machine, harness);
   discoveries.set(key, { at: Date.now(), sessions });
@@ -640,7 +641,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 +674,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 +715,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 +1248,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). */
@@ -2555,6 +2560,7 @@ export async function deliver({ root, slugs = null }) {
  */
 export async function tick({ root, lanes, settings, only = null, slugs = null, dispatch = true, params = {}, deploymentReceipt = false }) {
   fleetRoot = root;
+  discoveries.clear();
   // a verb's flags (dispatch --max, --failure-limit) are params of this tick, read by the workflow as its own
   const deployment = deploymentReceipt ? workflowRevision(root) : null;
   const loaded = await loadWorkflow(root, { fresh: true });

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@volter/supercode-orchestrator",
-  "version": "0.5.58",
+  "version": "0.5.60",
   "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": {