@akagilnc/pi-workflow-roles 0.1.2107 → 0.1.2114

package/README.md

@@ -11,7 +11,7 @@ pi install npm:@akagilnc/pi-workflow-roles
 export PATH="$HOME/.pi/agent/npm/node_modules/.bin:$PATH"
 ```
 
-Update with `pi update npm:@akagilnc/pi-workflow-roles`—never a second global `npm install -g`. Inspect with `ak-role roles` and `ak-role help <role>`; set per-seat model defaults with `ak-role config set judge openai-codex/gpt-5.6-sol:high`; set or clear a persistent labor engine (judge|reviewer) with `ak-role config set-engine <seat> <name>` / `ak-role config unset-engine <seat>`.
+Update with `pi update npm:@akagilnc/pi-workflow-roles`—never a second global `npm install -g`. Inspect with `ak-role roles` and `ak-role help <role>`; set per-seat model defaults with `ak-role config set judge openai-codex/gpt-5.6-sol:high`; set or clear a persistent labor engine (callable roles) with `ak-role config set-engine <seat> <name>` / `ak-role config unset-engine <seat>`.
 
 ## Reading results
 
@@ -31,16 +31,16 @@ Every run also prepares Navigator advice in the same Terminal. Configure it like
 
 ```bash
 ak-role config set navigator openai-codex/gpt-5.6-luna:medium
-# persistent labor engine (judge|reviewer only); one-shot override remains --engine
+# persistent labor engine (callable roles; not navigator); one-shot override remains --engine
 ak-role config set-engine judge opus
 ak-role config unset-engine judge
 ```
 
-`config set` stores the seat model default; `config set-engine` / `unset-engine` store or clear the persistent labor-engine name on judge|reviewer only (same seats as `--engine`). Usage and refusal text are owned by `ak-role config` in the public CLI.
+`config set` stores the seat model default; `config set-engine` / `unset-engine` store or clear the persistent labor-engine name on callable roles (same seats as `--engine`; navigator refused — no independent activation). Usage and refusal text are owned by `ak-role config` in the public CLI.
 
 Receipts are typed, so callers compose roles without parsing prose; ordering and stopping stay caller-owned. Programmatic consumers derive contracts from the exported schemas in `src/package-contracts/`, not from this guide.
 
-When a Judge/Reviewer labor-engine detour fails and the seat continues the labor on the main road, the typed receipt may carry a mechanical `engineLaborFallback` field: `{ engine, failure, laborBy: "seat" }`. It appears only after a real detour failure that fell back to seat labor (including package-owned idle timeout on the detour tool)—not on detour success or caller cancel. First failure wins for the activation; model-forged `engineLaborFallback` keys are stripped unless the package latch recorded one. Sole producer: `src/engine-labor-fallback.ts`; decision record: [ADR 0071](docs/adr/0071-engine-detour-failure-seat-fallback-declaration.md). This README only projects that contract.
+When a labor-engine detour fails and the seat continues the labor on the main road, the typed receipt may carry a mechanical `engineLaborFallback` field: `{ engine, failure, laborBy: "seat" }`. It appears only after a real detour failure that fell back to seat labor (including package-owned idle timeout on the detour tool)—not on detour success or caller cancel. First failure wins for the activation; model-forged `engineLaborFallback` keys are stripped unless the package latch recorded one. Sole producer: `src/engine-labor-fallback.ts`; decision record: [ADR 0071](docs/adr/0071-engine-detour-failure-seat-fallback-declaration.md). This README only projects that contract.
 
 ## Call the roles
 
@@ -121,7 +121,7 @@ Generated from `src/public-cli/option-definitions.ts`. Prefer `ak-role help <com
 | --- | --- | --- | --- | --- | --- | --- | --- |
 | `--model` | — | `provider/model` | no | no | option | — | Override the effective seat model for this invocation (before or after the command). |
 | `--thinking` | — | `level` | no | no | option | — | Override thinking level: off\|minimal\|low\|medium\|high\|xhigh\|max. |
-| `--engine` | — | `name` | no | no | option | — | Optional Judge/Reviewer labor engine for this invocation (owner pool-directive name; packaged notes attached when present; judge+reviewer only). |
+| `--engine` | — | `name` | no | no | option | — | Optional labor engine for this invocation (owner pool-directive name; packaged notes attached when present; any role). |
 | `--help` | `-h` | — | no | no | option | — | Show public CLI help and exit. |
 
 ### `judge`

package/README.zh-CN.md

@@ -11,7 +11,7 @@ pi install npm:@akagilnc/pi-workflow-roles
 export PATH="$HOME/.pi/agent/npm/node_modules/.bin:$PATH"
 ```
 
-更新用 `pi update npm:@akagilnc/pi-workflow-roles`——勿另起全局 `npm install -g`。查看能力：`ak-role roles`、`ak-role help <role>`；设席位模型默认：`ak-role config set judge openai-codex/gpt-5.6-sol:high`；设或清持久劳务引擎（仅 judge|reviewer）：`ak-role config set-engine <seat> <name>` / `ak-role config unset-engine <seat>`。
+更新用 `pi update npm:@akagilnc/pi-workflow-roles`——勿另起全局 `npm install -g`。查看能力：`ak-role roles`、`ak-role help <role>`；设席位模型默认：`ak-role config set judge openai-codex/gpt-5.6-sol:high`；设或清持久劳务引擎（可调用角色）：`ak-role config set-engine <seat> <name>` / `ak-role config unset-engine <seat>`。
 
 ## 读结果
 
@@ -31,16 +31,16 @@ ak-role judge --attach ./plan.md "Review this plan." > result.txt
 
 ```bash
 ak-role config set navigator openai-codex/gpt-5.6-luna:medium
-# 持久劳务引擎（仅 judge|reviewer）；一次性覆盖仍用 --engine
+# 持久劳务引擎（可调用角色；不含 navigator）；一次性覆盖仍用 --engine
 ak-role config set-engine judge opus
 ak-role config unset-engine judge
 ```
 
-`config set` 存席位模型默认；`config set-engine` / `unset-engine` 在 judge|reviewer 上写入或清除持久劳务引擎名（与 `--engine` 同轴）。用法与拒绝文案以公开 CLI 的 `ak-role config` 为准。
+`config set` 存席位模型默认；`config set-engine` / `unset-engine` 在可调用角色上写入或清除持久劳务引擎名（与 `--engine` 同轴；拒收 navigator——无独立 activation）。用法与拒绝文案以公开 CLI 的 `ak-role config` 为准。
 
 回执是 typed 的，调用者不必解析散文即可组合角色；顺序与停止归调用者。编程消费者从 `src/package-contracts/` 导出推导契约，不从本文。
 
-当 Judge/Reviewer 劳务引擎绕行失败、座席回到主路继续劳务时，typed 回执可带机械字段 `engineLaborFallback`：`{ engine, failure, laborBy: "seat" }`。仅在真实绕行失败并座席顶班后出现（含 detour 工具命中 package-owned idle 超时）——成功绕行或调用方 cancel 不出现。同一次 activation 内先到先得；无包内 latch 时剥离模型伪造的 `engineLaborFallback` 键。唯一构造点：`src/engine-labor-fallback.ts`；决策记录：[ADR 0071](docs/adr/0071-engine-detour-failure-seat-fallback-declaration.md)。本文只投影该契约。
+当劳务引擎绕行失败、座席回到主路继续劳务时，typed 回执可带机械字段 `engineLaborFallback`：`{ engine, failure, laborBy: "seat" }`。仅在真实绕行失败并座席顶班后出现（含 detour 工具命中 package-owned idle 超时）——成功绕行或调用方 cancel 不出现。同一次 activation 内先到先得；无包内 latch 时剥离模型伪造的 `engineLaborFallback` 键。唯一构造点：`src/engine-labor-fallback.ts`；决策记录：[ADR 0071](docs/adr/0071-engine-detour-failure-seat-fallback-declaration.md)。本文只投影该契约。
 
 ## 调用百官
 
@@ -150,7 +150,7 @@ Codex fast 档：开启：`echo "fast_mode = on" > ~/.pi-codex-fast`；关闭：
 | --- | --- | --- | --- | --- | --- | --- | --- |
 | `--model` | — | `provider/model` | 否 | 否 | option | — | 覆盖本调用有效席位模型（可置于子命令前或后）。 |
 | `--thinking` | — | `level` | 否 | 否 | option | — | 覆盖 thinking 档位：off\|minimal\|low\|medium\|high\|xhigh\|max。 |
-| `--engine` | — | `name` | 否 | 否 | option | — | 本调用可选 Judge/Reviewer 劳动引擎（池令名字；有包内调法笔记则附卷；仅 Judge+Reviewer）。 |
+| `--engine` | — | `name` | 否 | 否 | option | — | 本调用可选劳动引擎（池令名字；有包内调法笔记则附卷；全部角色可用）。 |
 | `--help` | `-h` | — | 否 | 否 | option | — | 显示公开 CLI 帮助并退出。 |
 
 ### `judge`

package/dist/engine-detour-tool.js

@@ -114,7 +114,7 @@ export function createEngineDetourToolDefinition(input) {
     });
 }
 /**
- * Register the engine-generic detour tool once for this process when Judge/Reviewer has
+ * Register the engine-generic detour tool once for this process when any role has
  * an engine activation signal. Returns whether registration occurred.
  * Once-latch is activation-scoped via the returned reset handle.
  */

package/dist/engine-detour.js

@@ -6,8 +6,21 @@
 import { spawn } from "node:child_process";
 /** Package-owned detour tool name (settlement whitelist + session principal). */
 export const ENGINE_DETOUR_TOOL_NAME = "ak_engine_detour";
-/** Env presence/name signal injected by public Judge run (registration gate only). */
+/** Env presence/name signal injected by public role runs (registration gate only). */
 export const AK_ROLE_ENGINE_ENV = "AK_ROLE_ENGINE";
+/**
+ * Sole AK_ROLE_ENGINE write seam for public role child env (#391 E2).
+ * Delete ambient inheritance first; own-key undefined mask survives process.env re-merge.
+ */
+export function applyEngineChildEnv(childEnv, engine) {
+    delete childEnv[AK_ROLE_ENGINE_ENV];
+    if (engine !== undefined && engine.trim() !== "") {
+        childEnv[AK_ROLE_ENGINE_ENV] = engine.trim();
+    }
+    else {
+        childEnv[AK_ROLE_ENGINE_ENV] = undefined;
+    }
+}
 export const ENGINE_DETOUR_EMPTY_STDOUT_DIAGNOSTIC = "engine detour produced empty stdout";
 export const ENGINE_DETOUR_ALREADY_USED_DIAGNOSTIC = "engine detour already used in this activation";
 function abortReasonError(signal) {

package/dist/engine-labor-fallback.js

@@ -46,7 +46,7 @@ export function withEngineLaborFallbackField(receipt, field) {
     const { engineLaborFallback: _forged, ...rest } = receipt;
     return rest;
 }
-/** Install the activation-scoped latch (Judge/Reviewer session_start). */
+/** Install the activation-scoped latch (any role session_start with engine). */
 export function installActivationEngineLaborFallbackLatch(latch) {
     activationLatch = latch;
 }