@akagilnc/pi-workflow-roles 0.1.5593 → 0.1.5598

package/src/role-runtime-dependencies.ts

@@ -45,8 +45,6 @@ export function createRoleRuntimeDependencies(packageRoot: string): RoleRuntimeD
     packageRoot,
     loadRoleReferenceMaterials: loadPackagedRoleReferenceMaterials,
     loadRoleSoul: loadRegisteredRoleSoul,
-    loadFixPacket: (path) => readFile(path, "utf8"),
-    loadCoderTask: (path) => readFile(path, "utf8"),
     loadDoctorCase,
     loadNotarySourceRun: loadNotarySourceRunLocator,
     loadMergerInput: async (path) => JSON.parse(await readFile(path, "utf8")),

package/src/role-runtime.ts

@@ -346,8 +346,6 @@ export {
 } from "./worker-role.ts";
 export { fixerOutputSchema, validateFixerOutput } from "./package-contracts/fixer-output.ts";
 export type { FixerBlocker, FixerClassResult, FixerPhase, FixerTestEvidence } from "./package-contracts/fixer-output.ts";
-export { fixerPrerequisiteSchema, fixerPrerequisitesSchema, parseFixerPrerequisites, validateFixerPrerequisites } from "./package-contracts/fixer-packet.ts";
-export type { FixerInvocationInput, FixerPrerequisite } from "./package-contracts/fixer-packet.ts";
 export {
   AUDITOR_SOUL_ROLES,
   AK_ROLE_AUDITOR_SUBJECT_ENV,
@@ -460,8 +458,6 @@ export type RoleRuntimeDependencies = {
   loadRoleReferenceMaterials?(role: PackagedRole): Promise<string>;
   /** One soul loader. The role argument selects the registry record. */
   loadRoleSoul(role: PackagedRole): Promise<string>;
-  loadFixPacket?(path: string): Promise<string>;
-  loadCoderTask?(path: string): Promise<string>;
   loadNotarySourceRun?(path: string): Promise<import("./notary-contracts.ts").NotarySourceRunLocator>;
   loadDoctorCase?(path: string): Promise<import("./doctor-contracts.ts").DoctorCase>;
   loadMergerInput?(path: string): Promise<unknown>;
@@ -1459,12 +1455,6 @@ export function createRoleRuntimeExtension(
       roleHost,
       {
         loadSoul: () => requireRoleSoul("fixer"),
-        async loadPacket(path) {
-          if (dependencies.loadFixPacket === undefined) {
-            throw new Error("Fixer packet loader is not configured");
-          }
-          return dependencies.loadFixPacket(path);
-        },
       },
       hostActions,
       { unfinishedReasonBounceLimit: deliveryLimit },
@@ -1473,12 +1463,6 @@ export function createRoleRuntimeExtension(
       roleHost,
       {
         loadSoul: () => requireRoleSoul("coder"),
-        async loadTask(path) {
-          if (dependencies.loadCoderTask === undefined) {
-            throw new Error("Coder task loader is not configured");
-          }
-          return dependencies.loadCoderTask(path);
-        },
       },
       hostActions,
       { unfinishedReasonBounceLimit: deliveryLimit },

package/src/worker-role.ts

@@ -10,11 +10,6 @@ import {
 import { validateFixerOutput, type FixerPhase } from "./package-contracts/fixer-output.ts";
 import { registerFiledSubmissionTool } from "./filed-submission.ts";
 import { roleSubmissionDeclaration } from "./role-submission-declarations.ts";
-import {
-  FixerPacketValidationError,
-  parseFixerPrerequisites,
-  type FixerInvocationInput,
-} from "./package-contracts/fixer-packet.ts";
 import {
   createWorkerSubmissionGate,
   WorkerCommitReminderError,
@@ -37,21 +32,8 @@ export {
   validateAcceptedWorkerDetails,
 } from "./package-contracts/worker-output.ts";
 export type { WorkerOutput, FixerOutput, CoderOutput };
+/** #1168: phase only — dispatch text is the first message, not a packet flag. */
 export const FIXER_FLAG_DEFINITIONS = {
-  packet: {
-    name: "ak-fix-packet",
-    definition: {
-      description: "Path to opaque prose instructions for the Fixer",
-      type: "string" as const,
-    },
-  },
-  prerequisites: {
-    name: "ak-fixer-prerequisites",
-    definition: {
-      description: "Optional path to a JSON array of typed Fixer prerequisites",
-      type: "string" as const,
-    },
-  },
   phase: {
     name: "ak-fixer-phase",
     definition: {
@@ -73,12 +55,10 @@ export type WorkerRoleHostActions = HostGatekeeperActions;
 
 export type FixerRoleDependencies = {
   loadSoul(): Promise<string>;
-  loadPacket(path: string): Promise<string>;
 };
 
 export type CoderRoleDependencies = {
   loadSoul(): Promise<string>;
-  loadTask(path: string): Promise<string>;
 };
 
 export type WorkerRoleRuntime = {
@@ -178,9 +158,6 @@ export function createFixerRoleRuntime(
   options?: { readonly unfinishedReasonBounceLimit?: number },
 ): WorkerRoleRuntime {
   let soul: string | undefined;
-  let packet: FixerInvocationInput | undefined;
-  let packetPath: string | undefined;
-  let prerequisitesPath: string | undefined;
   let phase: WorkerPhase | undefined;
   let lifecycleRegistered = false;
   // #1132: ADR 0050 缺理由催全次数. The in-process seam passes the value it
@@ -190,14 +167,6 @@ export function createFixerRoleRuntime(
       ?? deliveryLimitFromEnv(process.env),
   });
 
-  pi.registerFlag(
-    FIXER_FLAG_DEFINITIONS.packet.name,
-    FIXER_FLAG_DEFINITIONS.packet.definition,
-  );
-  pi.registerFlag(
-    FIXER_FLAG_DEFINITIONS.prerequisites.name,
-    FIXER_FLAG_DEFINITIONS.prerequisites.definition,
-  );
   pi.registerFlag(
     FIXER_FLAG_DEFINITIONS.phase.name,
     FIXER_FLAG_DEFINITIONS.phase.definition,
@@ -214,33 +183,12 @@ export function createFixerRoleRuntime(
         );
       }
       phase = selectedPhase;
-      const resolvedPacketPath = pi.getFlag(FIXER_FLAG_DEFINITIONS.packet.name);
-      if (typeof resolvedPacketPath !== "string" || resolvedPacketPath.trim().length === 0) {
-        throw new Error("Fixer role requires --ak-fix-packet");
-      }
-      packetPath = resolvedPacketPath;
-      const instructions = await dependencies.loadPacket(packetPath);
-      if (instructions.trim().length === 0) {
-        throw new FixerPacketValidationError(
-          new Error("Fixer instructions must be nonblank"),
-        );
-      }
-      const resolvedPrerequisitesPath = pi.getFlag(FIXER_FLAG_DEFINITIONS.prerequisites.name);
-      if (resolvedPrerequisitesPath !== undefined && (typeof resolvedPrerequisitesPath !== "string" || resolvedPrerequisitesPath.trim().length === 0)) {
-        throw new Error("Fixer --ak-fixer-prerequisites path must be nonblank when supplied");
-      }
-      prerequisitesPath =
-        typeof resolvedPrerequisitesPath === "string" ? resolvedPrerequisitesPath : undefined;
-      const prerequisites = prerequisitesPath !== undefined
-        ? parseFixerPrerequisites(await dependencies.loadPacket(prerequisitesPath))
-        : Object.freeze([]);
-      packet = Object.freeze({ instructions, prerequisites });
 
       if (!lifecycleRegistered) {
         lifecycleRegistered = true;
         registerWorkerSubmission(pi, "fixer", {
-          notReady: "修内司修理包与阶段未装载",
-          ready: () => packet !== undefined && phase !== undefined,
+          notReady: "修内司阶段未装载",
+          ready: () => phase !== undefined,
           phase: () => phase,
           project: (parameters, current) => deepFreeze(validateFixerOutput(parameters, current)),
           submissionGate,
@@ -257,17 +205,12 @@ export function createFixerRoleRuntime(
             reason: fixerBashSeatbeltDenyReason(matched),
           };
         });
+        // #1168: soul + phase only; dispatch text is the first message.
         pi.on("before_agent_start", (event) => {
           if (soul === undefined) throw new Error("修内司职分未装载");
-          if (packetPath === undefined) throw new Error("修内司修理包路径未装载");
-          // Path delivery only — body is self-fetched; no inline duplicate of flag bytes (#632).
-          const prerequisitesBlock =
-            prerequisitesPath === undefined
-              ? ""
-              : `\n\n<fixer_prerequisites_path>\n${prerequisitesPath}\n</fixer_prerequisites_path>`;
           return {
             systemPrompt:
-              `${event.systemPrompt}\n\n<fixer_soul>\n${soul}\n</fixer_soul>\n\n<fixer_phase>\n${phase ?? ""}\n</fixer_phase>\n\n<fix_packet_path>\n${packetPath}\n</fix_packet_path>${prerequisitesBlock}`,
+              `${event.systemPrompt}\n\n<fixer_soul>\n${soul}\n</fixer_soul>\n\n<fixer_phase>\n${phase ?? ""}\n</fixer_phase>`,
           };
         });
       }
@@ -285,7 +228,6 @@ export function createCoderRoleRuntime(
   options?: { readonly unfinishedReasonBounceLimit?: number },
 ): WorkerRoleRuntime {
   let soul: string | undefined;
-  let task: string | undefined;
   let phase: WorkerPhase | undefined;
   let lifecycleRegistered = false;
   // #1132: same resolved ceiling as the fixer gate. Absent = Pi child env.
@@ -294,10 +236,6 @@ export function createCoderRoleRuntime(
       ?? deliveryLimitFromEnv(process.env),
   });
 
-  pi.registerFlag("ak-coder-task", {
-    description: "Markdown task assigned to the coder role",
-    type: "string",
-  });
   pi.registerFlag("ak-coder-phase", {
     description:
       "Coder phase: plan (inspect and propose an implementation plan; no edits or commits) or apply (execute the approved plan and verify the first implementation)",
@@ -305,7 +243,7 @@ export function createCoderRoleRuntime(
   });
 
   return {
-    async activate(ctx) {
+    async activate() {
       soul = (await dependencies.loadSoul()).trim();
       if (soul.length === 0) throw new Error("Coder soul is empty");
       const selectedPhase = pi.getFlag("ak-coder-phase");
@@ -315,28 +253,23 @@ export function createCoderRoleRuntime(
         );
       }
       phase = selectedPhase;
-      const taskPath = pi.getFlag("ak-coder-task");
-      if (typeof taskPath !== "string" || taskPath.trim().length === 0) {
-        throw new Error("Coder role requires --ak-coder-task");
-      }
-      task = (await dependencies.loadTask(taskPath)).trim();
-      if (task.length === 0) throw new Error("Coder task is empty");
 
       if (!lifecycleRegistered) {
         lifecycleRegistered = true;
         registerWorkerSubmission(pi, "coder", {
-          notReady: "将作监任务与阶段未装载",
-          ready: () => task !== undefined && phase !== undefined,
+          notReady: "将作监阶段未装载",
+          ready: () => phase !== undefined,
           phase: () => phase,
           project: (parameters, current) => validateWorkerOutput(parameters, current, "Coder"),
           submissionGate,
           hostActions,
         });
-        pi.on("before_agent_start", (event, ctx) => {
+        // #1168: soul + phase only; dispatch text is the first message.
+        pi.on("before_agent_start", (event) => {
           if (soul === undefined) throw new Error("将作监职分未装载");
           return {
             systemPrompt:
-              `${event.systemPrompt}\n\n<coder_soul>\n${soul}\n</coder_soul>\n\n<coder_phase>\n${phase ?? ""}\n</coder_phase>\n\n<coder_task>\n${task ?? ""}\n</coder_task>`,
+              `${event.systemPrompt}\n\n<coder_soul>\n${soul}\n</coder_soul>\n\n<coder_phase>\n${phase ?? ""}\n</coder_phase>`,
           };
         });
       }

package/dist/package-contracts/fixer-packet.js

@@ -1,77 +0,0 @@
-import { Type } from "typebox";
-import { Value } from "typebox/value";
-import { isRecord } from "../unknown-value.js";
-const FIXER_PREREQUISITE_ID_PATTERN = "^[A-Za-z0-9][A-Za-z0-9._-]*$";
-const fixerPrerequisiteSchema = Type.Object({
-  id: Type.String({ pattern: FIXER_PREREQUISITE_ID_PATTERN }),
-  requirement: Type.String({ pattern: "\\S" })
-});
-const fixerPrerequisitesSchema = Type.Array(fixerPrerequisiteSchema);
-function causeMessage(cause) {
-  if (cause instanceof Error) return cause.message;
-  if (typeof cause === "string") return cause;
-  try {
-    return JSON.stringify(cause) ?? String(cause);
-  } catch {
-    return String(cause);
-  }
-}
-class FixerPacketValidationError extends Error {
-  code = "AK_INVALID_FIX_PACKET";
-  constructor(cause) {
-    const prefix = "Fixer prerequisites or instructions violate the invocation contract";
-    super(
-      cause === void 0 ? prefix : `${prefix}: ${causeMessage(cause)}`,
-      cause === void 0 ? void 0 : { cause }
-    );
-    this.name = "FixerPacketValidationError";
-  }
-}
-function fail(cause) {
-  throw new FixerPacketValidationError(cause);
-}
-function parseFailure(value) {
-  if (!Array.isArray(value)) fail(new Error("Fixer prerequisites must be a JSON array"));
-  for (const entry of value) {
-    if (!isRecord(entry)) {
-      fail(new Error("Fixer prerequisite entry must be an object with id and requirement fields"));
-    }
-    if (typeof entry.id !== "string" || !new RegExp(FIXER_PREREQUISITE_ID_PATTERN).test(entry.id)) {
-      fail(new Error(`Fixer prerequisite id violates pattern ${FIXER_PREREQUISITE_ID_PATTERN}`));
-    }
-    if (typeof entry.requirement !== "string" || !/\S/.test(entry.requirement)) {
-      fail(new Error("Fixer prerequisite requirement must be nonblank"));
-    }
-  }
-  fail(new Error("Fixer prerequisites violate the attachment schema"));
-}
-function validateFixerPrerequisites(value) {
-  if (!Value.Check(fixerPrerequisitesSchema, value)) parseFailure(value);
-  const entries = value;
-  const ids = /* @__PURE__ */ new Set();
-  const prerequisites = entries.map((entry) => {
-    if (ids.has(entry.id)) {
-      fail(new Error(`Fixer prerequisites contain duplicate id: ${entry.id}`));
-    }
-    ids.add(entry.id);
-    return Object.freeze({ id: entry.id, requirement: entry.requirement });
-  });
-  return Object.freeze(prerequisites);
-}
-function parseFixerPrerequisites(source) {
-  let decoded;
-  try {
-    decoded = JSON.parse(source);
-  } catch (error) {
-    fail(error);
-  }
-  return validateFixerPrerequisites(decoded);
-}
-export {
-  FIXER_PREREQUISITE_ID_PATTERN,
-  FixerPacketValidationError,
-  fixerPrerequisiteSchema,
-  fixerPrerequisitesSchema,
-  parseFixerPrerequisites,
-  validateFixerPrerequisites
-};

package/packets/fixer-prerequisites.json

@@ -1,6 +0,0 @@
-[
-  {
-    "id": "owner.choice",
-    "requirement": "The controlling owner decision required before this repair exists."
-  }
-]

package/src/package-contracts/fixer-packet.ts

@@ -1,91 +0,0 @@
-import { Type, type Static } from "typebox";
-import { Value } from "typebox/value";
-
-import { isRecord } from "../unknown-value.ts";
-
-export const FIXER_PREREQUISITE_ID_PATTERN = "^[A-Za-z0-9][A-Za-z0-9._-]*$";
-
-export const fixerPrerequisiteSchema = Type.Object({
-  id: Type.String({ pattern: FIXER_PREREQUISITE_ID_PATTERN }),
-  requirement: Type.String({ pattern: "\\S" }),
-});
-
-export const fixerPrerequisitesSchema = Type.Array(fixerPrerequisiteSchema);
-export type FixerPrerequisite = Readonly<Static<typeof fixerPrerequisiteSchema>>;
-export type FixerInvocationInput = Readonly<{
-  instructions: string;
-  prerequisites: readonly FixerPrerequisite[];
-}>;
-
-function causeMessage(cause: unknown): string {
-  if (cause instanceof Error) return cause.message;
-  if (typeof cause === "string") return cause;
-  try {
-    return JSON.stringify(cause) ?? String(cause);
-  } catch {
-    return String(cause);
-  }
-}
-
-export class FixerPacketValidationError extends Error {
-  readonly code = "AK_INVALID_FIX_PACKET";
-
-  constructor(cause?: unknown) {
-    const prefix = "Fixer prerequisites or instructions violate the invocation contract";
-    super(
-      cause === undefined ? prefix : `${prefix}: ${causeMessage(cause)}`,
-      cause === undefined ? undefined : { cause },
-    );
-    this.name = "FixerPacketValidationError";
-  }
-}
-
-function fail(cause: unknown): never {
-  throw new FixerPacketValidationError(cause);
-}
-
-function parseFailure(value: unknown): never {
-  if (!Array.isArray(value)) fail(new Error("Fixer prerequisites must be a JSON array"));
-  for (const entry of value) {
-    if (!isRecord(entry)) {
-      fail(new Error("Fixer prerequisite entry must be an object with id and requirement fields"));
-    }
-    if (
-      typeof (entry as Record<string, unknown>).id !== "string" ||
-      !(new RegExp(FIXER_PREREQUISITE_ID_PATTERN)).test((entry as { id: string }).id)
-    ) {
-      fail(new Error(`Fixer prerequisite id violates pattern ${FIXER_PREREQUISITE_ID_PATTERN}`));
-    }
-    if (
-      typeof (entry as Record<string, unknown>).requirement !== "string" ||
-      !/\S/.test((entry as { requirement: string }).requirement)
-    ) {
-      fail(new Error("Fixer prerequisite requirement must be nonblank"));
-    }
-  }
-  fail(new Error("Fixer prerequisites violate the attachment schema"));
-}
-
-export function validateFixerPrerequisites(value: unknown): readonly FixerPrerequisite[] {
-  if (!Value.Check(fixerPrerequisitesSchema, value)) parseFailure(value);
-  const entries = value as Static<typeof fixerPrerequisitesSchema>;
-  const ids = new Set<string>();
-  const prerequisites = entries.map((entry) => {
-    if (ids.has(entry.id)) {
-      fail(new Error(`Fixer prerequisites contain duplicate id: ${entry.id}`));
-    }
-    ids.add(entry.id);
-    return Object.freeze({ id: entry.id, requirement: entry.requirement });
-  });
-  return Object.freeze(prerequisites);
-}
-
-export function parseFixerPrerequisites(source: string): readonly FixerPrerequisite[] {
-  let decoded: unknown;
-  try {
-    decoded = JSON.parse(source);
-  } catch (error) {
-    fail(error);
-  }
-  return validateFixerPrerequisites(decoded);
-}