@akagilnc/pi-workflow-roles 0.1.4403 → 0.1.4419

package/dist/migrate-book-topology.js

@@ -14526,18 +14526,32 @@ var init_option_definitions = __esm({
           zh: "\u5FC5\u586B\uFF1B\u9489\u4F4F\u5BA1\u67E5\u76EE\u6807\u7684 fixed-point revision\u3002"
         }
       },
+      {
+        id: "lens",
+        owner: "reviewer",
+        canonical: "--lens",
+        aliases: [],
+        valueMetavar: "completeness|correctness",
+        required: true,
+        repeatable: false,
+        form: "option",
+        description: {
+          en: "Required single review lens: completeness or correctness (no default).",
+          zh: "\u5FC5\u586B\u5355 lens\uFF1Acompleteness \u6216 correctness\uFF08\u65E0\u9ED8\u8BA4\u503C\uFF09\u3002"
+        }
+      },
       {
         id: "authority-ref",
         owner: "reviewer",
         canonical: "--authority-ref",
         aliases: [],
         valueMetavar: "ref",
-        required: false,
+        required: true,
         repeatable: true,
         form: "option",
         description: {
-          en: "Durable authority reference/URL (repeatable; refs only, not inline prose).",
-          zh: "\u6301\u4E45 authority \u5F15\u7528/URL\uFF08\u53EF\u91CD\u590D\uFF1B\u4EC5 ref\uFF0C\u975E\u5185\u8054\u6563\u6587\uFF09\u3002"
+          en: "Required durable authority reference/URL (repeatable; refs only, not inline prose).",
+          zh: "\u5FC5\u586B\u6301\u4E45 authority \u5F15\u7528/URL\uFF08\u53EF\u91CD\u590D\uFF1B\u4EC5 ref\uFF0C\u975E\u5185\u8054\u6563\u6587\uFF09\u3002"
         }
       }
     ];
@@ -14729,10 +14743,13 @@ var init_option_definitions = __esm({
       },
       reviewer: {
         command: "reviewer",
-        summary: "Fixed-target two-axis review (Standards + Spec).",
-        usage: ["ak-role reviewer --base <revision> [options] <instruction>"],
+        summary: "Fixed-target single-lens review (completeness or correctness).",
+        usage: [
+          "ak-role reviewer --base <revision> --lens completeness|correctness --authority-ref <ref> [options] <instruction>"
+        ],
         examples: [
-          'ak-role reviewer --base main "Review the branch against the governing issue and repository authority."'
+          'ak-role reviewer --base main --lens completeness --authority-ref docs/adr/0001.md "Review the branch."',
+          "ak-role reviewer --base main --lens correctness --authority-ref CLAUDE.md --authority-ref docs/adr/0001.md"
         ]
       },
       collector: {

package/dist/package-resources/method-skill.js

@@ -26,7 +26,12 @@ const SHA256_RE = /^[0-9a-f]{64}$/;
 const REQUIRED_COMPANIONS = {
     tdd: ["tests.md", "mocking.md", "agents/openai.yaml"],
     "diagnosing-bugs": ["agents/openai.yaml", "scripts/hitl-loop.template.sh"],
-    "code-review": ["agents/openai.yaml"],
+    "ak-cross-m-review": [
+        "CONTEXT.md",
+        "LICENSE",
+        "prompts/cmr-completeness.md",
+        "prompts/cmr-reviewer.md",
+    ],
     "resolving-merge-conflicts": ["agents/openai.yaml"],
 };
 /** Git blob OID for raw file bytes (sha1 of `blob <size>\\0` + content). */

package/dist/pi/role-turn-host.js

@@ -134,7 +134,9 @@ function piUserDialogueBody(request) {
             const _exhaustive = request.continuation;
             return _exhaustive;
         })();
-    return applyPiNativeSkillInvocation(request.methods, rawPrompt);
+    return request.continuation.kind === "resume" && request.activation.role === "reviewer"
+        ? rawPrompt
+        : applyPiNativeSkillInvocation(request.methods, rawPrompt);
 }
 const execFileAsync = promisify(execFile);
 async function resolveSelectedPi(command, cwd, env) {

package/dist/public-cli/invocation.js

@@ -379,6 +379,48 @@ function requireOptionPath(flag, value) {
     }
     return value;
 }
+/**
+ * Shared Skill-arg token rule for caller-controlled values projected into the
+ * space-joined Skill invocation line. Rejects blank, whitespace (smuggles the
+ * next option), and a leading `-` (read as the next Skill option). Not a
+ * general free-text gate — only the projection admission seam.
+ */
+function requireSkillArgToken(value, messages) {
+    if (value === undefined || value.trim() === "") {
+        throw new CliUsageError(messages.empty);
+    }
+    if (/\s/.test(value)) {
+        throw new CliUsageError(messages.whitespace);
+    }
+    if (value.startsWith("-")) {
+        throw new CliUsageError(messages.optionLike);
+    }
+    return value;
+}
+/**
+ * Reviewer --base admission: nonempty single Skill-arg token (shared rule with
+ * requireAuthorityRef). Multi-token / option-like values smuggle extra flags
+ * (e.g. `--lens all`, `--authority x`).
+ */
+export function requireReviewerBaseRevision(value) {
+    return requireSkillArgToken(value, {
+        empty: "--base requires a nonempty revision",
+        whitespace: "--base requires a single-token revision",
+        optionLike: "--base requires a single-token revision",
+    });
+}
+/** Shared ReviewerLens predicate — sole interpretation owner for fresh + durable. */
+export function isReviewerLens(value) {
+    return value === "completeness" || value === "correctness";
+}
+/** Public --lens enum; sole owner for parse + fresh admission. */
+export function requireReviewerLens(value) {
+    const trimmed = (value ?? "").trim();
+    if (!isReviewerLens(trimmed)) {
+        throw new CliUsageError("--lens requires completeness or correctness");
+    }
+    return trimmed;
+}
 /** True when token is a retained rejected spelling for the owner (#342). */
 function isRejectedPublicSpelling(owner, token) {
     for (const entry of REJECTED_PUBLIC_SPELLINGS) {
@@ -398,18 +440,16 @@ function roleOptions(owner) {
 /**
  * Public --authority-ref admission grammar (refs-only).
  * Unique owner for fresh argv and durable resume restore — no string-only parallel.
- * Accepts durable reference tokens as-is; rejects blank and inline Spec prose
- * (whitespace-bearing sentences). Does not fetch, normalize, or judge content.
+ * Accepts durable reference tokens as-is; rejects blank, inline Spec prose
+ * (whitespace-bearing sentences), and option-like leading `-` (Skill-arg boundary).
+ * Does not fetch, normalize, or judge content.
  */
 export function requireAuthorityRef(value) {
-    if (value === undefined || value.trim() === "") {
-        throw new CliUsageError("--authority-ref requires a nonempty durable reference");
-    }
-    // Spec prose sentences contain whitespace; durable public refs are single tokens.
-    if (/\s/.test(value)) {
-        throw new CliUsageError("--authority-ref requires a durable reference, not inline Spec prose");
-    }
-    return value;
+    return requireSkillArgToken(value, {
+        empty: "--authority-ref requires a nonempty durable reference",
+        whitespace: "--authority-ref requires a durable reference, not inline Spec prose",
+        optionLike: "--authority-ref requires a durable reference, not inline Spec prose",
+    });
 }
 /**
  * Parse Judge-specific argv after the `judge` token.
@@ -1663,12 +1703,6 @@ export async function admitNotaryInvocation(options) {
 export function buildNotaryTransportPrompt(_admitted, engineMaterial) {
     return appendEngineSessionMaterial([NOTARY_FIXED_KICKOFF], engineMaterial).join("\n");
 }
-/**
- * Parse Reviewer-specific argv after the `reviewer` token.
- * Public flags: --project, required --base, optional repeatable --authority-ref.
- * Reviewer gathers its own evidence; users submit neither attachments nor capability packets.
- * Caller instruction remains scope/procedure provenance — not Spec authority.
- */
 /**
  * Parse Gleaner-Left argv after the `gleaner-left` token.
  * Public flags: --project, required --base. No --attach / ticket face (unanchored self-fetch).
@@ -1788,6 +1822,7 @@ export function parseReviewerArgv(args) {
     const authorityRefs = [];
     let project;
     let baseRevision;
+    let lens;
     const positional = [];
     const tokens = [...args];
     const definitions = roleOptions("reviewer");
@@ -1805,7 +1840,12 @@ export function parseReviewerArgv(args) {
                 continue;
             }
             if (taken.def.id === "base") {
-                baseRevision = requireOptionPath(taken.def.canonical, taken.value);
+                baseRevision = requireReviewerBaseRevision(taken.value);
+                continue;
+            }
+            if (taken.def.id === "lens") {
+                // Empty and other non-enum values share one message — do not borrow path helper.
+                lens = requireReviewerLens(taken.value);
                 continue;
             }
             if (taken.def.id === "authority-ref") {
@@ -1820,29 +1860,32 @@ export function parseReviewerArgv(args) {
         }
         positional.push(token);
     }
-    // Unconditional required (e.g. --base) from typed table via shared consumer (#342).
+    // Unconditional required (--base/--lens/--authority-ref) from typed table (#342).
     options.assertRequired();
     return {
         instruction: positional.join(" "),
         attachmentPaths,
         baseRevision: baseRevision,
+        lens: lens,
         authorityRefs,
         ...(project === undefined ? {} : { project }),
     };
 }
 /**
- * Admit a Reviewer Role run on the fixed base only.
- * Caller instruction is optional provenance; Reviewer acquires issue/authority independently.
- * Optional authorityRefs are frozen as durable references only — not Spec prose.
+ * Admit a Reviewer Role run on the fixed base + lens + authority set.
+ * Caller instruction is optional provenance; typed fields project Skill inputs.
+ * authorityRefs are frozen as durable references only — not Spec prose.
  */
 export async function admitReviewerInvocation(options) {
     if (options.project !== undefined) {
         requireOptionPath("--project", options.project);
     }
-    if (options.baseRevision.trim() === "") {
-        throw new CliUsageError("--base requires a nonempty revision");
+    const baseRevision = requireReviewerBaseRevision(options.baseRevision);
+    const lens = requireReviewerLens(options.lens);
+    if (options.authorityRefs.length === 0) {
+        throw new CliUsageError("reviewer requires --authority-ref <ref>");
     }
-    const authorityRefs = Object.freeze((options.authorityRefs ?? []).map((ref) => requireAuthorityRef(ref)));
+    const authorityRefs = Object.freeze(options.authorityRefs.map((ref) => requireAuthorityRef(ref)));
     const projectRoot = resolve(options.project ?? options.cwd);
     const runId = (options.createRunId ?? uuidv7)();
     const { principal, sessionDirectory, sessionFile, runDirectory, attachmentsDirectory, ledgerHome, bookKey, } = issueAdmissionPlacement(options.principalAuthority, {
@@ -1865,7 +1908,8 @@ export async function admitReviewerInvocation(options) {
         principal,
         instruction,
         instructionEmpty,
-        baseRevision: options.baseRevision,
+        baseRevision,
+        lens,
         authorityRefs: [...authorityRefs],
         attachments: attachments.map((a) => ({
             provenancePath: a.provenancePath,
@@ -1892,19 +1936,26 @@ export async function admitReviewerInvocation(options) {
         runDirectory,
         principal,
         admittedRequestPath,
-        baseRevision: options.baseRevision,
+        baseRevision,
+        lens,
         authorityRefs,
     };
 }
+/** Frozen Skill arg projection shared by initial and resume (never reverse-parsed). */
+export function buildReviewerSkillArgProjection(admitted) {
+    return [
+        `--base ${admitted.baseRevision}`,
+        `--lens ${admitted.lens}`,
+        ...admitted.authorityRefs.map((ref) => `--authority ${ref}`),
+    ].join(" ");
+}
 /**
- * Build the Pi prompt transport for an admitted Reviewer request.
- * #836: caller instruction is no longer dropped (A3.5).
- * Fixed base + optional caller words + engine material.
+ * Build the host-neutral prompt transport for an admitted Reviewer request.
+ * Typed base/lens/authority project to Skill invocation args (never reverse-parsed from prose).
+ * Optional caller words + engine material follow.
  */
 export function buildReviewerTransportPrompt(admitted, engineMaterial) {
-    const lines = [
-        `本次审查的固定基点：${admitted.baseRevision}`,
-    ];
+    const lines = [buildReviewerSkillArgProjection(admitted)];
     if (!admitted.instructionEmpty && admitted.instruction.trim() !== "") {
         lines.push("", admitted.instruction);
     }