@akagilnc/pi-workflow-roles 0.1.3873 → 0.1.3884

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

@@ -1,18 +1,13 @@
 /**
- * Public Merger Role run: derive active-merge envelope → force package
- * merge-only method → post-admission coordinator → settle Terminal result (#114 / #517).
+ * Public Merger Role run: read Git materials → force package merge-only method
+ * → post-admission coordinator → settle Terminal result (#114 / #517 / #827).
  * #526: execution via RoleTurnHost; argv is Pi adapter internal.
  */
 import type {
-  DurablePrincipalAuthority,
   MethodBinding,
   RoleTurnKnownFailure,
   RoleTurnRequest,
 } from "../host-contracts.ts";
-import { mkdir, writeFile } from "node:fs/promises";
-import { join, resolve } from "node:path";
-
-import { ensureRealDirectoryTree } from "../activation-ledger-topology.ts";
 import { engineSessionMaterialFromOptions } from "../package-resources/engine-material.ts";
 import {
   loadPackagedMethodSkillMaterial,
@@ -20,13 +15,10 @@ import {
   type PackagedMethodSkillMaterial,
   type PackagedMethodSkillProvenance,
 } from "../package-resources/method-skill.ts";
-import { uuidv7 } from "../uuidv7.ts";
 import { CliUsageError } from "./cli-errors.ts";
 import {
   admitMergerInvocation,
   buildMergerTransportPrompt,
-  issueAdmissionPlacement,
-  MergerEnvelopeDerivationError,
   type AdmittedMergerInvocation,
 } from "./invocation.ts";
 import {
@@ -116,82 +108,6 @@ async function loadMergerMethodMaterial(
   );
 }
 
-/**
- * Structural admit shell when envelope derivation fails: keeps role-correct
- * Terminal identity for honest activation-class settlement without inventing
- * parents/conflicts (those stay empty; the gate never receives a guessed packet).
- */
-async function admitMergerShellForActivationFailure(options: {
-  home: string;
-  cwd: string;
-  instruction: string;
-  project?: string;
-  createRunId?: () => string;
-  principalAuthority: DurablePrincipalAuthority;
-}): Promise<AdmittedMergerInvocation> {
-  const projectRoot = resolve(options.project ?? options.cwd);
-  const runId = (options.createRunId ?? uuidv7)();
-  const {
-    principal,
-    sessionDirectory,
-    sessionFile,
-    runDirectory,
-    ledgerHome,
-    bookKey,
-  } = issueAdmissionPlacement(options.principalAuthority, {
-    cwd: projectRoot,
-    runId,
-    role: "merger",
-    home: options.home,
-  });
-  ensureRealDirectoryTree(ledgerHome, sessionDirectory);
-  await mkdir(runDirectory, { recursive: true });
-  const emptyDerived = {
-    targetObjectId: "",
-    sourceObjectId: "",
-    automaticMergeTreeId: "",
-    expectedConflictPaths: [] as string[],
-    resolutionScope: [] as string[],
-  };
-  const admittedRequestPath = join(runDirectory, "admitted-request.json");
-  const mergerInputPath = join(runDirectory, "merger-input.json");
-  await writeFile(
-    admittedRequestPath,
-    `${JSON.stringify(
-      {
-        role: "merger",
-        runId,
-        bookKey,
-        projectRoot,
-        instruction: options.instruction,
-        instructionEmpty: false,
-        mergerInputPath,
-        derived: emptyDerived,
-        attachments: [],
-        sessionDirectory,
-        sessionFile,
-      },
-      null,
-      2,
-    )}\n`,
-    "utf8",
-  );
-  return {
-    role: "merger",
-    runId,
-    bookKey,
-    projectRoot,
-    instruction: options.instruction,
-    instructionEmpty: false,
-    attachments: [],
-    runDirectory,
-    principal,
-    admittedRequestPath,
-    mergerInputPath,
-    derived: emptyDerived,
-  };
-}
-
 function mergerTurnOptions(
   admitted: AdmittedMergerInvocation,
   env: MergerRunEnv,
@@ -263,33 +179,6 @@ export async function runPublicMerger(
       presentStructuralRejection(error, io);
       return { exitCode: 2 };
     }
-    if (error instanceof MergerEnvelopeDerivationError) {
-      const shell = await admitMergerShellForActivationFailure({
-        home: env.home,
-        principalAuthority: env.principalAuthority,
-        cwd: env.cwd,
-        instruction: parsed.instruction,
-        ...(parsed.project === undefined ? {} : { project: parsed.project }),
-        ...(env.createRunId === undefined
-          ? {}
-          : { createRunId: env.createRunId }),
-      });
-      await markRunAdmitted(shell, env.principalAuthority);
-      return (await presentControlledFailure(
-        shell,
-        {
-          timedOut: false,
-          code: null,
-          stderr: "",
-          thrown: error,
-          knownCause: "activation",
-          knownDiagnostic: error.message,
-        },
-        mergerAdapters(env.packageRoot),
-        env.principalAuthority,
-        io,
-      )) as { exitCode: number; admitted: AdmittedMergerInvocation; terminal: TerminalResult };
-    }
     throw error;
   }
 

package/src/public-cli/option-definitions.ts

@@ -670,7 +670,7 @@ const NOTARY_OPTIONS = [
 ] as const satisfies readonly PublicOptionDefinition[];
 
 const MERGER_OPTIONS = [
-  // Merger project face differs: requires an in-progress ordinary merge root.
+  // Merger project face: worktree whose Git materials are handed to the role.
   {
     id: "project",
     owner: "merger",
@@ -681,8 +681,8 @@ const MERGER_OPTIONS = [
     repeatable: false,
     form: "option",
     description: {
-      en: "Project root with one ordinary in-progress merge (defaults to cwd).",
-      zh: "已有进行中 ordinary merge 的项目根（默认 cwd）。",
+      en: "Project root whose Git merge materials are read (defaults to cwd).",
+      zh: "读取 Git merge 材料的项目根（默认 cwd）。",
     },
   },
   bindOwner("merger", SHARED_ATTACH_SEMANTICS),
@@ -1231,10 +1231,10 @@ const ROLE_COMMAND_HELP = {
   },
   merger: {
     command: "merger",
-    summary: "Resolve one ordinary merge already in conflict.",
+    summary: "Reconcile merge materials in a worktree (escalate when nothing to merge).",
     usage: ["ak-role merger [options] <instruction>"],
     examples: [
-      'ak-role merger --project /path/to/worktree "Reconcile the active merge."',
+      'ak-role merger --project /path/to/worktree "Reconcile the merge."',
     ],
   },
   inspector: {

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

@@ -1386,7 +1386,6 @@ async function loadResumableRunRecord(
         if (
           typeof d.targetObjectId === "string" &&
           typeof d.sourceObjectId === "string" &&
-          typeof d.automaticMergeTreeId === "string" &&
           Array.isArray(d.expectedConflictPaths) &&
           Array.isArray(d.resolutionScope) &&
           d.expectedConflictPaths.every((p) => typeof p === "string") &&
@@ -1395,7 +1394,6 @@ async function loadResumableRunRecord(
           derived = {
             targetObjectId: d.targetObjectId,
             sourceObjectId: d.sourceObjectId,
-            automaticMergeTreeId: d.automaticMergeTreeId,
             expectedConflictPaths: d.expectedConflictPaths as string[],
             resolutionScope: d.resolutionScope as string[],
           };

package/src/role-runtime-dependencies.ts

@@ -5,7 +5,6 @@ import { loadCanonicalSkillBinding as loadHomeCanonicalSkillBinding } from "./ca
 import { createGhCollectorGitHubTransport, createGhIssueSoftFetcher } from "./collector-github.ts";
 import { createPiDoctorAuditor } from "./doctor-auditor.ts";
 import { loadDoctorCase } from "./doctor-evidence.ts";
-import { createProductionMergerGitState } from "./merger-git-state.ts";
 import { createNativeNavigatorSessionFactory, createNavigatorAttendance } from "./navigator-attendance.ts";
 import { loadNavigatorWorkContext } from "./navigator-work-context.ts";
 import { loadNotarySourceRunLocator } from "./notary-source-run.ts";
@@ -53,7 +52,6 @@ export function createRoleRuntimeDependencies(packageRoot: string): RoleRuntimeD
     loadNotarySourceRun: loadNotarySourceRunLocator,
     loadMergerSoul: () => loadMainRoleSessionMaterials("merger"),
     loadMergerInput: async (path) => JSON.parse(await readFile(path, "utf8")),
-    createMergerGitState: (repositoryRoot) => createProductionMergerGitState(repositoryRoot),
     async loadCanonicalSkillBinding(name) {
       if (name === "tdd") {
         return loadPackagedCanonicalSkillBinding(packageRoot, "tdd");

package/src/role-runtime.ts

@@ -423,7 +423,7 @@ export { MERGER_INPUT_FLAG, createMergerRoleRuntime } from "./merger-role.ts";
 export { MERGER_OUTPUT_TOOL_NAME, mergerInputSchema, mergerOutputSchema, validateMergerInput, validateMergerOutput } from "./merger-contracts.ts";
 export type { MergerInput, MergerMaterial, MergerOutput } from "./merger-contracts.ts";
 export { createProductionMergerGitState } from "./merger-git-state.ts";
-export type { MergerGitState, ActiveMergerGitState, CompletedMergerGitState } from "./merger-git-state.ts";
+export type { MergerGitState, ActiveMergerGitState } from "./merger-git-state.ts";
 export type { MergerRoleDependencies } from "./merger-role.ts";
 
 type WorkerArmable = {
@@ -627,8 +627,6 @@ export type RoleRuntimeDependencies = {
   loadDoctorCase?(path: string): Promise<import("./doctor-contracts.ts").DoctorCase>;
   loadMergerSoul?(): Promise<string>;
   loadMergerInput?(path: string): Promise<unknown>;
-  mergerGitState?: MergerRoleDependencies["gitState"];
-  createMergerGitState?(repositoryRoot: string): MergerRoleDependencies["gitState"];
   auditDoctorCompliance?(options: { context: HostContext; signal?: AbortSignal }): Promise<ComplianceDecision>;
   createCollectorClock?(): CollectorClock;
   createNavigatorAttendance?(options: { context: HostContext; role: string; phase: NavigatorPhase; subjectKey: string; subject: string; authority: string; contextError?: unknown; invocationId: string; onEvent: (event: import("./navigator-attendance.ts").NavigatorEvent, report: import("./navigator-attendance.ts").NavigatorReport) => void | Promise<void> }): NavigatorAttendanceDependency | Promise<NavigatorAttendanceDependency>;
@@ -1682,15 +1680,10 @@ export function createRoleRuntimeExtension(
         },
       },
     );
-    let sessionMergerGitState = dependencies.mergerGitState;
     const merger = createMergerRoleRuntime(roleHost, {
       async loadSoul() { if (!dependencies.loadMergerSoul) throw new Error("Merger runtime dependencies are not configured"); return dependencies.loadMergerSoul(); },
       async loadInput(path) { if (!dependencies.loadMergerInput) throw new Error("Merger runtime dependencies are not configured"); return dependencies.loadMergerInput(path); },
-      gitState: {
-        activeMerge() { if (!sessionMergerGitState) throw new Error("Merger runtime dependencies are not configured"); return sessionMergerGitState.activeMerge(); },
-        completedMerge(mergeCommitId, automaticMergeTreeId) { if (!sessionMergerGitState) throw new Error("Merger runtime dependencies are not configured"); return sessionMergerGitState.completedMerge(mergeCommitId, automaticMergeTreeId); },
-      },
-    }, hostActions);
+    });
     // #676 E: shared envelope owns collector lifecycle (mode/fork, tool surface,
     // event gates). Role module supplies business activate/tools/materials only.
     let activeCollector: CollectorActivation | undefined;
@@ -1997,10 +1990,6 @@ export function createRoleRuntimeExtension(
         auditor,
         diarist,
         merger: async () => {
-          if (dependencies.mergerGitState === undefined) {
-            sessionMergerGitState = dependencies.createMergerGitState?.(ctx.cwd);
-          }
-          if (sessionMergerGitState === undefined) throw new Error("Merger runtime dependencies are not configured");
           await merger.activate();
         },
       };