@akagilnc/pi-workflow-roles 0.1.4980 → 0.1.4992

package/dist/analyst-gate-cycles-read.js

@@ -81,7 +81,7 @@ function optionalDispatchReason(raw) {
     return raw;
 }
 function isGateTerminatingToolName(toolName) {
-    return DISPATCH_TOOLS.has(toolName) || OFFICER_TOOL_TO_FACE[toolName] !== undefined;
+    return toolName === "ak_submission_output" || DISPATCH_TOOLS.has(toolName) || OFFICER_TOOL_TO_FACE[toolName] !== undefined;
 }
 /**
  * toolCallIds whose paired toolResult is an accepted receipt (`isError === false`).
@@ -176,7 +176,7 @@ function extractAllAcceptedGateToolCalls(rows) {
     }
     return out;
 }
-function projectAcceptedGateCall(filePath, rows, call) {
+function projectAcceptedGateCall(filePath, rows, call, pointerOfficer) {
     // Continuous memory volumes carry many summons; span only this binding's interval.
     const span = requireAcceptedGateSpan(intervalRowsForGateCall(rows, call.rowIndex), filePath);
     const status = readAcceptedGateStatus(call.args);
@@ -208,7 +208,9 @@ function projectAcceptedGateCall(filePath, rows, call) {
             ...bindingFields,
         };
     }
-    const officer = OFFICER_TOOL_TO_FACE[call.toolName];
+    const officer = call.toolName === "ak_submission_output"
+        ? pointerOfficer
+        : OFFICER_TOOL_TO_FACE[call.toolName];
     if (officer === undefined) {
         // isGateTerminatingToolName already screened; keep loud if tables drift.
         throw new Error(`accepted gate receipt has unknown officer tool ${call.toolName} in ${filePath}`);
@@ -225,7 +227,7 @@ function projectAcceptedGateCall(filePath, rows, call) {
         ...bindingFields,
     };
 }
-async function classifyAuditorVolume(filePath) {
+async function classifyAuditorVolume(filePath, pointerOfficer) {
     // Canonical JSONL errors propagate — failure honesty (never wash to fewer rounds).
     const rows = await readLedgerSessionJsonl(filePath);
     const accepted = extractAllAcceptedGateToolCalls(rows);
@@ -234,7 +236,9 @@ async function classifyAuditorVolume(filePath) {
         return [];
     const volumes = [];
     for (const call of accepted) {
-        const projected = projectAcceptedGateCall(filePath, rows, call);
+        const projected = call.toolName === "ak_submission_output" && pointerOfficer === undefined
+            ? undefined // Shared tool alone carries no officer identity; a typed pointer does.
+            : projectAcceptedGateCall(filePath, rows, call, pointerOfficer);
         if (projected !== undefined)
             volumes.push(projected);
     }
@@ -330,7 +334,9 @@ async function resolveOfficerSessionFromPointerFile(pointerPath) {
     if (typeof sessionFile !== "string" || sessionFile.trim() === "") {
         throw new Error(`direct officer run pointer missing sessionFile in ${pointerPath}`);
     }
-    return sessionFile;
+    const officer = raw.officer === "inspector" || raw.officer === "notary"
+        ? raw.officer : undefined;
+    return { sessionFile, ...(officer === undefined ? {} : { officer }) };
 }
 /**
  * Read and pair gate-cycle rounds from one or more auditor-roles directories.
@@ -368,17 +374,16 @@ export async function readAnalystGateCyclesFromAuditorRoles(auditorRolesDirector
         for (const name of names) {
             const path = join(directory, name);
             const fromPointer = name.endsWith(".pointer.json");
-            const sessionPath = fromPointer
+            const pointer = fromPointer
                 ? await resolveOfficerSessionFromPointerFile(path)
-                : path;
-            if (sessionPath === undefined)
-                continue;
+                : undefined;
+            const sessionPath = pointer?.sessionFile ?? path;
             if (fromPointer) {
                 if (classifiedOfficerSessions.has(sessionPath))
                     continue;
                 classifiedOfficerSessions.add(sessionPath);
             }
-            const classified = await classifyAuditorVolume(sessionPath);
+            const classified = await classifyAuditorVolume(sessionPath, pointer?.officer);
             for (const volume of classified) {
                 if (options.parentSessionFile !== undefined &&
                     volume.parentSessionFile !== undefined &&

package/dist/analyst-metric-families/gate-cycles.js

@@ -31,9 +31,9 @@ function emptyAccum() {
 function absorbRound(accum, round) {
     accum.rounds += 1;
     accum.wallSum += round.officerWallMs;
-    if (round.status === "bounce")
+    if (round.status === "bounce" || round.status === "continue")
         accum.bounceCount += 1;
-    if (round.status === "pass")
+    if (round.status === "pass" || round.status === "converged")
         accum.passCount += 1;
 }
 function finishOfficer(officer, accum) {

package/dist/audit-escalation.js

@@ -89,8 +89,8 @@ export function isAuditEscalationResult(value) {
  */
 export async function disposeComplianceDecision(decision, handlers, deliveredOutput) {
     switch (decision.status) {
-        case "pass":
-            return await handlers.pass(decision.usage);
+        case "converged":
+            return await handlers.converged(decision.usage);
         case "no-receipt":
             // The parent candidate remains accepted, but its parallel audit leg is a
             // public typed fact and must not be collapsed into an ordinary pass.
@@ -104,8 +104,8 @@ export async function disposeComplianceDecision(decision, handlers, deliveredOut
                 throw new Error("Compliance received projection handler is unavailable");
             }
             return await handlers.received(decision, decision.usage === undefined ? {} : { usage: decision.usage });
-        case "bounce":
-            return await handlers.bounce(decision.violations);
+        case "continue":
+            return await handlers.continue(decision.violations);
         case "escalate":
             return await handlers.escalate(projectAuditEscalation(decision, deliveredOutput));
         case "transport_failure":

package/dist/compliance-transport.js

@@ -21,8 +21,8 @@ function tryReadComplianceCandidate(arguments_, usage) {
   }
   const args = arguments_;
   const status = args.status;
-  if (status === "pass") return { status, receipt: arguments_, ...usage === void 0 ? {} : { usage } };
-  if (status === "bounce") return { status, violations: readListField(args.violations), receipt: arguments_, ...usage === void 0 ? {} : { usage } };
+  if (status === "converged") return { status, receipt: arguments_, ...usage === void 0 ? {} : { usage } };
+  if (status === "continue") return { status, violations: readListField(args.violations), receipt: arguments_, ...usage === void 0 ? {} : { usage } };
   if (status === "escalate") {
     return {
       status,

package/dist/countersign-contracts.js

@@ -1,18 +1,19 @@
-const COUNTERSIGN_OUTPUT_TOOL_NAME = "ak_countersign_output";
+import { REVIEW_SUBMISSION_OUTPUT_TOOL_NAME } from "./review-submission.js";
+const COUNTERSIGN_OUTPUT_TOOL_NAME = REVIEW_SUBMISSION_OUTPUT_TOOL_NAME;
 function validateRecordedCountersignOutput(verdict) {
   if (verdict === null || typeof verdict !== "object" || Array.isArray(verdict)) {
     throw new Error("Countersign verdict has no execution discriminator");
   }
-  let countersignStatus;
+  let status;
   try {
-    countersignStatus = verdict.countersignStatus;
+    status = verdict.status;
   } catch {
     throw new Error("Countersign verdict has no execution discriminator");
   }
-  if (typeof countersignStatus !== "string") {
+  if (typeof status !== "string") {
     throw new Error("Countersign verdict has no execution discriminator");
   }
-  if (["converged", "continue", "escalate"].includes(countersignStatus)) {
+  if (["converged", "continue", "escalate"].includes(status)) {
     return verdict;
   }
   throw new Error("Countersign verdict has no execution discriminator");

package/dist/doctor-evidence.js

@@ -59,6 +59,8 @@ function deriveSession(content, id) {
     let accepted, observedCommit, turns = 0, calls = 0, tokens = 0;
     const statuses = [], commits = [];
     for (const row of rows) {
+        if (row.type === "custom" && row.customType === "ak-role-submission-closure" && record(row.data) && row.data.isError === false && typeof row.data.toolName === "string" && isTerminatingToolName(row.data.toolName))
+            accepted = row;
         const message = record(row.message) ? row.message : undefined;
         if (message?.role === "assistant") {
             for (const part of Array.isArray(message.content) ? message.content : [])
@@ -81,7 +83,6 @@ function deriveSession(content, id) {
                     continue;
                 throw error;
             }
-            accepted = row;
             const facts = acceptedFacts(message.toolName, details);
             if (facts.commit && facts.commit !== observedCommit) {
                 commits.push({ source: id, commit: facts.commit });

package/dist/gatekeeper-role.js

@@ -1,11 +1,12 @@
 import { auditorRunDirectory } from "./auditor-dossier-tool.js";
-import { packagedGateDecision, packagedGateStageLabel } from "./packaged-role-registry.js";
+import { packagedGateStageLabel } from "./packaged-role-registry.js";
 import { GatekeeperDecisionError } from "./submission-errors.js";
 import { INSPECTOR_OUTPUT_TOOL_NAME } from "./inspector-contracts.js";
+import { REVIEW_SUBMISSION_OUTPUT_TOOL_NAME } from "./review-submission.js";
 import { GATEKEEPER_OUTPUT_TOOL_NAME, gatekeeperDecisionSchema, gatekeeperOutputSchema, } from "./package-contracts/gatekeeper-output.js";
 import { runIdFromRunDirectory } from "./run-terminal-artifacts.js";
 export const INSPECTOR_OUTPUT_TOOL = INSPECTOR_OUTPUT_TOOL_NAME;
-export const NOTARY_OUTPUT_TOOL = "ak_notary_output";
+export const NOTARY_OUTPUT_TOOL = REVIEW_SUBMISSION_OUTPUT_TOOL_NAME;
 function gateSeatLabel(stage) {
     return packagedGateStageLabel(stage) ?? stage;
 }
@@ -67,17 +68,17 @@ function readRecord(value) {
 }
 /**
  * Read only the conclusion field for queueing (#753).
- * pass | bounce | escalate → queue signal + raw receipt.
+ * converged | continue | escalate → queue signal + raw receipt.
  * Anything else accepted → needs_reask (resume speaker), never unreadable/parent-stand.
  * `fallbackStatus` is the terminal outcome.status when the receipt body has no status key
  * (keeps missing-args sentinel intact as the receipt).
  */
-/** Map 给事中 countersignStatus onto the shared gate queue words (#969). */
-function gateStatusFromCountersign(status) {
+/** Map the shared review verdict onto the gate queue words (#1028). */
+function reviewQueueStatus(status) {
     if (status === "converged")
-        return "pass";
+        return "converged";
     if (status === "continue")
-        return "bounce";
+        return "continue";
     if (status === "escalate")
         return "escalate";
     return undefined;
@@ -85,31 +86,14 @@ function gateStatusFromCountersign(status) {
 function projectOfficerDecision(officer, decision, fallbackStatus) {
     const receipt = retainedReceipt(decision);
     const record = readRecord(decision);
-    // 给事中: only countersignStatus, strict three-word map; never generic status,
-    // never unmapped raw word, never fallbackStatus (#969 / ADR 0040/0055).
-    if (packagedGateDecision(officer) === "countersign-status") {
-        const countersignStatus = record !== undefined && typeof record.countersignStatus === "string"
-            ? record.countersignStatus
-            : undefined;
-        const status = typeof countersignStatus === "string"
-            ? gateStatusFromCountersign(countersignStatus)
-            : undefined;
-        if (status === "pass") {
-            return { status: "pass", officer, receipt };
-        }
-        if (status === "bounce" || status === "escalate") {
-            return { status, officer, receipt };
-        }
-        return { status: "needs_reask", officer, receipt };
-    }
-    // inspector / notary / auditor: shared gate words on generic status.
     const status = (record !== undefined && typeof record.status === "string" ? record.status : undefined)
         ?? fallbackStatus;
-    if (status === "pass") {
-        return { status: "pass", officer, receipt };
+    const queueStatus = typeof status === "string" ? reviewQueueStatus(status) : undefined;
+    if (queueStatus === "converged") {
+        return { status: "converged", officer, receipt };
     }
-    if (status === "bounce" || status === "escalate") {
-        return { status, officer, receipt };
+    if (queueStatus === "continue" || queueStatus === "escalate") {
+        return { status: queueStatus, officer, receipt };
     }
     return { status: "needs_reask", officer, receipt };
 }
@@ -161,8 +145,8 @@ export function officerRunIdFromSummoned(summoned) {
     return undefined;
 }
 function withOfficerRunId(result, summoned) {
-    if (result.status !== "pass"
-        && result.status !== "bounce"
+    if (result.status !== "converged"
+        && result.status !== "continue"
         && result.status !== "escalate"
         && result.status !== "needs_reask") {
         return result;
@@ -233,23 +217,18 @@ function projectOfficerTerminal(officer, summoned) {
     }, summoned);
 }
 /**
- * Plain-language re-ask when the officer conclusion is not pass|bounce|escalate.
+ * Plain-language re-ask when the officer conclusion is not converged|continue|escalate.
  * Not a packaged engine handbook line (#755 exception for 读不出三态).
- * inspector / notary / auditor keep shared gate words; 给事中 uses own three-state
- * field and words so reask does not invite non-contract values (#969 / ADR 0055).
+ * Every review officer uses the same `status` field and three queue words (#1028).
  */
-export const OFFICER_CONCLUSION_REASK = "上次交卷的结论不是 pass、bounce、escalate 三态之一。请重新输出，结论字段写明其一；打回或上呈的话就是给对方看的原文。";
-/** 给事中 re-ask: countersignStatus converged|continue|escalate only (#969). */
-export const COUNTERSIGN_CONCLUSION_REASK = "上次交卷的 countersignStatus 不是 converged、continue、escalate 三态之一。请重新输出，countersignStatus 写明其一；封驳或上呈的话就是给对方看的原文。";
+export const OFFICER_CONCLUSION_REASK = "上次交卷的 status 不是 converged、continue、escalate 三态之一。请重新输出，status 写明其一；continue 或 escalate 的话就是给对方看的原文。";
 /** Pick the re-ask line for the officer under review. */
 export function officerConclusionReask(officer) {
-    return packagedGateDecision(officer) === "countersign-status"
-        ? COUNTERSIGN_CONCLUSION_REASK
-        : OFFICER_CONCLUSION_REASK;
+    return OFFICER_CONCLUSION_REASK;
 }
 /**
  * Summon (via injected seam) + project. Default summonGateOfficer drive lives
- * on the shared envelope (`gatekeeper-pass-envelope.ts`, ADR 0018 / #675) —
+ * on the shared envelope (`submission-gate.ts`, ADR 0018 / #675) —
  * role module keeps projection only.
  */
 export async function projectGatekeeperRun(options) {