@akagilnc/pi-workflow-roles 0.1.5368 → 0.1.5375

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

@@ -7,6 +7,7 @@
  */
 import { Type } from "typebox";
 
+import { isRecord } from "../unknown-value.ts";
 import { openToolObject } from "../open-tool-schema.ts";
 import { withTerminatingOutputDeclarations } from "./terminating-infrastructure.ts";
 
@@ -33,19 +34,18 @@ export type NavigatorAdvice = { readonly prose: string };
 export function projectLawfulNavigatorOutput(value: unknown): NavigatorAdvice | undefined {
   if (value === undefined || value === null) return undefined;
   if (typeof value === "string") return { prose: value };
-  if (typeof value === "object" && !Array.isArray(value)) {
-    const record = value as Record<string, unknown>;
-    if (typeof record.prose === "string") return { prose: record.prose };
-    if (record.prose !== undefined && record.prose !== null) {
-      return { prose: typeof record.prose === "string" ? record.prose : String(record.prose) };
+  if (isRecord(value)) {
+    if (typeof value.prose === "string") return { prose: value.prose };
+    if (value.prose !== undefined && value.prose !== null) {
+      return { prose: typeof value.prose === "string" ? value.prose : String(value.prose) };
     }
     // Free-form object submission: present the whole body as prose when it has content.
-    const keys = Object.keys(record);
+    const keys = Object.keys(value);
     if (keys.length === 0) return { prose: "" };
     try {
-      return { prose: JSON.stringify(record) };
+      return { prose: JSON.stringify(value) };
     } catch {
-      return { prose: String(record) };
+      return { prose: String(value) };
     }
   }
   return { prose: String(value) };

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

@@ -13,6 +13,7 @@
  */
 import { Type, type TSchema } from "typebox";
 import type { CorrectableSubmissionError } from "../submission-correctable-error.ts";
+import { isRecord } from "../unknown-value.ts";
 
 export const INFRASTRUCTURE_FAILURE_DECLARATION_KEY =
   "infrastructureFailure" as const;
@@ -94,26 +95,11 @@ type TerminatingInfrastructureHostActions<C> = {
 function isInfrastructureFailureDeclaration(
   parameters: unknown,
 ): boolean {
-  if (
-    parameters === null ||
-    typeof parameters !== "object" ||
-    Array.isArray(parameters)
-  ) {
-    return false;
-  }
-  const record = parameters as Record<string, unknown>;
-  if (!Object.hasOwn(record, INFRASTRUCTURE_FAILURE_DECLARATION_KEY)) return false;
-  const declaration = record[INFRASTRUCTURE_FAILURE_DECLARATION_KEY];
-  if (
-    declaration === null ||
-    typeof declaration !== "object" ||
-    Array.isArray(declaration)
-  ) {
-    return false;
-  }
-  const diagnostic = (declaration as Record<string, unknown>)[
-    INFRASTRUCTURE_FAILURE_DIAGNOSTIC_KEY
-  ];
+  if (!isRecord(parameters)) return false;
+  if (!Object.hasOwn(parameters, INFRASTRUCTURE_FAILURE_DECLARATION_KEY)) return false;
+  const declaration = parameters[INFRASTRUCTURE_FAILURE_DECLARATION_KEY];
+  if (!isRecord(declaration)) return false;
+  const diagnostic = declaration[INFRASTRUCTURE_FAILURE_DIAGNOSTIC_KEY];
   return typeof diagnostic === "string" && diagnostic.trim().length > 0;
 }
 

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

@@ -36,6 +36,8 @@ import {
   type WorkerOutput,
 } from "./worker-output.ts";
 
+import { isRecord } from "../unknown-value.ts";
+
 export {
   CODER_OUTPUT_TOOL_NAME,
   COLLECTOR_OUTPUT_TOOL,
@@ -114,7 +116,6 @@ export class AcceptedDetailsContractError extends CorrectableSubmissionError {
   }
 }
 
-
 /**
  * #836: status allowlist rejection deleted. Original object is the receipt.
  * Callers that need a typed view may still cast; code must not bounce on status.
@@ -123,7 +124,7 @@ export function validateAcceptedDetails(
   _toolName: TerminatingToolName,
   details: unknown,
 ): AcceptedDetails {
-  if (details !== null && typeof details === "object" && !Array.isArray(details)) {
+  if (isRecord(details)) {
     return details as AcceptedDetails;
   }
   // Non-object payload still records as empty object rather than rejecting.
@@ -178,18 +179,13 @@ export function deepEqual(a: unknown, b: unknown): boolean {
     if (!Array.isArray(b) || a.length !== b.length) return false;
     return a.every((item, index) => deepEqual(item, b[index]));
   }
-  if (typeof a === "object") {
-    if (typeof b !== "object" || b === null || Array.isArray(b)) return false;
-    const aKeys = Object.keys(a as object).sort();
-    const bKeys = Object.keys(b as object).sort();
+  if (isRecord(a)) {
+    if (!isRecord(b)) return false;
+    const aKeys = Object.keys(a).sort();
+    const bKeys = Object.keys(b).sort();
     if (aKeys.length !== bKeys.length) return false;
     if (!aKeys.every((key, index) => key === bKeys[index])) return false;
-    return aKeys.every((key) =>
-      deepEqual(
-        (a as Record<string, unknown>)[key],
-        (b as Record<string, unknown>)[key],
-      )
-    );
+    return aKeys.every((key) => deepEqual(a[key], b[key]));
   }
   return false;
 }

package/src/pi/adapter.ts

@@ -20,7 +20,6 @@ import { createRoleRuntimeExtension, type RoleRuntimeDependencies } from "../rol
 import { renderAgentStartMaterials } from "../agent-start-materials.ts";
 import { readUserDialogueStdin } from "../user-dialogue-stdin.ts";
 
-
 export type PiRoleHostAdapter = RoleEnvelopeHost;
 
 const piContexts = new WeakMap<HostContext, ExtensionContext>();

package/src/pi/role-turn-host.ts

@@ -26,7 +26,6 @@ import { applyEngineChildEnv, ENGINE_MODEL_FLAG_NAME, normalizeEngineName } from
 import { projectActivationFlags } from "../role-activation-flags.ts";
 import { encodeUserDialogueStdin } from "../user-dialogue-stdin.ts";
 
-
 /** Package-relative Internal role entrypoint (ADR 0052; same path as public-cli registry). */
 const INTERNAL_ROLE_ENTRYPOINT_RELATIVE = "extensions/role-runtime.ts";
 

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

@@ -41,6 +41,8 @@ import type {
 } from "./invocation.ts";
 import { presentControlledFailure, presentStructuralRejection } from "./settlement.ts";
 
+import { errorText } from "../unknown-value.ts";
+
 export type AnalystRunEnv = {
   readonly home: string;
 };
@@ -97,7 +99,7 @@ export async function buildAnalystIssueModeInputFromPublicArgv(
 
 /**
  * Attachment JSON → library AnalystSweepModeInput via the sole schema (#337).
- * No parallel hand shape; rejects missing/extra/wrong-type fields only.
+ * No parallel hand shape; rejects missing or wrong-type fields only.
  */
 export function parseAnalystSweepModeInputFromJsonValue(
   value: unknown,
@@ -141,7 +143,7 @@ export async function buildAnalystSweepModeInputFromAttachmentPaths(
   try {
     text = exactUtf8(bytes, "analyst sweep attachment");
   } catch (error) {
-    const detail = error instanceof Error ? error.message : String(error);
+    const detail = errorText(error);
     throw new CliUsageError(detail, { cause: error });
   }
 

package/src/public-cli/auto-resume.ts

@@ -42,6 +42,8 @@ import {
 import type { CliIo } from "./cli-io.ts";
 import { serializeThrownValue } from "../serialize-thrown-value.ts";
 
+import { errorText, isEnoent } from "../unknown-value.ts";
+
 const dummyIo: CliIo = { stdout: () => {}, stderr: () => {} };
 
 /**
@@ -132,7 +134,7 @@ export class TurnDispatchedFailure extends Error {
   constructor(cause: unknown) {
     super(
       `settlement failed after the host turn genuinely started: ${
-        cause instanceof Error ? cause.message : String(cause)
+        errorText(cause)
       }`,
       { cause },
     );
@@ -164,7 +166,7 @@ export async function ensureRealArtifactsDirectory(runDirectory: string): Promis
       throw new Error("run artifact retention: artifacts path is not a real directory");
     }
   } catch (error) {
-    if (!isMissingPathError(error)) throw error;
+    if (!isEnoent(error)) throw error;
     await mkdir(artifactsDir, { recursive: true });
     const created = await lstat(artifactsDir);
     if (created.isSymbolicLink() || !created.isDirectory()) {
@@ -174,15 +176,6 @@ export async function ensureRealArtifactsDirectory(runDirectory: string): Promis
   return artifactsDir;
 }
 
-/** ENOENT identity shared with settlement.ts's hardened audit-artifact path. */
-function isMissingPathError(error: unknown): boolean {
-  return (
-    error instanceof Error &&
-    "code" in error &&
-    (error as { code?: unknown }).code === "ENOENT"
-  );
-}
-
 /** Cycle- and bigint-safe JSON replacer so serialization itself cannot drop data. */
 function jsonSafeReplacer(): (key: string, value: unknown) => unknown {
   const seen = new WeakSet<object>();

package/src/public-cli/cli.ts

@@ -105,6 +105,8 @@ import {
 } from "./settlement.ts";
 import type { TerminalResult } from "./terminal.ts";
 
+import { errorText } from "../unknown-value.ts";
+
 export {
   buildExplicitInternalActivationArgs,
   resolveInternalRoleEntrypoint,
@@ -237,8 +239,6 @@ export type CliEnv = {
   signal?: AbortSignal;
 };
 
-
-
 type RoleEnvironmentOptions = {
   role: PublicCallableRole;
   home: string;
@@ -542,7 +542,7 @@ function requireLegalEngineModel(model: string): string {
     return assertLegalEngineModel(model);
   } catch (error) {
     throw new CliUsageError(
-      error instanceof Error ? error.message : String(error),
+      errorText(error),
       { cause: error },
     );
   }
@@ -553,7 +553,7 @@ function requireLegalEngineName(name: string): string {
     return assertLegalEngineName(name);
   } catch (error) {
     throw new CliUsageError(
-      error instanceof Error ? error.message : String(error),
+      errorText(error),
       { cause: error },
     );
   }
@@ -585,7 +585,7 @@ function loadAndValidateConfig(
       validatePublicCliConfigAxes(config, packageRoot);
     } catch (error) {
       throw new CliUsageError(
-        error instanceof Error ? error.message : String(error),
+        errorText(error),
         { cause: error },
       );
     }
@@ -917,7 +917,7 @@ async function runConfigCommand(
     try {
       config = setPersistentSeatHost(config, seat, unset ? undefined : args[2]!);
     } catch (error) {
-      throw new CliUsageError(error instanceof Error ? error.message : String(error), { cause: error });
+      throw new CliUsageError(errorText(error), { cause: error });
     }
     await savePublicCliConfig(config, home);
     io.stdout(renderConfig(config, home));
@@ -941,7 +941,7 @@ async function runConfigCommand(
       config = setPersistentSeatEngine(config, seat, name, engineModel);
     } catch (error) {
       throw new CliUsageError(
-        error instanceof Error ? error.message : String(error),
+        errorText(error),
         { cause: error },
       );
     }
@@ -963,7 +963,7 @@ async function runConfigCommand(
       config = setPersistentSeatEngine(config, seat, undefined);
     } catch (error) {
       throw new CliUsageError(
-        error instanceof Error ? error.message : String(error),
+        errorText(error),
         { cause: error },
       );
     }
@@ -987,7 +987,7 @@ async function runConfigCommand(
       config = setPersistentSeatEngineModel(config, seat, model);
     } catch (error) {
       throw new CliUsageError(
-        error instanceof Error ? error.message : String(error),
+        errorText(error),
         { cause: error },
       );
     }
@@ -1009,7 +1009,7 @@ async function runConfigCommand(
       config = setPersistentSeatEngineModel(config, seat, undefined);
     } catch (error) {
       throw new CliUsageError(
-        error instanceof Error ? error.message : String(error),
+        errorText(error),
         { cause: error },
       );
     }

package/src/public-cli/config.ts

@@ -27,6 +27,7 @@ import {
   type PublicConfigurableSeat,
   type PublicThinkingLevel,
 } from "./registry.ts";
+import { isEnoent, isRecord } from "../unknown-value.ts";
 
 /** Province officers that may carry a persistent model override (#453). Registry order. */
 type ProvinceConfigRole = Extract<
@@ -134,13 +135,7 @@ export async function loadPublicCliConfig(
     const raw = await readFile(path, "utf8");
     return parsePublicCliConfig(JSON.parse(raw));
   } catch (error) {
-    if (
-      error instanceof Error &&
-      "code" in error &&
-      (error as NodeJS.ErrnoException).code === "ENOENT"
-    ) {
-      return { seats: {} };
-    }
+    if (isEnoent(error)) return { seats: {} };
     throw error;
   }
 }
@@ -481,7 +476,7 @@ function serializePublicCliConfig(config: PublicCliConfig): {
 }
 
 function parsePublicCliConfig(value: unknown): PublicCliConfig {
-  if (value === null || typeof value !== "object" || Array.isArray(value)) {
+  if (!isRecord(value)) {
     throw new Error("public CLI config must be an object");
   }
   const record = value as {
@@ -500,11 +495,7 @@ function parsePublicCliConfig(value: unknown): PublicCliConfig {
   // disk-shaped seats map and a memory-shaped bucket both survive.
   const unknownSeats: Record<string, unknown> = {};
   if (record.unknownSeats !== undefined) {
-    if (
-      record.unknownSeats === null ||
-      typeof record.unknownSeats !== "object" ||
-      Array.isArray(record.unknownSeats)
-    ) {
+    if (!isRecord(record.unknownSeats)) {
       throw new Error("public CLI config.unknownSeats must be an object");
     }
     Object.assign(unknownSeats, record.unknownSeats as Record<string, unknown>);
@@ -517,11 +508,7 @@ function parsePublicCliConfig(value: unknown): PublicCliConfig {
   if (record.seats === undefined) {
     return withOpaque({});
   }
-  if (
-    record.seats === null ||
-    typeof record.seats !== "object" ||
-    Array.isArray(record.seats)
-  ) {
+  if (!isRecord(record.seats)) {
     throw new Error("public CLI config.seats must be an object");
   }
   const seats: PublicCliConfig["seats"] = {};
@@ -542,7 +529,7 @@ function parsePublicCliConfig(value: unknown): PublicCliConfig {
 }
 
 function parseSeatModelConfig(value: unknown, seat: string): PersistentSeatConfig {
-  if (value === null || typeof value !== "object" || Array.isArray(value)) {
+  if (!isRecord(value)) {
     throw new Error(`config seat ${seat} must be an object`);
   }
   const raw = value as Record<string, unknown>;
@@ -825,7 +812,7 @@ export function listRolesForDisplay(
 export function credentialProvidersFromAuthData(
   data: unknown,
 ): CredentialProviders {
-  if (data === null || typeof data !== "object" || Array.isArray(data)) {
+  if (!isRecord(data)) {
     return { "openai-codex": false, xai: false };
   }
   const record = data as Record<string, unknown>;
@@ -850,13 +837,7 @@ export async function loadCredentialProviders(
     const raw = await readFile(join(agentDir, "auth.json"), "utf8");
     return credentialProvidersFromAuthData(JSON.parse(raw));
   } catch (error) {
-    if (
-      error instanceof Error &&
-      "code" in error &&
-      (error as NodeJS.ErrnoException).code === "ENOENT"
-    ) {
-      return { "openai-codex": false, xai: false };
-    }
+    if (isEnoent(error)) return { "openai-codex": false, xai: false };
     throw error;
   }
 }

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

@@ -9,6 +9,8 @@ import type { PostAdmissionEnv } from "./post-admission.ts";
 import type { CliIo } from "./cli-io.ts";
 import type { TerminalRoleOutcome } from "./terminal.ts";
 
+import { isRecord } from "../unknown-value.ts";
+
 export type CountersignRunEnv = PostAdmissionEnv & {
   principalAuthority: DurablePrincipalAuthority;
   createRunId?: () => string;
@@ -51,10 +53,6 @@ export type CourtDiaristInvocationResult = {
   readonly failedWithoutEscalate?: { readonly diagnostic: string };
 };
 
-function isRecord(value: unknown): value is Record<string, unknown> {
-  return typeof value === "object" && value !== null && !Array.isArray(value);
-}
-
 function isEscalatePayload(payload: unknown): boolean {
   if (!isRecord(payload)) return false;
   return (

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

@@ -101,6 +101,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"
@@ -159,7 +161,7 @@ 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)
+  const status = isRecord(payload)
     ? (payload as Record<string, unknown>).status
     : undefined;
   return typeof status === "string" && route.statuses.has(status)
@@ -659,7 +661,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 };
@@ -997,7 +999,7 @@ 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") {

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);
   }