@akagilnc/pi-workflow-roles 0.1.5577 → 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,15 +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 { pickEngineAxis } from "../package-resources/engine-material.js";
 import { readStoredHostSessionId } from "../session-identity.js";
 import { reportRunRecord } from "../sitian-facade.js";
 import { createReceiptDeliveryPolicy, deliveryLimitFromConfig, } from "../receipt-delivery-policy.js";
@@ -52,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) {
@@ -926,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({
@@ -946,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) {
@@ -1047,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 };
 }
 /**
@@ -1138,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).
@@ -1345,13 +1315,7 @@ export async function runPostAdmissionOneShot(input) {
             ...input.request,
             continuation: {
                 kind: "resume",
-                prompt: buildAutoResumeContinuationPrompt({
-                    packageRoot: input.env.packageRoot,
-                    ...pickEngineAxis({
-                        engine: input.effectiveEngine ?? input.env.engine,
-                        engineModel: input.env.engineModel,
-                    }),
-                }),
+                prompt: buildAutoResumeContinuationPrompt(),
             },
         }),
         adapters: input.adapters,

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

@@ -13,9 +13,8 @@ import { isSafePositiveTicketNumber } from "../run-ticket-number.js";
 import { CliUsageError } from "./cli-errors.js";
 import { parseCollectorRepository } from "../collector-config.js";
 import { retainedRunPathsMatch, rewriteAdmittedRoleRunPage, rewriteRunDirectoryPathValue, } from "../role-run-relocation.js";
-import { appendEngineSessionMaterial, engineSessionMaterialFromOptions, pickEngineAxis, } from "../package-resources/engine-material.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
@@ -33,14 +32,12 @@ export const AUTO_RESUME_LIMIT = 2;
 export const RESUME_TRANSPORT_ENVELOPE = "继续。";
 /**
  * Auto-resume continuation only (#959 / ADR 0080).
- * Always non-empty: Chinese neutral envelope plus optional engine pointers.
+ * Always non-empty Chinese neutral envelope. Engine / outsourcing material
+ * rides startup readingMaterial (#1167), never this continuation.
  * Never call this from manual `ak-role resume`.
  */
-export function buildAutoResumeContinuationPrompt(options) {
-    return appendEngineSessionMaterial([RESUME_TRANSPORT_ENVELOPE], engineSessionMaterialFromOptions({
-        packageRoot: options.packageRoot,
-        ...pickEngineAxis(options),
-    })).join("\n");
+export function buildAutoResumeContinuationPrompt() {
+    return RESUME_TRANSPORT_ENVELOPE;
 }
 const WRITER_LOCK_FILE = "writer.lock";
 export async function writeRoleRunState(runDirectory, record) {
@@ -54,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
@@ -784,7 +781,6 @@ async function loadResumableRunRecord(home, runId, authority) {
     let repository;
     let repositoryDisplay;
     let requestManifestPath;
-    let manifestDigest;
     let waitWindowMs;
     let issueNumber;
     let caseRunsPath;
@@ -810,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;
@@ -845,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;
             }
@@ -993,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 }),
@@ -1182,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;
@@ -1208,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.5577",
+  "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/package-resources/engine-material.ts

@@ -1,23 +1,27 @@
 /**
- * Packaged engine method-material seam (#356 T1 / ADR 0069 / #376).
+ * Packaged engine method-material seam (#356 T1 / ADR 0069 / #376 / #1167).
  * Engine names are owner pool-directive labels — not a closed material catalog.
  * Material body is optional data for the LLM, not a code contract.
  * Only path-safety syntax is checked at real I/O seams.
+ * Delivery is startup readingMaterial (handbook bodies), never the transport prompt.
  */
-import { existsSync, readdirSync } from "node:fs";
+import { existsSync, readdirSync, readFileSync } from "node:fs";
 import { join } from "node:path";
 
 const ENGINE_MATERIAL_RELATIVE_ROOT = "resources/engines" as const;
+const ENGINE_DISPATCH_RELATIVE_PATH = "resources/engine-dispatch.md" as const;
 
 export type EngineSessionMaterial = Readonly<{
   name: string;
-  /** Present only when a packaged notes file exists for this name. */
-  materialPath?: string;
   /**
    * Optional owner pool-directive model id for multi-model engines (#883).
    * Opaque pass-through coordinate for the seat — not validated against a catalog.
    */
   model?: string;
+  /** Packaged per-engine handbook body when notes exist. */
+  handbook?: string;
+  /** Shared engine-dispatch.md body when the per-engine handbook exists. */
+  dispatchHandbook?: string;
 }>;
 
 /** Project engine name + optional model for env / request / material spreads. */
@@ -57,6 +61,10 @@ export function resolveEngineMaterialDirectory(packageRoot: string): string {
   return join(packageRoot, ENGINE_MATERIAL_RELATIVE_ROOT);
 }
 
+export function resolveEngineDispatchMaterialPath(packageRoot: string): string {
+  return join(packageRoot, ENGINE_DISPATCH_RELATIVE_PATH);
+}
+
 /**
  * Enumerate packaged engine notes stems (discovery only — not a legal-name gate).
  * Only `*.md` stems that pass name syntax are listed.
@@ -99,7 +107,7 @@ export function assertLegalEngineName(name: string): string {
 /**
  * Resolve optional engine options into session material coordinates.
  * No engine → undefined (caller keeps default prompt bytes).
- * Engine with packaged notes → name + absolute material path.
+ * Engine with packaged notes → name + handbook body + dispatch handbook body.
  * Engine without notes → name only (pass-through; no warning).
  */
 export function engineSessionMaterialFromOptions(options: {
@@ -119,41 +127,40 @@ export function engineSessionMaterialFromOptions(options: {
       : assertLegalEngineModel(options.engineModel);
   const materialPath = resolveEngineMaterialPath(options.packageRoot, name);
   const modelField = model === undefined ? {} : { model };
-  if (existsSync(materialPath)) {
-    return Object.freeze({ name, materialPath, ...modelField });
+  if (!existsSync(materialPath)) {
+    return Object.freeze({ name, ...modelField });
   }
-  return Object.freeze({ name, ...modelField });
+  const handbook = readFileSync(materialPath, "utf8");
+  const dispatchPath = resolveEngineDispatchMaterialPath(options.packageRoot);
+  const dispatchField = existsSync(dispatchPath)
+    ? { dispatchHandbook: readFileSync(dispatchPath, "utf8") }
+    : {};
+  return Object.freeze({ name, handbook, ...dispatchField, ...modelField });
 }
 
 /**
- * Append engine method-material delivery to session initial material lines.
- * No engine → identity copy (byte-stable when joined the same way).
- * With notes → Chinese neutral handbook header + engine name + absolute material path.
- * Name only → engine name coordinate only (no handbook header, no path, no warning).
- * Never delivers material body.
+ * Typed startup readingMaterial for a resolved engine session material.
+ * Undefined engine → undefined (no outsourcing segment).
  */
-export function appendEngineSessionMaterial(
-  lines: readonly string[],
+export function engineSessionReadingMaterial(
   engineMaterial?: EngineSessionMaterial,
-): string[] {
-  if (engineMaterial === undefined) {
-    return [...lines];
-  }
-  const out = [...lines];
-  out.push("");
-  if (engineMaterial.materialPath !== undefined) {
-    out.push("本次配置的劳务引擎及其手册：");
-    out.push(`- engine: ${engineMaterial.name}`);
-    if (engineMaterial.model !== undefined) {
-      out.push(`- engineModel: ${engineMaterial.model}`);
-    }
-    out.push(`- ${engineMaterial.materialPath}`);
-  } else {
-    // Name-only pass-through: no packaged bytes to claim as handbook.
-    out.push(`- engine: ${engineMaterial.name}`);
-    if (engineMaterial.model !== undefined) {
-      out.push(`- engineModel: ${engineMaterial.model}`);
+):
+  | {
+      readonly kind: "engine-session-material";
+      readonly name: string;
+      readonly model?: string;
+      readonly handbook?: string;
+      readonly dispatchHandbook?: string;
     }
-  }
-  return out;
+  | undefined {
+  if (engineMaterial === undefined) return undefined;
+  return {
+    kind: "engine-session-material" as const,
+    name: engineMaterial.name,
+    ...(engineMaterial.model === undefined ? {} : { model: engineMaterial.model }),
+    ...(engineMaterial.handbook === undefined ? {} : { handbook: engineMaterial.handbook }),
+    ...(engineMaterial.dispatchHandbook === undefined
+      ? {}
+      : { dispatchHandbook: engineMaterial.dispatchHandbook }),
+  };
 }