@volter/supercode-orchestrator 0.5.154 → 0.5.156

package/board/cli.mjs

@@ -279,6 +279,9 @@ async function requestedSponsor(args) {
   return { requestedByUnverified: { observation: args['requested-by'], label: resolved?.label ?? 'unresolved', request: args.request ?? null } };
 }
 
+// `list --json` reads each card's parents and standing block per card up to this many cards, and in one read of each past it
+const LIST_PER_CARD = 64;
+
 const tagsOf = (card) => { const tags = Array.isArray(card.tags) ? card.tags : json(card.tags, null); return Array.isArray(tags) ? tags : card.tenant ? [card.tenant] : []; };
 
 const say = (args, human, value) => { current().out(args.json ? `${JSON.stringify(value, null, 2)}\n` : `${human}\n`); };
@@ -520,20 +523,36 @@ export async function runVerb(argv) {
       if (args.json) {
         // beyond Hermes's fields: the workflow's own status, the cards it waits on and its open run, so a reader of the
         // whole board (a board file's sync) needs no `show` per card
-        const extra = withBoard((b) => new Map(cards.map((t) => {
-          const run = t.current_run_id ? b.db.prepare('SELECT id, status, started_at, metadata FROM task_runs WHERE id = ?').get(t.current_run_id) : null;
-          const session = run ? json(run.metadata, {})?.supercode ?? {} : {};
-          return [t.id, {
-            workflow_status: t.stage ?? t.status,
-            parents: b.db.prepare('SELECT parent_id FROM task_links WHERE child_id = ? ORDER BY parent_id').all(t.id).map((r) => r.parent_id),
-            run: run ? { id: run.id, status: run.status, started_at: run.started_at, session: session.address ?? session.session_id ?? null, execution: executionOf(run) } : null,
-            // the message whose answer its standing block waits for (`block --waiting-on`)
-            waiting_on: (() => {
-              const last = b.db.prepare("SELECT kind, payload FROM task_events WHERE task_id = ? AND kind IN ('blocked', 'unblocked', 'dependency_wait') ORDER BY id DESC LIMIT 1").get(t.id);
-              return last?.kind === 'blocked' ? json(last.payload, {})?.waiting_on ?? null : null;
-            })(),
-          }];
-        })));
+        const extra = withBoard((b) => {
+          // A whole board's parents and standing blocks are read once, not once per card. Asked per card, the block's
+          // read fetched every event row of every card through the card's index to answer from the few of these
+          // kinds (on 2026-10-11: 45,670 rows of a 17 MB table for 2,154 cards, to find 332), on the owner's event
+          // loop, for as long as the listing took. A short listing still asks per card: it reads less than the scan.
+          const whole = cards.length > LIST_PER_CARD;
+          const parentsOf = new Map(), standing = new Map();
+          if (whole) {
+            for (const row of b.db.prepare('SELECT child_id, parent_id FROM task_links ORDER BY child_id, parent_id').all()) {
+              if (!parentsOf.has(row.child_id)) parentsOf.set(row.child_id, []);
+              parentsOf.get(row.child_id).push(row.parent_id);
+            }
+            // in id order, so the last one kept for a card is its latest
+            for (const row of b.db.prepare("SELECT task_id, kind, payload FROM task_events WHERE kind IN ('blocked', 'unblocked', 'dependency_wait') ORDER BY id").all()) standing.set(row.task_id, row);
+          }
+          return new Map(cards.map((t) => {
+            const run = t.current_run_id ? b.db.prepare('SELECT id, status, started_at, metadata FROM task_runs WHERE id = ?').get(t.current_run_id) : null;
+            const session = run ? json(run.metadata, {})?.supercode ?? {} : {};
+            return [t.id, {
+              workflow_status: t.stage ?? t.status,
+              parents: whole ? parentsOf.get(t.id) ?? [] : b.db.prepare('SELECT parent_id FROM task_links WHERE child_id = ? ORDER BY parent_id').all(t.id).map((r) => r.parent_id),
+              run: run ? { id: run.id, status: run.status, started_at: run.started_at, session: session.address ?? session.session_id ?? null, execution: executionOf(run) } : null,
+              // the message whose answer its standing block waits for (`block --waiting-on`)
+              waiting_on: (() => {
+                const last = whole ? standing.get(t.id) : b.db.prepare("SELECT kind, payload FROM task_events WHERE task_id = ? AND kind IN ('blocked', 'unblocked', 'dependency_wait') ORDER BY id DESC LIMIT 1").get(t.id);
+                return last?.kind === 'blocked' ? json(last.payload, {})?.waiting_on ?? null : null;
+              })(),
+            }];
+          }));
+        });
         say(args, '', cards.map((t) => ({ ...cardDict(t), ...extra.get(t.id) })));
         return 0;
       }

package/board/dispatch.mjs

@@ -838,8 +838,11 @@ async function startCards(board, fleet, lanes, settings, report, entry, effects,
   };
   for (const task of everyCard) {
     if (only && task.id !== only) continue;
+    // a subtask or a card with a run is never started here, whatever its scope says: no scope is built to learn it (the
+    // scope of each of a board's 2,155 unarchived cards, 2,039 of them subtasks or running, was 0.8 s of every round)
+    if (task.subtask_of || task.current_run_id) continue;
     const state = scopeFor(task).card;
-    if (task.subtask_of || state.archived || task.current_run_id || ['done', 'cancelled'].includes(state.status)) continue;
+    if (state.archived || ['done', 'cancelled'].includes(state.status)) continue;
     if (state.dispatch_hold) { waiting(task, 'dispatch held: an explicit start or verified adoption must release it'); continue; }
     if (workflow.run && state.paused) { waiting(task, `paused: ${state.paused.reason ?? 'manager pause'}`); continue; }
     if (workflow.run && state.blocked_by.length) { waiting(task, `blocked by ${state.blocked_by.map(b => b.id).join(', ')}`); continue; }
@@ -3087,9 +3090,10 @@ async function activatePullRequestReviews(board, entry, report, effects, { only
     const eligible = db => {
       const task = getTask(db, id);
       if (!task) return false;
+      // a status the platform cannot request a review from needs no scope built to say so
+      if (!transitionsFor(entry.workflow, statusOf(db, task), 'request_review').some(t => !t.by || t.by.includes('platform'))) return false;
       const scope = scopeOf(db, board, task, { params: entry.params });
-      if (!transitionsFor(entry.workflow, statusOf(db, task), 'request_review').some(t => !t.by || t.by.includes('platform')) ||
-        scope.card.dispatch_hold || scope.card.paused || scope.card.blocked_by.length) return false;
+      if (scope.card.dispatch_hold || scope.card.paused || scope.card.blocked_by.length) return false;
       // The card's Tasks are its subtasks; an open one holds the review, and says so.
       const open = scope.card.subtasks.filter(t => !['done', 'cancelled'].includes(t.status));
       return open.length ? open : true;

package/board/store.mjs

@@ -150,6 +150,10 @@ export function layOut(db, { fresh }) {
   db.exec(PUBLICATION_SCHEMA);
   const have = new Set(db.prepare('PRAGMA table_info(supercode_cards)').all().map((c) => c.name));
   for (const [column, type] of Object.entries(OURS_COLUMNS)) if (!have.has(column)) db.exec(`ALTER TABLE supercode_cards ADD COLUMN ${column} ${type}`);
+  // A card's subtasks are read by the arc they name, for every scope built (workflow.mjs scopeOf): without this index
+  // each read walked every card's row. Laid out here, after the column it names exists on a store of any age and before
+  // any statement is prepared; no statement names it, so a store without it is only slower.
+  db.exec('CREATE INDEX IF NOT EXISTS idx_supercode_cards_subtask_of ON supercode_cards(subtask_of)');
   const effectColumns = new Set(db.prepare('PRAGMA table_info(supercode_effects)').all().map((c) => c.name));
   for (const [column, type] of Object.entries(EFFECT_COLUMNS)) if (!effectColumns.has(column)) db.exec(`ALTER TABLE supercode_effects ADD COLUMN ${column} ${type}`);
   // what changed is marked by the store itself, whoever writes it (published.mjs)

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@volter/supercode-orchestrator",
-  "version": "0.5.154",
+  "version": "0.5.156",
   "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": {
@@ -24,7 +24,7 @@
     "check-types": "node scripts/generate-types.mjs --check"
   },
   "dependencies": {
-    "@volter/supercode-harness-sdk": "^0.3.91",
+    "@volter/supercode-harness-sdk": "^0.3.96",
     "@volter/teams": "0.2.65",
     "@volter/ztrack": "^2.4.23",
     "yaml": "^2.9.0"