@akagilnc/pi-workflow-roles 0.1.5368 → 0.1.5382

package/src/public-cli/host-providers.ts

@@ -12,6 +12,7 @@ import { readFileSync } from "node:fs";
 import { join } from "node:path";
 
 import type { SeatModelConfig } from "./config.ts";
+import { isEnoent, isRecord } from "../unknown-value.ts";
 
 /** host → seat-provider → host-facing provider. */
 export type HostProvidersTable = Readonly<
@@ -31,25 +32,19 @@ export function hermesProviderModelsCachePath(home: string): string {
 }
 
 export function parseHostProvidersTable(value: unknown): HostProvidersTable {
-  if (value === null || typeof value !== "object" || Array.isArray(value)) {
+  if (!isRecord(value)) {
     throw new Error("host-providers.json must be an object");
   }
   const out: Record<string, Record<string, string>> = {};
-  for (const [host, byProvider] of Object.entries(value as Record<string, unknown>)) {
+  for (const [host, byProvider] of Object.entries(value)) {
     if (host.trim() === "") {
       throw new Error("host-providers.json host key must be non-empty");
     }
-    if (
-      byProvider === null ||
-      typeof byProvider !== "object" ||
-      Array.isArray(byProvider)
-    ) {
+    if (!isRecord(byProvider)) {
       throw new Error(`host-providers.json[${host}] must be an object`);
     }
     const providers: Record<string, string> = {};
-    for (const [seatProvider, hostProvider] of Object.entries(
-      byProvider as Record<string, unknown>,
-    )) {
+    for (const [seatProvider, hostProvider] of Object.entries(byProvider)) {
       if (seatProvider.trim() === "") {
         throw new Error(
           `host-providers.json[${host}] seat-provider key must be non-empty`,
@@ -79,13 +74,7 @@ export function loadHostProvidersTable(home: string): HostProvidersTable {
     const raw = readFileSync(path, "utf8");
     return parseHostProvidersTable(JSON.parse(raw) as unknown);
   } catch (error) {
-    if (
-      error instanceof Error &&
-      "code" in error &&
-      (error as NodeJS.ErrnoException).code === "ENOENT"
-    ) {
-      return {};
-    }
+    if (isEnoent(error)) return {};
     throw error;
   }
 }
@@ -115,24 +104,18 @@ export function listHermesProvidersForModel(
   try {
     text = readFileSync(path, "utf8");
   } catch (error) {
-    if (
-      error instanceof Error &&
-      "code" in error &&
-      (error as NodeJS.ErrnoException).code === "ENOENT"
-    ) {
-      return [];
-    }
+    if (isEnoent(error)) return [];
     throw error;
   }
   const raw: unknown = JSON.parse(text);
-  if (raw === null || typeof raw !== "object" || Array.isArray(raw)) {
+  if (!isRecord(raw)) {
     throw new Error(
       `hermes provider_models_cache.json must be an object: ${path}`,
     );
   }
   const found: string[] = [];
-  for (const [provider, entry] of Object.entries(raw as Record<string, unknown>)) {
-    if (entry === null || typeof entry !== "object" || Array.isArray(entry)) {
+  for (const [provider, entry] of Object.entries(raw)) {
+    if (!isRecord(entry)) {
       continue;
     }
     const models = (entry as { models?: unknown }).models;

package/src/public-cli/instruction-seat-run.ts

@@ -11,6 +11,8 @@ import { createPiDoctorAuditor } from "../doctor-auditor.ts";
 
 import type { DurablePrincipalAuthority, HostContext, RoleTurnRequest } from "../host-contracts.ts";
 import { officerConclusionReask, gateOfficerForSubject } from "../gatekeeper-role.ts";
+import { receivedDiscriminator } from "../submission-errors.ts";
+import { REVIEW_QUEUE_STATUSES } from "../review-submission.ts";
 import { runJudgeGates } from "../judge-role.ts";
 import { WORKER_DONE_STATUSES } from "../worker-submission-contracts.ts";
 import { SECRETARIAT_GATE_OFFICER_ENTRY_TYPE } from "../secretariat-contracts.ts";
@@ -101,6 +103,8 @@ import {
   latestPayloadEscalated,
   type CountersignRunEnv,
 } from "./countersign-run.ts";
+import { isRecord, errorText } from "../unknown-value.ts";
+
 export type InstructionSeatRunEnv = PostAdmissionEnv & Pick<
   CountersignRunEnv,
   "reviewReask" | "gateReviewInstruction" | "parentRunPath"
@@ -114,7 +118,6 @@ type SeatRunResult = {
 
 const AUDITED_ROLES = new Set<PackagedRole>(["judge", "fixer", "coder", "secretariat", "countersign", "doctor"]);
 const SECRETARIAT_STATUS_REASK = "secretariatStatus 不是 converged、escalate 之一。请重新交卷，secretariatStatus 写明其一。";
-const COUNTERSIGN_STATUS_REASK = "status 不是 converged、continue、escalate 三态之一。请重新交卷，status 写明其一。";
 const WORKER_ROUTING_STATUSES: ReadonlySet<string> = new Set([
   "planned", "completed", "refused", "partially_completed", "unfinished",
 ]);
@@ -125,11 +128,10 @@ const DOCTOR_STATUS_REASK =
   "status 不是 completed、refused 之一。请重新交卷，status 写明其一。";
 const POST_SUBMISSION_ROUTING: Partial<Record<PackagedRole, {
   readonly statuses: ReadonlySet<string>;
-  readonly reask: string;
+  readonly reask?: string;
 }>> = {
   judge: {
-    statuses: new Set(["converged", "continue", "escalate"]),
-    reask: "status 不是 converged、continue、escalate 三态之一。请重新交卷，status 写明其一。",
+    statuses: REVIEW_QUEUE_STATUSES,
   },
   coder: {
     statuses: WORKER_ROUTING_STATUSES,
@@ -159,12 +161,10 @@ function unreadablePostSubmissionStatus(
   const route = POST_SUBMISSION_ROUTING[admitted.role];
   if (route === undefined) return undefined;
   const payload = terminal.roleOutcome.payloads?.at(-1);
-  const status = payload !== null && typeof payload === "object" && !Array.isArray(payload)
-    ? (payload as Record<string, unknown>).status
-    : undefined;
+  const status = receivedDiscriminator(payload, "status");
   return typeof status === "string" && route.statuses.has(status)
     ? undefined
-    : route.reask;
+    : route.reask ?? officerConclusionReask(status);
 }
 
 async function reaskUnreadablePostSubmissionStatus(
@@ -659,7 +659,7 @@ export async function runPublicInstructionSeat(
       auditorSource = resolved.runDirectory;
       auditorTicket = await readBoardTicketNumber(resolved.runDirectory);
     } catch (error) {
-      presentStructuralRejection(new CliUsageError(error instanceof Error ? error.message : String(error)), io);
+      presentStructuralRejection(new CliUsageError(errorText(error)), io);
       return { exitCode: 2 };
     }
     if (auditorSource === undefined) return { exitCode: 2 };
@@ -906,7 +906,6 @@ export async function runPublicInstructionSeatResume(
   return resume();
 }
 
-const QUEUE_CONCLUSIONS = new Set(["converged", "continue", "escalate"]);
 const GATE_CHILD_ROLES = new Set(["notary", "auditor", "inspector", "countersign"]);
 
 /**
@@ -931,7 +930,7 @@ async function queueConclusionFromChild(
       undefined,
     );
     const status = latestQueueStatus(terminal);
-    if (terminal !== undefined && status !== undefined && QUEUE_CONCLUSIONS.has(status)) {
+    if (terminal !== undefined && typeof status === "string" && REVIEW_QUEUE_STATUSES.has(status)) {
       return { admitted: current, terminal, status };
     }
     const reasked = await runPublicInstructionSeatResume({
@@ -942,7 +941,7 @@ async function queueConclusionFromChild(
       return { stop: reasked };
     }
     const reaskedStatus = latestQueueStatus(reasked.terminal);
-    if (reaskedStatus !== undefined && QUEUE_CONCLUSIONS.has(reaskedStatus)) {
+    if (typeof reaskedStatus === "string" && REVIEW_QUEUE_STATUSES.has(reaskedStatus)) {
       return { admitted: reasked.admitted, terminal: reasked.terminal, status: reaskedStatus };
     }
     if (latestPayloadEscalated(reasked.terminal.roleOutcome)) return { stop: reasked };
@@ -997,15 +996,15 @@ async function auditSubmittedRole(
     || turn.terminal?.roleOutcome.kind !== "accepted") return turn;
   const accepted = turn.terminal.roleOutcome.payloads?.at(-1);
   if (accepted === undefined) throw new Error("accepted submission has no payload");
-  const record = accepted !== null && typeof accepted === "object" && !Array.isArray(accepted)
+  const record = isRecord(accepted)
     ? accepted as Record<string, unknown> : undefined;
   const status = admitted.role === "secretariat" ? record?.secretariatStatus : record?.status;
   if (admitted.role === "secretariat" && status !== "converged" && status !== "escalate") {
     return runPublicInstructionSeatResume({ runId: admitted.runId, message: SECRETARIAT_STATUS_REASK },
       { ...env, autoResumeLimit: 0 }, io);
   }
-  if (admitted.role === "countersign" && (typeof status !== "string" || !QUEUE_CONCLUSIONS.has(status))) {
-    return runPublicInstructionSeatResume({ runId: admitted.runId, message: COUNTERSIGN_STATUS_REASK },
+  if (admitted.role === "countersign" && (typeof status !== "string" || !REVIEW_QUEUE_STATUSES.has(status))) {
+    return runPublicInstructionSeatResume({ runId: admitted.runId, message: officerConclusionReask(status) },
       { ...env, autoResumeLimit: 0 }, io);
   }
   // Self-escalation is a valid open routing state for these seats only.

package/src/public-cli/invocation.ts

@@ -110,6 +110,8 @@ import {
 } from "./option-definitions.ts";
 import type { PublicThinkingLevel } from "./registry.ts";
 
+import { errorText, isRecord } from "../unknown-value.ts";
+
 export type FrozenAttachment = {
   /** Original caller path retained only as provenance. */
   readonly provenancePath: string;
@@ -1571,7 +1573,7 @@ export async function admitPublicRole(
         try {
           repository = parseCollectorRepository(parsed.repo);
         } catch (error) {
-          const detail = error instanceof Error ? error.message : String(error);
+          const detail = errorText(error);
           throw new CliUsageError(detail, { cause: error });
         }
       } else {
@@ -1585,7 +1587,7 @@ export async function admitPublicRole(
           manifestCanonicalJson = manifest.canonicalJson;
         } catch (error) {
           throw new CliUsageError(
-            error instanceof Error ? error.message : String(error),
+            errorText(error),
             { cause: error },
           );
         }
@@ -1640,7 +1642,7 @@ export async function admitPublicRole(
             });
           } catch (error) {
             if (error instanceof CliUsageError) throw error;
-            const detail = error instanceof Error ? error.message : String(error);
+            const detail = errorText(error);
             throw new CliUsageError(detail, { cause: error });
           }
           if (parsed.runs === undefined) {
@@ -1658,7 +1660,7 @@ export async function admitPublicRole(
             caseRunsPath = await realpath(caseRunsPath);
           } catch (error) {
             if (error instanceof CliUsageError) throw error;
-            const detail = error instanceof Error ? error.message : String(error);
+            const detail = errorText(error);
             throw new CliUsageError(
               `doctor case could not be constructed from retained evidence: ${detail}`,
               { cause: error },
@@ -2203,8 +2205,8 @@ export async function loadAdmittedJudgeRequest(
     const raw = JSON.parse(
       await readFile(join(runDirectory, "admitted-request.json"), "utf8"),
     ) as unknown;
-    if (raw === null || typeof raw !== "object" || Array.isArray(raw)) return undefined;
-    const record = raw as Record<string, unknown>;
+    if (!isRecord(raw)) return undefined;
+    const record = raw;
     if (!packagedPublicInstructionSubject(record.role)) return undefined;
     if (typeof record.instruction !== "string") return undefined;
     if (typeof record.instructionEmpty !== "boolean") return undefined;
@@ -2246,7 +2248,7 @@ export type AdmitFixerInvocationOptions = {
 
 function parsePositivePrOption(raw: string | undefined): number {
   if (raw === undefined || raw.trim() === "") throw new CliUsageError("--pr requires a positive pull request number");
-  try { return parseCollectorPrNumber(raw); } catch (error) { throw new CliUsageError(error instanceof Error ? error.message : String(error), { cause: error }); }
+  try { return parseCollectorPrNumber(raw); } catch (error) { throw new CliUsageError(errorText(error), { cause: error }); }
 }
 function parseRepoOption(raw: string | undefined): string {
   if (raw === undefined || raw.trim() === "") throw new CliUsageError("--repo requires owner/repo");
@@ -2294,7 +2296,7 @@ export function resolveGitHubRemoteRepository(
   try {
     return parseCollectorRepository(ownerRepo);
   } catch (error) {
-    const detail = error instanceof Error ? error.message : String(error);
+    const detail = errorText(error);
     throw new CliUsageError(detail, { cause: error });
   }
 }
@@ -2379,7 +2381,7 @@ export async function resolveDoctorCaseRunsPath(options: {
   try {
     real = await realpath(resolved);
   } catch (error) {
-    const detail = error instanceof Error ? error.message : String(error);
+    const detail = errorText(error);
     throw new CliUsageError(
       `doctor --runs is not a readable retained runs root: ${detail}`,
       { cause: error },
@@ -2401,7 +2403,6 @@ export async function resolveDoctorCaseRunsPath(options: {
   return real;
 }
 
-
 export type AdmitReviewerInvocationOptions = {
   home: string;
   principalAuthority: DurablePrincipalAuthority;

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

@@ -122,6 +122,8 @@ import {
   writeHardenedArtifactFile,
 } from "./auto-resume.ts";
 
+import { isRecord, errorText } from "../unknown-value.ts";
+
 function projectRelocatedTurnIdentity(
   request: RoleTurnRequest,
   result: { readonly terminal?: TerminalResult },
@@ -137,7 +139,7 @@ function projectRelocatedTurnIdentity(
   mutableRequest.runDirectory = admitted.runDirectory;
   mutableRequest.principal = admitted.principal;
   const activation = mutableRequest.activation;
-  if (activation !== null && typeof activation === "object" && !Array.isArray(activation)) {
+  if (isRecord(activation)) {
     for (const field of ["taskPath", "packetPath", "prerequisitesPath", "inputPath", "requestManifestPath"] as const) {
       const record = activation as Record<string, unknown>;
       if (field in record) record[field] = rewrite(record[field]);
@@ -733,7 +735,7 @@ export async function dispatchPostAdmissionTurn<
             continue;
           }
           const payload = row.accepted;
-          if (payload === null || typeof payload !== "object" || Array.isArray(payload)) continue;
+          if (!isRecord(payload)) continue;
           ticketNumber = readDeclaredTicketNumber(
             (payload as { ticketNumber?: unknown }).ticketNumber,
           );
@@ -1802,7 +1804,7 @@ async function presentResumeFailurePointer(
       cause.thrownCause = formatErrorCauseDetail(thrown.cause);
     }
     presentStructuralRejection({
-      message: thrown instanceof Error ? thrown.message : String(thrown),
+      message: errorText(thrown),
       cause,
     }, io);
   }

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

@@ -71,6 +71,8 @@ import {
   type ReviewerLens,
 } from "./invocation.ts";
 
+import { isRecord, errorText, isEnoent } from "../unknown-value.ts";
+
 /** Providers eligible for v1 typed-429 resume (Codex / xAI only).
  * @deprecated v1 429-only gate removed by #416 (owner 2026-08-22: "根本不要有限制"). Kept for compatibility; do not use for new branching.
  */
@@ -241,8 +243,8 @@ type RoleRunStateDisk = {
 function parseSameTicketSummonsMaterials(
   raw: unknown,
 ): SameTicketSummonsMaterials | undefined {
-  if (raw === null || typeof raw !== "object" || Array.isArray(raw)) return undefined;
-  const record = raw as Record<string, unknown>;
+  if (!isRecord(raw)) return undefined;
+  const record = raw;
   const instruction =
     typeof record.instruction === "string" ? record.instruction : undefined;
   const instructionEmpty =
@@ -256,9 +258,7 @@ function parseSameTicketSummonsMaterials(
       : undefined;
   let sourceRun: NotarySourceRunLocator | undefined;
   if (
-    record.sourceRun !== null &&
-    typeof record.sourceRun === "object" &&
-    !Array.isArray(record.sourceRun)
+    isRecord(record.sourceRun)
   ) {
     const sr = record.sourceRun as Record<string, unknown>;
     if (
@@ -297,8 +297,8 @@ function parseSameTicketSummonsMaterials(
 }
 
 function parseCurrentCourtState(raw: unknown): CurrentCourtState | undefined {
-  if (raw === null || typeof raw !== "object" || Array.isArray(raw)) return undefined;
-  const record = raw as Record<string, unknown>;
+  if (!isRecord(raw)) return undefined;
+  const record = raw;
   if (typeof record.courtAttemptId !== "string" || record.courtAttemptId.length === 0) {
     return undefined;
   }
@@ -333,10 +333,10 @@ function parseRoleRunIdentity(raw: unknown, runDirectory: string): {
   readonly runDirectory: string;
   readonly state: RoleRunState;
 } {
-  if (raw === null || typeof raw !== "object" || Array.isArray(raw)) {
+  if (!isRecord(raw)) {
     throw new TypeError("Invalid run-state.json");
   }
-  const record = raw as Record<string, unknown>;
+  const record = raw;
   if (typeof record.runId !== "string" || record.runId.trim() === "") {
     throw new TypeError("Invalid run-state.json");
   }
@@ -390,11 +390,8 @@ async function readRoleRunStateDisk(
   };
   let resumable: TypedHttp429Observation | undefined;
   if (record.resumable !== undefined && record.resumable !== null) {
-    if (
-      typeof record.resumable === "object" &&
-      !Array.isArray(record.resumable)
-    ) {
-      const r = record.resumable as Record<string, unknown>;
+    if (isRecord(record.resumable)) {
+      const r = record.resumable;
       if (
         r.httpStatus === 429 &&
         typeof r.provider === "string" &&
@@ -702,7 +699,6 @@ export type RunWriterLease = {
 /** Kind of a writer-lease diagnostic sent on the existing sink. */
 export type WriterLeaseDiagnosticKind = "stale-reclaimed";
 
-
 /**
  * True error identity for diagnostics — name/code/message as-is, never a
  * guessed label (failure-honesty constitution).
@@ -741,7 +737,6 @@ function isProcessAlive(pid: number): boolean {
   }
 }
 
-
 export type WriterLockAutopsy =
   | { verdict: "absent" }
   | { verdict: "unknown"; reason: "unparseable"; content: string }
@@ -1072,10 +1067,10 @@ export async function readRunParentPath(
     if (errorCodeOf(error) === "ENOENT") return undefined;
     throw error;
   }
-  if (raw === null || typeof raw !== "object" || Array.isArray(raw)) {
+  if (!isRecord(raw)) {
     return undefined;
   }
-  const record = raw as Record<string, unknown>;
+  const record = raw;
   if (typeof record.sourceRunPath === "string" && record.sourceRunPath.trim() !== "") {
     return record.sourceRunPath;
   }
@@ -1280,7 +1275,7 @@ async function loadResumableRunRecord(
     const raw: unknown = JSON.parse(
       await readFile(run.admittedRequestPath, "utf8"),
     );
-    if (raw !== null && typeof raw === "object" && !Array.isArray(raw)) {
+    if (isRecord(raw)) {
       const record = raw as Record<string, unknown>;
       const storedRunDirectory =
         typeof record.runDirectory === "string" && record.runDirectory.trim() !== ""
@@ -1353,9 +1348,7 @@ async function loadResumableRunRecord(
         caseRunsPath = record.caseRunsPath;
       }
       if (
-        record.caseIdentity !== null &&
-        typeof record.caseIdentity === "object" &&
-        !Array.isArray(record.caseIdentity)
+        isRecord(record.caseIdentity)
       ) {
         const ci = record.caseIdentity as Record<string, unknown>;
         if (
@@ -1373,9 +1366,7 @@ async function loadResumableRunRecord(
         sourceRunPath = record.sourceRunPath;
       }
       if (
-        record.sourceRun !== null &&
-        typeof record.sourceRun === "object" &&
-        !Array.isArray(record.sourceRun)
+        isRecord(record.sourceRun)
       ) {
         const sr = record.sourceRun as Record<string, unknown>;
         if (
@@ -1413,9 +1404,7 @@ async function loadResumableRunRecord(
         mergerInputPath = record.mergerInputPath;
       }
       if (
-        record.derived !== null &&
-        typeof record.derived === "object" &&
-        !Array.isArray(record.derived)
+        isRecord(record.derived)
       ) {
         const d = record.derived as Record<string, unknown>;
         if (
@@ -1452,9 +1441,7 @@ async function loadResumableRunRecord(
       await readFile(join(run.runDirectory, "invocation.json"), "utf8"),
     );
     if (
-      invocationRaw !== null &&
-      typeof invocationRaw === "object" &&
-      !Array.isArray(invocationRaw)
+      isRecord(invocationRaw)
     ) {
       const rec = invocationRaw as Record<string, unknown>;
       if (typeof rec.provider === "string" && typeof rec.model === "string") {
@@ -1478,13 +1465,7 @@ async function loadResumableRunRecord(
     ) {
       throw error;
     }
-    if (
-      !(
-        error instanceof Error &&
-        "code" in error &&
-        (error as { code?: unknown }).code === "ENOENT"
-      )
-    ) {
+    if (!isEnoent(error)) {
       throw new CliUsageError(
         `role run invocation identity is unreadable: ${runId}`,
         { cause: error },
@@ -1583,7 +1564,6 @@ function seatLoadedResult<R extends AdmittedRoleInvocation>(
   };
 }
 
-
 export async function peekRoleRunRole(
   home: string,
   runId: string,
@@ -1594,7 +1574,6 @@ export async function peekRoleRunRole(
   return run?.role;
 }
 
-
 export type LoadedResumablePublicRole = {
   readonly admitted: AdmittedRoleInvocation;
   readonly run: RoleRunRecord;
@@ -1793,7 +1772,7 @@ function admitResumedRole(loaded: {
       try {
         parsedRepository = parseCollectorRepository(repositoryDisplay);
       } catch (error) {
-        const detail = error instanceof Error ? error.message : String(error);
+        const detail = errorText(error);
         throw new CliUsageError(detail, { cause: error });
       }
       if (parsedRepository.canonical !== repository) {

package/src/public-cli/settlement.ts

@@ -346,6 +346,8 @@ import {
   type TerminalRoleOutcome,
 } from "./terminal.ts";
 
+import { isEnoent, isMissingPathError, isRecord } from "../unknown-value.ts";
+
 export type { ControlledFailureCause };
 
 export {
@@ -467,7 +469,7 @@ export async function inspectJudgeSession(
     await readFile(sessionFile, "utf8");
     return { state: "present" };
   } catch (error) {
-    if (isMissingPathError(error)) return { state: "missing" };
+    if (isEnoent(error)) return { state: "missing" };
     return {
       state: "unreadable",
       diagnostic:
@@ -834,24 +836,6 @@ type SessionEntry = {
   parentSession?: string;
 };
 
-function isMissingPathError(error: unknown): boolean {
-  return (
-    error instanceof Error &&
-    "code" in error &&
-    (error as { code?: unknown }).code === "ENOENT"
-  );
-}
-
-/** Face clear only: path not enterable as a face (ENOENT or file mid-path). */
-function isAbsentFacePathError(error: unknown): boolean {
-  return (
-    error instanceof Error &&
-    "code" in error &&
-    ((error as { code?: unknown }).code === "ENOENT" ||
-      (error as { code?: unknown }).code === "ENOTDIR")
-  );
-}
-
 /**
  * Preserve session-read failure identity as a typed session cause.
  * SyntaxError keeps its name so durable settlement does not wash malformed JSONL
@@ -1064,7 +1048,7 @@ async function loadBoundAuditorVolumes(
   try {
     parentEntries = await readBoundSessionEntries(sessionFile);
   } catch (error) {
-    if (isMissingPathError(error)) return undefined;
+    if (isEnoent(error)) return undefined;
     throw sessionReadFailure(error, "failed to read parent session for auditor binding");
   }
   const parentId = parentEntries.find((entry) => entry.type === "session")?.id;
@@ -1078,7 +1062,7 @@ async function loadBoundAuditorVolumes(
       names = await readdir(childDirectory);
       sawAnyDirectory = true;
     } catch (error) {
-      if (isMissingPathError(error)) continue;
+      if (isEnoent(error)) continue;
       throw sessionReadFailure(error, "failed to read bound auditor session directory");
     }
     for (const file of names.filter((name) => name.endsWith(".jsonl")).sort().reverse()) {
@@ -1313,7 +1297,7 @@ export async function resolveAuditedRunnerFailureResolution(input: {
     );
     if (terminatingFailure !== undefined) return resolutionOf(terminatingFailure);
   } catch (error) {
-    if (!isMissingPathError(error)) {
+    if (!isEnoent(error)) {
       const failure = sessionReadFailure(error, "failed to recover typed terminating-tool failure");
       return resolutionOf({
         cause: "session",
@@ -1459,10 +1443,6 @@ export function controlledFailureInputFromResolution(
   };
 }
 
-function isRecord(value: unknown): value is Record<string, unknown> {
-  return typeof value === "object" && value !== null && !Array.isArray(value);
-}
-
 function safelyRead(object: object, key: string): { readable: true; value: unknown } | { readable: false } {
   try {
     return { readable: true, value: (object as Record<string, unknown>)[key] };
@@ -1478,9 +1458,7 @@ function toolResultText(message: SessionMessage): string {
   return content
     .map((part) => {
       if (
-        typeof part === "object" &&
-        part !== null &&
-        !Array.isArray(part) &&
+        isRecord(part) &&
         typeof (part as { text?: unknown }).text === "string"
       ) {
         return (part as { text: string }).text;
@@ -2234,7 +2212,7 @@ async function extractNavigatorFactFromAdmittedSession(
     const entries = await readBoundSessionEntries(sessionFile);
     return extractNavigatorFact(entries);
   } catch (error) {
-    if (isMissingPathError(error)) {
+    if (isEnoent(error)) {
       return {
         disposition: "unavailable",
         source: "unknown",
@@ -2261,7 +2239,7 @@ async function removeFaceIfPresent(path: string): Promise<void> {
     // force:true already ignores ENOENT; ENOTDIR = face path not enterable
     // (artifacts-as-file mid-path) — same absent-face semantics as the reader.
     // Other errno stay loud (失败诚实).
-    if (isAbsentFacePathError(error)) return;
+    if (isMissingPathError(error)) return;
     throw error;
   }
 }
@@ -2392,7 +2370,7 @@ async function readLawfulSettlementEntries(
     return await readBoundSessionEntries(sessionFile);
   } catch (error) {
     // Missing path is absence of a lawful outcome; callers classify via session inspect.
-    if (isMissingPathError(error)) return undefined;
+    if (isEnoent(error)) return undefined;
     // Malformed JSONL and other read failures keep typed session identity.
     throw error instanceof Error &&
       (error as { knownCause?: unknown }).knownCause === "session"
@@ -2839,7 +2817,7 @@ function countersignTerminalFromEntries(
     if (entry?.type !== "custom") continue;
     if (entry.customType !== SECRETARIAT_GATE_OFFICER_ENTRY_TYPE) continue;
     const data =
-      entry.data !== null && typeof entry.data === "object" && !Array.isArray(entry.data)
+      isRecord(entry.data)
         ? (entry.data as Record<string, unknown>)
         : undefined;
     if (data === undefined || !packagedDurableOfficerEntry(data.officer)) continue;

package/src/public-role-summons.ts

@@ -26,10 +26,11 @@ import type {
 import type { TerminalResult } from "./public-cli/terminal.ts";
 import type { HostSelectionFailure, NamedRoleTurnHostAdapter } from "./public-cli/role-turn-host-resolution.ts";
 
+import { isRecord, errorText } from "./unknown-value.ts";
+
 /** Env published by the parent activation so nested summons never re-derive root. */
 export const AK_ROLE_PACKAGE_ROOT_ENV = "AK_ROLE_PACKAGE_ROOT" as const;
 
-
 export type PublicSummonRole =
   | "inspector"
   | "notary"
@@ -591,7 +592,7 @@ export async function summonParallelReviewerLenses(options: {
     if (error instanceof AggregateError) {
       return [error.message, ...error.errors.map(describeFailure)].join("\n");
     }
-    return error instanceof Error ? error.message : String(error);
+    return errorText(error);
   };
   const failedResult = (error: unknown): PublicSummonResult => ({
     exitCode: 1,
@@ -839,7 +840,7 @@ export async function summonGateOfficer(options: {
     const correlationId = parentRunId;
     const { isSafePositiveTicketNumber, readBoardTicketNumber } = await import("./run-ticket-number.ts");
     const parentTicket = await readBoardTicketNumber(options.sourceRunDirectory);
-    const submittedTicket = options.submission !== null && typeof options.submission === "object" && !Array.isArray(options.submission)
+    const submittedTicket = isRecord(options.submission)
       ? (options.submission as { ticketNumber?: unknown }).ticketNumber
       : undefined;
     const courtTicket = isSafePositiveTicketNumber(submittedTicket) ? submittedTicket : parentTicket;

package/src/receipt-delivery-policy.ts

@@ -1,3 +1,4 @@
+import { isRecord } from "./unknown-value.ts";
 /** Shared accepted-receipt delivery budget for role, auditor, and Navigator sessions (#288). */
 export const RECEIPT_DELIVERY_TURN_LIMIT = 2 as const;
 
@@ -14,10 +15,6 @@ export type NoReceiptLifecycleFacts = {
   acceptedReceipt: false;
 };
 
-function isRecord(value: unknown): value is Record<string, unknown> {
-  return typeof value === "object" && value !== null && !Array.isArray(value);
-}
-
 /** Read only the facts required by Terminal consumers; persisted extensions are ignored. */
 export function parseNoReceiptLifecycleFacts(input: unknown): NoReceiptLifecycleFacts {
   if (!isRecord(input)