@akagilnc/pi-workflow-roles 0.1.5365 → 0.1.5375

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-cli/terminal.ts

@@ -7,6 +7,7 @@
  */
 import type { NoReceiptLifecycleFacts } from "../receipt-delivery-policy.ts";
 import type { ControlledFailureCause } from "../host-contracts.ts";
+import type { PackagedRole } from "../packaged-role-registry.ts";
 
 export type { ControlledFailureCause } from "../host-contracts.ts";
 
@@ -22,23 +23,7 @@ export type TerminalArtifactRef = {
 };
 
 /** Public callable roles that currently produce Terminal outcomes. */
-export type TerminalRoleName =
-  | "judge"
-  | "coder"
-  | "fixer"
-  | "collector"
-  | "doctor"
-  | "reviewer"
-  | "merger"
-  | "notary"
-  | "countersign"
-  | "gleaner-left"
-  | "inspector"
-  | "gatekeeper"
-  | "navigator"
-  | "auditor"
-  | "diarist"
-  | "secretariat";
+export type TerminalRoleName = PackagedRole;
 
 export type NoReceiptTerminalOutcome = NoReceiptLifecycleFacts & {
   kind: "no_receipt";

package/src/public-cli/turn-request.ts

@@ -27,6 +27,32 @@ export type ResumeModelConfig = {
   readonly thinking?: PublicThinkingLevel;
 };
 
+/**
+ * Env axes shared by every public turn projection.
+ * Callers add correlation, continuation, host, and station-only fields.
+ */
+export function projectPublicTurnAxes(env: {
+  readonly packageRoot: string;
+  readonly home: string;
+  readonly agentDir: string;
+  readonly model?: SeatModelConfig;
+  readonly engine?: string;
+  readonly engineModel?: string;
+  readonly timeoutMs?: number;
+}): Pick<
+  RoleTurnRequestProjectionOptions,
+  "packageRoot" | "home" | "agentDir" | "model" | "engine" | "engineModel" | "timeoutMs"
+> {
+  return {
+    packageRoot: env.packageRoot,
+    home: env.home,
+    agentDir: env.agentDir,
+    ...(env.model === undefined ? {} : { model: env.model }),
+    ...pickEngineAxis(env),
+    ...(env.timeoutMs === undefined ? {} : { timeoutMs: env.timeoutMs }),
+  };
+}
+
 export type RoleTurnRequestProjectionOptions = {
   packageRoot: string;
   home: string;

package/src/public-role-summons.ts

@@ -25,12 +25,12 @@ import type {
 } from "./host-contracts.ts";
 import type { TerminalResult } from "./public-cli/terminal.ts";
 import type { HostSelectionFailure, NamedRoleTurnHostAdapter } from "./public-cli/role-turn-host-resolution.ts";
-import { pickEngineAxis } from "./package-resources/engine-material.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"
@@ -210,18 +210,6 @@ async function resolveSummonHome(options: PublicSummonRequest): Promise<string>
   return packageMachineHome();
 }
 
-/** Seat axes only — no parent-env fallback (#675 / #617 DK-3 / #883). */
-function projectSeatEngine(seat: EffectiveSeat): {
-  engine?: string;
-  engineModel?: string;
-} {
-  return pickEngineAxis(seat);
-}
-
-function projectSeatHost(seat: EffectiveSeat): { host?: string } {
-  return seat.host === undefined ? {} : { host: seat.host };
-}
-
 function hostSelectionFailureFromUnknown(error: unknown): HostSelectionFailure | undefined {
   if (typeof error !== "object" || error === null) return undefined;
   if ((error as { name?: unknown }).name !== "HostSelectionError") return undefined;
@@ -273,18 +261,17 @@ async function createSummonEnv(
   const [
     { piDurablePrincipalAuthority },
     { appendPiSessionCustomEntry },
-    { resolveRoleTurnHost },
-    { missingResolvedSeatModelMessage, resolvedSeatWithModel },
+    { openRoleSeatRuntime },
+    { missingResolvedSeatModelMessage },
   ] = await Promise.all([
     import("./pi/durable-principal.ts"),
     import("./pi/role-turn-host.ts"),
-    import("./public-cli/role-turn-host-resolution.ts"),
+    import("./public-cli/role-seat-runtime.ts"),
     import("./public-cli/config.ts"),
   ]);
   const principalAuthority = options.principalAuthority ?? piDurablePrincipalAuthority;
-  // Host first → argv (afterHost) → missing-model → provider projection (#617/#178/#840).
-  const roleTurnHost = resolveRoleTurnHost(
-    {
+  const opened = openRoleSeatRuntime({
+    resolution: {
       packageRoot: options.packageRoot,
       ...(options.roleTurnHost === undefined ? {} : { roleTurnHost: options.roleTurnHost }),
       ...(options.hostAdapters === undefined ? {} : { hostAdapters: options.hostAdapters }),
@@ -293,29 +280,19 @@ async function createSummonEnv(
         : { extraPiArgs: options.extraPiArgs }),
       ...(options.timeoutMs === undefined ? {} : { timeoutMs: options.timeoutMs }),
     },
-    { role: options.role, seat: options.seat, principalAuthority },
-  );
-  afterHost?.();
-  const seatWithModel = resolvedSeatWithModel(options.seat);
-  if (seatWithModel === undefined) {
+    principalAuthority,
+    role: options.role,
+    seat: options.seat,
+    home: options.home,
+    ...(afterHost === undefined ? {} : { afterHost }),
+    ...(options.hostFacingModel === undefined ? {} : { hostFacingModel: options.hostFacingModel }),
+  });
+  if (!opened.ok) {
     return {
       ok: false,
       missingModelMessage: missingResolvedSeatModelMessage(options.role),
     };
   }
-  const hostName = seatWithModel.host ?? "pi";
-  let hostFacingSelection: RoleTurnModelConfig | undefined = options.hostFacingModel;
-  if (hostFacingSelection === undefined) {
-    const { loadHostProvidersTable, projectHostFacingProvider } = await import(
-      "./public-cli/host-providers.ts"
-    );
-    hostFacingSelection = projectHostFacingProvider(
-      seatWithModel.selection,
-      hostName,
-      loadHostProvidersTable(options.home),
-      options.home,
-    );
-  }
   return {
     ok: true,
     env: {
@@ -324,12 +301,13 @@ async function createSummonEnv(
       agentDir: options.agentDir,
       sessionAppender: appendPiSessionCustomEntry,
       packageRoot: options.packageRoot,
-      roleTurnHost,
+      roleTurnHost: opened.roleTurnHost,
       cwd: options.cwd,
       credentials: options.credentials,
-      ...(hostFacingSelection === undefined ? {} : { model: hostFacingSelection }),
-      ...projectSeatEngine(seatWithModel),
-      ...projectSeatHost(seatWithModel),
+      ...(opened.model === undefined ? {} : { model: opened.model }),
+      ...(opened.engine === undefined ? {} : { engine: opened.engine }),
+      ...(opened.engineModel === undefined ? {} : { engineModel: opened.engineModel }),
+      host: opened.host,
       ...(options.timeoutMs === undefined ? {} : { timeoutMs: options.timeoutMs }),
     },
   };
@@ -344,16 +322,14 @@ export async function summonPublicRole(
 ): Promise<PublicSummonResult> {
   const packageRoot = resolveSummonsPackageRoot(options.packageRoot);
   const home = await resolveSummonHome(options);
-  const agentDir =
-    options.agentDir
-    ?? process.env.PI_CODING_AGENT_DIR
-    ?? join(home, ".pi", "agent");
   const {
     loadCredentialProviders,
     loadPublicCliConfig,
     resolveEffectiveSeat,
+    resolvePublicAgentDir,
     validatePublicCliConfigAxes,
   } = await import("./public-cli/config.ts");
+  const agentDir = resolvePublicAgentDir(options.agentDir, home);
   const credentials =
     options.credentials ?? (await loadCredentialProviders(agentDir));
   const config = await loadPublicCliConfig(home);
@@ -616,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,
@@ -864,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)

package/src/reviewer-role.ts

@@ -41,7 +41,6 @@ export type ReviewerRoleDependencies = {
 };
 export type ReviewerRoleHostActions = { failInfrastructure(error: unknown, ctx: HostContext, toolCallId?: string): never };
 
-
 export type ReviewerActivation = Readonly<{
   fixedBaseRevision: string;
   soul: string;

package/src/role-envelope.ts

@@ -35,6 +35,8 @@ import {
 } from "./navigator-invocation-identity.ts";
 import { serializeThrownValue } from "./serialize-thrown-value.ts";
 
+import { isRecord, errorText } from "./unknown-value.ts";
+
 export { projectActivationFlags };
 
 type Handler = HostEventRegistration[1];
@@ -158,7 +160,7 @@ export async function prepareRoleEnvelope(options: {
       try {
         await appendFile(sessionFile, line, "utf8");
       } catch (error) {
-        const diagnostic = error instanceof Error ? error.message : String(error);
+        const diagnostic = errorText(error);
         armDurableWriteFailure?.(customType, diagnostic);
       }
     });
@@ -291,7 +293,7 @@ export async function prepareRoleEnvelope(options: {
     content: ContentPart[],
   ): void {
     if (infrastructureRoundFailure !== undefined) return;
-    const record = typeof details === "object" && details !== null && !Array.isArray(details)
+    const record = isRecord(details)
       ? details as Record<string, unknown>
       : undefined;
     const isInfra = record !== undefined && (
@@ -336,7 +338,7 @@ export async function prepareRoleEnvelope(options: {
     content: ContentPart[];
     details: Record<string, unknown>;
   } {
-    const diagnostic = error instanceof Error ? error.message : String(error);
+    const diagnostic = errorText(error);
     const content: ContentPart[] = [{ type: "text", text: diagnostic }];
     const errorCode = typeof (error as unknown as { code?: unknown })?.code === "string"
       ? (error as unknown as { code: string }).code

package/src/role-run-placement.ts

@@ -7,6 +7,8 @@ import {
 } from "./activation-ledger-topology.ts";
 import { requireSafePositiveTicketNumber } from "./run-ticket-number.ts";
 
+import { isEnoent } from "./unknown-value.ts";
+
 export type RoleRunSubject =
   | { readonly ticketNumber: number }
   | { readonly unbound: true };
@@ -19,14 +21,6 @@ export type RoleRunPlacement = {
   readonly attachmentsDirectory: string;
 };
 
-function isMissingPathError(error: unknown): boolean {
-  return (
-    error instanceof Error &&
-    "code" in error &&
-    (error as NodeJS.ErrnoException).code === "ENOENT"
-  );
-}
-
 /**
  * Sole book-level run directory walk: flat legacy `runs/` plus each
  * `subject/runs/` child (ticket / unbound). One authority for read-side
@@ -42,7 +36,7 @@ export async function listBookRunDirectories(bookDir: string): Promise<string[]>
     try {
       entries = await readdir(runsDir, { withFileTypes: true });
     } catch (error) {
-      if (isMissingPathError(error)) return;
+      if (isEnoent(error)) return;
       throw error;
     }
     for (const entry of entries) {
@@ -60,7 +54,7 @@ export async function listBookRunDirectories(bookDir: string): Promise<string[]>
   try {
     subjects = await readdir(bookDir, { withFileTypes: true });
   } catch (error) {
-    if (isMissingPathError(error)) return out.sort();
+    if (isEnoent(error)) return out.sort();
     throw error;
   }
   for (const subject of subjects) {
@@ -107,6 +101,11 @@ export function roleRunArtifactsDirectory(runDirectory: string): string {
   return join(runDirectory, "artifacts");
 }
 
+/** Canonical unbound placement: a run directory under `<book>/unbound/runs/`. */
+export function isUnboundRunDirectory(runDirectory: string): boolean {
+  return runDirectory.replaceAll("\\", "/").includes("/unbound/runs/");
+}
+
 /** The placement seam owns creation for both new runs and resumed legacy runs. */
 export function ensureRoleRunDirectory(
   ledgerHome: string,

package/src/role-run-relocation.ts

@@ -12,6 +12,8 @@ import { join, sep } from "node:path";
 
 import { AUDITOR_PARENT_ATTEMPT_BINDING_ENTRY_TYPE } from "./compliance-transport.ts";
 
+import { isRecord, isEnoent } from "./unknown-value.ts";
+
 const ADMITTED_PAGE_FIELDS = [
   "runDirectory",
   "admittedRequestPath",
@@ -50,14 +52,6 @@ export type RunDirectoryPathRewrite = {
   readonly newRunDirectory: string;
 };
 
-function isEnoent(error: unknown): boolean {
-  return (
-    error instanceof Error &&
-    "code" in error &&
-    (error as NodeJS.ErrnoException).code === "ENOENT"
-  );
-}
-
 /** Rewrite one path value that points at or under oldRunDirectory. */
 export function rewriteRunDirectoryPathValue(
   value: unknown,
@@ -158,16 +152,12 @@ function collectRewrites(input: {
   return out;
 }
 
-function isPlainObject(value: unknown): value is Record<string, unknown> {
-  return value !== null && typeof value === "object" && !Array.isArray(value);
-}
-
 /** Typed notary/source-run locator: only runDirectory is a path. */
 function rewriteSourceRunLocator(
   value: unknown,
   rewrites: readonly RunDirectoryPathRewrite[],
 ): void {
-  if (!isPlainObject(value)) return;
+  if (!isRecord(value)) return;
   rewriteRunDirectoryPathFieldsAgainstRewrites(
     value,
     SOURCE_RUN_LOCATOR_FIELDS,
@@ -180,7 +170,7 @@ function rewriteSummonsMaterials(
   value: unknown,
   rewrites: readonly RunDirectoryPathRewrite[],
 ): void {
-  if (!isPlainObject(value)) return;
+  if (!isRecord(value)) return;
   rewriteRunDirectoryPathFieldsAgainstRewrites(
     value,
     SUMMONS_PATH_FIELDS,
@@ -204,12 +194,12 @@ export function rewriteAdmittedRoleRunPage(
   rewriteSourceRunLocator(page.sourceRun, rewrites);
   if (Array.isArray(page.attachments)) {
     for (const attachment of page.attachments) {
-      if (isPlainObject(attachment)) {
+      if (isRecord(attachment)) {
         rewriteRunDirectoryPathFieldsAgainstRewrites(attachment, ["frozenPath"], rewrites);
       }
     }
   }
-  if (isPlainObject(page.principal)) {
+  if (isRecord(page.principal)) {
     rewriteRunDirectoryPathFieldsAgainstRewrites(
       page.principal,
       ["sessionDirectory", "sessionFile"],
@@ -224,7 +214,7 @@ async function rewriteOfficerPointerFile(
 ): Promise<void> {
   if (!existsSync(path)) return;
   const page = JSON.parse(await readFile(path, "utf8")) as unknown;
-  if (!isPlainObject(page)) return;
+  if (!isRecord(page)) return;
   // Only the typed direct-officer pointer shape — never arbitrary .pointer.json.
   if (page.kind !== "direct-officer-run-pointer") return;
   const before = JSON.stringify(page);
@@ -267,7 +257,7 @@ async function rewriteSitianRecordsJsonl(
       out.push(line);
       continue;
     }
-    if (!isPlainObject(parsed)) {
+    if (!isRecord(parsed)) {
       out.push(line);
       continue;
     }
@@ -325,7 +315,7 @@ async function rewriteSessionTranscriptBindings(
       out.push(line);
       continue;
     }
-    if (!isPlainObject(parsed)) {
+    if (!isRecord(parsed)) {
       out.push(line);
       continue;
     }
@@ -341,8 +331,8 @@ async function rewriteSessionTranscriptBindings(
     } else if (
       parsed.type === "custom" &&
       parsed.customType === AUDITOR_PARENT_ATTEMPT_BINDING_ENTRY_TYPE &&
-      isPlainObject(parsed.data) &&
-      isPlainObject(parsed.data.parent)
+      isRecord(parsed.data) &&
+      isRecord(parsed.data.parent)
     ) {
       const parent = parsed.data.parent;
       const before = parent.sessionFile;
@@ -454,7 +444,7 @@ export async function rewriteRoleRunDurablePages(input: {
       RUN_STATE_PAGE_FIELDS,
       rewrites,
     );
-    if (isPlainObject(page.principal)) {
+    if (isRecord(page.principal)) {
       rewriteRunDirectoryPathFieldsAgainstRewrites(
         page.principal,
         ["sessionDirectory", "sessionFile"],
@@ -462,7 +452,7 @@ export async function rewriteRoleRunDurablePages(input: {
       );
     }
     // Open court summons: source-run locators + frozen attachment path pointers.
-    if (isPlainObject(page.currentCourt)) {
+    if (isRecord(page.currentCourt)) {
       rewriteSummonsMaterials(page.currentCourt.summons, rewrites);
     }
     if (JSON.stringify(page) !== before) {

package/src/role-runtime.ts

@@ -188,7 +188,7 @@ function decodeReviewerAdmittedInputs(getFlag: (name: string) => unknown): Revie
       parsed = JSON.parse(rawAuthorityRefs);
     } catch (error) {
       throw new Error(
-        `Reviewer authority refs transport error: JSON decode failed: ${error instanceof Error ? error.message : String(error)}`,
+        `Reviewer authority refs transport error: JSON decode failed: ${errorText(error)}`,
       );
     }
     if (!Array.isArray(parsed) || parsed.some((ref) => typeof ref !== "string")) {
@@ -297,6 +297,8 @@ export type { GatekeeperResult, GatekeeperSubject, SubmissionGateNonPassResult,
 import { ParentQueueReaskError, unreadableDiscriminatorNotice } from "./submission-errors.ts";
 import { REVIEW_QUEUE_STATUSES } from "./review-submission.ts";
 
+import { isRecord, errorText, isEnoent } from "./unknown-value.ts";
+
 export {
   DOCTOR_EVIDENCE_TOOL_NAME,
   DOCTOR_OUTPUT_TOOL_NAME,
@@ -507,7 +509,7 @@ export function publicNavigatorSettlement(role: string, phase: NavigatorPhase, e
     return { kind: "role_infrastructure_failure", role, phase };
   }
   // accepted/human — project role/phase status; classifier already rejected infra/contradiction.
-  const details = typeof event.details === "object" && event.details !== null && !Array.isArray(event.details)
+  const details = isRecord(event.details)
     ? event.details as Record<string, unknown>
     : {};
   const status = typeof details.status === "string"
@@ -710,7 +712,7 @@ export function createNavigatorRoleRuntime(
           dependencies.recordRoutePlaybookReadFailure?.(undefined);
           if (content.trim() !== "") parts.push(content);
         } catch (error) {
-          const message = error instanceof Error ? error.message : String(error);
+          const message = errorText(error);
           if (message.trim() !== "") {
             dependencies.recordRoutePlaybookReadFailure?.(message);
             parts.push(message);
@@ -854,7 +856,7 @@ export function createDiaristRoleRuntime(
       soulTag: "diarist",
       beforeAccept: async ({ parameters, ctx }) => {
         const submitted =
-          parameters !== null && typeof parameters === "object" && !Array.isArray(parameters)
+          isRecord(parameters)
             ? (parameters as Record<string, unknown>)
             : undefined;
         // Routing belongs to the public seam after this tool call has returned.
@@ -1073,7 +1075,7 @@ export function createRoleRuntimeExtension(
     ): void => {
       if (attendance === undefined) return;
       const recordDisposeFailure = (error: unknown): void => {
-        const diagnostic = error instanceof Error ? error.message : String(error);
+        const diagnostic = errorText(error);
         try {
           sitianReport({
             level: "event",
@@ -1087,7 +1089,7 @@ export function createRoleRuntimeExtension(
           try {
             envelopeHost.appendEntry?.("ak-navigator-dispose-failure", {
               diagnostic,
-              recordFailure: recordError instanceof Error ? recordError.message : String(recordError),
+              recordFailure: errorText(recordError),
             });
           } catch {
             // Failure already diagnosed; recording must not create unhandled rejection (#959).
@@ -1655,15 +1657,7 @@ export function createRoleRuntimeExtension(
       try {
         await recordTypedProviderHttpStatus(runDir, { httpStatus: status, provider });
       } catch (error) {
-        if (
-          status >= 200 &&
-          status < 300 &&
-          error instanceof Error &&
-          "code" in error &&
-          (error as { code?: unknown }).code === "ENOENT"
-        ) {
-          return;
-        }
+        if (status >= 200 && status < 300 && isEnoent(error)) return;
         failInfrastructure(error, ctx);
       }
     };