@volter/supercode-orchestrator 0.5.126 → 0.5.127

package/board/ci.mjs

@@ -411,12 +411,20 @@ export async function inspectCompletionEvidence(board, id, entry, payload = {},
     // Merged source is immutable and needs no surviving workspace. Resolve an
     // unmerged candidate's checkout as before; a broken legacy path never
     // prevents a verdict on already merged commits.
-    const mergedOnly = urls.length && (await Promise.all(urls.map(readPullRequest))).every(pr => pr.state === 'MERGED');
+    const read = await Promise.all(urls.map(readPullRequest));
+    const mergedOnly = urls.length && read.every(pr => pr.state === 'MERGED');
     const own = mergedOnly ? null : await repositoryWorkspace(board, id);
     if (!own && !urls.length) {
       return state;
     }
-    if (!own && !mergedOnly) throw new Error('An unmerged PR candidate needs a resolved repository workspace.');
+    // A card with no repository of its own (scratch) is reviewed through the branches it registered: each unmerged PR
+    // on one of them (or on its wt/<card> branch) is read at its pushed head, and needs no checkout to exist here.
+    if (!own && !mergedOnly) {
+      const registered = registeredBranches(board, id);
+      const declared = (pr) => namedWorkBranch(id, pr.headRefName) || registered.some((r) => r.branch === pr.headRefName && r.repository.toLowerCase() === repositoryOf(pr.url).toLowerCase());
+      const stray = read.find((pr) => pr.state !== 'MERGED' && !declared(pr));
+      if (stray) throw new Error(`An unmerged PR candidate needs a repository workspace: ${stray.url} (${stray.headRefName}) is on no branch the card or its tasks declare; register it with create "<what>" --subtask-of ${id} --workspace worktree:${repositoryOf(stray.url)} --branch ${stray.headRefName}`);
+    }
     // A verdict with no requested PR (a scratch arc) keeps its own contract. An arc's review
     // always names every opened PR it changed, independent of completion_ci.
     if (stage === 'review_verdict' && !asked && !urls.length) return state;
@@ -452,7 +460,7 @@ async function verifyPullRequest(board, id, entry, own, url, supplied, stage) {
   // card's own checkout, and this machine's checkout of the PR's branch when it keeps one, must also hold that head
   // with nothing uncommitted. A card on another machine is reviewed at its pushed head: its checkout is not read from here.
   const observed = await readPullRequest(url);
-  const path = !own?.here || observed.state === 'MERGED' ? null : await originOf(own.path) === repository && await run('git', ['branch', '--show-current'], own.path) === observed.headRefName ? own.path : await branchCheckout(board, id, repository, observed.headRefName);
+  const path = observed.state === 'MERGED' || (own && !own.here) ? null : !own ? await branchCheckout(board, id, repository, observed.headRefName) : await originOf(own.path) === repository && await run('git', ['branch', '--show-current'], own.path) === observed.headRefName ? own.path : await branchCheckout(board, id, repository, observed.headRefName);
   // A PR merged before its review (an outage fix landed first) is still reviewed: its head is fixed for good, so the
   // verdict is recorded on that head, and nothing about a worktree or its base can move under it any more.
   const merged = observed.state === 'MERGED';

package/package.json

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