@akagilnc/pi-workflow-roles 0.1.3629 → 0.1.3631

package/dist/acp-host/production-host.js

@@ -3207,6 +3207,7 @@ function notaryDecisiveFacts(output) {
   if (status === "pass" || status === "bounce") {
     const findings = output.findings;
     facts.findingsCount = Array.isArray(findings) ? findings.length : 0;
+    facts.findings = Array.isArray(findings) ? findings : [];
   }
   if (status === "bounce") {
     facts.disposition = "rewrite";
@@ -10659,7 +10660,7 @@ function failureReason(error) {
 }
 function asStringArray3(value) {
   if (!Array.isArray(value)) return [];
-  return value.filter((item) => typeof item === "string");
+  return value.map((item) => typeof item === "string" ? item : JSON.stringify(item));
 }
 function isRecord14(value) {
   return typeof value === "object" && value !== null && !Array.isArray(value);
@@ -24667,13 +24668,11 @@ async function prepareAcpRoleEnvelope(options) {
                 });
                 reply(socket, rpc.id, {
                   content: projected.content,
-                  structuredContent: projected.details,
                   isError: true
                 });
               } catch {
                 reply(socket, rpc.id, {
                   content: declared.content,
-                  structuredContent: declared.details,
                   isError: true
                 });
               }
@@ -24686,7 +24685,7 @@ async function prepareAcpRoleEnvelope(options) {
                 details: result.details,
                 isError: false
               });
-              reply(socket, rpc.id, { content: projected.content, structuredContent: projected.details, ...projected.isError ? { isError: true } : {} });
+              reply(socket, rpc.id, { content: projected.content, ...projected.isError ? { isError: true } : {} });
             } catch (error) {
               let content;
               let details;
@@ -24710,7 +24709,7 @@ async function prepareAcpRoleEnvelope(options) {
                 details,
                 isError: true
               });
-              reply(socket, rpc.id, { content: projected.content, structuredContent: projected.details, ...projected.isError ? { isError: true } : {} });
+              reply(socket, rpc.id, { content: projected.content, ...projected.isError ? { isError: true } : {} });
             }
           } catch (error) {
             reply(socket, rpc.id, void 0, error);

package/dist/gatekeeper-role.js

@@ -51,7 +51,9 @@ function failureReason(error) {
 function asStringArray(value) {
     if (!Array.isArray(value))
         return [];
-    return value.filter((item) => typeof item === "string");
+    // Officer findings may be structured objects (category/law/evidence); the
+    // parent role must see them verbatim, not an empty list (#750 evidence).
+    return value.map((item) => (typeof item === "string" ? item : JSON.stringify(item)));
 }
 /** Serializable stand-in when the child tool call had no arguments object. */
 export const MISSING_ARGUMENTS_SUBMISSION = Object.freeze({ missing: "arguments" });

package/dist/notary-contracts.js

@@ -80,6 +80,7 @@ function notaryDecisiveFacts(output) {
   if (status === "pass" || status === "bounce") {
     const findings = output.findings;
     facts.findingsCount = Array.isArray(findings) ? findings.length : 0;
+    facts.findings = Array.isArray(findings) ? findings : [];
   }
   if (status === "bounce") {
     facts.disposition = "rewrite";

package/dist/public-cli/main.js

@@ -14902,6 +14902,7 @@ function notaryDecisiveFacts(output) {
   if (status === "pass" || status === "bounce") {
     const findings = output.findings;
     facts.findingsCount = Array.isArray(findings) ? findings.length : 0;
+    facts.findings = Array.isArray(findings) ? findings : [];
   }
   if (status === "bounce") {
     facts.disposition = "rewrite";

package/package.json

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

package/src/acp-host/role-envelope.ts

@@ -474,14 +474,12 @@ export async function prepareAcpRoleEnvelope(options: {
                 });
                 reply(socket, rpc.id, {
                   content: projected.content,
-                  structuredContent: projected.details,
                   isError: true,
                 });
               } catch {
                 // Slot already filled; durable projection may have partially failed.
                 reply(socket, rpc.id, {
                   content: declared.content,
-                  structuredContent: declared.details,
                   isError: true,
                 });
               }
@@ -497,7 +495,7 @@ export async function prepareAcpRoleEnvelope(options: {
               // Candidate only: do not emit turn_end here. Seal waits for the typed ACP
               // round boundary (closeRound after session/prompt), so delayed siblings stay
               // in the same round instead of becoming silent post-seal anomalies.
-              reply(socket, rpc.id, { content: projected.content, structuredContent: projected.details, ...(projected.isError ? { isError: true } : {}) });
+              reply(socket, rpc.id, { content: projected.content, ...(projected.isError ? { isError: true } : {}) });
             } catch (error) {
               let content: ContentPart[];
               let details: Record<string, unknown>;
@@ -529,7 +527,7 @@ export async function prepareAcpRoleEnvelope(options: {
                 details,
                 isError: true,
               });
-              reply(socket, rpc.id, { content: projected.content, structuredContent: projected.details, ...(projected.isError ? { isError: true } : {}) });
+              reply(socket, rpc.id, { content: projected.content, ...(projected.isError ? { isError: true } : {}) });
             }
           } catch (error) { reply(socket, rpc.id, undefined, error); }
         })();

package/src/gatekeeper-role.ts

@@ -146,7 +146,9 @@ function failureReason(error: unknown): string {
 
 function asStringArray(value: unknown): readonly string[] {
   if (!Array.isArray(value)) return [];
-  return value.filter((item): item is string => typeof item === "string");
+  // Officer findings may be structured objects (category/law/evidence); the
+  // parent role must see them verbatim, not an empty list (#750 evidence).
+  return value.map((item) => (typeof item === "string" ? item : JSON.stringify(item)));
 }
 
 /** Serializable stand-in when the child tool call had no arguments object. */

package/src/notary-contracts.ts

@@ -128,6 +128,8 @@ export function notaryDecisiveFacts(output: NotaryOutput): Record<string, unknow
   if (status === "pass" || status === "bounce") {
     const findings = (output as { findings?: unknown }).findings;
     facts.findingsCount = Array.isArray(findings) ? findings.length : 0;
+    // The parent role reads the findings themselves; the count alone dropped them (#750).
+    facts.findings = Array.isArray(findings) ? findings : [];
   }
   if (status === "bounce") {
     facts.disposition = "rewrite";