@akagilnc/pi-workflow-roles 0.1.5241 → 0.1.5252

package/dist/public-cli/option-definitions.js

@@ -429,13 +429,12 @@ const COLLECTOR_OPTIONS = [
         canonical: "--pr",
         aliases: [],
         valueMetavar: "number",
-        // #676 D1: optional when context uniquely determines the PR; ambiguous → require explicit.
         required: false,
         repeatable: false,
         form: "option",
         description: {
-            en: "Positive GitHub pull request number. Optional when unique branch/HEAD association binds the PR, or when the Collector role can decide the target from task materials via bind-target; multi-candidate git context or a role that cannot decide requires an explicit value.",
-            zh: "正整数 GitHub PR 号。分支/HEAD 唯一关联可绑定时可省略；亦可由通进司从任务材料经 bind-target 判定。git 多候选或角色无法判定时必须显式提供。",
+            en: "Positive GitHub pull request number. Optional when the Collector role locates the PR via host CLI from task materials.",
+            zh: "正整数 GitHub PR 号。可选；省略时由通进司经宿主 CLI 从任务材料自行定位。",
         },
     },
     {
@@ -462,22 +461,8 @@ const COLLECTOR_OPTIONS = [
         repeatable: false,
         form: "option",
         description: {
-            en: "Optional request manifest JSON path ({requests:[{id,body}]}). Role may also decide request bodies from handbook/field activity without this file.",
-            zh: "可选 request manifest JSON 路径（{requests:[{id,body}]}）。角色亦可依手册/现场自行判定请求正文，不必依赖本文件。",
-        },
-    },
-    {
-        id: "wait-ms",
-        owner: "collector",
-        canonical: "--wait-ms",
-        aliases: [],
-        valueMetavar: "ms",
-        required: false,
-        repeatable: false,
-        form: "option",
-        description: {
-            en: "Wait-window duration in milliseconds after the work step opens (default 600000 = 10 minutes). Does not require a code change to adjust.",
-            zh: "工作步骤开启后的等待窗时长（毫秒；默认 600000＝十分钟）。改配置即可，无需改代码。",
+            en: "Optional request manifest JSON path ({requests:[{id,body}]}). Parsed for semantics and passed to the Collector as materials — not a machine collection directive.",
+            zh: "可选 request manifest JSON 路径（{requests:[{id,body}]}）。按语义解析后作为材料递给通进司，不是机器代收 finding 的指令。",
         },
     },
 ];

package/dist/public-cli/settlement.js

@@ -17,18 +17,16 @@ import { isV1ResumableProvider, readLatestTypedProviderHttpObservation, readType
 import { AUDITOR_COMPLIANCE_FAILURE_ENTRY_TYPE, AUDITOR_PARENT_ATTEMPT_BINDING_ENTRY_TYPE, COMPLIANCE_RESPONSE_ENTRY_TYPE, readComplianceCandidate, } from "../compliance-transport.js";
 // COMPLIANCE_RESPONSE_ENTRY_TYPE remains for boundRetainedAuditResponse (call/result
 // interval binding on historical session bytes). Provider-stop retain authority is Sitian.
-import { COLLECTOR_BIND_TARGET_TOOL, COLLECTOR_OBSERVE_TOOL, COLLECTOR_READ_TOOL, COLLECTOR_REQUEST_TOOL, COLLECTOR_WAIT_TOOL, } from "../collector-ledger.js";
-import { ENGINE_DETOUR_TOOL_NAME } from "../engine-detour.js";
+import { ENGINE_DETOUR_TOOL_NAME, } from "../engine-detour.js";
 import { projectEngineDetourToolUsageForPublicTerminal, readEngineDetourToolUsage, readInvocationEngineMounted, runDirectoryFromSessionDirectory, sessionFileFromSessionDirectory, withEngineDetourToolUsageFact, } from "../engine-detour-usage.js";
 import { JUDGE_OUTPUT_TOOL_NAME, } from "../package-contracts/judge-output.js";
-import { COLLECTOR_OUTPUT_TOOL, } from "../package-contracts/collector-output.js";
 import { CODER_OUTPUT_TOOL_NAME, FIXER_OUTPUT_TOOL_NAME, } from "../package-contracts/worker-output.js";
 import { DOCTOR_OUTPUT_TOOL_NAME, } from "../doctor-contracts.js";
 import { DOCTOR_CANDIDATE_ENTRY_TYPE } from "../dossier-resolution.js";
 import { REVIEWER_OUTPUT_TOOL_NAME, } from "../package-contracts/reviewer-output.js";
-import { packagedAuditToolName, packagedDurableOfficerEntry, packagedProjectsTargetBindRejection, packagedRoleAcceptedOutputTool, packagedRoleMetadata, packagedSkipsGateOnInfrastructureStage, } from "../packaged-role-registry.js";
+import { packagedAuditToolName, packagedDurableOfficerEntry, packagedRoleAcceptedOutputTool, packagedRoleMetadata, packagedSkipsGateOnInfrastructureStage, } from "../packaged-role-registry.js";
 import { SECRETARIAT_COUNTERSIGN_TERMINAL_FACT_KEY, SECRETARIAT_GATE_OFFICER_ENTRY_TYPE, } from "../secretariat-contracts.js";
-import { classifyPackagedRoleTerminalResult, findLatestDurablePackagedRoleTerminal, hasNavigatorInfrastructureFailureBase, isAcceptedPackagedRoleTerminalResult, isReceiptSettlementBindingClear, NAVIGATOR_INVOCATION_ENTRY, NAVIGATOR_ROUTE_PLAYBOOK_FAILURE_ENTRY, parseInvocationMarkerIdentity, } from "../navigator-invocation-identity.js";
+import { classifyPackagedRoleTerminalResult, findLatestDurablePackagedRoleTerminal, hasNavigatorInfrastructureFailureBase, isAcceptedPackagedRoleTerminalResult, NAVIGATOR_ROUTE_PLAYBOOK_FAILURE_ENTRY, } from "../navigator-invocation-identity.js";
 import { NO_RECEIPT_LIFECYCLE_ENTRY_TYPE, RECEIPT_DELIVERY_TURN_LIMIT, noReceiptLifecycleFacts, parseNoReceiptLifecycleFacts, } from "../receipt-delivery-policy.js";
 import { roleRunArtifactsDirectory } from "../role-run-placement.js";
 import { rewriteRunDirectoryPathValue } from "../role-run-relocation.js";
@@ -1037,35 +1035,6 @@ function toolResultText(message) {
         .join("")
         .trim();
 }
-/**
- * #676 D1/J3: durable bind-target rejection on the current attempt → public no_receipt facts.
- * Callers must learn they need explicit --pr without opening the session 正本.
- * Authority is the latest bind-target toolResult only — a later success clears an earlier failure.
- */
-function extractCollectorTargetBindRejection(entries) {
-    for (let i = entries.length - 1; i >= 0; i -= 1) {
-        const entry = entries[i];
-        if (entry?.type !== "message")
-            continue;
-        const message = entry.message;
-        if (message?.role !== "toolResult")
-            continue;
-        if (message.toolName !== COLLECTOR_BIND_TARGET_TOOL)
-            continue;
-        // Latest bind result wins: success means the attempt is no longer rejected.
-        if (message.isError !== true)
-            return undefined;
-        const diagnostic = toolResultText(message);
-        if (diagnostic.length === 0)
-            return undefined;
-        const details = message.details;
-        const code = isRecord(details) && typeof details.code === "string" && details.code.trim() !== ""
-            ? details.code
-            : undefined;
-        return code === undefined ? { diagnostic } : { diagnostic, code };
-    }
-    return undefined;
-}
 function boundErroredToolCandidate(entries, resultIndex, message, toolName) {
     if (message.toolName !== toolName || message.isError !== true)
         return undefined;
@@ -1075,21 +1044,6 @@ function boundErroredToolCandidate(entries, resultIndex, message, toolName) {
         ? undefined
         : { candidate: bound.candidate, diagnostic, callIndex: bound.callIndex };
 }
-/** Collector operational tools that fail closed via host infrastructure abort. */
-const COLLECTOR_INFRASTRUCTURE_TOOLS = new Set([
-    COLLECTOR_OBSERVE_TOOL,
-    COLLECTOR_READ_TOOL,
-    COLLECTOR_REQUEST_TOOL,
-    COLLECTOR_WAIT_TOOL,
-]);
-const COLLECTOR_INFRASTRUCTURE_FAILURE_SPEC = {
-    matchTool: (toolName) => COLLECTOR_INFRASTRUCTURE_TOOLS.has(toolName),
-    cause: "activation",
-    identityName: "CollectorInfrastructureError",
-    // read alone has a correctable rejection mode (unknown/non-openable pointers);
-    // only its typed infrastructure-failure fact counts as a real host failure.
-    requireInfrastructureFact: (toolName) => toolName === COLLECTOR_READ_TOOL,
-};
 const ENGINE_DETOUR_INFRASTRUCTURE_FAILURE_SPEC = {
     matchTool: (toolName) => toolName === ENGINE_DETOUR_TOOL_NAME,
     cause: "output",
@@ -1146,21 +1100,6 @@ async function readInfrastructureToolFailure(sessionFile, spec, options = {}) {
         return undefined;
     }
 }
-/**
- * Prefer a real Collector infrastructure tool failure already on the session
- * principal over a later secondary provider-stop (failure-honesty).
- * Observe/request/wait host failures keep their diagnostic identity (e.g. HTTP 404).
- */
-export function extractCollectorInfrastructureFailure(entries) {
-    return extractInfrastructureToolFailure(entries, COLLECTOR_INFRASTRUCTURE_FAILURE_SPEC);
-}
-/** Read the bound session principal for a Collector infrastructure tool failure. */
-export async function readCollectorInfrastructureFailure(sessionFile) {
-    return readInfrastructureToolFailure(sessionFile, COLLECTOR_INFRASTRUCTURE_FAILURE_SPEC, 
-    // Multi-attempt resume: only a current-attempt infrastructure failure
-    // may preempt the current failure cause (#633).
-    { currentAttemptOnly: true });
-}
 /**
  * Prefer a real engine-detour infrastructure tool failure already on the session
  * principal over a later secondary provider-stop (failure-honesty / #357 T2).
@@ -1189,10 +1128,6 @@ const RUNNER_FAILURE_POLICY = {
         read: readEngineDetourInfrastructureFailure,
         rank: "record-first",
     },
-    "collector-known-first": {
-        read: readCollectorInfrastructureFailure,
-        rank: "known-first",
-    },
 };
 /**
  * Runner-failure rank for one seat. The composition-root `runnerFailure`
@@ -1418,19 +1353,6 @@ export async function appendRunAttemptHistory(source, outcome) {
     }
     catch { }
 }
-/**
- * Minimal attendance provenance against the bound marker (ADR 0043).
- * Keep only invocationId + post-terminal ordering. Runtime-self-produced
- * role/phase/subject/version are not re-reconciled here (ADR 0042).
- */
-function navigatorAttendanceCorrelatedWithBoundMarker(details, attendanceIndex, terminalIndex, marker) {
-    if (attendanceIndex <= terminalIndex)
-        return false;
-    // Exact current invocation token is the bound marker principal.
-    if (details.invocationId !== marker.invocationId)
-        return false;
-    return true;
-}
 function parseNavigatorAttendanceDetails(details) {
     const disposition = details.disposition;
     const advisoryDiagnostic = typeof details.routePlaybookReadFailure === "string"
@@ -1651,10 +1573,9 @@ export function extractNavigatorFact(entries) {
     return message === undefined ? fact : { ...fact, advisoryDiagnostic: message };
 }
 function extractNavigatorAttendanceFact(entries) {
-    // Affirmative attendance only. Missing / uncorrelated / unparseable is never no-advice.
-    // Minimal provenance: latest durable terminal + nearest preceding marker +
-    // invocationId + post-terminal order. Marker↔terminal cardinality belongs to
-    // receipt settlement (isReceiptSettlementBindingClear), not attendance extraction.
+    // Affirmative attendance only. Missing / unparseable is never no-advice.
+    // The shared lifecycle places the settled event on this submission's closure;
+    // an unrelated attendance message cannot supply another call's result.
     const terminal = findLatestDurablePackagedRoleTerminal(entries);
     if (terminal === undefined) {
         return {
@@ -1663,49 +1584,14 @@ function extractNavigatorAttendanceFact(entries) {
             reason: "Navigator attendance has no durable packaged role terminal",
         };
     }
-    let markerIndex = -1;
-    for (let i = terminal.index - 1; i >= 0; i -= 1) {
+    for (let i = terminal.index; i < entries.length; i += 1) {
         const entry = entries[i];
-        if (entry?.type === "custom" && entry.customType === NAVIGATOR_INVOCATION_ENTRY) {
-            markerIndex = i;
+        if (entry?.type !== "custom" || entry.customType !== "ak-role-submission-closure")
+            continue;
+        const details = isRecord(entry.data) ? entry.data.navigator : undefined;
+        if (!isRecord(details))
             break;
-        }
-    }
-    if (markerIndex < 0) {
-        return {
-            disposition: "unavailable",
-            source: "unknown",
-            reason: "Navigator attendance is uncorrelated with session invocation facts",
-        };
-    }
-    const marker = parseInvocationMarkerIdentity(entries[markerIndex]?.data);
-    if (marker === undefined) {
-        return {
-            disposition: "unavailable",
-            source: "unknown",
-            reason: "Navigator attendance is uncorrelated with session invocation facts",
-        };
-    }
-    for (let i = entries.length - 1; i >= 0; i -= 1) {
-        const entry = entries[i];
-        if (entry?.type === "custom_message" && entry.customType === "ak-navigator-attendance") {
-            const details = entry.message?.details ?? entry.details;
-            if (!isRecord(details)) {
-                return {
-                    disposition: "unavailable",
-                    source: "unknown",
-                    reason: "Navigator attendance is unparseable",
-                };
-            }
-            if (!navigatorAttendanceCorrelatedWithBoundMarker(details, i, terminal.index, marker)) {
-                return {
-                    disposition: "unavailable",
-                    source: "unknown",
-                    reason: "Navigator attendance is uncorrelated with session invocation facts",
-                };
-            }
-            return parseNavigatorAttendanceDetails(details);
-        }
+        return parseNavigatorAttendanceDetails(details);
     }
     // Absence is not successful no-advice — require affirmative typed attendance.
     return {
@@ -2441,18 +2327,6 @@ export async function settleFailureTerminalResult(admitted, failure, authority,
                     facts.runPointer === admitted.runDirectory &&
                     facts.attemptPointer === `current:${admitted.runDirectory}`) {
                     let decisiveFacts = facts;
-                    // #676 D1/J3: project durable bind-target rejection onto public no_receipt facts.
-                    if (packagedProjectsTargetBindRejection(admitted.role)) {
-                        const bindRejection = extractCollectorTargetBindRejection(entries.slice(attemptStart));
-                        if (bindRejection !== undefined) {
-                            decisiveFacts = {
-                                ...facts,
-                                targetBindRejected: true,
-                                targetBindDiagnostic: bindRejection.diagnostic,
-                                ...(bindRejection.code === undefined ? {} : { targetBindCode: bindRejection.code }),
-                            };
-                        }
-                    }
                     // #478: no_receipt is still a public Terminal — project accepted gate facts.
                     // #953: empty public terminal face — clear every reader-adoptable prior face.
                     await clearOppositeTerminalArtifactFace(admitted.runDirectory);

package/package.json

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

package/resources/collector-bot-handbook.md

@@ -4,6 +4,8 @@
 内容是角色阅读与修正的工作记忆，不是代码状态规则；临时额度/故障不得固化为永久行为。
 参与者依本仓使用与现场活动辨认，不把下列名称当作全仓固定名单。
 
+#1088：用宿主 CLI（如 `gh`）自行取证与请求；代码不代收、不选择、不归并 finding。取证受限时如实报告。
+
 ## Codex（hosted GitHub connector）
 
 - 官方：https://learn.chatgpt.com/docs/third-party/github
@@ -29,9 +31,9 @@
 
 ## 工序
 
-判定任务材料后绑定本仓唯一 PR（prNumber，或由 issueNumber 解析到唯一 PR）。已有显式 PR 绑定时不必再绑。多义或无法确定时不猜，交给调用方明确目标。未绑定前不观察。观察把证据存成不可变快照；上下文里的正文只到头部，其余按 evidenceId 开卷。请求发在所引最新快照 HEAD 上；清单外的 requestId 要带上正文。交卷交 findings 指针，不把模板通知当成 finding；没有 finding 就不要编一条。未完成写现场原因，不写成没问题。
+从任务材料与 `--pr` / `--request-manifest` 材料判定目标后，用宿主 CLI 读取有评审的 PR，按各 reviewer 已记录的完成形态收集 findings，整理并交卷。未绑定 PR 时用 `gh` 等自行定位；无法确定时不猜，交给调用方明确目标。交卷交按机器身份归组的出席、材料与 findings；没有 finding 就不要编一条。未完成写现场原因，不写成没问题。
 
 ## 修正规则
 
-- 优先引用官方文档；现场差异记入仓库差异条并附 PR/证据指针
+- 优先引用官方文档；现场差异附 PR/证据指针
 - 未知或暂未响应 ≠ 通过或确定死亡

package/src/collector-config.ts

@@ -1,7 +1,6 @@
 import { createHash } from "node:crypto";
 import { readFile } from "node:fs/promises";
 
-export const COLLECTOR_HOST = "github.com" as const;
 export const COLLECTOR_OWNER_PATTERN = /^[A-Za-z0-9](?:[A-Za-z0-9-]{0,37}[A-Za-z0-9])?$/;
 export const COLLECTOR_REPO_PATTERN = /^[A-Za-z0-9](?:[A-Za-z0-9._-]{0,98}[A-Za-z0-9])?$/;