@akagilnc/pi-workflow-roles 0.1.4800 → 0.1.4870

package/dist/public-cli/turn-request.js

@@ -1,4 +1,7 @@
 import { pickEngineAxis } from "../package-resources/engine-material.js";
+import { resolvePackagedMethodSkillPath, } from "../package-resources/method-skill.js";
+import { packagedRoleActivationFlags, packagedRoleMetadata, } from "../packaged-role-registry.js";
+import { parentRunPathFromGatePointerInstruction } from "./run-lifecycle.js";
 export function projectRoleTurnRequest(admitted, roleDetails, options) {
     const cwd = options.cwd ?? admitted.projectRoot ?? admitted.repoRoot ?? admitted.cwd;
     if (cwd === undefined)
@@ -31,3 +34,76 @@ export function projectRoleTurnRequest(admitted, roleDetails, options) {
         ...(options.stationChild === undefined ? {} : { stationChild: options.stationChild }),
     };
 }
+/**
+ * Packaged method this admitted run settles with.
+ * Coder carries one only on apply. Other seats read `settleMethod`.
+ */
+export function packagedSettleSkill(admitted) {
+    const record = packagedRoleMetadata(admitted.role);
+    if (record === undefined)
+        return undefined;
+    if ("applyMethod" in record && record.applyMethod !== undefined) {
+        return "phase" in admitted && admitted.phase === "apply" ? record.applyMethod : undefined;
+    }
+    if ("settleMethod" in record && record.settleMethod !== undefined)
+        return record.settleMethod;
+    return undefined;
+}
+function readAdmittedPath(admitted, path) {
+    let current = admitted;
+    for (const key of path.split(".")) {
+        if (current === null || typeof current !== "object")
+            return undefined;
+        current = current[key];
+    }
+    return current;
+}
+/** Activation object for one admitted run. Field copies come from the composition-root record. */
+function activationForAdmitted(admitted) {
+    if (packagedRoleMetadata(admitted.role) === undefined) {
+        throw new Error(`no turn projection for ${admitted.role}`);
+    }
+    const activation = { role: admitted.role };
+    for (const spec of packagedRoleActivationFlags(admitted.role)) {
+        let value = readAdmittedPath(admitted, spec.from ?? spec.field);
+        if (spec.fallback === "gate-pointer") {
+            const trimmed = typeof value === "string" ? value.trim() : "";
+            value = trimmed !== ""
+                ? trimmed
+                : parentRunPathFromGatePointerInstruction(admitted.instruction);
+            if (typeof value !== "string" || value === "")
+                continue;
+        }
+        if (value === undefined)
+            continue;
+        if (spec.text === true) {
+            if (typeof value !== "number")
+                continue;
+            value = String(value);
+        }
+        activation[spec.field] = value;
+    }
+    return activation;
+}
+/** Skill bindings declared on the composition-root record for this admitted run. */
+function methodBindings(admitted, packageRoot) {
+    const record = packagedRoleMetadata(admitted.role);
+    const names = [];
+    if (record !== undefined && "methodSkills" in record && record.methodSkills !== undefined) {
+        names.push(...record.methodSkills);
+    }
+    const apply = packagedSettleSkill(admitted);
+    if (apply !== undefined && !names.includes(apply))
+        names.push(apply);
+    return names.map((name) => ({
+        kind: "skill",
+        path: resolvePackagedMethodSkillPath(packageRoot, name),
+    }));
+}
+/** Registry activation and method bindings for one admitted public seat. */
+export function admittedSeatTurnDetails(admitted, packageRoot) {
+    return {
+        activation: activationForAdmitted(admitted),
+        methods: methodBindings(admitted, packageRoot),
+    };
+}

package/dist/public-role-summons.js

@@ -236,118 +236,42 @@ async function summonPublicRole(options) {
       ...stderr2 === void 0 ? {} : { stderr: stderr2 }
     };
   }
-  async function runPrepared(parse, run) {
+  async function runPrepared(parse2, run) {
     let parsedArgv;
     const prepared = await prepareSummonEnv(() => {
-      parsedArgv = parse(options.argv);
+      parsedArgv = parse2(options.argv);
     });
     if (!prepared.ok) return { fail: projectPrepareFailure(prepared) };
     return { ok: await run(prepared.env, () => parsedArgv) };
   }
+  const [{ runPublicInstructionSeat, runPublicInstructionSeatResume }, { PUBLIC_ROLE_ARGV }] = await Promise.all([
+    import("./public-cli/instruction-seat-run.js"),
+    import("./public-cli/cli.js")
+  ]);
+  const { packagedRoleMetadata } = await import("./packaged-role-registry.js");
+  const record = packagedRoleMetadata(options.role);
+  const parse = ((args) => PUBLIC_ROLE_ARGV[options.role].parse(args));
+  const resumeRunId = record?.summonResume === true && typeof options.resumeRunId === "string" && options.resumeRunId.trim() !== "" ? options.resumeRunId.trim() : void 0;
   let result;
-  switch (options.role) {
-    case "notary": {
-      const [{ runPublicNotary }, { parseNotaryArgv }] = await Promise.all([
-        import("./public-cli/notary-run.js"),
-        import("./public-cli/invocation.js")
-      ]);
-      const stepped = await runPrepared(parseNotaryArgv, (env, once) => runPublicNotary(options.argv, env, io, once));
-      if ("fail" in stepped) return stepped.fail;
-      result = stepped.ok;
-      break;
-    }
-    case "inspector": {
-      const [{ runPublicInspector }, { parseInspectorArgv }] = await Promise.all([
-        import("./public-cli/inspector-run.js"),
-        import("./public-cli/invocation.js")
-      ]);
-      const stepped = await runPrepared(parseInspectorArgv, (env, once) => runPublicInspector(options.argv, env, io, once));
-      if ("fail" in stepped) return stepped.fail;
-      result = stepped.ok;
-      break;
-    }
-    case "auditor":
-    case "navigator":
-    case "gatekeeper": {
-      const seat2 = options.role;
-      const [{ runPublicInstructionSeat, runPublicInstructionSeatResume }, invocation] = await Promise.all([
-        import("./public-cli/instruction-seat-run.js"),
-        import("./public-cli/invocation.js")
-      ]);
-      const parse = seat2 === "auditor" ? invocation.parseAuditorArgv : seat2 === "navigator" ? invocation.parseNavigatorArgv : invocation.parseGatekeeperArgv;
-      const resumeRunId = typeof options.resumeRunId === "string" && options.resumeRunId.trim() !== "" ? options.resumeRunId.trim() : void 0;
-      if (resumeRunId !== void 0) {
-        const instruction = options.argv[0] ?? "";
-        const stepped = await runPrepared(parse, (env) => runPublicInstructionSeatResume(
-          {
-            runId: resumeRunId,
-            summons: {
-              instruction,
-              instructionEmpty: instruction.trim() === ""
-            }
-          },
-          env,
-          io
-        ));
-        if ("fail" in stepped) return stepped.fail;
-        result = stepped.ok;
-      } else {
-        const stepped = await runPrepared(parse, (env, once) => runPublicInstructionSeat(options.argv, env, io, seat2, once));
-        if ("fail" in stepped) return stepped.fail;
-        result = stepped.ok;
-      }
-      break;
-    }
-    case "judge": {
-      const [{ runPublicJudge }, { parseJudgeArgv }] = await Promise.all([
-        import("./public-cli/judge-run.js"),
-        import("./public-cli/invocation.js")
-      ]);
-      const stepped = await runPrepared(parseJudgeArgv, (env, once) => runPublicJudge(options.argv, env, io, once));
-      if ("fail" in stepped) return stepped.fail;
-      result = stepped.ok;
-      break;
-    }
-    case "doctor": {
-      const [{ runPublicDoctor }, { parseDoctorArgv }] = await Promise.all([
-        import("./public-cli/doctor-run.js"),
-        import("./public-cli/invocation.js")
-      ]);
-      const stepped = await runPrepared(parseDoctorArgv, (env, once) => runPublicDoctor(options.argv, env, io, once));
-      if ("fail" in stepped) return stepped.fail;
-      result = stepped.ok;
-      break;
-    }
-    case "diarist": {
-      const [{ runPublicDiarist }, { parseDiaristArgv }] = await Promise.all([
-        import("./public-cli/diarist-run.js"),
-        import("./public-cli/invocation.js")
-      ]);
-      const stepped = await runPrepared(parseDiaristArgv, (env, once) => runPublicDiarist(options.argv, env, io, once));
-      if ("fail" in stepped) return stepped.fail;
-      result = stepped.ok;
-      break;
-    }
-    case "countersign": {
-      const [{ runPublicCountersign }, { parseCountersignArgv }] = await Promise.all([
-        import("./public-cli/countersign-run.js"),
-        import("./public-cli/invocation.js")
-      ]);
-      const stepped = await runPrepared(parseCountersignArgv, (env, once) => runPublicCountersign(options.argv, env, io, once));
-      if ("fail" in stepped) return stepped.fail;
-      result = stepped.ok;
-      break;
-    }
-    case "reviewer": {
-      const [{ runPublicReviewer }, { parseReviewerArgv }] = await Promise.all([
-        import("./public-cli/reviewer-run.js"),
-        import("./public-cli/invocation.js")
-      ]);
-      const stepped = await runPrepared(parseReviewerArgv, (env, once) => runPublicReviewer(options.argv, env, io, once));
-      if ("fail" in stepped) return stepped.fail;
-      result = stepped.ok;
-      break;
-    }
+  if (resumeRunId !== void 0) {
+    const instruction = options.argv[0] ?? "";
+    const stepped = await runPrepared(parse, (env) => runPublicInstructionSeatResume(
+      {
+        runId: resumeRunId,
+        summons: {
+          instruction,
+          instructionEmpty: instruction.trim() === ""
+        }
+      },
+      env,
+      io
+    ));
+    if ("fail" in stepped) return stepped.fail;
+    result = stepped.ok;
+  } else {
+    const stepped = await runPrepared(parse, (env, once) => runPublicInstructionSeat(options.argv, env, io, options.role, once));
+    if ("fail" in stepped) return stepped.fail;
+    result = stepped.ok;
   }
   const stderr = captured?.stderrText();
   const runDirectory = result.admitted?.runDirectory;
@@ -672,16 +596,18 @@ async function summonGateOfficer(options) {
     ...options.hostAdapters === void 0 ? {} : { hostAdapters: options.hostAdapters },
     ...options.createRunId === void 0 ? {} : { createRunId: options.createRunId }
   };
-  if (options.officer === "notary") {
+  const { packagedGateSummon } = await import("./packaged-role-registry.js");
+  const gateSummon = packagedGateSummon(options.officer);
+  if (gateSummon === "source-run") {
     return summonPublicRole({
-      role: "notary",
+      role: options.officer,
       argv: ["--source-run", options.sourceRunDirectory, "--project", options.cwd],
       ...common
     });
   }
-  if (options.officer === "auditor") {
+  if (gateSummon === "subject-source") {
     return summonPublicRole({
-      role: "auditor",
+      role: options.officer,
       argv: [
         "--subject",
         "judge",
@@ -692,7 +618,7 @@ async function summonGateOfficer(options) {
       ...common
     });
   }
-  if (options.officer === "countersign") {
+  if (gateSummon === "parent-instruction") {
     const { runIdFromRunDirectory } = await import("./run-terminal-artifacts.js");
     const correlationId = runIdFromRunDirectory(options.sourceRunDirectory);
     if (correlationId === void 0 || correlationId.trim() === "") {
@@ -704,7 +630,7 @@ async function summonGateOfficer(options) {
     const parentTicket = await readBoardTicketNumber(options.sourceRunDirectory);
     const instruction = options.reask ?? gateReviewInstruction ?? "";
     return summonPublicRole({
-      role: "countersign",
+      role: options.officer,
       argv: ["--project", options.cwd, "--", instruction],
       ...common,
       correlationId,
@@ -713,7 +639,7 @@ async function summonGateOfficer(options) {
     });
   }
   return summonPublicRole({
-    role: "inspector",
+    role: options.officer,
     argv: [`\u5377\u5B97\u6307\u9488\uFF1A${options.sourceRunDirectory}`],
     ...common
   });

package/dist/role-activation-flags.js

@@ -1,54 +1,32 @@
-import { packagedRoleInputFlag, packagedRolePhaseFlag } from "./packaged-role-registry.js";
-/** Project closed RoleTurnActivation onto the shared host flag map. */
+import { packagedRoleActivationFlags } from "./packaged-role-registry.js";
+/**
+ * Project closed RoleTurnActivation onto the shared host flag map.
+ * Seat flag names and fields come from the composition-root record.
+ * countersign ticketNumber stays off this map (no private transport flag, #632).
+ */
 export function projectActivationFlags(request) {
     const activation = request.activation;
     const flags = new Map([["ak-role", activation.role]]);
     if (request.stationChild === true) {
         flags.set("ak-station-child", true);
     }
-    const inputFlag = packagedRoleInputFlag(activation.role);
-    const phaseFlag = packagedRolePhaseFlag(activation.role);
-    if ("phase" in activation && phaseFlag !== undefined)
-        flags.set(phaseFlag, activation.phase);
-    if (inputFlag !== undefined) {
-        const path = "taskPath" in activation ? activation.taskPath
-            : "packetPath" in activation ? activation.packetPath
-                : "casePath" in activation ? activation.casePath
-                    : "inputPath" in activation ? activation.inputPath
-                        : "sourceRun" in activation ? activation.sourceRun
-                            : undefined;
-        if (path !== undefined)
-            flags.set(inputFlag, path);
-    }
-    if (activation.role === "fixer" && activation.prerequisitesPath !== undefined) {
-        flags.set("ak-fixer-prerequisites", activation.prerequisitesPath);
-    }
-    if (activation.role === "reviewer") {
-        flags.set("ak-review-base", activation.baseRevision);
-        flags.set("ak-review-lens", activation.lens);
-        flags.set("ak-review-authority-refs", JSON.stringify(activation.authorityRefs));
-        if (activation.ticketNumber !== undefined) {
-            flags.set("ak-review-ticket-number", String(activation.ticketNumber));
-        }
-    }
-    // countersign ticketNumber stays on activation/admission/invocation only —
-    // no private transport flag (inner-gate material path deleted in #632).
-    if (activation.role === "notary" && activation.ticketNumber !== undefined) {
-        flags.set("ak-notary-ticket-number", String(activation.ticketNumber));
-    }
-    if (activation.role === "gleaner-left") {
-        flags.set("ak-gleaner-left-base", activation.baseRevision);
-    }
-    if (activation.role === "collector") {
-        flags.set("ak-collector-repo", activation.repo);
-        // #676 D1: pr optional at admission; omit flag when role binds from materials.
-        if (activation.pr !== undefined)
-            flags.set("ak-collector-pr", activation.pr);
-        if (activation.requestManifestPath !== undefined) {
-            flags.set("ak-collector-request-manifest", activation.requestManifestPath);
-        }
-        if (activation.waitMs !== undefined)
-            flags.set("ak-collector-wait-ms", activation.waitMs);
+    const values = activation;
+    for (const spec of packagedRoleActivationFlags(activation.role)) {
+        if (spec.flag === undefined)
+            continue;
+        const text = activationFlagText(values[spec.field]);
+        if (text !== undefined)
+            flags.set(spec.flag, text);
     }
     return flags;
 }
+function activationFlagText(value) {
+    if (typeof value === "string")
+        return value;
+    if (typeof value === "number")
+        return String(value);
+    if (Array.isArray(value) && value.every((item) => typeof item === "string")) {
+        return JSON.stringify(value);
+    }
+    return undefined;
+}

package/dist/secretariat-contracts.js

@@ -1,10 +1,8 @@
 const SECRETARIAT_OUTPUT_TOOL_NAME = "ak_secretariat_output";
 const SECRETARIAT_SUMMON_COUNTERSIGN_TOOL_NAME = "ak_secretariat_summon_countersign";
-const SECRETARIAT_ACCEPTED_TEXT = "\u4E2D\u4E66\u7701\u56DE\u6267\u5DF2\u63A5\u53D7";
 const SECRETARIAT_GATE_OFFICER_ENTRY_TYPE = "ak-secretariat-gate-officer";
 const SECRETARIAT_COUNTERSIGN_TERMINAL_FACT_KEY = "countersignTerminal";
 export {
-  SECRETARIAT_ACCEPTED_TEXT,
   SECRETARIAT_COUNTERSIGN_TERMINAL_FACT_KEY,
   SECRETARIAT_GATE_OFFICER_ENTRY_TYPE,
   SECRETARIAT_OUTPUT_TOOL_NAME,

package/dist/session-identity.js

@@ -46,13 +46,22 @@ export function resolveHostAwareSessionAvailability(host, principalAuthority) {
     if (host === undefined || host === DEFAULT_ROLE_TURN_HOST) {
         return (principal) => principalAuthority.isAvailable(principal);
     }
+    // Unregistered names have no binding file: absence, not a guess at pi.
+    return async (principal) => (await readStoredHostSessionId(host, principalAuthority, principal)) !== undefined;
+}
+/**
+ * Native session/thread id already stored for this principal on `host`.
+ * Public explicit resume reads it once and hands it to the host adapter.
+ * Pi has no separate binding file. A missing file is absence, not the package run id.
+ */
+export async function readStoredHostSessionId(host, authority, principal) {
+    if (host === undefined || host === DEFAULT_ROLE_TURN_HOST)
+        return undefined;
     const description = lookupHostDescription(host) ?? lookupHeadlessHostDescription(host);
-    if (description === undefined) {
-        // Host selection already fails closed on an unregistered name (#510) long
-        // before a turn dispatches; mirror the same fail-closed default here
-        // rather than guessing at pi's binding for a name pi does not own.
-        return () => Promise.resolve(false);
-    }
-    const sessionIdentity = createSessionIdentityAuthority(principalAuthority, description.sessionBindingFile);
-    return async (principal) => (await sessionIdentity.load(principal)) !== undefined;
+    if (description === undefined)
+        return undefined;
+    const sessionId = await createSessionIdentityAuthority(authority, description.sessionBindingFile).load(principal);
+    if (sessionId === undefined || sessionId === "")
+        return undefined;
+    return sessionId;
 }

package/dist/session-opening-materials.js

@@ -5,7 +5,8 @@ import { fileURLToPath, pathToFileURL } from "node:url";
 import {
   INSPECTOR_SESSION_MATERIALS,
   NOTARY_SESSION_MATERIALS,
-  PUBLIC_ROLE_RECORDS
+  PUBLIC_ROLE_RECORDS,
+  packagedRoleSessionMaterials
 } from "./packaged-role-registry.js";
 function resolvePackageRootDir(moduleUrl = import.meta.url) {
   let dir = dirname(fileURLToPath(moduleUrl));
@@ -51,7 +52,7 @@ function loadMainRoleReferenceMaterials(role) {
 }
 const GATEKEEPER_SESSION_MATERIALS = {
   // #639: single authority — the public gatekeeper record owns the province list.
-  gatekeeper: PUBLIC_ROLE_RECORDS.find((entry) => entry.role === "gatekeeper").sessionMaterials,
+  gatekeeper: packagedRoleSessionMaterials("gatekeeper"),
   inspector: INSPECTOR_SESSION_MATERIALS,
   notary: NOTARY_SESSION_MATERIALS
 };

package/extensions/role-runtime.ts

@@ -1,8 +1,5 @@
-import { readFile } from "node:fs/promises";
 import { fileURLToPath } from "node:url";
 
-import { loadDoctorCase } from "../src/doctor-evidence.ts";
-import { loadNotarySourceRunLocator } from "../src/notary-source-run.ts";
 import { createPiRoleRuntimeExtension } from "../src/pi/adapter.ts";
 
 import {
@@ -14,35 +11,21 @@ import {
 } from "@earendil-works/pi-coding-agent";
 import type { Message } from "@earendil-works/pi-ai";
 
-import { createGhCollectorGitHubTransport } from "../src/collector-github.ts";
-import { createPiDoctorAuditor } from "../src/doctor-auditor.ts";
 import {
-  createNativeNavigatorSessionFactory,
-  createNavigatorAttendance,
   registerNavigatorModelCommand,
   resolveNavigatorAuthorityMaterial,
   type NavigatorTargetRole,
 } from "../src/navigator-attendance.ts";
 import { loadNavigatorWorkContext as loadHostNeutralNavigatorWorkContext } from "../src/navigator-work-context.ts";
 export { resolveNavigatorAuthorityMaterial };
-import { loadCanonicalSkillBinding as loadHomeCanonicalSkillBinding } from "../src/canonical-skill-binding.ts";
-import { loadPackagedCanonicalSkillBinding } from "../src/package-resources/method-skill-binding.ts";
 import { JUDGE_OUTPUT_TOOL_NAME } from "../src/package-contracts/judge-output.ts";
 import { readOAuthKeepaliveProviders } from "../src/oauth-keepalive.ts";
 import {
   formatNavigatorRoleHelp,
   type RoleRuntimeDependencies,
 } from "../src/role-runtime.ts";
-import { loadAuditorSoulFromSubjectInput } from "../src/auditor-soul.ts";
-import { loadPackagedRoleReferenceMaterials } from "../src/role-runtime-dependencies.ts";
-import {
-  loadGatekeeperSessionMaterials,
-  loadMainRoleSessionMaterials,
-} from "../src/session-opening-materials.ts";
-const extensionPath = fileURLToPath(import.meta.url);
+import { createRoleRuntimeDependencies } from "../src/role-runtime-dependencies.ts";
 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;
@@ -110,68 +93,11 @@ export async function loadNavigatorWorkContext(
   });
 }
 
-export function createPiRoleRuntimeDependencies(pi: ExtensionAPI): RoleRuntimeDependencies {
-  const navigatorSessionFactory = createNativeNavigatorSessionFactory();
-  return {
-    packageRoot,
-    loadRoleReferenceMaterials: loadPackagedRoleReferenceMaterials,
-    loadJudgeSoul: () => loadMainRoleSessionMaterials("judge"),
-    loadFixerSoul: () => loadMainRoleSessionMaterials("fixer"),
-    loadFixPacket: (path) => readFile(path, "utf8"),
-    loadCoderSoul: () => loadMainRoleSessionMaterials("coder"),
-    loadCoderTask: (path) => readFile(path, "utf8"),
-    loadReviewerSoul: () => loadMainRoleSessionMaterials("reviewer"),
-    loadCollectorSoul: () => loadMainRoleSessionMaterials("collector"),
-    loadCollectorHandbookSeed: () => readFile(collectorHandbookSeedPath, "utf8"),
-    createCollectorTransport: () => createGhCollectorGitHubTransport(),
-    loadDoctorSoul: () => loadMainRoleSessionMaterials("doctor"),
-    loadDoctorCase,
-    // #590: auditors / navigator / reviewer consume HostContext directly (no Pi WeakMap recover).
-    auditDoctorCompliance: (options) => createPiDoctorAuditor()(options),
-    loadNotarySoul: () => loadMainRoleSessionMaterials("notary"),
-    loadCountersignSoul: () => loadMainRoleSessionMaterials("countersign"),
-    loadGleanerLeftSoul: () => loadMainRoleSessionMaterials("gleaner-left"),
-    loadDiaristSoul: () => loadMainRoleSessionMaterials("diarist"),
-    loadSecretariatSoul: () => loadMainRoleSessionMaterials("secretariat"),
-    loadInspectorSoul: () => loadMainRoleSessionMaterials("inspector"),
-    loadGatekeeperSoul: () => loadGatekeeperSessionMaterials("gatekeeper"),
-    loadNavigatorSoul: () => loadMainRoleSessionMaterials("navigator"),
-    loadAuditorSoul: () => loadAuditorSoulFromSubjectInput(),
-    loadNotarySourceRun: loadNotarySourceRunLocator,
-    loadNavigatorWorkContext: (options) => loadNavigatorWorkContext(pi, options),
-    createNavigatorAttendance: (options) => {
-      return createNavigatorAttendance({
-        context: options.context,
-        role: options.role,
-        phase: options.phase,
-        subjectKey: options.subjectKey,
-        subject: options.subject,
-        authority: options.authority,
-        invocationId: options.invocationId,
-        loadSoul: () => loadMainRoleSessionMaterials("navigator"),
-        loadRoutePlaybook: () => readFile(navigatorRoutePlaybookPath, "utf8"),
-        // In-process help: subprocess pi --help is reserved for cold-install proofs.
-        // N child pi processes cannot fit the accepted 3s post-role grace under CI load.
-        loadRoleHelp: async (role) => formatNavigatorRoleHelp(role),
-        createSession: navigatorSessionFactory,
-        contextError: options.contextError,
-        onEvent: options.onEvent,
-      });
-    },
-    loadMergerSoul: () => loadMainRoleSessionMaterials("merger"),
-    loadMergerInput: async (path) => JSON.parse(await readFile(path, "utf8")),
-    async loadCanonicalSkillBinding(name) {
-      // Coder TDD (#109) and Reviewer ak-cross-m-review (#917) are package-owned.
-      // Optional Fixer diagnosing-bugs is available via --skill without this binding.
-      if (name === "tdd") {
-        return loadPackagedCanonicalSkillBinding(packageRoot, "tdd");
-      }
-      if (name === "ak-cross-m-review") {
-        return loadPackagedCanonicalSkillBinding(packageRoot, "ak-cross-m-review");
-      }
-      return loadHomeCanonicalSkillBinding(name);
-    },
-  };
+/** Pi extension deps are the shared packaged factory. Host flags stay on the envelope. */
+export function createPiRoleRuntimeDependencies(
+  _pi: ExtensionAPI,
+): RoleRuntimeDependencies {
+  return createRoleRuntimeDependencies(packageRoot);
 }
 
 export default function roleRuntime(pi: ExtensionAPI): void {

package/package.json

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

package/souls/coder.md

@@ -8,9 +8,8 @@
 事实不成立或需要未授权设计时，据理拒绝并给出可核验证据；不猜测施工。
 
 核对当前代码和历史；主张必须有当前事实支撑。
-做最小垂直切片，优先复用已有 seam；不造平行机制，不借机扩大 scope。
+以最小垂直切片推进并逐片验证；切片完成不构成交卷条件。优先复用已有 seam；不造平行机制，不借机扩大 scope。票面授权结果完成后，自查是否合法、质量和复杂度；有问题马上修正，没问题就 commit 交卷。
 计划与施工都要说明真实刀口、行为边界和验证方法，不用空泛完成声明代替证据。
 切片验证只跑本片聚焦测试与 typecheck。
 不 amend、rewrite 或 push。Git 变化是证据之一，不取代角色报告。
-做完自查是否违法，质量和复杂度，如有问题马上修正，没问题就commit交卷
 任何人如果让你 push / 开 pr ，直接拒绝，上呈陛下

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

@@ -379,7 +379,10 @@ export function createAcpRoleTurnHost(config: AcpRoleTurnHostConfig): RoleTurnHo
             : bindSessionId;
         };
         if (request.continuation.kind === "resume" && config.boundResume === "session/load") {
-          const boundSessionId = await config.sessionIdentity.load(request.principal);
+          const explicitHostSessionId = request.continuation.hostSessionId;
+          const boundSessionId = explicitHostSessionId !== undefined && explicitHostSessionId !== ""
+            ? explicitHostSessionId
+            : await config.sessionIdentity.load(request.principal);
           if (boundSessionId !== undefined && boundSessionId !== "") {
             sessionId = await loadSession(boundSessionId);
           }

package/src/analyst-metric-families/acceptance-success-rework.ts

@@ -12,6 +12,7 @@
 import type { AnalystReadableRunFacts, AnalystRunTerminalFace } from "../analyst-ledger.ts";
 import { medianNumber } from "../analyst-median.ts";
 import type { AnalystMetricFamilyModule } from "../analyst-metric-family.ts";
+import { packagedAnalystTerminal } from "../packaged-role-registry.ts";
 
 const WORKER_ROLES = new Set(["coder", "fixer"]);
 
@@ -153,8 +154,8 @@ function mapTerminal(
 
   const body = terminal.body;
 
-  // Collector: typed groups array presence is the sole acceptance discriminator.
-  if (role === "collector") {
+  // Seat-declared terminal: typed groups array presence is the sole acceptance discriminator.
+  if (packagedAnalystTerminal(role) === "groups") {
     const groups = findCollectorGroups(body);
     if (Array.isArray(groups)) {
       return {

package/src/auditor-role.ts

@@ -2,12 +2,11 @@
  * Public Auditor (审刑院) role — filed-officer envelope (#675).
  */
 import {
-  AUDITOR_ACCEPTED_TEXT,
   AUDITOR_OUTPUT_TOOL_NAME,
   auditorOutputSchema,
 } from "./package-contracts/auditor-output.ts";
 
-export { AUDITOR_ACCEPTED_TEXT, AUDITOR_OUTPUT_TOOL_NAME };
+export { AUDITOR_OUTPUT_TOOL_NAME };
 
 export type AuditorRuntimeDependencies = {
   loadSoul(): Promise<string>;

package/src/auditor-soul.ts

@@ -54,7 +54,7 @@ export const AUDITOR_SESSION_MATERIALS = {
 >;
 
 export function isAuditorSoulRole(value: unknown): value is AuditorSoulRole {
-  return value === "judge" || value === "doctor";
+  return typeof value === "string" && Object.hasOwn(AUDITOR_SESSION_MATERIALS, value);
 }
 
 /** Resolve audited subject from explicit value or the subject-input env. */

package/src/collector-role.ts

@@ -55,7 +55,7 @@ import {
   collectorRequestArgsSchema,
   collectorWaitArgsSchema,
 } from "./collector-tool-schemas.ts";
-import { COLLECTOR_ACCEPTED_TEXT } from "./package-contracts/collector-output.ts";
+import { packagedRoleAcceptedText } from "./packaged-role-registry.ts";
 import { CorrectableSubmissionError, isCorrectableExecuteError } from "./submission-correctable-error.ts";
 import { CollectorUnknownEvidenceError } from "./collector-identity.ts";
 
@@ -666,7 +666,7 @@ export function createCollectorRoleRuntime(
             return {
               content: [{
                 type: "text" as const,
-                text: COLLECTOR_ACCEPTED_TEXT,
+                text: packagedRoleAcceptedText("collector"),
               }],
               details: params,
               terminate: true as const,

package/src/countersign-contracts.ts

@@ -6,7 +6,6 @@
  */
 
 export const COUNTERSIGN_OUTPUT_TOOL_NAME = "ak_countersign_output";
-export const COUNTERSIGN_ACCEPTED_TEXT = "给事中回执已接受";
 
 export type CountersignVerdict =
   | { countersignStatus: "converged"; note?: string; evidence?: unknown }