@akagilnc/pi-workflow-roles 0.1.2331 → 0.1.2336

package/dist/package-contracts/reviewer-output.js

@@ -15,12 +15,32 @@ function read(value, key) {
         return undefined;
     }
 }
+/** Canonical per-axis string slots only — no shape reject, no prose parse (ADR 0057). */
+function readAmendments(output) {
+    const amendments = read(output, "amendments");
+    if (!isRecord(amendments))
+        return undefined;
+    const slots = {};
+    for (const axis of ["standards", "spec"]) {
+        const value = read(amendments, axis);
+        if (typeof value === "string")
+            slots[axis] = value;
+    }
+    if (slots.standards === undefined && slots.spec === undefined)
+        return undefined;
+    return Object.freeze(slots);
+}
 export function validateReviewerIntent(output) {
     const status = read(output, "status");
-    if (status === "completed")
-        return { status };
-    if (status === "refused")
-        return { status, diagnostic: read(output, "diagnostic") };
+    const amendments = readAmendments(output);
+    if (status === "completed") {
+        return amendments === undefined ? { status } : { status, amendments };
+    }
+    if (status === "refused") {
+        return amendments === undefined
+            ? { status, diagnostic: read(output, "diagnostic") }
+            : { status, diagnostic: read(output, "diagnostic"), amendments };
+    }
     throw new Error("Reviewer output has no recognized execution intent");
 }
 /** Validate runtime-owned facts at their real identity seams (target pins + plain text). */

package/dist/public-cli/main.js

@@ -21190,13 +21190,16 @@ function reviewerDecisiveFacts(output) {
   const status = safelyRead(candidate, "status");
   const outcomes = safelyRead(candidate, "outcomes");
   const reports = safelyRead(candidate, "reports");
+  const amendments = safelyRead(candidate, "amendments");
   const axes = reviewerAxes(outcomes.readable ? outcomes.value : void 0);
   const reportAxes = reviewerAxes(reports.readable ? reports.value : void 0);
+  const amendmentAxes = reviewerAxes(amendments.readable ? amendments.value : void 0);
   const acceptedBatch = safelyRead(candidate, "acceptedBatch");
   const specDisposition = safelyRead(candidate, "specDisposition");
   const facts = {
     axes,
     reportAxes,
+    amendmentAxes,
     acceptedBatchPresent: acceptedBatch.readable && acceptedBatch.value !== void 0,
     ...auditNoReceiptDecisiveFact(candidate),
     // #380: spread sole-built field; do not re-key here (S1).

package/package.json

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

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

@@ -11,9 +11,11 @@ import type { ReviewerAcceptedEvidence, ReviewerFailureClassification, ReviewerW
 export const REVIEWER_OUTPUT_TOOL_NAME = "ak_reviewer_output";
 export const REVIEWER_ACCEPTED_TEXT = "Reviewer report accepted";
 
+/** Seat-owned per-axis delta relative to child reports — not a replacement report. */
+export type ReviewerAmendments = Readonly<Partial<Record<"standards" | "spec", string>>>;
 export type ReviewerIntent =
-  | Readonly<{ status: "completed" }>
-  | Readonly<{ status: "refused"; diagnostic: string }>;
+  | Readonly<{ status: "completed"; amendments?: ReviewerAmendments }>
+  | Readonly<{ status: "refused"; diagnostic: string; amendments?: ReviewerAmendments }>;
 /** Child report is plain text — no length/digest identity shell (ADR 0031). */
 export type VerbatimChildReport = Readonly<{ text: string }>;
 /** Prompt projection on the receipt face is plain text (ADR 0031). */
@@ -44,6 +46,8 @@ type RuntimeReviewerReceiptV2Clean = Readonly<{
   /** Self-fetch bytes + source annotation when Spec primary path produced material (#343). */
   specFetchedMaterial?: ReviewerAcceptedEvidence["specFetchedMaterial"];
   reports: Readonly<Partial<Record<"standards" | "spec", VerbatimChildReport>>>;
+  /** Seat-owned per-axis deltas; separate from runtime-owned child reports. */
+  amendments?: ReviewerAmendments;
   outcomes: Readonly<Partial<Record<"standards" | "spec", RuntimeReviewerOutcome>>>;
   identities: Readonly<{
     canonicalSkill: ReviewerReceiptSkillContent;
@@ -68,10 +72,30 @@ function read(value: unknown, key: string): unknown {
   try { return value[key]; } catch { return undefined; }
 }
 
+/** Canonical per-axis string slots only — no shape reject, no prose parse (ADR 0057). */
+function readAmendments(output: unknown): ReviewerAmendments | undefined {
+  const amendments = read(output, "amendments");
+  if (!isRecord(amendments)) return undefined;
+  const slots: Partial<Record<"standards" | "spec", string>> = {};
+  for (const axis of ["standards", "spec"] as const) {
+    const value = read(amendments, axis);
+    if (typeof value === "string") slots[axis] = value;
+  }
+  if (slots.standards === undefined && slots.spec === undefined) return undefined;
+  return Object.freeze(slots);
+}
+
 export function validateReviewerIntent(output: unknown): ReviewerIntent {
   const status = read(output, "status");
-  if (status === "completed") return { status };
-  if (status === "refused") return { status, diagnostic: read(output, "diagnostic") as string };
+  const amendments = readAmendments(output);
+  if (status === "completed") {
+    return amendments === undefined ? { status } : { status, amendments };
+  }
+  if (status === "refused") {
+    return amendments === undefined
+      ? { status, diagnostic: read(output, "diagnostic") as string }
+      : { status, diagnostic: read(output, "diagnostic") as string, amendments };
+  }
   throw new Error("Reviewer output has no recognized execution intent");
 }
 

package/src/public-cli/settlement.ts

@@ -1309,13 +1309,17 @@ function reviewerDecisiveFacts(
   const status = safelyRead(candidate, "status");
   const outcomes = safelyRead(candidate, "outcomes");
   const reports = safelyRead(candidate, "reports");
+  const amendments = safelyRead(candidate, "amendments");
   const axes = reviewerAxes(outcomes.readable ? outcomes.value : undefined);
   const reportAxes = reviewerAxes(reports.readable ? reports.value : undefined);
+  // Typed presence only — never copy amendment prose into public facts.
+  const amendmentAxes = reviewerAxes(amendments.readable ? amendments.value : undefined);
   const acceptedBatch = safelyRead(candidate, "acceptedBatch");
   const specDisposition = safelyRead(candidate, "specDisposition");
   const facts: Record<string, unknown> = {
     axes,
     reportAxes,
+    amendmentAxes,
     acceptedBatchPresent: acceptedBatch.readable && acceptedBatch.value !== undefined,
     ...auditNoReceiptDecisiveFact(candidate),
     // #380: spread sole-built field; do not re-key here (S1).

package/src/reviewer-role.ts

@@ -32,9 +32,20 @@ export type ReviewerAdmittedInputs = Readonly<{
   ticketNumber?: number;
 }>;
 
+const reviewerAmendmentsSchema = Type.Object({
+  standards: Type.Optional(Type.String({ description: "Delta relative to the Standards child report: added finding, withdrawal, or factual correction." })),
+  spec: Type.Optional(Type.String({ description: "Delta relative to the Spec child report: added finding, withdrawal, or factual correction." })),
+}, { additionalProperties: true, description: "Optional per-axis deltas relative to child reports; not a replacement report. Omit axes with no delta." });
 const reviewerOutputVariants = Type.Union([
-  Type.Object({ status: Type.Literal("completed", { description: "Reviewer dispatch completed." }) }, { additionalProperties: false }),
-  Type.Object({ status: Type.Literal("refused", { description: "Reviewer dispatch was lawfully refused." }), diagnostic: Type.String({ minLength: 1, description: "Diagnostic explaining the refusal." }) }, { additionalProperties: false }),
+  Type.Object({
+    status: Type.Literal("completed", { description: "Reviewer dispatch completed." }),
+    amendments: Type.Optional(reviewerAmendmentsSchema),
+  }, { additionalProperties: false }),
+  Type.Object({
+    status: Type.Literal("refused", { description: "Reviewer dispatch was lawfully refused." }),
+    diagnostic: Type.String({ minLength: 1, description: "Diagnostic explaining the refusal." }),
+    amendments: Type.Optional(reviewerAmendmentsSchema),
+  }, { additionalProperties: false }),
 ]);
 const reviewerOutputSchema = openToolObjectFromUnion(reviewerOutputVariants);
 export type ReviewerRoleDependencies = {

package/src/reviewer-settlement.ts

@@ -43,6 +43,8 @@ export function assembleRuntimeReviewerReceipt(input: {
   }
   const accepted = input.record.accepted;
   const skillText = accepted?.input.canonicalSkill ?? input.canonicalSkillText;
+  // Seat-owned amendments sit beside reports; never rewrite runtime-owned child bytes.
+  const amendments = input.intent.amendments;
   return freeze({
     version: 2,
     status: input.intent.status,
@@ -62,6 +64,7 @@ export function assembleRuntimeReviewerReceipt(input: {
         : { specFetchedMaterial: accepted.specFetchedMaterial }),
     }),
     reports,
+    ...(amendments === undefined ? {} : { amendments }),
     outcomes,
     identities: {
       canonicalSkill: { text: skillText },