@akagilnc/pi-workflow-roles 0.1.5015 → 0.1.5063

package/src/public-cli/run-lifecycle.ts

@@ -1127,11 +1127,10 @@ async function runHasFormedSessionPrincipal(runDirectory: string): Promise<boole
 /**
  * Locate the latest retained run for one seat under a book (#637 / #747).
  * Same walk surface as findRunDirectoryById (listBookRunDirectories). Match by
- * parent run path (officer / gate seats, #747 / #987). A typed ticket number
- * does not select a run. runId is UUIDv7 — lexicographic max is latest among
- * runs that formed a session principal. No parallel index. Public ticket-number
- * selection of a prior run was deleted (#987 Result 7); callers use explicit
- * `ak-role resume <runId>`.
+ * parent run path (officer / gate seats, #747 / #987), narrowed to a supplied
+ * typed ticket for same-ticket gate re-summons. runId is UUIDv7 — lexicographic
+ * max is latest among runs that formed a session principal. No parallel index;
+ * callers can use explicit `ak-role resume <runId>`.
  * Only a truly missing book directory means no history; damage/permission errors propagate.
  */
 export async function findLatestRunIdForSeatTicket(input: {
@@ -1148,12 +1147,8 @@ export async function findLatestRunIdForSeatTicket(input: {
   const bookDir = activationBookDirectory(ledgerHome, input.bookKey);
   let runDirectories: string[];
   try {
-    // A typed ticket narrows the existing walk surface to that ticket's runs;
-    // same-parent matching below still selects the actual prior leg.
     runDirectories = await listBookRunDirectories(
-      input.ticketNumber === undefined
-        ? bookDir
-        : join(bookDir, String(input.ticketNumber)),
+      input.ticketNumber === undefined ? bookDir : join(bookDir, String(input.ticketNumber)),
     );
   } catch (error) {
     if (errorCodeOf(error) === "ENOENT") return undefined;
@@ -1167,7 +1162,9 @@ export async function findLatestRunIdForSeatTicket(input: {
     const runId = entry.slice(0, entry.length - suffix.length);
     if (runId.length === 0) continue;
     const parentPath = await readRunParentPath(runDirectory);
-    if (parentPath !== input.parentRunPath) continue;
+    // The parent's ticket can change after this child was admitted. Within one
+    // book the run leaf is its stable identity; directory placement is not.
+    if (parentPath !== input.parentRunPath && basename(parentPath ?? "") !== basename(input.parentRunPath)) continue;
     // Durable fact: never resume-select a provisional that never formed principal.
     if (!(await runHasFormedSessionPrincipal(runDirectory))) continue;
     if (best === undefined || runId > best) best = runId;
@@ -1328,6 +1325,7 @@ async function loadResumableRunRecord(
   home: string,
   runId: string,
   authority: DurablePrincipalAuthority,
+  allowUnformedAdmitted = false,
 ): Promise<{
   readonly run: RoleRunRecord;
   readonly principal: DurablePrincipal;
@@ -1350,7 +1348,7 @@ async function loadResumableRunRecord(
     throw new CliUsageError(`unknown role run id: ${runId}`);
   }
   const { run, principal } = materialized;
-  if (!(await isDurablePrincipalAvailable(principal, authority))) {
+  if (!(allowUnformedAdmitted && run.state === "admitted") && !(await isDurablePrincipalAvailable(principal, authority))) {
     throw new CliUsageError(
       `role run Pi session principal is unavailable: ${runId}`,
     );
@@ -1757,8 +1755,9 @@ export async function loadResumablePublicRole(
   home: string,
   runId: string,
   authority: DurablePrincipalAuthority,
+  allowUnformedAdmitted = false,
 ): Promise<LoadedResumablePublicRole> {
-  const loaded = await loadResumableRunRecord(home, runId, authority);
+  const loaded = await loadResumableRunRecord(home, runId, authority, allowUnformedAdmitted);
   return seatLoadedResult(loaded, admitResumedRole(loaded));
 }
 

package/src/public-cli/seat-ticket-binding.ts

@@ -1,10 +1,9 @@
 /**
  * Shared same-parent resume seam for gate / officer seats (#635 / #637 / #747 / #987).
  *
- * Lookup key is parent run path only (#747 / #987 Result 7). Public seats no
- * longer select a prior run by ticket number — callers use explicit
- * `ak-role resume <runId>`. Officer and gate countersign same-parent re-summons
- * keep this seam. Lookup/resume failures propagate (失败诚实) — never wash into
+ * Lookup uses the parent run path; a typed ticket narrows same-ticket gate
+ * re-summons, never creates an issue identity. Callers can also use explicit
+ * `ak-role resume <runId>`. Lookup/resume failures propagate (失败诚实) — never wash into
  * a fresh mint. Returns undefined when the caller declared an explicit fresh
  * summons (`ak-role new`) or when no prior run exists; both mint new.
  * freshSummons is required so no seat can drift back into its own skip branch.
@@ -18,7 +17,7 @@ import {
 
 /**
  * Sole same-seat → resume decision by parent run path (#637 / #724 / #747 / #987).
- * A typed ticket number is a bind key, not a resume key. When a prior run is
+ * A supplied typed ticket only narrows the prior-run search. When a prior run is
  * found, resume carries this summons' materials. Lookup/resume failures
  * propagate (失败诚实) — never wash into a fresh mint. Returns undefined when
  * the caller declared an explicit fresh summons (`ak-role new`), the parent
@@ -45,9 +44,7 @@ export async function tryResumeSameTicketSeatRun<T>(input: {
     bookKey: resolveBookKeyFromGit(input.projectRoot),
     role: input.role,
     parentRunPath: input.parentRunPath,
-    ...(input.ticketNumber === undefined
-      ? {}
-      : { ticketNumber: input.ticketNumber }),
+    ...(input.ticketNumber === undefined ? {} : { ticketNumber: input.ticketNumber }),
   });
   if (previousRunId === undefined) return undefined;
   return await input.resume(previousRunId, input.summons);

package/src/public-cli/settlement.ts

@@ -92,14 +92,6 @@ import {
   SECRETARIAT_COUNTERSIGN_TERMINAL_FACT_KEY,
   SECRETARIAT_GATE_OFFICER_ENTRY_TYPE,
 } from "../secretariat-contracts.ts";
-import {
-  observePackagedMethodSkillInvocation,
-  resolvePackagedMethodSkillPath,
-  type ObservedPackagedMethodSkillInvocation,
-  type PackagedMethodSkillMaterial,
-  type PackagedMethodSkillName,
-  type PackagedMethodSkillProvenance,
-} from "../package-resources/method-skill.ts";
 import {
   classifyPackagedRoleTerminalResult,
   findLatestDurablePackagedRoleTerminal,
@@ -2482,21 +2474,14 @@ async function publishAcceptedTerminalArtifacts(
   ];
 }
 
-type MethodPublicationOptions = {
-  readonly methodProvenance?: PackagedMethodSkillProvenance;
-  readonly methodSkillPath?: string;
-  readonly methodSkillConfiguredPath?: string;
-};
-
 /** Publish one accepted seat's report/evidence face from the shared leaf table. */
 export async function publishSeatAcceptedArtifacts(
   admitted: AdmittedRoleInvocation,
   roleOutcome: TerminalRoleOutcome,
   coordinates: DurablePrincipalCoordinates,
   entries: readonly SessionEntry[] = [],
-  options: MethodPublicationOptions = {},
 ): Promise<TerminalArtifactRef[]> {
-  return publishDeclaredSeatArtifacts(admitted, roleOutcome, coordinates, entries, options);
+  return publishDeclaredSeatArtifacts(admitted, roleOutcome, coordinates, entries);
 }
 
 /**
@@ -2623,13 +2608,12 @@ async function settleSealedAcceptedOrToolResidual(
 /**
  * One settlement for every registered seat. The registry `settlement` leaf
  * picks sealed ledger, sealed-or-residual, or the accepted-tool scan.
- * Seat evidence and the observed-method tail stay on the artifact face.
+ * Seat evidence stays on the artifact face.
  */
 async function settleSeat(
   admitted: AdmittedRoleInvocation,
   authority: DurablePrincipalAuthority,
   scope: SettlementCourtScope | undefined,
-  options: MethodPublicationOptions,
 ): Promise<TerminalResult | undefined> {
   const record = packagedRoleMetadata(admitted.role);
   if (record === undefined) {
@@ -2642,7 +2626,7 @@ async function settleSeat(
     roleOutcome: Extract<TerminalRoleOutcome, { kind: "accepted" | "audit_escalation" }>,
     coordinates: DurablePrincipalCoordinates,
     entries: readonly SessionEntry[],
-  ) => publishDeclaredSeatArtifacts(admitted, roleOutcome, coordinates, entries, options);
+  ) => publishDeclaredSeatArtifacts(admitted, roleOutcome, coordinates, entries);
   if (record.settlement === "residual") {
     return settleSealedAcceptedOrToolResidual(
       admitted,
@@ -2669,10 +2653,9 @@ async function settleSeat(
 export async function trySettleSeatTerminalResult(
   admitted: AdmittedRoleInvocation,
   authority: DurablePrincipalAuthority,
-  options: MethodPublicationOptions = {},
   scope?: SettlementCourtScope,
 ): Promise<TerminalResult | undefined> {
-  return settleSeat(admitted, authority, scope, options);
+  return settleSeat(admitted, authority, scope);
 }
 
 /**
@@ -2682,10 +2665,9 @@ export async function trySettleSeatTerminalResult(
 export async function settleSeatTerminalResult(
   admitted: AdmittedRoleInvocation,
   authority: DurablePrincipalAuthority,
-  options: MethodPublicationOptions = {},
   scope?: SettlementCourtScope,
 ): Promise<TerminalResult> {
-  const settled = await settleSeat(admitted, authority, scope, options);
+  const settled = await settleSeat(admitted, authority, scope);
   if (settled === undefined) {
     const label = admitted.role.charAt(0).toUpperCase() + admitted.role.slice(1);
     throw new Error(
@@ -2695,71 +2677,6 @@ export async function settleSeatTerminalResult(
   return settled;
 }
 
-function sessionMessageText(message: SessionMessage | undefined): string {
-  if (message === undefined) return "";
-  if (typeof message.content === "string") return message.content;
-  if (!Array.isArray(message.content)) return "";
-  const parts: string[] = [];
-  for (const part of message.content) {
-    if (
-      typeof part === "object" &&
-      part !== null &&
-      !Array.isArray(part) &&
-      (part as { type?: unknown }).type === "text" &&
-      typeof (part as { text?: unknown }).text === "string"
-    ) {
-      parts.push((part as { text: string }).text);
-    }
-  }
-  return parts.join("\n");
-}
-
-/**
- * Locations a loaded packaged method may be observed at.
- * The name is the loaded material; the configured path is the package copy.
- */
-export function observedMethodSkillOptions(
-  packageRoot: string,
-  material: PackagedMethodSkillMaterial,
-): {
-  readonly methodProvenance: PackagedMethodSkillProvenance;
-  readonly methodSkillPath: string;
-  readonly methodSkillConfiguredPath: string;
-} {
-  return {
-    methodProvenance: material.provenance,
-    methodSkillPath: material.skillPath,
-    methodSkillConfiguredPath: resolvePackagedMethodSkillPath(packageRoot, material.name),
-  };
-}
-
-/**
- * Observe packaged method Skill expansions from the session.
- * The skill name is the caller's packaged method; home locations never count.
- */
-function extractObservedMethodInvocations(
-  entries: readonly SessionEntry[],
-  options: {
-    readonly name: PackagedMethodSkillName;
-    readonly allowedLocations: readonly string[];
-  },
-): readonly ObservedPackagedMethodSkillInvocation[] {
-  const observed: ObservedPackagedMethodSkillInvocation[] = [];
-  for (const entry of entries) {
-    if (entry?.type !== "message") continue;
-    const message = entry.message;
-    if (message?.role !== "user") continue;
-    const text = sessionMessageText(message);
-    if (text.length === 0) continue;
-    const hit = observePackagedMethodSkillInvocation(text, {
-      name: options.name,
-      allowedLocations: options.allowedLocations,
-    });
-    if (hit !== undefined) observed.push(hit);
-  }
-  return Object.freeze(observed);
-}
-
 /**
  * #836: cost and the auditor's no-receipt facts ride beside the role's
  * testimony on the current attempt's candidate entry. Absence is omitted,
@@ -2834,49 +2751,11 @@ function doctorReportFacts(
   };
 }
 
-function observedMethodTail(
-  face: PackagedArtifactFace,
-  entries: readonly SessionEntry[],
-  options: {
-    readonly methodProvenance?: PackagedMethodSkillProvenance;
-    readonly methodSkillPath?: string;
-    readonly methodSkillConfiguredPath?: string;
-  },
-): Record<string, unknown> {
-  if (face.method === undefined) return {};
-  if (face.method === "optional") {
-    return options.methodProvenance === undefined
-      ? {}
-      : { methodProvenance: options.methodProvenance };
-  }
-  if (
-    options.methodProvenance === undefined
-    || options.methodSkillPath === undefined
-    || options.methodSkillConfiguredPath === undefined
-  ) {
-    throw new Error("observed method publication is missing packaged method coordinates");
-  }
-  const methodInvocations = extractObservedMethodInvocations(entries, {
-    name: options.methodProvenance.name,
-    allowedLocations: [options.methodSkillPath, options.methodSkillConfiguredPath],
-  });
-  return {
-    methodProvenance: options.methodProvenance,
-    methodInvocationObserved: methodInvocations.length > 0,
-    methodInvocations,
-  };
-}
-
 async function publishDeclaredSeatArtifacts(
   admitted: AdmittedRoleInvocation,
   roleOutcome: TerminalRoleOutcome,
   coordinates: DurablePrincipalCoordinates,
   entries: readonly SessionEntry[],
-  options: {
-    readonly methodProvenance?: PackagedMethodSkillProvenance;
-    readonly methodSkillPath?: string;
-    readonly methodSkillConfiguredPath?: string;
-  } = {},
 ): Promise<TerminalArtifactRef[]> {
   const face = seatArtifactFace(admitted.role);
   const phase = face.reportPhase === true
@@ -2898,7 +2777,6 @@ async function publishDeclaredSeatArtifacts(
       sessionFile: coordinates.sessionFile,
       admittedRequestPath: admitted.admittedRequestPath,
       attachments: acceptedArtifactAttachmentRefs(admitted.attachments),
-      ...observedMethodTail(face, entries, options),
     },
   });
 }
@@ -3040,18 +2918,8 @@ export async function trySettlePublicSeat(
   admitted: AdmittedRoleInvocation,
   authority: DurablePrincipalAuthority,
   scope: { readonly courtAttemptId?: string } | undefined,
-  material: PackagedMethodSkillMaterial | undefined,
-  packageRoot: string,
 ): Promise<TerminalResult | undefined> {
-  const face = seatArtifactFace(admitted.role);
-  if (face.method === "observed" && material === undefined) return undefined;
-  const options: MethodPublicationOptions =
-    face.method === "observed" && material !== undefined
-      ? observedMethodSkillOptions(packageRoot, material)
-      : face.method === "optional" && material !== undefined
-        ? { methodProvenance: material.provenance }
-        : {};
-  return settleSeat(admitted, authority, scope, options);
+  return settleSeat(admitted, authority, scope);
 }
 
 /**

package/src/public-cli/turn-request.ts

@@ -10,16 +10,13 @@ import type {
   RoleTurnRequest,
 } from "../host-contracts.ts";
 import { pickEngineAxis } from "../package-resources/engine-material.ts";
-import {
-  resolvePackagedMethodSkillPath,
-  type PackagedMethodSkillName,
-} from "../package-resources/method-skill.ts";
 import {
   packagedRoleActivationFlags,
   packagedRoleMetadata,
 } from "../packaged-role-registry.ts";
 import type { SeatModelConfig } from "./config.ts";
 import type { AdmittedRoleInvocation } from "./invocation.ts";
+import { installedMethodSkillPath } from "./machine-method-skills.ts";
 import { parentRunPathFromGatePointerInstruction } from "./run-lifecycle.ts";
 import type { PublicThinkingLevel } from "./registry.ts";
 
@@ -33,6 +30,7 @@ export type ResumeModelConfig = {
 export type RoleTurnRequestProjectionOptions = {
   packageRoot: string;
   home: string;
+  host?: string;
   agentDir: string;
   model?: SeatModelConfig;
   engine?: string;
@@ -103,7 +101,7 @@ export function projectRoleTurnRequest(
  */
 export function packagedSettleSkill(
   admitted: AdmittedRoleInvocation,
-): PackagedMethodSkillName | undefined {
+): string | undefined {
   const record = packagedRoleMetadata(admitted.role);
   if (record === undefined) return undefined;
   if ("applyMethod" in record && record.applyMethod !== undefined) {
@@ -148,33 +146,43 @@ function activationForAdmitted(admitted: AdmittedRoleInvocation): RoleTurnActiva
 }
 
 /** Skill bindings declared on the composition-root record for this admitted run. */
-function methodBindings(
-  admitted: AdmittedRoleInvocation,
-  packageRoot: string,
-): readonly MethodBinding[] {
+export function requiredMethodSkills(admitted: AdmittedRoleInvocation): readonly string[] {
   const record = packagedRoleMetadata(admitted.role);
-  const names: PackagedMethodSkillName[] = [];
+  const names: string[] = [];
   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" as const,
-    path: resolvePackagedMethodSkillPath(packageRoot, name),
-  }));
+  return names;
+}
+
+function methodBindings(
+  admitted: AdmittedRoleInvocation,
+  home: string,
+  host: string | undefined,
+): readonly MethodBinding[] {
+  if (host !== undefined && host !== "pi") return [];
+  return requiredMethodSkills(admitted).flatMap((name) => {
+    const path = installedMethodSkillPath(home, name);
+    return path === undefined ? [] : [{
+      kind: "skill" as const,
+      path,
+    }];
+  });
 }
 
 /** Registry activation and method bindings for one admitted public seat. */
 export function admittedSeatTurnDetails(
   admitted: AdmittedRoleInvocation,
-  packageRoot: string,
+  home: string,
+  host: string | undefined,
 ): {
   readonly activation: RoleTurnActivation;
   readonly methods: readonly MethodBinding[];
 } {
   return {
     activation: activationForAdmitted(admitted),
-    methods: methodBindings(admitted, packageRoot),
+    methods: methodBindings(admitted, home, host),
   };
 }

package/src/public-role-summons.ts

@@ -535,7 +535,16 @@ export async function summonPublicRole(
   }
 
   const stderr = captured?.stderrText();
-  const runDirectory = result.admitted?.runDirectory;
+  // A parked officer can return its court diarist's escalation verbatim. The
+  // terminal's independent run, not the parked officer, is the gate pointer.
+  const runDirectory = result.terminal !== undefined
+    && result.terminal.roleOutcome.role !== options.role
+    && typeof result.terminal.runId === "string"
+    ? await (await import("./public-cli/run-lifecycle.ts")).findRunDirectoryById(
+      home,
+      result.terminal.runId,
+    )
+    : result.admitted?.runDirectory;
   return {
     exitCode: result.exitCode,
     ...(result.terminal === undefined ? {} : { terminal: result.terminal }),
@@ -851,8 +860,12 @@ export async function summonGateOfficer(options: {
         `countersign gate summon requires parent runId from sourceRunDirectory: ${options.sourceRunDirectory}`,
       );
     }
-    const { readBoardTicketNumber } = await import("./run-ticket-number.ts");
+    const { isSafePositiveTicketNumber, readBoardTicketNumber } = await import("./run-ticket-number.ts");
     const parentTicket = await readBoardTicketNumber(options.sourceRunDirectory);
+    const submittedTicket = options.submission !== null && typeof options.submission === "object" && !Array.isArray(options.submission)
+      ? (options.submission as { ticketNumber?: unknown }).ticketNumber
+      : undefined;
+    const courtTicket = isSafePositiveTicketNumber(submittedTicket) ? submittedTicket : parentTicket;
     // Argv instruction = reask or parent payload bytes only (never a fabricated line).
     const instruction = options.reask ?? gateReviewInstruction ?? "";
     return summonPublicRole({
@@ -861,7 +874,7 @@ export async function summonGateOfficer(options: {
       ...common,
       correlationId,
       parentRunPath: options.sourceRunDirectory,
-      ...(parentTicket === undefined ? {} : { boundTicketNumber: parentTicket }),
+      ...(courtTicket === undefined ? {} : { boundTicketNumber: courtTicket }),
     });
   }
   // Inspector: argv stays pure 卷宗指针 (#747 parentRunPath lookup key).

package/src/reviewer-role.ts

@@ -3,8 +3,6 @@ import { Type } from "typebox";
 import { openToolObjectFromUnion } from "./open-tool-schema.ts";
 import { withTerminatingOutputDeclarations } from "./package-contracts/terminating-infrastructure.ts";
 
-import type { AnyCanonicalSkillBinding, CanonicalSkillBinding } from "./canonical-skill-binding.ts";
-export type { CanonicalSkillBinding };
 import { REVIEWER_OUTPUT_TOOL_NAME, type ReviewerIntent } from "./package-contracts/reviewer-output.ts";
 
 export { REVIEWER_OUTPUT_TOOL_NAME };
@@ -59,7 +57,6 @@ export const reviewerOutputSchema = withTerminatingOutputDeclarations(
 );
 export type ReviewerRoleDependencies = {
   loadSoul(): Promise<string>;
-  loadCanonicalSkillBinding(name: "ak-cross-m-review"): Promise<AnyCanonicalSkillBinding>;
 };
 export type ReviewerRoleHostActions = { failInfrastructure(error: unknown, ctx: HostContext, toolCallId?: string): never };
 
@@ -67,8 +64,6 @@ export type ReviewerRoleHostActions = { failInfrastructure(error: unknown, ctx:
 export type ReviewerActivation = Readonly<{
   fixedBaseRevision: string;
   soul: string;
-  /** Frozen ak-cross-m-review binding — envelope owns expansion capture against this data. */
-  skillBinding: CanonicalSkillBinding<"ak-cross-m-review">;
 }>;
 
 /**
@@ -84,7 +79,6 @@ export function createReviewerRoleRuntime(
   activate(ctx: HostContext | undefined, admitted: ReviewerAdmittedInputs): Promise<ReviewerActivation>;
 } {
   let soul: string | undefined;
-  let binding: CanonicalSkillBinding<"ak-cross-m-review"> | undefined;
   let registered = false;
   let fixedBaseRevision: string | undefined;
 
@@ -93,15 +87,12 @@ export function createReviewerRoleRuntime(
       soul = (await dependencies.loadSoul()).trim();
       if (!soul) throw new Error("Reviewer soul is empty");
       fixedBaseRevision = admitted.baseRevision;
-      const loaded = await dependencies.loadCanonicalSkillBinding("ak-cross-m-review");
-      if (loaded.name !== "ak-cross-m-review") throw new Error("Canonical Skill binding loader returned tdd for ak-cross-m-review");
-      binding = loaded;
 
       if (!registered) {
         registered = true;
         pi.registerTool({ name: REVIEWER_OUTPUT_TOOL_NAME, label: "御史台输出", description: "提交御史台终局回执。本席自调 ak-cross-m-review skill。", promptSnippet: "提交御史台终局回执", parameters: reviewerOutputSchema,
           async execute(_id: string, parameters: unknown): Promise<HostToolResult<unknown>> {
-            if (!soul || !binding) throw new Error("御史台输入未装载");
+            if (!soul) throw new Error("御史台输入未装载");
             return {
               content: [],
               details: parameters,
@@ -111,11 +102,9 @@ export function createReviewerRoleRuntime(
       }
       const activatedSoul = soul;
       const activatedBase = fixedBaseRevision;
-      const activatedBinding = binding;
       return Object.freeze({
         fixedBaseRevision: activatedBase,
         soul: activatedSoul,
-        skillBinding: activatedBinding,
       });
     },
   };

package/src/role-envelope.ts

@@ -22,7 +22,6 @@ import type {
   RoleTurnKnownFailure,
   RoleTurnRequest,
 } from "./host-contracts.ts";
-import { hostMethodSkills, installWorkspaceMethodSkills } from "./host-native-method.ts";
 import { packagedRoleOutputTool } from "./packaged-role-registry.ts";
 import {
   createRoleRuntimeExtension,
@@ -219,10 +218,6 @@ export async function prepareRoleEnvelope(options: {
     deliverSubmissionRejection(_value) {
       // #836 删 1/A4.5: do not arm closeRound retry with「终局交卷并非本轮唯一工具调用」.
     },
-    capabilities: {
-      // #922: native loaders own expansion; package does not pre-read bodies (ADR 0032).
-      skillExpansion() { return undefined; },
-    },
     registerFlag(name, definition) { if (!flags.has(name) && definition.default !== undefined) flags.set(name, definition.default); },
     getFlag(name) { return flags.get(name); },
     registerTool(tool) { tools.set(tool.name, tool); },
@@ -730,14 +725,6 @@ export async function prepareRoleEnvelope(options: {
     }
     const jsonSchema = terminatingToolJsonSchema(terminating.parameters);
 
-    if (request.host === "codex" && hostMethodSkills(request.methods).length > 0) {
-      const packageRoot = options.dependencies.packageRoot;
-      if (typeof packageRoot !== "string" || packageRoot === "") {
-        throw new Error("codex project Skill catalog requires packageRoot");
-      }
-      await installWorkspaceMethodSkills(request.cwd, packageRoot);
-    }
-
     return {
       mcpServers: [{
         name: `ak-${request.activation.role}`,

package/src/role-runtime-dependencies.ts

@@ -7,7 +7,6 @@ import { loadDoctorCase } from "./doctor-evidence.ts";
 import { createNativeNavigatorSessionFactory, createNavigatorAttendance } from "./navigator-attendance.ts";
 import { loadNavigatorWorkContext } from "./navigator-work-context.ts";
 import { loadNotarySourceRunLocator } from "./notary-source-run.ts";
-import { loadPackagedCanonicalSkillBinding } from "./package-resources/method-skill-binding.ts";
 import { formatNavigatorRoleHelp, type RoleRuntimeDependencies } from "./role-runtime.ts";
 import {
   loadAuditorReferenceMaterialsFromSubjectInput,
@@ -58,18 +57,6 @@ export function createRoleRuntimeDependencies(packageRoot: string): RoleRuntimeD
     loadDoctorCase,
     loadNotarySourceRun: loadNotarySourceRunLocator,
     loadMergerInput: async (path) => JSON.parse(await readFile(path, "utf8")),
-    async loadCanonicalSkillBinding(name) {
-      switch (name) {
-        case "tdd":
-          return loadPackagedCanonicalSkillBinding(packageRoot, name);
-        case "ak-cross-m-review":
-          return loadPackagedCanonicalSkillBinding(packageRoot, name);
-        default: {
-          const unexpected: never = name;
-          throw new Error(`Canonical skill is not packaged: ${String(unexpected)}`);
-        }
-      }
-    },
     // #590: doctor compliance still on disposeCompliance path; judge→auditor is gate queue (#756).
     auditDoctorCompliance: (options) => doctorAuditor(options),
     loadNavigatorWorkContext: (options) => loadNavigatorWorkContext({