@akagilnc/pi-workflow-roles 0.1.4403 → 0.1.4419

package/src/role-runtime.ts

@@ -140,7 +140,6 @@ import {
   type ReviewerActivation,
   type ReviewerAdmittedInputs,
 } from "./reviewer-role.ts";
-import type { ReviewerPinnedGitReader } from "./reviewer-dispatch.ts";
 import type { GatekeeperNonPassResult } from "./gatekeeper-role.ts";
 
 /**
@@ -156,16 +155,16 @@ const REVIEWER_TRANSPORT_FLAGS = Object.freeze([
     }),
   }),
   Object.freeze({
-    name: "ak-review-scope-keys",
+    name: "ak-review-lens",
     definition: Object.freeze({
-      description: "Optional comma-separated exact class keys limiting Reviewer scope",
+      description: "Caller-selected single review lens: completeness or correctness",
       type: "string" as const,
     }),
   }),
   Object.freeze({
     name: "ak-review-authority-refs",
     definition: Object.freeze({
-      description: "JSON array of durable authority references for Spec-axis material only",
+      description: "JSON array of durable authority references projected as Skill --authority inputs",
       type: "string" as const,
     }),
   }),
@@ -207,19 +206,6 @@ export const STATION_CHILD_FLAG = Object.freeze({
  * Envelope-owned; necessary JSON decode only (public --authority-ref owns grammar).
  */
 function decodeReviewerAdmittedInputs(getFlag: (name: string) => unknown): ReviewerAdmittedInputs {
-  let reviewScopeKeys: readonly string[] | undefined;
-  const rawScopeKeys = getFlag("ak-review-scope-keys");
-  if (rawScopeKeys !== undefined) {
-    if (typeof rawScopeKeys !== "string" || rawScopeKeys.length === 0) {
-      throw new Error("Reviewer scope keys must be a nonempty comma-separated string");
-    }
-    const parsed = rawScopeKeys.split(",");
-    if (parsed.some((key) => key.trim().length === 0) || new Set(parsed).size !== parsed.length) {
-      throw new Error("Reviewer scope keys contain a blank or exact duplicate key");
-    }
-    reviewScopeKeys = Object.freeze(parsed);
-  }
-
   let authorityRefs: readonly string[] | undefined;
   const rawAuthorityRefs = getFlag("ak-review-authority-refs");
   if (rawAuthorityRefs !== undefined) {
@@ -251,9 +237,13 @@ function decodeReviewerAdmittedInputs(getFlag: (name: string) => unknown): Revie
   if (typeof baseRevision !== "string" || !baseRevision.trim()) {
     throw new Error("Reviewer role requires --ak-review-base");
   }
+  const rawLens = getFlag("ak-review-lens");
+  if (rawLens !== "completeness" && rawLens !== "correctness") {
+    throw new Error("Reviewer role requires --ak-review-lens completeness|correctness");
+  }
   return Object.freeze({
     baseRevision,
-    ...(reviewScopeKeys === undefined ? {} : { reviewScopeKeys }),
+    lens: rawLens,
     ...(authorityRefs === undefined ? {} : { authorityRefs }),
     ...(ticketNumber === undefined ? {} : { ticketNumber }),
   });
@@ -383,7 +373,6 @@ export {
 } from "./judge-role.ts";
 export { ENGINE_DETOUR_TOOL_NAME, AK_ROLE_ENGINE_ENV } from "./engine-detour.ts";
 export {
-  AGENT_TOOL_NAME,
   REVIEWER_OUTPUT_TOOL_NAME,
   type ReviewerIntent,
 } from "./reviewer-role.ts";
@@ -423,7 +412,6 @@ export {
   COLLECTOR_REQUEST_TOOL,
   COLLECTOR_WAIT_TOOL,
 } from "./collector-role.ts";
-export type { ReviewerPinnedGitReader } from "./reviewer-dispatch.ts";
 export type { CollectorReceipt } from "./package-contracts/collector-output.ts";
 export type { CollectorGitHubTransport } from "./collector-github.ts";
 export type { CollectorClock } from "./collector-evidence.ts";
@@ -599,7 +587,6 @@ export type RoleRuntimeDependencies = {
   loadCoderSoul?(): Promise<string>;
   loadCoderTask?(path: string): Promise<string>;
   loadReviewerSoul?(): Promise<string>;
-  createReviewerPinnedGitReader?(): Promise<ReviewerPinnedGitReader>;
   loadCollectorSoul?(): Promise<string>;
   /** #677: optional packaged seed for first-use general bot handbook. */
   loadCollectorHandbookSeed?(): Promise<string>;
@@ -623,7 +610,7 @@ export type RoleRuntimeDependencies = {
   createNavigatorAttendance?(options: { context: HostContext; role: string; phase: NavigatorPhase; subjectKey: string; subject: string; authority: string; contextError?: unknown; invocationId: string; onEvent: (event: import("./navigator-attendance.ts").NavigatorEvent, report: import("./navigator-attendance.ts").NavigatorReport) => void | Promise<void> }): NavigatorAttendanceDependency | Promise<NavigatorAttendanceDependency>;
   loadNavigatorWorkContext?(options: { context: HostContext; role: string; phase: NavigatorPhase; getFlag?: (name: string) => unknown }): Promise<NavigatorWorkContext>;
   loadCanonicalSkillBinding?(
-    name: "tdd" | "code-review",
+    name: "tdd" | "ak-cross-m-review",
   ): Promise<AnyCanonicalSkillBinding>;
   activationClock?(): string;
   activationTraceWriter?: (record: ActivationTraceRecord) => void | Promise<void>;
@@ -1782,10 +1769,6 @@ export function createRoleRuntimeExtension(
           }
           return dependencies.loadReviewerSoul();
         },
-        async createPinnedGitReader() {
-          if (dependencies.createReviewerPinnedGitReader === undefined) throw new Error("Reviewer runtime dependencies are not configured");
-          return dependencies.createReviewerPinnedGitReader();
-        },
         async loadCanonicalSkillBinding(name) {
           if (dependencies.loadCanonicalSkillBinding === undefined) {
             throw new Error("Reviewer runtime dependencies are not configured");

package/src/worker-role.ts

@@ -390,7 +390,7 @@ export function createCoderRoleRuntime(
           const loaded = await dependencies.loadCanonicalSkillBinding("tdd");
           if (loaded.name !== "tdd") {
             throw new Error(
-              "Canonical Skill binding loader returned code-review for tdd",
+              "Canonical Skill binding loader returned ak-cross-m-review for tdd",
             );
           }
           binding = loaded;

package/dist/adr-path-refs.js

@@ -1,23 +0,0 @@
-/**
- * Shared docs/adr path extraction from free text (issue body / ticket face).
- * Single authority for reviewer Spec discovery.
- *
- * Shape only: segments under docs/adr/ without `..` or empty parts. Real IO
- * confinement still lives at the read seam (ADR 0038).
- */
-/** docs/adr paths referenced inside free text (order of first appearance). */
-// Each segment must start with alnum so `.` / `..` traversal claims never match.
-const ADR_PATH_IN_BODY = /docs\/adr\/(?:[A-Za-z0-9][A-Za-z0-9._-]*\/)*[A-Za-z0-9][A-Za-z0-9._-]*\.md/g;
-/** Extract unique docs/adr/*.md paths; preserve first-seen order. */
-export function extractReferencedAdrPaths(text) {
-    const seen = new Set();
-    const paths = [];
-    for (const match of text.matchAll(ADR_PATH_IN_BODY)) {
-        const path = match[0];
-        if (seen.has(path))
-            continue;
-        seen.add(path);
-        paths.push(path);
-    }
-    return Object.freeze(paths);
-}

package/dist/public-cli/reviewer-dispatch-rejection.js

@@ -1,77 +0,0 @@
-/**
- * Durable Reviewer-rejection child→parent page under AK_ROLE_RUN_DIR.
- * Stays in public-cli (AK artifact owner); role-runtime imports this neighbor.
- */
-import { writeFileSync } from "node:fs";
-import { readFile, unlink } from "node:fs/promises";
-import { join } from "node:path";
-import { REVIEWER_PREFLIGHT_VIOLATIONS, } from "../reviewer-dispatch.js";
-const REVIEWER_DISPATCH_REJECTION_FILE = "typed-known-failure.json";
-function isReviewerPreflightViolation(value) {
-    return (typeof value === "string" &&
-        REVIEWER_PREFLIGHT_VIOLATIONS.includes(value));
-}
-function reviewerDispatchRejectionPath(runDirectory) {
-    return join(runDirectory, REVIEWER_DISPATCH_REJECTION_FILE);
-}
-/**
- * Clear any prior attempt's Reviewer rejection page so resume/retry cannot
- * inherit a stale knownFailure.details.
- */
-export async function clearReviewerDispatchRejection(runDirectory) {
-    try {
-        await unlink(reviewerDispatchRejectionPath(runDirectory));
-    }
-    catch (error) {
-        if (error instanceof Error &&
-            "code" in error &&
-            error.code === "ENOENT") {
-            return;
-        }
-        throw error;
-    }
-}
-/**
- * Synchronous durable write for Reviewer dispatch rejection (child process exit is sync).
- * Parent public CLI recovers via readReviewerDispatchRejection into knownFailure.
- */
-export function recordReviewerDispatchRejectionSync(runDirectory, rejection) {
-    writeFileSync(reviewerDispatchRejectionPath(runDirectory), `${JSON.stringify(rejection)}\n`, "utf8");
-}
-/** Recover a child-written Reviewer dispatch rejection through a fixed mapping. */
-export async function readReviewerDispatchRejection(runDirectory) {
-    let raw;
-    try {
-        raw = await readFile(reviewerDispatchRejectionPath(runDirectory), "utf8");
-    }
-    catch (error) {
-        if (error instanceof Error &&
-            "code" in error &&
-            error.code === "ENOENT") {
-            return undefined;
-        }
-        throw error;
-    }
-    const parsed = JSON.parse(raw);
-    if (typeof parsed !== "object" || parsed === null || Array.isArray(parsed)) {
-        const error = new Error("Reviewer dispatch rejection page must be a JSON object");
-        error.name = "ReviewerDispatchRejectionContractError";
-        throw error;
-    }
-    const record = parsed;
-    if (typeof record.diagnostic !== "string" ||
-        record.diagnostic.trim() === "" ||
-        !Array.isArray(record.violations) ||
-        record.violations.length === 0 ||
-        record.violations.some((value) => !isReviewerPreflightViolation(value))) {
-        const error = new Error("Reviewer dispatch rejection page has unusable required fields");
-        error.name = "ReviewerDispatchRejectionContractError";
-        throw error;
-    }
-    return {
-        cause: "activation",
-        diagnostic: record.diagnostic,
-        identity: { name: "ReviewerDispatchRejectionError" },
-        details: { violations: Object.freeze([...record.violations]) },
-    };
-}

package/dist/reviewer-dispatch.js

@@ -1,14 +0,0 @@
-/**
- * #836 删 8: no code-compiled axis dispatch.
- * Parent seat invokes packaged code-review skill; this module does not construct or run children.
- */
-export { branchNamesAtPinnedHead, createReviewerPinnedGitReader, immutableReviewerPin, } from "./reviewer-pinned-git.js";
-export { extractReferencedAdrPaths } from "./adr-path-refs.js";
-export { sha256Hex } from "./sha256.js";
-export { isReviewerPromptText as isReviewerPromptIdentity, sameReviewerPromptText as sameReviewerPromptIdentity, } from "./reviewer-prompt-identity.js";
-export const REVIEWER_PREFLIGHT_VIOLATIONS = [
-    "base-invalid",
-    "range-invalid",
-    "prompt-identity-invalid",
-    "target-drift",
-];

package/dist/reviewer-git-snapshot.js

@@ -1,32 +0,0 @@
-export const REVIEW_REF_PREFIXES = ["refs/heads", "refs/tags", "refs/remotes"];
-export function parseReviewerRefSnapshot(raw) {
-    const refs = {};
-    for (const line of raw.split("\n")) {
-        if (!line)
-            continue;
-        const fields = line.split("\0");
-        if (fields.length !== 5 || !fields[0] || !fields[1] || !fields[3]) {
-            throw new Error(`Malformed Git ref snapshot line: ${line}`);
-        }
-        const peeledCommitId = fields[3] === "commit"
-            ? fields[1]
-            : fields[4] === "commit" && fields[2] ? fields[2] : null;
-        refs[fields[0]] = Object.freeze({
-            objectId: fields[1],
-            peeledCommitId,
-        });
-    }
-    return refs;
-}
-export function reviewerRefSnapshotArgs() {
-    return ["for-each-ref", "--format=%(refname)%00%(objectname)%00%(*objectname)%00%(objecttype)%00%(*objecttype)", ...REVIEW_REF_PREFIXES];
-}
-export function immutableReviewerRefs(refs) {
-    return Object.freeze(Object.fromEntries(Object.entries(refs)
-        .sort(([a], [b]) => a.localeCompare(b))
-        .map(([name, entry]) => [name, Object.freeze({ objectId: entry.objectId, peeledCommitId: entry.peeledCommitId })])));
-}
-export function sameReviewerPinnedTarget(actual, expected) {
-    return actual.repositoryRoot === expected.repositoryRoot && actual.objectFormat === expected.objectFormat &&
-        actual.targetHead === expected.targetHead;
-}

package/dist/reviewer-pinned-git.js

@@ -1,308 +0,0 @@
-import { execFile } from "node:child_process";
-import { access, realpath } from "node:fs/promises";
-import { promisify } from "node:util";
-import { immutableReviewerRefs, parseReviewerRefSnapshot, reviewerRefSnapshotArgs } from "./reviewer-git-snapshot.js";
-import { sha256Hex } from "./sha256.js";
-import { ReviewerCorrectablePreflightError } from "./reviewer-preflight-error.js";
-const execFileAsync = promisify(execFile);
-async function execGit(args, options) {
-    // Pin C locale at the sole Git exec seam so English diagnostic classifiers stay honest under translated gettext installs.
-    try {
-        return await execFileAsync("git", args, {
-            ...options,
-            env: { ...process.env, LC_ALL: "C" },
-        });
-    }
-    catch (error) {
-        const source = error;
-        const wrapped = new Error("git process failed", { cause: error });
-        Object.assign(wrapped, { code: source.code ?? null, signal: source.signal ?? null, timedOut: source.killed === true && source.signal === "SIGTERM", aborted: source.name === "AbortError", stderr: String(source.stderr ?? ""), stdout: String(source.stdout ?? "") });
-        throw wrapped;
-    }
-}
-function exitCode(error) { const code = typeof error === "object" && error !== null ? error.code : undefined; return typeof code === "number" ? code : undefined; }
-function gitStderr(error) {
-    if (typeof error !== "object" || error === null)
-        return "";
-    const stderr = error.stderr;
-    return typeof stderr === "string" ? stderr : "";
-}
-/** Confirmed `origin` remote absence only (`git remote get-url origin`). */
-function isConfirmedMissingOriginRemote(error) {
-    return /No such remote ['"]origin['"]/.test(gitStderr(error));
-}
-/** Confirmed path-at-pinned-tree absence only — exit 128 alone is not enough. */
-function isConfirmedPinnedPathAbsent(error, path) {
-    const stderr = gitStderr(error);
-    const quoted = `'${path}'`;
-    return (stderr.includes(`path ${quoted} does not exist in `) ||
-        stderr.includes(`path ${quoted} exists on disk, but not in `));
-}
-async function repositoryIsAvailable(root) { try {
-    await access(`${root}/.git`);
-    return { available: true };
-}
-catch (cause) {
-    return { available: false, cause };
-} }
-export const immutableReviewerPin = (pin) => Object.freeze({
-    repositoryRoot: pin.repositoryRoot, objectFormat: pin.objectFormat, targetHead: pin.targetHead, refs: immutableReviewerRefs(pin.refs),
-});
-/** Short name from a full ref, stripping heads/tags/remotes namespaces (and remote remote-name). */
-function shortNameFromPinnedRef(refName) {
-    const short = refName.startsWith("refs/heads/")
-        ? refName.slice("refs/heads/".length)
-        : refName.startsWith("refs/tags/")
-            ? refName.slice("refs/tags/".length)
-            : refName.startsWith("refs/remotes/")
-                ? refName.slice("refs/remotes/".length).replace(/^[^/]+\//, "")
-                : refName;
-    const trimmed = short.trim();
-    return trimmed === "" ? undefined : trimmed;
-}
-/**
- * Branch-only short names at pinned targetHead for ticket-number provenance (#343).
- * Heads and remotes only — tags never supply branch-token ticket candidates.
- */
-export function branchNamesAtPinnedHead(pin) {
-    const names = new Set();
-    for (const [refName, entry] of Object.entries(pin.refs)) {
-        if (entry.peeledCommitId !== pin.targetHead)
-            continue;
-        if (!refName.startsWith("refs/heads/") && !refName.startsWith("refs/remotes/"))
-            continue;
-        const short = shortNameFromPinnedRef(refName);
-        if (short !== undefined)
-            names.add(short);
-    }
-    return Object.freeze([...names]);
-}
-async function gitText(root, args) {
-    const { stdout } = await execGit(["-C", root, ...args], { encoding: "utf8" });
-    return stdout.trim();
-}
-/** Concrete fixed-point Git I/O; dispatch owns all policy applied to these reads. */
-export async function createReviewerPinnedGitReader(root = process.cwd()) {
-    const discoveredRoot = await gitText(root, ["rev-parse", "--show-toplevel"]);
-    const repositoryRoot = await realpath(discoveredRoot);
-    const objectFormat = await gitText(repositoryRoot, ["rev-parse", "--show-object-format"]);
-    if (objectFormat !== "sha1" && objectFormat !== "sha256")
-        throw new Error("Unsupported Git object format");
-    const oidWidth = objectFormat === "sha1" ? 40 : 64;
-    const targetHead = await gitText(repositoryRoot, ["rev-parse", "HEAD^{commit}"]);
-    const reachableCommitIds = Object.freeze((await gitText(repositoryRoot, ["rev-list", targetHead])).split("\n").filter(Boolean));
-    const refs = parseReviewerRefSnapshot(await gitText(repositoryRoot, reviewerRefSnapshotArgs()));
-    const pin = immutableReviewerPin({ repositoryRoot, objectFormat, targetHead, refs });
-    const invalid = (code, diagnostic, cause) => {
-        throw new ReviewerCorrectablePreflightError(code, diagnostic, cause === undefined ? undefined : { cause });
-    };
-    const symbolic = (base) => {
-        const selected = Object.hasOwn(refs, base) ? base : (() => {
-            const candidates = [`refs/heads/${base}`, `refs/tags/${base}`, `refs/remotes/${base}`].filter((name) => Object.hasOwn(refs, name));
-            if (candidates.length > 1)
-                invalid("base-invalid", "base revision is ambiguous across pinned refs");
-            return candidates[0];
-        })();
-        if (selected === undefined)
-            return undefined;
-        const commit = refs[selected]?.peeledCommitId;
-        if (commit === null)
-            invalid("base-invalid", "base revision ref must resolve to a commit");
-        return commit ?? undefined;
-    };
-    return Object.freeze({
-        pin,
-        async snapshot() {
-            const liveObjectFormat = await gitText(repositoryRoot, ["rev-parse", "--show-object-format"]);
-            if (liveObjectFormat !== "sha1" && liveObjectFormat !== "sha256")
-                throw new Error("Unsupported Git object format");
-            return immutableReviewerPin({ repositoryRoot, objectFormat: liveObjectFormat, targetHead: await gitText(repositoryRoot, ["rev-parse", "HEAD^{commit}"]), refs: parseReviewerRefSnapshot(await gitText(repositoryRoot, reviewerRefSnapshotArgs())) });
-        },
-        async resolve(base) {
-            if (!/^[A-Za-z0-9._/~^+-]+$/.test(base) || base.startsWith("-") || base.includes("..") || base.includes("@{")) {
-                invalid("base-invalid", "base revision syntax is invalid or uses a forbidden revision form");
-            }
-            let commit;
-            const headExpression = /^HEAD((?:~[0-9]+|\^[0-9]+)*)$/.exec(base);
-            if (headExpression) {
-                try {
-                    commit = await gitText(repositoryRoot, ["rev-parse", "--verify", `${targetHead}${headExpression[1]}^{commit}`]);
-                }
-                catch (error) {
-                    if (exitCode(error) === 128) {
-                        const repository = await repositoryIsAvailable(repositoryRoot);
-                        if (repository.available)
-                            invalid("base-invalid", "base revision HEAD ancestry expression must resolve to a reachable commit", error);
-                    }
-                    throw error;
-                }
-            }
-            else if (new RegExp(`^[0-9a-f]{${oidWidth}}$`).test(base))
-                commit = base;
-            else if (new RegExp(`^[0-9a-f]{4,${oidWidth - 1}}$`).test(base) && !(objectFormat === "sha256" && base.length === 40)) {
-                const matches = reachableCommitIds.filter((candidate) => candidate.startsWith(base));
-                if (matches.length !== 1)
-                    invalid("base-invalid", "base revision abbreviation must identify exactly one reachable commit");
-                commit = matches[0];
-            }
-            else
-                commit = symbolic(base);
-            if (commit === undefined)
-                invalid("base-invalid", "base revision must name an existing pinned ref or reachable commit");
-            try {
-                commit = await gitText(repositoryRoot, ["rev-parse", "--verify", `${commit}^{commit}`]);
-            }
-            catch (error) {
-                if (exitCode(error) === 128) {
-                    const repository = await repositoryIsAvailable(repositoryRoot);
-                    if (repository.available)
-                        invalid("base-invalid", "base revision must resolve to an existing commit", error);
-                }
-                throw error;
-            }
-            try {
-                await gitText(repositoryRoot, ["merge-base", "--is-ancestor", commit, targetHead]);
-            }
-            catch (error) {
-                if (exitCode(error) === 1)
-                    invalid("base-invalid", "base revision must be an ancestor of the pinned target", error);
-                throw error;
-            }
-            return commit;
-        },
-        async range(base) {
-            let mergeBase;
-            try {
-                mergeBase = await gitText(repositoryRoot, ["merge-base", base, targetHead]);
-            }
-            catch (error) {
-                if (exitCode(error) === 1) {
-                    invalid("range-invalid", "review range requires a common ancestor for base and pinned target", error);
-                }
-                throw error;
-            }
-            if (!mergeBase)
-                invalid("range-invalid", "review range requires a common ancestor for base and pinned target");
-            const diffCommand = `git diff ${mergeBase}...${targetHead}`;
-            const [{ stdout: diff }, commitsText] = await Promise.all([
-                execGit(["-C", repositoryRoot, "diff", `${mergeBase}...${targetHead}`], { encoding: "buffer", maxBuffer: 64 * 1024 * 1024 }),
-                gitText(repositoryRoot, ["rev-list", "--reverse", `${mergeBase}..${targetHead}`]),
-            ]);
-            if (diff.length === 0)
-                invalid("range-invalid", "review range must contain a non-empty diff between base and pinned target");
-            return Object.freeze({ base: mergeBase, target: targetHead, diffCommand, diffSha256: sha256Hex(Uint8Array.from(diff)), commits: Object.freeze(commitsText ? commitsText.split("\n") : []) });
-        },
-        async featureTokens() {
-            // Pinned ref snapshot is the target-tree fact — no live branch/symbolic-ref walk,
-            // no catch-to-empty. Detached/remote-only tips surface via refs/remotes/* entries.
-            // Includes tags for local Spec-path matching only; ticket branch source is separate.
-            const names = new Set();
-            for (const [refName, entry] of Object.entries(pin.refs)) {
-                if (entry.peeledCommitId !== targetHead)
-                    continue;
-                const short = shortNameFromPinnedRef(refName);
-                if (short !== undefined)
-                    names.add(short);
-            }
-            return Object.freeze([...names]);
-        },
-        async listSpecCandidatePaths() {
-            const roots = ["docs", "specs", ".scratch"];
-            // git ls-tree exits 0 with empty stdout when none of the roots exist at targetHead.
-            // Other Git/I-O failures keep their true cause for the dispatch preflight path.
-            const text = await gitText(repositoryRoot, [
-                "ls-tree",
-                "-r",
-                "--name-only",
-                targetHead,
-                "--",
-                ...roots,
-            ]);
-            return Object.freeze(text === "" ? [] : text.split("\n").filter((line) => line.length > 0));
-        },
-        async originRepository() {
-            let remoteUrl;
-            try {
-                remoteUrl = await gitText(repositoryRoot, ["remote", "get-url", "origin"]);
-            }
-            catch (error) {
-                // Only confirmed origin absence softens to unavailable; other Git failures keep true cause.
-                if (isConfirmedMissingOriginRemote(error))
-                    return undefined;
-                throw error;
-            }
-            // Non-github / unparseable remote URL = self-fetch unavailable (soft degrade).
-            return parseGitHubOriginRemote(remoteUrl);
-        },
-        async commitMessagesNewestFirst(base) {
-            const text = await gitText(repositoryRoot, [
-                "log",
-                "--format=%s",
-                `${base}..${targetHead}`,
-            ]);
-            return Object.freeze(text === "" ? [] : text.split("\n"));
-        },
-        async readPinnedText(path) {
-            // Reject path traversal / absolute paths — Spec material is relative tree paths only.
-            if (path.length === 0 ||
-                path.startsWith("/") ||
-                path.includes("\0") ||
-                path.split("/").some((part) => part === ".." || part === "")) {
-                return undefined;
-            }
-            try {
-                const { stdout } = await execGit(["-C", repositoryRoot, "show", `${targetHead}:${path}`], { encoding: "utf8", maxBuffer: 8 * 1024 * 1024 });
-                return stdout;
-            }
-            catch (error) {
-                // Only confirmed path-at-pinned-tree absence softens to missing; exit 128 is not a blanket.
-                if (isConfirmedPinnedPathAbsent(error, path))
-                    return undefined;
-                throw error;
-            }
-        },
-    });
-}
-/**
- * Parse github.com owner/repo from a git remote URL.
- * Supports scp-like SSH, ssh://, https://, and git:// shapes. Soft: undefined when not github.
- */
-export function parseGitHubOriginRemote(remoteUrl) {
-    const trimmed = remoteUrl.trim();
-    if (trimmed.length === 0)
-        return undefined;
-    const scp = /^git@github\.com:([^/\s]+)\/([^/\s]+?)(?:\.git)?$/i.exec(trimmed);
-    if (scp)
-        return normalizeOrigin(scp[1], scp[2]);
-    const ssh = /^ssh:\/\/git@github\.com\/([^/\s]+)\/([^/\s]+?)(?:\.git)?\/?$/i.exec(trimmed);
-    if (ssh)
-        return normalizeOrigin(ssh[1], ssh[2]);
-    let parsed;
-    try {
-        parsed = new URL(trimmed);
-    }
-    catch {
-        return undefined;
-    }
-    if (!/^github\.com$/i.test(parsed.hostname))
-        return undefined;
-    if (parsed.search !== "" || parsed.hash !== "")
-        return undefined;
-    const parts = parsed.pathname.split("/").filter((p) => p.length > 0);
-    if (parts.length !== 2)
-        return undefined;
-    return normalizeOrigin(parts[0], parts[1]);
-}
-function normalizeOrigin(ownerRaw, repoRaw) {
-    const owner = ownerRaw.trim();
-    const repo = stripGitSuffix(repoRaw.trim());
-    if (owner.length === 0 || repo.length === 0)
-        return undefined;
-    // Conservative identity: no path separators or URL material inside segments.
-    if (/[/?#@\\]/.test(owner) || /[/?#@\\]/.test(repo))
-        return undefined;
-    return Object.freeze({ owner, repo });
-}
-function stripGitSuffix(name) {
-    return name.toLowerCase().endsWith(".git") ? name.slice(0, -4) : name;
-}

package/dist/reviewer-preflight-error.js

@@ -1,15 +0,0 @@
-export const REVIEWER_CORRECTABLE_PREFLIGHT_CODES = [
-    "base-invalid",
-    "range-invalid",
-];
-/** Closed policy error shared by concrete Git reads and dispatch compilation. */
-export class ReviewerCorrectablePreflightError extends Error {
-    code;
-    diagnostic;
-    constructor(code, diagnostic = `${code} constraint failed`, options) {
-        super(`${code}: ${diagnostic}`, options);
-        this.code = code;
-        this.diagnostic = diagnostic;
-        this.name = "ReviewerCorrectablePreflightError";
-    }
-}

package/dist/reviewer-prompt-identity.js

@@ -1,10 +0,0 @@
-/**
- * ADR 0031: Reviewer model-visible text remains plain text.
- * No length/digest identity shell; callers compare strings directly when needed.
- */
-export function isReviewerPromptText(value) {
-    return typeof value === "string";
-}
-export function sameReviewerPromptText(first, second) {
-    return first === second;
-}