@akagilnc/pi-workflow-roles 0.1.5368 → 0.1.5382

package/dist/public-cli/config.js

@@ -19,6 +19,7 @@ import {
   isPublicConfigurableSeat,
   seatModelOnly
 } from "./registry.js";
+import { isEnoent, isRecord } from "../unknown-value.js";
 const GATE_OFFICER_SEATS = PUBLIC_ROLE_RECORDS.flatMap(
   (record) => "provinceConfig" in record && record.provinceConfig === true ? [record.role] : []
 );
@@ -37,9 +38,7 @@ async function loadPublicCliConfig(home) {
     const raw = await readFile(path, "utf8");
     return parsePublicCliConfig(JSON.parse(raw));
   } catch (error) {
-    if (error instanceof Error && "code" in error && error.code === "ENOENT") {
-      return { seats: {} };
-    }
+    if (isEnoent(error)) return { seats: {} };
     throw error;
   }
 }
@@ -260,7 +259,7 @@ function serializePublicCliConfig(config) {
   };
 }
 function parsePublicCliConfig(value) {
-  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;
@@ -270,7 +269,7 @@ function parsePublicCliConfig(value) {
   }
   const unknownSeats = {};
   if (record.unknownSeats !== void 0) {
-    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);
@@ -283,7 +282,7 @@ function parsePublicCliConfig(value) {
   if (record.seats === void 0) {
     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 = {};
@@ -299,7 +298,7 @@ function parsePublicCliConfig(value) {
   return withOpaque(seats);
 }
 function parseSeatModelConfig(value, seat) {
-  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;
@@ -481,7 +480,7 @@ function listRolesForDisplay(config, credentials, invocation) {
   return effectiveSeatConfigurations(config, credentials, invocation);
 }
 function credentialProvidersFromAuthData(data) {
-  if (data === null || typeof data !== "object" || Array.isArray(data)) {
+  if (!isRecord(data)) {
     return { "openai-codex": false, xai: false };
   }
   const record = data;
@@ -498,9 +497,7 @@ async function loadCredentialProviders(agentDir) {
     const raw = await readFile(join(agentDir, "auth.json"), "utf8");
     return credentialProvidersFromAuthData(JSON.parse(raw));
   } catch (error) {
-    if (error instanceof Error && "code" in error && error.code === "ENOENT") {
-      return { "openai-codex": false, xai: false };
-    }
+    if (isEnoent(error)) return { "openai-codex": false, xai: false };
     throw error;
   }
 }

package/dist/public-cli/countersign-run.js

@@ -1,7 +1,5 @@
 import { readableGateItem } from "../readable-gate-item.js";
-function isRecord(value) {
-    return typeof value === "object" && value !== null && !Array.isArray(value);
-}
+import { isRecord } from "../unknown-value.js";
 function isEscalatePayload(payload) {
     if (!isRecord(payload))
         return false;

package/dist/public-cli/host-providers.js

@@ -10,6 +10,7 @@
  */
 import { readFileSync } from "node:fs";
 import { join } from "node:path";
+import { isEnoent, isRecord } from "../unknown-value.js";
 export function hostProvidersPath(home) {
     if (typeof home !== "string" || home.trim() === "") {
         throw new Error("home must be explicitly provided");
@@ -21,7 +22,7 @@ export function hermesProviderModelsCachePath(home) {
     return join(home, ".hermes", "provider_models_cache.json");
 }
 export function parseHostProvidersTable(value) {
-    if (value === null || typeof value !== "object" || Array.isArray(value)) {
+    if (!isRecord(value)) {
         throw new Error("host-providers.json must be an object");
     }
     const out = {};
@@ -29,9 +30,7 @@ export function parseHostProvidersTable(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 = {};
@@ -61,11 +60,8 @@ export function loadHostProvidersTable(home) {
         return parseHostProvidersTable(JSON.parse(raw));
     }
     catch (error) {
-        if (error instanceof Error &&
-            "code" in error &&
-            error.code === "ENOENT") {
+        if (isEnoent(error))
             return {};
-        }
         throw error;
     }
 }
@@ -86,20 +82,17 @@ export function listHermesProvidersForModel(home, seatModel) {
         text = readFileSync(path, "utf8");
     }
     catch (error) {
-        if (error instanceof Error &&
-            "code" in error &&
-            error.code === "ENOENT") {
+        if (isEnoent(error))
             return [];
-        }
         throw error;
     }
     const raw = 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 = [];
     for (const [provider, entry] of Object.entries(raw)) {
-        if (entry === null || typeof entry !== "object" || Array.isArray(entry)) {
+        if (!isRecord(entry)) {
             continue;
         }
         const models = entry.models;

package/dist/public-cli/instruction-seat-run.js

@@ -9,6 +9,8 @@ import { AUDITOR_DOSSIER_PROMPT } from "../compliance-transport.js";
 import { bookDirectOfficerRunPointer } from "../archivist-record-pointer.js";
 import { createPiDoctorAuditor } from "../doctor-auditor.js";
 import { officerConclusionReask, gateOfficerForSubject } from "../gatekeeper-role.js";
+import { receivedDiscriminator } from "../submission-errors.js";
+import { REVIEW_QUEUE_STATUSES } from "../review-submission.js";
 import { runJudgeGates } from "../judge-role.js";
 import { WORKER_DONE_STATUSES } from "../worker-submission-contracts.js";
 import { SECRETARIAT_GATE_OFFICER_ENTRY_TYPE } from "../secretariat-contracts.js";
@@ -34,9 +36,9 @@ import { warnMissingMethodSkills } from "./machine-method-skills.js";
 import { formatTerminalResult, isLawfulTypedTerminalOutcome, } from "./terminal.js";
 import { admittedSeatTurnDetails, projectPublicTurnAxes, projectRoleTurnRequest, requiredMethodSkills, } from "./turn-request.js";
 import { invokeCourtDiarist, latestPayloadEscalated, } from "./countersign-run.js";
+import { isRecord, errorText } from "../unknown-value.js";
 const AUDITED_ROLES = new Set(["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 = new Set([
     "planned", "completed", "refused", "partially_completed", "unfinished",
 ]);
@@ -45,8 +47,7 @@ const DOCTOR_ROUTING_STATUSES = new Set(["completed", "refused"]);
 const DOCTOR_STATUS_REASK = "status 不是 completed、refused 之一。请重新交卷，status 写明其一。";
 const POST_SUBMISSION_ROUTING = {
     judge: {
-        statuses: new Set(["converged", "continue", "escalate"]),
-        reask: "status 不是 converged、continue、escalate 三态之一。请重新交卷，status 写明其一。",
+        statuses: REVIEW_QUEUE_STATUSES,
     },
     coder: {
         statuses: WORKER_ROUTING_STATUSES,
@@ -74,12 +75,10 @@ function unreadablePostSubmissionStatus(admitted, terminal) {
     if (route === undefined)
         return undefined;
     const payload = terminal.roleOutcome.payloads?.at(-1);
-    const status = payload !== null && typeof payload === "object" && !Array.isArray(payload)
-        ? payload.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(admitted, terminal, env, io) {
     const message = unreadablePostSubmissionStatus(admitted, terminal);
@@ -484,7 +483,7 @@ export async function runPublicInstructionSeat(argv, env, io, role, parseArgv) {
             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)
@@ -715,7 +714,6 @@ export async function runPublicInstructionSeatResume(request, env, io) {
     }
     return resume();
 }
-const QUEUE_CONCLUSIONS = new Set(["converged", "continue", "escalate"]);
 const GATE_CHILD_ROLES = new Set(["notary", "auditor", "inspector", "countersign"]);
 /**
  * A gate child's conclusion outside the three states goes back to that child.
@@ -728,7 +726,7 @@ async function queueConclusionFromChild(child, env, io) {
     for (;;) {
         const terminal = await trySettlePublicSeat(current, env.principalAuthority, 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({
@@ -739,7 +737,7 @@ async function queueConclusionFromChild(child, env, io) {
             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))
@@ -785,14 +783,14 @@ async function auditSubmittedRole(turn, env, io, passedOfficer, passedReceipt, p
     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 : 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 }, { ...env, autoResumeLimit: 0 }, io);
+    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.
     // Doctor's declared domain is completed|refused — an open "escalate" must

package/dist/public-cli/invocation.js

@@ -27,6 +27,7 @@ import { NotarySourceRunError, resolveNotarySourceRunLocator, } from "../notary-
 import { appendEngineSessionMaterial, } from "../package-resources/engine-material.js";
 import { CliUsageError } from "./cli-errors.js";
 import { REJECTED_PUBLIC_SPELLINGS, createTypedOptionConsumer, evaluateAnalystModeOptionContract, optionsForOwner, resolveAnalystMode, } from "./option-definitions.js";
+import { errorText, isRecord } from "../unknown-value.js";
 /** Issue a principal from the single authoritative role-run placement. */
 export function issueAdmissionPlacement(authority, request) {
     const ledgerHome = resolveActivationLedgerHome(request.home);
@@ -880,7 +881,7 @@ export async function admitPublicRole(role, parsed, env, override) {
                     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 });
                 }
             }
@@ -895,7 +896,7 @@ export async function admitPublicRole(role, parsed, env, override) {
                     manifestCanonicalJson = manifest.canonicalJson;
                 }
                 catch (error) {
-                    throw new CliUsageError(error instanceof Error ? error.message : String(error), { cause: error });
+                    throw new CliUsageError(errorText(error), { cause: error });
                 }
             }
             const manifestDigest = manifest.digest;
@@ -948,7 +949,7 @@ export async function admitPublicRole(role, parsed, env, override) {
                     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) {
@@ -966,7 +967,7 @@ export async function admitPublicRole(role, parsed, env, override) {
                     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 });
                     }
                     frozenAttachments = await freezeAttachments(attachmentPaths, placed.attachmentsDirectory);
@@ -1322,7 +1323,7 @@ export async function materializeCountersignInvocation(admitted, options) {
 export async function loadAdmittedJudgeRequest(runDirectory) {
     try {
         const raw = JSON.parse(await readFile(join(runDirectory, "admitted-request.json"), "utf8"));
-        if (raw === null || typeof raw !== "object" || Array.isArray(raw))
+        if (!isRecord(raw))
             return undefined;
         const record = raw;
         if (!packagedPublicInstructionSubject(record.role))
@@ -1354,7 +1355,7 @@ function parsePositivePrOption(raw) {
         return parseCollectorPrNumber(raw);
     }
     catch (error) {
-        throw new CliUsageError(error instanceof Error ? error.message : String(error), { cause: error });
+        throw new CliUsageError(errorText(error), { cause: error });
     }
 }
 function parseRepoOption(raw) {
@@ -1396,7 +1397,7 @@ export function resolveGitHubRemoteRepository(projectRoot) {
         return parseCollectorRepository(ownerRepo);
     }
     catch (error) {
-        const detail = error instanceof Error ? error.message : String(error);
+        const detail = errorText(error);
         throw new CliUsageError(detail, { cause: error });
     }
 }
@@ -1456,7 +1457,7 @@ export async function resolveDoctorCaseRunsPath(options) {
         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 });
     }
     const normalized = real.split(sep).join("/");