@akagilnc/pi-workflow-roles 0.1.5368 → 0.1.5375

package/dist/ticket-provenance-contracts.js

@@ -1,3 +1,4 @@
+import { isRecord } from "./unknown-value.js";
 /**
  * 起居录（ticket-provenance）typed 形状 —— ADR 0075 / ADR 0081 / #1090。
  *
@@ -7,9 +8,6 @@
  */
 /** Sitian kind for per-ticket court diary volumes. */
 export const TICKET_PROVENANCE_KIND = "ticket-provenance";
-function isRecord(value) {
-    return typeof value === "object" && value !== null && !Array.isArray(value);
-}
 function positiveInteger(value) {
     if (typeof value === "number" && Number.isSafeInteger(value) && value >= 1)
         return value;

package/dist/ticket-provenance.js

@@ -11,6 +11,7 @@ import { isSafePositiveTicketNumber } from "./run-ticket-number.js";
 import { adaptSessionDialogue, nativeEventId } from "./session-dialogue.js";
 import { appendSitianRecordBlock, appendSitianRecord, resolveSitianRecordPath, } from "./sitian-facade.js";
 import { TICKET_PROVENANCE_KIND, } from "./ticket-provenance-contracts.js";
+import { errorText } from "./unknown-value.js";
 /**
  * Typed input failure for diarist bounds/session path (reask, not infrastructure).
  * Accept hook discriminates with instanceof — never Error.message prefixes.
@@ -199,7 +200,7 @@ async function projectSessionRanges(input) {
             throw error;
         const code = errnoCode(error);
         if (code === "ENOENT" || code === "ENOTDIR" || code === "EISDIR") {
-            const detail = error instanceof Error ? error.message : String(error);
+            const detail = errorText(error);
             throw new TicketProvenanceInputError(`session unreadable: ${input.session.path} (${detail})`, { cause: error });
         }
         throw error;

package/dist/typed-provider-http.js

@@ -1,5 +1,6 @@
 import { readFile, unlink, writeFile } from "node:fs/promises";
 import { join } from "node:path";
+import { isEnoent, isRecord } from "./unknown-value.js";
 const TYPED_HTTP_FILE = "typed-provider-http.json";
 function typedProviderHttpPath(runDirectory) {
   return join(runDirectory, TYPED_HTTP_FILE);
@@ -8,9 +9,7 @@ async function clearTypedProviderHttpObservation(runDirectory) {
   try {
     await unlink(typedProviderHttpPath(runDirectory));
   } catch (error) {
-    if (error instanceof Error && "code" in error && error.code === "ENOENT") {
-      return;
-    }
+    if (isEnoent(error)) return;
     throw error;
   }
 }
@@ -35,13 +34,11 @@ async function readLatestTypedProviderHttpObservation(runDirectory) {
   try {
     text = await readFile(typedProviderHttpPath(runDirectory), "utf8");
   } catch (error) {
-    if (error instanceof Error && "code" in error && error.code === "ENOENT") {
-      return void 0;
-    }
+    if (isEnoent(error)) return void 0;
     throw error;
   }
   const raw = JSON.parse(text);
-  if (raw === null || typeof raw !== "object" || Array.isArray(raw)) {
+  if (!isRecord(raw)) {
     throw new Error("typed provider HTTP observation must be a JSON object");
   }
   const record = raw;

package/dist/unknown-value.js

@@ -0,0 +1,28 @@
+import { Guard } from "typebox/guard";
+/**
+ * Sole readers for a plain object, a Node errno, and an Error message.
+ * Callers choose which codes count; ENOTDIR is not ENOENT.
+ * Plain objects are TypeBox `Guard.IsObjectNotArray`.
+ */
+export function isRecord(value) {
+    return Guard.IsObjectNotArray(value);
+}
+export function errnoCode(error) {
+    return error !== null && typeof error === "object" && "code" in error
+        && typeof error.code === "string"
+        ? error.code
+        : undefined;
+}
+/** `Error.message`, otherwise `String(value)`. */
+export function errorText(error) {
+    return error instanceof Error ? error.message : String(error);
+}
+/** True only for ENOENT on an Error. ENOTDIR and every other code stay distinct. */
+export function isEnoent(error) {
+    return error instanceof Error && errnoCode(error) === "ENOENT";
+}
+/** ENOENT or ENOTDIR on an Error. */
+export function isMissingPathError(error) {
+    const code = errnoCode(error);
+    return error instanceof Error && (code === "ENOENT" || code === "ENOTDIR");
+}

package/dist/user-dialogue-stdin.js

@@ -3,12 +3,13 @@
  * Pi's piped-stdin reader trims the wrapper; the body field keeps original bytes.
  * No size cap and no path/pointer substitute — the whole dialogue rides the envelope.
  */
+import { isRecord } from "./unknown-value.js";
 export const USER_DIALOGUE_STDIN_KIND = "ak-user-dialogue";
 export function encodeUserDialogueStdin(body) {
     return JSON.stringify({ kind: USER_DIALOGUE_STDIN_KIND, body });
 }
 function isUserDialogueEnvelope(value) {
-    if (value === null || typeof value !== "object" || Array.isArray(value))
+    if (!isRecord(value))
         return false;
     const record = value;
     return record.kind === USER_DIALOGUE_STDIN_KIND && typeof record.body === "string";

package/package.json

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

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

@@ -27,6 +27,8 @@ import {
 } from "../prepared-role-turn.ts";
 import { acpModelId, type AcpHostDescription } from "./description.ts";
 
+import { isRecord } from "../unknown-value.ts";
+
 /**
  * #959: collect free-form agent text from ACP session/update stream.
  * Used only when the navigator seat spoke prose without calling the output tool.
@@ -42,7 +44,7 @@ function acpAgentTextChunk(params: Readonly<Record<string, unknown>>): string |
   let textFallback: unknown;
 
   const nested = params.update;
-  if (typeof nested === "object" && nested !== null && !Array.isArray(nested)) {
+  if (isRecord(nested)) {
     const record = nested as Record<string, unknown>;
     kind = record.sessionUpdate;
     content = record.content;
@@ -55,7 +57,7 @@ function acpAgentTextChunk(params: Readonly<Record<string, unknown>>): string |
 
   if (kind !== "agent_message_chunk" && kind !== "agent_message") return undefined;
   if (typeof content === "string" && content.length > 0) return content;
-  if (typeof content === "object" && content !== null && !Array.isArray(content)) {
+  if (isRecord(content)) {
     const record = content as Record<string, unknown>;
     if (typeof record.text === "string" && record.text.length > 0) return record.text;
   }

package/src/activation-ledger-git.ts

@@ -1,6 +1,8 @@
 import { execFileSync } from "node:child_process";
 import { basename, dirname, isAbsolute, resolve } from "node:path";
 
+import { errnoCode } from "./unknown-value.ts";
+
 /** Git discovery env vars that must not influence book-key resolution from cwd. */
 const GIT_DISCOVERY_ENV_KEYS = [
   "GIT_DIR",
@@ -60,8 +62,7 @@ export class ActivationGitRepositoryRequiredError extends Error {
 
 /** Spawn/OS failure identity for the git child — not a repository-status result. */
 function isGitSpawnInfrastructureError(error: unknown): boolean {
-  if (error === null || typeof error !== "object" || !("code" in error)) return false;
-  const code = (error as { code: unknown }).code;
+  const code = errnoCode(error);
   return code === "ENOENT" || code === "EACCES" || code === "EPERM";
 }
 

package/src/activation-ledger-topology.ts

@@ -7,6 +7,10 @@ import {
 import { userInfo } from "node:os";
 import { basename, dirname, isAbsolute, join, relative, resolve, sep } from "node:path";
 
+import { errnoCode, errorText } from "./unknown-value.ts";
+
+export { errnoCode, errorText };
+
 /**
  * Typed activation-ledger path/fs failure. Callers discriminate with instanceof/code;
  * never by parsing message prose. Original filesystem causes are retained.
@@ -128,18 +132,6 @@ export function physicallyContainedIn(root: string, candidate: string): boolean
   return pathContainedIn(physicalPathIdentity(root), physicalPathIdentity(candidate));
 }
 
-export function errnoCode(error: unknown): string | undefined {
-  return error !== null && typeof error === "object" && "code" in error
-    && typeof (error as { code: unknown }).code === "string"
-    ? (error as { code: string }).code
-    : undefined;
-}
-
-export function errorText(error: unknown): string {
-  if (!(error instanceof Error)) return String(error);
-  return error.message;
-}
-
 /**
  * Ensure the configured ledger root is a physical directory identity.
  * lstat before create/realpath: a root symlink is never admitted, even when its

package/src/analyst-entry.ts

@@ -46,6 +46,8 @@ import {
   type AnalystUnreadableRun,
 } from "./analyst-page.ts";
 
+import { isMissingPathError } from "./unknown-value.ts";
+
 /** #338 compute-if-missing failure — issue identity + real cause (CLI → ControlledFailure). */
 export class AnalystIssueComputeError extends Error {
   readonly code = "analyst-issue-compute-failed" as const;
@@ -80,14 +82,6 @@ export class AnalystIssueComputeError extends Error {
   }
 }
 
-function isMissingPathError(error: unknown): boolean {
-  return (
-    error instanceof Error
-    && "code" in error
-    && (error.code === "ENOENT" || error.code === "ENOTDIR")
-  );
-}
-
 /** Issue-mode typed input — book × ticket scope (#399). */
 export type AnalystIssueModeInput = {
   readonly mode: "issue";
@@ -123,7 +117,7 @@ export type AnalystIssueModeInput = {
  * Sole sweep-mode input contract (#298/#329/#337).
  * Schema is the single definition; TS types are derived (no parallel hand shape).
  * projectRoot = string (not nonempty); changedLines optional finite non-negative;
- * 0 remains typed 空缺; no extra keys.
+ * 0 remains typed 空缺. Extra keys are not a reject reason.
  */
 export const analystSweepModeInputSchema = Type.Object(
   {
@@ -137,11 +131,9 @@ export const analystSweepModeInputSchema = Type.Object(
             Type.Number({ minimum: 0, maximum: Number.MAX_VALUE }),
           ),
         },
-        { additionalProperties: false },
       ),
     ),
   },
-  { additionalProperties: false },
 );
 
 /** Sweep-mode typed input — 已并 PR 清单 + LOC → 补算缺页 + 维护全库索引. */

package/src/analyst-gate-cycles-read.ts

@@ -38,6 +38,7 @@ import {
   type LedgerSessionRow,
 } from "./ledger-session-read.ts";
 
+import { isRecord, errorText, isEnoent } from "./unknown-value.ts";
 
 /** One completed gate round: direct officer receipt or historical province/officer pair. */
 /** Honest origin discriminant: direct summons vs historical province dispatch. */
@@ -88,10 +89,6 @@ const OFFICER_ARG_ALIASES: Readonly<Record<string, "inspector" | "notary">> = {
   notary: "notary",
 };
 
-function isRecord(value: unknown): value is Record<string, unknown> {
-  return typeof value === "object" && value !== null && !Array.isArray(value);
-}
-
 function isParentAttemptBindingRow(row: LedgerSessionRow): boolean {
   return row.type === "custom" && row.customType === AUDITOR_PARENT_ATTEMPT_BINDING_ENTRY_TYPE;
 }
@@ -104,15 +101,6 @@ function intervalRowsForGateCall(
   return intervalRowsAroundAnchor(rows, callRowIndex, isParentAttemptBindingRow).rows;
 }
 
-/** Only true absence (ENOENT). ENOTDIR is damaged topology — must stay loud. */
-function isMissingDirectoryError(error: unknown): boolean {
-  return (
-    error instanceof Error
-    && "code" in error
-    && error.code === "ENOENT"
-  );
-}
-
 function normalizeOfficerArg(raw: unknown): "inspector" | "notary" | undefined {
   if (typeof raw !== "string") return undefined;
   return OFFICER_ARG_ALIASES[raw.trim()];
@@ -438,7 +426,7 @@ async function resolveOfficerSessionFromPointerFile(
     raw = JSON.parse(await readFile(pointerPath, "utf8"));
   } catch (error) {
     throw new Error(
-      `direct officer run pointer unreadable in ${pointerPath}: ${error instanceof Error ? error.message : String(error)}`,
+      `direct officer run pointer unreadable in ${pointerPath}: ${errorText(error)}`,
       { cause: error },
     );
   }
@@ -490,7 +478,7 @@ export async function readAnalystGateCyclesFromAuditorRoles(
         .map((e) => e.name)
         .sort();
     } catch (error) {
-      if (isMissingDirectoryError(error)) continue;
+      if (isEnoent(error)) continue;
       throw error;
     }
     for (const name of names) {

package/src/analyst-index.ts

@@ -20,6 +20,7 @@
  */
 import { open, readFile, unlink } from "node:fs/promises";
 import { dirname, join } from "node:path";
+import { setTimeout as sleep } from "node:timers/promises";
 
 import { writeFileAtomically } from "./atomic-write.ts";
 import {
@@ -28,6 +29,7 @@ import {
   physicalPathIdentity,
 } from "./activation-ledger-topology.ts";
 import { resolveAnalystBookKey } from "./analyst-book-key.ts";
+import { isMissingPathError } from "./unknown-value.ts";
 import type {
   AnalystIssueMetricsPage,
   AnalystOptionalMetricNumber,
@@ -38,12 +40,6 @@ const LIBRARY_INDEX_LOCK_NAME = ".library-index.lock";
 const LIBRARY_INDEX_LOCK_TIMEOUT_MS = 30_000;
 const LIBRARY_INDEX_LOCK_RETRY_MS = 15;
 
-function sleep(ms: number): Promise<void> {
-  return new Promise((resolve) => {
-    setTimeout(resolve, ms);
-  });
-}
-
 /**
  * Exclusive create lock for the library-index read→upsert→write critical section.
  * Same-directory sibling of the index file; not a second index and not a daemon.
@@ -270,13 +266,7 @@ export async function readAnalystLibraryIndexPage(
   try {
     raw = await readFile(path, "utf8");
   } catch (error) {
-    if (
-      error instanceof Error
-      && "code" in error
-      && (error.code === "ENOENT" || error.code === "ENOTDIR")
-    ) {
-      return undefined;
-    }
+    if (isMissingPathError(error)) return undefined;
     throw error;
   }
   const parsed: unknown = JSON.parse(raw);

package/src/analyst-ledger.ts

@@ -49,23 +49,9 @@ import {
   type AnalystGateCycleRound,
 } from "./analyst-gate-cycles-read.ts";
 
-export type { AnalystGateCycleRound } from "./analyst-gate-cycles-read.ts";
-
-function isMissingPathError(error: unknown): boolean {
-  return (
-    error instanceof Error
-    && "code" in error
-    && (error.code === "ENOENT" || error.code === "ENOTDIR")
-  );
-}
+import { isRecord, errorText, isMissingPathError } from "./unknown-value.ts";
 
-function errorText(error: unknown): string {
-  return error instanceof Error ? error.message : String(error);
-}
-
-function isRecord(value: unknown): value is Record<string, unknown> {
-  return typeof value === "object" && value !== null && !Array.isArray(value);
-}
+export type { AnalystGateCycleRound } from "./analyst-gate-cycles-read.ts";
 
 /** Existing public-cli run-state live phases — not a new state machine. */
 const LIVE_RUN_STATES = new Set(["admitted", "running", "resumable"]);

package/src/analyst-metric-families/acceptance-success-rework.ts

@@ -14,6 +14,8 @@ import { medianNumber } from "../analyst-median.ts";
 import type { AnalystMetricFamilyModule } from "../analyst-metric-family.ts";
 import { packagedAnalystTerminal } from "../packaged-role-registry.ts";
 
+import { isRecord } from "../unknown-value.ts";
+
 const WORKER_ROLES = new Set(["coder", "fixer"]);
 
 /** Lawful acceptance vocabulary by role (PRD 受理终态映射). */
@@ -95,10 +97,6 @@ export type AnalystAcceptanceSuccessReworkSection = {
   readonly rework: AnalystReworkLens;
 };
 
-function isRecord(value: unknown): value is Record<string, unknown> {
-  return typeof value === "object" && value !== null && !Array.isArray(value);
-}
-
 /** Locate collector groups array: receipt.groups or top-level groups. */
 function findCollectorGroups(body: Record<string, unknown>): unknown {
   if (Array.isArray(body.groups)) return body.groups;

package/src/analyst-metric-families/round-timeline.ts

@@ -17,6 +17,8 @@ import type {
   AnalystUnreadableRun,
 } from "../analyst-page.ts";
 
+import { isRecord } from "../unknown-value.ts";
+
 export type AnalystRoundTimelineTerminal =
   | {
       readonly kind: "receipt";
@@ -66,10 +68,6 @@ export type AnalystRoundTimelineSection = {
   readonly lanes: readonly AnalystRoundTimelineLane[];
 };
 
-function isRecord(value: unknown): value is Record<string, unknown> {
-  return typeof value === "object" && value !== null && !Array.isArray(value);
-}
-
 function readClassCount(body: Record<string, unknown>): number | undefined {
   if (!isRecord(body.outcome)) return undefined;
   if (!isRecord(body.outcome.decisiveFacts)) return undefined;

package/src/analyst-page.ts

@@ -6,9 +6,9 @@
  * top-level sections via directory discovery — B/C waves add family files
  * without forking the page writer or editing a shared registry list.
  */
-import { createHash } from "node:crypto";
 import { dirname, join } from "node:path";
 
+import { sha256Hex } from "./sha256.ts";
 import { writeFileAtomically } from "./atomic-write.ts";
 import {
   assertLedgerFileInsideHome,
@@ -161,7 +161,7 @@ export function analystIssuePageKey(address: AnalystIssuePageAddress): string {
   } else if (address.scopeRootIdentity !== undefined) {
     parts.push("root", physicalPathIdentity(address.scopeRootIdentity));
   }
-  return createHash("sha256").update(parts.join("\0")).digest("hex").slice(0, 32);
+  return sha256Hex(parts.join("\0")).slice(0, 32);
 }
 
 export function analystIssuePagePath(

package/src/archivist-record-topology.ts

@@ -7,9 +7,9 @@
  * books/<book>/navigator/<work-subject-digest>/. Callers must not re-hash
  * subject or re-join book/navigator/digest themselves.
  */
-import { createHash } from "node:crypto";
 import { join } from "node:path";
 
+import { sha256Hex } from "./sha256.ts";
 import { resolveBookKeyFromGit } from "./activation-ledger-git.ts";
 import {
   activationBookDirectory,
@@ -52,7 +52,7 @@ export function resolveNavigatorWorkSubjectPlacement(input: {
     }
   }
   ledgerHome ??= resolveActivationLedgerHome(input.home);
-  const digest = createHash("sha256").update(input.subject).digest("hex").slice(0, 32);
+  const digest = sha256Hex(input.subject).slice(0, 32);
   return {
     ledgerHome,
     sessionDir: join(

package/src/audit-escalation.ts

@@ -1,7 +1,8 @@
+import { isRecord } from "./unknown-value.ts";
 /** Historical audit-escalation discriminator retained for reading pre-#1057 run artifacts. */
 export const AUDIT_ESCALATION_KIND = "audit_escalation" as const;
 
 export function isAuditEscalationResult(value: unknown): value is { readonly kind: typeof AUDIT_ESCALATION_KIND } {
-  return typeof value === "object" && value !== null && !Array.isArray(value)
+  return isRecord(value)
     && (value as Record<string, unknown>).kind === AUDIT_ESCALATION_KIND;
 }

package/src/auditor-soul.ts

@@ -1,6 +1,8 @@
 import { readFile } from "node:fs/promises";
 import { basename, join } from "node:path";
 
+import { isRecord } from "./unknown-value.ts";
+
 import {
   joinPackageMaterials,
   readPackageMaterial,
@@ -126,7 +128,7 @@ export async function readAuditorResumeBinding(runDirectory: string): Promise<
     if ((error as NodeJS.ErrnoException).code === "ENOENT") return undefined;
     throw error;
   }
-  if (raw === null || typeof raw !== "object" || Array.isArray(raw)) return undefined;
+  if (!isRecord(raw)) return undefined;
   const record = raw as { sourceRunPath?: unknown; auditorSubject?: unknown };
   if (typeof record.sourceRunPath !== "string" || record.sourceRunPath.trim() === "") return undefined;
   const subject = isAuditorSoulRole(record.auditorSubject)

package/src/book-topology-migration-placement.ts

@@ -19,22 +19,11 @@ import {
   readMigrationDerivedTicketNumber,
 } from "./run-ticket-number.ts";
 
+import { isRecord, isEnoent } from "./unknown-value.ts";
+
 const TICKET_NUMBER_RE = /^[1-9][0-9]*$/;
 const RUN_DIR_NAME_RE = /^([^@]+)@([^@]+)$/;
 
-function isRecord(value: unknown): value is Record<string, unknown> {
-  return typeof value === "object" && value !== null && !Array.isArray(value);
-}
-
-/** Sole migration ENOENT projection — non-ENOENT must propagate. */
-export function isMigrationEnoent(error: unknown): boolean {
-  return (
-    error instanceof Error &&
-    "code" in error &&
-    (error as NodeJS.ErrnoException).code === "ENOENT"
-  );
-}
-
 /** Book-key directories under a books root; missing root → []. */
 export async function listMigrationBookKeys(
   booksDirectory: string,
@@ -46,7 +35,7 @@ export async function listMigrationBookKeys(
       .map((entry) => entry.name)
       .sort((a, b) => a.localeCompare(b));
   } catch (error) {
-    if (isMigrationEnoent(error)) return [];
+    if (isEnoent(error)) return [];
     throw error;
   }
 }
@@ -59,7 +48,7 @@ export async function listMigrationDirents(
     const entries = await readdir(directory, { withFileTypes: true });
     return [...entries].sort((a, b) => a.name.localeCompare(b.name));
   } catch (error) {
-    if (isMigrationEnoent(error)) return [];
+    if (isEnoent(error)) return [];
     throw error;
   }
 }
@@ -109,7 +98,7 @@ async function readProjectRootFromRun(
         return { projectRoot, sourcePage: page };
       }
     } catch (error) {
-      if (isMigrationEnoent(error)) continue;
+      if (isEnoent(error)) continue;
       throw error;
     }
   }

package/src/book-topology-migrators-t11.ts

@@ -25,7 +25,6 @@ import {
 import {
   findPlacedMigratingRun,
   isFlatRunsRelative,
-  isMigrationEnoent,
   isTicketNumberString,
   listBackupRunLeaves,
   listMigrationBookKeys,
@@ -37,6 +36,8 @@ import {
   type RunDirectoryPathRewrite,
 } from "./role-run-relocation.ts";
 
+import { isRecord, errorText, isEnoent } from "./unknown-value.ts";
+
 const AUDITOR_ROLES_PARTITION = "auditor-roles";
 const ISSUES_PARTITION = "issues";
 const DEPRECATED_KINDS_PARTITION = "deprecated-kinds";
@@ -101,7 +102,7 @@ async function directoryExists(path: string): Promise<boolean> {
     const info = await stat(path);
     return info.isDirectory();
   } catch (error) {
-    if (isMigrationEnoent(error)) return false;
+    if (isEnoent(error)) return false;
     throw error;
   }
 }
@@ -219,7 +220,7 @@ async function readJsonlParentSession(path: string): Promise<string | undefined>
   try {
     raw = await readFile(path, "utf8");
   } catch (error) {
-    if (isMigrationEnoent(error)) return undefined;
+    if (isEnoent(error)) return undefined;
     throw error;
   }
   const newline = raw.indexOf("\n");
@@ -227,7 +228,7 @@ async function readJsonlParentSession(path: string): Promise<string | undefined>
   if (line.length === 0) return undefined;
   try {
     const header: unknown = JSON.parse(line);
-    if (header === null || typeof header !== "object" || Array.isArray(header)) {
+    if (!isRecord(header)) {
       return undefined;
     }
     const parentSession = (header as { parentSession?: unknown }).parentSession;
@@ -268,13 +269,13 @@ async function findTrueVolumeParent(
   try {
     ledgerRaw = await readFile(join(volumeDirectory, CURRENT_SESSION_LEDGER), "utf8");
   } catch (error) {
-    if (isMigrationEnoent(error)) return undefined;
+    if (isEnoent(error)) return undefined;
     throw error;
   }
   let sessionFileField: string;
   try {
     const ledger: unknown = JSON.parse(ledgerRaw);
-    if (ledger === null || typeof ledger !== "object" || Array.isArray(ledger)) {
+    if (!isRecord(ledger)) {
       return undefined;
     }
     const sessionFile = (ledger as { sessionFile?: unknown }).sessionFile;
@@ -298,7 +299,7 @@ async function findTrueVolumeParent(
     const info = await lstat(liveSession);
     if (!info.isFile()) return undefined;
   } catch (error) {
-    if (isMigrationEnoent(error)) return undefined;
+    if (isEnoent(error)) return undefined;
     throw error;
   }
 
@@ -413,7 +414,7 @@ async function readSessionFileField(path: string): Promise<string | undefined> {
   try {
     raw = await readFile(path, "utf8");
   } catch (error) {
-    if (isMigrationEnoent(error)) return undefined;
+    if (isEnoent(error)) return undefined;
     throw error;
   }
   let parsed: unknown;
@@ -422,12 +423,12 @@ async function readSessionFileField(path: string): Promise<string | undefined> {
   } catch (error) {
     throw new Error(
       `book topology migration cannot read current-session pointer at ${path}: invalid JSON (${
-        error instanceof Error ? error.message : String(error)
+        errorText(error)
       })`,
       { cause: error },
     );
   }
-  if (parsed === null || typeof parsed !== "object" || Array.isArray(parsed)) {
+  if (!isRecord(parsed)) {
     throw new Error(
       `book topology migration cannot read current-session pointer at ${path}: expected a JSON object`,
     );
@@ -480,7 +481,7 @@ async function mergeRewrittenCurrentSession(
     srcStat = await stat(srcPath);
     srcRaw = await readFile(srcPath, "utf8");
   } catch (error) {
-    if (isMigrationEnoent(error)) return;
+    if (isEnoent(error)) return;
     throw error;
   }
 
@@ -488,7 +489,7 @@ async function mergeRewrittenCurrentSession(
   let nextSessionFile: string | undefined;
   try {
     const parsed: unknown = JSON.parse(srcRaw);
-    if (parsed !== null && typeof parsed === "object" && !Array.isArray(parsed)) {
+    if (isRecord(parsed)) {
       const record = parsed as Record<string, unknown>;
       if (typeof record.sessionFile === "string") {
         const rewritten = rewritePathIntoDestination(
@@ -510,7 +511,7 @@ async function mergeRewrittenCurrentSession(
   try {
     destStat = await stat(destPath);
   } catch (error) {
-    if (!isMigrationEnoent(error)) throw error;
+    if (!isEnoent(error)) throw error;
   }
   await writeWinningCurrentSession(
     destPath,
@@ -656,7 +657,7 @@ async function copyIssuesNonRunTree(source: string, destination: string): Promis
         `book topology migration refuses to overwrite ${to} with ${from}`,
       );
     } catch (error) {
-      if (!isMigrationEnoent(error)) throw error;
+      if (!isEnoent(error)) throw error;
     }
     await cp(from, to, { preserveTimestamps: true });
   }
@@ -719,7 +720,7 @@ async function unlinkIfPresent(path: string): Promise<void> {
   try {
     await unlink(path);
   } catch (error) {
-    if (!isMigrationEnoent(error)) throw error;
+    if (!isEnoent(error)) throw error;
   }
 }