@pugi/cli 0.1.0-beta.23 → 0.1.0-beta.25

package/dist/core/hooks/runner.js

@@ -0,0 +1,236 @@
+/**
+ * Pugi hooks MVP — runner (Leak L12, first pass).
+ *
+ * Spawns the shell command declared in `hooks-mvp.json`, applies the
+ * timeout watchdog, captures stdout / stderr, and surfaces a
+ * structured result. Two events are wired in the MVP (SessionStart,
+ * PreToolUse); the runner itself is event-agnostic so the fast-follow
+ * PR can attach the remaining 6 events without changing this file.
+ *
+ * Safety properties:
+ *   - 30 s default timeout (per task spec); SIGTERM then SIGKILL with
+ *     a 2 s grace window.
+ *   - 1 MiB output cap per stream — a misbehaving hook (`yes`) cannot
+ *     OOM the parent CLI by buffering unbounded data.
+ *   - Spawn failures are caught + logged; the session never crashes
+ *     because of a missing binary or a syntax error in the command.
+ *   - Hook errors are atomic-appended to `<workspaceRoot>/.pugi/logs/
+ *     hooks.log`. Multiple sessions can write concurrently without
+ *     interleaving because `appendFileSync` opens with O_APPEND.
+ *
+ * Brand voice: ASCII only.
+ */
+import { spawn } from 'node:child_process';
+import { appendFileSync, existsSync, mkdirSync } from 'node:fs';
+import { resolve } from 'node:path';
+import { DEFAULT_HOOK_TIMEOUT_MS, isToolEvent, } from './registry.js';
+const HOOK_STREAM_CAP_BYTES = 1024 * 1024;
+const SIGKILL_GRACE_MS = 2_000;
+/**
+ * Fire every matching hook for `event` sequentially. Sequential (not
+ * parallel) is the intentional default — operators frequently chain
+ * `git add` -> `eslint --fix` style hooks that would race otherwise.
+ * Returns a `HookFireOutcome` with the per-invocation results.
+ */
+export async function fireHooks(opts) {
+    const { config, event, payload, toolName, workspaceRoot, env } = opts;
+    const matching = config.listMatching(event, toolName);
+    if (matching.length === 0) {
+        return { event, results: [], anyBlocked: false };
+    }
+    const logger = workspaceRoot ? new HookLogger(workspaceRoot) : undefined;
+    const results = [];
+    let anyBlocked = false;
+    for (const entry of matching) {
+        const timeoutMs = entry.timeoutMs ?? DEFAULT_HOOK_TIMEOUT_MS;
+        const result = await executeOne(entry.command, payload, timeoutMs, env);
+        // Blocking semantics only honored for PreToolUse in the MVP.
+        // Other events can declare `blocking: true` but the runner just
+        // logs that intent — it does NOT short-circuit. The fast-follow
+        // PR threads PostToolUse + UserPromptSubmit blocking through.
+        const blockable = entry.blocking === true && event === 'PreToolUse';
+        const blocked = blockable && !result.ok;
+        if (blocked) {
+            anyBlocked = true;
+            result.blocked = true;
+            result.blockSentinel = `HOOK_BLOCKED: ${truncate(entry.command, 80)} exited ${result.exitCode}`;
+        }
+        if (logger && !result.ok) {
+            logger.recordFailure(event, entry.command, result);
+        }
+        results.push(result);
+    }
+    return { event, results, anyBlocked };
+}
+async function executeOne(command, payload, timeoutMs, env) {
+    const startedAt = Date.now();
+    return new Promise((resolvePromise) => {
+        const payloadJson = JSON.stringify(payload);
+        const childEnv = {
+            ...(env ?? process.env),
+            PUGI_HOOK_PAYLOAD: payloadJson,
+            PUGI_HOOK_EVENT: payload.event,
+            PUGI_HOOK_SESSION_ID: payload.sessionId,
+        };
+        const child = spawn('/bin/sh', ['-c', command], {
+            env: childEnv,
+            stdio: ['pipe', 'pipe', 'pipe'],
+        });
+        const state = {
+            stdout: '',
+            stderr: '',
+            killedForTimeout: false,
+            killedForStreamCap: false,
+        };
+        const escalateKill = () => {
+            if (state.sigKillTimer)
+                return;
+            state.sigKillTimer = setTimeout(() => {
+                if (!child.killed)
+                    child.kill('SIGKILL');
+            }, SIGKILL_GRACE_MS);
+            if (state.sigKillTimer.unref)
+                state.sigKillTimer.unref();
+        };
+        const enforceStreamCap = () => {
+            if (state.killedForStreamCap)
+                return;
+            if (state.stdout.length + state.stderr.length <= HOOK_STREAM_CAP_BYTES)
+                return;
+            state.killedForStreamCap = true;
+            child.kill('SIGTERM');
+            escalateKill();
+        };
+        child.stdout?.on('data', (chunk) => {
+            if (state.killedForStreamCap)
+                return;
+            state.stdout += chunk.toString('utf8');
+            enforceStreamCap();
+        });
+        child.stderr?.on('data', (chunk) => {
+            if (state.killedForStreamCap)
+                return;
+            state.stderr += chunk.toString('utf8');
+            enforceStreamCap();
+        });
+        // Best-effort stdin payload — hook scripts that want to read it can
+        // (e.g. `jq .`); scripts that ignore stdin will EPIPE on our write
+        // which we swallow because the env var carries the same data.
+        if (child.stdin) {
+            child.stdin.on('error', () => {
+                // EPIPE is benign — see above.
+            });
+            child.stdin.end(payloadJson);
+        }
+        const timer = setTimeout(() => {
+            state.killedForTimeout = true;
+            child.kill('SIGTERM');
+            escalateKill();
+        }, timeoutMs);
+        if (timer.unref)
+            timer.unref();
+        child.on('error', (error) => {
+            clearTimeout(timer);
+            if (state.sigKillTimer)
+                clearTimeout(state.sigKillTimer);
+            resolvePromise({
+                command: truncate(command, 200),
+                exitCode: -1,
+                stdoutBytes: state.stdout.length,
+                stderrBytes: state.stderr.length,
+                elapsedMs: Date.now() - startedAt,
+                ok: false,
+                blocked: false,
+                timedOut: false,
+                // No blockSentinel here — spawn errors are not the same as
+                // blocking-failure semantics. The caller logs them generically.
+            });
+        });
+        child.on('close', (code, signal) => {
+            clearTimeout(timer);
+            if (state.sigKillTimer)
+                clearTimeout(state.sigKillTimer);
+            let exitCode;
+            if (code !== null) {
+                exitCode = code;
+            }
+            else if (signal === 'SIGTERM') {
+                exitCode = -15;
+            }
+            else if (signal === 'SIGKILL') {
+                exitCode = -9;
+            }
+            else {
+                exitCode = -1;
+            }
+            const ok = exitCode === 0 &&
+                !state.killedForTimeout &&
+                !state.killedForStreamCap;
+            resolvePromise({
+                command: truncate(command, 200),
+                exitCode,
+                stdoutBytes: state.stdout.length,
+                stderrBytes: state.stderr.length,
+                elapsedMs: Date.now() - startedAt,
+                ok,
+                blocked: false,
+                timedOut: state.killedForTimeout,
+            });
+        });
+    });
+}
+/**
+ * Append-only failure log at `<workspaceRoot>/.pugi/logs/hooks.log`.
+ * Each line is a JSON record so log scrapers can `jq` over it.
+ */
+class HookLogger {
+    path;
+    prepared = false;
+    constructor(workspaceRoot) {
+        this.path = resolve(workspaceRoot, '.pugi', 'logs', 'hooks.log');
+    }
+    recordFailure(event, command, result) {
+        this.prepareDir();
+        const line = JSON.stringify({
+            ts: new Date().toISOString(),
+            event,
+            command: truncate(command, 200),
+            exitCode: result.exitCode,
+            timedOut: result.timedOut,
+            elapsedMs: result.elapsedMs,
+            stdoutBytes: result.stdoutBytes,
+            stderrBytes: result.stderrBytes,
+            toolEvent: isToolEvent(event),
+        });
+        try {
+            appendFileSync(this.path, `${line}\n`, 'utf8');
+        }
+        catch {
+            // Logging is best-effort — the session must not crash when the
+            // disk is full or the directory is read-only. The runner has
+            // already returned the result; dropping the log line is the
+            // safe fallback.
+        }
+    }
+    prepareDir() {
+        if (this.prepared)
+            return;
+        const dir = resolve(this.path, '..');
+        if (!existsSync(dir)) {
+            try {
+                mkdirSync(dir, { recursive: true });
+            }
+            catch {
+                // ignored — appendFileSync will surface a fresh error on the
+                // write path, which we also swallow.
+            }
+        }
+        this.prepared = true;
+    }
+}
+function truncate(value, max) {
+    if (value.length <= max)
+        return value;
+    return `${value.slice(0, max - 3)}...`;
+}
+//# sourceMappingURL=runner.js.map

package/dist/core/init/scaffold.js

@@ -0,0 +1,195 @@
+/**
+ * Workspace scaffold — extracted from `pugi init` so the bare REPL boot
+ * can call it automatically when the operator launches `pugi` in a
+ * fresh directory (CEO directive 2026-05-26).
+ *
+ * Before this module, `pugi init` was the only path that materialised
+ * `.pugi/` + the canonical config files. Launching the REPL in an empty
+ * directory printed `workspace: (not bound - run /init OR cd into
+ * project)` and instructed the operator to Ctrl+C, run `pugi init`,
+ * relaunch. That round trip is hostile on a first-touch install — CEO
+ * escalated "auto = решение" on 2026-05-26.
+ *
+ * The module is intentionally side-effect free at import time: the
+ * scaffold runs only when `ensureWorkspaceInitialized` is called. The
+ * scaffold is also idempotent — every file write is gated by an
+ * `existsSync` check, so re-running against a workspace that already has
+ * `.pugi/settings.json` (e.g. a manual `pugi init` followed by auto-init
+ * on next REPL launch) is a no-op. The function is safe to call before
+ * any other init logic.
+ *
+ * Two CRITICAL invariants:
+ *
+ *   1. **Atomic per-file.** Every write uses `existsSync` + `writeFileSync`
+ *      against the final path. There is no read-modify-write pattern that
+ *      could lose data on a concurrent `pugi init` race. The one path
+ *      that DOES mutate an existing file — `.gitignore` (append `.pugi/`
+ *      marker) — also gates on the marker being absent before appending,
+ *      so the worst-case race is a duplicate marker line that the next
+ *      run skips.
+ *
+ *   2. **Silent by default.** When `opts.silent` is true (the REPL
+ *      auto-init path) the scaffold writes NOTHING to stderr/stdout.
+ *      The REPL bootstrap runs before Ink mounts, and a stray
+ *      stdout/stderr write at that point would land on the operator's
+ *      shell ABOVE the alt-screen entry — visible until they scroll up,
+ *      and noisy in a CI tail. The explicit `pugi init` path stays
+ *      verbose via the standalone command in `runtime/cli.ts`.
+ */
+import { existsSync, mkdirSync, readFileSync, writeFileSync } from 'node:fs';
+import { resolve } from 'node:path';
+import { emptyIndex } from '../index-store.js';
+/**
+ * Materialise the canonical `.pugi/` workspace scaffold under `cwd`.
+ * Returns a `{created, dir, createdPaths, skippedPaths}` summary so the
+ * caller can log a one-shot "initialized" line on the first call without
+ * re-checking the filesystem.
+ *
+ * The scaffold mirrors `pugi init` minus the bundled default-skills
+ * install (that is a heavier operation gated on the `--no-defaults`
+ * flag, and the standalone `pugi init` command keeps owning it).
+ *
+ * Idempotent: every file write gates on `existsSync`, so re-running
+ * against an existing workspace is a no-op and returns
+ * `{created: false}` with every path in `skippedPaths`.
+ */
+export function ensureWorkspaceInitialized(cwd, opts = {}) {
+    const silent = opts.silent !== false;
+    const pugiDir = resolve(cwd, '.pugi');
+    // Local trackers so the existing helpers (mkdirIfMissing /
+    // writeJsonIfMissing / writeTextIfMissing) keep their (created, skipped)
+    // signature. The explicit `pugi init` command forwards these straight
+    // into its JSON payload.
+    const created = [];
+    const skipped = [];
+    mkdirIfMissing(pugiDir, created, skipped);
+    mkdirIfMissing(resolve(pugiDir, 'artifacts'), created, skipped);
+    mkdirIfMissing(resolve(pugiDir, 'sessions'), created, skipped);
+    mkdirIfMissing(resolve(pugiDir, 'skills'), created, skipped);
+    writeJsonIfMissing(resolve(pugiDir, 'settings.json'), {
+        schema: 1,
+        workflow: {
+            brand: 'pugi',
+            legacyName: 'codeforge',
+            approvals: 'auto',
+            notAutomatic: [],
+            defaultBaseBranch: 'dev',
+            branchPrefixes: ['feature', 'fix', 'refactor', 'chore'],
+            aiCoAuthorTrailers: false,
+        },
+        permissions: {
+            mode: 'auto',
+            allow: [],
+            deny: [],
+            notAutomatic: [],
+        },
+        privacy: {
+            mode: 'balanced',
+            telemetry: 'off',
+        },
+        artifacts: {
+            defaultPath: '.pugi/artifacts',
+            promoteExplicitly: true,
+        },
+    }, created, skipped);
+    writeJsonIfMissing(resolve(pugiDir, 'mcp.json'), { schema: 1, servers: [] }, created, skipped);
+    writeJsonIfMissing(resolve(pugiDir, 'index.json'), emptyIndex(), created, skipped);
+    writeTextIfMissing(resolve(pugiDir, 'PUGI.md'), [
+        '# Pugi Project Context',
+        '',
+        '## Product Workflow',
+        '',
+        '- Public product name: Pugi',
+        '- Default flow: idea -> build -> review',
+        '- Approvals are automatic by default until a repo, environment, workflow, or action is marked notAutomatic.',
+        '- Do not add AI Co-Authored-By trailers.',
+        '- Generated code, comments, commits, PR text, and technical docs default to English.',
+        '',
+        '## Project Notes',
+        '',
+        '- Add repo-specific architecture, commands, and business rules here.',
+        '- Do not store secrets, real IPs, private key paths, tokens, or credentials here.',
+        '',
+    ].join('\n'), created, skipped);
+    writeTextIfMissing(resolve(cwd, '.pugiignore'), [
+        '# Pugi ignore rules',
+        '.env',
+        '.env.*',
+        '!.env.example',
+        'node_modules/',
+        'dist/',
+        '.next/',
+        'coverage/',
+        '*.log',
+        '*.pem',
+        '*.key',
+        '*.crt',
+        '*.p12',
+        '*.sql',
+        '*.dump',
+        '',
+    ].join('\n'), created, skipped);
+    ensurePugiGitIgnore(cwd, created, skipped);
+    // `silent` is honoured implicitly — this module never writes to
+    // stdout/stderr. The flag exists so the standalone `pugi init` command
+    // can layer its own logger on top (it does, in runtime/cli.ts), while
+    // the auto-init REPL path leaves the boot stream untouched. We
+    // reference the flag here to defeat the lint "unused" warning and to
+    // document the contract in the source.
+    void silent;
+    return {
+        created: created.length > 0,
+        dir: pugiDir,
+        createdPaths: created,
+        skippedPaths: skipped,
+    };
+}
+/* ------------------------------------------------------------------ */
+/* Helpers (mirror the previous in-file implementations in cli.ts)     */
+/* ------------------------------------------------------------------ */
+function mkdirIfMissing(path, created, skipped) {
+    if (existsSync(path)) {
+        skipped.push(path);
+        return;
+    }
+    mkdirSync(path, { recursive: true });
+    created.push(path);
+}
+function writeJsonIfMissing(path, value, created, skipped) {
+    writeTextIfMissing(path, `${JSON.stringify(value, null, 2)}\n`, created, skipped);
+}
+function writeTextIfMissing(path, value, created, skipped) {
+    if (existsSync(path)) {
+        skipped.push(path);
+        return;
+    }
+    writeFileSync(path, value, { encoding: 'utf8', mode: 0o600 });
+    created.push(path);
+}
+/**
+ * Ensure the workspace `.gitignore` ignores `.pugi/`. The function is
+ * additive: it leaves an existing `.gitignore` body intact and appends
+ * the marker only when none of `.pugi/`, `/.pugi/`, or `.pugi` is
+ * already present. On a fresh repo with no `.gitignore` it creates the
+ * file with the single marker line. Mode 0o600 matches the rest of the
+ * scaffold so a paranoid CI does not surface "world-readable" warnings.
+ */
+function ensurePugiGitIgnore(cwd, created, skipped) {
+    const gitignorePath = resolve(cwd, '.gitignore');
+    const marker = '.pugi/';
+    if (!existsSync(gitignorePath)) {
+        writeFileSync(gitignorePath, `${marker}\n`, { encoding: 'utf8', mode: 0o600 });
+        created.push(gitignorePath);
+        return;
+    }
+    const current = readFileSync(gitignorePath, 'utf8');
+    const lines = current.split('\n').map((line) => line.trim());
+    if (lines.includes(marker) || lines.includes('/.pugi/') || lines.includes('.pugi')) {
+        skipped.push(gitignorePath);
+        return;
+    }
+    const next = current.endsWith('\n') ? `${current}${marker}\n` : `${current}\n${marker}\n`;
+    writeFileSync(gitignorePath, next, { encoding: 'utf8' });
+    created.push(`${gitignorePath} (+${marker})`);
+}
+//# sourceMappingURL=scaffold.js.map

package/dist/core/lsp/cache.js

@@ -0,0 +1,105 @@
+/**
+ * Per-process LSP client cache — Leak L15.
+ *
+ * The α7.7 `runtime/commands/lsp.ts` CLI surface spawns one LSP server
+ * per invocation and stops it at the end. That is correct for the
+ * one-shot `pugi lsp hover ...` shape but wrong for L15's
+ * post-edit auto-diagnostics: every successful `edit`/`write` would
+ * otherwise pay the ~2-3s cold-start of `typescript-language-server`,
+ * which is unusable inside an agent loop.
+ *
+ * This module owns a singleton map keyed by `LspLanguage` with lazy
+ * initialization (`getOrStart`). The first edit of a TS file in a
+ * session pays cold-start; every subsequent edit of any TS/TSX file
+ * in the same workspace reuses the warm client.
+ *
+ * Lifecycle:
+ *   - `getOrStart(lang, cwd)` — spawn if missing, return cached otherwise.
+ *   - `stopAll()` — graceful shutdown of every cached client. Called from
+ *     `runCli` exit so a Ctrl-C never leaves zombie LSP processes behind.
+ *   - `reset()` — test-only escape hatch, drops the cache without
+ *     touching child processes (specs inject stubs that own their own
+ *     lifecycle).
+ *
+ * Failure handling: a startup failure is NOT cached. The next call
+ * tries again. This keeps the cache from poisoning a session when the
+ * operator installs the missing LSP binary mid-session and re-edits.
+ *
+ * Brand voice: ASCII only, no emoji, no banned words.
+ */
+import { isLspLanguageDisabled, startLspClient, } from './client.js';
+const cache = new Map();
+/**
+ * Return a warm client for `lang`, starting one if needed. The
+ * workspace `cwd` is captured at cache-insert time; if a subsequent
+ * call asks for the same language with a different `cwd` we tear
+ * down the old client and start a fresh one. This handles the
+ * agent-worktree case where the same process hops between workspace
+ * roots inside one Node lifetime.
+ */
+export async function getOrStartLspClient(lang, opts) {
+    // β7 L9: respect the per-language disable toggle BEFORE we attempt to
+    // spawn. The check is cheap and keeps the disabled path from paying
+    // the `npx --yes` warmup cost on first use.
+    if (isLspLanguageDisabled(lang, opts.lspSettings)) {
+        return {
+            ok: false,
+            reason: 'lsp_disabled',
+            detail: `${lang} is disabled via .pugi/settings.json::lsp`,
+        };
+    }
+    const existing = cache.get(lang);
+    if (existing && existing.cwd === opts.cwd) {
+        return { ok: true, client: existing.client };
+    }
+    if (existing && existing.cwd !== opts.cwd) {
+        // Workspace switched — stop the old client and fall through to spawn.
+        try {
+            await existing.client.stop();
+        }
+        catch {
+            // best effort; stop() is idempotent + swallow-safe
+        }
+        cache.delete(lang);
+    }
+    const result = await startLspClient(lang, opts);
+    if (!result.ok) {
+        return { ok: false, reason: result.reason, detail: result.detail };
+    }
+    cache.set(lang, { client: result.value, cwd: opts.cwd });
+    return { ok: true, client: result.value };
+}
+/** Look up the cached client without starting one. Returns undefined when missing. */
+export function peekLspClient(lang) {
+    return cache.get(lang)?.client;
+}
+/** Snapshot of currently-cached languages — used by `pugi lsp status` debug output. */
+export function listCachedLanguages() {
+    return Array.from(cache.keys());
+}
+/**
+ * Stop every cached client and clear the cache. Called from `runCli`
+ * exit and from specs that own the lifecycle of their stub servers.
+ */
+export async function stopAllLspClients() {
+    const snapshot = Array.from(cache.values());
+    cache.clear();
+    await Promise.all(snapshot.map(async (entry) => {
+        try {
+            await entry.client.stop();
+        }
+        catch {
+            // best effort — shutting down anyway
+        }
+    }));
+}
+/**
+ * Test-only: drop the cache map WITHOUT calling stop on the children.
+ * Specs that inject stub servers manage the stub lifecycle themselves;
+ * this lets a spec swap a stub mid-test without the cache holding a
+ * stale reference to a torn-down process.
+ */
+export function __resetLspCacheForTests() {
+    cache.clear();
+}
+//# sourceMappingURL=cache.js.map

package/dist/core/lsp/language-detect.js

@@ -0,0 +1,66 @@
+/**
+ * Language-from-extension detection — Leak L15.
+ *
+ * Single source of truth for "given a file path, which `LspLanguage`
+ * slug do we route to". The α7.7 `runtime/commands/lsp.ts` shipped its
+ * own inline `inferLanguage` switch; L15 (post-edit auto-diagnostics)
+ * needs the same lookup from `core/engine/tool-bridge.ts`, so we lift
+ * the table into a dedicated module to avoid a second copy drifting
+ * out of sync.
+ *
+ * Returning `undefined` is the calling code's signal to silently skip
+ * LSP — an unsupported extension is NOT an error, it just means "no
+ * diagnostics for this file". The tool-bridge hook treats this as a
+ * no-op envelope tail.
+ *
+ * Adding a new language requires THREE coordinated changes:
+ *   1. Add the `LspLanguage` slug + server descriptor in `client.ts`.
+ *   2. Map its extensions here.
+ *   3. Add a `lsp-language-matrix` spec row exercising the new ext.
+ *
+ * Brand voice: ASCII only, no emoji, no banned words.
+ */
+import { extname } from 'node:path';
+/**
+ * Lower-case extension (including the dot) → LSP language slug.
+ * Mirror of the switch in `runtime/commands/lsp.ts::inferLanguage`.
+ * The table form lets tests assert coverage and lets new languages
+ * land with one edit instead of two.
+ */
+export const EXTENSION_TO_LANGUAGE = {
+    '.ts': 'ts',
+    '.tsx': 'ts',
+    '.mts': 'ts',
+    '.cts': 'ts',
+    '.js': 'js',
+    '.jsx': 'js',
+    '.mjs': 'js',
+    '.cjs': 'js',
+    '.py': 'py',
+    '.pyi': 'py',
+    '.go': 'go',
+    '.rs': 'rust',
+};
+/**
+ * Infer the `LspLanguage` for a workspace-relative or absolute path.
+ * Returns `undefined` for unmapped extensions — the caller decides
+ * whether that is silently skipped (post-edit hook) or surfaced as
+ * `language_unsupported` (`pugi lsp` CLI).
+ */
+export function languageForFile(file) {
+    const ext = extname(file).toLowerCase();
+    if (!ext)
+        return undefined;
+    return EXTENSION_TO_LANGUAGE[ext];
+}
+/**
+ * Return every extension currently mapped to the given language.
+ * Used by the matrix spec to assert coverage without re-typing the
+ * extension list.
+ */
+export function extensionsForLanguage(lang) {
+    return Object.entries(EXTENSION_TO_LANGUAGE)
+        .filter(([, value]) => value === lang)
+        .map(([ext]) => ext);
+}
+//# sourceMappingURL=language-detect.js.map