@volter/supercode-orchestrator 0.5.155 → 0.5.157

package/board/cli.mjs

@@ -1590,14 +1590,22 @@ async function answerVerbs(root, log) {
 
 /** One request to the owner, answered whole: the verb run in the caller's call, its output kept for the answer. A
  *  verb that takes more than a second is a slow line of its own, with the steps that ended while it ran (its own and,
- *  in this one process, any other call's that held it). */
+ *  in this one process, any other call's that held it) and what the process used while it ran (`process`): its
+ *  processor time beside the line's `ms` says whether the time was computed or waited for, with the major page faults
+ *  and the waits (voluntary context switches) and preemptions (involuntary) the kernel counted. */
 async function answerOne(root, line) {
   const began = performance.now();
+  const before = process.resourceUsage();
   let named = null, by = null;
   try { return await answerRequest(root, line, (verb, caller) => { named = verb; by = caller; }); }
   finally {
     const ms = performance.now() - began;
-    if (named) recordSlow({ kind: 'call', name: `owner verb ${named}`, ms, slowest: null, extra: { caller: by, steps: slowLog.stepsWithin?.(began, performance.now()) ?? null } });
+    if (named && ms > slowLog.SLOW_MS) {
+      const after = process.resourceUsage();
+      const used = { cpu_ms: Math.round((after.userCPUTime + after.systemCPUTime - before.userCPUTime - before.systemCPUTime) / 1000), major_faults: after.majorPageFault - before.majorPageFault,
+        waits: after.voluntaryContextSwitches - before.voluntaryContextSwitches, preempted: after.involuntaryContextSwitches - before.involuntaryContextSwitches };
+      recordSlow({ kind: 'call', name: `owner verb ${named}`, ms, slowest: null, extra: { caller: by, steps: slowLog.stepsWithin?.(began, performance.now()) ?? null, process: used } });
+    }
   }
 }
 

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/board/ztrack.mjs

@@ -21,6 +21,15 @@ import * as slowLog from '@volter/supercode-harness-sdk/slow-log';
 
 /** Run `fn` as a named step of the running command (the slow log names its slowest step). */
 const timedStep = (step, fn) => { const started = performance.now(); try { return fn(); } finally { noteStep(step, performance.now() - started); } };
+// One statement that takes this long is a step of its own, named by its text: a verb that only reads (`show`) runs in
+// no transaction and has no other step, so a hold inside it was a slow line that named no statement (8.7 s on
+// 2026-10-11, where every statement of `show` measures under 80 ms on a copy of the same store).
+const SLOW_STATEMENT_MS = 100;
+const timedStatement = (sql, fn) => {
+  const started = performance.now();
+  try { return fn(); }
+  finally { const ms = performance.now() - started; if (ms >= SLOW_STATEMENT_MS) noteStep(`board statement: ${String(sql).replace(/\s+/g, ' ').trim().slice(0, 120)}`, ms); }
+};
 import { fire, workflowFor } from './workflow.mjs';
 
 const require = createRequire(import.meta.url);
@@ -128,10 +137,10 @@ class ZtrackBoard {
       boardErrors: self.errors,
       prepare(sql) {
         return {
-          get: (...args) => self.read(() => self.statement(sql).get(...args)),
-          all: (...args) => self.read(() => self.statement(sql).all(...args)),
-          iterate: (...args) => self.read(() => self.statement(sql).all(...args))[Symbol.iterator](),
-          run: (...args) => self.active ? self.noting(() => self.statement(sql).run(...args)) : self.tx(() => self.statement(sql).run(...args)),
+          get: (...args) => self.read(() => timedStatement(sql, () => self.statement(sql).get(...args))),
+          all: (...args) => self.read(() => timedStatement(sql, () => self.statement(sql).all(...args))),
+          iterate: (...args) => self.read(() => timedStatement(sql, () => self.statement(sql).all(...args)))[Symbol.iterator](),
+          run: (...args) => self.active ? self.noting(() => timedStatement(sql, () => self.statement(sql).run(...args))) : self.tx(() => timedStatement(sql, () => self.statement(sql).run(...args))),
         };
       },
       exec(sql) { return self.active ? self.noting(() => self.conn.exec(sql)) : self.tx(() => self.conn.exec(sql)); },

package/package.json

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