peaks-loop 4.0.0-beta.6 → 4.0.0-beta.8

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/migrate-service.d.ts

@@ -1,63 +0,0 @@
-/**
- * Slice 028 (Q2=A): `peaks standards migrate` rewrites a consumer
- * project's `CLAUDE.md` to drop the legacy heartbeat block.
- *
- * The legacy block (rendered by `peaks standards init` / `peaks
- * standards update` before slice 028) contained instructions that:
- *   1. pointed the LLM at the legacy `.peaks/.active-skill.json` path;
- *   2. required the LLM to invoke `peaks skill heartbeat:touch` and
- *      `peaks skill presence:clear` on every turn;
- *   3. ended with an `External reference: https://github.com/affaan-m/...`
- *      line that was peaks-loop-internal, not consumer-facing.
- *
- * The replacement text matches the post-slice-025 peaks-loop repo's own
- * `CLAUDE.md`: route the LLM through `peaks skill presence --json` and
- * render a compact status header when a valid skill is active. This
- * service is the deterministic in-place rewriter that brings existing
- * consumer trees in line with the new template.
- *
- * Behavior:
- *   - `migrateStandards({ project, apply: true })`     — rewrites the
- *     file when the legacy block is present, returns
- *     `applied: true`.
- *   - `migrateStandards({ project, dryRun: true })`    — returns the
- *     would-change preview, no write.
- *   - `migrateStandards({ project })`                  — defaults to a
- *     dry-run. `--apply` is the only opt-in for a destructive write.
- *   - File missing → `file: null`, no throw.
- *   - Legacy block not present → `foundOldBlock: false`, no write.
- */
-export declare const NEW_TEMPLATE_TEXT = "Active Peaks-Loop skill presence: at the start of every response, run `peaks skill presence --json` to read the active skill marker. The CLI handles canonical-path resolution (`.peaks/_runtime/active-skill.json` with back-compat fallback to `.peaks/.active-skill.json`); do not read those files directly. When the response includes a valid skill name, display the compact status header: `Peaks-Loop Skill: <skill> | Peaks-Loop Gate: <gate> | Next: <one short action>`. Display the header on every turn while the CLI returns an active skill; omit when the CLI returns no active skill.";
-export type MigrateStandardsInput = {
-    project: string;
-    apply?: boolean;
-    dryRun?: boolean;
-};
-export type MigrateStandardsData = {
-    file: string | null;
-    foundOldBlock: boolean;
-    wouldChange: boolean;
-    applied: boolean;
-    before: {
-        lines: number;
-    } | null;
-    after: {
-        lines: number;
-    } | null;
-    nextActions: string[];
-};
-export type MigrateStandardsResult = {
-    ok: true;
-    data: MigrateStandardsData;
-    warnings: string[];
-};
-export declare function detectLegacyBlock(content: string): {
-    found: boolean;
-    start: number;
-    end: number;
-};
-export declare function rewriteLegacyBlock(content: string, newText?: string): {
-    rewritten: string;
-    replaced: boolean;
-};
-export declare function migrateStandards(input: MigrateStandardsInput): MigrateStandardsResult;

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

@@ -1,193 +0,0 @@
-import { existsSync, readFileSync, writeFileSync } from 'node:fs';
-import { isAbsolute, resolve } from 'node:path';
-/**
- * Slice 028 (Q2=A): `peaks standards migrate` rewrites a consumer
- * project's `CLAUDE.md` to drop the legacy heartbeat block.
- *
- * The legacy block (rendered by `peaks standards init` / `peaks
- * standards update` before slice 028) contained instructions that:
- *   1. pointed the LLM at the legacy `.peaks/.active-skill.json` path;
- *   2. required the LLM to invoke `peaks skill heartbeat:touch` and
- *      `peaks skill presence:clear` on every turn;
- *   3. ended with an `External reference: https://github.com/affaan-m/...`
- *      line that was peaks-loop-internal, not consumer-facing.
- *
- * The replacement text matches the post-slice-025 peaks-loop repo's own
- * `CLAUDE.md`: route the LLM through `peaks skill presence --json` and
- * render a compact status header when a valid skill is active. This
- * service is the deterministic in-place rewriter that brings existing
- * consumer trees in line with the new template.
- *
- * Behavior:
- *   - `migrateStandards({ project, apply: true })`     — rewrites the
- *     file when the legacy block is present, returns
- *     `applied: true`.
- *   - `migrateStandards({ project, dryRun: true })`    — returns the
- *     would-change preview, no write.
- *   - `migrateStandards({ project })`                  — defaults to a
- *     dry-run. `--apply` is the only opt-in for a destructive write.
- *   - File missing → `file: null`, no throw.
- *   - Legacy block not present → `foundOldBlock: false`, no write.
- */
-export const NEW_TEMPLATE_TEXT = 'Active Peaks-Loop skill presence: at the start of every response, run `peaks skill presence --json` to read the active skill marker. The CLI handles canonical-path resolution (`.peaks/_runtime/active-skill.json` with back-compat fallback to `.peaks/.active-skill.json`); do not read those files directly. When the response includes a valid skill name, display the compact status header: `Peaks-Loop Skill: <skill> | Peaks-Loop Gate: <gate> | Next: <one short action>`. Display the header on every turn while the CLI returns an active skill; omit when the CLI returns no active skill.';
-const LEGACY_BLOCK_OPENER_LINE = 'Peaks-Loop 心跳检测 (heartbeat check)';
-const LEGACY_BLOCK_CLOSER = 'External reference: https://github.com/affaan-m/everything-claude-code';
-const LEGACY_MARKER_FALLBACK = 'Do NOT skip step 3-5. The CLI heartbeat:touch command';
-const FORBIDDEN_LEGACY_STRINGS = [
-    'heartbeat:touch',
-    'presence:clear',
-    'Default runbook',
-    'Startup sequence',
-    'Swarm parallel phase'
-];
-const NEW_TEMPLATE_FINGERPRINT = 'peaks skill presence --json';
-export function detectLegacyBlock(content) {
-    const openerIndex = content.indexOf(LEGACY_BLOCK_OPENER_LINE);
-    if (openerIndex >= 0) {
-        // Walk back to the start of the `<!--` line. The opener text is
-        // typically indented on the line after `<!--` (the legacy block
-        // is a multi-line HTML comment), so we cut from the most recent
-        // `<!--` line above. If no `<!--` is found in the surrounding
-        // 4 lines, fall back to the start of the opener's own line.
-        const htmlCommentIndex = content.lastIndexOf('<!--', openerIndex);
-        const previousNewlineBeforeOpener = content.lastIndexOf('\n', openerIndex);
-        let start;
-        if (htmlCommentIndex < 0 || htmlCommentIndex < previousNewlineBeforeOpener - 200) {
-            // No `<!--` line within a reasonable distance — start of the
-            // opener's own line.
-            start = previousNewlineBeforeOpener + 1;
-        }
-        else {
-            start = content.lastIndexOf('\n', htmlCommentIndex) + 1;
-        }
-        const closerIndex = content.indexOf(LEGACY_BLOCK_CLOSER, openerIndex);
-        let endIndex;
-        if (closerIndex < 0) {
-            const tailIndex = content.indexOf(LEGACY_MARKER_FALLBACK, openerIndex);
-            if (tailIndex < 0) {
-                endIndex = content.length;
-            }
-            else {
-                endIndex = content.indexOf('\n', tailIndex);
-                if (endIndex < 0)
-                    endIndex = content.length;
-            }
-        }
-        else {
-            endIndex = content.indexOf('\n', closerIndex);
-            if (endIndex < 0)
-                endIndex = content.length;
-        }
-        return { found: true, start, end: endIndex };
-    }
-    // Fallback: opener stripped by editor / re-format. Detect by the
-    // first forbidden string that survives the rewrite, then walk back
-    // to the start of the line.
-    for (const marker of FORBIDDEN_LEGACY_STRINGS) {
-        const idx = content.indexOf(marker);
-        if (idx >= 0) {
-            const start = content.lastIndexOf('\n', idx) + 1;
-            return { found: true, start, end: content.length };
-        }
-    }
-    return { found: false, start: -1, end: -1 };
-}
-export function rewriteLegacyBlock(content, newText = NEW_TEMPLATE_TEXT) {
-    const detection = detectLegacyBlock(content);
-    if (!detection.found) {
-        return { rewritten: content, replaced: false };
-    }
-    const before = content.slice(0, detection.start);
-    const after = content.slice(detection.end);
-    const trimmedBefore = before.replace(/\s+$/u, '\n');
-    const cleanedAfter = after.replace(/^\n+/u, '\n');
-    const rewritten = `${trimmedBefore}${newText}${cleanedAfter}`;
-    return { rewritten, replaced: true };
-}
-export function migrateStandards(input) {
-    const project = input.project;
-    const projectRoot = isAbsolute(project) ? project : resolve(project);
-    const filePath = resolve(projectRoot, 'CLAUDE.md');
-    const apply = input.apply === true;
-    const dryRun = input.dryRun === true || apply === false;
-    if (!existsSync(filePath)) {
-        return {
-            ok: true,
-            data: {
-                file: null,
-                foundOldBlock: false,
-                wouldChange: false,
-                applied: false,
-                before: null,
-                after: null,
-                nextActions: ['CLAUDE.md does not exist; nothing to migrate']
-            },
-            warnings: []
-        };
-    }
-    const original = readFileSync(filePath, 'utf8');
-    const detection = detectLegacyBlock(original);
-    if (!detection.found) {
-        if (original.includes(NEW_TEMPLATE_FINGERPRINT)) {
-            return {
-                ok: true,
-                data: {
-                    file: filePath,
-                    foundOldBlock: false,
-                    wouldChange: false,
-                    applied: false,
-                    before: { lines: original.split('\n').length },
-                    after: null,
-                    nextActions: ['CLAUDE.md is already up to date']
-                },
-                warnings: []
-            };
-        }
-        return {
-            ok: true,
-            data: {
-                file: filePath,
-                foundOldBlock: false,
-                wouldChange: false,
-                applied: false,
-                before: { lines: original.split('\n').length },
-                after: null,
-                nextActions: ['CLAUDE.md has no peaks-loop block; nothing to migrate']
-            },
-            warnings: []
-        };
-    }
-    const { rewritten } = rewriteLegacyBlock(original);
-    const nextActions = [];
-    if (dryRun) {
-        nextActions.push('Re-run with --apply to perform the rewrite');
-        return {
-            ok: true,
-            data: {
-                file: filePath,
-                foundOldBlock: true,
-                wouldChange: true,
-                applied: false,
-                before: { lines: original.split('\n').length },
-                after: { lines: rewritten.split('\n').length },
-                nextActions
-            },
-            warnings: []
-        };
-    }
-    writeFileSync(filePath, rewritten, 'utf8');
-    nextActions.push('CLAUDE.md rewritten; no further action required');
-    return {
-        ok: true,
-        data: {
-            file: filePath,
-            foundOldBlock: true,
-            wouldChange: true,
-            applied: true,
-            before: { lines: original.split('\n').length },
-            after: { lines: rewritten.split('\n').length },
-            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;