@akagilnc/pi-workflow-roles 0.1.4802 → 0.1.4874

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

@@ -4,56 +4,46 @@
  */
 
 import {
-  COLLECTOR_ACCEPTED_TEXT,
   COLLECTOR_OUTPUT_TOOL,
   validateAcceptedCollectorReceipt,
   type CollectorReceipt,
 } from "./collector-output.ts";
 import {
-  JUDGE_ACCEPTED_TEXT,
   JUDGE_OUTPUT_TOOL_NAME,
   validateAcceptedJudgeDetails,
   type JudgeVerdict,
 } from "./judge-output.ts";
 import {
-  REVIEWER_ACCEPTED_TEXT,
   REVIEWER_OUTPUT_TOOL_NAME,
   validateReviewerIntent,
   type ReviewerIntent,
 } from "./reviewer-output.ts";
+import { PACKAGED_ROLE_REGISTRY } from "../packaged-role-registry.ts";
 import { CorrectableSubmissionError } from "../submission-correctable-error.ts";
-import { DOCTOR_ACCEPTED_TEXT, DOCTOR_OUTPUT_TOOL_NAME, validateDoctorSubmissionShape, validateRecordedDoctorOutput, type DoctorOutput, type DoctorSubmission } from "../doctor-contracts.ts";
-import { GATEKEEPER_ACCEPTED_TEXT, GATEKEEPER_OUTPUT_TOOL_NAME, validateRecordedGatekeeperOutput, type GatekeeperDirectOutput } from "./gatekeeper-output.ts";
-import { NAVIGATOR_ACCEPTED_TEXT, NAVIGATOR_OUTPUT_TOOL_NAME, validateRecordedNavigatorOutput, type NavigatorAdvice } from "./navigator-output.ts";
-import { AUDITOR_ACCEPTED_TEXT, AUDITOR_OUTPUT_TOOL_NAME, validateRecordedAuditorOutput, type AuditorOutput } from "./auditor-output.ts";
-import { MERGER_ACCEPTED_TEXT, MERGER_OUTPUT_TOOL_NAME, validateMergerOutput, type MergerOutput } from "../merger-contracts.ts";
-import { NOTARY_ACCEPTED_TEXT, NOTARY_OUTPUT_TOOL_NAME, validateRecordedNotaryOutput, type NotaryOutput } from "../notary-contracts.ts";
-import { COUNTERSIGN_ACCEPTED_TEXT, COUNTERSIGN_OUTPUT_TOOL_NAME, validateRecordedCountersignOutput, type CountersignVerdict } from "../countersign-contracts.ts";
-import { GLEANER_LEFT_ACCEPTED_TEXT, GLEANER_LEFT_OUTPUT_TOOL_NAME, validateRecordedGleanerLeftOutput, type GleanerLeftOutput } from "../gleaner-left-contracts.ts";
-import { INSPECTOR_ACCEPTED_TEXT, INSPECTOR_OUTPUT_TOOL_NAME, validateRecordedInspectorOutput, type InspectorOutput } from "../inspector-contracts.ts";
-import { DIARIST_ACCEPTED_TEXT, DIARIST_OUTPUT_TOOL_NAME, validateRecordedDiaristOutput, type DiaristOutput } from "../diarist-contracts.ts";
-import { SECRETARIAT_ACCEPTED_TEXT, SECRETARIAT_OUTPUT_TOOL_NAME, type SecretariatVerdict } from "../secretariat-contracts.ts";
+import { DOCTOR_OUTPUT_TOOL_NAME, validateDoctorSubmissionShape, validateRecordedDoctorOutput, type DoctorOutput, type DoctorSubmission } from "../doctor-contracts.ts";
+import { GATEKEEPER_OUTPUT_TOOL_NAME, validateRecordedGatekeeperOutput, type GatekeeperDirectOutput } from "./gatekeeper-output.ts";
+import { NAVIGATOR_OUTPUT_TOOL_NAME, validateRecordedNavigatorOutput, type NavigatorAdvice } from "./navigator-output.ts";
+import { AUDITOR_OUTPUT_TOOL_NAME, validateRecordedAuditorOutput, type AuditorOutput } from "./auditor-output.ts";
+import { MERGER_OUTPUT_TOOL_NAME, validateMergerOutput, type MergerOutput } from "../merger-contracts.ts";
+import { NOTARY_OUTPUT_TOOL_NAME, validateRecordedNotaryOutput, type NotaryOutput } from "../notary-contracts.ts";
+import { COUNTERSIGN_OUTPUT_TOOL_NAME, validateRecordedCountersignOutput, type CountersignVerdict } from "../countersign-contracts.ts";
+import { GLEANER_LEFT_OUTPUT_TOOL_NAME, validateRecordedGleanerLeftOutput, type GleanerLeftOutput } from "../gleaner-left-contracts.ts";
+import { INSPECTOR_OUTPUT_TOOL_NAME, validateRecordedInspectorOutput, type InspectorOutput } from "../inspector-contracts.ts";
+import { DIARIST_OUTPUT_TOOL_NAME, validateRecordedDiaristOutput, type DiaristOutput } from "../diarist-contracts.ts";
+import { SECRETARIAT_OUTPUT_TOOL_NAME, type SecretariatVerdict } from "../secretariat-contracts.ts";
 import {
-  CODER_ACCEPTED_TEXT,
   CODER_OUTPUT_TOOL_NAME,
-  FIXER_ACCEPTED_TEXT,
   FIXER_OUTPUT_TOOL_NAME,
   validateAcceptedWorkerDetails,
   type WorkerOutput,
 } from "./worker-output.ts";
 
 export {
-  CODER_ACCEPTED_TEXT,
   CODER_OUTPUT_TOOL_NAME,
-  COLLECTOR_ACCEPTED_TEXT,
   COLLECTOR_OUTPUT_TOOL,
-  FIXER_ACCEPTED_TEXT,
   FIXER_OUTPUT_TOOL_NAME,
-  JUDGE_ACCEPTED_TEXT,
   JUDGE_OUTPUT_TOOL_NAME,
-  REVIEWER_ACCEPTED_TEXT,
   REVIEWER_OUTPUT_TOOL_NAME,
-  MERGER_ACCEPTED_TEXT,
   MERGER_OUTPUT_TOOL_NAME,
   validateAcceptedCollectorReceipt,
   validateAcceptedJudgeDetails,
@@ -90,24 +80,7 @@ export type {
   SecretariatVerdict,
 };
 
-export const TERMINATING_TOOL_NAMES = [
-  CODER_OUTPUT_TOOL_NAME,
-  FIXER_OUTPUT_TOOL_NAME,
-  REVIEWER_OUTPUT_TOOL_NAME,
-  JUDGE_OUTPUT_TOOL_NAME,
-  COLLECTOR_OUTPUT_TOOL,
-  DOCTOR_OUTPUT_TOOL_NAME,
-  MERGER_OUTPUT_TOOL_NAME,
-  NOTARY_OUTPUT_TOOL_NAME,
-  COUNTERSIGN_OUTPUT_TOOL_NAME,
-  GLEANER_LEFT_OUTPUT_TOOL_NAME,
-  INSPECTOR_OUTPUT_TOOL_NAME,
-  GATEKEEPER_OUTPUT_TOOL_NAME,
-  NAVIGATOR_OUTPUT_TOOL_NAME,
-  AUDITOR_OUTPUT_TOOL_NAME,
-  DIARIST_OUTPUT_TOOL_NAME,
-  SECRETARIAT_OUTPUT_TOOL_NAME,
-] as const;
+export const TERMINATING_TOOL_NAMES = PACKAGED_ROLE_REGISTRY.map((entry) => entry.outputTool);
 
 export type TerminatingToolName = (typeof TERMINATING_TOOL_NAMES)[number];
 
@@ -134,43 +107,6 @@ export function isTerminatingToolName(
   return (TERMINATING_TOOL_NAMES as readonly string[]).includes(name);
 }
 
-export function acceptedTextFor(toolName: TerminatingToolName): string {
-  switch (toolName) {
-    case CODER_OUTPUT_TOOL_NAME:
-      return CODER_ACCEPTED_TEXT;
-    case FIXER_OUTPUT_TOOL_NAME:
-      return FIXER_ACCEPTED_TEXT;
-    case REVIEWER_OUTPUT_TOOL_NAME:
-      return REVIEWER_ACCEPTED_TEXT;
-    case JUDGE_OUTPUT_TOOL_NAME:
-      return JUDGE_ACCEPTED_TEXT;
-    case COLLECTOR_OUTPUT_TOOL:
-      return COLLECTOR_ACCEPTED_TEXT;
-    case DOCTOR_OUTPUT_TOOL_NAME:
-      return DOCTOR_ACCEPTED_TEXT;
-    case MERGER_OUTPUT_TOOL_NAME:
-      return MERGER_ACCEPTED_TEXT;
-    case NOTARY_OUTPUT_TOOL_NAME:
-      return NOTARY_ACCEPTED_TEXT;
-    case COUNTERSIGN_OUTPUT_TOOL_NAME:
-      return COUNTERSIGN_ACCEPTED_TEXT;
-    case GLEANER_LEFT_OUTPUT_TOOL_NAME:
-      return GLEANER_LEFT_ACCEPTED_TEXT;
-    case INSPECTOR_OUTPUT_TOOL_NAME:
-      return INSPECTOR_ACCEPTED_TEXT;
-    case GATEKEEPER_OUTPUT_TOOL_NAME:
-      return GATEKEEPER_ACCEPTED_TEXT;
-    case NAVIGATOR_OUTPUT_TOOL_NAME:
-      return NAVIGATOR_ACCEPTED_TEXT;
-    case AUDITOR_OUTPUT_TOOL_NAME:
-      return AUDITOR_ACCEPTED_TEXT;
-    case DIARIST_OUTPUT_TOOL_NAME:
-      return DIARIST_ACCEPTED_TEXT;
-    case SECRETARIAT_OUTPUT_TOOL_NAME:
-      return SECRETARIAT_ACCEPTED_TEXT;
-  }
-}
-
 export class AcceptedDetailsContractError extends CorrectableSubmissionError {
   readonly code = "accepted_details_contract" as const;
   constructor(message: string, options?: ErrorOptions) {
@@ -215,28 +151,20 @@ export type AcceptedFacts = {
 /** Read status/commit leaves the role wrote — never invent defaults (#836). */
 export function acceptedFacts(toolName: TerminatingToolName, details: AcceptedDetails): AcceptedFacts {
   const record = details as Record<string, unknown>;
-  switch (toolName) {
-    case JUDGE_OUTPUT_TOOL_NAME:
-      return typeof record.judgeStatus === "string" ? { status: record.judgeStatus } : {};
-    case COUNTERSIGN_OUTPUT_TOOL_NAME:
-      return typeof record.countersignStatus === "string" ? { status: record.countersignStatus } : {};
-    case SECRETARIAT_OUTPUT_TOOL_NAME:
-      return typeof record.secretariatStatus === "string" ? { status: record.secretariatStatus } : {};
-    case MERGER_OUTPUT_TOOL_NAME: {
-      const status = typeof record.status === "string" ? record.status : undefined;
-      return {
-        ...(status === undefined ? {} : { status }),
-        ...(status === "completed" && typeof record.mergeCommitId === "string"
-          ? { commit: record.mergeCommitId }
-          : {}),
-      };
-    }
-    case COLLECTOR_OUTPUT_TOOL:
-      // Collector has no status leaf — do not invent "collected".
-      return typeof record.status === "string" ? { status: record.status } : {};
-    default:
-      return typeof record.status === "string" ? { status: record.status } : {};
-  }
+  const entry = PACKAGED_ROLE_REGISTRY.find((candidate) => candidate.outputTool === toolName);
+  const statusKey = entry !== undefined && "receiptStatusKey" in entry ? entry.receiptStatusKey : "status";
+  const status = typeof record[statusKey] === "string" ? record[statusKey] : undefined;
+  const commitKey = entry !== undefined && "receiptCommitKey" in entry ? entry.receiptCommitKey : undefined;
+  const commitWhen = entry !== undefined && "receiptCommitWhen" in entry ? entry.receiptCommitWhen : undefined;
+  const commit = commitKey !== undefined
+    && status === commitWhen
+    && typeof record[commitKey] === "string"
+    ? record[commitKey]
+    : undefined;
+  return {
+    ...(status === undefined ? {} : { status }),
+    ...(commit === undefined ? {} : { commit }),
+  };
 }
 
 /** Deep structural equality for lifecycle agreement checks. */

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

@@ -1,7 +1,6 @@
 /** Package-owned independent Coder and Fixer output leaves. */
 
 export {
-  FIXER_ACCEPTED_TEXT,
   FIXER_OUTPUT_TOOL_NAME,
   fixerOutputSchema,
   validateFixerOutput,
@@ -17,7 +16,6 @@ export type { FixerInvocationInput, FixerPrerequisite } from "./fixer-packet.ts"
 import { validateFixerOutput, type FixerOutput } from "./fixer-output.ts";
 
 export const CODER_OUTPUT_TOOL_NAME = "ak_coder_output";
-export const CODER_ACCEPTED_TEXT = "将作监回执已接受";
 export type WorkerRoleLabel = "Coder" | "Fixer";
 export type CoderOutput =
   | { status: "planned"; report: string }