@akagilnc/pi-workflow-roles 0.1.3353 → 0.1.3356

package/dist/public-cli/main.js

@@ -17301,10 +17301,6 @@ async function readTypedHttp429Observation(runDirectory) {
   if (!isV1ResumableProvider(observation.provider)) return void 0;
   return { httpStatus: 429, provider: observation.provider };
 }
-function isV1ResumableFailure(input) {
-  if (input.hasLawfulTerminalResult) return false;
-  return input.typedHttp429 !== void 0;
-}
 function renderResumeCommand(runId) {
   return `ak-role resume ${runId}`;
 }
@@ -24930,14 +24926,6 @@ async function readLawfulJudgeRoleOutcome(admitted, authority) {
   }
   return auditEscalationLedgerOutcome(admitted, "judge", authority);
 }
-async function hasLawfulJudgeTerminalResult(admitted, authority) {
-  try {
-    const outcome = await readLawfulJudgeRoleOutcome(admitted, authority);
-    return outcome !== void 0 && isLawfulTypedTerminalOutcome(outcome);
-  } catch {
-    return false;
-  }
-}
 async function settleLawfulJudgeTerminalResult(admitted, authority) {
   const roleOutcome = await readLawfulJudgeRoleOutcome(admitted, authority);
   if (roleOutcome === void 0) return void 0;
@@ -25563,25 +25551,9 @@ async function trySettleNavigatorTerminalResult(admitted, authority) {
 async function trySettleCoderTerminalResult(admitted, authority, options = {}) {
   return settleLawfulCoderTerminalResult(admitted, authority, options);
 }
-async function hasLawfulCoderTerminalResult(admitted, authority) {
-  try {
-    const outcome = await sealedLedgerOutcome(admitted);
-    return outcome?.role === "coder";
-  } catch {
-    return false;
-  }
-}
 async function trySettleFixerTerminalResult(admitted, authority, options) {
   return settleLawfulFixerTerminalResult(admitted, authority, options);
 }
-async function hasLawfulFixerTerminalResult(admitted, authority) {
-  try {
-    const outcome = await sealedLedgerOutcome(admitted);
-    return outcome?.role === "fixer";
-  } catch {
-    return false;
-  }
-}
 function extractReviewerMethodInvocations(entries, options) {
   const observed = [];
   for (const entry of entries) {
@@ -25697,14 +25669,6 @@ async function settleLawfulReviewerTerminalResult(admitted, authority, options)
 async function trySettleReviewerTerminalResult(admitted, authority, options) {
   return settleLawfulReviewerTerminalResult(admitted, authority, options);
 }
-async function hasLawfulReviewerTerminalResult(admitted, authority) {
-  try {
-    const outcome = await sealedLedgerOutcome(admitted);
-    return outcome?.role === "reviewer";
-  } catch {
-    return false;
-  }
-}
 function mergerDecisiveFacts(output) {
   const candidate = output;
   const facts = {};
@@ -25862,14 +25826,6 @@ async function settleLawfulMergerTerminalResult(admitted, authority, options) {
 async function trySettleMergerTerminalResult(admitted, authority, options) {
   return settleLawfulMergerTerminalResult(admitted, authority, options);
 }
-async function hasLawfulMergerTerminalResult(admitted, authority) {
-  try {
-    const outcome = await sealedLedgerOutcome(admitted);
-    return outcome?.role === "merger";
-  } catch {
-    return false;
-  }
-}
 function publicationAttemptFromError(path, error) {
   if (error instanceof Error) {
     const identity = {
@@ -26764,13 +26720,9 @@ async function presentControlledFailure2(admitted, failureInput, adapters, autho
   });
   let resumable = false;
   const typedHttp429 = resumeObservation.typedHttp429;
-  if (adapters.isResumableRole === true && admitted.principal !== void 0) {
+  if (admitted.principal !== void 0) {
     const sessionPrincipalAvailable = await authority.isAvailable(admitted.principal);
-    const hasLawful = adapters.hasLawfulTerminalResult !== void 0 ? await adapters.hasLawfulTerminalResult(admitted, authority) : false;
-    resumable = sessionPrincipalAvailable && isV1ResumableFailure({
-      hasLawfulTerminalResult: hasLawful,
-      ...typedHttp429 === void 0 ? {} : { typedHttp429 }
-    });
+    resumable = sessionPrincipalAvailable && typedHttp429 !== void 0;
   }
   if (resumable && typedHttp429 !== void 0) {
     await markRunResumable(admitted.runDirectory, typedHttp429);
@@ -27138,9 +27090,7 @@ function coderAdapters(methodProvenance) {
   return {
     trySettle: (admitted, authority) => trySettleCoderTerminalResult(admitted, authority, {
       ...methodProvenance === void 0 ? {} : { methodProvenance }
-    }),
-    hasLawfulTerminalResult: (admitted, authority) => hasLawfulCoderTerminalResult(admitted, authority),
-    isResumableRole: true
+    })
   };
 }
 async function runPublicCoder(argv, env, io, parseCoderArgv2) {
@@ -29368,9 +29318,7 @@ function fixerAdapters(packageRoot2, methodMaterial) {
         packageRoot2,
         "diagnosing-bugs"
       )
-    }),
-    hasLawfulTerminalResult: (admitted, authority) => hasLawfulFixerTerminalResult(admitted, authority),
-    isResumableRole: true
+    })
   };
 }
 async function loadFixerMethodMaterial(packageRoot2) {
@@ -29728,8 +29676,6 @@ function buildJudgeTurnRequest(admitted, options) {
 function judgeAdapters() {
   return {
     trySettle: (admitted, authority) => trySettleJudgeTerminalResult(admitted, authority),
-    hasLawfulTerminalResult: (admitted, authority) => hasLawfulJudgeTerminalResult(admitted, authority),
-    isResumableRole: true,
     resolveRunnerKnownFailure: async ({ result: result2, sessionFile }) => {
       const infrastructureFailure = await readEngineDetourInfrastructureFailure(sessionFile);
       return result2.knownFailure ?? (infrastructureFailure === void 0 ? void 0 : {
@@ -29866,9 +29812,7 @@ function mergerAdapters(packageRoot2, methodMaterial) {
         "resolving-merge-conflicts"
       )
     }),
-    shouldPresentSettled: (t) => isLawfulTypedTerminalOutcome(t.roleOutcome) || t.roleOutcome.kind === "incomplete",
-    hasLawfulTerminalResult: (admitted, authority) => hasLawfulMergerTerminalResult(admitted, authority),
-    isResumableRole: true
+    shouldPresentSettled: (t) => isLawfulTypedTerminalOutcome(t.roleOutcome) || t.roleOutcome.kind === "incomplete"
   };
 }
 async function loadMergerMethodMaterial(packageRoot2) {
@@ -30152,8 +30096,6 @@ function reviewerAdapters(packageRoot2, methodMaterial) {
         "code-review"
       )
     }),
-    hasLawfulTerminalResult: (admitted, authority) => hasLawfulReviewerTerminalResult(admitted, authority),
-    isResumableRole: true,
     resolveRunnerKnownFailure: async ({ result: result2, sessionFile }) => {
       const infrastructureFailure = await readEngineDetourInfrastructureFailure(sessionFile);
       return infrastructureFailure === void 0 ? result2.knownFailure : {

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@akagilnc/pi-workflow-roles",
-  "version": "0.1.3353",
+  "version": "0.1.3356",
   "description": "Soul-bound workflow roles for Pi",
   "type": "module",
   "license": "Apache-2.0",

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

@@ -28,7 +28,6 @@ import {
   type PublicResumeRequest,
 } from "./run-lifecycle.ts";
 import {
-  hasLawfulCoderTerminalResult,
   presentStructuralRejection,
   trySettleCoderTerminalResult,
 } from "./settlement.ts";
@@ -87,8 +86,6 @@ function coderAdapters(
       trySettleCoderTerminalResult(admitted, authority, {
         ...(methodProvenance === undefined ? {} : { methodProvenance }),
       }),
-    hasLawfulTerminalResult: (admitted, authority) => hasLawfulCoderTerminalResult(admitted, authority),
-    isResumableRole: true,
   };
 }
 

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

@@ -31,7 +31,6 @@ import {
   type PublicResumeRequest,
 } from "./run-lifecycle.ts";
 import {
-  hasLawfulFixerTerminalResult,
   presentStructuralRejection,
   trySettleFixerTerminalResult,
 } from "./settlement.ts";
@@ -99,8 +98,6 @@ function fixerAdapters(
               "diagnosing-bugs",
             ),
           }),
-    hasLawfulTerminalResult: (admitted, authority) => hasLawfulFixerTerminalResult(admitted, authority),
-    isResumableRole: true,
   };
 }
 

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

@@ -22,7 +22,6 @@ import {
   type PublicResumeRequest,
 } from "./run-lifecycle.ts";
 import {
-  hasLawfulJudgeTerminalResult,
   presentStructuralRejection,
   readEngineDetourInfrastructureFailure,
   trySettleJudgeTerminalResult,
@@ -56,8 +55,6 @@ export function buildJudgeTurnRequest(
 function judgeAdapters(): PostAdmissionAdapters<AdmittedJudgeInvocation> {
   return {
     trySettle: (admitted, authority) => trySettleJudgeTerminalResult(admitted, authority),
-    hasLawfulTerminalResult: (admitted, authority) => hasLawfulJudgeTerminalResult(admitted, authority),
-    isResumableRole: true,
     resolveRunnerKnownFailure: async ({ result, sessionFile }) => {
       const infrastructureFailure = await readEngineDetourInfrastructureFailure(sessionFile);
       return (

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

@@ -36,7 +36,6 @@ import {
   type PublicResumeRequest,
 } from "./run-lifecycle.ts";
 import {
-  hasLawfulMergerTerminalResult,
   isLawfulTypedTerminalOutcome,
   presentStructuralRejection,
   trySettleMergerTerminalResult,
@@ -105,8 +104,6 @@ function mergerAdapters(
           }),
     shouldPresentSettled: (t) =>
       isLawfulTypedTerminalOutcome(t.roleOutcome) || t.roleOutcome.kind === "incomplete",
-    hasLawfulTerminalResult: (admitted, authority) => hasLawfulMergerTerminalResult(admitted, authority),
-    isResumableRole: true,
   };
 }
 

package/src/public-cli/post-admission.ts

@@ -34,7 +34,6 @@ import {
 import {
   acquireRunWriterLease,
   clearTypedProviderHttpObservation,
-  isV1ResumableFailure,
   markRunResumable,
   markRunRunning,
   markRunTerminal,
@@ -115,8 +114,6 @@ export type PostAdmissionAdapters<
     result: RoleTurnResult;
     sessionFile: string;
   }) => Promise<RoleTurnKnownFailure | undefined>;
-  hasLawfulTerminalResult?: (admitted: A, authority: DurablePrincipalAuthority) => Promise<boolean>;
-  isResumableRole?: boolean;
   beforeDispatch?: (admitted: A) => Promise<void> | void;
 };
 
@@ -190,19 +187,13 @@ export async function presentControlledFailure<
     ...(session === undefined ? {} : { session }),
   });
 
+  // #665 / #416: resume hint is seat-uniform — principal available && typed 429 → show.
+  // Do not fork the public terminal face on per-seat hasLawful / isResumableRole (ADR 0040).
   let resumable = false;
   const typedHttp429 = resumeObservation.typedHttp429;
-  if (adapters.isResumableRole === true && admitted.principal !== undefined) {
+  if (admitted.principal !== undefined) {
     const sessionPrincipalAvailable = await authority.isAvailable(admitted.principal);
-    const hasLawful = adapters.hasLawfulTerminalResult !== undefined
-      ? await adapters.hasLawfulTerminalResult(admitted, authority)
-      : false;
-    resumable =
-      sessionPrincipalAvailable &&
-      isV1ResumableFailure({
-        hasLawfulTerminalResult: hasLawful,
-        ...(typedHttp429 === undefined ? {} : { typedHttp429 }),
-      });
+    resumable = sessionPrincipalAvailable && typedHttp429 !== undefined;
   }
 
   if (resumable && typedHttp429 !== undefined) {

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

@@ -31,7 +31,6 @@ import {
 } from "./run-lifecycle.ts";
 import { clearReviewerDispatchRejection } from "./reviewer-dispatch-rejection.ts";
 import {
-  hasLawfulReviewerTerminalResult,
   presentStructuralRejection,
   readEngineDetourInfrastructureFailure,
   trySettleReviewerTerminalResult,
@@ -96,8 +95,6 @@ function reviewerAdapters(
               "code-review",
             ),
           }),
-    hasLawfulTerminalResult: (admitted, authority) => hasLawfulReviewerTerminalResult(admitted, authority),
-    isResumableRole: true,
     resolveRunnerKnownFailure: async ({ result, sessionFile }) => {
       const infrastructureFailure = await readEngineDetourInfrastructureFailure(sessionFile);
       return infrastructureFailure === undefined

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

@@ -175,19 +175,6 @@ export async function readTypedHttp429Observation(
   return { httpStatus: 429, provider: observation.provider };
 }
 
-/**
- * @deprecated #416: 429-only resumability predicate removed from gating. Kept for compatibility.
- * True only when a typed HTTP 429 was observed and no lawful terminal exists.
- * Callers must pass the lawful-terminal fact from settlement, not re-infer it.
- */
-export function isV1ResumableFailure(input: {
-  readonly hasLawfulTerminalResult: boolean;
-  readonly typedHttp429?: TypedHttp429Observation;
-}): boolean {
-  if (input.hasLawfulTerminalResult) return false;
-  return input.typedHttp429 !== undefined;
-}
-
 /** Complete public resume command. Run ID is revealed only through this command text. */
 export function renderResumeCommand(runId: string): string {
   return `ak-role resume ${runId}`;

package/src/public-cli/settlement.ts

@@ -2513,23 +2513,6 @@ export async function readLawfulJudgeRoleOutcome(
   return auditEscalationLedgerOutcome(admitted, "judge", authority);
 }
 
-/**
- * Independent confirmation that a lawful Judge terminal result is present in session.
- * Used for resume qualification — must not depend on artifact publication success.
- * Unreadable sessions are not a confirmed lawful result (returns false).
- */
-export async function hasLawfulJudgeTerminalResult(
-  admitted: AdmittedJudgeInvocation,
-  authority: DurablePrincipalAuthority,
-): Promise<boolean> {
-  try {
-    const outcome = await readLawfulJudgeRoleOutcome(admitted, authority);
-    return outcome !== undefined && isLawfulTypedTerminalOutcome(outcome);
-  } catch {
-    return false;
-  }
-}
-
 /**
  * Single lawful-settlement implementation (session → outcome/Navigator/artifacts).
  *
@@ -3581,18 +3564,6 @@ export async function trySettleCoderTerminalResult(
   return settleLawfulCoderTerminalResult(admitted, authority, options);
 }
 
-export async function hasLawfulCoderTerminalResult(
-  admitted: AdmittedCoderInvocation,
-  authority: DurablePrincipalAuthority,
-): Promise<boolean> {
-  try {
-    const outcome = await sealedLedgerOutcome(admitted);
-    return outcome?.role === "coder";
-  } catch {
-    return false;
-  }
-}
-
 /** Try to settle a lawful Fixer Terminal; undefined only for genuine absence. */
 export async function trySettleFixerTerminalResult(
   admitted: AdmittedFixerInvocation,
@@ -3606,18 +3577,6 @@ export async function trySettleFixerTerminalResult(
   return settleLawfulFixerTerminalResult(admitted, authority, options);
 }
 
-export async function hasLawfulFixerTerminalResult(
-  admitted: AdmittedFixerInvocation,
-  authority: DurablePrincipalAuthority,
-): Promise<boolean> {
-  try {
-    const outcome = await sealedLedgerOutcome(admitted);
-    return outcome?.role === "fixer";
-  } catch {
-    return false;
-  }
-}
-
 /**
  * Observe forced Reviewer code-review Skill expansions from the session.
  * Expansion evidence is package-path only; ambient home locations never count.
@@ -3807,18 +3766,6 @@ export async function trySettleReviewerTerminalResult(
   return settleLawfulReviewerTerminalResult(admitted, authority, options);
 }
 
-export async function hasLawfulReviewerTerminalResult(
-  admitted: AdmittedReviewerInvocation,
-  authority: DurablePrincipalAuthority,
-): Promise<boolean> {
-  try {
-    const outcome = await sealedLedgerOutcome(admitted);
-    return outcome?.role === "reviewer";
-  } catch {
-    return false;
-  }
-}
-
 function mergerDecisiveFacts(output: MergerOutput): Record<string, unknown> {
   const candidate = output as unknown as object;
   const facts: Record<string, unknown> = {};
@@ -4057,18 +4004,6 @@ export async function trySettleMergerTerminalResult(
   return settleLawfulMergerTerminalResult(admitted, authority, options);
 }
 
-export async function hasLawfulMergerTerminalResult(
-  admitted: AdmittedMergerInvocation,
-  authority: DurablePrincipalAuthority,
-): Promise<boolean> {
-  try {
-    const outcome = await sealedLedgerOutcome(admitted);
-    return outcome?.role === "merger";
-  } catch {
-    return false;
-  }
-}
-
 /** One failed attempt to place a durable failure artifact (path is private layout). */
 type PublicationAttempt = {
   readonly path: string;