peaks-loop 4.0.0-beta.3 → 4.0.0-beta.30

package/dist/src/cli/commands/qa-commands.d.ts

@@ -1,94 +0,0 @@
-/**
- * Slice 2026-06-16-playwright-restart-loop — G1, G2, G4, G5, G6.
- *
- * `peaks qa run` — runs a peaks-qa slice for the active project.
- *
- * The slice is the user-facing surface for the QA gate family
- * (functional / security / browser E2E). The browser-E2E gate
- * specifically is what the restart-loop detector guards. The
- * detector is wired into the slice here:
- *
- *   1. We construct a `BrowserRestartDetector` (configurable via
- *      `--max-browser-restarts` / `--no-restart-detector`).
- *   2. We construct a `BrowserEventLogger` that writes the
- *      per-slice JSONL log to
- *      `.peaks/_runtime/<session-id>/qa/browser-events.jsonl`.
- *   3. The slice entry exposes `runQaSlice` so tests can
- *      feed a synthetic event log and assert the diagnostic.
- *
- * CLI surface (PRD G5 + AC4/AC5/AC6):
- *   peaks qa run --project <X> [--no-browser] [--max-browser-restarts N]
- *                       [--no-restart-detector] [--session-id <sid>] [--json]
- *
- * `--no-browser` (G5) skips the browser E2E gate entirely. The
- * resulting gate list marks the browser gate as `status: skipped`
- * with reason `--no-browser`. This is a slice-level opt-out for
- * backend-only or already-covered-by-unit-tests work.
- */
-import { Command } from 'commander';
-import { type ProgramIO } from '../cli-helpers.js';
-import { type BrowserEvent } from '../../services/qa/browser-restart-detector.js';
-import type { MutReportJson } from '../../services/mut/types.js';
-export type QaRunOptions = {
-    project: string;
-    browser?: boolean;
-    restartDetector?: boolean;
-    maxBrowserRestarts?: string;
-    sessionId?: string;
-    json?: boolean;
-    mutation?: boolean;
-};
-export type QaGateStatus = {
-    readonly name: string;
-    readonly status: 'passed' | 'skipped' | 'failed';
-    readonly reason?: string;
-    /**
-     * Plan 2 / Task 8 — when the gate consumes an artifact with a
-     * deterministic signature, this field carries that signature for
-     * downstream audit-trail consumers. Currently set only on the
-     * `mutation` gate to MUT.sig (the `sha256` of mut-report.json).
-     */
-    readonly mutSig?: string;
-};
-export type QaRunResult = {
-    readonly sessionId: string;
-    readonly project: string;
-    readonly browserEnabled: boolean;
-    readonly gates: readonly QaGateStatus[];
-    readonly detectorTriggered: boolean;
-    readonly diagnostic?: string;
-    readonly subAgentPromptHint: string;
-    /**
-     * Plan 2 / Task 8 — MUT.sig when peaks-mut ran in this session.
-     * Undefined when peaks-mut was not run (no mut-report.json) or
-     * the user passed --no-mutation.
-     */
-    readonly mutSig?: string;
-};
-export declare const DEFAULT_MAX_BROWSER_RESTARTS = 3;
-export type RunQaSliceInput = {
-    readonly project: string;
-    readonly sessionId: string;
-    readonly browserEnabled: boolean;
-    readonly maxRestarts: number;
-    readonly detectorEnabled: boolean;
-    readonly events: readonly BrowserEvent[];
-    readonly mutationReport: MutReportJson | null;
-    readonly mutationEnabled: boolean;
-};
-export declare function runQaSlice(input: RunQaSliceInput): QaRunResult;
-/**
- * Commander 12.x sets `options.X = false` for `--no-X` (POSITIVE form),
- * not `options.noX = true`. Reads use the positive form; the default is
- * `true` and the boolean form is `false` only when the user passes `--no-X`.
- *
- * Exported for unit testing the parser contract in isolation from
- * the rest of the action handler.
- */
-export declare function readQaRunOptions(options: QaRunOptions): {
-    readonly browserEnabled: boolean;
-    readonly detectorEnabled: boolean;
-    readonly maxRestarts: number;
-    readonly mutationEnabled: boolean;
-};
-export declare function registerQaCommands(program: Command, io: ProgramIO): void;

package/dist/src/cli/commands/qa-commands.js

@@ -1,271 +0,0 @@
-/**
- * Slice 2026-06-16-playwright-restart-loop — G1, G2, G4, G5, G6.
- *
- * `peaks qa run` — runs a peaks-qa slice for the active project.
- *
- * The slice is the user-facing surface for the QA gate family
- * (functional / security / browser E2E). The browser-E2E gate
- * specifically is what the restart-loop detector guards. The
- * detector is wired into the slice here:
- *
- *   1. We construct a `BrowserRestartDetector` (configurable via
- *      `--max-browser-restarts` / `--no-restart-detector`).
- *   2. We construct a `BrowserEventLogger` that writes the
- *      per-slice JSONL log to
- *      `.peaks/_runtime/<session-id>/qa/browser-events.jsonl`.
- *   3. The slice entry exposes `runQaSlice` so tests can
- *      feed a synthetic event log and assert the diagnostic.
- *
- * CLI surface (PRD G5 + AC4/AC5/AC6):
- *   peaks qa run --project <X> [--no-browser] [--max-browser-restarts N]
- *                       [--no-restart-detector] [--session-id <sid>] [--json]
- *
- * `--no-browser` (G5) skips the browser E2E gate entirely. The
- * resulting gate list marks the browser gate as `status: skipped`
- * with reason `--no-browser`. This is a slice-level opt-out for
- * backend-only or already-covered-by-unit-tests work.
- */
-import { join } from 'node:path';
-import { addJsonOption, printResult } from '../cli-helpers.js';
-import { fail, ok } from '../../shared/result.js';
-import { BrowserRestartDetector } from '../../services/qa/browser-restart-detector.js';
-import { BrowserEventLogger } from '../../services/qa/browser-event-logger.js';
-import { BROWSER_REUSE_HINT } from '../../services/qa/browser-reuse-hint.js';
-// Plan 1 / Task 9 — auto-build peaks-context before peaks-qa runs.
-import { buildContext } from '../../services/context/context-builder.js';
-// Plan 1 / Task 10 — production fetcher (replaces mockFetcher).
-import { createHeadroomFetcher } from '../../services/context/headroom-fetcher.js';
-// Plan 2 / Task 8 — consume MUT.sig from peaks-mut into verdict envelope.
-import { loadMutReport, mutReportPath } from '../../services/mut/index.js';
-function buildHeadroomFetcher(sid) {
-    return createHeadroomFetcher({
-        cacheDir: `.peaks/_runtime/${sid}/doc-cache`,
-        // remoteFetcher wired in a future slice (headroom-ai programmatic API).
-    });
-}
-async function ensureContextForQa(goal, project, sid) {
-    const out = `.peaks/_runtime/${sid}/context.json`;
-    try {
-        await buildContext({
-            goal,
-            project,
-            audience: 'peaks-qa',
-            depsMode: 'locked',
-            docBudgetTokens: 8000,
-            out,
-            fetcher: buildHeadroomFetcher(sid),
-        });
-    }
-    catch (error) {
-        // Plan 1 / Task 9 — context is a pre-step, not a precondition.
-        // Task 11 will upgrade this to a hard precondition once the qa
-        // slice actually consumes context.json.
-        const message = error instanceof Error ? error.message : 'unknown context build failure';
-        process.stderr.write(`[peaks-context] qa pre-step skipped: ${message}\n`);
-    }
-}
-export const DEFAULT_MAX_BROWSER_RESTARTS = 3;
-export function runQaSlice(input) {
-    const detector = new BrowserRestartDetector({
-        maxRestarts: input.maxRestarts,
-        enabled: input.detectorEnabled
-    });
-    const logPath = join(input.project, '.peaks', '_runtime', input.sessionId, 'qa', 'browser-events.jsonl');
-    const logger = new BrowserEventLogger({ filePath: logPath });
-    let pendingCloseTs = null;
-    // PRD AC4 + P1: when browser E2E is off, the detector must NOT
-    // record events at all (otherwise a user passing both
-    // --no-browser AND a hot detector would see spurious
-    // detectorTriggered=true and exit code 2).
-    if (input.browserEnabled) {
-        for (const ev of input.events) {
-            detector.record(ev);
-            if (ev.tool === 'browser_close') {
-                pendingCloseTs = ev.ts;
-            }
-            else if (ev.tool === 'browser_navigate' && pendingCloseTs !== null) {
-                const closeMs = Date.parse(pendingCloseTs);
-                const navMs = Date.parse(ev.ts);
-                logger.append({
-                    kind: 'spurious_restart',
-                    ts: ev.ts,
-                    sessionId: input.sessionId,
-                    closeTs: pendingCloseTs,
-                    navigateTs: ev.ts,
-                    deltaMs: navMs - closeMs
-                });
-                pendingCloseTs = null;
-            }
-            else if (ev.tool === 'browser_navigate') {
-                pendingCloseTs = null;
-            }
-        }
-    }
-    const browserGate = input.browserEnabled
-        ? detector.shouldHalt()
-            ? { name: 'browser-e2e', status: 'failed', reason: detector.diagnostic() }
-            : { name: 'browser-e2e', status: 'passed' }
-        : { name: 'browser-e2e', status: 'skipped', reason: '--no-browser' };
-    // Plan 2 / Task 8 — MUT.sig gate.
-    // Gate semantics (per spec lines 1292-1341):
-    //   - `skipped` when peaks-mut was not run (no report) OR --no-mutation
-    //   - `passed`  when the report's `thresholds.passed` is true
-    //   - `failed`  when the report's `thresholds.passed` is false
-    // "Skipped" is NOT a failure — the gate is a no-op so existing
-    // sessions that have not yet adopted peaks-mut keep passing.
-    const mutationGate = !input.mutationEnabled
-        ? { name: 'mutation', status: 'skipped', reason: '--no-mutation' }
-        : input.mutationReport === null
-            ? { name: 'mutation', status: 'skipped', reason: 'peaks mut not run' }
-            : input.mutationReport.thresholds.passed
-                ? {
-                    name: 'mutation',
-                    status: 'passed',
-                    mutSig: input.mutationReport.sha256
-                }
-                : {
-                    name: 'mutation',
-                    status: 'failed',
-                    reason: deriveMutationFailureReason(input.mutationReport),
-                    mutSig: input.mutationReport.sha256
-                };
-    const gates = [
-        { name: 'functional', status: 'passed' },
-        { name: 'security', status: 'passed' },
-        browserGate,
-        mutationGate
-    ];
-    return {
-        sessionId: input.sessionId,
-        project: input.project,
-        browserEnabled: input.browserEnabled,
-        gates,
-        detectorTriggered: detector.shouldHalt(),
-        ...(detector.shouldHalt() ? { diagnostic: detector.diagnostic() } : {}),
-        subAgentPromptHint: BROWSER_REUSE_HINT,
-        // Surface MUT.sig ONLY when the gate is actually evaluated
-        // (mutation enabled AND a report was loaded). --no-mutation or
-        // missing report -> undefined, so downstream consumers cannot
-        // mistake a skipped gate for a chain-trail anchor.
-        ...(input.mutationEnabled && input.mutationReport !== null
-            ? { mutSig: input.mutationReport.sha256 }
-            : {})
-    };
-}
-/**
- * Plan 2 / Task 8 — pure helper that explains WHY the mutation gate
- * failed. Used only when the report's `thresholds.passed` is false.
- * Reads the threshold block from the report itself (already
- * CLI-computed by `buildMutReport`) so the message matches what
- * peaks-mut's CLI surface prints.
- */
-function deriveMutationFailureReason(report) {
-    const t = report.thresholds;
-    const fragments = [];
-    if (report.mutation.killRate < t.mutationKillRateMin) {
-        fragments.push(`mutation kill rate ${report.mutation.killRate.toFixed(2)} < ${t.mutationKillRateMin.toFixed(2)}`);
-    }
-    if (report.assertions.weakRate > t.weakAssertionRateMax) {
-        fragments.push(`weak assertion rate ${report.assertions.weakRate.toFixed(2)} > ${t.weakAssertionRateMax.toFixed(2)}`);
-    }
-    if (fragments.length === 0) {
-        return `mutation thresholds failed (MUT.sig=${report.sha256})`;
-    }
-    return `${fragments.join('; ')} (MUT.sig=${report.sha256})`;
-}
-function resolveMaxRestarts(raw) {
-    if (raw === undefined)
-        return DEFAULT_MAX_BROWSER_RESTARTS;
-    const n = Number.parseInt(raw, 10);
-    if (Number.isNaN(n) || n < 1) {
-        return DEFAULT_MAX_BROWSER_RESTARTS;
-    }
-    return n;
-}
-/**
- * Commander 12.x sets `options.X = false` for `--no-X` (POSITIVE form),
- * not `options.noX = true`. Reads use the positive form; the default is
- * `true` and the boolean form is `false` only when the user passes `--no-X`.
- *
- * Exported for unit testing the parser contract in isolation from
- * the rest of the action handler.
- */
-export function readQaRunOptions(options) {
-    return {
-        browserEnabled: options.browser !== false,
-        detectorEnabled: options.restartDetector !== false,
-        maxRestarts: resolveMaxRestarts(options.maxBrowserRestarts),
-        mutationEnabled: options.mutation !== false
-    };
-}
-export function registerQaCommands(program, io) {
-    const qa = program
-        .command('qa')
-        .description('peaks-qa slice: run QA gates (functional / security / browser E2E / mutation) for the active project');
-    addJsonOption(qa
-        .command('run')
-        .description('Run the peaks-qa slice (PRD 2026-06-16-playwright-restart-loop; Plan 2 mut gate)')
-        .option('--project <path>', 'project the gates evaluate against (default: current directory)', '.')
-        .option('--session-id <sid>', 'session id; defaults to "ad-hoc" for one-shot runs', 'ad-hoc')
-        .option('--no-browser', 'skip the browser E2E gate entirely (PRD G5 / AC4)')
-        .option('--max-browser-restarts <n>', 'halt threshold for close->navigate pairs in this slice (default 3, PRD AC5)', String(DEFAULT_MAX_BROWSER_RESTARTS))
-        .option('--no-restart-detector', 'disable the restart-loop detector escape hatch (PRD AC6)')
-        // Plan 2 / Task 8 — MUT.sig gate opt-out (mirrors --no-browser).
-        .option('--no-mutation', 'skip the mutation gate even if .peaks/_runtime/<sid>/mut/mut-report.json exists')).action(async (options) => {
-        try {
-            const { browserEnabled, detectorEnabled, maxRestarts, mutationEnabled } = readQaRunOptions(options);
-            // Plan 1 / Task 9 — pre-build peaks-context before peaks-qa runs.
-            // Goal is audience-scoped doc retrieval only; pass a placeholder
-            // when the user did not supply one.
-            const qaProject = options.project;
-            const qaSid = options.sessionId ?? 'ad-hoc';
-            await ensureContextForQa('qa gate run', qaProject, qaSid);
-            // Plan 2 / Task 8 — load peaks-mut's report (MUT.sig) if present.
-            // Returns null when peaks-mut was not run; the gate treats that
-            // as `skipped`, never as `failed`. loadMutReport itself never
-            // throws (per its docstring) so this is safe in the action.
-            const mutationReport = mutationEnabled
-                ? await loadMutReport(qaSid)
-                : null;
-            // Production slice: no synthetic events to feed; in real
-            // dogfood the LLM tool dispatcher would push events into the
-            // detector. Here we record an empty event log so the
-            // gate-list shape is stable for downstream parsers.
-            const result = runQaSlice({
-                project: options.project,
-                sessionId: options.sessionId ?? 'ad-hoc',
-                browserEnabled,
-                maxRestarts,
-                detectorEnabled,
-                events: [],
-                mutationReport,
-                mutationEnabled
-            });
-            const mutationGate = result.gates.find((g) => g.name === 'mutation');
-            const mutationFailed = mutationGate?.status === 'failed';
-            const nextActions = result.detectorTriggered
-                ? [
-                    'Stop the slice and inspect the diagnostic above',
-                    'Re-run with --no-restart-detector if the close/reopen was intentional',
-                    'See .peaks/memory/playwright-restart-loop-2026-06-16.md'
-                ]
-                : mutationFailed
-                    ? [
-                        'Re-run peaks mut and address the threshold breaches (see reason)',
-                        'Re-run with --no-mutation to bypass the gate for this slice',
-                        `mut-report path: ${mutReportPath(qaSid)}`
-                    ]
-                    : browserEnabled
-                        ? ['No action required; browser gate passed']
-                        : ['Browser E2E skipped; run without --no-browser when the slice needs E2E'];
-            printResult(io, ok('qa.run', result, [], nextActions), options.json);
-            if (result.detectorTriggered || mutationFailed) {
-                process.exitCode = 2;
-            }
-        }
-        catch (error) {
-            printResult(io, fail('qa.run', 'QA_RUN_FAILED', error instanceof Error ? error.message : 'Unknown error', { project: options.project }, ['Check the project path and rerun']), options.json);
-            process.exitCode = 1;
-        }
-    });
-}

package/dist/src/cli/commands/release-commands.js

@@ -1,231 +0,0 @@
-/**
- * v2.15.0 follow-up — G15: release / hotfix CLI.
- *
- *   - `peaks release plan <version>`            — start a new release
- *   - `peaks release canary --percent <10|50>`   — advance to canary stage
- *   - `peaks release promote`                   — promote to 100% + start watch
- *   - `peaks release watch`                     — show watch window status
- *   - `peaks release rollback`                  — emergency rollback
- *   - `peaks release hotfix <version>`          — start a hotfix (forces
- *                                                  rollback of any active
- *                                                  release; skips 'planned'
- *                                                  stage)
- *
- * State machine: planned → canary-10 → canary-50 → promoted → watching → done
- * Side branches: → rolled-back (from any pre-done stage), → hotfixed (from watching).
- *
- * Real deployment (k8s rollout, LB config, monitoring integration) is
- * OUT OF SCOPE for this slice.
- */
-import { findProjectRoot } from '../../services/config/config-safety.js';
-import { hotfixRelease, isReleaseStage, planRelease, readReleaseState, rollbackRelease, transitionRelease, watchWindow, writeReleaseState } from '../../services/release/release-state.js';
-import { fail, ok } from '../../shared/result.js';
-import { addJsonOption, printResult } from '../cli-helpers.js';
-const CANARY_PERCENTS = {
-    10: 'canary-10',
-    50: 'canary-50'
-};
-export function registerReleaseCommands(program, io) {
-    const release = program
-        .command('release')
-        .description('v2.15.0 follow-up G15: canary → promote → watch → done / hotfix state machine.');
-    // 1. plan
-    addJsonOption(release
-        .command('plan <version>')
-        .description('Start a new release. Stores the version in the canary pipeline state. ' +
-        'Fails when there is already an active release in any non-terminal stage.')
-        .option('--project <path>', 'project root (default: cwd)')).action((version, opts) => {
-        const projectRoot = opts.project ?? findProjectRoot(process.cwd()) ?? process.cwd();
-        const state = readReleaseState(projectRoot);
-        const result = planRelease(state, version);
-        if ('error' in result) {
-            printResult(io, fail('release.plan', 'CONFLICT', result.error, { projectRoot }, [
-                'Run `peaks release rollback` or `peaks release hotfix` to clear the active release.'
-            ]), opts.json ?? false);
-            return;
-        }
-        writeReleaseState(projectRoot, result.state);
-        printResult(io, ok('release.plan', {
-            projectRoot,
-            version: result.record.version,
-            currentStage: result.record.currentStage
-        }, [], [
-            'Run `peaks release canary --percent 10` to begin the canary phase.'
-        ]), opts.json ?? false);
-    });
-    // 2. canary
-    addJsonOption(release
-        .command('canary')
-        .description('Advance the active release to a canary stage. Two percent values supported: ' +
-        '10 (first canary, requires stage=planned) and 50 (second canary, requires ' +
-        'stage=canary-10). Use `peaks release promote` to complete.')
-        .requiredOption('--percent <10|50>', 'canary percent (10 or 50)')
-        .option('--note <text>', 'optional note for the stage transition')
-        .option('--project <path>', 'project root (default: cwd)')).action((opts) => {
-        const percent = Number.parseInt(opts.percent, 10);
-        if (percent !== 10 && percent !== 50) {
-            printResult(io, fail('release.canary', 'INVALID_PERCENT', `--percent must be 10 or 50 (got "${opts.percent}")`, {}, []), opts.json ?? false);
-            return;
-        }
-        const projectRoot = opts.project ?? findProjectRoot(process.cwd()) ?? process.cwd();
-        const state = readReleaseState(projectRoot);
-        const targetStage = CANARY_PERCENTS[percent];
-        const result = transitionRelease(state, targetStage, opts.note);
-        if ('error' in result) {
-            printResult(io, fail('release.canary', 'INVALID_TRANSITION', result.error, { projectRoot }, []), opts.json ?? false);
-            return;
-        }
-        writeReleaseState(projectRoot, result.state);
-        printResult(io, ok('release.canary', {
-            projectRoot,
-            percent,
-            currentStage: targetStage,
-            nextAction: percent === 10
-                ? 'peaks release canary --percent 50'
-                : 'peaks release promote'
-        }, [], []), opts.json ?? false);
-    });
-    // 3. promote
-    addJsonOption(release
-        .command('promote')
-        .description('Promote the canary to 100% (full release). Requires stage=canary-50. ' +
-        'Records the promoted-at timestamp and starts the 24h watch window.')
-        .option('--note <text>', 'optional note')
-        .option('--project <path>', 'project root (default: cwd)')).action((opts) => {
-        const projectRoot = opts.project ?? findProjectRoot(process.cwd()) ?? process.cwd();
-        const state = readReleaseState(projectRoot);
-        const result = transitionRelease(state, 'promoted', opts.note);
-        if ('error' in result) {
-            printResult(io, fail('release.promote', 'INVALID_TRANSITION', result.error, { projectRoot }, []), opts.json ?? false);
-            return;
-        }
-        writeReleaseState(projectRoot, result.state);
-        printResult(io, ok('release.promote', {
-            projectRoot,
-            currentStage: 'promoted',
-            promotedAt: result.state.active?.promotedAt
-        }, [], [
-            'Watch window started. Run `peaks release watch` to check progress; `peaks release rollback` for emergency.'
-        ]), opts.json ?? false);
-    });
-    // 4. watch
-    addJsonOption(release
-        .command('watch')
-        .description('Show the watch window status for the current promoted release. ' +
-        '24h window from promoted-at. After the window, run `peaks release done` ' +
-        'to mark the release complete.')
-        .option('--project <path>', 'project root (default: cwd)')).action((opts) => {
-        const projectRoot = opts.project ?? findProjectRoot(process.cwd()) ?? process.cwd();
-        const state = readReleaseState(projectRoot);
-        if (state.active === null) {
-            printResult(io, fail('release.watch', 'NO_ACTIVE', 'no active release to watch', { projectRoot }, [
-                'Run `peaks release plan <version>` to start one.'
-            ]), opts.json ?? false);
-            return;
-        }
-        const win = watchWindow(state.active);
-        const readyForDone = win.percentComplete >= 1.0;
-        printResult(io, ok('release.watch', {
-            projectRoot,
-            version: state.active.version,
-            currentStage: state.active.currentStage,
-            window: {
-                elapsedMs: win.elapsedMs,
-                remainingMs: win.remainingMs,
-                windowMs: win.windowMs,
-                percentComplete: Math.round(win.percentComplete * 100) / 100
-            },
-            readyForDone
-        }, readyForDone
-            ? ['Watch window complete. Run `peaks release done` to mark the release done.']
-            : []), opts.json ?? false);
-    });
-    // 5. done (implicit helper; not in original spec but useful)
-    addJsonOption(release
-        .command('done')
-        .description('Mark the active release as done. Requires the watch window to be ' +
-        'complete (24h after promoted-at).')
-        .option('--project <path>', 'project root (default: cwd)')).action((opts) => {
-        const projectRoot = opts.project ?? findProjectRoot(process.cwd()) ?? process.cwd();
-        const state = readReleaseState(projectRoot);
-        if (state.active === null) {
-            printResult(io, fail('release.done', 'NO_ACTIVE', 'no active release', { projectRoot }, []), opts.json ?? false);
-            return;
-        }
-        if (state.active.currentStage !== 'watching') {
-            printResult(io, fail('release.done', 'INVALID_STAGE', `must be in 'watching' stage to mark done (current: ${state.active.currentStage})`, { projectRoot }, [
-                'Run `peaks release watch` to check progress; transition requires the watch window to complete.'
-            ]), opts.json ?? false);
-            return;
-        }
-        const win = watchWindow(state.active);
-        if (win.percentComplete < 1.0) {
-            printResult(io, fail('release.done', 'WATCH_INCOMPLETE', `watch window not yet complete (${Math.round(win.percentComplete * 100)}% elapsed)`, { projectRoot }, []), opts.json ?? false);
-            return;
-        }
-        const result = transitionRelease(state, 'done');
-        if ('error' in result) {
-            printResult(io, fail('release.done', 'INVALID_TRANSITION', result.error, { projectRoot }, []), opts.json ?? false);
-            return;
-        }
-        // Move to history.
-        const finalRecord = result.state.active;
-        const newState = { version: 1, active: null, history: [...result.state.history, finalRecord] };
-        writeReleaseState(projectRoot, newState);
-        printResult(io, ok('release.done', {
-            projectRoot,
-            version: finalRecord.version,
-            doneAt: finalRecord.doneAt
-        }, [], []), opts.json ?? false);
-    });
-    // 6. rollback
-    addJsonOption(release
-        .command('rollback')
-        .description('Emergency rollback of the active release. Moves the active release to ' +
-        'the history with currentStage=rolled-back. Available from any ' +
-        'pre-done stage.')
-        .option('--note <text>', 'optional rollback reason')
-        .option('--project <path>', 'project root (default: cwd)')).action((opts) => {
-        const projectRoot = opts.project ?? findProjectRoot(process.cwd()) ?? process.cwd();
-        const state = readReleaseState(projectRoot);
-        const result = rollbackRelease(state, opts.note);
-        if ('error' in result) {
-            printResult(io, fail('release.rollback', 'INVALID_TRANSITION', result.error, { projectRoot }, []), opts.json ?? false);
-            return;
-        }
-        writeReleaseState(projectRoot, result.state);
-        printResult(io, ok('release.rollback', {
-            projectRoot,
-            rolledBack: result.record.version,
-            finalStage: result.record.currentStage
-        }, [], [
-            'Run `peaks release hotfix <version>` to start a hotfix on the previous release.'
-        ]), opts.json ?? false);
-    });
-    // 7. hotfix
-    addJsonOption(release
-        .command('hotfix <version>')
-        .description('Start a hotfix release. Forces a rollback of any active release, ' +
-        'skips the `planned` stage, and starts the new release at canary-10. ' +
-        'Use this for紧急修复 — minimal ceremony, no full prd ceremony.')
-        .option('--note <text>', 'optional hotfix note')
-        .option('--project <path>', 'project root (default: cwd)')).action((version, opts) => {
-        const projectRoot = opts.project ?? findProjectRoot(process.cwd()) ?? process.cwd();
-        const state = readReleaseState(projectRoot);
-        const result = hotfixRelease(state, version, opts.note);
-        if ('error' in result) {
-            printResult(io, fail('release.hotfix', 'HOTFIX_FAILED', result.error, { projectRoot }, []), opts.json ?? false);
-            return;
-        }
-        writeReleaseState(projectRoot, result.state);
-        printResult(io, ok('release.hotfix', {
-            projectRoot,
-            version: result.record.version,
-            currentStage: result.record.currentStage
-        }, [], [
-            'Hotfix started at canary-10. Run `peaks release canary --percent 50` to advance.'
-        ]), opts.json ?? false);
-    });
-}
-// Re-export for tests / external consumers.
-export { isReleaseStage };