@akagilnc/pi-workflow-roles 0.1.2086 → 0.1.2089

package/dist/public-cli/main.js

@@ -14439,9 +14439,12 @@ function setPersistentSeatConfig(config, seat, selection) {
     }
   };
 }
+function isEngineAxisSeat(seat) {
+  return seat === "judge" || seat === "reviewer";
+}
 function setPersistentSeatEngine(config, seat, engine) {
-  if (seat !== "judge") {
-    throw new Error(`engine axis is judge-only; refused seat ${seat}`);
+  if (!isEngineAxisSeat(seat)) {
+    throw new Error(`engine axis is judge+reviewer only; refused seat ${seat}`);
   }
   const previous = config.seats[seat];
   if (previous === void 0) {
@@ -14472,9 +14475,9 @@ function validatePublicCliConfigEngines(config, _packageRoot) {
   for (const seat of Object.keys(config.seats)) {
     const row = config.seats[seat];
     if (row?.engine === void 0) continue;
-    if (seat !== "judge") {
+    if (!isEngineAxisSeat(seat)) {
       throw new Error(
-        `config seat ${seat} engine is not allowed; engine axis is judge-only`
+        `config seat ${seat} engine is not allowed; engine axis is judge+reviewer only`
       );
     }
     try {
@@ -14609,13 +14612,13 @@ function pickStartupCandidate(seat, credentials) {
   return void 0;
 }
 function attachEngineAxis(seat, config, invocation) {
-  if (seat.seat !== "judge") {
+  if (!isEngineAxisSeat(seat.seat)) {
     return {
       ...seat,
       engineSource: "unconfigured"
     };
   }
-  const persistentEngine = config.seats.judge?.engine;
+  const persistentEngine = config.seats[seat.seat]?.engine;
   if (invocation?.engine !== void 0) {
     return {
       ...seat,
@@ -15917,8 +15920,8 @@ var init_option_definitions = __esm({
         repeatable: false,
         form: "option",
         description: {
-          en: "Optional Judge labor engine for this invocation (owner pool-directive name; packaged notes attached when present; judge-only).",
-          zh: "\u672C\u8C03\u7528\u53EF\u9009 Judge \u52B3\u52A8\u5F15\u64CE\uFF08\u6C60\u4EE4\u540D\u5B57\uFF1B\u6709\u5305\u5185\u8C03\u6CD5\u7B14\u8BB0\u5219\u9644\u5377\uFF1B\u4EC5 Judge\uFF09\u3002"
+          en: "Optional Judge/Reviewer labor engine for this invocation (owner pool-directive name; packaged notes attached when present; judge+reviewer only).",
+          zh: "\u672C\u8C03\u7528\u53EF\u9009 Judge/Reviewer \u52B3\u52A8\u5F15\u64CE\uFF08\u6C60\u4EE4\u540D\u5B57\uFF1B\u6709\u5305\u5185\u8C03\u6CD5\u7B14\u8BB0\u5219\u9644\u5377\uFF1B\u4EC5 Judge+Reviewer\uFF09\u3002"
         }
       },
       {
@@ -17494,11 +17497,12 @@ async function admitReviewerInvocation(options) {
     ...ticketFields
   };
 }
-function buildReviewerTransportPrompt(admitted) {
-  return [
+function buildReviewerTransportPrompt(admitted, engineMaterial) {
+  const lines = [
     `Base revision for the fixed review target: ${admitted.baseRevision}`,
     "Use this exact revision as the fixed review point."
-  ].join("\n");
+  ];
+  return appendEngineSessionMaterial(lines, engineMaterial).join("\n");
 }
 function parseMergerArgv(args) {
   const attachmentPaths = [];
@@ -19156,6 +19160,16 @@ var init_auditor_dossier_tool = __esm({
   }
 });
 
+// src/engine-detour.ts
+var ENGINE_DETOUR_TOOL_NAME, AK_ROLE_ENGINE_ENV;
+var init_engine_detour = __esm({
+  "src/engine-detour.ts"() {
+    "use strict";
+    ENGINE_DETOUR_TOOL_NAME = "ak_engine_detour";
+    AK_ROLE_ENGINE_ENV = "AK_ROLE_ENGINE";
+  }
+});
+
 // src/stream-idle-guard.ts
 var init_stream_idle_guard = __esm({
   "src/stream-idle-guard.ts"() {
@@ -19214,6 +19228,26 @@ var init_package_owned_tool_idle = __esm({
   }
 });
 
+// src/engine-detour-tool.ts
+var engineDetourArgsSchema;
+var init_engine_detour_tool = __esm({
+  "src/engine-detour-tool.ts"() {
+    "use strict";
+    init_build();
+    init_engine_detour();
+    init_package_owned_tool_idle();
+    engineDetourArgsSchema = typebox_exports.Object(
+      {
+        argv: typebox_exports.Array(typebox_exports.String({ minLength: 1 }), {
+          minItems: 1,
+          description: "Executable argv for one engine subprocess. First element is the command (PATH lookup); remaining elements are arguments. Build argv from the host CLI actual interface for the configured engine name; when optional packaged notes are present in the session prompt, follow those bytes. Do not invent package flags."
+        })
+      },
+      { additionalProperties: false }
+    );
+  }
+});
+
 // src/receipt-delivery-policy.ts
 function isRecord4(value) {
   return typeof value === "object" && value !== null && !Array.isArray(value);
@@ -19249,6 +19283,9 @@ var init_evidence_child_executor = __esm({
   "src/evidence-child-executor.ts"() {
     "use strict";
     init_compliance_transport();
+    init_engine_detour_tool();
+    init_engine_detour();
+    init_engine_material();
     init_package_owned_tool_idle();
     init_receipt_delivery_policy();
     init_reviewer_construction();
@@ -19403,16 +19440,6 @@ var init_collector_ledger = __esm({
   }
 });
 
-// src/engine-detour.ts
-var ENGINE_DETOUR_TOOL_NAME, AK_ROLE_ENGINE_ENV;
-var init_engine_detour = __esm({
-  "src/engine-detour.ts"() {
-    "use strict";
-    ENGINE_DETOUR_TOOL_NAME = "ak_engine_detour";
-    AK_ROLE_ENGINE_ENV = "AK_ROLE_ENGINE";
-  }
-});
-
 // src/work-subject-identity.ts
 var init_work_subject_identity = __esm({
   "src/work-subject-identity.ts"() {
@@ -24360,7 +24387,10 @@ function buildReviewerTicketNumberArgs(ticketNumber) {
   return ticketNumber === void 0 ? [] : ["--ak-review-ticket-number", String(ticketNumber)];
 }
 function buildReviewerActivationExtraArgs(admitted, options) {
-  const prompt = buildReviewerTransportPrompt(admitted);
+  const prompt = buildReviewerTransportPrompt(
+    admitted,
+    engineSessionMaterialFromOptions(options)
+  );
   const skillPath = resolvePackagedMethodSkillPath(
     options.packageRoot,
     "code-review"
@@ -24468,7 +24498,7 @@ async function presentControlledFailure8(admitted, failureInput, io) {
   };
 }
 async function dispatchAdmittedReviewer(input) {
-  const { admitted, env, io, extraArgs, lease, methodMaterial } = input;
+  const { admitted, env, io, extraArgs, lease, methodMaterial, effectiveEngine } = input;
   try {
     const missingCredential = missingCredentialPreDispatchFailure(
       env.model,
@@ -24481,7 +24511,7 @@ async function dispatchAdmittedReviewer(input) {
         io
       );
     }
-    await markRunRunning(admitted.runDirectory, env.model);
+    await markRunRunning(admitted.runDirectory, env.model, effectiveEngine);
     await clearTypedProviderHttpObservation(admitted.runDirectory);
     await clearReviewerDispatchRejection(admitted.runDirectory);
     const childEnv = {
@@ -24490,6 +24520,12 @@ async function dispatchAdmittedReviewer(input) {
       PI_CODING_AGENT_DIR: env.agentDir,
       AK_ROLE_RUN_DIR: admitted.runDirectory
     };
+    delete childEnv[AK_ROLE_ENGINE_ENV];
+    if (env.engine !== void 0 && env.engine.trim() !== "") {
+      childEnv[AK_ROLE_ENGINE_ENV] = env.engine.trim();
+    } else {
+      childEnv[AK_ROLE_ENGINE_ENV] = void 0;
+    }
     const correlationId = admitted.correlationId ?? env.correlationId;
     if (correlationId !== void 0 && correlationId.trim() !== "") {
       childEnv.AK_CORRELATION_ID = correlationId;
@@ -24571,13 +24607,20 @@ async function dispatchAdmittedReviewer(input) {
         terminal: auditIncomplete
       };
     }
+    const infrastructureFailure = await readEngineDetourInfrastructureFailure(
+      admitted.sessionFile
+    );
     const credentialFailure = postRunMissingCredentialFailure(
       result2,
       env.model,
       env.credentials
     );
     const resolution = await resolveAuditedRunnerFailureResolution({
-      runner: result2.knownFailure,
+      runner: infrastructureFailure === void 0 ? result2.knownFailure : {
+        cause: infrastructureFailure.cause,
+        diagnostic: infrastructureFailure.diagnostic,
+        ...infrastructureFailure.identity === void 0 ? {} : { identity: infrastructureFailure.identity }
+      },
       sessionFile: admitted.sessionFile,
       credential: credentialFailure,
       runDirectory: admitted.runDirectory
@@ -24651,6 +24694,7 @@ async function runPublicReviewer(argv, env, io, parseReviewerArgv2) {
   const extraArgs = buildReviewerActivationExtraArgs(admitted, {
     packageRoot: env.packageRoot,
     ...env.model === void 0 ? {} : { model: env.model },
+    ...env.engine === void 0 ? {} : { engine: env.engine },
     ...env.extraPiArgs === void 0 ? {} : { extraPiArgs: env.extraPiArgs }
   });
   return await dispatchAdmittedReviewer({
@@ -24662,7 +24706,9 @@ async function runPublicReviewer(argv, env, io, parseReviewerArgv2) {
     io,
     extraArgs,
     lease,
-    methodMaterial
+    methodMaterial,
+    // #378: only initial Reviewer dispatch records mechanical engine provenance.
+    ...env.engine === void 0 ? {} : { effectiveEngine: env.engine }
   });
 }
 async function runPublicReviewerResume(argv, env, io) {
@@ -24738,6 +24784,8 @@ async function runPublicReviewerResume(argv, env, io) {
 var init_reviewer_run = __esm({
   "src/public-cli/reviewer-run.ts"() {
     "use strict";
+    init_engine_detour();
+    init_engine_material();
     init_method_skill();
     init_explicit_internal();
     init_cli_errors();
@@ -27025,7 +27073,7 @@ function renderHelp() {
     "",
     "Role options: ak-role help <command>",
     "Persistent config: ak-role config set <seat> <provider/model:thinking>",
-    "Persistent engine (judge only): ak-role config set-engine judge <name> | unset-engine judge",
+    "Persistent engine (judge|reviewer): ak-role config set-engine <seat> <name> | unset-engine <seat>",
     "Effective seats: ak-role roles"
   );
   return `${lines.join("\n")}
@@ -27130,7 +27178,7 @@ async function runConfigCommand(args, home, packageRoot2, io) {
   if (args[0] === "set-engine") {
     if (args.length !== 3) {
       throw new CliUsageError(
-        "usage: ak-role config set-engine judge <name>"
+        "usage: ak-role config set-engine <judge|reviewer> <name>"
       );
     }
     const seat = args[1];
@@ -27138,9 +27186,9 @@ async function runConfigCommand(args, home, packageRoot2, io) {
     if (!isPublicConfigurableSeat(seat)) {
       throw new CliUsageError(`unknown configurable seat: ${seat}`);
     }
-    if (seat !== "judge") {
+    if (!isEngineAxisSeat(seat)) {
       throw new CliUsageError(
-        `engine axis is judge-only; refused seat ${seat}`
+        `engine axis is judge+reviewer only; refused seat ${seat}`
       );
     }
     requireLegalEngineName(name);
@@ -27160,16 +27208,16 @@ async function runConfigCommand(args, home, packageRoot2, io) {
   if (args[0] === "unset-engine") {
     if (args.length !== 2) {
       throw new CliUsageError(
-        "usage: ak-role config unset-engine judge"
+        "usage: ak-role config unset-engine <judge|reviewer>"
       );
     }
     const seat = args[1];
     if (!isPublicConfigurableSeat(seat)) {
       throw new CliUsageError(`unknown configurable seat: ${seat}`);
     }
-    if (seat !== "judge") {
+    if (!isEngineAxisSeat(seat)) {
       throw new CliUsageError(
-        `engine axis is judge-only; refused seat ${seat}`
+        `engine axis is judge+reviewer only; refused seat ${seat}`
       );
     }
     let config = await loadAndValidateConfig(home, packageRoot2);
@@ -27195,9 +27243,9 @@ async function runAkRole(argv, env) {
     const parsed = parseArgv(argv);
     if (parsed.engine !== void 0) {
       requireLegalEngineName(parsed.engine);
-      if (!parsed.help && parsed.command !== void 0 && parsed.command !== "help" && parsed.command !== "judge") {
+      if (!parsed.help && parsed.command !== void 0 && parsed.command !== "help" && !isEngineAxisSeat(parsed.command)) {
         throw new CliUsageError(
-          `engine axis is judge-only; refused command ${parsed.command}`
+          `engine axis is judge+reviewer only; refused command ${parsed.command}`
         );
       }
     }
@@ -27528,6 +27576,7 @@ async function runAkRole(argv, env) {
           ...env.correlationId === void 0 ? {} : { correlationId: env.correlationId },
           ...env.piRunner === void 0 ? {} : { piRunner: env.piRunner },
           ...seat.selection === void 0 ? {} : { model: seat.selection },
+          ...seat.engine === void 0 ? {} : { engine: seat.engine },
           ...env.reviewerExtraPiArgs === void 0 ? {} : { extraPiArgs: env.reviewerExtraPiArgs },
           ...env.reviewerTimeoutMs === void 0 ? {} : { timeoutMs: env.reviewerTimeoutMs },
           ...env.createRunId === void 0 ? {} : { createRunId: env.createRunId }

package/dist/reviewer-agent.js

@@ -73,6 +73,9 @@ export function createReviewerAgentRunner(dependencies = {}) {
                         ...(dependencies.credentialScratchParent === undefined
                             ? {}
                             : { credentialScratchParent: dependencies.credentialScratchParent }),
+                        ...(dependencies.packageRoot === undefined
+                            ? {}
+                            : { packageRoot: dependencies.packageRoot }),
                     });
                     const disposition = await workspaceOwner.dispose(workspace);
                     return [leg.axis, Object.freeze({ status: "successful", report: child.report, usage: child.usage, target: batch.target, prompt: child.prompt, workspaceDisposition: disposition })];

package/extensions/role-runtime.ts

@@ -242,7 +242,7 @@ export async function loadNavigatorWorkContext(
 }
 
 export default function roleRuntime(pi: ExtensionAPI): void {
-  const reviewerAgent = createReviewerAgentRunner();
+  const reviewerAgent = createReviewerAgentRunner({ packageRoot });
   registerNavigatorModelCommand(pi);
   const navigatorSessionFactory = createNativeNavigatorSessionFactory();
   // #351: static provider list from extension setting (default ["kimi-coding"]).

package/package.json

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

package/src/engine-detour-tool.ts

@@ -1,12 +1,13 @@
 /**
- * Package-owned engine detour tool (#357 T2).
- * Registered by shared role-runtime when Judge + engine activation signal is present.
- * Forbidden in judge-role.ts (lifecycle ban — no spawn in role modules).
+ * Package-owned engine detour tool (#357 T2 / #378).
+ * Registered by shared role-runtime when Judge|Reviewer + engine activation signal is present.
+ * Evidence-child legs install the same definition via customTools (no spawn in role modules).
  */
 import type {
   AgentToolResult,
   ExtensionAPI,
   ExtensionContext,
+  ToolDefinition,
 } from "@earendil-works/pi-coding-agent";
 import { Type, type Static } from "typebox";
 
@@ -18,6 +19,7 @@ import {
   isEngineDetourFailure,
   runEngineDetourOnce,
 } from "./engine-detour.ts";
+import { wrapPackageOwnedToolDefinition } from "./package-owned-tool-idle.ts";
 
 const engineDetourArgsSchema = Type.Object(
   {
@@ -45,28 +47,21 @@ export type EngineDetourToolRegistration = {
   readonly registered: boolean;
 };
 
+type EngineDetourLatch = { used: boolean };
+
 /**
- * Register the engine-generic detour tool once for this process when Judge has
- * an engine activation signal. Returns whether registration occurred.
- * Once-latch is activation-scoped via the returned reset handle.
+ * Build one once-latch detour tool definition for a configured engine name.
+ * `latch` is shared so parent registration can reset between activations.
+ * `fail` owns host abort (parent) vs throw (evidence child).
  */
-export function registerEngineDetourTool(
-  pi: ExtensionAPI,
-  hostActions: EngineDetourHostActions,
-): EngineDetourToolRegistration & { resetLatch(): void } {
-  const engineName = engineNameFromEnv();
-  if (engineName === undefined) {
-    return {
-      registered: false,
-      resetLatch() {
-        /* no-op when unregistered */
-      },
-    };
-  }
-
-  let used = false;
-
-  pi.registerTool({
+export function createEngineDetourToolDefinition(input: {
+  engineName: string;
+  latch?: EngineDetourLatch;
+  fail: (error: Error, toolCallId: string, ctx: ExtensionContext) => never;
+}): ToolDefinition {
+  const latch = input.latch ?? { used: false };
+  const engineName = input.engineName;
+  return wrapPackageOwnedToolDefinition({
     name: ENGINE_DETOUR_TOOL_NAME,
     label: "Engine Detour",
     description:
@@ -75,7 +70,7 @@ export function registerEngineDetourTool(
     promptGuidelines: [
       `Use ${ENGINE_DETOUR_TOOL_NAME} exactly once for the configured engine (${engineName}). Optional packaged notes are guidance when present; a bare engine name alone is also a valid call path.`,
       "Pass argv for the host CLI of this engine name — first element is the executable name on PATH. Follow optional packaged notes when delivered; otherwise act from the engine name and the host CLI actual interface. Do not invent package flags.",
-      "On success, use the returned stdout as labor content for the existing typed submission tool.",
+      "On success, use the returned stdout as labor content for the existing typed submission / report path.",
     ],
     parameters: engineDetourArgsSchema,
     async execute(
@@ -85,22 +80,22 @@ export function registerEngineDetourTool(
       _onUpdate,
       ctx,
     ): Promise<AgentToolResult<unknown>> {
-      if (used) {
-        hostActions.failInfrastructure(
+      if (latch.used) {
+        input.fail(
           new Error(ENGINE_DETOUR_ALREADY_USED_DIAGNOSTIC),
-          ctx,
           toolCallId,
+          ctx,
         );
       }
-      used = true;
+      latch.used = true;
 
       const args = params as EngineDetourArgs;
       const argv = Array.isArray(args.argv) ? args.argv : [];
       if (argv.length === 0 || argv.some((part) => typeof part !== "string" || part.length === 0)) {
-        hostActions.failInfrastructure(
+        input.fail(
           new Error("engine detour argv must be a non-empty string array"),
-          ctx,
           toolCallId,
+          ctx,
         );
       }
 
@@ -112,14 +107,18 @@ export function registerEngineDetourTool(
           ...(signal === undefined ? {} : { signal }),
         });
       } catch (error) {
-        hostActions.failInfrastructure(error, ctx, toolCallId);
+        input.fail(
+          error instanceof Error ? error : new Error(String(error)),
+          toolCallId,
+          ctx,
+        );
       }
 
       if (isEngineDetourFailure(result)) {
-        hostActions.failInfrastructure(
+        input.fail(
           new Error(engineDetourFailureDiagnostic(result)),
-          ctx,
           toolCallId,
+          ctx,
         );
       }
 
@@ -131,12 +130,42 @@ export function registerEngineDetourTool(
         },
       };
     },
+  }) as ToolDefinition;
+}
+
+/**
+ * Register the engine-generic detour tool once for this process when Judge/Reviewer has
+ * an engine activation signal. Returns whether registration occurred.
+ * Once-latch is activation-scoped via the returned reset handle.
+ */
+export function registerEngineDetourTool(
+  pi: ExtensionAPI,
+  hostActions: EngineDetourHostActions,
+): EngineDetourToolRegistration & { resetLatch(): void } {
+  const engineName = engineNameFromEnv();
+  if (engineName === undefined) {
+    return {
+      registered: false,
+      resetLatch() {
+        /* no-op when unregistered */
+      },
+    };
+  }
+
+  const latch: EngineDetourLatch = { used: false };
+  const definition = createEngineDetourToolDefinition({
+    engineName,
+    latch,
+    fail(error, toolCallId, ctx) {
+      hostActions.failInfrastructure(error, ctx, toolCallId);
+    },
   });
+  pi.registerTool(definition);
 
   return {
     registered: true,
     resetLatch() {
-      used = false;
+      latch.used = false;
     },
   };
 }

package/src/evidence-child-executor.ts

@@ -29,6 +29,13 @@ import {
   prepareComplianceDispatch,
   type AuditorParentAttemptBinding,
 } from "./compliance-transport.ts";
+import { createEngineDetourToolDefinition } from "./engine-detour-tool.ts";
+import { engineNameFromEnv } from "./engine-detour.ts";
+import {
+  appendEngineSessionMaterial,
+  engineSessionMaterialFromOptions,
+  type EngineSessionMaterial,
+} from "./package-resources/engine-material.ts";
 import { wrapPackageOwnedToolDefinition } from "./package-owned-tool-idle.ts";
 import { createReceiptDeliveryPolicy, NO_RECEIPT_LIFECYCLE_ENTRY_TYPE, RECEIPT_DELIVERY_PROMPT, type NoReceiptLifecycleFacts } from "./receipt-delivery-policy.ts";
 import { REVIEWER_VERIFICATION_BOUNDARY } from "./reviewer-construction.ts";
@@ -41,13 +48,16 @@ import {
 } from "./upstream-error-testimony.ts";
 
 /** Package-owned system prompt for Reviewer Standards/Spec evidence children (private carrier). */
-function buildEvidenceChildSystemPrompt(): string {
-  return [
+function buildEvidenceChildSystemPrompt(
+  engineMaterial?: EngineSessionMaterial,
+): string {
+  const lines = [
     "Work only in the supplied workspace.",
     "Use the available evidence tools to investigate. Do not commit, push, or mutate remotes.",
     REVIEWER_VERIFICATION_BOUNDARY,
     "Return one substantive non-blank report.",
-  ].join("\n");
+  ];
+  return appendEngineSessionMaterial(lines, engineMaterial).join("\n");
 }
 
 // ── shared constants / types ──────────────────────────────────────────────
@@ -540,6 +550,11 @@ export type EvidenceChildExecuteOptions = Readonly<{
   signal?: AbortSignal;
   /** Parent directory for credential/config scratch. Defaults to os.tmpdir(). */
   credentialScratchParent?: string;
+  /**
+   * Package root for optional engine method-material resolution (#378).
+   * Required only when AK_ROLE_ENGINE is set and packaged notes should attach.
+   */
+  packageRoot?: string;
 }>;
 
 export async function executeEvidenceChild(
@@ -568,6 +583,32 @@ export async function executeEvidenceChild(
       } catch (error) {
         throw classifiedError(error, "provider");
       }
+      // #378: when labor engine is configured, legs get the same detour tool + material
+      // dual-path as the parent seat (ADR 0069 detour-rejoins-main-road).
+      const engineName = engineNameFromEnv();
+      const engineMaterial =
+        engineName === undefined
+          ? undefined
+          : options.packageRoot === undefined || options.packageRoot.trim() === ""
+            // Name-only when package root is unavailable (still a valid #376 path).
+            ? Object.freeze({ name: engineName })
+            : engineSessionMaterialFromOptions({
+                engine: engineName,
+                packageRoot: options.packageRoot,
+              });
+      // #378: detour failure is durable for this leg — later assistant reports cannot wash it.
+      let engineDetourFailure: Error | undefined;
+      const engineDetourTool =
+        engineName === undefined
+          ? undefined
+          : createEngineDetourToolDefinition({
+              engineName,
+              fail(error) {
+                const failure = error instanceof Error ? error : new Error(String(error));
+                engineDetourFailure = failure;
+                throw failure;
+              },
+            });
       // No tools allowlist — Pi defaults + unrestricted evidence surface (ADR 0064).
       // Single createAgentSession owner: in-process-session.ts.
       const { session, dispose } = await openInProcessAgentSession({
@@ -576,7 +617,10 @@ export async function executeEvidenceChild(
         model: inherited.model,
         thinkingLevel: context.thinkingLevel ?? "off",
         modelRuntime: inherited.runtime,
-        systemPrompt: buildEvidenceChildSystemPrompt(),
+        systemPrompt: buildEvidenceChildSystemPrompt(engineMaterial),
+        ...(engineDetourTool === undefined
+          ? {}
+          : { customTools: [engineDetourTool] }),
         sessionManager: createRecordSession({
           cwd: workspace,
           kind: "evidence-children",
@@ -598,8 +642,18 @@ export async function executeEvidenceChild(
         try {
           await session.prompt(delivered);
         } catch (error) {
+          // Engine detour fail becomes isError toolResult and must outrank a later
+          // provider-shaped throw from the same prompt turn (#378).
+          if (engineDetourFailure !== undefined) {
+            throw classifiedError(engineDetourFailure, "child");
+          }
           throw classifiedError(error, "provider");
         }
+        // Launched-leg detour non-zero / empty stdout / spawn failure is infrastructure:
+        // reject the leg even when the model still emits a non-blank report afterward.
+        if (engineDetourFailure !== undefined) {
+          throw classifiedError(engineDetourFailure, "child");
+        }
         if (signal?.aborted) throw new Error("Evidence child was cancelled");
         const lastAssistant = [...session.messages]
           .reverse()