@akagilnc/pi-workflow-roles 0.1.5583 → 0.1.5598

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. */
@@ -370,8 +370,8 @@ const FIXER_OPTIONS = [
         repeatable: false,
         form: "option",
         description: {
-            en: "JSON array of {id, requirement} prerequisite objects.",
-            zh: "{id, requirement} 前置条件 JSON 数组路径。",
+            en: "Caller prerequisites path passed as-is to the role. Package does not read, copy, or validate; use an absolute path when cwd differs from --project.",
+            zh: "调用方前置条件文件路径原样递给衙门。包不读、不复制、不校验；调用方当前目录与 --project 不同时请给绝对路径。",
         },
     },
 ];
@@ -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 ["inputPath"]) {
             const record = activation;
             if (field in record)
                 record[field] = rewrite(record[field]);
         }
+        // #1165/#1168: caller file-flag paths are not activation fields to 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
@@ -766,10 +766,7 @@ async function loadResumableRunRecord(home, runId, authority) {
     let instructionEmpty = true;
     let attachments = [];
     let phase;
-    let taskPath;
-    let packetPath;
     let prerequisitesPath;
-    let prerequisites;
     let baseRevision;
     let lens;
     let authorityRefs;
@@ -781,7 +778,6 @@ async function loadResumableRunRecord(home, runId, authority) {
     let repository;
     let repositoryDisplay;
     let requestManifestPath;
-    let manifestDigest;
     let waitWindowMs;
     let issueNumber;
     let caseRunsPath;
@@ -807,24 +803,17 @@ 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;
             }
-            if (typeof record.taskPath === "string" && record.taskPath.trim() !== "") {
-                taskPath = record.taskPath;
-            }
-            if (typeof record.packetPath === "string" && record.packetPath.trim() !== "") {
-                packetPath = record.packetPath;
-            }
-            if (typeof record.prerequisitesPath === "string" &&
-                record.prerequisitesPath.trim() !== "") {
+            // #1168: opaque caller path on resume — do not trim-filter provided values.
+            if (typeof record.prerequisitesPath === "string") {
                 prerequisitesPath = record.prerequisitesPath;
             }
-            if (Array.isArray(record.prerequisites)) {
-                prerequisites = record.prerequisites;
-            }
             if (typeof record.baseRevision === "string" &&
                 record.baseRevision.trim() !== "") {
                 baseRevision = record.baseRevision;
@@ -842,12 +831,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;
             }
@@ -975,10 +962,7 @@ async function loadResumableRunRecord(home, runId, authority) {
             instructionEmpty,
             attachments,
             ...(phase === undefined ? {} : { phase }),
-            ...(taskPath === undefined ? {} : { taskPath }),
-            ...(packetPath === undefined ? {} : { packetPath }),
             ...(prerequisitesPath === undefined ? {} : { prerequisitesPath }),
-            ...(prerequisites === undefined ? {} : { prerequisites }),
             ...(baseRevision === undefined ? {} : { baseRevision }),
             ...(lens === undefined ? {} : { lens }),
             ...(authorityRefs === undefined ? {} : { authorityRefs }),
@@ -990,7 +974,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 }),
@@ -1081,10 +1064,6 @@ function admitResumedRole(loaded) {
         }
         case "worker-task": {
             const phase = resumedWorkerPhase(fields.phase ?? loaded.run.phase, record.phases, role, runId);
-            const taskPath = fields.taskPath;
-            if (taskPath === undefined) {
-                throw new CliUsageError(`role run admitted coder task path is missing: ${runId}`);
-            }
             if (fields.instruction.trim() === "") {
                 throw new CliUsageError(`role run admitted coder task is blank: ${runId}`);
             }
@@ -1093,30 +1072,22 @@ function admitResumedRole(loaded) {
                 phase,
                 ...base,
                 instructionEmpty: false,
-                taskPath,
             };
             return admitted;
         }
         case "worker-packet": {
             const phase = resumedWorkerPhase(fields.phase ?? loaded.run.phase, record.phases, role, runId);
-            const packetPath = fields.packetPath;
-            if (packetPath === undefined) {
-                throw new CliUsageError(`role run admitted fixer packet path is missing: ${runId}`);
-            }
             if (fields.instruction.trim() === "") {
                 throw new CliUsageError(`role run admitted fixer instruction is blank: ${runId}`);
             }
-            const prerequisites = fields.prerequisites ?? Object.freeze([]);
             const admitted = {
                 role: "fixer",
                 phase,
                 ...base,
                 instructionEmpty: false,
-                packetPath,
                 ...(fields.prerequisitesPath === undefined
                     ? {}
                     : { prerequisitesPath: fields.prerequisitesPath }),
-                prerequisites,
             };
             return admitted;
         }
@@ -1179,10 +1150,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 +1174,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

@@ -19,10 +19,9 @@ const ADMITTED_PAGE_FIELDS = [
     "admittedRequestPath", // pre-#1161 pages only
     "sessionDirectory",
     "sessionFile",
-    "taskPath",
-    "packetPath",
-    "prerequisitesPath",
-    "requestManifestPath",
+    // #1165/#1168: caller file-flag paths (requestManifestPath, prerequisitesPath)
+    // stay as given. Obsolete copy fields (taskPath/packetPath) are not rewritten —
+    // stock volumes keep their bytes; live relocate no longer maintains them.
     "mergerInputPath",
     "sourceRunPath",
 ];
@@ -150,28 +149,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 +337,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 +357,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.5598",
   "description": "Soul-bound workflow roles for Pi",
   "type": "module",
   "license": "Apache-2.0",

package/scripts/build-package.mjs

@@ -55,7 +55,6 @@ const entries = [
   "open-tool-schema",
   "package-contracts/collector-output",
   "package-contracts/fixer-output",
-  "package-contracts/fixer-packet",
   "package-contracts/gatekeeper-output",
   "package-contracts/judge-output",
   "package-contracts/navigator-output",

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/host-contracts.ts

@@ -87,13 +87,10 @@ export type RoleTurnActivation =
   | {
       readonly role: "coder";
       readonly phase: string;
-      readonly taskPath: string;
     }
   | {
       readonly role: "fixer";
       readonly phase: string;
-      readonly packetPath: string;
-      readonly prerequisitesPath?: string;
     }
   | {
       readonly role: "reviewer";

package/src/merger-contracts.ts

@@ -11,7 +11,8 @@ export const mergerInputSchema = Type.Object({
   attemptId: Type.String({ minLength: 1, description: "对账 attempt 身份（transport）" }),
   targetObjectId: Type.String({ description: "材料：target parent OID；可空" }),
   sourceObjectId: Type.String({ description: "材料：source parent OID；可空" }),
-  materials: Type.Object({ task: materialSchema, authority: materialSchema, targetIntent: materialSchema, sourceIntent: materialSchema }),
+  // #1168: task/authority dispatch copies removed; git-fact intents remain.
+  materials: Type.Object({ targetIntent: materialSchema, sourceIntent: materialSchema }),
   expectedConflictPaths: Type.Array(Type.String(), { description: "材料：当前未合并路径；可空" }),
   resolutionScope: Type.Array(Type.String(), { description: "材料：解析范围提示；可空" }),
   authorizedChecks: Type.Array(checkSchema),
@@ -62,7 +63,10 @@ function readMaterial(value: unknown, label: string): MergerMaterial {
 export function validateMergerInput(value: unknown): MergerInput {
   if (!isRecord(value) || blank(value.attemptId)) fail("Merger input requires a non-empty attemptId for accounting");
   if (!isRecord(value.materials)) fail("Merger materials are missing");
-  const materials = { task: readMaterial(value.materials.task, "task"), authority: readMaterial(value.materials.authority, "authority"), targetIntent: readMaterial(value.materials.targetIntent, "targetIntent"), sourceIntent: readMaterial(value.materials.sourceIntent, "sourceIntent") };
+  const materials = {
+    targetIntent: readMaterial(value.materials.targetIntent, "targetIntent"),
+    sourceIntent: readMaterial(value.materials.sourceIntent, "sourceIntent"),
+  };
   const authorizedChecks = Array.isArray(value.authorizedChecks) ? value.authorizedChecks.filter((check): check is Record<string, unknown> => isRecord(check)).map(c => ({ name: asString(c.name), argv: asStringArray(c.argv).filter(a => a.trim().length > 0) })).filter(c => c.name.trim().length > 0 && c.argv.length > 0) : [];
   return deepFreeze(structuredClone({ attemptId: (value.attemptId as string).trim(), targetObjectId: asString(value.targetObjectId), sourceObjectId: asString(value.sourceObjectId), materials, expectedConflictPaths: asStringArray(value.expectedConflictPaths), resolutionScope: asStringArray(value.resolutionScope), authorizedChecks }) as MergerInput);
 }

package/src/merger-role.ts

@@ -21,6 +21,7 @@ function materialText(input: MergerInput, key: keyof MergerInput["materials"]):
  * Merger (校书郎) role runtime.
  * Submission tool records every call (#836); host end is the final.
  * Git state / completion verification are not attendance or rejection gates (#827).
+ * #1168: dispatch text is the first message; assignment carries git facts only.
  */
 export function createMergerRoleRuntime(pi: RoleHost, dependencies: MergerRoleDependencies) {
   let activation: { soul: string; input: Readonly<MergerInput> } | undefined;
@@ -41,7 +42,16 @@ export function createMergerRoleRuntime(pi: RoleHost, dependencies: MergerRoleDe
         });
         pi.on("before_agent_start", (event) => {
           if (!activation) throw new Error("校书郎未激活");
-          const admitted = { attemptId: activation.input.attemptId, targetObjectId: activation.input.targetObjectId, sourceObjectId: activation.input.sourceObjectId, task: materialText(activation.input, "task"), authority: materialText(activation.input, "authority"), targetIntent: materialText(activation.input, "targetIntent"), sourceIntent: materialText(activation.input, "sourceIntent"), expectedConflictPaths: activation.input.expectedConflictPaths, resolutionScope: activation.input.resolutionScope, authorizedChecks: activation.input.authorizedChecks };
+          const admitted = {
+            attemptId: activation.input.attemptId,
+            targetObjectId: activation.input.targetObjectId,
+            sourceObjectId: activation.input.sourceObjectId,
+            targetIntent: materialText(activation.input, "targetIntent"),
+            sourceIntent: materialText(activation.input, "sourceIntent"),
+            expectedConflictPaths: activation.input.expectedConflictPaths,
+            resolutionScope: activation.input.resolutionScope,
+            authorizedChecks: activation.input.authorizedChecks,
+          };
           return { systemPrompt: `${event.systemPrompt}\n\n<merger_soul>\n${activation.soul}\n</merger_soul>\n\n<merger_assignment>\n${JSON.stringify(admitted)}\n</merger_assignment>` };
         });
       }

package/src/navigator-attendance.ts

@@ -229,14 +229,14 @@ export function navigatorSubjectKey(
 /**
  * Role inputs for one ad-hoc work item live below role-specific run folders.
  * The folder and filename are transport, not identity: the shared work root
- * keeps task.md, fix-packet.json, and other natural inputs on one subject.
+ * keeps natural inputs on one subject.
  */
 export function navigatorSubjectKeyForInput(subjectRoot: string, reference: string, cwd = process.cwd()): string {
   if (issueRoot(subjectRoot) !== undefined || !subjectRoot.includes("/.ak/work/")) return subjectRoot;
   const resolvedReference = resolve(cwd, reference);
   const marker = "/runs/";
   if (resolvedReference.includes(marker)) {
-    // The work root, not task.md/fix-packet.json/etc., is the stable subject.
+    // The work root, not a role-input filename, is the stable subject.
     // Different roots remain isolated without inventing a filename convention.
     return subjectRoot;
   }