@akagilnc/pi-workflow-roles 0.1.5368 → 0.1.5382

package/src/worker-submission-gates.ts

@@ -17,6 +17,8 @@ import {
 } from "./submission-errors.ts";
 import { WORKER_DONE_STATUSES } from "./worker-submission-contracts.ts";
 
+import { isRecord } from "./unknown-value.ts";
+
 export { WorkerCommitReminderError, WorkerPrefixReminderError, WorkerUnfinishedReasonReminderError } from "./submission-errors.ts";
 export { WORKER_DONE_STATUSES } from "./worker-submission-contracts.ts";
 
@@ -152,15 +154,11 @@ function uninstallPackageWorkerHooks(cwd: string): void {
   }
 }
 
-function isRecord(value: unknown): value is Record<string, unknown> {
-  return typeof value === "object" && value !== null && !Array.isArray(value);
-}
-
 /** ADR 0050: a written explanation, including text carried by an object key. */
 function containsWrittenReason(value: unknown): boolean {
   if (typeof value === "string") return value.trim().length > 0;
   if (Array.isArray(value)) return value.some(containsWrittenReason);
-  if (typeof value === "object" && value !== null) {
+  if (isRecord(value)) {
     return Object.entries(value).some(
       ([key, item]) => containsWrittenReason(key) || containsWrittenReason(item),
     );
@@ -169,8 +167,8 @@ function containsWrittenReason(value: unknown): boolean {
 }
 
 function unfinishedReasonPresent(details?: unknown): boolean {
-  if (typeof details !== "object" || details === null || Array.isArray(details)) return false;
-  return containsWrittenReason((details as { reason?: unknown }).reason);
+  if (!isRecord(details)) return false;
+  return containsWrittenReason(details.reason);
 }
 
 function readGateState(session: HostRecordSession): {
@@ -347,3 +345,6 @@ export function createWorkerSubmissionGate(
     },
   };
 }
+
+/** The gate object returned above. Callers reference this instead of restating assertAcceptable. */
+export type WorkerSubmissionGate = ReturnType<typeof createWorkerSubmissionGate>;