@akagilnc/pi-workflow-roles 0.1.5712 → 0.1.5714

package/dist/acp-host/production-host.js

@@ -18961,35 +18961,8 @@ async function summonParallelReviewerLenses(options) {
     const failure = failedResult(error);
     return { completeness: failure, correctness: failure };
   };
-  let sourceProjectRoot;
-  let targetCommit;
-  const statusArgs = [
-    "status",
-    "--porcelain=v1",
-    "--untracked-files=all",
-    "--",
-    ":/",
-    ":(top,exclude).claude/worktrees/**"
-  ];
   try {
-    sourceProjectRoot = await resolveReviewerGitToplevel(options.projectRoot);
-    const { stdout: statusStdout } = await execFileAsync3("git", statusArgs, {
-      cwd: sourceProjectRoot
-    });
-    if (statusStdout !== "") {
-      const diagnostic = [
-        "Reviewer target status gate failed:",
-        "git status --porcelain=v1 --untracked-files=all -- :/ ':(top,exclude).claude/worktrees/**'",
-        statusStdout
-      ].join("\n");
-      return dualFailure(new Error(diagnostic));
-    }
-    const { stdout: targetStdout } = await execFileAsync3(
-      "git",
-      ["rev-parse", "--verify", "HEAD^{commit}"],
-      { cwd: sourceProjectRoot }
-    );
-    targetCommit = targetStdout.trim();
+    const sourceProjectRoot = await resolveReviewerGitToplevel(options.projectRoot);
     await execFileAsync3(
       "git",
       ["rev-parse", "--verify", `${options.baseRevision}^{commit}`],
@@ -19025,55 +18998,10 @@ async function summonParallelReviewerLenses(options) {
     summon("completeness"),
     summon("correctness")
   ]);
-  let results = {
+  return {
     completeness: settled[0].status === "fulfilled" ? settled[0].value : failedResult(settled[0].reason),
     correctness: settled[1].status === "fulfilled" ? settled[1].value : failedResult(settled[1].reason)
   };
-  let sealDiagnostic;
-  try {
-    const { stdout: headBeforeStdout } = await execFileAsync3(
-      "git",
-      ["rev-parse", "--verify", "HEAD^{commit}"],
-      { cwd: sourceProjectRoot }
-    );
-    const { stdout: sealedStatusStdout } = await execFileAsync3("git", statusArgs, {
-      cwd: sourceProjectRoot
-    });
-    const { stdout: headAfterStdout } = await execFileAsync3(
-      "git",
-      ["rev-parse", "--verify", "HEAD^{commit}"],
-      { cwd: sourceProjectRoot }
-    );
-    const headBefore = headBeforeStdout.trim();
-    const headAfter = headAfterStdout.trim();
-    if (headBefore !== targetCommit || headAfter !== targetCommit || sealedStatusStdout !== "") {
-      sealDiagnostic = [
-        "Reviewer target final seal failed:",
-        `HEAD before status => ${headBefore}`,
-        `HEAD after status => ${headAfter}`,
-        `expected PRE_HEAD ${targetCommit}`,
-        "git status --porcelain=v1 --untracked-files=all -- :/ ':(top,exclude).claude/worktrees/**'",
-        sealedStatusStdout
-      ].join("\n");
-    }
-  } catch (error) {
-    sealDiagnostic = `Reviewer target final seal failed:
-${serializeThrownValue(error)}`;
-  }
-  if (sealDiagnostic !== void 0) {
-    const withSealFailure = (result) => ({
-      ...result,
-      exitCode: 1,
-      stderr: [result.stderr, sealDiagnostic].filter(
-        (text) => typeof text === "string" && text !== ""
-      ).join("\n")
-    });
-    results = {
-      completeness: withSealFailure(results.completeness),
-      correctness: withSealFailure(results.correctness)
-    };
-  }
-  return results;
 }
 async function summonGateOfficer(options) {
   let home = options.home;

package/dist/headless-host/production-host.js

@@ -19155,35 +19155,8 @@ async function summonParallelReviewerLenses(options) {
     const failure = failedResult(error);
     return { completeness: failure, correctness: failure };
   };
-  let sourceProjectRoot;
-  let targetCommit;
-  const statusArgs = [
-    "status",
-    "--porcelain=v1",
-    "--untracked-files=all",
-    "--",
-    ":/",
-    ":(top,exclude).claude/worktrees/**"
-  ];
   try {
-    sourceProjectRoot = await resolveReviewerGitToplevel(options.projectRoot);
-    const { stdout: statusStdout } = await execFileAsync3("git", statusArgs, {
-      cwd: sourceProjectRoot
-    });
-    if (statusStdout !== "") {
-      const diagnostic = [
-        "Reviewer target status gate failed:",
-        "git status --porcelain=v1 --untracked-files=all -- :/ ':(top,exclude).claude/worktrees/**'",
-        statusStdout
-      ].join("\n");
-      return dualFailure(new Error(diagnostic));
-    }
-    const { stdout: targetStdout } = await execFileAsync3(
-      "git",
-      ["rev-parse", "--verify", "HEAD^{commit}"],
-      { cwd: sourceProjectRoot }
-    );
-    targetCommit = targetStdout.trim();
+    const sourceProjectRoot = await resolveReviewerGitToplevel(options.projectRoot);
     await execFileAsync3(
       "git",
       ["rev-parse", "--verify", `${options.baseRevision}^{commit}`],
@@ -19219,55 +19192,10 @@ async function summonParallelReviewerLenses(options) {
     summon("completeness"),
     summon("correctness")
   ]);
-  let results = {
+  return {
     completeness: settled[0].status === "fulfilled" ? settled[0].value : failedResult(settled[0].reason),
     correctness: settled[1].status === "fulfilled" ? settled[1].value : failedResult(settled[1].reason)
   };
-  let sealDiagnostic;
-  try {
-    const { stdout: headBeforeStdout } = await execFileAsync3(
-      "git",
-      ["rev-parse", "--verify", "HEAD^{commit}"],
-      { cwd: sourceProjectRoot }
-    );
-    const { stdout: sealedStatusStdout } = await execFileAsync3("git", statusArgs, {
-      cwd: sourceProjectRoot
-    });
-    const { stdout: headAfterStdout } = await execFileAsync3(
-      "git",
-      ["rev-parse", "--verify", "HEAD^{commit}"],
-      { cwd: sourceProjectRoot }
-    );
-    const headBefore = headBeforeStdout.trim();
-    const headAfter = headAfterStdout.trim();
-    if (headBefore !== targetCommit || headAfter !== targetCommit || sealedStatusStdout !== "") {
-      sealDiagnostic = [
-        "Reviewer target final seal failed:",
-        `HEAD before status => ${headBefore}`,
-        `HEAD after status => ${headAfter}`,
-        `expected PRE_HEAD ${targetCommit}`,
-        "git status --porcelain=v1 --untracked-files=all -- :/ ':(top,exclude).claude/worktrees/**'",
-        sealedStatusStdout
-      ].join("\n");
-    }
-  } catch (error) {
-    sealDiagnostic = `Reviewer target final seal failed:
-${serializeThrownValue(error)}`;
-  }
-  if (sealDiagnostic !== void 0) {
-    const withSealFailure = (result) => ({
-      ...result,
-      exitCode: 1,
-      stderr: [result.stderr, sealDiagnostic].filter(
-        (text) => typeof text === "string" && text !== ""
-      ).join("\n")
-    });
-    results = {
-      completeness: withSealFailure(results.completeness),
-      correctness: withSealFailure(results.correctness)
-    };
-  }
-  return results;
 }
 async function summonGateOfficer(options) {
   let home = options.home;

package/dist/public-cli/main.js

@@ -20841,35 +20841,8 @@ async function summonParallelReviewerLenses(options) {
     const failure = failedResult(error);
     return { completeness: failure, correctness: failure };
   };
-  let sourceProjectRoot;
-  let targetCommit;
-  const statusArgs = [
-    "status",
-    "--porcelain=v1",
-    "--untracked-files=all",
-    "--",
-    ":/",
-    ":(top,exclude).claude/worktrees/**"
-  ];
   try {
-    sourceProjectRoot = await resolveReviewerGitToplevel(options.projectRoot);
-    const { stdout: statusStdout } = await execFileAsync3("git", statusArgs, {
-      cwd: sourceProjectRoot
-    });
-    if (statusStdout !== "") {
-      const diagnostic = [
-        "Reviewer target status gate failed:",
-        "git status --porcelain=v1 --untracked-files=all -- :/ ':(top,exclude).claude/worktrees/**'",
-        statusStdout
-      ].join("\n");
-      return dualFailure(new Error(diagnostic));
-    }
-    const { stdout: targetStdout } = await execFileAsync3(
-      "git",
-      ["rev-parse", "--verify", "HEAD^{commit}"],
-      { cwd: sourceProjectRoot }
-    );
-    targetCommit = targetStdout.trim();
+    const sourceProjectRoot = await resolveReviewerGitToplevel(options.projectRoot);
     await execFileAsync3(
       "git",
       ["rev-parse", "--verify", `${options.baseRevision}^{commit}`],
@@ -20905,55 +20878,10 @@ async function summonParallelReviewerLenses(options) {
     summon("completeness"),
     summon("correctness")
   ]);
-  let results = {
+  return {
     completeness: settled[0].status === "fulfilled" ? settled[0].value : failedResult(settled[0].reason),
     correctness: settled[1].status === "fulfilled" ? settled[1].value : failedResult(settled[1].reason)
   };
-  let sealDiagnostic;
-  try {
-    const { stdout: headBeforeStdout } = await execFileAsync3(
-      "git",
-      ["rev-parse", "--verify", "HEAD^{commit}"],
-      { cwd: sourceProjectRoot }
-    );
-    const { stdout: sealedStatusStdout } = await execFileAsync3("git", statusArgs, {
-      cwd: sourceProjectRoot
-    });
-    const { stdout: headAfterStdout } = await execFileAsync3(
-      "git",
-      ["rev-parse", "--verify", "HEAD^{commit}"],
-      { cwd: sourceProjectRoot }
-    );
-    const headBefore = headBeforeStdout.trim();
-    const headAfter = headAfterStdout.trim();
-    if (headBefore !== targetCommit || headAfter !== targetCommit || sealedStatusStdout !== "") {
-      sealDiagnostic = [
-        "Reviewer target final seal failed:",
-        `HEAD before status => ${headBefore}`,
-        `HEAD after status => ${headAfter}`,
-        `expected PRE_HEAD ${targetCommit}`,
-        "git status --porcelain=v1 --untracked-files=all -- :/ ':(top,exclude).claude/worktrees/**'",
-        sealedStatusStdout
-      ].join("\n");
-    }
-  } catch (error) {
-    sealDiagnostic = `Reviewer target final seal failed:
-${serializeThrownValue(error)}`;
-  }
-  if (sealDiagnostic !== void 0) {
-    const withSealFailure = (result) => ({
-      ...result,
-      exitCode: 1,
-      stderr: [result.stderr, sealDiagnostic].filter(
-        (text) => typeof text === "string" && text !== ""
-      ).join("\n")
-    });
-    results = {
-      completeness: withSealFailure(results.completeness),
-      correctness: withSealFailure(results.correctness)
-    };
-  }
-  return results;
 }
 async function summonGateOfficer(options) {
   let home = options.home;

package/dist/public-role-summons.js

@@ -298,35 +298,8 @@ async function summonParallelReviewerLenses(options) {
     const failure = failedResult(error);
     return { completeness: failure, correctness: failure };
   };
-  let sourceProjectRoot;
-  let targetCommit;
-  const statusArgs = [
-    "status",
-    "--porcelain=v1",
-    "--untracked-files=all",
-    "--",
-    ":/",
-    ":(top,exclude).claude/worktrees/**"
-  ];
   try {
-    sourceProjectRoot = await resolveReviewerGitToplevel(options.projectRoot);
-    const { stdout: statusStdout } = await execFileAsync("git", statusArgs, {
-      cwd: sourceProjectRoot
-    });
-    if (statusStdout !== "") {
-      const diagnostic = [
-        "Reviewer target status gate failed:",
-        "git status --porcelain=v1 --untracked-files=all -- :/ ':(top,exclude).claude/worktrees/**'",
-        statusStdout
-      ].join("\n");
-      return dualFailure(new Error(diagnostic));
-    }
-    const { stdout: targetStdout } = await execFileAsync(
-      "git",
-      ["rev-parse", "--verify", "HEAD^{commit}"],
-      { cwd: sourceProjectRoot }
-    );
-    targetCommit = targetStdout.trim();
+    const sourceProjectRoot = await resolveReviewerGitToplevel(options.projectRoot);
     await execFileAsync(
       "git",
       ["rev-parse", "--verify", `${options.baseRevision}^{commit}`],
@@ -362,55 +335,10 @@ async function summonParallelReviewerLenses(options) {
     summon("completeness"),
     summon("correctness")
   ]);
-  let results = {
+  return {
     completeness: settled[0].status === "fulfilled" ? settled[0].value : failedResult(settled[0].reason),
     correctness: settled[1].status === "fulfilled" ? settled[1].value : failedResult(settled[1].reason)
   };
-  let sealDiagnostic;
-  try {
-    const { stdout: headBeforeStdout } = await execFileAsync(
-      "git",
-      ["rev-parse", "--verify", "HEAD^{commit}"],
-      { cwd: sourceProjectRoot }
-    );
-    const { stdout: sealedStatusStdout } = await execFileAsync("git", statusArgs, {
-      cwd: sourceProjectRoot
-    });
-    const { stdout: headAfterStdout } = await execFileAsync(
-      "git",
-      ["rev-parse", "--verify", "HEAD^{commit}"],
-      { cwd: sourceProjectRoot }
-    );
-    const headBefore = headBeforeStdout.trim();
-    const headAfter = headAfterStdout.trim();
-    if (headBefore !== targetCommit || headAfter !== targetCommit || sealedStatusStdout !== "") {
-      sealDiagnostic = [
-        "Reviewer target final seal failed:",
-        `HEAD before status => ${headBefore}`,
-        `HEAD after status => ${headAfter}`,
-        `expected PRE_HEAD ${targetCommit}`,
-        "git status --porcelain=v1 --untracked-files=all -- :/ ':(top,exclude).claude/worktrees/**'",
-        sealedStatusStdout
-      ].join("\n");
-    }
-  } catch (error) {
-    sealDiagnostic = `Reviewer target final seal failed:
-${serializeThrownValue(error)}`;
-  }
-  if (sealDiagnostic !== void 0) {
-    const withSealFailure = (result) => ({
-      ...result,
-      exitCode: 1,
-      stderr: [result.stderr, sealDiagnostic].filter(
-        (text) => typeof text === "string" && text !== ""
-      ).join("\n")
-    });
-    results = {
-      completeness: withSealFailure(results.completeness),
-      correctness: withSealFailure(results.correctness)
-    };
-  }
-  return results;
 }
 async function summonGateOfficer(options) {
   let home = options.home;

package/package.json

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

package/src/public-role-summons.ts

@@ -554,7 +554,7 @@ function withReviewerLens(
   return [...argv.slice(0, separator), "--lens", lens, ...argv.slice(separator)];
 }
 
-/** Git toplevel for Reviewer target status / seal checks (subdir-safe). */
+/** Git toplevel for Reviewer pre-dispatch --base existence check (subdir-safe). */
 async function resolveReviewerGitToplevel(projectRoot: string): Promise<string> {
   const callerProjectRoot = await realpath(projectRoot);
   return await realpath((await execFileAsync(
@@ -570,6 +570,10 @@ async function resolveReviewerGitToplevel(projectRoot: string): Promise<string>
  * caller's public argv and only adds `--lens` (10a). No seat-specific ephemeral
  * copy — same shared tree as other seats. Lifecycle stays in this shared
  * summons seam, never in the role module (#946 / ADR 0018).
+ *
+ * Work-tree dirt and mid-batch HEAD drift are not package hard-stops (#1133):
+ * each ordinary Reviewer leg judges and reports object state itself; this seam
+ * records original child terminals as returned.
  */
 export async function summonParallelReviewerLenses(options: {
   /** Public argv after the role token; must not already carry `--lens`. */
@@ -606,37 +610,10 @@ export async function summonParallelReviewerLenses(options: {
     return { completeness: failure, correctness: failure } as const;
   };
 
-  // Pre-dispatch target checks use git toplevel (subdir-safe). Every failure
-  // keeps the existing dual-child batch surface.
-  let sourceProjectRoot: string;
-  let targetCommit: string;
-  const statusArgs = [
-    "status",
-    "--porcelain=v1",
-    "--untracked-files=all",
-    "--",
-    ":/",
-    ":(top,exclude).claude/worktrees/**",
-  ] as const;
+  // Pre-dispatch: only verify typed --base resolves (subdir-safe). Missing base
+  // keeps the dual-child batch surface. Status / HEAD are not package gates.
   try {
-    sourceProjectRoot = await resolveReviewerGitToplevel(options.projectRoot);
-    const { stdout: statusStdout } = await execFileAsync("git", statusArgs, {
-      cwd: sourceProjectRoot,
-    });
-    if (statusStdout !== "") {
-      const diagnostic = [
-        "Reviewer target status gate failed:",
-        "git status --porcelain=v1 --untracked-files=all -- :/ ':(top,exclude).claude/worktrees/**'",
-        statusStdout,
-      ].join("\n");
-      return dualFailure(new Error(diagnostic));
-    }
-    const { stdout: targetStdout } = await execFileAsync(
-      "git",
-      ["rev-parse", "--verify", "HEAD^{commit}"],
-      { cwd: sourceProjectRoot },
-    );
-    targetCommit = targetStdout.trim();
+    const sourceProjectRoot = await resolveReviewerGitToplevel(options.projectRoot);
     // Typed base from the public parse — covers --base value and --base=value alike.
     // Child legs still carry the original argv token unchanged (10a).
     await execFileAsync(
@@ -682,7 +659,7 @@ export async function summonParallelReviewerLenses(options: {
     summon("completeness"),
     summon("correctness"),
   ]);
-  let results = {
+  return {
     completeness: settled[0].status === "fulfilled"
       ? settled[0].value
       : failedResult(settled[0].reason),
@@ -690,55 +667,6 @@ export async function summonParallelReviewerLenses(options: {
       ? settled[1].value
       : failedResult(settled[1].reason),
   };
-
-  let sealDiagnostic: string | undefined;
-  try {
-    const { stdout: headBeforeStdout } = await execFileAsync(
-      "git",
-      ["rev-parse", "--verify", "HEAD^{commit}"],
-      { cwd: sourceProjectRoot },
-    );
-    const { stdout: sealedStatusStdout } = await execFileAsync("git", statusArgs, {
-      cwd: sourceProjectRoot,
-    });
-    const { stdout: headAfterStdout } = await execFileAsync(
-      "git",
-      ["rev-parse", "--verify", "HEAD^{commit}"],
-      { cwd: sourceProjectRoot },
-    );
-    const headBefore = headBeforeStdout.trim();
-    const headAfter = headAfterStdout.trim();
-    if (
-      headBefore !== targetCommit
-      || headAfter !== targetCommit
-      || sealedStatusStdout !== ""
-    ) {
-      sealDiagnostic = [
-        "Reviewer target final seal failed:",
-        `HEAD before status => ${headBefore}`,
-        `HEAD after status => ${headAfter}`,
-        `expected PRE_HEAD ${targetCommit}`,
-        "git status --porcelain=v1 --untracked-files=all -- :/ ':(top,exclude).claude/worktrees/**'",
-        sealedStatusStdout,
-      ].join("\n");
-    }
-  } catch (error) {
-    sealDiagnostic = `Reviewer target final seal failed:\n${serializeThrownValue(error)}`;
-  }
-  if (sealDiagnostic !== undefined) {
-    const withSealFailure = (result: PublicSummonResult): PublicSummonResult => ({
-      ...result,
-      exitCode: 1,
-      stderr: [result.stderr, sealDiagnostic].filter(
-        (text): text is string => typeof text === "string" && text !== "",
-      ).join("\n"),
-    });
-    results = {
-      completeness: withSealFailure(results.completeness),
-      correctness: withSealFailure(results.correctness),
-    };
-  }
-  return results;
 }
 
 /** Gate officer summons: notary/auditor via --source-run; inspector via pointer; countersign via parentRunPath (#969 / #987). */