@akagilnc/pi-workflow-roles 0.1.2525 → 0.1.2537

package/README.md

@@ -1,6 +1,6 @@
 # @akagilnc/pi-workflow-roles
 
-Packaged workflow roles for [Pi](https://pi.dev): `judge`, `fixer`, `coder`, `reviewer`, `collector`, `doctor`, `merger`, `notary`, `analyst`. 中文说明见 [README.zh-CN.md](https://github.com/Akagilnc/ak-pi-workflow-roles/blob/main/README.zh-CN.md)。
+Packaged workflow roles for [Pi](https://pi.dev): `judge`, `fixer`, `coder`, `reviewer`, `collector`, `doctor`, `merger`, `notary`, `inspector`, `analyst`. 中文说明见 [README.zh-CN.md](https://github.com/Akagilnc/ak-pi-workflow-roles/blob/main/README.zh-CN.md)。
 
 ## Install
 
@@ -48,7 +48,7 @@ Seat and Gate-officer configuration:
 ```bash
 ak-role config set judge <provider/model[:thinking]>
 ak-role config set navigator <provider/model[:thinking]>
-# Gate officers (automatic on submission; not caller commands except direct notary)
+# Gate officers (automatic on submission; inspector and notary also have direct commands)
 ak-role config set gatekeeper <provider/model[:thinking]>
 ak-role config set inspector <provider/model[:thinking]>
 ak-role config set notary <provider/model[:thinking]>
@@ -95,6 +95,9 @@ ak-role merger --project /path/to/worktree "Reconcile the active merge."
 # notary — document-fidelity check on one retained source run; one-shot
 ak-role notary --source-run <runId@role|path>
 
+# inspector — direct complexity and test-quality check
+ak-role inspector --attach ./change.patch "Review this material."
+
 # analyst — deterministic metrics; bare call = whole book
 ak-role analyst
 

package/dist/inspector-contracts.js

@@ -0,0 +1 @@
+export const INSPECTOR_OUTPUT_TOOL_NAME = "ak_inspector_output";

package/dist/package-contracts/terminating-tools.js

@@ -6,6 +6,7 @@ import { COLLECTOR_ACCEPTED_TEXT, COLLECTOR_OUTPUT_TOOL, validateAcceptedCollect
 import { JUDGE_ACCEPTED_TEXT, JUDGE_OUTPUT_TOOL_NAME, validateAcceptedJudgeDetails, } from "./judge-output.js";
 import { REVIEWER_ACCEPTED_TEXT, REVIEWER_OUTPUT_TOOL_NAME, projectReviewerIntentToReceipt, validateReviewerIntent, validateRuntimeReviewerReceipt, } from "./reviewer-output.js";
 import { isAuditEscalationResult } from "../audit-escalation.js";
+import { INSPECTOR_OUTPUT_TOOL_NAME } from "../inspector-contracts.js";
 import { DOCTOR_ACCEPTED_TEXT, DOCTOR_OUTPUT_TOOL_NAME, validateDoctorSubmissionShape, validateRecordedDoctorOutput } from "../doctor-contracts.js";
 import { MERGER_ACCEPTED_TEXT, MERGER_OUTPUT_TOOL_NAME, validateMergerOutput } from "../merger-contracts.js";
 import { NOTARY_ACCEPTED_TEXT, NOTARY_OUTPUT_TOOL_NAME, validateRecordedNotaryOutput } from "../notary-contracts.js";
@@ -20,6 +21,7 @@ export const TERMINATING_TOOL_NAMES = [
     DOCTOR_OUTPUT_TOOL_NAME,
     MERGER_OUTPUT_TOOL_NAME,
     NOTARY_OUTPUT_TOOL_NAME,
+    INSPECTOR_OUTPUT_TOOL_NAME,
 ];
 export function isTerminatingToolName(name) {
     return TERMINATING_TOOL_NAMES.includes(name);
@@ -42,6 +44,8 @@ export function acceptedTextFor(toolName) {
             return MERGER_ACCEPTED_TEXT;
         case NOTARY_OUTPUT_TOOL_NAME:
             return NOTARY_ACCEPTED_TEXT;
+        case INSPECTOR_OUTPUT_TOOL_NAME:
+            return "给事中回执已受理";
     }
 }
 export class AcceptedDetailsContractError extends Error {
@@ -82,6 +86,7 @@ export function validateAcceptedDetails(toolName, details) {
         [DOCTOR_OUTPUT_TOOL_NAME]: ["completed", "refused"],
         [MERGER_OUTPUT_TOOL_NAME]: ["completed", "escalate"],
         [NOTARY_OUTPUT_TOOL_NAME]: ["pass", "bounce"],
+        [INSPECTOR_OUTPUT_TOOL_NAME]: ["pass", "bounce"],
     };
     const collectorDiscriminator = toolName === COLLECTOR_OUTPUT_TOOL && Array.isArray(candidate?.groups);
     const baseDiscriminator = discriminator;
@@ -109,6 +114,8 @@ export function validateAcceptedDetails(toolName, details) {
                 return validateMergerOutput(details);
             case NOTARY_OUTPUT_TOOL_NAME:
                 return validateRecordedNotaryOutput(details);
+            case INSPECTOR_OUTPUT_TOOL_NAME:
+                return candidate;
         }
     }
     catch (error) {
@@ -146,7 +153,8 @@ export function acceptedFacts(toolName, details) {
         case FIXER_OUTPUT_TOOL_NAME:
         case REVIEWER_OUTPUT_TOOL_NAME:
         case DOCTOR_OUTPUT_TOOL_NAME:
-        case NOTARY_OUTPUT_TOOL_NAME: return { status: details.status };
+        case NOTARY_OUTPUT_TOOL_NAME:
+        case INSPECTOR_OUTPUT_TOOL_NAME: return { status: details.status };
         case JUDGE_OUTPUT_TOOL_NAME: return { status: details.judgeStatus };
         case MERGER_OUTPUT_TOOL_NAME: {
             const output = details;

package/dist/packaged-role-registry.js

@@ -5,6 +5,7 @@ import { CODER_OUTPUT_TOOL_NAME, FIXER_OUTPUT_TOOL_NAME } from "./package-contra
 import { DOCTOR_OUTPUT_TOOL_NAME } from "./doctor-contracts.js";
 import { MERGER_OUTPUT_TOOL_NAME } from "./merger-contracts.js";
 import { NOTARY_OUTPUT_TOOL_NAME } from "./notary-contracts.js";
+import { INSPECTOR_OUTPUT_TOOL_NAME } from "./inspector-contracts.js";
 const PACKAGED_ROLE_REGISTRY = [
   { role: "judge", phases: [null], outputTool: JUDGE_OUTPUT_TOOL_NAME, inputFlag: void 0, phaseFlag: void 0, activationStage: "load-and-install" },
   { role: "fixer", phases: ["plan", "apply"], outputTool: FIXER_OUTPUT_TOOL_NAME, inputFlag: "ak-fix-packet", phaseFlag: "ak-fixer-phase", activationStage: "load-and-install" },
@@ -14,7 +15,8 @@ const PACKAGED_ROLE_REGISTRY = [
   { role: "collector", phases: [null], outputTool: COLLECTOR_OUTPUT_TOOL, inputFlag: void 0, phaseFlag: void 0, activationStage: "load-and-install" },
   { role: "doctor", phases: [null], outputTool: DOCTOR_OUTPUT_TOOL_NAME, inputFlag: "ak-doctor-case", phaseFlag: void 0, activationStage: "load-and-install" },
   { role: "merger", phases: [null], outputTool: MERGER_OUTPUT_TOOL_NAME, inputFlag: "ak-merger-input", phaseFlag: void 0, activationStage: "prepare-git-and-install" },
-  { role: "notary", phases: [null], outputTool: NOTARY_OUTPUT_TOOL_NAME, inputFlag: "ak-notary-source-run", phaseFlag: void 0, activationStage: "load-and-install" }
+  { role: "notary", phases: [null], outputTool: NOTARY_OUTPUT_TOOL_NAME, inputFlag: "ak-notary-source-run", phaseFlag: void 0, activationStage: "load-and-install" },
+  { role: "inspector", phases: [null], outputTool: INSPECTOR_OUTPUT_TOOL_NAME, inputFlag: void 0, phaseFlag: void 0, activationStage: "load-and-install" }
 ];
 function packagedRoleMetadata(role) {
   return PACKAGED_ROLE_REGISTRY.find((entry) => entry.role === role);

package/dist/public-cli/config.js

@@ -67,21 +67,21 @@ export function setPersistentSeatConfig(config, seat, selection) {
 /**
  * Clear a gate officer's persistent model override (#453).
  * Scope is GateOfficerSeat only — non-province seats have no destructive clear seam.
- * Only notary may retain an engine-only residual so direct notary activation keeps
+ * Directly callable gate officers may retain an engine-only residual so activation keeps
  * its labor engine while model resolution returns to startup / province inheritance.
- * gatekeeper/inspector drop the whole row. Already-absent seats are a no-op.
+ * Gatekeeper drops the whole row. Already-absent seats are a no-op.
  */
 export function clearPersistentSeatConfig(config, seat) {
     const previous = config.seats[seat];
     if (previous === undefined)
         return config;
-    // Engine-only residual ownership is notary-only — never widen to other officers.
-    if (seat === "notary" && previous.engine !== undefined) {
+    // Direct officer engine ownership survives an independent model clear.
+    if ((seat === "notary" || seat === "inspector") && previous.engine !== undefined) {
         return {
             ...config,
             seats: {
                 ...config.seats,
-                notary: { engine: previous.engine },
+                [seat]: { engine: previous.engine },
             },
         };
     }
@@ -111,8 +111,8 @@ export function isEngineAxisSeat(seat) {
 }
 /**
  * Set or clear persistent engine on a callable role seat (#356 / #378 / #391 / #453).
- * First engine still requires an existing seat row (model, or notary engine residual).
- * Clearing engine from a notary engine-only residual drops the empty row; clearing
+ * First engine still requires an existing seat row (model, or direct-officer engine residual).
+ * Clearing engine from an engine-only residual drops the empty row; clearing
  * engine from a model+engine row leaves model-only. Seat type is PublicCallableRole
  * (navigator excluded at the type boundary).
  */
@@ -297,10 +297,10 @@ function parseSeatModelConfig(value, seat) {
         // validatePublicCliConfigEngines → assertLegalEngineName (single authority).
         throw new Error(`config seat ${seat} engine must be a string`);
     }
-    // #453: engine-only residual is legal only for notary after model clear.
+    // Direct officer engine-only residual remains legal after model clear.
     // All other seats keep the baseline provider/model required contract.
     if (!hasProvider) {
-        if (seat === "notary" && typeof raw.engine === "string") {
+        if ((seat === "notary" || seat === "inspector") && typeof raw.engine === "string") {
             if (raw.thinking !== undefined) {
                 throw new Error(`config seat ${seat} thinking requires provider/model`);
             }