peaks-loop 4.0.54 → 4.1.1

package/dist/services/release/version-precheck-service.d.ts

@@ -2,17 +2,15 @@
  * rid-010 — peaks release precheck (Phase 4 slice 1).
  *
  * Module path: src/services/release/version-precheck-service.ts
- * Mirror of publish.yml gate-cli-version step §(A). The §(B) tarball-content gate
- * (Layer 5 of the 5-layer root cause) stays CI-only — AC-7 grep test pins
- * publish.yml so §(B) cannot drift silently.
+ * Mirrors publish.yml gate-cli-version §(A) — verified in code 2026-10-03, not
+ * just per the old comment: `runRootVsShared` covers §(A)'s shared-dist check;
+ * §(A′) (runtime RUNTIME_VERSION) has no layer here, and the §(B)
+ * tarball-content gate (Layer 5 of the 5-layer root cause) stays CI-only —
+ * AC-7 grep test pins publish.yml so §(B) cannot drift silently.
  *
- * 4-layer version precheck mirroring publish.yml gate-cli-version step. Designed
- * to run BEFORE `peaks release canary` so developers catch CLI_VERSION lag /
- * tag collision / changeset staged / workspace drift without waiting for CI.
- *
- * Mirrors the §(A) on-disk gate from publish.yml. The §(B) tarball-content gate
- * (Layer 5 of the 5-layer root cause) stays CI-only — AC-7 grep test pins
- * publish.yml so §(B) cannot drift silently.
+ * 4-layer version precheck. Designed to run BEFORE `peaks release canary` so
+ * developers catch CLI_VERSION lag / tag collision / changeset staged /
+ * workspace drift without waiting for CI.
  *
  * Design contract:
  *   - Pure functions. NO `process.exitCode` mutation here (cli-helpers owns it).
@@ -20,33 +18,10 @@
  *   - Layer A and B are blockers by default; C and D are warnings (--strict
  *     upgrades them to blockers; hotfix path uses default warning).
  */
-export type LayerStatus = 'ok' | 'warning' | 'blocker';
-export interface LayerResult {
-    readonly status: LayerStatus;
-    readonly message: string;
-    readonly remediation: string;
-    readonly observed?: Readonly<Record<string, unknown>>;
-}
-export interface PrecheckOptions {
-    readonly projectRoot: string;
-    readonly strict?: boolean;
-}
-export interface PrecheckEnvelope {
-    readonly ok: boolean;
-    readonly overall: LayerStatus;
-    /** `null` when the root `package.json` is missing or unreadable — see `rootVsShared`. */
-    readonly rootVersion: string | null;
-    readonly strict: boolean;
-    readonly snapshotAt: string;
-    readonly layers: {
-        readonly rootVsShared: LayerResult;
-        readonly tagCollision: LayerResult;
-        readonly changesetStaged: LayerResult;
-        readonly workspaceLockstep: LayerResult;
-    };
-}
+import { type LayerResult, type PrecheckEnvelope, type PrecheckOptions } from './version-precheck-support.js';
+export type { LayerStatus, LayerResult, PrecheckOptions, PrecheckEnvelope } from './version-precheck-support.js';
+export { runChangesetStaged } from './version-precheck-layer-changeset.js';
 export declare function runRootVsShared(opts: PrecheckOptions): LayerResult;
 export declare function runTagCollision(opts: PrecheckOptions): LayerResult;
-export declare function runChangesetStaged(opts: PrecheckOptions): LayerResult;
 export declare function runWorkspaceLockstep(opts: PrecheckOptions): LayerResult;
 export declare function runAllLayers(opts: PrecheckOptions): PrecheckEnvelope;

package/dist/services/release/version-precheck-service.js

@@ -2,17 +2,15 @@
  * rid-010 — peaks release precheck (Phase 4 slice 1).
  *
  * Module path: src/services/release/version-precheck-service.ts
- * Mirror of publish.yml gate-cli-version step §(A). The §(B) tarball-content gate
- * (Layer 5 of the 5-layer root cause) stays CI-only — AC-7 grep test pins
- * publish.yml so §(B) cannot drift silently.
+ * Mirrors publish.yml gate-cli-version §(A) — verified in code 2026-10-03, not
+ * just per the old comment: `runRootVsShared` covers §(A)'s shared-dist check;
+ * §(A′) (runtime RUNTIME_VERSION) has no layer here, and the §(B)
+ * tarball-content gate (Layer 5 of the 5-layer root cause) stays CI-only —
+ * AC-7 grep test pins publish.yml so §(B) cannot drift silently.
  *
- * 4-layer version precheck mirroring publish.yml gate-cli-version step. Designed
- * to run BEFORE `peaks release canary` so developers catch CLI_VERSION lag /
- * tag collision / changeset staged / workspace drift without waiting for CI.
- *
- * Mirrors the §(A) on-disk gate from publish.yml. The §(B) tarball-content gate
- * (Layer 5 of the 5-layer root cause) stays CI-only — AC-7 grep test pins
- * publish.yml so §(B) cannot drift silently.
+ * 4-layer version precheck. Designed to run BEFORE `peaks release canary` so
+ * developers catch CLI_VERSION lag / tag collision / changeset staged /
+ * workspace drift without waiting for CI.
  *
  * Design contract:
  *   - Pure functions. NO `process.exitCode` mutation here (cli-helpers owns it).
@@ -20,63 +18,12 @@
  *   - Layer A and B are blockers by default; C and D are warnings (--strict
  *     upgrades them to blockers; hotfix path uses default warning).
  */
-import { readdirSync, readFileSync } from 'node:fs';
+import { readFileSync } from 'node:fs';
 import { join } from 'node:path';
 import { spawnSync } from 'node:child_process';
-const SEMVER_RE = /^\d+\.\d+\.\d+(?:-[0-9A-Za-z.-]+)?(?:\+[0-9A-Za-z.-]+)?$/;
-// ---------------------------------------------------------------------------
-// Helpers
-// ---------------------------------------------------------------------------
-function upgrade(result, strict) {
-    if (strict === true && result.status === 'warning') {
-        return {
-            ...result,
-            status: 'blocker',
-            remediation: `${result.remediation} (strict mode: warning promoted to blocker)`
-        };
-    }
-    return result;
-}
-/** Read + parse a JSON manifest, mapping both failure modes onto a blocker description. */
-function readPackageJson(path, label) {
-    let raw;
-    try {
-        raw = readFileSync(path, 'utf8');
-    }
-    catch {
-        return {
-            ok: false,
-            message: `${label} is missing or unreadable at ${path}`,
-            remediation: `run \`peaks release precheck\` from a project root that has a readable ${label}`
-        };
-    }
-    try {
-        return { ok: true, value: JSON.parse(raw) };
-    }
-    catch {
-        return {
-            ok: false,
-            message: `${label} at ${path} is not valid JSON`,
-            remediation: `fix the ${label} syntax; \`peaks release precheck\` cannot gate a version it cannot parse`
-        };
-    }
-}
-function readRootVersion(projectRoot) {
-    const path = join(projectRoot, 'package.json');
-    const read = readPackageJson(path, 'root package.json');
-    if (!read.ok) {
-        return read;
-    }
-    const pkg = read.value;
-    if (typeof pkg.version !== 'string' || !SEMVER_RE.test(pkg.version)) {
-        return {
-            ok: false,
-            message: `invalid root package.json#version: ${JSON.stringify(pkg.version)}`,
-            remediation: 'set root package.json#version to a clean semver (e.g. 4.0.0) before running `peaks release precheck`'
-        };
-    }
-    return { ok: true, version: pkg.version };
-}
+import { readPackageJson, readRootVersion, SEMVER_RE, upgrade } from './version-precheck-support.js';
+import { runChangesetStaged } from './version-precheck-layer-changeset.js';
+export { runChangesetStaged } from './version-precheck-layer-changeset.js';
 function rollup(layers, strict) {
     const statuses = Object.values(layers).map((l) => l.status);
     if (statuses.includes('blocker')) {
@@ -106,9 +53,12 @@ export function runRootVsShared(opts) {
     let distExists = false;
     try {
         const raw = readFileSync(sharedDist, 'utf8');
-        const match = raw.match(/CLI_VERSION\s*=\s*"([^"]+)"/);
-        if (match && match[1] !== undefined) {
-            sharedVersion = match[1];
+        // Quote-tolerant: the old double-quote-only regex blanked on prettier's
+        // single-quoted emit (rid 2026-10-03-release-gate-quote-brittle).
+        const match = raw.match(/CLI_VERSION\s*=\s*(?:"([^"]+)"|'([^']+)'|`([^`]+)`)/);
+        const captured = match?.[1] ?? match?.[2] ?? match?.[3];
+        if (captured !== undefined) {
+            sharedVersion = captured;
         }
         distExists = true;
     }
@@ -128,7 +78,8 @@ export function runRootVsShared(opts) {
         return {
             status: 'blocker',
             message: `peaks-loop-shared/dist/version.js does not contain a parseable CLI_VERSION`,
-            remediation: 'inspect packages/peaks-loop-shared/dist/version.js; ensure CLI_VERSION = "<semver>" is exported',
+            remediation: 'inspect packages/peaks-loop-shared/dist/version.js; ensure CLI_VERSION is exported ' +
+                'as a quoted string literal (single or double quotes both parse)',
             observed: { rootVersion, sharedVersion: null }
         };
     }
@@ -214,41 +165,6 @@ export function runTagCollision(opts) {
     };
 }
 // ---------------------------------------------------------------------------
-// Layer C — changesetStaged (mirrors publish.yml gate-changeset step)
-// ---------------------------------------------------------------------------
-export function runChangesetStaged(opts) {
-    const changesetDir = join(opts.projectRoot, '.changeset');
-    let files = [];
-    try {
-        files = readdirSync(changesetDir);
-    }
-    catch {
-        return {
-            status: 'ok',
-            message: '.changeset/ directory is absent — no changeset is staged',
-            remediation: '',
-            observed: { stagedFiles: [] }
-        };
-    }
-    const staged = files.filter((f) => f.endsWith('.md') && f !== 'README.md');
-    if (staged.length === 0) {
-        return {
-            status: 'ok',
-            message: '.changeset/ has no staged *.md files',
-            remediation: '',
-            observed: { stagedFiles: [] }
-        };
-    }
-    return {
-        status: 'warning',
-        message: `.changeset/ has ${staged.length} staged change(s): ${staged.join(', ')}`,
-        remediation: 'publish.yml gate-changeset will BLOCK publish with these staged files. ' +
-            'Either: (a) run `peaks changeset publish` to drain the staged changesets, ' +
-            '(b) move them out of .changeset/, or (c) re-run with --strict to enforce blocker exit here.',
-        observed: { stagedFiles: staged }
-    };
-}
-// ---------------------------------------------------------------------------
 // Layer D — workspaceLockstep (CLI-only; not in publish.yml today)
 // ---------------------------------------------------------------------------
 export function runWorkspaceLockstep(opts) {

package/dist/services/release/version-precheck-support.d.ts

@@ -0,0 +1,53 @@
+export type LayerStatus = 'ok' | 'warning' | 'blocker';
+export interface LayerResult {
+    readonly status: LayerStatus;
+    readonly message: string;
+    readonly remediation: string;
+    readonly observed?: Readonly<Record<string, unknown>>;
+}
+export interface PrecheckOptions {
+    readonly projectRoot: string;
+    readonly strict?: boolean;
+}
+export interface PrecheckEnvelope {
+    readonly ok: boolean;
+    readonly overall: LayerStatus;
+    /** `null` when the root `package.json` is missing or unreadable — see `rootVsShared`. */
+    readonly rootVersion: string | null;
+    readonly strict: boolean;
+    readonly snapshotAt: string;
+    readonly layers: {
+        readonly rootVsShared: LayerResult;
+        readonly tagCollision: LayerResult;
+        readonly changesetStaged: LayerResult;
+        readonly workspaceLockstep: LayerResult;
+    };
+}
+export declare const SEMVER_RE: RegExp;
+export declare function upgrade(result: LayerResult, strict: boolean | undefined): LayerResult;
+/**
+ * Outcome of reading a `package.json`. Never throws: an unreadable root manifest
+ * is an ordinary precheck blocker, not an unhandled CLI error. Before this, a
+ * project without `package.json` made `peaks release canary` crash with
+ * `{ command: "cli", code: "UNHANDLED_ERROR" }` instead of a `PRECHECK_BLOCKER`
+ * naming the missing file.
+ */
+export type PackageJsonRead<T> = {
+    readonly ok: true;
+    readonly value: T;
+} | {
+    readonly ok: false;
+    readonly message: string;
+    readonly remediation: string;
+};
+/** Read + parse a JSON manifest, mapping both failure modes onto a blocker description. */
+export declare function readPackageJson<T>(path: string, label: string): PackageJsonRead<T>;
+export type RootVersionRead = {
+    readonly ok: true;
+    readonly version: string;
+} | {
+    readonly ok: false;
+    readonly message: string;
+    readonly remediation: string;
+};
+export declare function readRootVersion(projectRoot: string): RootVersionRead;

package/dist/services/release/version-precheck-support.js

@@ -0,0 +1,64 @@
+// src/services/release/version-precheck-support.ts
+//
+// Wave-5 class-A hoist (rid 2026-10-01-wave5-w5-3-compact-release): the type
+// layer and the manifest readers moved VERBATIM out of
+// `version-precheck-service.ts` (:28-132 at the split) for the 300-raw-line
+// cap. The parent re-exports the four public types so
+// './version-precheck-service.js' keeps resolving them
+// (release-commands.ts, tests/unit/release/...).
+import { readFileSync } from 'node:fs';
+import { join } from 'node:path';
+export const SEMVER_RE = /^\d+\.\d+\.\d+(?:-[0-9A-Za-z.-]+)?(?:\+[0-9A-Za-z.-]+)?$/;
+// ---------------------------------------------------------------------------
+// Helpers
+// ---------------------------------------------------------------------------
+export function upgrade(result, strict) {
+    if (strict === true && result.status === 'warning') {
+        return {
+            ...result,
+            status: 'blocker',
+            remediation: `${result.remediation} (strict mode: warning promoted to blocker)`
+        };
+    }
+    return result;
+}
+/** Read + parse a JSON manifest, mapping both failure modes onto a blocker description. */
+export function readPackageJson(path, label) {
+    let raw;
+    try {
+        raw = readFileSync(path, 'utf8');
+    }
+    catch {
+        return {
+            ok: false,
+            message: `${label} is missing or unreadable at ${path}`,
+            remediation: `run \`peaks release precheck\` from a project root that has a readable ${label}`
+        };
+    }
+    try {
+        return { ok: true, value: JSON.parse(raw) };
+    }
+    catch {
+        return {
+            ok: false,
+            message: `${label} at ${path} is not valid JSON`,
+            remediation: `fix the ${label} syntax; \`peaks release precheck\` cannot gate a version it cannot parse`
+        };
+    }
+}
+export function readRootVersion(projectRoot) {
+    const path = join(projectRoot, 'package.json');
+    const read = readPackageJson(path, 'root package.json');
+    if (!read.ok) {
+        return read;
+    }
+    const pkg = read.value;
+    if (typeof pkg.version !== 'string' || !SEMVER_RE.test(pkg.version)) {
+        return {
+            ok: false,
+            message: `invalid root package.json#version: ${JSON.stringify(pkg.version)}`,
+            remediation: 'set root package.json#version to a clean semver (e.g. 4.0.0) before running `peaks release precheck`'
+        };
+    }
+    return { ok: true, version: pkg.version };
+}

package/dist/services/retrospective/retrospective-index.js

@@ -8,8 +8,20 @@
  */
 import { existsSync, readFileSync } from 'node:fs';
 import { join, resolve } from 'node:path';
-const VALID_TYPES = new Set(['refactor', 'feature', 'bugfix', 'config', 'docs', 'chore']);
-const VALID_OUTCOMES = new Set(['shipped', 'blocked', 'in-flight', 'cancelled']);
+const VALID_TYPES = new Set([
+    'refactor',
+    'feature',
+    'bugfix',
+    'config',
+    'docs',
+    'chore'
+]);
+const VALID_OUTCOMES = new Set([
+    'shipped',
+    'blocked',
+    'in-flight',
+    'cancelled'
+]);
 export function loadRetrospectiveIndex(projectRoot) {
     const resolvedRoot = resolve(projectRoot);
     const indexPath = join(resolvedRoot, '.peaks', 'retrospective', 'index.json');
@@ -96,7 +108,9 @@ function isRetrospectiveEntry(value) {
         return false;
     if (!v.keyDecisions.every((decision) => typeof decision === 'string'))
         return false;
-    if (typeof v.lessonsLearned !== 'number' || !Number.isInteger(v.lessonsLearned) || v.lessonsLearned < 0)
+    if (typeof v.lessonsLearned !== 'number' ||
+        !Number.isInteger(v.lessonsLearned) ||
+        v.lessonsLearned < 0)
         return false;
     if (!Array.isArray(v.artifactPaths))
         return false;

package/dist/services/retrospective/retrospective-search-service.js

@@ -57,7 +57,11 @@ export function searchRetrospective(input) {
     if (input.outcome !== undefined) {
         candidates = candidates.filter((e) => e.outcome === input.outcome);
     }
-    const matches = fuzzyMatchWithKey(input.query, candidates, { keyFn: (e) => `${e.title} ${e.summary}`, limit, caseSensitive: false });
+    const matches = fuzzyMatchWithKey(input.query, candidates, {
+        keyFn: (e) => `${e.title} ${e.summary}`,
+        limit,
+        caseSensitive: false
+    });
     return matches.map((m) => {
         const entry = m.item;
         return {
@@ -69,7 +73,7 @@ export function searchRetrospective(input) {
             outcome: entry.outcome,
             artifactPaths: entry.artifactPaths,
             score: m.score,
-            positions: m.positions,
+            positions: m.positions
         };
     });
 }

package/dist/services/retrospective/retrospective-show.js

@@ -60,7 +60,10 @@ export function showRetrospective(options) {
 }
 function synthesizeBody(entry, projectRoot) {
     if (entry.artifactPaths.length === 0) {
-        return { body: renderEntryHeader(entry), warnings: ['entry has no artifactPaths; body is the index summary only'] };
+        return {
+            body: renderEntryHeader(entry),
+            warnings: ['entry has no artifactPaths; body is the index summary only']
+        };
     }
     const sections = [];
     const warnings = [];

package/dist/services/reviewer/model-family.js

@@ -22,7 +22,13 @@ import { createHash } from 'node:crypto';
  */
 const RULES = [
     // Anthropic (incl. bedrock-hosted anthropic) — most specific first.
-    { family: 'claude', test: (m) => /^(claude|anthropic\.)/i.test(m) || /^anthropic\.claude-/i.test(m) || /^us\.anthropic\./i.test(m) || /^bedrock\/.+\/claude-/i.test(m) },
+    {
+        family: 'claude',
+        test: (m) => /^(claude|anthropic\.)/i.test(m) ||
+            /^anthropic\.claude-/i.test(m) ||
+            /^us\.anthropic\./i.test(m) ||
+            /^bedrock\/.+\/claude-/i.test(m)
+    },
     // OpenAI / o-series — order: most specific prefix first.
     { family: 'gpt-5', test: (m) => /^gpt-5/i.test(m) },
     { family: 'gpt-4o', test: (m) => /^gpt-4o/i.test(m) },

package/dist/services/reviewer/providers/anthropic.d.ts

@@ -1,2 +1,7 @@
+/**
+ * anthropic provider — Anthropic Messages API. Reads API key from the
+ * env var named in `provider.apiKeyEnv` (default `ANTHROPIC_API_KEY`).
+ * Per A4 prohibition, NO SDK; we use fetch directly.
+ */
 import type { ProviderCallInput, ProviderCallResult } from './ollama.js';
 export declare function callAnthropic(input: ProviderCallInput): Promise<ProviderCallResult>;

package/dist/services/reviewer/providers/anthropic.js

@@ -1,3 +1,8 @@
+/**
+ * anthropic provider — Anthropic Messages API. Reads API key from the
+ * env var named in `provider.apiKeyEnv` (default `ANTHROPIC_API_KEY`).
+ * Per A4 prohibition, NO SDK; we use fetch directly.
+ */
 const ANTHROPIC_ENDPOINT = 'https://api.anthropic.com/v1/messages';
 const ANTHROPIC_VERSION = '2023-06-08';
 const REQUEST_TIMEOUT_MS = 30_000;
@@ -31,7 +36,11 @@ export async function callAnthropic(input) {
         });
         if (!res.ok) {
             const detail = await safeReadError(res);
-            return { ok: false, error: `anthropic http ${res.status}: ${detail}`, latencyMs: Date.now() - start };
+            return {
+                ok: false,
+                error: `anthropic http ${res.status}: ${detail}`,
+                latencyMs: Date.now() - start
+            };
         }
         const json = (await res.json());
         const textBlock = (json.content ?? []).find((b) => b.type === 'text' && typeof b.text === 'string');

package/dist/services/reviewer/providers/openai.d.ts

@@ -1,2 +1,7 @@
+/**
+ * openai provider — OpenAI Chat Completions. Reads API key from the
+ * env var named in `provider.apiKeyEnv` (default `OPENAI_API_KEY`).
+ * Per A4 prohibition, NO SDK; we use fetch directly.
+ */
 import type { ProviderCallInput, ProviderCallResult } from './ollama.js';
 export declare function callOpenAI(input: ProviderCallInput): Promise<ProviderCallResult>;

package/dist/services/reviewer/providers/openai.js

@@ -1,3 +1,8 @@
+/**
+ * openai provider — OpenAI Chat Completions. Reads API key from the
+ * env var named in `provider.apiKeyEnv` (default `OPENAI_API_KEY`).
+ * Per A4 prohibition, NO SDK; we use fetch directly.
+ */
 const DEFAULT_ENDPOINT = 'https://api.openai.com/v1/chat/completions';
 const DEFAULT_API_KEY_ENV = 'OPENAI_API_KEY';
 const REQUEST_TIMEOUT_MS = 30_000;
@@ -32,7 +37,11 @@ export async function callOpenAI(input) {
         });
         if (!res.ok) {
             const detail = await safeReadError(res);
-            return { ok: false, error: `openai http ${res.status}: ${detail}`, latencyMs: Date.now() - start };
+            return {
+                ok: false,
+                error: `openai http ${res.status}: ${detail}`,
+                latencyMs: Date.now() - start
+            };
         }
         const json = (await res.json());
         const text = typeof json.choices?.[0]?.message?.content === 'string'

package/dist/services/reviewer/reviewer-config.js

@@ -71,11 +71,13 @@ export function loadReviewerConfig(options = {}) {
     if (providers.length < 2)
         return { ok: false, reason: 'no-reviewer-config' };
     const selectionRaw = reviewer['selection'];
-    const selection = typeof selectionRaw === 'string' && VALID_SELECTION.includes(selectionRaw)
+    const selection = typeof selectionRaw === 'string' &&
+        VALID_SELECTION.includes(selectionRaw)
         ? selectionRaw
         : 'round-robin';
     const fallbackRaw = reviewer['fallbackOnError'];
-    const fallbackOnError = typeof fallbackRaw === 'string' && VALID_FALLBACK.includes(fallbackRaw)
+    const fallbackOnError = typeof fallbackRaw === 'string' &&
+        VALID_FALLBACK.includes(fallbackRaw)
         ? fallbackRaw
         : 'skip';
     const rdProviderNameRaw = reviewer['rdProviderName'];

package/dist/services/reviewer/reviewer-service.d.ts

@@ -1,39 +1,5 @@
-import { type SelectionState } from './selection-strategies.js';
-export type ReviewerViolation = {
-    kind: string;
-    file: string;
-    line: number;
-    hint: string;
-};
-export type ReviewerEnvelope = {
-    reviewerId: string;
-    modelId: string;
-    modelFamily: string;
-    passed: boolean;
-    violations: ReviewerViolation[];
-    gateAction: 'block' | 'allow' | 'warn';
-    reason: string;
-};
-export type ReviewerRunInput = {
-    rid: string;
-    /** Slice context blob fed to the prompt. Plain text — no PII (A1.5). */
-    context: string;
-    /** Optional override of the selection-mode state for cross-slice round-robin. */
-    state?: SelectionState;
-    /** Optional injected fetch (testability). */
-    fetchImpl?: typeof fetch;
-    /** Optional injected rng (random mode tests). */
-    rng?: () => number;
-};
-export type ReviewerRunOutput = {
-    ok: true;
-    envelope: ReviewerEnvelope;
-    nextState: SelectionState;
-} | {
-    ok: false;
-    reason: 'no-reviewer-config';
-    envelope: ReviewerEnvelope;
-};
+import { type ReviewerEnvelope, type ReviewerRunInput, type ReviewerRunOutput } from './reviewer-types.js';
+export type { ReviewerViolation, ReviewerEnvelope, ReviewerRunInput, ReviewerRunOutput } from './reviewer-types.js';
 export declare const REVIEWER_ID = "third-party-reviewer-v2.14.0";
 /**
  * Validate a parsed JSON value against the ReviewerEnvelope contract

package/dist/services/reviewer/reviewer-service.js

@@ -122,7 +122,8 @@ export function extractFirstJsonObject(text) {
                     try {
                         return JSON.parse(candidate.slice(start, i + 1));
                     }
-                    catch { // TODO(g2): legacy silent catch — grace: 1 minor release (v2.14.0)
+                    catch {
+                        // TODO(g2): legacy silent catch — grace: 1 minor release (v2.14.0)
                         // TODO(g2): brace-balanced scan may slice mid-string or trap on malformed JSON;
                         // null sentinel is intentional for slice B-1 reviewer parser — caller distinguishes
                         // via ParseResult.ok flag at the call site (extractFirstJsonObject contract).

package/dist/services/reviewer/reviewer-types.d.ts

@@ -0,0 +1,43 @@
+/**
+ * reviewer-types.ts — public type surface of the G4 third-party reviewer.
+ *
+ * Extracted verbatim from `reviewer-service.ts` for the 300-line file-size
+ * cap (`.peaks/docs/lint-gate.md` §4 row 5). `reviewer-service.ts` re-exports
+ * every name below, so importers keep using the original path unchanged.
+ */
+import { type SelectionState } from './selection-strategies.js';
+export type ReviewerViolation = {
+    kind: string;
+    file: string;
+    line: number;
+    hint: string;
+};
+export type ReviewerEnvelope = {
+    reviewerId: string;
+    modelId: string;
+    modelFamily: string;
+    passed: boolean;
+    violations: ReviewerViolation[];
+    gateAction: 'block' | 'allow' | 'warn';
+    reason: string;
+};
+export type ReviewerRunInput = {
+    rid: string;
+    /** Slice context blob fed to the prompt. Plain text — no PII (A1.5). */
+    context: string;
+    /** Optional override of the selection-mode state for cross-slice round-robin. */
+    state?: SelectionState;
+    /** Optional injected fetch (testability). */
+    fetchImpl?: typeof fetch;
+    /** Optional injected rng (random mode tests). */
+    rng?: () => number;
+};
+export type ReviewerRunOutput = {
+    ok: true;
+    envelope: ReviewerEnvelope;
+    nextState: SelectionState;
+} | {
+    ok: false;
+    reason: 'no-reviewer-config';
+    envelope: ReviewerEnvelope;
+};

package/dist/services/reviewer/reviewer-types.js

@@ -0,0 +1 @@
+export {};

package/dist/services/role/role-registry.js

@@ -79,5 +79,13 @@ export const DEFAULT_PERMISSIONS = [
 export const DEFAULT_SENIOR_FE_ROLE = {
     name: 'senior-fe',
     description: 'Senior Frontend (业务资深 + 后端半盲 + 24h AI 程序员)',
-    permissions: ['prd.write', 'prd.confirm', 'rd.review', 'qa.accept', 'qa.reject', 'smoke.define', 'smoke.run']
+    permissions: [
+        'prd.write',
+        'prd.confirm',
+        'rd.review',
+        'qa.accept',
+        'qa.reject',
+        'smoke.define',
+        'smoke.run'
+    ]
 };

package/dist/services/runtime/runtime-detector.js

@@ -50,7 +50,10 @@ function detectCopilot(opts) {
  *  unknown. The first match wins. */
 export function detectRuntime(opts = {}) {
     if (detectClaudeCode(opts)) {
-        return { vendor: 'claude-code', reason: 'CLAUDE_CODE_ENTRYPOINT / CLAUDE_CODE / ~/.claude matched' };
+        return {
+            vendor: 'claude-code',
+            reason: 'CLAUDE_CODE_ENTRYPOINT / CLAUDE_CODE / ~/.claude matched'
+        };
     }
     if (detectCodex(opts)) {
         return { vendor: 'codex', reason: 'CODEX_HOME / ~/.codex matched' };