@volter/supercode-orchestrator 0.5.155 → 0.5.156

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.155",
+  "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.95",
+    "@volter/supercode-harness-sdk": "^0.3.96",
     "@volter/teams": "0.2.65",
     "@volter/ztrack": "^2.4.23",
     "yaml": "^2.9.0"