@akagilnc/pi-workflow-roles 0.1.2139 → 0.1.2146

package/src/navigator-attendance.ts

@@ -8,6 +8,7 @@ import { createAssistantMessageEventStream } from "@earendil-works/pi-ai";
 import { Type, type Static } from "typebox";
 import { Value } from "typebox/value";
 
+import { seatFallbackBaseStatus } from "./engine-labor-fallback.ts";
 import {
   NAVIGATOR_INVOCATION_ENTRY,
   mintNavigatorInvocationId,
@@ -461,6 +462,18 @@ export type NavigatorCandidateSelection = {
   readonly matchedToSettlement: boolean;
 };
 
+/** Match route-playbook statuses against settlement, treating seat-fallback taint as base-equivalent. */
+function statusListMatchesSettlement(
+  candidateStatuses: readonly string[],
+  settlementStatus: string,
+): boolean {
+  if (candidateStatuses.includes(settlementStatus)) return true;
+  const settlementBase = seatFallbackBaseStatus(settlementStatus);
+  return candidateStatuses.some(
+    (status) => seatFallbackBaseStatus(status) === settlementBase,
+  );
+}
+
 export function selectNavigatorCandidate(
   candidates: readonly NavigatorCandidate[],
   settlement: NavigatorSettlement,
@@ -477,7 +490,9 @@ export function selectNavigatorCandidate(
   if (rolePhaseMatched.length > 0) {
     if (settlement.status !== undefined) {
       const statusSpecific = rolePhaseMatched.find(
-        (candidate) => candidate.matches?.statuses?.includes(settlement.status!) === true,
+        (candidate) =>
+          candidate.matches?.statuses !== undefined &&
+          statusListMatchesSettlement(candidate.matches.statuses, settlement.status!),
       );
       if (statusSpecific !== undefined) {
         return { candidate: statusSpecific, matchedToSettlement: true };

package/src/package-contracts/judge-output.ts

@@ -1,5 +1,12 @@
 /** Package-owned Judge output leaf — no role registration surface. */
 
+import {
+  seatFallbackBaseStatus,
+  seatFallbackStatusHasLawfulEvidence,
+  type SeatFallbackTaintedStatus,
+  type WithEngineLaborFallback,
+} from "../engine-labor-fallback.ts";
+
 export const JUDGE_OUTPUT_TOOL_NAME = "ak_judge_output";
 export const JUDGE_ACCEPTED_TEXT = "Judge verdict accepted";
 
@@ -10,7 +17,7 @@ export type JudgeClass = {
   disposition: string;
 };
 
-export type JudgeVerdict =
+type JudgeVerdictClean =
   | { judgeStatus: "converged"; note?: string; evidence?: unknown }
   | {
     judgeStatus: "continue";
@@ -26,6 +33,11 @@ export type JudgeVerdict =
     evidence?: unknown;
   };
 
+/** Clean submission shape or seat-fallback tainted accepted receipt (ADR 0071). */
+export type JudgeVerdict =
+  | JudgeVerdictClean
+  | WithEngineLaborFallback<JudgeVerdictClean>;
+
 export function validateAcceptedJudgeDetails(verdict: unknown): JudgeVerdict {
   if (verdict === null || typeof verdict !== "object" || Array.isArray(verdict)) throw new Error("Judge verdict has no execution discriminator");
   let judgeStatus: unknown;
@@ -34,6 +46,17 @@ export function validateAcceptedJudgeDetails(verdict: unknown): JudgeVerdict {
   } catch {
     throw new Error("Judge verdict has no execution discriminator");
   }
-  if (["converged", "continue", "escalate"].includes(String(judgeStatus))) return verdict as JudgeVerdict;
+  if (typeof judgeStatus !== "string") {
+    throw new Error("Judge verdict has no execution discriminator");
+  }
+  const base = seatFallbackBaseStatus(judgeStatus) as SeatFallbackTaintedStatus<
+    "converged" | "continue" | "escalate"
+  >;
+  if (
+    ["converged", "continue", "escalate"].includes(base) &&
+    seatFallbackStatusHasLawfulEvidence(judgeStatus, verdict)
+  ) {
+    return verdict as JudgeVerdict;
+  }
   throw new Error("Judge verdict has no execution discriminator");
 }

package/src/package-contracts/reviewer-output.ts

@@ -1,5 +1,11 @@
 /** Package-owned Reviewer intent and runtime-receipt leaves — no role registration surface. */
 
+import {
+  seatFallbackBaseStatus,
+  seatFallbackStatusHasLawfulEvidence,
+  type SeatFallbackTaintedStatus,
+  type WithEngineLaborFallback,
+} from "../engine-labor-fallback.ts";
 import type { ReviewerAcceptedEvidence, ReviewerFailureClassification, ReviewerWorkspaceDisposition } from "../reviewer-execution-ledger.ts";
 
 export const REVIEWER_OUTPUT_TOOL_NAME = "ak_reviewer_output";
@@ -28,7 +34,7 @@ export type RuntimeReviewerAcceptedBatch = Readonly<{
 }>;
 /** Honest Spec-child disposition on the receipt face. */
 export type RuntimeReviewerSpecDisposition = "launched" | "skipped-missing";
-export type RuntimeReviewerReceiptV2 = Readonly<{
+type RuntimeReviewerReceiptV2Clean = Readonly<{
   version: 2;
   status: "completed" | "refused";
   diagnostic?: string;
@@ -45,6 +51,14 @@ export type RuntimeReviewerReceiptV2 = Readonly<{
     target?: ReviewerAcceptedEvidence["target"];
   }>;
 }>;
+/** Clean runtime receipt or seat-fallback tainted accepted receipt (ADR 0071). */
+export type RuntimeReviewerReceiptV2 =
+  | RuntimeReviewerReceiptV2Clean
+  | WithEngineLaborFallback<
+      Omit<RuntimeReviewerReceiptV2Clean, "status"> & {
+        status: SeatFallbackTaintedStatus<"completed" | "refused">;
+      }
+    >;
 
 function isRecord(value: unknown): value is Record<string, unknown> {
   return typeof value === "object" && value !== null && !Array.isArray(value);
@@ -63,6 +77,11 @@ export function validateReviewerIntent(output: unknown): ReviewerIntent {
 
 /** Validate runtime-owned facts at their real identity seams (target pins + plain text). */
 export function validateRuntimeReviewerReceipt(output: unknown): RuntimeReviewerReceiptV2 {
+  const status = read(output, "status");
+  // ADR 0071: tainted top-level status requires latch-shaped engineLaborFallback evidence.
+  if (typeof status === "string" && !seatFallbackStatusHasLawfulEvidence(status, output)) {
+    throw new Error("Reviewer receipt has no recognized execution discriminator");
+  }
   const acceptedBatch = read(output, "acceptedBatch");
   const identities = read(output, "identities");
   const construction = read(identities, "construction");
@@ -129,7 +148,9 @@ export function validateRuntimeReviewerReceipt(output: unknown): RuntimeReviewer
 export function projectReviewerIntentToReceipt(intentValue: unknown, receiptValue: unknown): RuntimeReviewerReceiptV2 {
   const intent = validateReviewerIntent(intentValue);
   const receipt = validateRuntimeReviewerReceipt(receiptValue);
-  if (receipt.status !== intent.status || (intent.status === "completed" ? receipt.diagnostic !== undefined : receipt.diagnostic !== intent.diagnostic)) {
+  const receiptStatus = String(receipt.status);
+  const receiptBase = seatFallbackBaseStatus(receiptStatus);
+  if (receiptBase !== intent.status || (intent.status === "completed" ? receipt.diagnostic !== undefined : receipt.diagnostic !== intent.diagnostic)) {
     throw new Error("Reviewer intent and runtime receipt disagree");
   }
   return receipt;

package/src/package-contracts/terminating-tools.ts

@@ -25,6 +25,10 @@ import {
   type RuntimeReviewerReceiptV2,
 } from "./reviewer-output.ts";
 import { isAuditEscalationResult } from "../audit-escalation.ts";
+import {
+  seatFallbackBaseStatus,
+  seatFallbackStatusHasLawfulEvidence,
+} from "../engine-labor-fallback.ts";
 import { DOCTOR_OUTPUT_TOOL_NAME, validateDoctorSubmissionShape, validateRecordedDoctorOutput, type DoctorOutput, type DoctorSubmission } from "../doctor-contracts.ts";
 import { MERGER_ACCEPTED_TEXT, MERGER_OUTPUT_TOOL_NAME, validateMergerOutput, type MergerOutput } from "../merger-contracts.ts";
 import {
@@ -159,10 +163,19 @@ export function validateAcceptedDetails(
     [MERGER_OUTPUT_TOOL_NAME]: ["completed", "escalate"],
   };
   const collectorDiscriminator = toolName === COLLECTOR_OUTPUT_TOOL && Array.isArray(candidate?.groups);
+  const baseDiscriminator = typeof discriminator === "string" ? seatFallbackBaseStatus(discriminator) : discriminator;
+  // ADR 0071: `-by-fallback` is not independently lawful — require latch-shaped evidence.
+  const taintedWithoutEvidence =
+    typeof discriminator === "string" &&
+    !seatFallbackStatusHasLawfulEvidence(discriminator, details);
   const runtimeBindingMissing =
-    (toolName === DOCTOR_OUTPUT_TOOL_NAME && discriminator === "completed" && !(candidate?.cost !== null && typeof candidate?.cost === "object")) ||
+    (toolName === DOCTOR_OUTPUT_TOOL_NAME && baseDiscriminator === "completed" && !(candidate?.cost !== null && typeof candidate?.cost === "object")) ||
     (toolName === REVIEWER_OUTPUT_TOOL_NAME && candidate?.version !== 2);
-  if (runtimeBindingMissing || (!collectorDiscriminator && (typeof discriminator !== "string" || !lawfulStatuses[toolName].includes(discriminator)))) {
+  if (
+    taintedWithoutEvidence ||
+    runtimeBindingMissing ||
+    (!collectorDiscriminator && (typeof discriminator !== "string" || !lawfulStatuses[toolName].includes(baseDiscriminator as string)))
+  ) {
     throw new AcceptedDetailsContractError("terminating receipt has no recognized execution discriminator");
   }
   try {
@@ -196,12 +209,14 @@ export function validateAcceptedLifecycle(
   const details = validateAcceptedDetails(toolName, detailsValue);
   if (toolName === DOCTOR_OUTPUT_TOOL_NAME) {
     const testimony = validateDoctorSubmissionShape(argumentsValue);
-    if (testimony.status === "refused") {
+    if (seatFallbackBaseStatus(String(testimony.status)) === "refused") {
       if (!deepEqual(testimony, details)) throw new Error("accepted tool lifecycle details mismatch");
       return details;
     }
-    const receipt = details as DoctorOutput;
-    if (receipt.status !== "completed") throw new Error("accepted tool lifecycle details mismatch");
+    const receipt = details as DoctorOutput & { cost?: unknown };
+    if (seatFallbackBaseStatus(String(receipt.status)) !== "completed") {
+      throw new Error("accepted tool lifecycle details mismatch");
+    }
     const { cost: _runtimeCost, ...projected } = receipt;
     if (!deepEqual(testimony, projected)) throw new Error("accepted tool lifecycle details mismatch");
     return details;
@@ -226,7 +241,8 @@ export function acceptedFacts(toolName: TerminatingToolName, details: AcceptedDe
     case JUDGE_OUTPUT_TOOL_NAME: return { status: (details as { judgeStatus: string }).judgeStatus };
     case MERGER_OUTPUT_TOOL_NAME: {
       const output = details as unknown as Record<string, unknown>;
-      return { status: output.status as string, ...(output.status === "completed" && typeof output.mergeCommitId === "string" ? { commit: output.mergeCommitId } : {}) };
+      const status = output.status as string;
+      return { status, ...(seatFallbackBaseStatus(status) === "completed" && typeof output.mergeCommitId === "string" ? { commit: output.mergeCommitId } : {}) };
     }
     case COLLECTOR_OUTPUT_TOOL:
       return { status: "collected" };

package/src/package-contracts/worker-output.ts

@@ -15,16 +15,24 @@ export type {
 } from "./fixer-output.ts";
 export { fixerPrerequisiteSchema, fixerPrerequisitesSchema, parseFixerPrerequisites, validateFixerPrerequisites } from "./fixer-packet.ts";
 export type { FixerInvocationInput, FixerPrerequisite } from "./fixer-packet.ts";
+import type { WithEngineLaborFallback } from "../engine-labor-fallback.ts";
 import { validateFixerOutput, type FixerOutput } from "./fixer-output.ts";
 
 export const CODER_OUTPUT_TOOL_NAME = "ak_coder_output";
 export const CODER_ACCEPTED_TEXT = "Coder report accepted";
 export type WorkerRoleLabel = "Coder" | "Fixer";
-export type CoderOutput =
+type CoderOutputClean =
   | { status: "planned"; report: string }
   | { status: "completed" | "refused"; report: string }
   | { status: "unfinished"; report: string; remainingScope: string; reason?: string };
-export type WorkerOutput = CoderOutput | FixerOutput;
+/** Clean submission shape or seat-fallback tainted accepted receipt (ADR 0071). */
+export type CoderOutput =
+  | CoderOutputClean
+  | WithEngineLaborFallback<CoderOutputClean>;
+export type WorkerOutput =
+  | CoderOutput
+  | FixerOutput
+  | WithEngineLaborFallback<FixerOutput>;
 
 export function validateAcceptedCoderDetails(output: unknown): CoderOutput {
   return output as CoderOutput;

package/src/public-cli/settlement.ts

@@ -34,7 +34,11 @@ import {
   COLLECTOR_WAIT_TOOL,
 } from "../collector-ledger.ts";
 import { ENGINE_DETOUR_TOOL_NAME } from "../engine-detour.ts";
-import { readEngineLaborFallbackFieldFrom } from "../engine-labor-fallback.ts";
+import {
+  readEngineLaborFallbackFieldFrom,
+  seatFallbackBaseStatus,
+  seatFallbackStatusHasLawfulEvidence,
+} from "../engine-labor-fallback.ts";
 import {
   JUDGE_OUTPUT_TOOL_NAME,
   type JudgeVerdict,
@@ -1059,7 +1063,7 @@ function auditNoReceiptDecisiveFact(candidate: object): Record<string, unknown>
 
 function judgeDecisiveFacts(
   verdict: object,
-  judgeStatus: JudgeVerdict["judgeStatus"],
+  judgeStatus: string,
 ): Record<string, unknown> {
   const facts: Record<string, unknown> = {
     judgeStatus,
@@ -1067,7 +1071,8 @@ function judgeDecisiveFacts(
     // #380: spread sole-built field; do not re-key here (S1).
     ...readEngineLaborFallbackFieldFrom(verdict),
   };
-  if (judgeStatus === "continue") {
+  const statusBase = seatFallbackBaseStatus(judgeStatus);
+  if (statusBase === "continue") {
     const fix = safelyRead(verdict, "fix");
     if (fix.readable && isRecord(fix.value)) {
       const summary = safelyRead(fix.value, "summary");
@@ -1093,7 +1098,7 @@ function judgeDecisiveFacts(
       }
     }
   }
-  if (judgeStatus === "escalate") {
+  if (statusBase === "escalate") {
     const gate = safelyRead(verdict, "decisionGate");
     if (gate.readable && isRecord(gate.value)) {
       const question = safelyRead(gate.value, "question");
@@ -1118,10 +1123,14 @@ function coderDecisiveFacts(output: CoderOutput): Record<string, unknown> {
   const status = safelyRead(candidate, "status");
   const facts: Record<string, unknown> = {};
   if (status.readable && typeof status.value === "string") facts.coderStatus = status.value;
+  const statusBase =
+    status.readable && typeof status.value === "string"
+      ? seatFallbackBaseStatus(status.value)
+      : undefined;
   const remainingScope = safelyRead(candidate, "remainingScope");
-  if (status.readable && status.value === "unfinished" && remainingScope.readable && typeof remainingScope.value === "string") facts.remainingScope = remainingScope.value;
+  if (statusBase === "unfinished" && remainingScope.readable && typeof remainingScope.value === "string") facts.remainingScope = remainingScope.value;
   const reason = safelyRead(candidate, "reason");
-  if (status.readable && status.value === "unfinished" && reason.readable && typeof reason.value === "string" && reason.value.trim().length > 0) {
+  if (statusBase === "unfinished" && reason.readable && typeof reason.value === "string" && reason.value.trim().length > 0) {
     facts.reason = reason.value;
   }
   const report = safelyRead(candidate, "report");
@@ -1134,14 +1143,18 @@ function fixerDecisiveFacts(output: FixerOutput): Record<string, unknown> {
   const status = safelyRead(candidate, "status");
   const facts: Record<string, unknown> = {};
   if (status.readable && typeof status.value === "string") facts.fixerStatus = status.value;
+  const statusBase =
+    status.readable && typeof status.value === "string"
+      ? seatFallbackBaseStatus(status.value)
+      : undefined;
   const remainingScope = safelyRead(candidate, "remainingScope");
-  if (status.readable && (status.value === "unfinished" || status.value === "refused") && remainingScope.readable && typeof remainingScope.value === "string") facts.remainingScope = remainingScope.value;
+  if ((statusBase === "unfinished" || statusBase === "refused") && remainingScope.readable && typeof remainingScope.value === "string") facts.remainingScope = remainingScope.value;
   const reason = safelyRead(candidate, "reason");
-  if (status.readable && status.value === "unfinished" && reason.readable && typeof reason.value === "string" && reason.value.trim().length > 0) {
+  if (statusBase === "unfinished" && reason.readable && typeof reason.value === "string" && reason.value.trim().length > 0) {
     facts.reason = reason.value;
   }
   const blockerRead = safelyRead(candidate, "blocker");
-  if (status.readable && status.value === "refused" && blockerRead.readable && isRecord(blockerRead.value)) {
+  if (statusBase === "refused" && blockerRead.readable && isRecord(blockerRead.value)) {
     const cause = safelyRead(blockerRead.value, "cause");
     if (cause.readable && typeof cause.value === "string") facts.blockerCause = cause.value;
     const prerequisiteId = safelyRead(blockerRead.value, "prerequisiteId");
@@ -1223,7 +1236,11 @@ function doctorDecisiveFacts(output: DoctorOutput): Record<string, unknown> {
   const status = safelyRead(candidate, "status");
   const facts: Record<string, unknown> = { ...auditNoReceiptDecisiveFact(candidate) };
   if (status.readable && typeof status.value === "string") facts.doctorStatus = status.value;
-  if (status.readable && status.value === "refused") {
+  const statusBase =
+    status.readable && typeof status.value === "string"
+      ? seatFallbackBaseStatus(status.value)
+      : undefined;
+  if (statusBase === "refused") {
     const reason = safelyRead(candidate, "reason");
     if (reason.readable && reason.value !== undefined) facts.reason = reason.value;
     const missing = safelyRead(candidate, "missingEvidence");
@@ -1277,7 +1294,11 @@ function reviewerDecisiveFacts(
     facts.specDisposition = specDisposition.value;
   }
   const diagnostic = safelyRead(candidate, "diagnostic");
-  if (status.readable && status.value === "refused" && diagnostic.readable) {
+  const statusBase =
+    status.readable && typeof status.value === "string"
+      ? seatFallbackBaseStatus(status.value)
+      : undefined;
+  if (statusBase === "refused" && diagnostic.readable) {
     facts.diagnosticPresent = typeof diagnostic.value === "string" && diagnostic.value.trim().length > 0;
   }
   return facts;
@@ -1980,9 +2001,12 @@ export function extractJudgeRoleOutcome(
     // must not become a second verdict-shape gate (ADR 0040).
     if (!isRecord(details)) continue;
     const statusRead = safelyRead(details, "judgeStatus");
-    if (!statusRead.readable) continue;
+    if (!statusRead.readable || typeof statusRead.value !== "string") continue;
     const judgeStatus = statusRead.value;
-    if (judgeStatus !== "converged" && judgeStatus !== "continue" && judgeStatus !== "escalate") continue;
+    const statusBase = seatFallbackBaseStatus(judgeStatus);
+    if (statusBase !== "converged" && statusBase !== "continue" && statusBase !== "escalate") continue;
+    // ADR 0071: `-by-fallback` without latch-shaped evidence is not a lawful terminal.
+    if (!seatFallbackStatusHasLawfulEvidence(judgeStatus, details)) continue;
     return {
       kind: "accepted",
       role: "judge",
@@ -2985,13 +3009,19 @@ async function settleLawfulDoctorTerminalResult(
   const extracted = extractDoctorRoleOutcome(entries);
   if (extracted === undefined) return undefined;
   // Bind completed receipt case identity to the admitted Issue evidence case.
+  // Seat-fallback may taint status; base semantics still require case binding.
   if (
     extracted.output !== undefined &&
-    extracted.output.status === "completed"
+    seatFallbackBaseStatus(String(extracted.output.status)) === "completed"
   ) {
+    const completedCase = (
+      extracted.output as {
+        case: { issueNumber: number; runsPath: string };
+      }
+    ).case;
     if (
-      extracted.output.case.issueNumber !== admitted.caseIdentity.issueNumber ||
-      extracted.output.case.runsPath !== admitted.caseIdentity.runsPath
+      completedCase.issueNumber !== admitted.caseIdentity.issueNumber ||
+      completedCase.runsPath !== admitted.caseIdentity.runsPath
     ) {
       const error = new Error(
         "Doctor receipt case identity does not match admitted case identity",
@@ -3331,7 +3361,11 @@ function mergerDecisiveFacts(output: MergerOutput): Record<string, unknown> {
   const attemptId = safelyRead(candidate, "attemptId");
   if (status.readable && typeof status.value === "string") facts.mergerStatus = status.value;
   if (attemptId.readable && attemptId.value !== undefined) facts.attemptId = attemptId.value;
-  const decisiveKey = status.readable && status.value === "completed" ? "mergeCommitId" : "diagnosis";
+  const statusBase =
+    status.readable && typeof status.value === "string"
+      ? seatFallbackBaseStatus(status.value)
+      : undefined;
+  const decisiveKey = statusBase === "completed" ? "mergeCommitId" : "diagnosis";
   const decisive = safelyRead(candidate, decisiveKey);
   if (decisive.readable && decisive.value !== undefined) facts[decisiveKey] = decisive.value;
   return facts;

package/src/role-runtime.ts

@@ -27,6 +27,8 @@ import {
   createEngineLaborFallbackLatch,
   installActivationEngineLaborFallbackLatch,
   restoreEngineLaborFallbackFromSessionEntries,
+  seatFallbackBaseStatus,
+  seatFallbackStatusHasLawfulEvidence,
 } from "./engine-labor-fallback.ts";
 import { createReceiptDeliveryPolicy, NO_RECEIPT_LIFECYCLE_ENTRY_TYPE, RECEIPT_DELIVERY_PROMPT } from "./receipt-delivery-policy.ts";
 import { createOAuthKeepalive, type OAuthKeepaliveOptions } from "./oauth-keepalive.ts";
@@ -564,7 +566,12 @@ export function publicNavigatorSettlement(role: string, phase: NavigatorPhase, e
   const status = typeof details.status === "string"
     ? details.status
     : typeof details.judgeStatus === "string" ? details.judgeStatus : undefined;
-  if (status === "escalate") {
+  // ADR 0071: `-by-fallback` without latch-shaped evidence is not a lawful terminal.
+  if (status !== undefined && !seatFallbackStatusHasLawfulEvidence(status, event.details)) {
+    return undefined;
+  }
+  // Seat-fallback taint keeps escalate semantics (base) while preserving the polluted token.
+  if (status !== undefined && seatFallbackBaseStatus(status) === "escalate") {
     return { kind: "human_decision", role, phase, status };
   }
   return { kind: "accepted", role, phase, ...(status === undefined ? {} : { status }) };

package/src/taishi-metric-families/acceptance-success-rework.ts

@@ -9,6 +9,7 @@
  * - success rate den = success-eligible accepted legs (no-receipt out; planned out)
  * - planned = plan-duty acceptance; never success numerator or denominator
  */
+import { seatFallbackBaseStatus } from "../engine-labor-fallback.ts";
 import type { TaishiReadableRunFacts, TaishiRunTerminalFace } from "../taishi-ledger.ts";
 import { medianNumber } from "../taishi-median.ts";
 import type { TaishiMetricFamilyModule } from "../taishi-metric-family.ts";
@@ -185,8 +186,10 @@ function mapTerminal(
     };
   }
 
+  // Seat-fallback taint is visible on the label; acceptance/success use base semantics.
+  const statusBase = seatFallbackBaseStatus(status);
   const acceptedSet = ACCEPTED_STATUS[role];
-  if (acceptedSet === undefined || !acceptedSet.has(status)) {
+  if (acceptedSet === undefined || !acceptedSet.has(statusBase)) {
     return {
       terminalLabel: status,
       accepted: false,
@@ -196,9 +199,9 @@ function mapTerminal(
     };
   }
 
-  const plannedDuty = WORKER_ROLES.has(role) && status === "planned";
+  const plannedDuty = WORKER_ROLES.has(role) && statusBase === "planned";
   const successSet = SUCCESS_STATUS[role] ?? new Set<string>();
-  const success = !plannedDuty && successSet.has(status);
+  const success = !plannedDuty && successSet.has(statusBase);
   const successEligible = !plannedDuty;
 
   return {