peaks-loop 4.1.0 → 4.1.2

package/dist/services/upgrade/1x-detector-service.js

@@ -1,96 +0,0 @@
-/**
- * 1.x → 2.0 detection service — TypeScript mirror of
- * `scripts/install-skills.mjs:detect1xProjectState`.
- *
- * Slice: 2026-06-12-code-step-0-55-1x-detection.
- *
- * The canonical implementation lives in the .mjs postinstall
- * (because the postinstall runs before any TS compile step).
- * This TS mirror exists so the peaks-code skill can call
- * `peaks upgrade --detect-1x --project <root> --json` and
- * read a structured JSON envelope to gate the
- * AskUserQuestion that prompts the 1.x → 2.0 upgrade.
- *
- * The two implementations MUST stay in parity. The
- * `tests/integration/upgrade/1x-detector-parity.test.ts`
- * test exercises both on the same fixture and asserts
- * their outputs match.
- */
-import { existsSync, readFileSync } from 'node:fs';
-import { homedir } from 'node:os';
-import { dirname, join } from 'node:path';
-const MAX_WALK_UP = 8;
-export function detect1xProjectState(cwd = process.cwd()) {
-    const home = homedir();
-    const signals = [];
-    let projectRoot = null;
-    let configPath = null;
-    // Walk up from cwd looking for .peaks/_runtime (signals
-    // we're inside a peaks project).
-    let dir = cwd;
-    for (let i = 0; i < MAX_WALK_UP; i += 1) {
-        const peaksRuntime = join(dir, '.peaks', '_runtime');
-        if (existsSync(peaksRuntime)) {
-            projectRoot = dir;
-            break;
-        }
-        const parent = dirname(dir);
-        if (parent === dir)
-            break;
-        dir = parent;
-    }
-    // Signal 1: ~/.peaks/config.json with 1.x version
-    const globalConfig = join(home, '.peaks', 'config.json');
-    if (existsSync(globalConfig)) {
-        try {
-            const raw = JSON.parse(readFileSync(globalConfig, 'utf8'));
-            if (typeof raw['version'] === 'string' && /^1\./.test(raw['version'])) {
-                signals.push(`global config at ${globalConfig} is 1.x (${raw['version']})`);
-                if (configPath === null)
-                    configPath = globalConfig;
-            }
-        }
-        catch {
-            // TODO(g2): legacy silent catch — grace: 1 minor release (v2.14.0)
-            // ignore parse error — the 1.x detection is best-effort
-        }
-    }
-    // Signal 2: .claude/rules/common/dev-preference.md with peaks progress
-    if (projectRoot !== null) {
-        const devPref = join(projectRoot, '.claude', 'rules', 'common', 'dev-preference.md');
-        if (existsSync(devPref)) {
-            try {
-                const body = readFileSync(devPref, 'utf8');
-                if (/peaks progress/i.test(body)) {
-                    signals.push(`${devPref} references "peaks progress" (1.x CLI surface, removed in slice #014)`);
-                }
-            }
-            catch {
-                // TODO(g2): legacy silent catch — grace: 1 minor release (v2.14.0)
-                // ignore
-            }
-        }
-        // Signal 3: project preferences.json missing or 1.x
-        const prefs = join(projectRoot, '.peaks', 'preferences.json');
-        if (!existsSync(prefs)) {
-            signals.push(`${prefs} does not exist (1.x project never migrated)`);
-        }
-        else {
-            try {
-                const raw = JSON.parse(readFileSync(prefs, 'utf8'));
-                if (raw['schema_version'] !== '2.0.0') {
-                    signals.push(`${prefs} has schema_version ${JSON.stringify(raw['schema_version'])}, expected '2.0.0'`);
-                }
-            }
-            catch {
-                signals.push(`${prefs} exists but is not valid JSON`);
-            }
-        }
-    }
-    return {
-        isOneX: signals.length > 0,
-        signals,
-        projectRoot,
-        configPath
-    };
-}

package/dist/services/upgrade/gitignore-migrate-service.d.ts

@@ -1,56 +0,0 @@
-/**
- * The 2.0 canonical ignore block. Mirrors the same paths
- * peaks-loop's own .gitignore uses; kept in lockstep manually
- * (the peaks-loop root .gitignore is the source of truth).
- *
- * Sentinel: every emitted line carries the marker comment so
- * the migration can detect "already-migrated" and stay
- * idempotent.
- */
-export declare const PEAKS_2_0_BLOCK_SENTINEL = "# peaks-loop 2.0 canonical ignore block \u2014 managed by `peaks upgrade --to 2.0`";
-export declare const CANONICAL_2_0_PEAKS_BLOCK: string;
-/**
- * Returns true when `line` is a wholesale ignore of the entire
- * `.peaks/` tree (the 1.x pattern that breaks 2.0 tracking).
- *
- * Examples that match:    `.peaks`  `.peaks/`  `/.peaks`  `/.peaks/`  `  .peaks/  `
- * Examples that DON'T:    `.peaks/_runtime/`  `# .peaks/ comment`  `.peaks_other`
- */
-export declare function isStaleWholesalePeaksRule(line: string): boolean;
-export interface MigrateGitignoreContentResult {
-    /** True if at least one stale wholesale .peaks rule was removed OR the canonical block was appended. */
-    readonly changed: boolean;
-    /** The exact lines that were removed (verbatim, with original whitespace). */
-    readonly removedRules: readonly string[];
-    /** The migrated content (always ends with a newline). */
-    readonly content: string;
-}
-/**
- * Pure function: takes the .gitignore content, returns the
- * migrated content + diff summary. No I/O.
- */
-export declare function migrateGitignoreContent(input: string): MigrateGitignoreContentResult;
-export interface MigrateGitignoreFileInput {
-    readonly projectRoot: string;
-    readonly apply?: boolean;
-}
-export interface MigrateGitignoreFileResult {
-    /** True if .gitignore is absent (nothing to migrate). */
-    readonly missing: boolean;
-    /** True if the migration would change the file. */
-    readonly changed: boolean;
-    /** True only if `apply: true` AND `changed: true`. */
-    readonly appliedWrite: boolean;
-    /** Absolute path to the timestamped backup file, or null. */
-    readonly backupPath: string | null;
-    /** Removed lines (verbatim). */
-    readonly removedRules: readonly string[];
-}
-/**
- * FS variant: reads .gitignore, runs migration, optionally
- * writes the result + a timestamped backup of the original.
- *
- * Date.now() is used for the backup suffix; in test environments
- * where determinism matters, the caller can stub Date via vi.
- */
-export declare function migrateGitignoreFile(input: MigrateGitignoreFileInput): MigrateGitignoreFileResult;

package/dist/services/upgrade/gitignore-migrate-service.js

@@ -1,170 +0,0 @@
-/**
- * .gitignore 1.x → 2.0 migration service.
- *
- * Real bug surfaced by ice-cola dogfood 2026-06-12:
- *   - 1.x consumer projects often have a wholesale `/.peaks/` (or
- *     `.peaks/` / `.peaks`) entry in their .gitignore — the 1.x
- *     install treated .peaks/ as the tool's ephemeral working
- *     directory.
- *   - In 2.0 the .peaks/ tree is split: most subdirectories
- *     remain ignored (_runtime, _sub_agents, audit, etc.) but
- *     several MUST be tracked: .peaks/standards/,
- *     .peaks/memory/*.md (durable LLM-authored memories),
- *     .peaks/PROJECT.md, and the user opt-in dotfiles.
- *   - A wholesale `.peaks/` ignore silently hides every 2.0
- *     tracked artifact, violating the "one-key completion" tenet
- *     because the user runs upgrade, sees 6/6 pass, then git
- *     status shows no new tracked files — the upgrade looks
- *     "done" but nothing landed in git.
- *
- * The service:
- *   1. Reads the project's .gitignore (string in / string out;
- *      no I/O in the pure function).
- *   2. Detects every line that is a wholesale-ignore of .peaks/
- *      (with or without leading slash, with or without trailing
- *      slash, ignoring surrounding whitespace).
- *   3. Removes those lines and appends the canonical 2.0 block
- *      (granular subpaths) if not already present.
- *   4. The FS variant `migrateGitignoreFile` adds backup +
- *      atomic write on top.
- *
- * Idempotent: re-running on a migrated .gitignore is a no-op.
- *
- * Spec reference: docs/superpowers/specs/2026-06-11-peaks-loop-l1-l2-l3-redesign.md §8.4
- * (per-project state in .peaks/preferences.json, durable
- * memories in .peaks/memory/*.md).
- */
-import { existsSync, readFileSync, writeFileSync } from 'node:fs';
-import { join } from 'node:path';
-/**
- * The 2.0 canonical ignore block. Mirrors the same paths
- * peaks-loop's own .gitignore uses; kept in lockstep manually
- * (the peaks-loop root .gitignore is the source of truth).
- *
- * Sentinel: every emitted line carries the marker comment so
- * the migration can detect "already-migrated" and stay
- * idempotent.
- */
-export const PEAKS_2_0_BLOCK_SENTINEL = '# peaks-loop 2.0 canonical ignore block — managed by `peaks upgrade --to 2.0`';
-export const CANONICAL_2_0_PEAKS_BLOCK = [
-    PEAKS_2_0_BLOCK_SENTINEL,
-    '.peaks/_runtime/',
-    '.peaks/_dogfood/',
-    '.peaks/_sub_agents/',
-    '.peaks/audit/',
-    '.peaks/system/',
-    '.peaks/runtime/',
-    '.peaks/preferences.json',
-    '.peaks/memory/upgrade-2.0-*.md'
-].join('\n');
-/**
- * Returns true when `line` is a wholesale ignore of the entire
- * `.peaks/` tree (the 1.x pattern that breaks 2.0 tracking).
- *
- * Examples that match:    `.peaks`  `.peaks/`  `/.peaks`  `/.peaks/`  `  .peaks/  `
- * Examples that DON'T:    `.peaks/_runtime/`  `# .peaks/ comment`  `.peaks_other`
- */
-export function isStaleWholesalePeaksRule(line) {
-    const trimmed = line.trim();
-    if (trimmed.length === 0)
-        return false;
-    if (trimmed.startsWith('#'))
-        return false;
-    // Strip leading slash (relative-to-repo-root convention) and
-    // trailing slash (directory marker); both equivalent in
-    // .gitignore semantics for wholesale ignore.
-    const normalized = trimmed.replace(/^\//, '').replace(/\/$/, '');
-    return normalized === '.peaks';
-}
-/**
- * Pure function: takes the .gitignore content, returns the
- * migrated content + diff summary. No I/O.
- */
-export function migrateGitignoreContent(input) {
-    const lines = input.split('\n');
-    const removed = [];
-    const kept = [];
-    for (const line of lines) {
-        if (isStaleWholesalePeaksRule(line)) {
-            removed.push(line.trim());
-        }
-        else {
-            kept.push(line);
-        }
-    }
-    // Already-migrated detection: if the canonical block is
-    // already present AND nothing was removed, this is a no-op.
-    const hasCanonicalBlock = input.includes(PEAKS_2_0_BLOCK_SENTINEL);
-    if (removed.length === 0 && hasCanonicalBlock) {
-        return { changed: false, removedRules: [], content: input };
-    }
-    // Already-migrated AND nothing stale: no-op, return original
-    // verbatim so byte-equality holds for idempotency.
-    if (removed.length === 0 && !hasCanonicalBlock) {
-        // Check whether there's anything peaks-related at all; if
-        // not, treat as no-op (the user explicitly chose not to
-        // ignore .peaks/ — respect that).
-        return { changed: false, removedRules: [], content: input };
-    }
-    // Removed at least one stale rule. Append canonical block
-    // if not already present.
-    let body = kept.join('\n');
-    if (!body.endsWith('\n') && body.length > 0) {
-        body += '\n';
-    }
-    if (!hasCanonicalBlock) {
-        body += CANONICAL_2_0_PEAKS_BLOCK + '\n';
-    }
-    return { changed: true, removedRules: removed, content: body };
-}
-/**
- * FS variant: reads .gitignore, runs migration, optionally
- * writes the result + a timestamped backup of the original.
- *
- * Date.now() is used for the backup suffix; in test environments
- * where determinism matters, the caller can stub Date via vi.
- */
-export function migrateGitignoreFile(input) {
-    const path = join(input.projectRoot, '.gitignore');
-    if (!existsSync(path)) {
-        return {
-            missing: true,
-            changed: false,
-            appliedWrite: false,
-            backupPath: null,
-            removedRules: []
-        };
-    }
-    const before = readFileSync(path, 'utf8');
-    const result = migrateGitignoreContent(before);
-    if (!result.changed) {
-        return {
-            missing: false,
-            changed: false,
-            appliedWrite: false,
-            backupPath: null,
-            removedRules: []
-        };
-    }
-    if (input.apply !== true) {
-        return {
-            missing: false,
-            changed: true,
-            appliedWrite: false,
-            backupPath: null,
-            removedRules: result.removedRules
-        };
-    }
-    // Apply: write backup first, then atomic-overwrite the original.
-    const ts = new Date().toISOString().replace(/[:.]/g, '-');
-    const backupPath = join(input.projectRoot, `.gitignore.peaks-2.0-backup-${ts}`);
-    writeFileSync(backupPath, before, 'utf8');
-    writeFileSync(path, result.content, 'utf8');
-    return {
-        missing: false,
-        changed: true,
-        appliedWrite: true,
-        backupPath,
-        removedRules: result.removedRules
-    };
-}

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

@@ -1,81 +0,0 @@
-/**
- * Pluggable executor for sub-step command spawns. Production
- * callers omit `executor` on `UpgradeInput`; the default
- * (`defaultSubstepExecutor`) wraps `node:child_process` `spawnSync`.
- * Tests pass a synchronous fake executor to avoid the spawn cost.
- *
- * Contract:
- *  - Return `status: 0` for success, non-zero for failure, `null`
- *    for synchronous spawn errors (binary not found, EACCES, …).
- *  - `stdout` / `stderr` are captured as UTF-8 strings.
- *  - `durationMs` is the wall-clock duration of the spawn
- *    (caller-facing field on `SubStepResult`).
- */
-export type SubstepExecutor = (command: string, args: readonly string[], timeoutMs: number) => {
-    status: number | null;
-    stdout: string;
-    stderr: string;
-    durationMs: number;
-};
-/**
- * Default substep executor — wraps `spawnSync` with the same
- * stdio / encoding / timeout config the umbrella has used since
- * v1. Exported for test coverage: the seam-preservation test
- * imports this directly to verify the normalized return shape
- * on a spawn failure (e.g. a non-existent command).
- */
-export declare const defaultSubstepExecutor: SubstepExecutor;
-export interface UpgradeInput {
-    readonly projectRoot: string;
-    /**
-     * When true, the umbrella is being invoked from the
-     * `npm i -g peaks-loop` postinstall. Suppresses the
-     * interactive prompts and accepts soft-fail on any
-     * sub-step.
-     */
-    readonly auto?: boolean;
-    /**
-     * When omitted, the sub-commands are inferred from PATH
-     * (the postinstall puts the binary there). The umbrella
-     * never blocks on a missing binary — it surfaces the
-     * install hint in `nextActions` instead.
-     */
-    readonly peaksBin?: string;
-    /**
-     * Optional injectable executor for sub-step command execution.
-     * When omitted, the umbrella uses `defaultSubstepExecutor`
-     * (real `spawnSync`). Tests pass a synchronous fake to avoid
-     * the per-sub-step `node peaks.js …` spawn cost.
-     */
-    readonly executor?: SubstepExecutor;
-}
-export interface SubStepResult {
-    readonly name: string;
-    readonly status: 'pass' | 'fail' | 'skipped';
-    readonly exitCode: number | null;
-    readonly stdout: string;
-    readonly stderr: string;
-    readonly durationMs: number;
-}
-export interface UpgradeResult {
-    readonly applied: boolean;
-    readonly fromVersion: string | null;
-    readonly toVersion: string;
-    readonly projectRoot: string;
-    readonly steps: readonly SubStepResult[];
-    readonly passedCount: number;
-    readonly failedCount: number;
-    readonly skippedCount: number;
-    readonly auditBefore: {
-        totalRedLines: number;
-        cliBacked: number;
-    } | null;
-    readonly auditAfter: {
-        totalRedLines: number;
-        cliBacked: number;
-    } | null;
-    readonly upgradeRecordPath: string | null;
-    readonly nextActions: readonly string[];
-    readonly warnings: readonly string[];
-}
-export declare function runUpgrade(input: UpgradeInput): UpgradeResult;