@volter/supercode-orchestrator 0.4.33 → 0.4.34

package/board/dispatch.mjs

@@ -523,7 +523,10 @@ function memoryPressure() {
 
 /** Record a guard reason once per change, not once per tick. */
 function noteGuard(db, id, reason) {
-  const last = db.prepare("SELECT payload FROM task_events WHERE task_id = ? AND kind IN ('respawn_guarded', 'claimed') ORDER BY id DESC LIMIT 1").get(id);
+  // 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
+  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 });
 }

package/board/engine.mjs

@@ -130,15 +130,6 @@ function retryStatus(db, taskId) {
   return source === 'review' || source === 'triage' ? source : 'ready';
 }
 
-/** The phase a card blocked out of comes back to (Hermes reads it off the latest interruption). */
-function resumeStatus(db, taskId) {
-  const row = one(db, `SELECT payload FROM task_events WHERE task_id = ? AND kind IN ('blocked', 'block_loop_detected', 'dependency_wait', 'gave_up',
-    'unblocked', 'changes_requested', 'review_reopened', 'status', 'reclaimed', 'stale', 'timed_out', 'crashed', 'spawn_failed', 'rate_limited', 'session_lost')
-    ORDER BY id DESC LIMIT 1`, taskId);
-  const p = json(row?.payload, {}) ?? {};
-  return ['resume_status', 'retry_status', 'source_status'].some((k) => p[k] === 'review') ? 'review' : 'ready';
-}
-
 // ------------------------------------------------------------------ creating and linking
 
 /**
@@ -290,35 +281,6 @@ export function extendClaim(db, id, { ttl, reason, extra = {} }) {
   return true;
 }
 
-/** Put a running card back where its run came from, closing the run with `outcome`. Answers the closed run id. */
-export function release(db, id, { outcome = 'reclaimed', error = null, metadata = undefined, kind = 'reclaimed', payload = {} } = {}) {
-  const task = one(db, 'SELECT status, claim_lock FROM tasks WHERE id = ?', id);
-  if (!task || task.status !== 'running') return null;
-  const retry = retryStatus(db, id);
-  run(db, 'UPDATE tasks SET status = ?, claim_lock = NULL, claim_expires = NULL, worker_pid = NULL WHERE id = ?', retry, id);
-  const runId = endRun(db, id, { outcome, status: outcome, error, metadata });
-  event(db, id, kind, { ...payload, retry_status: retry }, runId);
-  return runId;
-}
-
-/**
- * Count a failure against the card (Hermes's circuit breaker): at the card's `max_retries`, else `failureLimit`,
- * consecutive failures it is blocked with a `gave_up` event. Answers true when it tripped.
- */
-export function recordFailure(db, id, error, { failureLimit = DEFAULT_FAILURE_LIMIT, outcome = 'crashed' } = {}) {
-  const task = one(db, 'SELECT consecutive_failures, max_retries, status FROM tasks WHERE id = ?', id);
-  if (!task) return false;
-  const failures = Number(task.consecutive_failures || 0) + 1;
-  run(db, 'UPDATE tasks SET consecutive_failures = ?, last_failure_error = ? WHERE id = ?', failures, String(error).slice(0, 500), id);
-  const limit = task.max_retries ?? failureLimit;
-  if (failures < limit) return false;
-  if (task.status === 'running') endRun(db, id, { outcome, error: String(error) });
-  run(db, "UPDATE tasks SET status = 'blocked', claim_lock = NULL, claim_expires = NULL, worker_pid = NULL WHERE id = ?", id);
-  event(db, id, 'gave_up', { error: String(error).slice(0, 500), failures, limit, outcome });
-  return true;
-}
-
-
 /**
  * A completed card's workspace goes, as Hermes cleans up on completion: a scratch directory under the board's own
  * workspaces, or a worktree (its branch stays); never while a child still waits on it, never a `dir`.
@@ -541,8 +503,8 @@ export const subscriptions = (db, id = null) => (id ? all(db, 'SELECT * FROM kan
 
 export function listCards(db, { status = null, assignee = null, machine = null, includeArchived = false } = {}) {
   const rows = all(db, `SELECT t.*, c.machine, c.subtask_of, c.stage FROM tasks t LEFT JOIN supercode_cards c ON c.task_id = t.id
-    WHERE (? IS NULL OR t.status = ?) AND (? IS NULL OR t.assignee = ?) AND (? IS NULL OR c.machine = ?) AND (? OR t.status != 'archived')
-    ORDER BY t.priority DESC, t.created_at ASC`, status, status, assignee, assignee, machine, machine, includeArchived ? 1 : 0);
+    WHERE (? IS NULL OR t.status = ? OR c.stage = ?) AND (? IS NULL OR t.assignee = ?) AND (? IS NULL OR c.machine = ?) AND (? OR t.status != 'archived')
+    ORDER BY t.priority DESC, t.created_at ASC`, status, status, status, assignee, assignee, machine, machine, includeArchived ? 1 : 0);
   return rows.map((r) => ({ ...r, skills: json(r.skills, []) ?? [] }));
 }
 

package/package.json

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