@akagilnc/pi-workflow-roles 0.1.5583 → 0.1.5593

package/dist/public-cli/option-definitions.js

@@ -261,7 +261,7 @@ const SHARED_PROJECT_SEMANTICS = {
     },
 };
 /**
- * Immutable shared semantics for the common frozen-file `--attach` face.
+ * Immutable shared semantics for the common `--attach` face (#1165).
  * Role rows bind owner only. Reviewer has no attach face; analyst sweep attach
  * keeps its own modes/selectsMode/description and must not use this binding.
  */
@@ -274,8 +274,8 @@ const SHARED_ATTACH_SEMANTICS = {
     repeatable: true,
     form: "option",
     description: {
-        en: "Attach a regular file; frozen at admission (repeatable).",
-        zh: "附加普通文件；受理即冻结（可重复）。",
+        en: "Caller file path passed as-is to the role (repeatable). Package does not read, copy, or validate; use an absolute path when cwd differs from --project.",
+        zh: "调用方文件路径原样递给衙门（可重复）。包不读、不复制、不校验；调用方当前目录与 --project 不同时请给绝对路径。",
     },
 };
 /** Minimal owner-binding: one immutable semantic row → one role table entry. */
@@ -461,8 +461,8 @@ const COLLECTOR_OPTIONS = [
         repeatable: false,
         form: "option",
         description: {
-            en: "Optional request manifest JSON path ({requests:[{id,body}]}). Parsed for semantics and passed to the Collector as materials — not a machine collection directive.",
-            zh: "可选 request manifest JSON 路径（{requests:[{id,body}]}）。按语义解析后作为材料递给通进司，不是机器代收 finding 的指令。",
+            en: "Caller path to a request manifest JSON file, passed as-is. Package does not read, copy, or validate; use an absolute path when cwd differs from --project.",
+            zh: "调用方 request manifest JSON 路径，原样传递。包不读、不复制、不校验；调用方当前目录与 --project 不同时请给绝对路径。",
         },
     },
 ];

package/dist/public-cli/post-admission.js

@@ -7,14 +7,12 @@
  */
 import { randomUUID } from "node:crypto";
 import { writeFile } from "node:fs/promises";
-import { isAbsolute, join, resolve } from "node:path";
 import { buildAutoResumeContinuationPrompt, findRunDirectoryById, readRoleRunState, RESUME_TRANSPORT_ENVELOPE, } from "./run-lifecycle.js";
 import { CliUsageError } from "./cli-errors.js";
 import { projectPublicTurnAxes } from "./turn-request.js";
-import { bindAdmittedTicketNumber, buildInstructionTransportPrompt, freezeAttachmentsIntoRun, relocateAdmittedRunToTicket, } from "./invocation.js";
+import { bindAdmittedTicketNumber, buildInstructionTransportPrompt, relocateAdmittedRunToTicket, } from "./invocation.js";
 import { readRecordedSubmissionRows } from "../submission-ledger.js";
 import { parseTicketNumber, readDeclaredTicketNumber } from "../run-ticket-number.js";
-import { pathContainedIn } from "../activation-ledger-topology.js";
 import { readStoredHostSessionId } from "../session-identity.js";
 import { reportRunRecord } from "../sitian-facade.js";
 import { createReceiptDeliveryPolicy, deliveryLimitFromConfig, } from "../receipt-delivery-policy.js";
@@ -51,11 +49,12 @@ function projectRelocatedTurnIdentity(request, result, admitted, relocation) {
     mutableRequest.principal = admitted.principal;
     const activation = mutableRequest.activation;
     if (isRecord(activation)) {
-        for (const field of ["taskPath", "packetPath", "prerequisitesPath", "inputPath", "requestManifestPath"]) {
+        for (const field of ["taskPath", "packetPath", "prerequisitesPath", "inputPath"]) {
             const record = activation;
             if (field in record)
                 record[field] = rewrite(record[field]);
         }
+        // #1165: activation.requestManifestPath is a caller path — do not rewrite.
     }
     if (result.terminal !== undefined) {
         for (const artifact of result.terminal.artifacts) {
@@ -925,7 +924,7 @@ export function resumeTurnRequestProjectionOptions(admitted, request, env, summo
     if (request.message !== undefined) {
         if (summonsPrepared !== undefined) {
             // #755: same-ticket review / open-court — caller words.
-            // #879 station-child officer: words only (attachments are independent freeze).
+            // #879 station-child officer: words only (paths remain on summons materials).
             prompt = officerDialogue
                 ? request.message
                 : buildInstructionTransportPrompt({
@@ -945,8 +944,10 @@ export function resumeTurnRequestProjectionOptions(admitted, request, env, summo
     else if (summonsPrepared !== undefined) {
         // #879 station-child officer: instruction bytes === peer body/reask (no wrap).
         // Other seats keep #755 instruction + optional attachment path listing.
+        // #1165 J3: officer dialogue is raw instruction bytes; do not gate on
+        // instructionEmpty metadata (resume of older admitted pages).
         prompt = officerDialogue
-            ? (summonsPrepared.instructionEmpty ? "" : summonsPrepared.instruction)
+            ? summonsPrepared.instruction
             : buildInstructionTransportPrompt(summonsPrepared);
     }
     else if (request.summons !== undefined) {
@@ -1046,34 +1047,18 @@ async function runSettledAutoResumeLoop(input) {
         }),
     });
 }
-function isAlreadyFrozenSummonsAttachment(runDirectory, attachmentPath) {
-    const absolute = isAbsolute(attachmentPath)
-        ? attachmentPath
-        : resolve(attachmentPath);
-    return pathContainedIn(join(runDirectory, "attachments"), absolute);
-}
 /**
- * Freeze same-ticket summons attachments into the retained run directory (#637).
- * No-op materials (no paths / instruction-only) skip the freeze.
- * Paths already under this run's attachments/ are the accepted freeze identity —
- * reuse them for the same internal re-summons flow.
- * Manual resume never calls this — old attachment semantics stay intact.
+ * Same-ticket summons materials: caller paths as-is (#1165). No copy or freeze.
  */
-export async function prepareSummonsResumeMaterials(runDirectory, summons) {
+export async function prepareSummonsResumeMaterials(_runDirectory, summons) {
     if (summons === undefined)
         return undefined;
     if (summons.instruction === undefined && (summons.attachmentPaths?.length ?? 0) === 0) {
         return undefined;
     }
     const instruction = summons.instruction ?? "";
-    const instructionEmpty = summons.instructionEmpty ?? instruction.trim() === "";
-    let attachments = [];
-    if (summons.attachmentPaths !== undefined && summons.attachmentPaths.length > 0) {
-        const alreadyFrozen = summons.attachmentPaths.every((path) => isAlreadyFrozenSummonsAttachment(runDirectory, path));
-        attachments = alreadyFrozen
-            ? summons.attachmentPaths.map((frozenPath) => ({ frozenPath }))
-            : await freezeAttachmentsIntoRun(summons.attachmentPaths, runDirectory);
-    }
+    const instructionEmpty = summons.instructionEmpty ?? instruction.length === 0;
+    const attachments = (summons.attachmentPaths ?? []).map((path) => ({ path }));
     return { instruction, instructionEmpty, attachments };
 }
 /**
@@ -1137,20 +1122,6 @@ export async function runPostAdmissionSeatResume(input) {
                 openCourtAttemptId = openCourt.courtAttemptId;
             }
         }
-        // Internal re-summons freezes external paths once and records that identity.
-        if (request.summons !== undefined) {
-            const prepared = await prepareSummonsResumeMaterials(admittedForBuild.runDirectory, request.summons);
-            if (prepared !== undefined &&
-                (request.summons.attachmentPaths?.length ?? 0) > 0) {
-                request = {
-                    ...request,
-                    summons: {
-                        ...request.summons,
-                        attachmentPaths: prepared.attachments.map((attachment) => attachment.frozenPath),
-                    },
-                };
-            }
-        }
         // Public explicit resume: attach the stored native id when the caller
         // left it unset (399c3c8c). In-call auto-resume / 催交 / gate retry omit
         // the field; those load inside the host adapter (host-contracts).

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

@@ -14,7 +14,7 @@ import { CliUsageError } from "./cli-errors.js";
 import { parseCollectorRepository } from "../collector-config.js";
 import { retainedRunPathsMatch, rewriteAdmittedRoleRunPage, rewriteRunDirectoryPathValue, } from "../role-run-relocation.js";
 import { packagedResumeSourcePath, packagedRoleMetadata, } from "../packaged-role-registry.js";
-import { admittedReviewerBaseFault, recordEffectiveInvocationModel, requireAuthorityRef, isReviewerLens, } from "./invocation.js";
+import { admittedReviewerBaseFault, recordEffectiveInvocationModel, requireAuthorityRef, isReviewerLens, normalizeAdmittedAttachment, } from "./invocation.js";
 import { readPageSync, writeSectionSync } from "../run-dossier.js";
 import { isRecord, errorText, isEnoent } from "../unknown-value.js";
 /** Default for the #422 configurable single-call auto-resume ceiling
@@ -51,7 +51,7 @@ function parseSameTicketSummonsMaterials(raw) {
     const instruction = typeof record.instruction === "string" ? record.instruction : undefined;
     const instructionEmpty = typeof record.instructionEmpty === "boolean" ? record.instructionEmpty : undefined;
     const attachmentPaths = Array.isArray(record.attachmentPaths)
-        ? record.attachmentPaths.filter((p) => typeof p === "string" && p.length > 0)
+        ? record.attachmentPaths.filter((p) => typeof p === "string")
         : undefined;
     const sourceRunPath = typeof record.sourceRunPath === "string" && record.sourceRunPath.trim() !== ""
         ? record.sourceRunPath
@@ -781,7 +781,6 @@ async function loadResumableRunRecord(home, runId, authority) {
     let repository;
     let repositoryDisplay;
     let requestManifestPath;
-    let manifestDigest;
     let waitWindowMs;
     let issueNumber;
     let caseRunsPath;
@@ -807,7 +806,9 @@ async function loadResumableRunRecord(home, runId, authority) {
                 instructionEmpty = record.instructionEmpty;
             }
             if (Array.isArray(record.attachments)) {
-                attachments = record.attachments;
+                attachments = record.attachments
+                    .map((item) => normalizeAdmittedAttachment(item))
+                    .filter((item) => item !== undefined);
             }
             if (record.phase === "plan" || record.phase === "apply") {
                 phase = record.phase;
@@ -842,12 +843,10 @@ async function loadResumableRunRecord(home, runId, authority) {
             if (typeof record.repositoryDisplay === "string" && record.repositoryDisplay.trim() !== "") {
                 repositoryDisplay = record.repositoryDisplay;
             }
-            if (typeof record.requestManifestPath === "string" && record.requestManifestPath.trim() !== "") {
+            // #1165: opaque caller path on resume — do not trim-filter provided values.
+            if (typeof record.requestManifestPath === "string") {
                 requestManifestPath = record.requestManifestPath;
             }
-            if (typeof record.manifestDigest === "string" && record.manifestDigest.trim() !== "") {
-                manifestDigest = record.manifestDigest;
-            }
             if (typeof record.waitWindowMs === "number" && Number.isSafeInteger(record.waitWindowMs) && record.waitWindowMs >= 1) {
                 waitWindowMs = record.waitWindowMs;
             }
@@ -990,7 +989,6 @@ async function loadResumableRunRecord(home, runId, authority) {
             ...(repository === undefined ? {} : { repository }),
             ...(repositoryDisplay === undefined ? {} : { repositoryDisplay }),
             ...(requestManifestPath === undefined ? {} : { requestManifestPath }),
-            ...(manifestDigest === undefined ? {} : { manifestDigest }),
             ...(waitWindowMs === undefined ? {} : { waitWindowMs }),
             ...(issueNumber === undefined ? {} : { issueNumber }),
             ...(caseRunsPath === undefined ? {} : { caseRunsPath }),
@@ -1179,10 +1177,8 @@ function admitResumedRole(loaded) {
             return admitted;
         }
         case "collect-target": {
-            const { prNumber, repository, repositoryDisplay, manifestDigest } = fields;
-            if (repository === undefined ||
-                repositoryDisplay === undefined ||
-                manifestDigest === undefined) {
+            const { prNumber, repository, repositoryDisplay } = fields;
+            if (repository === undefined || repositoryDisplay === undefined) {
                 throw new CliUsageError(`role run admitted collector repository identity is missing: ${runId}`);
             }
             let parsedRepository;
@@ -1205,7 +1201,6 @@ function admitResumedRole(loaded) {
                     ? {}
                     : { requestManifestPath: fields.requestManifestPath }),
                 ...(fields.waitWindowMs === undefined ? {} : { waitWindowMs: fields.waitWindowMs }),
-                manifestDigest,
             };
             return admitted;
         }

package/dist/public-role-summons.js

@@ -249,7 +249,7 @@ async function summonPublicRole(options) {
         runId: resumeRunId,
         summons: {
           instruction,
-          instructionEmpty: instruction.trim() === ""
+          instructionEmpty: instruction.length === 0
         }
       },
       env,

package/dist/role-run-relocation.js

@@ -22,7 +22,7 @@ const ADMITTED_PAGE_FIELDS = [
     "taskPath",
     "packetPath",
     "prerequisitesPath",
-    "requestManifestPath",
+    // #1165: requestManifestPath is a caller path — never rewrite.
     "mergerInputPath",
     "sourceRunPath",
 ];
@@ -150,28 +150,19 @@ function rewriteSourceRunLocator(value, rewrites) {
         return;
     rewriteRunDirectoryPathFieldsAgainstRewrites(value, SOURCE_RUN_LOCATOR_FIELDS, rewrites);
 }
-/** Same-ticket summons materials: typed source-run + frozen attachment pointers. */
+/** Same-ticket summons materials: typed source-run; caller attach paths stay as-is (#1165). */
 function rewriteSummonsMaterials(value, rewrites) {
     if (!isRecord(value))
         return;
     rewriteRunDirectoryPathFieldsAgainstRewrites(value, SUMMONS_PATH_FIELDS, rewrites);
     rewriteSourceRunLocator(value.sourceRun, rewrites);
-    // Bare-resume frozen attachment pointers (string[]), not free text / bytes.
-    if (Array.isArray(value.attachmentPaths)) {
-        value.attachmentPaths = value.attachmentPaths.map((path) => rewriteRunDirectoryPathValueAgainstRewrites(path, rewrites));
-    }
+    // #1165: summons.attachmentPaths are caller paths — do not rewrite.
 }
 /** Project one admitted page in memory after its containing run moved. */
 export function rewriteAdmittedRoleRunPage(page, rewrites) {
     rewriteRunDirectoryPathFieldsAgainstRewrites(page, ADMITTED_PAGE_FIELDS, rewrites);
     rewriteSourceRunLocator(page.sourceRun, rewrites);
-    if (Array.isArray(page.attachments)) {
-        for (const attachment of page.attachments) {
-            if (isRecord(attachment)) {
-                rewriteRunDirectoryPathFieldsAgainstRewrites(attachment, ["frozenPath"], rewrites);
-            }
-        }
-    }
+    // #1165: admitted attachments are caller paths outside the run directory — do not rewrite.
     if (isRecord(page.principal)) {
         rewriteRunDirectoryPathFieldsAgainstRewrites(page.principal, ["sessionDirectory", "sessionFile"], rewrites);
     }
@@ -347,12 +338,13 @@ async function rewriteNestedMachinePathPages(pagesDirectory, rewrites) {
     await walk(sessionRoot);
 }
 /**
- * Rewrite the admitted / invocation / runState fact rows (path fields: attachment
- * frozenPath / summons.attachmentPaths pointers only), then nested package-owned
- * session seams. Callers may rewrite before or after the filesystem move/copy:
- * `pagesDirectory` is where the pages currently live on disk; path strings that
- * still name `oldRunDirectory` become `newRunDirectory`. `crossRunRewrites`
- * covers officer/parent pointers that landed under a different final placement.
+ * Rewrite the admitted / invocation / runState fact rows (typed run placement
+ * paths only; caller --attach / --request-manifest paths stay as-is, #1165),
+ * then nested package-owned session seams. Callers may rewrite before or after
+ * the filesystem move/copy: `pagesDirectory` is where the pages currently live
+ * on disk; path strings that still name `oldRunDirectory` become
+ * `newRunDirectory`. `crossRunRewrites` covers officer/parent pointers that
+ * landed under a different final placement.
  */
 export async function rewriteRoleRunDurablePages(input) {
     const { pagesDirectory } = input;
@@ -366,7 +358,7 @@ export async function rewriteRoleRunDurablePages(input) {
             if (isRecord(page.principal)) {
                 rewriteRunDirectoryPathFieldsAgainstRewrites(page.principal, ["sessionDirectory", "sessionFile"], rewrites);
             }
-            // Open court summons: source-run locators + frozen attachment path pointers.
+            // Open court summons: source-run locators only; caller attach paths stay (#1165).
             if (isRecord(page.currentCourt)) {
                 rewriteSummonsMaterials(page.currentCourt.summons, rewrites);
             }

package/package.json

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

package/src/collector-config.ts

@@ -1,7 +1,3 @@
-import { readFile } from "node:fs/promises";
-
-import { sha256Hex } from "./sha256.ts";
-import { isRecord } from "./unknown-value.ts";
 import { isTicketNumberString, parseTicketNumber } from "./run-ticket-number.ts";
 
 export const COLLECTOR_OWNER_PATTERN = /^[A-Za-z0-9](?:[A-Za-z0-9-]{0,37}[A-Za-z0-9])?$/;
@@ -14,14 +10,6 @@ export type CollectorRepository = {
   repo: string;
 };
 
-export type CollectorRequestConfig = { id: string; requestBody: string };
-export type CollectorManifest = {
-  requests: readonly CollectorRequestConfig[];
-  canonicalJson: string;
-  digest: string;
-  sourcePath?: string;
-};
-
 function fail(message: string, cause?: unknown): never {
   throw new Error(message, cause === undefined ? undefined : { cause });
 }
@@ -54,33 +42,3 @@ export function parseCollectorPrNumber(raw: unknown): number {
     : "Collector pull request number must be a positive safe integer");
   return value;
 }
-
-function canonicalManifest(requests: readonly CollectorRequestConfig[]): string {
-  return `${JSON.stringify({ requests: requests.map((request) => ({ id: request.id, body: request.requestBody })) })}\n`;
-}
-
-export function emptyCollectorManifest(): CollectorManifest {
-  const canonicalJson = canonicalManifest([]);
-  return { requests: [], canonicalJson, digest: sha256Hex(canonicalJson) };
-}
-
-/** Optional request configuration. It names requests, never expected observers. */
-export async function loadCollectorManifest(path: string): Promise<CollectorManifest> {
-  let bytes: Buffer;
-  try { bytes = await readFile(path); } catch (error) { fail(`Collector request manifest is unreadable at ${path}`, error); }
-  let parsed: unknown;
-  try { parsed = JSON.parse(new TextDecoder("utf-8", { fatal: true }).decode(bytes)); } catch (error) { fail("Collector request manifest must be UTF-8 JSON", error); }
-  if (!isRecord(parsed)) fail("Collector request manifest must be an object");
-  const rawRequests = parsed.requests ?? [];
-  if (!Array.isArray(rawRequests)) fail("Collector request manifest requests must be an array");
-  const requests: CollectorRequestConfig[] = [];
-  const ids = new Set<string>();
-  for (const [index, item] of rawRequests.entries()) {
-    if (!isRecord(item) || typeof item.id !== "string" || item.id.length === 0 || typeof item.body !== "string" || item.body.trim() === "") fail(`Collector request manifest requests[${index}] is invalid`);
-    if (ids.has(item.id)) fail(`Collector request manifest has duplicate request id "${item.id}"`);
-    ids.add(item.id);
-    requests.push({ id: item.id, requestBody: item.body });
-  }
-  const canonicalJson = canonicalManifest(requests);
-  return { requests, canonicalJson, digest: sha256Hex(canonicalJson), sourcePath: path };
-}

package/src/collector-role.ts

@@ -2,17 +2,15 @@
  * Collector business: soul/material assembly and sole submission tool.
  * #1088: code no longer observes/requests/merges GitHub findings — the LLM
  * uses host CLI tools. Envelope owns host-surface + construction seatbelt.
+ * #1165: --request-manifest is a caller path only; package does not load it.
  */
 import type { RoleHost, HostContext } from "./host-contracts.ts";
 import { registerFiledSubmissionTool } from "./filed-submission.ts";
 import { roleSubmissionDeclaration } from "./role-submission-declarations.ts";
 
 import {
-  emptyCollectorManifest,
-  loadCollectorManifest,
   parseCollectorPrNumber,
   parseCollectorRepository,
-  type CollectorManifest,
   type CollectorRepository,
 } from "./collector-config.ts";
 import {
@@ -47,7 +45,7 @@ export const COLLECTOR_TRANSPORT_FLAGS = Object.freeze([
     name: "ak-collector-request-manifest",
     definition: Object.freeze({
       description:
-        "Path to the Collector v1 request manifest JSON file (caller guidance; not a machine collection directive).",
+        "Caller path to a request manifest JSON file (passed as-is; package does not read or validate).",
       type: "string" as const,
     }),
   }),
@@ -62,7 +60,7 @@ export type CollectorActivation = {
   soul: string;
   repository: CollectorRepository;
   prNumber: number | undefined;
-  manifest: CollectorManifest;
+  requestManifestPath?: string;
 };
 
 function buildMethodContext(activation: CollectorActivation): string {
@@ -71,19 +69,11 @@ function buildMethodContext(activation: CollectorActivation): string {
     `host: github.com`,
     `repository: ${activation.repository.canonical}`,
     `prNumber: ${activation.prNumber === undefined ? "未绑定" : String(activation.prNumber)}`,
-    `requests: ${JSON.stringify(activation.manifest.requests.map((request) => ({ id: request.id })))}`,
-    `manifestDigest: ${activation.manifest.digest}`,
+    ...(activation.requestManifestPath === undefined
+      ? []
+      : [`requestManifestPath: ${activation.requestManifestPath}`]),
     "</collector_method>",
   ];
-  if (activation.manifest.requests.length > 0) {
-    const payload = JSON.stringify({
-      requests: activation.manifest.requests.map((request) => ({
-        id: request.id,
-        body: request.requestBody,
-      })),
-    }).replaceAll("<", "\\u003c");
-    lines.push("", "<collector_request_manifest>", payload, "</collector_request_manifest>");
-  }
   return lines.join("\n");
 }
 
@@ -123,15 +113,15 @@ export function createCollectorRoleRuntime(
       } else if (typeof prFlag === "number") {
         prNumber = parseCollectorPrNumber(prFlag);
       }
-      const manifest = typeof requestManifestFlag === "string" && requestManifestFlag.trim().length > 0
-        ? await loadCollectorManifest(requestManifestFlag)
-        : emptyCollectorManifest();
+      // #1165: opaque caller path — keep spaces/"" when the flag was provided.
+      const requestManifestPath =
+        typeof requestManifestFlag === "string" ? requestManifestFlag : undefined;
 
       return {
         soul,
         repository,
         prNumber,
-        manifest,
+        ...(requestManifestPath === undefined ? {} : { requestManifestPath }),
       };
     },
 

package/src/public-cli/instruction-seat-run.ts

@@ -48,7 +48,6 @@ import {
   recordAdmittedCorrelation,
   recordChildDiaristRun,
   relocateAdmittedRunToTicket,
-  withPreparedAttachments,
   type AdmittedCountersignInvocation,
   type AdmittedRoleInvocation,
   type PublicSeatParse,
@@ -618,7 +617,7 @@ async function runCountersignBody(
       summons: {
         sourceRunPath: gateParentRunPath,
         instruction: resumeInstruction,
-        instructionEmpty: resumeInstruction.trim() === "",
+        instructionEmpty: resumeInstruction.length === 0,
       },
     });
     if (resumed != null) return resumed;
@@ -634,12 +633,12 @@ async function runCountersignBody(
   }
 
   try {
-    return await withPreparedAttachments(parsed.attachmentPaths ?? [], async (preparedAttachments) => {
+    {
       const materializeAdmission = async (ticketNumber?: number): Promise<void> => {
         await materializeCountersignInvocation(admitted, {
           home: env.home,
           principalAuthority: env.principalAuthority,
-          preparedAttachments,
+          attachmentPaths: parsed.attachmentPaths ?? [],
           ...(env.model === undefined ? {} : { model: env.model }),
           ...(ticketNumber === undefined ? {} : { ticketNumber }),
         });
@@ -676,7 +675,7 @@ async function runCountersignBody(
       });
       await relocateAdmittedRunToTicket(admitted, env.principalAuthority);
       return result;
-    });
+    }
   } catch (error) {
     const rejected = usageExit(error, io);
     if (rejected !== undefined) return rejected;
@@ -749,7 +748,7 @@ export async function runPublicInstructionSeat(
     const summons: SameTicketSummonsMaterials = {
       ...sameParentInstruction(env, {
         instruction: parsed.instruction ?? "",
-        instructionEmpty: (parsed.instruction ?? "").trim() === "",
+        instructionEmpty: (parsed.instruction ?? "").length === 0,
       }),
       attachmentPaths: parsed.attachmentPaths ?? [],
     };
@@ -775,7 +774,7 @@ export async function runPublicInstructionSeat(
         sourceRunPath: parentRunPath,
         ...sameParentInstruction(env, {
           instruction: parsed.instruction ?? "",
-          instructionEmpty: (parsed.instruction ?? "").trim() === "",
+          instructionEmpty: (parsed.instruction ?? "").length === 0,
         }),
         attachmentPaths: parsed.attachmentPaths ?? [],
       };
@@ -865,7 +864,7 @@ export async function runPublicInstructionSeat(
           instruction: parsed.instruction ?? "",
           projectRoot: admitted.projectRoot,
           failureLabel: "secretariat unbound summons",
-          attachmentPaths: admitted.attachments.map((attachment) => attachment.frozenPath),
+          attachmentPaths: admitted.attachments.map((attachment) => attachment.path),
           correlationId: admitted.runId,
         }, env, io);
       } catch (error) {