@volter/supercode-orchestrator 0.5.63 → 0.5.64

package/board/dispatch.mjs

@@ -2396,7 +2396,7 @@ export async function plan({ root, lanes, settings, slugs = null, params = {} })
     const board = openBoard(root, slug);
     try {
       board.label = boardLabel(root, slug);
-      await activatePullRequestReviews(board, entry, report, [], { dry: true });
+      await activatePullRequestReviews(board, entry, report, [], { dry: true, fleet });
       startCards(board, fleet, lanes, settings, report, entry, [], { dry: true, held });
     } finally { board.close(); }
     out.push(report);
@@ -2417,7 +2417,19 @@ function noCodeReviewDue(board, id) {
 
 /** An opened PR asks for the declared review transition once the arc's tasks are closed. */
 // A task written in the card's own Tasks section that did not become a subtask holds its review too.
-async function activatePullRequestReviews(board, entry, report, effects, { only = null, dry = false } = {}) {
+/** The arc's session still in a turn: its review, and the close of that session the request brings (end_run), wait for
+ *  the turn to end, so work in flight is never cut (t_f34710bc's coder had a proof take running when its review was
+ *  requested at 15:01:13Z on 2026-10-04, and the close ended it). A session the reading cannot place waits on nothing. */
+function turnInFlight(board, id, fleet) {
+  if (!fleet?.ok) return null;
+  const session = currentRun(board.db, id)?.metadata?.supercode;
+  if (!session?.session_id || session.surface === 'headless' || session.close_requested) return null;
+  let turn = fleet.sessions?.find((x) => x.address === addressOf(session))?.turn ?? null;
+  if (turn == null && session.surface === 'native') try { turn = nativeReading(nativeSessions(session.machine, session.harness ?? 'claude-code', session.session_id), session).row?.activity?.turn ?? null; } catch { /* unread: waits on nothing */ }
+  return turn === 'working' ? `its session ${addressOf(session)} is in a turn; the review is requested once the turn ends` : null;
+}
+
+async function activatePullRequestReviews(board, entry, report, effects, { only = null, dry = false, fleet = null } = {}) {
   const candidates = view(board, db => db.prepare(`SELECT t.id FROM tasks t JOIN supercode_cards c ON c.task_id=t.id
     WHERE c.subtask_of IS NULL AND c.every IS NULL
     AND t.workspace_kind!='scratch' ${only ? 'AND t.id=?' : ''}`).all(...(only ? [only] : [])));
@@ -2435,6 +2447,8 @@ async function activatePullRequestReviews(board, entry, report, effects, { only
     const ready = view(board, eligible);
     if (!ready) continue;
     if (ready !== true) { report.queued.push([id, `review waits: ${ready.length} open task(s) in the card's Tasks: ${ready[0].id} ${ready[0].title}`]); continue; }
+    const inTurn = turnInFlight(board, id, fleet);
+    if (inTurn) { report.queued.push([id, `review waits: ${inTurn}`]); continue; }
     let payload = null;
     try {
       payload = openReviewPullRequests(board, id);
@@ -2604,7 +2618,7 @@ export async function tick({ root, lanes, settings, only = null, slugs = null, d
       report.unsettled = [];
       report.promoted = recomputeReady(board, effects, report.unsettled);
       tx(board,db=>auditEndedRuns(db,board,entry));
-      if (dispatch) await activatePullRequestReviews(board, entry, report, effects, { only });
+      if (dispatch) await activatePullRequestReviews(board, entry, report, effects, { only, fleet });
       if (dispatch && !stopping) startCards(board, fleet, lanes, settings, report, entry, effects, { only, held });
       phase('start');
       if (dispatch && !only && !report.paused) observeIdle(board, fleet, settings, report, entry, effects);

package/package.json

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