peaks-loop 4.0.0-beta → 4.0.0-beta.10

package/dist/src/services/standards/ide-aware-standards-service.js

@@ -1,89 +0,0 @@
-import { detectAllResourceTargets, getStandardsProfile, } from '../ide/resource-profile.js';
-import { executeProjectStandardsInit, executeProjectStandardsUpdate, } from './project-standards-service.js';
-import { detectInstalledIde } from '../ide/ide-detector.js';
-function warnUnregisteredIde(ideId, projectRoot) {
-    process.stderr.write(`peaks standards: IDE '${ideId}' has no standardsProfile declared; ` +
-        `falling back to the legacy Claude Code path (CLAUDE.md + .claude/rules/**) ` +
-        `for project '${projectRoot}'. This is a slice #011 follow-up gap; ` +
-        `see .peaks/memory/ide-adapter-resource-profile-framework.md.\n`);
-}
-function warnNoIdeDetected(projectRoot) {
-    process.stderr.write(`peaks standards: no IDE detected in '${projectRoot}'; ` +
-        `writing to the legacy Claude Code path (CLAUDE.md + .claude/rules/**). ` +
-        `Pass --ide <id> to bypass detection.\n`);
-}
-/**
- * Resolve the active IDE id for a standards call. Order of precedence:
- *   1. explicit `options.ideId`
- *   2. `IdeRegistry.detect()` from `options.projectRoot`
- *   3. `null` (no detected IDE — caller falls back to legacy)
- */
-export function resolveStandardsIdeId(options) {
-    if (options.ideId !== undefined) {
-        return options.ideId;
-    }
-    return detectInstalledIde(options.projectRoot);
-}
-/**
- * Run `peaks standards init` with IDE-aware dispatch.
- *
- * When the resolved IDE has a `standardsProfile`, the call still
- * delegates to the existing `executeProjectStandardsInit` (the
- * profile maps to the Claude Code path; future per-IDE writers
- * plug in here). When the IDE is unregistered for the standards
- * profile, the call delegates to the legacy path + emits a stderr
- * warning.
- */
-export function executeProjectStandardsInitIdeAware(options) {
-    const ideId = resolveStandardsIdeId(options);
-    if (ideId === null) {
-        warnNoIdeDetected(options.projectRoot);
-        return executeProjectStandardsInit(options);
-    }
-    const profile = getStandardsProfile(ideId);
-    if (profile === null) {
-        warnUnregisteredIde(ideId, options.projectRoot);
-        return executeProjectStandardsInit(options);
-    }
-    // Claude Code path: profile matches the legacy writer. Future per-IDE
-    // writers (markdown+frontmatter, multiple rule roots, etc.) plug in
-    // here by branching on `profile.format` / `profile.rulesDir`.
-    return executeProjectStandardsInit(options);
-}
-/**
- * Run `peaks standards update` with IDE-aware dispatch.
- *
- * Same dispatch rules as `executeProjectStandardsInitIdeAware`.
- */
-export function executeProjectStandardsUpdateIdeAware(options) {
-    const ideId = resolveStandardsIdeId(options);
-    if (ideId === null) {
-        warnNoIdeDetected(options.projectRoot);
-        return executeProjectStandardsUpdate(options);
-    }
-    const profile = getStandardsProfile(ideId);
-    if (profile === null) {
-        warnUnregisteredIde(ideId, options.projectRoot);
-        return executeProjectStandardsUpdate(options);
-    }
-    return executeProjectStandardsUpdate(options);
-}
-/**
- * Test seam + integration-test helper: returns the resolved IDE id
- * for the call, plus the active standards profile. Exported for
- * the integration test in `tests/unit/standards/ide-aware-standards-service.test.ts`
- * to assert the dispatch decision without running the full write.
- */
-export function inspectStandardsDispatch(options) {
-    const ideId = resolveStandardsIdeId(options);
-    if (ideId === null) {
-        return { ideId: null, profile: null };
-    }
-    return { ideId, profile: getStandardsProfile(ideId) };
-}
-/**
- * Test seam: the resource-profile accessor exposes
- * `detectAllResourceTargets` for callers that need to enumerate
- * across all registered IDEs. Re-exported here for convenience.
- */
-export { detectAllResourceTargets };

package/dist/src/services/standards/migrate-claude-rules-service.js

@@ -1,193 +0,0 @@
-/**
- * peaks standards migrate — .claude/rules/ tree thinning.
- * Slice: 2026-06-12-standards-migrate-claude-rules.
- *
- * The 1.x peaks-loop install copied a thick .claude/rules
- * tree (skill-first / CLI-auxiliary / dogfood / commit-trailer
- * rules) into consumer projects. In 2.0, the canonical rules
- * live at .peaks/standards/ and every markdown file under
- * .claude/rules becomes a 2-line pointer to the canonical path.
- *
- * The service:
- *   1. Backs up the existing `.claude/rules/` tree to
- *      `.claude/rules/.peaks-2.0-backup-<ts>/` (timestamped;
- *      safe to run multiple times).
- *   2. Replaces each .md file under .claude/rules (recursive)
- *      with a 2-line pointer.
- *   3. Scaffolds the 2.0 canonical rules at
- *      `.peaks/standards/{common,typescript}/`, but
- *      never overwrites existing files in `.peaks/standards/`.
- *
- * All operations are gated by `apply: true`. Dry-run mode
- * returns the would-change diff without writing.
- */
-import { existsSync, mkdirSync, readdirSync, readFileSync, statSync, writeFileSync } from 'node:fs';
-import { join } from 'node:path';
-const POINTER_TEXT = (canonicalPath) => `# Canonical peaks-loop 2.0 rules live at: ${canonicalPath}\n# This file is a 2-line pointer. Edit the canonical file instead.\n`;
-function timestampSlug() {
-    return new Date().toISOString().replace(/[:.]/g, '-');
-}
-function readMarkdownFilesRecursive(root) {
-    if (!existsSync(root))
-        return [];
-    const out = [];
-    const stat = statSync(root);
-    if (stat.isFile()) {
-        return root.endsWith('.md') ? [root] : [];
-    }
-    if (!stat.isDirectory())
-        return [];
-    for (const entry of readdirSync(root)) {
-        out.push(...readMarkdownFilesRecursive(join(root, entry)));
-    }
-    return out;
-}
-function isAlreadyPointer(filePath) {
-    if (!existsSync(filePath))
-        return false;
-    try {
-        const body = readFileSync(filePath, 'utf8');
-        return body.includes('Canonical peaks-loop 2.0 rules live at:');
-    }
-    catch {
-        return false;
-    }
-}
-const CANONICAL_2_0_DEV_PREFERENCE = `# Peaks-Loop dev preference (2.0 canonical)
-
-> Project-local preference, captured from the 1.x install + re-rendered with the 2.0 vocabulary.
-> Scope: applies to every iteration, adjustment, fix, or tweak on this project.
-> Reading: read this **before** opening a new CLI command or routing a new feature through a CLI surface.
-
-## Rule 1 — Skill-first, CLI-auxiliary
-
-When designing or modifying a peaks-loop feature, default to the **skill-first** design. CLI commands are **invoked by the skill prompt** when they are the right primitive: a side effect that must be atomic, a gate that must be machine-enforced, a probe that needs structured JSON, or a backstop that prevents the LLM from skipping a step. Behaviour only an LLM in a skill prompt would use lives **in the relevant skill's SKILL.md**, not as a new CLI command. See \`.claude/rules/common/dev-preference.md\` for the decision template.
-
-## Rule 2 — Dogfood on every adjustment
-
-**Every adjustment, iteration, or fix-problem operation must be dogfood-tested in the current project before the work is declared complete.** No exceptions for "it's a small change", "just a comment update", or "just a SKILL.md line". The unit test suite is a subset of "current effect"; the dogfood is the full set. If a change passes unit tests but breaks a CLI command, the change is a regression.
-
-## Rule 3 — Commits belong to the human
-
-**No AI co-author trailer.** The commit is the human's. **Identity is global gitconfig only** (\`~/.gitconfig\`). Do not set, override, or shadow \`user.name\` / \`user.email\` at the repo level, via env vars, or via \`git -c user.*=...\`. The commit's recorded author and committer must both equal the global identity.
-`;
-const CANONICAL_2_0_CODING_STYLE_TS = `# TypeScript Coding Standards (2.0 canonical)
-
-> Project-local standards, derived from the 1.x install + re-rendered with the 2.0 vocabulary.
-
-- Apply project-local conventions before generic typescript guidance.
-- Keep public APIs typed or documented according to typescript ecosystem norms.
-- Do not add new \`any\` types; use explicit domain types, generics, or \`unknown\` with narrowing.
-- Prefer standard tooling and existing project scripts for formatting, linting, tests, and coverage.
-- peaks-rd must check this file before planning code changes in typescript projects.
-`;
-const CANONICAL_2_0_COMMON_FILES = [
-    { relPath: 'common/dev-preference.md', content: CANONICAL_2_0_DEV_PREFERENCE },
-    {
-        relPath: 'common/coding-style.md',
-        content: '# Coding Standards (2.0 canonical)\n\n- Prefer simple, readable code over clever abstractions.\n- Keep functions focused and files cohesive.\n- Use immutable updates unless a language-specific convention explicitly favors mutation.\n- Validate user input, external data, file paths, and configuration at system boundaries.\n- Preserve existing project conventions when they are stricter than this baseline.\n',
-    },
-    {
-        relPath: 'common/code-review.md',
-        content: '# Code Review Standards (2.0 canonical)\n\n- Review diffs for correctness, maintainability, test coverage, and regression risk.\n- Treat missing tests for changed behavior as a blocker unless the change is documentation-only.\n- Verify code paths that handle filesystem, external APIs, credentials, user input, or generated artifacts.\n',
-    },
-    {
-        relPath: 'common/security.md',
-        content: '# Security Review Standards (2.0 canonical)\n\n- Never hardcode secrets, API keys, passwords, tokens, or credentials.\n- Do not send private code or secrets to external services without explicit user authorization.\n- Guard filesystem writes against path traversal, symlink, and junction escapes.\n- Require explicit confirmation for destructive actions, external state changes, and credential use.\n',
-    },
-    { relPath: 'typescript/coding-style.md', content: CANONICAL_2_0_CODING_STYLE_TS },
-];
-export function migrateClaudeRules(input) {
-    const projectRoot = input.projectRoot;
-    const apply = input.apply === true;
-    const warnings = [];
-    const nextActions = [];
-    const claudeRulesDir = join(projectRoot, '.claude', 'rules');
-    const peaksStandardsDir = join(projectRoot, '.peaks', 'standards');
-    const canonicalRelPath = '.peaks/standards/';
-    const existingRulesFiles = readMarkdownFilesRecursive(claudeRulesDir);
-    const thickFiles = existingRulesFiles.filter((f) => !isAlreadyPointer(f));
-    const hasThickFiles = thickFiles.length > 0;
-    // The backup path is computed eagerly (so dry-run can preview
-    // the would-create location) but only created on disk in
-    // apply mode. In dry-run mode we still return the path so
-    // the user can see where the backup will land.
-    const computedBackupPath = hasThickFiles ? join(claudeRulesDir, `.peaks-2.0-backup-${timestampSlug()}`) : null;
-    const backupPath = apply ? computedBackupPath : null;
-    const thinnedFiles = [];
-    const scaffoldedFiles = [];
-    const preservedFiles = [];
-    // wouldChange is true iff there is at least one thick file to
-    // thin. An empty .claude/rules/ is NOT a wouldChange (no-op).
-    const wouldChange = hasThickFiles;
-    if (apply && hasThickFiles) {
-        // Step 1: backup
-        if (backupPath !== null) {
-            try {
-                mkdirSync(backupPath, { recursive: true });
-                for (const file of thickFiles) {
-                    const body = readFileSync(file, 'utf8');
-                    const rel = file.slice(claudeRulesDir.length + 1);
-                    writeFileSync(join(backupPath, rel), body, 'utf8');
-                }
-            }
-            catch (err) {
-                warnings.push(`Backup step failed: ${err instanceof Error ? err.message : String(err)}`);
-            }
-        }
-        // Step 2: replace each .md with a 2-line pointer
-        for (const file of thickFiles) {
-            try {
-                writeFileSync(file, POINTER_TEXT(canonicalRelPath), 'utf8');
-                thinnedFiles.push(file);
-            }
-            catch (err) {
-                warnings.push(`Thin step failed for ${file}: ${err instanceof Error ? err.message : String(err)}`);
-            }
-        }
-        // Step 3: scaffold .peaks/standards/ — never overwrite existing
-        for (const file of CANONICAL_2_0_COMMON_FILES) {
-            const dest = join(peaksStandardsDir, file.relPath);
-            if (existsSync(dest)) {
-                preservedFiles.push(dest);
-                continue;
-            }
-            try {
-                mkdirSync(join(dest, '..'), { recursive: true });
-                writeFileSync(dest, file.content, 'utf8');
-                scaffoldedFiles.push(dest);
-            }
-            catch (err) {
-                warnings.push(`Scaffold step failed for ${file.relPath}: ${err instanceof Error ? err.message : String(err)}`);
-            }
-        }
-    }
-    if (thinnedFiles.length > 0) {
-        nextActions.push(`Thinned ${thinnedFiles.length} .md file(s) under .claude/rules (recursive) → 2-line pointer.`);
-    }
-    if (scaffoldedFiles.length > 0) {
-        nextActions.push(`Scaffolded ${scaffoldedFiles.length} 2.0 canonical rule(s) at .peaks/standards/.`);
-    }
-    if (preservedFiles.length > 0) {
-        nextActions.push(`Preserved ${preservedFiles.length} existing .peaks/standards/ file(s) (no overwrite).`);
-    }
-    if (backupPath !== null) {
-        nextActions.push(`Backup at ${backupPath} (git-ignored).`);
-    }
-    if (!apply && wouldChange) {
-        nextActions.push('Re-run with --apply to perform the migration.');
-    }
-    return {
-        ok: true,
-        data: {
-            backupPath,
-            thinnedFiles,
-            scaffoldedFiles,
-            preservedFiles,
-            wouldChange,
-            applied: apply && hasThickFiles,
-            nextActions,
-        },
-        warnings,
-    };
-}

package/dist/src/services/standards/missing-standards-detector.d.ts

@@ -1,79 +0,0 @@
-/**
- * Slice 2026-06-16-peaks-code-auto-scaffold (RD#7) — missing-standards-detector.
- *
- * Read-side companion to the `peaks standards init` / `peaks standards update`
- * writers in `./project-standards-service.ts`. The writer KNOWS how to scaffold
- * `.claude/rules/{common,<language>}/` from the curated baseline; this module
- * detects when the consumer project's rules tree is missing or empty and emits
- * a copy-pasteable diagnostic for the operator.
- *
- * Why a separate read module: the writer is `apply`-gated and write-bounded
- * (it touches the filesystem via `prevalidateWrites`/`assertNotHomedirBaseline`).
- * The detector is read-only and platform-aware (win32 path separators for
- * Windows consumers) — coupling the two would force the writer to grow a
- * read-only flag and a platform-switching argument.
- *
- * Cross-platform path rendering (AC6): the detector switches on
- * `process.platform` and renders the project path with `path.sep` so Windows
- * consumers see `C:\Users\<u>\project\.claude\rules\` verbatim rather than the
- * POSIX form. The detection logic itself (readdirSync on `node:fs`) is
- * platform-neutral.
- *
- * Once-per-session dedup (AC7): `peaks workspace init` writes a marker at
- * `.peaks/_runtime/<sessionId>/.standards-checked` after the first detection
- * pass. `detectMissingProjectStandards` does NOT consult the marker — that's
- * the caller's responsibility (the CLI command decides whether to skip per
- * AC7's "once per session" contract).
- */
-import type { StandardsLanguage } from './project-standards-service.js';
-export type MissingProjectStandardsDiagnostic = {
-    readonly missing: boolean;
-    /**
-     * Absolute path to the rules root (`<projectRoot>/.claude/rules`). Rendered
-     * with the platform-native separator so Windows consumers see
-     * `C:\...\.claude\rules` verbatim.
-     */
-    readonly path: string;
-    /**
-     * Copy-pasteable remediation hint for the operator. Always present, even
-     * when `missing: false`, so downstream callers can show "if you ever need
-     * to re-scaffold: ..." as a nextAction without re-running the detector.
-     */
-    readonly remediation: string;
-    readonly language: StandardsLanguage;
-};
-/**
- * Detect whether a consumer project's `.claude/rules/` tree is missing or
- * empty.
- *
- * Rules (per PRD R2):
- *   - `.claude/rules/common/` MUST exist AND contain at least one `.md` file.
- *   - `.claude/rules/<language>/` MUST exist AND contain at least one `.md`
- *     file — except for `generic`, where the language dir is OPTIONAL.
- *
- * Returns `{ missing: true, ... }` when either required dir is absent or
- * empty. The diagnostic is always populated so the caller can surface the
- * remediation hint unconditionally (the CLI uses the same string for both
- * the stderr banner and the JSON envelope warning).
- */
-export declare function detectMissingProjectStandards(projectRoot: string, language: StandardsLanguage): MissingProjectStandardsDiagnostic;
-/**
- * Once-per-session marker file path. The CLI writes this file under
- * `.peaks/_runtime/<sessionId>/` after the first diagnostic pass so that
- * subsequent `peaks workspace init` invocations within the same session
- * skip the stderr banner (the diagnostic is still computable for
- * programmatic consumers via `detectMissingProjectStandards`).
- */
-export declare function getStandardsCheckedMarkerPath(projectRoot: string, sessionId: string): string;
-/**
- * Mark this session as "standards checked" so subsequent invocations can
- * skip the stderr diagnostic. Idempotent: existing marker is left as-is.
- * Returns true if the marker was newly created, false if it already existed.
- */
-export declare function markStandardsChecked(projectRoot: string, sessionId: string): boolean;
-/**
- * Test seam: check whether the once-per-session marker exists. Caller is
- * responsible for routing the result into the AC7 dedup logic.
- */
-export declare function hasStandardsCheckedMarker(projectRoot: string, sessionId: string): boolean;
-export declare const platformPathSeparator: "\\" | "/";

package/dist/src/services/standards/missing-standards-detector.js

@@ -1,132 +0,0 @@
-/**
- * Slice 2026-06-16-peaks-code-auto-scaffold (RD#7) — missing-standards-detector.
- *
- * Read-side companion to the `peaks standards init` / `peaks standards update`
- * writers in `./project-standards-service.ts`. The writer KNOWS how to scaffold
- * `.claude/rules/{common,<language>}/` from the curated baseline; this module
- * detects when the consumer project's rules tree is missing or empty and emits
- * a copy-pasteable diagnostic for the operator.
- *
- * Why a separate read module: the writer is `apply`-gated and write-bounded
- * (it touches the filesystem via `prevalidateWrites`/`assertNotHomedirBaseline`).
- * The detector is read-only and platform-aware (win32 path separators for
- * Windows consumers) — coupling the two would force the writer to grow a
- * read-only flag and a platform-switching argument.
- *
- * Cross-platform path rendering (AC6): the detector switches on
- * `process.platform` and renders the project path with `path.sep` so Windows
- * consumers see `C:\Users\<u>\project\.claude\rules\` verbatim rather than the
- * POSIX form. The detection logic itself (readdirSync on `node:fs`) is
- * platform-neutral.
- *
- * Once-per-session dedup (AC7): `peaks workspace init` writes a marker at
- * `.peaks/_runtime/<sessionId>/.standards-checked` after the first detection
- * pass. `detectMissingProjectStandards` does NOT consult the marker — that's
- * the caller's responsibility (the CLI command decides whether to skip per
- * AC7's "once per session" contract).
- */
-import { existsSync, mkdirSync, readdirSync, statSync, writeFileSync } from 'node:fs';
-import { isAbsolute, join, sep as pathSep } from 'node:path';
-const SUPPORTED_LANGUAGES = new Set([
-    'generic',
-    'typescript',
-    'javascript',
-    'python',
-    'go',
-    'rust'
-]);
-function normalizeProjectRoot(projectRoot) {
-    return isAbsolute(projectRoot) ? projectRoot : join(process.cwd(), projectRoot);
-}
-function renderForPlatform(projectRoot) {
-    // AC6: render with the platform-native separator. On win32 we replace '/'
-    // with '\\' in the project path; the rest of the path is constructed with
-    // the same separator via the literal '\\' join below.
-    if (process.platform === 'win32') {
-        const nativeRoot = projectRoot.replace(/\//g, '\\');
-        return `${nativeRoot}\\.claude\\rules`;
-    }
-    return `${projectRoot}/.claude/rules`;
-}
-function hasPopulatedMarkdown(dir) {
-    if (!existsSync(dir))
-        return false;
-    const stat = statSync(dir);
-    if (!stat.isDirectory())
-        return false;
-    const entries = readdirSync(dir);
-    return entries.some((entry) => entry.toLowerCase().endsWith('.md'));
-}
-/**
- * Detect whether a consumer project's `.claude/rules/` tree is missing or
- * empty.
- *
- * Rules (per PRD R2):
- *   - `.claude/rules/common/` MUST exist AND contain at least one `.md` file.
- *   - `.claude/rules/<language>/` MUST exist AND contain at least one `.md`
- *     file — except for `generic`, where the language dir is OPTIONAL.
- *
- * Returns `{ missing: true, ... }` when either required dir is absent or
- * empty. The diagnostic is always populated so the caller can surface the
- * remediation hint unconditionally (the CLI uses the same string for both
- * the stderr banner and the JSON envelope warning).
- */
-export function detectMissingProjectStandards(projectRoot, language) {
-    if (!SUPPORTED_LANGUAGES.has(language)) {
-        throw new Error(`Unsupported standards language: ${String(language)}`);
-    }
-    const normalizedRoot = normalizeProjectRoot(projectRoot);
-    const renderedPath = renderForPlatform(normalizedRoot);
-    const commonDir = join(normalizedRoot, '.claude', 'rules', 'common');
-    const languageDir = language === 'generic'
-        ? null
-        : join(normalizedRoot, '.claude', 'rules', language);
-    const commonOk = hasPopulatedMarkdown(commonDir);
-    const languageOk = languageDir === null ? true : hasPopulatedMarkdown(languageDir);
-    const missing = !(commonOk && languageOk);
-    // Remediation message: copy-pasteable, mentions language, references the
-    // resolved project path, and hints at the `--init-standards` opt-in flag.
-    const remediation = missing
-        ? `⚠ no project-local standards found at ${renderedPath} — run \`peaks standards init --project ${normalizedRoot} --apply\` to scaffold, or pass --init-standards to peaks workspace init for one-shot auto-apply (language: ${language}).`
-        : `Project-local standards already present at ${renderedPath}. Re-run \`peaks standards init --project ${normalizedRoot} --apply\` to refresh from the curated baseline.`;
-    return {
-        missing,
-        path: renderedPath,
-        remediation,
-        language
-    };
-}
-/**
- * Once-per-session marker file path. The CLI writes this file under
- * `.peaks/_runtime/<sessionId>/` after the first diagnostic pass so that
- * subsequent `peaks workspace init` invocations within the same session
- * skip the stderr banner (the diagnostic is still computable for
- * programmatic consumers via `detectMissingProjectStandards`).
- */
-export function getStandardsCheckedMarkerPath(projectRoot, sessionId) {
-    const normalizedRoot = normalizeProjectRoot(projectRoot);
-    // Canonical session runtime dir — see `.peaks/memory/session-dir-canonical-resolver-must-route-all-writes.md`.
-    return join(normalizedRoot, '.peaks', '_runtime', sessionId, '.standards-checked');
-}
-/**
- * Mark this session as "standards checked" so subsequent invocations can
- * skip the stderr diagnostic. Idempotent: existing marker is left as-is.
- * Returns true if the marker was newly created, false if it already existed.
- */
-export function markStandardsChecked(projectRoot, sessionId) {
-    const markerPath = getStandardsCheckedMarkerPath(projectRoot, sessionId);
-    if (existsSync(markerPath))
-        return false;
-    mkdirSync(join(markerPath, '..'), { recursive: true });
-    writeFileSync(markerPath, 'standards-checked\n', 'utf8');
-    return true;
-}
-/**
- * Test seam: check whether the once-per-session marker exists. Caller is
- * responsible for routing the result into the AC7 dedup logic.
- */
-export function hasStandardsCheckedMarker(projectRoot, sessionId) {
-    return existsSync(getStandardsCheckedMarkerPath(projectRoot, sessionId));
-}
-// Re-export path.sep for tests that want to assert the platform invariant.
-export const platformPathSeparator = pathSep;