@akagilnc/pi-workflow-roles 0.1.2383 → 0.1.2397

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

@@ -168,7 +168,7 @@ export function validateAcceptedDetails(
     [COLLECTOR_OUTPUT_TOOL]: [],
     [DOCTOR_OUTPUT_TOOL_NAME]: ["completed", "refused"],
     [MERGER_OUTPUT_TOOL_NAME]: ["completed", "escalate"],
-    [NOTARY_OUTPUT_TOOL_NAME]: ["pass", "bounce", "incomplete"],
+    [NOTARY_OUTPUT_TOOL_NAME]: ["pass", "bounce"],
   };
   const collectorDiscriminator = toolName === COLLECTOR_OUTPUT_TOOL && Array.isArray(candidate?.groups);
   const baseDiscriminator = typeof discriminator === "string" ? seatFallbackBaseStatus(discriminator) : discriminator;

package/src/public-cli/doctor-run.ts

@@ -21,7 +21,6 @@ import {
 } from "./one-shot-dispatch.ts";
 import {
   presentStructuralRejection,
-  trySettleComplianceAuditIncompleteTerminalResult,
   trySettleDoctorTerminalResult,
 } from "./settlement.ts";
 import type { CliIo } from "./cli-io.ts";
@@ -122,7 +121,6 @@ export async function runPublicDoctor(
       trySettle: trySettleDoctorTerminalResult,
       shouldPresentSettled: (terminal) =>
         isLawfulTypedTerminalOutcome(terminal.roleOutcome),
-      trySettleSecondary: trySettleComplianceAuditIncompleteTerminalResult,
     },
     ...(env.engine === undefined ? {} : { effectiveEngine: env.engine }),
   });

package/src/public-cli/fixer-run.ts

@@ -70,7 +70,6 @@ import {
   explicitInternalKnownFailureClassificationInput,
   settleFailureTerminalResult,
   trySettleFixerTerminalResult,
-  trySettleComplianceAuditIncompleteTerminalResult,
 } from "./settlement.ts";
 import type { CliIo } from "./cli-io.ts";
 import type {
@@ -390,20 +389,6 @@ async function dispatchAdmittedFixer(input: {
       };
     }
 
-    const auditIncomplete = await trySettleComplianceAuditIncompleteTerminalResult(admitted);
-    if (auditIncomplete !== undefined) {
-      await markRunTerminal(admitted.runDirectory).catch(() => undefined);
-      if (auditIncomplete.roleOutcome.kind === "failure") {
-        presentFailureTerminal(auditIncomplete, io);
-      } else {
-        io.stdout(formatTerminalResult(auditIncomplete));
-      }
-      return {
-        exitCode: exitCodeForTerminalOutcome(auditIncomplete.roleOutcome),
-        admitted,
-        terminal: auditIncomplete,
-      };
-    }
 
     const credentialFailure = postRunMissingCredentialFailure(
       result,

package/src/public-cli/judge-run.ts

@@ -64,7 +64,6 @@ import {
   readEngineDetourInfrastructureFailure,
   settleJudgeFailureTerminalResult,
   trySettleJudgeTerminalResult,
-  trySettleComplianceAuditIncompleteTerminalResult,
 } from "./settlement.ts";
 import type { CliIo } from "./cli-io.ts";
 import type {
@@ -370,21 +369,6 @@ async function dispatchAdmittedJudge(input: {
       };
     }
 
-    const auditIncomplete = await trySettleComplianceAuditIncompleteTerminalResult(admitted);
-    if (auditIncomplete !== undefined) {
-      await markRunTerminal(admitted.runDirectory).catch(() => undefined);
-      if (auditIncomplete.roleOutcome.kind === "failure") {
-        presentFailureTerminal(auditIncomplete, io);
-      } else {
-        io.stdout(formatTerminalResult(auditIncomplete));
-      }
-      return {
-        exitCode: exitCodeForTerminalOutcome(auditIncomplete.roleOutcome),
-        admitted,
-        terminal: auditIncomplete,
-      };
-    }
-
     // Prefer engine-detour infrastructure failure already on the session principal
     // over a later secondary provider-stop after abort (#357 T2 / collector-isomorphic).
     const infrastructureFailure = await readEngineDetourInfrastructureFailure(

package/src/public-cli/notary-run.ts

@@ -109,7 +109,7 @@ export async function runPublicNotary(
     extraArgs,
     adapters: {
       trySettle: trySettleNotaryTerminalResult,
-      // Accepted receipts and residual incomplete share one present path.
+      // Accepted receipts and failure terminals both present via shared path.
       shouldPresentSettled: () => true,
     },
     ...(env.engine === undefined ? {} : { effectiveEngine: env.engine }),

package/src/public-cli/one-shot-dispatch.ts

@@ -59,7 +59,7 @@ export type OneShotRunEnv = {
 /**
  * Role-specific settlement hooks. Lifecycle coordination stays in this module.
  * - Doctor: present only lawful typed outcomes; optional compliance-audit secondary.
- * - Notary: present any settled terminal (accepted + residual incomplete).
+ * - Notary: present any settled terminal (accepted or failure).
  */
 export type OneShotSettlementAdapters<A extends AdmittedRoleInvocation> = {
   trySettle: (admitted: A) => Promise<TerminalResult | undefined>;

package/src/public-cli/reviewer-run.ts

@@ -72,7 +72,6 @@ import {
   readEngineDetourInfrastructureFailure,
   settleFailureTerminalResult,
   trySettleReviewerTerminalResult,
-  trySettleComplianceAuditIncompleteTerminalResult,
 } from "./settlement.ts";
 import type { CliIo } from "./cli-io.ts";
 import type {
@@ -398,20 +397,6 @@ async function dispatchAdmittedReviewer(input: {
       };
     }
 
-    const auditIncomplete = await trySettleComplianceAuditIncompleteTerminalResult(admitted);
-    if (auditIncomplete !== undefined) {
-      await markRunTerminal(admitted.runDirectory).catch(() => undefined);
-      if (auditIncomplete.roleOutcome.kind === "failure") {
-        presentFailureTerminal(auditIncomplete, io);
-      } else {
-        io.stdout(formatTerminalResult(auditIncomplete));
-      }
-      return {
-        exitCode: exitCodeForTerminalOutcome(auditIncomplete.roleOutcome),
-        admitted,
-        terminal: auditIncomplete,
-      };
-    }
 
     // Prefer engine-detour infrastructure failure already on the session principal
     // over a later secondary knownFailure / provider-stop after abort (#357 T2 / #378).