@sublang/playbook 8.0.0 → 9.0.0

package/reference/sdlc/decide.playbook/decide.playbook.ts

@@ -114,20 +114,51 @@ function snapshotDecideRuntimeOptions(value: unknown): PlaybookRuntimeOptions {
   return Object.freeze({});
 }
 
-const STATE_DESCRIPTIONS: Readonly<Record<string, string>> = {
-  ready: 'Waiting for a topic to decide.',
-  askCoderProposal: 'Coder independently proposes a spec design.',
-  askReviewerProposal: 'Reviewer independently proposes a spec design.',
-  waitCoderProposalReply: 'Coder waits for Boss to answer a question.',
-  waitReviewerProposalReply: 'Reviewer waits for Boss to answer a question.',
-  commitCoderProposal: 'Coder writes and commits Coder’s independent proposal.',
-  awaitBossReply: 'Waiting for Boss to answer Coder’s question.',
-  reviewCommit: 'REVIEW examines the committed proposal.',
-  failed: 'DECIDE failed and is waiting for a new topic.',
-  reportedReviewFailure:
-    'DECIDE reports REVIEW’s failure and its last commit.',
-  done: 'DECIDE completed with an approved commit.',
-};
+interface AuthoredStateConfig {
+  readonly meta?: {
+    readonly playbook?: {
+      readonly stateId?: unknown;
+      readonly description?: unknown;
+    };
+  };
+  readonly states?: Readonly<Record<string, AuthoredStateConfig>>;
+}
+
+function authoredStateDescriptions(
+  states: Readonly<Record<string, AuthoredStateConfig>> | undefined,
+): Readonly<Record<string, string>> {
+  const descriptions: Record<string, string> = {};
+  const visit = (
+    children: Readonly<Record<string, AuthoredStateConfig>> | undefined,
+  ): void => {
+    for (const state of Object.values(children ?? {})) {
+      const stateId = state.meta?.playbook?.stateId;
+      const description = state.meta?.playbook?.description;
+      if (
+        typeof stateId === 'string' &&
+        typeof description === 'string' &&
+        description.trim().length > 0
+      ) {
+        const existing = descriptions[stateId];
+        if (existing !== undefined && existing !== description) {
+          throw new Error(
+            `DECIDE state ${stateId} declares conflicting descriptions`,
+          );
+        }
+        descriptions[stateId] = description;
+      }
+      visit(state.states);
+    }
+  };
+  visit(states);
+  return Object.freeze(descriptions);
+}
+
+const STATE_DESCRIPTIONS = authoredStateDescriptions(
+  decideMachine.config.states as
+    | Readonly<Record<string, AuthoredStateConfig>>
+    | undefined,
+);
 
 const ROLE_STATES = [
   { stateId: 'askCoderProposal', role: 'coder', sourceItem: 'DECIDE-1' },
@@ -584,6 +615,27 @@ function isAbortFailure(error: unknown, signal: AbortSignal): boolean {
   return signal.aborted && Object.is(error, signal.reason);
 }
 
+interface AbortReasonClassifier {
+  isAbortReason(error: unknown): boolean;
+}
+
+function abortReasonClassifier(
+  ...sources: readonly (AbortSignal | AbortReasonClassifier | undefined)[]
+): AbortReasonClassifier {
+  const captured = sources.filter(
+    (source): source is AbortSignal | AbortReasonClassifier =>
+      source !== undefined,
+  );
+  return Object.freeze({
+    isAbortReason: (error: unknown): boolean =>
+      captured.some((source) =>
+        source instanceof AbortSignal
+          ? isAbortFailure(error, source)
+          : source.isAbortReason(error),
+      ),
+  });
+}
+
 function pendingQuestionsFromContext(
   context: Record<string, unknown>,
 ): PendingBossQuestion[] {
@@ -635,6 +687,33 @@ const STATUS_STATE_IDS: ReadonlySet<string> = new Set([
   'failed',
 ]);
 
+// PBRT-45: a question is pending only while its authored reply-wait state
+// is active. The context retains an answered question through the resumed
+// player call so the Q+A continuation prompt can quote it, and each branch
+// keeps its own entry through the parallel region — so an unfiltered
+// projection would report the answered question as still awaiting during
+// the resume, and both branch questions after only one remains pending.
+const RESUME_WAIT_STATE_IDS: Readonly<Record<string, string>> = {
+  ...Object.fromEntries(
+    Object.entries(WAIT_STATE_RESUME_IDS).map(([waitStateId, resumeStateId]) => [
+      resumeStateId,
+      waitStateId,
+    ]),
+  ),
+  commitCoderProposal: 'awaitBossReply',
+};
+
+function pendingQuestionsForState(
+  state: PlaybookState,
+  context: Record<string, unknown>,
+): PendingBossQuestion[] {
+  return pendingQuestionsFromContext(context).filter((pending) =>
+    state.activeStateIds.includes(
+      RESUME_WAIT_STATE_IDS[pending.resumeStateId] ?? '',
+    ),
+  );
+}
+
 function questionForWaitState(
   stateId: string,
   pendingQuestions: readonly PendingBossQuestion[],
@@ -688,7 +767,7 @@ function telemetryPayload(
   event: unknown,
   context: Record<string, unknown>,
 ): JsonValue {
-  const pendingBossQuestions = pendingQuestionsFromContext(context);
+  const pendingBossQuestions = pendingQuestionsForState(state, context);
   const prior = previousState ?? state;
   const payload = {
     from: prior.value,
@@ -716,6 +795,9 @@ export const createPlaybookRuntime: PlaybookRuntimeFactory<
   let sessionIdentity: SessionIdentity | undefined;
   let actor: ReturnType<typeof createActor> | undefined;
   let currentSignal: AbortSignal | undefined;
+  let currentAborts: AbortReasonClassifier | undefined;
+  const actorSettlementAborts: AbortReasonClassifier[] = [];
+  let actorSettlementErrorAborts: AbortReasonClassifier | undefined;
   let currentTurnId: number | undefined;
   let previousState: PlaybookState | undefined;
   let suppressInspectionEmissions = false;
@@ -754,26 +836,35 @@ export const createPlaybookRuntime: PlaybookRuntimeFactory<
   ): void => {
     if (!isAbortFailure(error, signal)) controlPlaneError ??= error;
   };
-  const latchInspectionError = (error: unknown): void => {
-    if (currentSignal !== undefined) {
-      latchControlPlaneError(error, currentSignal);
-    } else {
-      collectFailure(emissionFailures, error);
-    }
+  const latchInspectionError = (
+    error: unknown,
+    aborts: AbortReasonClassifier | undefined = currentAborts,
+  ): void => {
+    if (aborts?.isAbortReason(error)) return;
+    if (currentSignal !== undefined) controlPlaneError ??= error;
+    else collectFailure(emissionFailures, error);
   };
-  const enqueue = (fn: () => Promise<void>): Promise<void> => {
+  const enqueue = (
+    fn: () => Promise<void>,
+    aborts: AbortReasonClassifier | undefined = currentAborts,
+  ): Promise<void> => {
+    const enqueueAborts = aborts;
     const queued = emissionQueue.add(fn);
     activeEmissionCalls.add(queued);
     void queued.then(
       () => activeEmissionCalls.delete(queued),
       (error: unknown) => {
         activeEmissionCalls.delete(queued);
-        collectFailure(emissionFailures, error);
+        if (!enqueueAborts?.isAbortReason(error)) {
+          collectFailure(emissionFailures, error);
+        }
       },
     );
     return queued;
   };
-  const flush = async (): Promise<void> => {
+  const flush = async (
+    _aborts: AbortReasonClassifier | undefined = currentAborts,
+  ): Promise<void> => {
     while (true) {
       const active = [...activeEmissionCalls];
       if (active.length > 0) await Promise.allSettled(active);
@@ -789,8 +880,15 @@ export const createPlaybookRuntime: PlaybookRuntimeFactory<
     if (emissionFailures.length === 0) return;
     const failures = emissionFailures;
     emissionFailures = [];
-    if (failures.length === 1) throw failures[0];
-    throw new AggregateError(failures, 'decide runtime emissions failed');
+    const failure =
+      failures.length === 1
+        ? failures[0]
+        : new AggregateError(failures, 'decide runtime emissions failed');
+    // Enqueue ownership already classified every stored failure as distinct.
+    // Preserve that classification if an unrelated public boundary drains
+    // it with a signal whose reason happens to be the same object.
+    if (currentSignal !== undefined) controlPlaneError ??= failure;
+    throw failure;
   };
   const drainBoundaryCallsAndEmissions = async (): Promise<void> => {
     while (true) {
@@ -1012,6 +1110,7 @@ export const createPlaybookRuntime: PlaybookRuntimeFactory<
     payload: unknown,
     meta: { turnId?: number; callId?: string } = {},
     describedEmission?: (runtimePorts: PlaybookPorts) => Promise<void>,
+    aborts?: AbortReasonClassifier,
   ): Promise<void> => {
     const runtimePorts = requirePorts();
     const identity = requireSessionIdentity();
@@ -1035,16 +1134,20 @@ export const createPlaybookRuntime: PlaybookRuntimeFactory<
       ...(meta.callId !== undefined ? { callId: meta.callId } : {}),
       payload: jsonPayload,
     });
-    return enqueue(async () => {
-      await runtimePorts.emitTelemetry({ topic: TRACE_TOPIC, payload: trace });
-      await describedEmission?.(runtimePorts);
-    });
+    return enqueue(
+      async () => {
+        await runtimePorts.emitTelemetry({ topic: TRACE_TOPIC, payload: trace });
+        await describedEmission?.(runtimePorts);
+      },
+      aborts,
+    );
   };
   const emitTrace = (
     type: PlaybookTraceType,
     payload: unknown,
     meta: { turnId?: number; callId?: string } = {},
-  ): Promise<void> => enqueueTracedEmission(type, payload, meta);
+    aborts?: AbortReasonClassifier,
+  ): Promise<void> => enqueueTracedEmission(type, payload, meta, undefined, aborts);
   const emitBoundaryStatus = async (
     message: string,
     state: PlaybookState,
@@ -1073,8 +1176,9 @@ export const createPlaybookRuntime: PlaybookRuntimeFactory<
     meta: { turnId?: number; callId?: string },
     signal: AbortSignal,
   ): Promise<void> => {
+    const aborts = abortReasonClassifier(signal);
     try {
-      await emitTrace(startedType, identity, meta);
+      await emitTrace(startedType, identity, meta, aborts);
     } catch (error) {
       latchControlPlaneError(error, signal);
       try {
@@ -1082,13 +1186,17 @@ export const createPlaybookRuntime: PlaybookRuntimeFactory<
           finishedType,
           {
             ...identity,
-            status: 'error',
+            // A started-trace sink rejection causally identical to the
+            // boundary reason is the abort's own evidence: the pair
+            // finishes 'aborted', not 'error' (DR-036 §4).
+            status: isAbortFailure(error, signal) ? 'aborted' : 'error',
             error: normalizeErrorFull(error) ?? {
               name: 'Error',
               message: String(error),
             },
           },
           meta,
+          aborts,
         );
       } catch {
         // Preserve the start failure after one best-effort finish attempt.
@@ -1103,6 +1211,7 @@ export const createPlaybookRuntime: PlaybookRuntimeFactory<
     purpose: 'boss-input-classification' | 'player-output-adjudication',
     callStateId: string | undefined,
   ): Promise<string> => {
+    const aborts = abortReasonClassifier(signal);
     const identity = {
       purpose,
       ...(callStateId !== undefined ? { stateId: callStateId } : {}),
@@ -1138,13 +1247,17 @@ export const createPlaybookRuntime: PlaybookRuntimeFactory<
           'judge.call.finished',
           {
             ...identity,
-            status: signal.aborted ? 'aborted' : 'error',
+            // Only the exact abort reason is cancellation; a distinct
+            // failure under an aborted signal stays an error
+            // (slc/link.md §Abort).
+            status: isAbortFailure(error, signal) ? 'aborted' : 'error',
             error: normalizeErrorFull(error) ?? {
               name: 'Error',
               message: String(error),
             },
           },
           { turnId: currentTurnId, callId },
+          aborts,
         );
         throw error;
       }
@@ -1153,6 +1266,7 @@ export const createPlaybookRuntime: PlaybookRuntimeFactory<
         'judge.call.finished',
         { ...identity, status: 'ok', reply: finalText },
         { turnId: currentTurnId, callId },
+        aborts,
       );
       return finalText;
     });
@@ -1178,6 +1292,7 @@ export const createPlaybookRuntime: PlaybookRuntimeFactory<
     playerId?: string;
     result: PlayerResult;
   }> => {
+    const aborts = abortReasonClassifier(signal);
     if (!ROLE_ID_SET.has(input.role)) {
       throw new TypeError(
         `DECIDE player input role must name a declared local role`,
@@ -1208,13 +1323,17 @@ export const createPlaybookRuntime: PlaybookRuntimeFactory<
         'player.call.finished',
         {
           ...identity,
-          status: signal.aborted ? 'aborted' : 'error',
+          // Only the exact abort reason is cancellation; a distinct
+          // failure under an aborted signal stays an error
+          // (slc/link.md §Abort).
+          status: isAbortFailure(error, signal) ? 'aborted' : 'error',
           error: normalizeErrorFull(error) ?? {
             name: 'Error',
             message: String(error),
           },
         },
         { turnId: currentTurnId, callId },
+        aborts,
       );
     if (inFlightPlayerKeys.has(playerKey)) {
       const error = new Error(
@@ -1310,6 +1429,7 @@ export const createPlaybookRuntime: PlaybookRuntimeFactory<
             : {}),
         },
         { turnId: currentTurnId, callId },
+        aborts,
       );
       return {
         roleId,
@@ -1335,64 +1455,70 @@ export const createPlaybookRuntime: PlaybookRuntimeFactory<
   const player = fromPromise<PlayerOutput, PlayerInput>(
     async ({ input, signal }) => {
       const combined = combineSignals(signal, currentSignal);
-      // XState starts invoked actors while publishing the entering snapshot.
-      // Yield through the runtime emission queue before crossing the player
-      // boundary so state trace/status always precede its call-start trace.
+      const settlementAborts = abortReasonClassifier(combined);
       try {
-        await flush();
-      } catch (error) {
-        latchControlPlaneError(error, combined);
-        throw error;
-      }
-      combined.throwIfAborted();
-
-      let { roleId, playerId, result } = await callPlayer(input, combined);
-      if (result.status === 'ok' && isEmptyFinalText(result.finalText)) {
-        // DR-028: an `ok` result whose finalText is missing, empty, or
-        // whitespace-only earns exactly one corrective re-ask — the same
-        // composed call repeated, traced by runPlayerCall as its own
-        // player-call pair, with the resume selection re-read from the
-        // token map the first result left (PBRT-38). An abort that lands
-        // between the two calls ends the turn without the re-ask (aborts
-        // are never retried), and a rejecting finish emission rejects
-        // `callPlayer` itself, so it never reaches this branch (PBRT-47).
+        // XState starts invoked actors while publishing the entering snapshot.
+        // Yield through the runtime emission queue before crossing the player
+        // boundary so state trace/status always precede its call-start trace.
+        combined.throwIfAborted();
+        try {
+          await flush(settlementAborts);
+        } catch (error) {
+          latchControlPlaneError(error, combined);
+          throw error;
+        }
         combined.throwIfAborted();
-        ({ roleId, playerId, result } = await callPlayer(input, combined));
-      }
-      if (result.status !== 'ok') {
-        throw new Error(
-          `${roleLabel(roleId)}${
-            playerId === undefined ? '' : ` (${playerId})`
-          } returned status "${result.status}"${
-            result.error ? `: ${result.error}` : ''
-          }`,
-        );
-      }
-      const finalText = result.finalText ?? '';
-      if (isEmptyFinalText(finalText)) {
-        throw new Error(
-          `${roleLabel(roleId)}${
-            playerId === undefined ? '' : ` (${playerId})`
-          } returned status "ok" with no finalText`,
-        );
-      }
-      combined.throwIfAborted();
 
-      try {
-        const prompt = buildAdjudicatorPrompt(input, finalText);
-        return parseAdjudication(
-          await callJudge(
-            prompt,
-            combined,
-            'player-output-adjudication',
-            input.stateId,
-          ),
-          input,
-          finalText,
-        );
-      } catch (error) {
-        latchControlPlaneError(error, combined);
-        throw error;
+        let { roleId, playerId, result } = await callPlayer(input, combined);
+        if (result.status === 'ok' && isEmptyFinalText(result.finalText)) {
+          // DR-028: an `ok` result whose finalText is missing, empty, or
+          // whitespace-only earns exactly one corrective re-ask — the same
+          // composed call repeated, traced by runPlayerCall as its own
+          // player-call pair, with the resume selection re-read from the
+          // token map the first result left (PBRT-38). An abort that lands
+          // between the two calls ends the turn without the re-ask (aborts
+          // are never retried), and a rejecting finish emission rejects
+          // `callPlayer` itself, so it never reaches this branch (PBRT-47).
+          combined.throwIfAborted();
+          ({ roleId, playerId, result } = await callPlayer(input, combined));
+        }
+        if (result.status !== 'ok') {
+          throw new Error(
+            `${roleLabel(roleId)}${
+              playerId === undefined ? '' : ` (${playerId})`
+            } returned status "${result.status}"${
+              result.error ? `: ${result.error}` : ''
+            }`,
+          );
+        }
+        const finalText = result.finalText ?? '';
+        if (isEmptyFinalText(finalText)) {
+          throw new Error(
+            `${roleLabel(roleId)}${
+              playerId === undefined ? '' : ` (${playerId})`
+            } returned status "ok" with no finalText`,
+          );
+        }
+        combined.throwIfAborted();
+
+        try {
+          const prompt = buildAdjudicatorPrompt(input, finalText);
+          return parseAdjudication(
+            await callJudge(
+              prompt,
+              combined,
+              'player-output-adjudication',
+              input.stateId,
+            ),
+            input,
+            finalText,
+          );
+        } catch (error) {
+          latchControlPlaneError(error, combined);
+          throw error;
+        }
+      } finally {
+        actorSettlementAborts.push(settlementAborts);
       }
     },
   );
@@ -1404,7 +1530,7 @@ export const createPlaybookRuntime: PlaybookRuntimeFactory<
       trackBoundaryCall(
         Promise.resolve(requirePorts().callPlaybook(request, signal)),
       ),
-    emitStarted: async (event) => {
+    emitStarted: async (event, aborts) => {
       playbookCallTurnIds.set(event.callId, currentTurnId);
       await emitTrace(
         'playbook.call.started',
@@ -1417,9 +1543,10 @@ export const createPlaybookRuntime: PlaybookRuntimeFactory<
           ...(currentTurnId === undefined ? {} : { turnId: currentTurnId }),
           callId: event.callId,
         },
+        aborts,
       );
     },
-    emitFinished: async (event) => {
+    emitFinished: async (event, aborts) => {
       const turnId = playbookCallTurnIds.get(event.callId);
       try {
         await emitTrace(
@@ -1434,23 +1561,32 @@ export const createPlaybookRuntime: PlaybookRuntimeFactory<
             ...(turnId === undefined ? {} : { turnId }),
             callId: event.callId,
           },
+          aborts,
         );
       } finally {
         playbookCallTurnIds.delete(event.callId);
       }
     },
     drain: flush,
-    bindResumeSignal: (signal) => {
+    bindResumeSignal: (signal, aborts) => {
       currentSignal = signal;
+      currentAborts = aborts ?? abortReasonClassifier(signal);
+    },
+    bindActorSettlement: (aborts) => {
+      actorSettlementAborts.push(aborts);
     },
-    onControlPlaneError: (error) => {
-      const signal = currentSignal;
-      if (!signal || !isAbortFailure(error, signal)) {
+    onControlPlaneError: (error, aborts) => {
+      if (
+        !aborts?.isAbortReason(error) &&
+        !currentAborts?.isAbortReason(error)
+      ) {
         controlPlaneError ??= error;
       }
     },
-    onBackgroundError: (error) => {
-      collectFailure(emissionFailures, error);
+    onBackgroundError: (error, aborts) => {
+      if (!aborts?.isAbortReason(error)) {
+        collectFailure(emissionFailures, error);
+      }
     },
   });
 
@@ -1458,10 +1594,27 @@ export const createPlaybookRuntime: PlaybookRuntimeFactory<
     actors: { player, playbook: nestedBridge.actorLogic },
   });
 
+  const consumeActorSettlementAborts = (
+    forSnapshot = false,
+  ): AbortReasonClassifier | undefined => {
+    const aborts = actorSettlementAborts.shift() ?? actorSettlementErrorAborts;
+    actorSettlementErrorAborts = undefined;
+    if (forSnapshot && aborts !== undefined) {
+      actorSettlementErrorAborts = aborts;
+      queueMicrotask(() => {
+        if (actorSettlementErrorAborts === aborts) {
+          actorSettlementErrorAborts = undefined;
+        }
+      });
+    }
+    return aborts;
+  };
+
   const inspect = (event: InspectionEvent): void => {
     if (event.type !== '@xstate.snapshot') return;
     if (actor === undefined || event.actorRef !== actor) return;
     if (suppressInspectionEmissions) return;
+    const settlementAborts = consumeActorSettlementAborts(true);
     try {
       const snapshot = event.snapshot as SnapshotFrom<typeof decideMachine>;
       const state = normalizePlaybookSnapshot(snapshot);
@@ -1482,6 +1635,7 @@ export const createPlaybookRuntime: PlaybookRuntimeFactory<
             topic: TELEMETRY_TOPIC,
             payload: describedFsmPayload,
           }),
+        settlementAborts,
       ).catch(() => undefined);
 
       const priorIds = new Set(previousState?.activeStateIds ?? []);
@@ -1507,6 +1661,7 @@ export const createPlaybookRuntime: PlaybookRuntimeFactory<
           tracePayload,
           { turnId: currentTurnId },
           (emissionPorts) => emissionPorts.emitStatus(message, data),
+          settlementAborts,
         ).catch(() => undefined);
       };
 
@@ -1552,7 +1707,7 @@ export const createPlaybookRuntime: PlaybookRuntimeFactory<
         );
       }
     } catch (error) {
-      latchInspectionError(error);
+      latchInspectionError(error, settlementAborts);
     }
   };
 
@@ -1571,6 +1726,16 @@ export const createPlaybookRuntime: PlaybookRuntimeFactory<
           }),
       inspect,
     });
+    // A synchronous FSM action throw errors the actor without any pending
+    // boundary await to observe it; unobserved, XState would surface it via
+    // reportUnhandledError as an uncaughtException. Observe it here: latch
+    // it as a control error while a turn signal is active (unless it is
+    // the abort reason itself), otherwise collect it with the emission
+    // failures (slc/link.md §Abort).
+    actor.subscribe({
+      error: (error) =>
+        latchInspectionError(error, consumeActorSettlementAborts()),
+    });
   };
 
   // PBRT-6: the single seam that stops this runtime's actor. Stopping a
@@ -1610,7 +1775,7 @@ export const createPlaybookRuntime: PlaybookRuntimeFactory<
     const state = normalizePlaybookSnapshot(snapshot, {
       pendingCall: nestedBridge.getPendingCall(),
     });
-    const pendingQuestions = pendingQuestionsFromContext(context);
+    const pendingQuestions = pendingQuestionsForState(state, context);
     if (
       pendingQuestions.length === 0 &&
       (snapshot.status === 'done' ||
@@ -1644,34 +1809,58 @@ export const createPlaybookRuntime: PlaybookRuntimeFactory<
     const pendingCall = nestedBridge.getPendingCall();
     const state = normalizePlaybookSnapshot(snapshot, { pendingCall });
     const context = snapshot.context as unknown as Record<string, unknown>;
-    if (signal?.aborted) {
-      return {
-        outcome: 'aborted',
-        state,
-        ...(signal.reason === undefined
-          ? {}
-          : {
-              error: normalizeErrorFull(signal.reason) ?? {
-                name: 'AbortError',
-                message: String(signal.reason),
-              },
-            }),
-      };
+    const abortedResult = (abortSignal: AbortSignal): PlaybookRunResult => ({
+      outcome: 'aborted',
+      state,
+      ...(abortSignal.reason === undefined
+        ? {}
+        : {
+            error: normalizeErrorFull(abortSignal.reason) ?? {
+              name: 'AbortError',
+              message: String(abortSignal.reason),
+            },
+          }),
+    });
+    if (snapshot.status === 'error') {
+      // An errored actor outranks a coincident abort unless the actor's
+      // error is the abort reason itself (slc/link.md §Abort).
+      const actorError = (snapshot as { error?: unknown }).error;
+      if (
+        actorError !== undefined &&
+        signal !== undefined &&
+        isAbortFailure(actorError, signal)
+      ) {
+        return abortedResult(signal);
+      }
+      throw (
+        actorError ?? new Error('decide runtime actor entered error status')
+      );
     }
+    // Terminal completion outranks a coincident abort (DR-036 §3): reporting
+    // 'aborted' over a completed machine would hide a terminal state that the
+    // next turn silently restarts, duplicating the workflow's side effects.
     if (snapshot.status === 'done') {
       const output = (snapshot as { output?: unknown }).output;
       if (output !== undefined) assertJsonSafe(output, 'terminal output');
+      const stateDescription = state.activeStateIds.includes('done')
+        ? STATE_DESCRIPTIONS.done
+        : state.activeStateIds.includes('reportedReviewFailure')
+          ? STATE_DESCRIPTIONS.reportedReviewFailure
+          : undefined;
+      if (stateDescription === undefined) {
+        throw new Error(
+          'decide runtime: completed actor has no authored final-state description',
+        );
+      }
       return {
         outcome: 'terminal',
         state,
+        stateDescription,
         ...(output === undefined ? {} : { output }),
       };
     }
-    if (snapshot.status === 'error') {
-      throw (
-        (snapshot as { error?: unknown }).error ??
-        new Error('decide runtime actor entered error status')
-      );
+    if (signal?.aborted) {
+      return abortedResult(signal);
     }
     if (state.activeStateIds.includes('failed')) {
       const error = normalizeErrorFull(context.lastError);
@@ -1741,6 +1930,9 @@ export const createPlaybookRuntime: PlaybookRuntimeFactory<
     judgeQueue.clear();
     actor = undefined;
     currentSignal = undefined;
+    currentAborts = undefined;
+    actorSettlementAborts.length = 0;
+    actorSettlementErrorAborts = undefined;
     currentTurnId = undefined;
     ports = undefined;
     sessionIdentity = undefined;
@@ -1866,7 +2058,7 @@ export const createPlaybookRuntime: PlaybookRuntimeFactory<
           playbookCall: playbookCallSequence,
         },
         state,
-        pendingBossQuestions: pendingQuestionsFromContext(context).map(
+        pendingBossQuestions: pendingQuestionsForState(state, context).map(
           (pending) => ({
             questionId: pending.questionId,
             asker: pending.asker,
@@ -1993,12 +2185,17 @@ export const createPlaybookRuntime: PlaybookRuntimeFactory<
       const turnId = ++turnSequence;
       currentTurnId = turnId;
       currentSignal = turn.signal;
+      currentAborts = abortReasonClassifier(turn.signal);
       controlPlaneError = undefined;
       let result: PlaybookRunResult = resultForSnapshot(turn.signal);
       let settlement: unknown = result;
       const failures: unknown[] = [];
       try {
         await emitTrace('boss.input.received', { text: turn.text }, { turnId });
+        // A boundary entered aborted records the attempted input, then refuses
+        // delivery before deterministic mapping or the classifier can perform
+        // any host-visible work (DR-036 §5).
+        turn.signal.throwIfAborted();
         if (turn.text.trim().length === 0) {
           const state = currentState();
           result = { outcome: 'no-action', state };
@@ -2031,15 +2228,18 @@ export const createPlaybookRuntime: PlaybookRuntimeFactory<
         };
       } catch (error) {
         const primaryError = controlPlaneError;
+        // Only a rejection that is the exact abort reason settles as the
+        // cancellation; a distinct failure observed while the signal is
+        // aborted remains a control error (slc/link.md §Abort).
         if (primaryError !== undefined) {
           collectFailure(failures, primaryError);
-        } else if (!turn.signal.aborted) {
+        } else if (!isAbortFailure(error, turn.signal)) {
           collectFailure(failures, error);
         }
         const state = currentState();
         const effectiveError = primaryError ?? error;
         result =
-          turn.signal.aborted && primaryError === undefined
+          isAbortFailure(error, turn.signal) && primaryError === undefined
             ? resultForSnapshot(turn.signal)
             : {
                 outcome: 'failed',
@@ -2060,13 +2260,13 @@ export const createPlaybookRuntime: PlaybookRuntimeFactory<
       } catch (error) {
         const primaryError = controlPlaneError;
         const effectiveError = primaryError ?? error;
-        collectFailure(failures, effectiveError);
+        // A drain rejection that is the exact abort reason evidences the
+        // cancellation, not a control-plane failure (slc/link.md §Abort).
+        const drainAborted = isAbortFailure(effectiveError, turn.signal);
+        if (!drainAborted) collectFailure(failures, effectiveError);
         const state = currentState();
         result = {
-          outcome:
-            turn.signal.aborted && primaryError === undefined
-              ? 'aborted'
-              : 'failed',
+          outcome: drainAborted ? 'aborted' : 'failed',
           state,
           error: normalizeErrorFull(effectiveError) ?? {
             name: 'Error',
@@ -2075,18 +2275,28 @@ export const createPlaybookRuntime: PlaybookRuntimeFactory<
         };
         settlement = { ...result, ...stateIdentity(state) };
       }
-      currentSignal = undefined;
       try {
         await emitTrace('boss.input.settled', settlement, { turnId });
       } catch (error) {
-        collectFailure(failures, error);
+        // A settlement-trace rejection that is the exact abort reason also
+        // evidences the cancellation (slc/link.md §Abort).
+        if (!isAbortFailure(error, turn.signal)) {
+          collectFailure(failures, error);
+        }
       }
       try {
         await flush();
       } catch (error) {
-        collectFailure(failures, error);
+        // A late flush rejection that is the exact abort reason likewise
+        // evidences the cancellation; the settled result already labels
+        // the turn aborted then (slc/link.md §Abort).
+        if (!isAbortFailure(error, turn.signal)) {
+          collectFailure(failures, error);
+        }
       } finally {
         const primaryError = controlPlaneError;
+        currentSignal = undefined;
+        currentAborts = undefined;
         currentTurnId = undefined;
         controlPlaneError = undefined;
         if (primaryError !== undefined) throw primaryError;
@@ -2122,6 +2332,7 @@ export const createPlaybookRuntime: PlaybookRuntimeFactory<
       }
       currentTurnId = playbookCallTurnIds.get(callId);
       currentSignal = signal;
+      currentAborts = abortReasonClassifier(signal);
       controlPlaneError = undefined;
       let runResult: PlaybookRunResult | undefined;
       let operationError: unknown;
@@ -2148,13 +2359,56 @@ export const createPlaybookRuntime: PlaybookRuntimeFactory<
       } catch (error) {
         drainError = error;
       }
-      const failure = controlPlaneError ?? drainError ?? operationError;
+      const aborts = currentAborts ?? abortReasonClassifier(signal);
+      // The control latch has already classified its failure as distinct
+      // under the operation that owned it. Only still-unclassified drain and
+      // operation candidates may be cancellation evidence for this resume.
+      const controlFailure = controlPlaneError;
+      const drainAbort =
+        controlFailure === undefined &&
+        drainError !== undefined &&
+        aborts.isAbortReason(drainError);
+      const operationAbort =
+        controlFailure === undefined &&
+        operationError !== undefined &&
+        aborts.isAbortReason(operationError);
+      const abortEvidence =
+        (drainAbort ? drainError : undefined) ??
+        (operationAbort ? operationError : undefined);
+      const failure =
+        controlFailure ??
+        (drainAbort ? undefined : drainError) ??
+        (operationAbort ? undefined : operationError);
       currentSignal = undefined;
+      currentAborts = undefined;
       currentTurnId = undefined;
       controlPlaneError = undefined;
       if (failure !== undefined) throw failure;
+      if (
+        abortEvidence !== undefined &&
+        runResult?.outcome !== 'terminal' &&
+        runResult?.outcome !== 'suspended'
+      ) {
+        const state = currentState();
+        runResult = {
+          outcome: 'aborted',
+          state,
+          error: normalizeErrorFull(abortEvidence) ?? {
+            name: 'AbortError',
+            message: String(abortEvidence),
+          },
+        };
+      }
       if (runResult === undefined) {
-        throw new Error('decide runtime: playbook resume produced no result');
+        if (signal.aborted) {
+          // Every candidate was the abort's own evidence: settle on the
+          // machine's state under the aborted boundary signal (DR-036 §4).
+          runResult = resultForSnapshot(signal);
+        } else {
+          throw new Error(
+            'decide runtime: playbook resume produced no result',
+          );
+        }
       }
       return runResult;
     },
@@ -2210,6 +2464,9 @@ export const createPlaybookRuntime: PlaybookRuntimeFactory<
           judgeQueue.clear();
           actor = undefined;
           currentSignal = undefined;
+          currentAborts = undefined;
+          actorSettlementAborts.length = 0;
+          actorSettlementErrorAborts = undefined;
           currentTurnId = undefined;
           ports = undefined;
           sessionIdentity = undefined;
@@ -2224,6 +2481,12 @@ export const createPlaybookRuntime: PlaybookRuntimeFactory<
       })();
       return disposalPromise;
     },
+
+    // @internal — test-only parity with the shared factory's bridge escape
+    // hatch. This is hidden by the PlaybookRuntime return type.
+    _getNestedBridge() {
+      return nestedBridge;
+    },
   };
 };
 
@@ -2237,6 +2500,7 @@ export const _internal = {
   parseAdjudication,
   combineSignals,
   pendingQuestionsFromContext,
+  pendingQuestionsForState,
   normalizeErrorCompact,
   normalizeErrorFull,
   STATE_DESCRIPTIONS,