@akagilnc/pi-workflow-roles 0.1.3783 → 0.1.3789

package/dist/public-cli/main.js

@@ -19741,8 +19741,8 @@ var init_option_definitions = __esm({
         repeatable: false,
         form: "option",
         description: {
-          en: "Optional request manifest JSON path ({requests:[{id,body}]}).",
-          zh: "\u53EF\u9009 request manifest JSON \u8DEF\u5F84\uFF08{requests:[{id,body}]}\uFF09\u3002"
+          en: "Optional request manifest JSON path ({requests:[{id,body}]}). Role may also decide request bodies from handbook/field activity without this file.",
+          zh: "\u53EF\u9009 request manifest JSON \u8DEF\u5F84\uFF08{requests:[{id,body}]}\uFF09\u3002\u89D2\u8272\u4EA6\u53EF\u4F9D\u624B\u518C/\u73B0\u573A\u81EA\u884C\u5224\u5B9A\u8BF7\u6C42\u6B63\u6587\uFF0C\u4E0D\u5FC5\u4F9D\u8D56\u672C\u6587\u4EF6\u3002"
         }
       }
     ];
@@ -23613,84 +23613,14 @@ var init_collector_identity = __esm({
   }
 });
 
-// src/collector-tool-schemas.ts
-var collectorObserveArgsSchema, collectorRequestArgsSchema, collectorReadArgsSchema, collectorWaitArgsSchema, collectorBindTargetArgsSchema, collectorFindingItemDeclaration, collectorOutputBaseSchema, collectorOutputArgsSchema;
-var init_collector_tool_schemas = __esm({
-  "src/collector-tool-schemas.ts"() {
-    "use strict";
-    init_build();
-    init_collector_evidence();
-    init_open_tool_schema();
-    init_terminating_infrastructure();
-    collectorObserveArgsSchema = typebox_exports.Object({}, { additionalProperties: false });
-    collectorRequestArgsSchema = typebox_exports.Object({
-      requestId: typebox_exports.String({ minLength: 1, description: "\u914D\u7F6E\u8BF7\u6C42\u8EAB\u4EFD" }),
-      snapshotId: typebox_exports.String({ minLength: 1, description: "\u6700\u65B0\u7559\u5B58\u89C2\u5BDF\u5FEB\u7167" })
-    }, { additionalProperties: false });
-    collectorReadArgsSchema = typebox_exports.Object({
-      evidenceId: typebox_exports.String({ minLength: 1, description: "observe \u8FD4\u56DE\u7684\u6750\u6599\u8BC1\u636E id\uFF08evidenceId\uFF09" })
-    }, { additionalProperties: false });
-    collectorWaitArgsSchema = typebox_exports.Object({
-      durationMs: typebox_exports.Integer({ minimum: 1, maximum: COLLECTOR_ELIGIBILITY_MS, description: "\u7B49\u5F85\u6BEB\u79D2\uFF1B\u5355\u6B21\u4E0A\u9650\u4E94\u5206\u949F\u4E14\u4E0D\u8D85\u5269\u4F59\u8D44\u683C" })
-    }, { additionalProperties: false });
-    collectorBindTargetArgsSchema = typebox_exports.Object({
-      prNumber: typebox_exports.Optional(typebox_exports.Unknown({
-        description: "\u89D2\u8272\u5224\u5B9A\u7684\u672C\u4ED3 PR \u53F7\uFF08\u6B63\u6574\u6570\uFF09\u3002\u4E0E issueNumber \u4E8C\u9009\u4E00\u6216\u540C\u6307\u552F\u4E00\u76EE\u6807\u3002\u5F62\u72B6\u6307\u5F15\uFF0C\u975E schema \u95F8\u3002"
-      })),
-      issueNumber: typebox_exports.Optional(typebox_exports.Unknown({
-        description: "\u89D2\u8272\u5224\u5B9A\u7684\u672C\u4ED3 issue \u53F7\uFF08\u6B63\u6574\u6570\uFF09\uFF1Bruntime \u7ECF\u7EBF\u4E0A\u5173\u8054\u89E3\u6790\u552F\u4E00 PR\u3002\u5F62\u72B6\u6307\u5F15\uFF0C\u975E schema \u95F8\u3002"
-      }))
-    }, { additionalProperties: true });
-    collectorFindingItemDeclaration = (() => {
-      const item = typebox_exports.Object(
-        {
-          evidenceId: typebox_exports.Unknown({
-            description: "observe \u8FD4\u56DE\u7684\u6750\u6599\u6307\u9488\uFF08\u5FC5\u586B\u8BED\u4E49\uFF09"
-          }),
-          category: typebox_exports.Unknown({
-            description: "\u7B80\u77ED\u5F52\u7C7B\u6807\u7B7E\uFF0C\u4E0D\u662F\u6458\u8981"
-          }),
-          summary: typebox_exports.Unknown({
-            description: "\u54EA\u4E2A bot\u3001\u4EC0\u4E48\u95EE\u9898\u7684\u6458\u8981\uFF1B\u4E0D\u8A8A\u6284\u6B63\u6587"
-          })
-        },
-        {
-          additionalProperties: true,
-          description: "\u5355\u6761 finding \u6307\u9488\uFF1AevidenceId + \u53EF\u9009 category/summary\u3002\u5F62\u72B6\u6307\u5F15\uFF0C\u975E schema \u95F8\u3002"
-        }
-      );
-      item.required = [];
-      return item;
-    })();
-    collectorOutputBaseSchema = openToolObject(
-      typebox_exports.Object({
-        // No root type:array — host must not shape-reject non-array findings (#676 C).
-        // Nested item declarations ride `items` for registration preservation (ADR 0057).
-        findings: typebox_exports.Unsafe({
-          description: "\u672C\u6B21\u6536\u96C6\u5230\u7684\u9010\u6761 findings\uFF08\u6307\u9488\u6570\u7EC4\u4E3A\u89C4\u8303\u5F62\uFF09\u3002\u96F6 finding \u7684\u6A21\u677F\u901A\u77E5\u4E0D\u5F97\u8FDB\u5165\uFF1B\u6B63\u5E38\u5B8C\u5DE5\u65E0 finding \u65F6\u7701\u7565\u3002\u5F62\u72B6\u6307\u5F15\uFF0C\u975E schema \u95F8\u3002",
-          items: collectorFindingItemDeclaration
-        }),
-        unfinishedReasons: typebox_exports.Unknown({
-          description: "\u672A\u5B8C\u6210\u539F\u56E0\u5B57\u7B26\u4E32\u6570\u7EC4\uFF08\u989D\u5EA6/\u6545\u969C/\u7B49\u5F85\u5C4A\u6EE1\u7B49\u73B0\u573A\u4F9D\u636E\uFF09\uFF1B\u4E0D\u5F97\u628A\u672A\u5B8C\u6210\u8868\u8FF0\u4E3A\u65E0\u95EE\u9898\u3002\u65E0\u53EF\u62A5\u544A\u65F6\u7701\u7565\u3002\u5F62\u72B6\u6307\u5F15\uFF0C\u975E schema \u95F8\u3002"
-        })
-      })
-    );
-    collectorOutputArgsSchema = withInfrastructureFailureDeclaration(
-      collectorOutputBaseSchema
-    );
-  }
-});
-
 // src/collector-ledger.ts
 var COLLECTOR_OBSERVE_TOOL, COLLECTOR_READ_TOOL, COLLECTOR_REQUEST_TOOL, COLLECTOR_WAIT_TOOL, COLLECTOR_BIND_TARGET_TOOL;
 var init_collector_ledger = __esm({
   "src/collector-ledger.ts"() {
     "use strict";
-    init_value2();
     init_collector_evidence();
     init_collector_github();
     init_collector_identity();
-    init_collector_tool_schemas();
     init_collector_output();
     COLLECTOR_OBSERVE_TOOL = "ak_collector_observe";
     COLLECTOR_READ_TOOL = "ak_collector_read";

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

@@ -483,8 +483,8 @@ const COLLECTOR_OPTIONS = [
         repeatable: false,
         form: "option",
         description: {
-            en: "Optional request manifest JSON path ({requests:[{id,body}]}).",
-            zh: "可选 request manifest JSON 路径（{requests:[{id,body}]}）。",
+            en: "Optional request manifest JSON path ({requests:[{id,body}]}). Role may also decide request bodies from handbook/field activity without this file.",
+            zh: "可选 request manifest JSON 路径（{requests:[{id,body}]}）。角色亦可依手册/现场自行判定请求正文，不必依赖本文件。",
         },
     },
 ];

package/dist/role-runtime.js

@@ -1223,6 +1223,9 @@ export function createRoleRuntimeExtension(dependencies) {
             ...(dependencies.createCollectorClock === undefined
                 ? {}
                 : { createClock: dependencies.createCollectorClock }),
+            ...(dependencies.loadCollectorHandbookSeed === undefined
+                ? {}
+                : { loadHandbookSeed: dependencies.loadCollectorHandbookSeed }),
         }, hostActions);
         // #676 J1: business tools + tool_call gate register only behind admission/activation.
         // before_agent_start / tool_result collector branches live on the shared envelope hooks above.

package/extensions/role-runtime.ts

@@ -40,6 +40,7 @@ import { loadGatekeeperSessionMaterials, loadMainRoleSessionMaterials } from "..
 const extensionPath = fileURLToPath(import.meta.url);
 const packageRoot = fileURLToPath(new URL("..", import.meta.url));
 const navigatorRoutePlaybookPath = fileURLToPath(new URL("../resources/navigator-route-playbook.md", import.meta.url));
+const collectorHandbookSeedPath = fileURLToPath(new URL("../resources/collector-bot-handbook.md", import.meta.url));
 // #675: nested public summons (gate/auditor) resolve root via env under jiti.
 if (process.env.AK_ROLE_PACKAGE_ROOT === undefined || process.env.AK_ROLE_PACKAGE_ROOT.trim() === "") {
   process.env.AK_ROLE_PACKAGE_ROOT = packageRoot;
@@ -122,6 +123,7 @@ export default function roleRuntime(pi: ExtensionAPI): void {
     createReviewerPinnedGitReader: () => createReviewerPinnedGitReader(),
     createReviewerIssueFetcher: () => createGhIssueSoftFetcher(),
     loadCollectorSoul: () => loadMainRoleSessionMaterials("collector"),
+    loadCollectorHandbookSeed: () => readFile(collectorHandbookSeedPath, "utf8"),
     createCollectorTransport: () => createGhCollectorGitHubTransport(),
     loadDoctorSoul: () => loadMainRoleSessionMaterials("doctor"),
     loadDoctorCase,

package/package.json

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

package/resources/collector-bot-handbook.md

@@ -0,0 +1,33 @@
+# 通进司 bot 工作记忆（通用）
+
+薄手册：触发方式、自动/手动条件、完成与不可继续的识别线索、官方出处。
+内容是角色阅读与修正的工作记忆，不是代码状态规则；临时额度/故障不得固化为永久行为。
+参与者依本仓使用与现场活动辨认，不把下列名称当作全仓固定名单。
+
+## Codex（hosted GitHub connector）
+
+- 官方：https://learn.chatgpt.com/docs/third-party/github
+- 常见主动请求：`@codex review`
+- 新建 PR 常有自动评审；是否已在审当前提交以现场 reviews/comments 为准
+
+## CodeRabbit
+
+- 官方：https://docs.coderabbit.ai/configuration/auto-review
+- 命令：https://docs.coderabbit.ai/guides/commands
+- 常见主动请求：`@coderabbitai review`；全量：`@coderabbitai full review`
+- 默认常为增量；本仓实际自动/暂停以现场与仓库差异条为准
+
+## Sourcery
+
+- 官方：https://docs.sourcery.ai/reviews/ 与 https://docs.sourcery.ai/reviews/commands/
+- 新建/更新与主动请求依文档与本仓现场
+
+## Cursor Bugbot / 自动化
+
+- 官方：https://prod.cursor.com/help/ai-features/bugbot 与 automations 文档
+- 内置 Bugbot 与同账号自定义任务须区分；某任务额度耗尽不推断其他任务失效
+
+## 修正规则
+
+- 优先引用官方文档；现场差异记入仓库差异条并附 PR/证据指针
+- 未知或暂未响应 ≠ 通过或确定死亡

package/src/acp-host/production-host.ts

@@ -39,6 +39,9 @@ export type ProductionAcpHostOptions = Readonly<{
 const navigatorRoutePlaybookPath = fileURLToPath(
   new URL("../../resources/navigator-route-playbook.md", import.meta.url),
 );
+const collectorHandbookSeedPath = fileURLToPath(
+  new URL("../../resources/collector-bot-handbook.md", import.meta.url),
+);
 
 /** Host-neutral packaged role runtime deps for the ACP parent-process envelope. */
 export function createAcpRoleRuntimeDependencies(packageRoot: string): RoleRuntimeDependencies {
@@ -55,6 +58,7 @@ export function createAcpRoleRuntimeDependencies(packageRoot: string): RoleRunti
     createReviewerPinnedGitReader: () => createReviewerPinnedGitReader(),
     createReviewerIssueFetcher: () => createGhIssueSoftFetcher(),
     loadCollectorSoul: () => loadMainRoleSessionMaterials("collector"),
+    loadCollectorHandbookSeed: () => readFile(collectorHandbookSeedPath, "utf8"),
     createCollectorTransport: () => createGhCollectorGitHubTransport(),
     loadDoctorSoul: () => loadMainRoleSessionMaterials("doctor"),
     loadDoctorCase,

package/src/collector-handbook.ts

@@ -0,0 +1,194 @@
+/**
+ * Collector bot handbook — opaque working memory under the book topology.
+ * Runtime stores and delivers UTF-8 text only; never parses free text into
+ * code state rules (parent #673 D3 / #677).
+ *
+ * Placement reuses the machine ledger home (ADR 0048) when the admitted
+ * session already sits under books/<bookKey>/ — no parallel persistence frame.
+ */
+import { readFile } from "node:fs/promises";
+import { join, sep } from "node:path";
+
+import { writeFileAtomically } from "./atomic-write.ts";
+import {
+  activationBookDirectory,
+  assertLedgerFileInsideHome,
+  ensureRealDirectoryTree,
+  resolveActivationLedgerHomeForPath,
+} from "./activation-ledger-topology.ts";
+import { COLLECTOR_HANDBOOK_MAX_BYTES } from "./collector-tool-schemas.ts";
+
+export type CollectorHandbookScope = "general" | "repo";
+
+export type CollectorHandbookRead = {
+  readonly general: string;
+  readonly repo: string;
+  readonly generalSource: "book" | "seed" | "empty";
+  readonly repoSource: "book" | "empty";
+  readonly generalPath: string;
+  readonly repoPath: string;
+};
+
+export type CollectorHandbookWriteResult = {
+  readonly scope: CollectorHandbookScope;
+  readonly path: string;
+  readonly byteLength: number;
+};
+
+export type CollectorHandbookStore = {
+  readonly root: string;
+  readonly repositoryCanonical: string;
+  read(): Promise<CollectorHandbookRead>;
+  write(scope: CollectorHandbookScope, body: string): Promise<CollectorHandbookWriteResult>;
+};
+
+export type CollectorHandbookPlacement = {
+  readonly ledgerHome: string;
+  readonly bookKey: string;
+  readonly root: string;
+};
+
+/**
+ * Derive handbook root from an admitted session/run path under
+ * `.ak-roles/books/<bookKey>/...`. Fails closed when topology is absent.
+ */
+export function resolveCollectorHandbookRoot(sessionPath: string): CollectorHandbookPlacement {
+  if (typeof sessionPath !== "string" || sessionPath.trim().length === 0) {
+    throw new Error("通进司手册要求非空 session 路径，且位于 books/<bookKey>/");
+  }
+  // Platform path segments only — never rewrite non-separator characters (POSIX `\\` is literal).
+  const segments = sessionPath.split(sep);
+  let bookKey: string | undefined;
+  for (let i = 0; i + 3 < segments.length; i += 1) {
+    if (segments[i] === ".ak-roles" && segments[i + 1] === "books") {
+      const candidate = segments[i + 2]!;
+      // Require a following segment so bookKey is a true path component under books/.
+      if (candidate.length > 0) {
+        bookKey = candidate;
+        break;
+      }
+    }
+  }
+  if (bookKey === undefined) {
+    throw new Error(
+      `通进司手册要求 session 位于 books/<bookKey>/；收到 ${sessionPath}`,
+    );
+  }
+  if (bookKey === "." || bookKey === "..") {
+    throw new Error(`通进司手册拒绝不安全 bookKey ${JSON.stringify(bookKey)}`);
+  }
+  const ledgerHome = resolveActivationLedgerHomeForPath(sessionPath);
+  const root = join(activationBookDirectory(ledgerHome, bookKey), "collector-handbook");
+  return { ledgerHome, bookKey, root };
+}
+
+/** Flat repo file name under handbook/repos/ — avoids nested owner/repo dirs. */
+function collectorHandbookRepoFileName(repositoryCanonical: string): string {
+  if (!/^[a-z0-9][a-z0-9._-]*\/[a-z0-9][a-z0-9._-]*$/.test(repositoryCanonical)) {
+    throw new Error(
+      `通进司手册仓库文件要求规范 owner/repo，收到 ${JSON.stringify(repositoryCanonical)}`,
+    );
+  }
+  return `${repositoryCanonical.replaceAll("/", "__")}.md`;
+}
+
+export function createCollectorHandbookStore(input: {
+  readonly ledgerHome: string;
+  readonly handbookRoot: string;
+  readonly repositoryCanonical: string;
+  readonly seedGeneral?: string;
+}): CollectorHandbookStore {
+  const generalPath = join(input.handbookRoot, "general.md");
+  const repoDir = join(input.handbookRoot, "repos");
+  const repoPath = join(repoDir, collectorHandbookRepoFileName(input.repositoryCanonical));
+
+  /**
+   * ADR 0038: confine at the real read seam. Parent dirs must be physical under
+   * ledger home; the leaf must not be a pre-existing symlink (even in-home).
+   */
+  /** Sole UTF-8 budget seam — write and read share COLLECTOR_HANDBOOK_MAX_BYTES. */
+  const assertHandbookBudget = (body: string, label: string): number => {
+    const byteLength = Buffer.byteLength(body, "utf8");
+    if (byteLength > COLLECTOR_HANDBOOK_MAX_BYTES) {
+      throw new Error(
+        `通进司手册${label} UTF-8 至多 ${COLLECTOR_HANDBOOK_MAX_BYTES} 字节，收到 ${byteLength}`,
+      );
+    }
+    return byteLength;
+  };
+
+  const readOptional = async (path: string, parentDir: string): Promise<string | undefined> => {
+    ensureRealDirectoryTree(input.ledgerHome, parentDir);
+    assertLedgerFileInsideHome(path, input.ledgerHome);
+    try {
+      const body = await readFile(path, "utf8");
+      assertHandbookBudget(body, "正文");
+      return body;
+    } catch (error) {
+      if (isNotFound(error)) return undefined;
+      throw error;
+    }
+  };
+
+  return {
+    root: input.handbookRoot,
+    repositoryCanonical: input.repositoryCanonical,
+    async read() {
+      const bookGeneral = await readOptional(generalPath, input.handbookRoot);
+      const bookRepo = await readOptional(repoPath, repoDir);
+      if (bookGeneral !== undefined) {
+        return {
+          general: bookGeneral,
+          repo: bookRepo ?? "",
+          generalSource: "book",
+          repoSource: bookRepo === undefined ? "empty" : "book",
+          generalPath,
+          repoPath,
+        };
+      }
+      const seed = input.seedGeneral;
+      if (typeof seed === "string" && seed.length > 0) {
+        assertHandbookBudget(seed, "正文");
+        return {
+          general: seed,
+          repo: bookRepo ?? "",
+          generalSource: "seed",
+          repoSource: bookRepo === undefined ? "empty" : "book",
+          generalPath,
+          repoPath,
+        };
+      }
+      return {
+        general: "",
+        repo: bookRepo ?? "",
+        generalSource: "empty",
+        repoSource: bookRepo === undefined ? "empty" : "book",
+        generalPath,
+        repoPath,
+      };
+    },
+    async write(scope, body) {
+      // body string shape = collectorHandbookWriteArgsSchema; byte budget is business (UTF-8).
+      const byteLength = assertHandbookBudget(body, "正文");
+      ensureRealDirectoryTree(input.ledgerHome, input.handbookRoot);
+      const path = scope === "general" ? generalPath : repoPath;
+      if (scope === "repo") {
+        ensureRealDirectoryTree(input.ledgerHome, repoDir);
+      }
+      assertLedgerFileInsideHome(path, input.ledgerHome);
+      await writeFileAtomically(path, body);
+      return {
+        scope,
+        path,
+        byteLength,
+      };
+    },
+  };
+}
+
+function isNotFound(error: unknown): boolean {
+  return typeof error === "object"
+    && error !== null
+    && "code" in error
+    && (error as { code?: unknown }).code === "ENOENT";
+}

package/src/collector-ledger.ts

@@ -1,5 +1,3 @@
-import Value from "typebox/value";
-
 import type { CollectorManifest, CollectorRepository } from "./collector-config.ts";
 import {
   applyEvidenceVersionHistory,
@@ -26,13 +24,6 @@ import {
   type GitHubPullRequest,
 } from "./collector-github.ts";
 import { CollectorNonOpenRequestError } from "./collector-identity.ts";
-import {
-  collectorObserveArgsSchema,
-  collectorOutputArgsSchema,
-  collectorReadArgsSchema,
-  collectorRequestArgsSchema,
-  collectorWaitArgsSchema,
-} from "./collector-tool-schemas.ts";
 import { COLLECTOR_OUTPUT_TOOL } from "./package-contracts/collector-output.ts";
 
 export const COLLECTOR_OBSERVE_TOOL = "ak_collector_observe";
@@ -41,6 +32,8 @@ export const COLLECTOR_REQUEST_TOOL = "ak_collector_request";
 export const COLLECTOR_WAIT_TOOL = "ak_collector_wait";
 /** #676 A: role-decided target bind — business tool, ledger-booked. */
 export const COLLECTOR_BIND_TARGET_TOOL = "ak_collector_bind_target";
+/** #677: opaque handbook write — business tool, ledger-booked. */
+export const COLLECTOR_HANDBOOK_WRITE_TOOL = "ak_collector_handbook_write";
 export { COLLECTOR_OUTPUT_TOOL };
 
 export const COLLECTOR_OPERATIONAL_TOOLS = [
@@ -49,6 +42,7 @@ export const COLLECTOR_OPERATIONAL_TOOLS = [
   COLLECTOR_READ_TOOL,
   COLLECTOR_REQUEST_TOOL,
   COLLECTOR_WAIT_TOOL,
+  COLLECTOR_HANDBOOK_WRITE_TOOL,
 ] as const;
 
 /**
@@ -57,7 +51,7 @@ export const COLLECTOR_OPERATIONAL_TOOLS = [
  * (evidenceId + htmlUrl) and is preserved in the volume, never unconditionally
  * transcribed into model context or receipt.
  */
-export const COLLECTOR_OBSERVE_BODY_HEAD_BYTES = 512;
+const COLLECTOR_OBSERVE_BODY_HEAD_BYTES = 512;
 
 function utf8SafePrefix(text: string, maxBytes: number): string {
   if (Buffer.byteLength(text, "utf8") <= maxBytes) return text;
@@ -108,7 +102,7 @@ export type ObserveModelView = {
  * (collector-side; provider-visible observe details carry only this bounded
  * projection), never unconditionally transcribed into model context.
  */
-export function projectObserveContextView(modelView: ObserveModelView): ObserveModelView {
+function projectObserveContextView(modelView: ObserveModelView): ObserveModelView {
   return {
     ...modelView,
     evidence: modelView.evidence.map((entry) =>
@@ -121,9 +115,9 @@ export function projectObserveContextView(modelView: ObserveModelView): ObserveM
 export type CollectorOperationalTool = (typeof COLLECTOR_OPERATIONAL_TOOLS)[number];
 
 export const COLLECTOR_ACTIVATION_ENTRY_TYPE = "ak-collector-activation" as const;
-export const COLLECTOR_SNAPSHOT_ENTRY_TYPE = "ak-collector-snapshot" as const;
-export const COLLECTOR_REQUEST_ENTRY_TYPE = "ak-collector-request" as const;
-export const COLLECTOR_WAIT_ENTRY_TYPE = "ak-collector-wait" as const;
+const COLLECTOR_SNAPSHOT_ENTRY_TYPE = "ak-collector-snapshot" as const;
+const COLLECTOR_REQUEST_ENTRY_TYPE = "ak-collector-request" as const;
+const COLLECTOR_WAIT_ENTRY_TYPE = "ak-collector-wait" as const;
 
 export type CollectorRequestAttempt = {
   attemptId: string;
@@ -217,7 +211,7 @@ export type CollectorLedger = {
   }>;
 
   request(
-    input: { requestId: string; snapshotId: string },
+    input: { requestId: string; snapshotId: string; body?: string },
     transport: CollectorGitHubTransport,
     clock: CollectorClock,
     signal?: AbortSignal,
@@ -243,29 +237,6 @@ function isOperationalTool(name: string): name is CollectorOperationalTool {
   return (COLLECTOR_OPERATIONAL_TOOLS as readonly string[]).includes(name);
 }
 
-/** Shared Check owner for Collector tool argument shapes (schema seam, not batch law). */
-export function collectorToolArgumentsValid(
-  name: string,
-  args: unknown,
-): boolean {
-  // Residual envelope: reject missing/null args before schema check.
-  if (args === undefined || args === null) return false;
-  switch (name) {
-    case COLLECTOR_OBSERVE_TOOL:
-      return Value.Check(collectorObserveArgsSchema, args);
-    case COLLECTOR_READ_TOOL:
-      return Value.Check(collectorReadArgsSchema, args);
-    case COLLECTOR_REQUEST_TOOL:
-      return Value.Check(collectorRequestArgsSchema, args);
-    case COLLECTOR_WAIT_TOOL:
-      return Value.Check(collectorWaitArgsSchema, args);
-    case COLLECTOR_OUTPUT_TOOL:
-      return Value.Check(collectorOutputArgsSchema, args);
-    default:
-      return false;
-  }
-}
-
 export function createCollectorLedger(
   config: CollectorConfigState,
   options?: CollectorLedgerOptions,
@@ -335,9 +306,7 @@ export function createCollectorLedger(
 
   const requireBoundPr = (): number => {
     if (config.prNumber === undefined) {
-      throw new Error(
-        "Collector PR target is unbound; call ak_collector_bind_target with the role-decided issue/PR or pass --pr",
-      );
+      throw new Error("通进司 PR 目标未绑定");
     }
     return config.prNumber;
   };
@@ -675,11 +644,11 @@ export function createCollectorLedger(
         throw new Error("通进司已产出输出候选，本局不再受理目标绑定");
       }
       if (!Number.isSafeInteger(prNumber) || prNumber < 1) {
-        throw new Error("Collector bind target requires a positive safe-integer PR number");
+        throw new Error("通进司绑定目标要求正安全整数 PR 号");
       }
       if (config.prNumber !== undefined && config.prNumber !== prNumber) {
         throw new Error(
-          `Collector target already bound to PR ${config.prNumber}; cannot rebind to ${prNumber}`,
+          `通进司目标已绑定 PR ${config.prNumber}，不可改绑为 ${prNumber}`,
         );
       }
       config.prNumber = prNumber;
@@ -915,9 +884,25 @@ export function createCollectorLedger(
         throw latchFatal("通进司请求时存在未恢复的传输失败");
       }
 
-      const request = config.manifest.requests.find((item) => item.id === input.requestId);
-      if (request === undefined) {
-        throw new Error(`未知通进司 requestId "${input.requestId}"`);
+      // Sole request identity seam: trim once, then use everywhere (lookup/marker/attemptKey).
+      // Schema pattern rejects leading/trailing whitespace at host; trim still collapses
+      // direct ledger callers and any residual padding into one stable id (#677 D2).
+      if (typeof input.requestId !== "string") {
+        throw new Error("通进司 requestId 须为字符串");
+      }
+      const requestId = input.requestId.trim();
+      if (requestId.length === 0) {
+        throw new Error("通进司 requestId 须为非空（首尾空白不构成身份）");
+      }
+      const configured = config.manifest.requests.find((item) => item.id === requestId);
+      const roleBody = typeof input.body === "string" ? input.body : undefined;
+      if (configured === undefined) {
+        // #677: role-decided trigger from handbook/field activity — body required when not in manifest.
+        if (roleBody === undefined || roleBody.trim().length === 0) {
+          throw new Error(
+            `未知通进司 requestId "${requestId}" 且未提供 body；请提交角色判定的请求正文或使用 request-manifest`,
+          );
+        }
       }
 
       const snapshot = snapshots.find((item) => item.snapshotId === input.snapshotId);
@@ -936,10 +921,12 @@ export function createCollectorLedger(
         throw latchFatal("通进司请求不在资格截止前");
       }
 
+      // Caller manifest body wins when requestId is configured; otherwise role body.
+      const configuredBody = configured?.requestBody ?? roleBody!;
       const { body, marker } = buildCollectorRequestBody({
-        configuredBody: request.requestBody,
+        configuredBody,
         manifestDigest: config.manifest.digest,
-        requestId: request.id,
+        requestId,
         headOid: snapshot.headOid,
       });
       const existingMarker = snapshot.evidenceIds.some((id) => {
@@ -951,7 +938,7 @@ export function createCollectorLedger(
       });
       if (existingMarker) {
         throw new Error(
-          `通进司在此 HEAD 已有同 marker 的已认证请求 "${input.requestId}"`,
+          `通进司在此 HEAD 已有同 marker 的已认证请求 "${requestId}"`,
         );
       }
 
@@ -960,11 +947,11 @@ export function createCollectorLedger(
         config.repository.canonical,
         String(boundPr),
         snapshot.headOid,
-        request.id,
+        requestId,
       ].join("|");
       if (attemptKeys.has(attemptKey)) {
         throw new Error(
-          `通进司进程内请求 "${request.id}" 在 HEAD ${snapshot.headOid} 的 attempt 已用`,
+          `通进司进程内请求 "${requestId}" 在 HEAD ${snapshot.headOid} 的 attempt 已用`,
         );
       }
 
@@ -972,7 +959,7 @@ export function createCollectorLedger(
       const attemptId = sha256Text(`${attemptKey}:${startedAt}`).slice(0, 16);
       const attempt: CollectorRequestAttempt = {
         attemptId,
-        requestId: request.id,
+        requestId,
         observedHead: snapshot.headOid,
         snapshotId: snapshot.snapshotId,
         marker,
@@ -1017,7 +1004,7 @@ export function createCollectorLedger(
         return {
           status: "succeeded",
           attemptId,
-          requestId: request.id,
+          requestId,
           observedHead: snapshot.headOid,
           marker,
           commentEvidenceId: record.evidenceId,
@@ -1032,7 +1019,7 @@ export function createCollectorLedger(
         return {
           status: "ambiguous_loss",
           attemptId,
-          requestId: request.id,
+          requestId,
           observedHead: snapshot.headOid,
           marker,
           diagnostics: result.diagnostics,
@@ -1051,14 +1038,7 @@ export function createCollectorLedger(
       if (activationTime === undefined) {
         throw latchFatal("通进司等待需要激活");
       }
-      if (!Number.isSafeInteger(input.durationMs) || input.durationMs < 1) {
-        throw new Error("通进司等待 durationMs 须为正安全整数");
-      }
-      if (input.durationMs > COLLECTOR_ELIGIBILITY_MS) {
-        throw new Error(
-          `通进司等待 durationMs 至多为 ${COLLECTOR_ELIGIBILITY_MS}`,
-        );
-      }
+      // durationMs shape authority = collectorWaitArgsSchema (host parameters).
       if (pastCutoff(clock)) {
         finalObservationRequired = true;
         throw latchFatal("通进司等待不在资格截止前");