lsd-pi 1.0.0 → 1.1.1

package/dist/resources/extensions/gsd/auto/resolve.js

@@ -1,90 +0,0 @@
-/**
- * auto/resolve.ts — Per-unit one-shot promise state and resolution.
- *
- * Module-level mutable state: `_currentResolve` and `_sessionSwitchInFlight`.
- * Setter functions are exported because ES modules can't mutate `let` vars
- * across module boundaries.
- *
- * Imports from: auto/types
- */
-import { debugLog } from "../debug-logger.js";
-// ─── Per-unit one-shot promise state ────────────────────────────────────────
-//
-// A single module-level resolve function scoped to the current unit execution.
-// No queue — if an agent_end arrives with no pending resolver, it is dropped
-// (logged as warning). This is simpler and safer than the previous session-
-// scoped pendingResolve + pendingAgentEndQueue pattern.
-let _currentResolve = null;
-let _sessionSwitchInFlight = false;
-// ─── Setters (needed for cross-module mutation) ─────────────────────────────
-export function _setCurrentResolve(fn) {
-    _currentResolve = fn;
-}
-export function _setSessionSwitchInFlight(v) {
-    _sessionSwitchInFlight = v;
-}
-export function _clearCurrentResolve() {
-    _currentResolve = null;
-}
-// ─── resolveAgentEnd ─────────────────────────────────────────────────────────
-/**
- * Called from the agent_end event handler in index.ts to resolve the
- * in-flight unit promise. One-shot: the resolver is nulled before calling
- * to prevent double-resolution from model fallback retries.
- *
- * If no resolver exists (event arrived between loop iterations or during
- * session switch), the event is dropped with a debug warning.
- */
-export function resolveAgentEnd(event) {
-    if (_sessionSwitchInFlight) {
-        debugLog("resolveAgentEnd", { status: "ignored-during-switch" });
-        return;
-    }
-    if (_currentResolve) {
-        debugLog("resolveAgentEnd", { status: "resolving", hasEvent: true });
-        const r = _currentResolve;
-        _currentResolve = null;
-        r({ status: "completed", event });
-    }
-    else {
-        debugLog("resolveAgentEnd", {
-            status: "no-pending-resolve",
-            warning: "agent_end with no pending unit",
-        });
-    }
-}
-export function isSessionSwitchInFlight() {
-    return _sessionSwitchInFlight;
-}
-// ─── resolveAgentEndCancelled ─────────────────────────────────────────────────
-/**
- * Force-resolve the pending unit promise with { status: "cancelled" }.
- *
- * Used by pauseAuto, handleAgentEnd early-return, and supervision catch
- * blocks to ensure the autoLoop is never stuck awaiting a promise that
- * will never resolve. Safe to call when no resolver is pending (no-op).
- */
-export function resolveAgentEndCancelled(errorContext) {
-    if (_currentResolve) {
-        debugLog("resolveAgentEndCancelled", { status: "resolving-cancelled" });
-        const r = _currentResolve;
-        _currentResolve = null;
-        r({ status: "cancelled", ...(errorContext ? { errorContext } : {}) });
-    }
-}
-// ─── resetPendingResolve (test helper) ───────────────────────────────────────
-/**
- * Reset module-level promise state. Only exported for test cleanup —
- * production code should never call this.
- */
-export function _resetPendingResolve() {
-    _currentResolve = null;
-    _sessionSwitchInFlight = false;
-}
-/**
- * No-op for backward compatibility with tests that previously set the
- * active session. The module no longer holds a session reference.
- */
-export function _setActiveSession(_session) {
-    // No-op — kept for test backward compatibility
-}

package/dist/resources/extensions/gsd/auto/run-unit.js

@@ -1,106 +0,0 @@
-/**
- * auto/run-unit.ts — Single unit execution: session create → prompt → await agent_end.
- *
- * Imports from: auto/types, auto/resolve
- */
-import { NEW_SESSION_TIMEOUT_MS } from "./session.js";
-import { _setCurrentResolve, _setSessionSwitchInFlight } from "./resolve.js";
-import { debugLog } from "../debug-logger.js";
-import { logWarning } from "../workflow-logger.js";
-/**
- * Execute a single unit: create a new session, send the prompt, and await
- * the agent_end promise. Returns a UnitResult describing what happened.
- *
- * The promise is one-shot: resolveAgentEnd() is the only way to resolve it.
- * On session creation failure or timeout, returns { status: 'cancelled' }
- * without awaiting the promise.
- */
-export async function runUnit(ctx, pi, s, unitType, unitId, prompt) {
-    debugLog("runUnit", { phase: "start", unitType, unitId });
-    // ── Session creation with timeout ──
-    debugLog("runUnit", { phase: "session-create", unitType, unitId });
-    let sessionResult;
-    let sessionTimeoutHandle;
-    _setSessionSwitchInFlight(true);
-    try {
-        const sessionPromise = s.cmdCtx.newSession().finally(() => {
-            _setSessionSwitchInFlight(false);
-        });
-        const timeoutPromise = new Promise((resolve) => {
-            sessionTimeoutHandle = setTimeout(() => resolve({ cancelled: true }), NEW_SESSION_TIMEOUT_MS);
-        });
-        sessionResult = await Promise.race([sessionPromise, timeoutPromise]);
-    }
-    catch (sessionErr) {
-        if (sessionTimeoutHandle)
-            clearTimeout(sessionTimeoutHandle);
-        const msg = sessionErr instanceof Error ? sessionErr.message : String(sessionErr);
-        debugLog("runUnit", {
-            phase: "session-error",
-            unitType,
-            unitId,
-            error: msg,
-        });
-        return { status: "cancelled", errorContext: { message: `Session creation failed: ${msg}`, category: "session-failed", isTransient: true } };
-    }
-    if (sessionTimeoutHandle)
-        clearTimeout(sessionTimeoutHandle);
-    if (sessionResult.cancelled) {
-        debugLog("runUnit-session-timeout", { unitType, unitId });
-        return { status: "cancelled", errorContext: { message: "Session creation timed out", category: "timeout", isTransient: true } };
-    }
-    if (!s.active) {
-        return { status: "cancelled" };
-    }
-    if (s.currentUnitModel && typeof pi.setModel === "function") {
-        const restored = await pi.setModel(s.currentUnitModel, { persist: false });
-        if (!restored) {
-            ctx.ui.notify(`Failed to restore ${s.currentUnitModel.provider}/${s.currentUnitModel.id} after session creation. Using session default.`, "warning");
-        }
-    }
-    // ── Create the agent_end promise (per-unit one-shot) ──
-    // This happens after newSession completes so session-switch agent_end events
-    // from the previous session cannot resolve the new unit.
-    _setSessionSwitchInFlight(false);
-    const unitPromise = new Promise((resolve) => {
-        _setCurrentResolve(resolve);
-    });
-    // Ensure cwd matches basePath before dispatch (#1389).
-    // async_bash and background jobs can drift cwd away from the worktree.
-    // Realigning here prevents commits from landing on the wrong branch.
-    try {
-        if (process.cwd() !== s.basePath) {
-            process.chdir(s.basePath);
-        }
-    }
-    catch (e) {
-        logWarning("engine", "Failed to chdir to basePath before dispatch", { basePath: s.basePath, error: String(e) });
-    }
-    // ── Send the prompt ──
-    debugLog("runUnit", { phase: "send-message", unitType, unitId });
-    pi.sendMessage({ customType: "gsd-auto", content: prompt, display: s.verbose }, { triggerTurn: true });
-    // ── Await agent_end ──
-    debugLog("runUnit", { phase: "awaiting-agent-end", unitType, unitId });
-    const result = await unitPromise;
-    debugLog("runUnit", {
-        phase: "agent-end-received",
-        unitType,
-        unitId,
-        status: result.status,
-    });
-    // Discard trailing follow-up messages (e.g. async_job_result notifications)
-    // from the completed unit. Without this, queued follow-ups trigger wasteful
-    // LLM turns before the next session can start (#1642).
-    // clearQueue() lives on AgentSession but isn't part of the typed
-    // ExtensionCommandContext interface — call it via runtime check.
-    try {
-        const cmdCtxAny = s.cmdCtx;
-        if (typeof cmdCtxAny?.clearQueue === "function") {
-            cmdCtxAny.clearQueue();
-        }
-    }
-    catch (e) {
-        logWarning("engine", "clearQueue failed after unit completion", { error: String(e) });
-    }
-    return result;
-}

package/dist/resources/extensions/gsd/auto/session.js

@@ -1,171 +0,0 @@
-/**
- * AutoSession — encapsulates all mutable auto-mode state into a single instance.
- *
- * Replaces ~40 module-level variables scattered across auto.ts with typed
- * properties on a class instance. Benefits:
- *
- * - reset() clears everything in one call (was 25+ manual resets in stopAuto)
- * - toJSON() provides diagnostic snapshots
- * - grep `s.` shows every state access
- * - Constructable for testing
- *
- * MAINTENANCE RULE: All new mutable auto-mode state MUST be added here as a
- * class property, not as a module-level variable in auto.ts. If the state
- * needs clearing on stop, add it to reset(). Tests in
- * auto-session-encapsulation.test.ts enforce that auto.ts has no module-level
- * `let` or `var` declarations.
- */
-// ─── Constants ───────────────────────────────────────────────────────────────
-export const MAX_UNIT_DISPATCHES = 3;
-export const STUB_RECOVERY_THRESHOLD = 2;
-export const MAX_LIFETIME_DISPATCHES = 6;
-export const NEW_SESSION_TIMEOUT_MS = 30_000;
-// ─── AutoSession ─────────────────────────────────────────────────────────────
-export class AutoSession {
-    // ── Lifecycle ────────────────────────────────────────────────────────────
-    active = false;
-    paused = false;
-    stepMode = false;
-    verbose = false;
-    activeEngineId = null;
-    activeRunDir = null;
-    cmdCtx = null;
-    // ── Paths ────────────────────────────────────────────────────────────────
-    basePath = "";
-    originalBasePath = "";
-    gitService = null;
-    // ── Dispatch counters ────────────────────────────────────────────────────
-    unitDispatchCount = new Map();
-    unitLifetimeDispatches = new Map();
-    unitRecoveryCount = new Map();
-    // ── Timers ───────────────────────────────────────────────────────────────
-    unitTimeoutHandle = null;
-    wrapupWarningHandle = null;
-    idleWatchdogHandle = null;
-    continueHereHandle = null;
-    // ── Current unit ─────────────────────────────────────────────────────────
-    currentUnit = null;
-    currentUnitRouting = null;
-    currentMilestoneId = null;
-    // ── Model state ──────────────────────────────────────────────────────────
-    autoModeStartModel = null;
-    currentUnitModel = null;
-    originalModelId = null;
-    originalModelProvider = null;
-    lastBudgetAlertLevel = 0;
-    // ── Recovery ─────────────────────────────────────────────────────────────
-    pendingCrashRecovery = null;
-    pendingVerificationRetry = null;
-    verificationRetryCount = new Map();
-    pausedSessionFile = null;
-    resourceVersionOnStart = null;
-    lastStateRebuildAt = 0;
-    // ── Sidecar queue ─────────────────────────────────────────────────────
-    sidecarQueue = [];
-    // ── Isolation degradation ────────────────────────────────────────────
-    /** Set to true when worktree creation fails; prevents merge of nonexistent branch. */
-    isolationDegraded = false;
-    // ── Merge guard ──────────────────────────────────────────────────────
-    /** Set to true after phases.ts successfully calls mergeAndExit, so that
-     *  stopAuto does not attempt the same merge a second time (#2645). */
-    milestoneMergedInPhases = false;
-    // ── Dispatch circuit breakers ──────────────────────────────────────
-    rewriteAttemptCount = 0;
-    // ── Metrics ──────────────────────────────────────────────────────────────
-    autoStartTime = 0;
-    lastPromptCharCount;
-    lastBaselineCharCount;
-    pendingQuickTasks = [];
-    // ── Signal handler ───────────────────────────────────────────────────────
-    sigtermHandler = null;
-    // ── Loop promise state ──────────────────────────────────────────────────
-    // Per-unit resolve function and session-switch guard live at module level
-    // in auto-loop.ts (_currentResolve, _sessionSwitchInFlight).
-    // ── Methods ──────────────────────────────────────────────────────────────
-    clearTimers() {
-        if (this.unitTimeoutHandle) {
-            clearTimeout(this.unitTimeoutHandle);
-            this.unitTimeoutHandle = null;
-        }
-        if (this.wrapupWarningHandle) {
-            clearTimeout(this.wrapupWarningHandle);
-            this.wrapupWarningHandle = null;
-        }
-        if (this.idleWatchdogHandle) {
-            clearInterval(this.idleWatchdogHandle);
-            this.idleWatchdogHandle = null;
-        }
-        if (this.continueHereHandle) {
-            clearInterval(this.continueHereHandle);
-            this.continueHereHandle = null;
-        }
-    }
-    resetDispatchCounters() {
-        this.unitDispatchCount.clear();
-        this.unitLifetimeDispatches.clear();
-    }
-    get lockBasePath() {
-        return this.originalBasePath || this.basePath;
-    }
-    reset() {
-        this.clearTimers();
-        // Lifecycle
-        this.active = false;
-        this.paused = false;
-        this.stepMode = false;
-        this.verbose = false;
-        this.activeEngineId = null;
-        this.activeRunDir = null;
-        this.cmdCtx = null;
-        // Paths
-        this.basePath = "";
-        this.originalBasePath = "";
-        this.gitService = null;
-        // Dispatch
-        this.unitDispatchCount.clear();
-        this.unitLifetimeDispatches.clear();
-        this.unitRecoveryCount.clear();
-        // Unit
-        this.currentUnit = null;
-        this.currentUnitRouting = null;
-        this.currentMilestoneId = null;
-        // Model
-        this.autoModeStartModel = null;
-        this.currentUnitModel = null;
-        this.originalModelId = null;
-        this.originalModelProvider = null;
-        this.lastBudgetAlertLevel = 0;
-        // Recovery
-        this.pendingCrashRecovery = null;
-        this.pendingVerificationRetry = null;
-        this.verificationRetryCount.clear();
-        this.pausedSessionFile = null;
-        this.resourceVersionOnStart = null;
-        this.lastStateRebuildAt = 0;
-        // Metrics
-        this.autoStartTime = 0;
-        this.lastPromptCharCount = undefined;
-        this.lastBaselineCharCount = undefined;
-        this.pendingQuickTasks = [];
-        this.sidecarQueue = [];
-        this.rewriteAttemptCount = 0;
-        this.isolationDegraded = false;
-        this.milestoneMergedInPhases = false;
-        // Signal handler
-        this.sigtermHandler = null;
-        // Loop promise state lives in auto-loop.ts module scope
-    }
-    toJSON() {
-        return {
-            active: this.active,
-            paused: this.paused,
-            stepMode: this.stepMode,
-            basePath: this.basePath,
-            activeEngineId: this.activeEngineId,
-            activeRunDir: this.activeRunDir,
-            currentMilestoneId: this.currentMilestoneId,
-            currentUnit: this.currentUnit,
-            unitDispatchCount: Object.fromEntries(this.unitDispatchCount),
-        };
-    }
-}

package/dist/resources/extensions/gsd/auto/types.js

@@ -1,23 +0,0 @@
-/**
- * auto/types.ts — Constants and types shared across auto-loop modules.
- *
- * Leaf node in the import DAG — no imports from auto/.
- */
-/**
- * Maximum total loop iterations before forced stop. Prevents runaway loops
- * when units alternate IDs (bypassing the same-unit stuck detector).
- * A milestone with 20 slices × 5 tasks × 3 phases ≈ 300 units. 500 gives
- * generous headroom including retries and sidecar work.
- */
-export const MAX_LOOP_ITERATIONS = 500;
-/** Maximum characters of failure/crash context included in recovery prompts. */
-export const MAX_RECOVERY_CHARS = 50_000;
-/** Data-driven budget threshold notifications (descending). The 100% entry
- *  triggers special enforcement logic (halt/pause/warn); sub-100 entries fire
- *  a simple notification. */
-export const BUDGET_THRESHOLDS = [
-    { pct: 100, label: "Budget ceiling reached", notifyLevel: "error", cmuxLevel: "error" },
-    { pct: 90, label: "Budget 90%", notifyLevel: "warning", cmuxLevel: "warning" },
-    { pct: 80, label: "Approaching budget ceiling — 80%", notifyLevel: "warning", cmuxLevel: "warning" },
-    { pct: 75, label: "Budget 75%", notifyLevel: "info", cmuxLevel: "progress" },
-];

package/dist/resources/extensions/gsd/auto-artifact-paths.js

@@ -1,116 +0,0 @@
-// GSD Auto-mode — Artifact Path Resolution
-//
-// resolveExpectedArtifactPath and diagnoseExpectedArtifact moved here from
-// auto-recovery.ts (Phase 5 dead-code cleanup). The artifact verification
-// function was removed entirely — callers now query WorkflowEngine directly.
-import { resolveMilestonePath, resolveSlicePath, relMilestoneFile, relSliceFile, buildMilestoneFileName, buildSliceFileName, buildTaskFileName, } from "./paths.js";
-import { parseUnitId } from "./unit-id.js";
-import { join } from "node:path";
-/**
- * Resolve the expected artifact for a unit to an absolute path.
- */
-export function resolveExpectedArtifactPath(unitType, unitId, base) {
-    const { milestone: mid, slice: sid, task: tid } = parseUnitId(unitId);
-    switch (unitType) {
-        case "discuss-milestone": {
-            const dir = resolveMilestonePath(base, mid);
-            return dir ? join(dir, buildMilestoneFileName(mid, "CONTEXT")) : null;
-        }
-        case "discuss-slice": {
-            const dir = resolveSlicePath(base, mid, sid);
-            return dir ? join(dir, buildSliceFileName(sid, "CONTEXT")) : null;
-        }
-        case "research-milestone": {
-            const dir = resolveMilestonePath(base, mid);
-            return dir ? join(dir, buildMilestoneFileName(mid, "RESEARCH")) : null;
-        }
-        case "plan-milestone": {
-            const dir = resolveMilestonePath(base, mid);
-            return dir ? join(dir, buildMilestoneFileName(mid, "ROADMAP")) : null;
-        }
-        case "research-slice": {
-            const dir = resolveSlicePath(base, mid, sid);
-            return dir ? join(dir, buildSliceFileName(sid, "RESEARCH")) : null;
-        }
-        case "plan-slice": {
-            const dir = resolveSlicePath(base, mid, sid);
-            return dir ? join(dir, buildSliceFileName(sid, "PLAN")) : null;
-        }
-        case "reassess-roadmap": {
-            const dir = resolveSlicePath(base, mid, sid);
-            return dir ? join(dir, buildSliceFileName(sid, "ASSESSMENT")) : null;
-        }
-        case "run-uat": {
-            const dir = resolveSlicePath(base, mid, sid);
-            return dir ? join(dir, buildSliceFileName(sid, "ASSESSMENT")) : null;
-        }
-        case "execute-task": {
-            const dir = resolveSlicePath(base, mid, sid);
-            return dir && tid
-                ? join(dir, "tasks", buildTaskFileName(tid, "SUMMARY"))
-                : null;
-        }
-        case "complete-slice": {
-            const dir = resolveSlicePath(base, mid, sid);
-            return dir ? join(dir, buildSliceFileName(sid, "SUMMARY")) : null;
-        }
-        case "validate-milestone": {
-            const dir = resolveMilestonePath(base, mid);
-            return dir ? join(dir, buildMilestoneFileName(mid, "VALIDATION")) : null;
-        }
-        case "complete-milestone": {
-            const dir = resolveMilestonePath(base, mid);
-            return dir ? join(dir, buildMilestoneFileName(mid, "SUMMARY")) : null;
-        }
-        case "replan-slice": {
-            const dir = resolveSlicePath(base, mid, sid);
-            return dir ? join(dir, buildSliceFileName(sid, "REPLAN")) : null;
-        }
-        case "rewrite-docs":
-            return null;
-        case "gate-evaluate":
-            // Gate evaluate writes to DB quality_gates table — verified via state derivation
-            return null;
-        case "reactive-execute":
-            // Reactive execute produces multiple task summaries — verified separately
-            return null;
-        default:
-            return null;
-    }
-}
-export function diagnoseExpectedArtifact(unitType, unitId, base) {
-    const { milestone: mid, slice: sid, task: tid } = parseUnitId(unitId);
-    switch (unitType) {
-        case "discuss-milestone":
-            return `${relMilestoneFile(base, mid, "CONTEXT")} (milestone context from discussion)`;
-        case "discuss-slice":
-            return `${relSliceFile(base, mid, sid, "CONTEXT")} (slice context from discussion)`;
-        case "research-milestone":
-            return `${relMilestoneFile(base, mid, "RESEARCH")} (milestone research)`;
-        case "plan-milestone":
-            return `${relMilestoneFile(base, mid, "ROADMAP")} (milestone roadmap)`;
-        case "research-slice":
-            return `${relSliceFile(base, mid, sid, "RESEARCH")} (slice research)`;
-        case "plan-slice":
-            return `${relSliceFile(base, mid, sid, "PLAN")} (slice plan)`;
-        case "execute-task": {
-            return `Task ${tid} marked [x] in ${relSliceFile(base, mid, sid, "PLAN")} + summary written`;
-        }
-        case "complete-slice":
-            return `Slice ${sid} marked [x] in ${relMilestoneFile(base, mid, "ROADMAP")} + summary + UAT written`;
-        case "replan-slice":
-            return `${relSliceFile(base, mid, sid, "REPLAN")} + updated ${relSliceFile(base, mid, sid, "PLAN")}`;
-        case "rewrite-docs":
-            return "Active overrides resolved in .gsd/OVERRIDES.md + plan documents updated";
-        case "reassess-roadmap":
-            return `${relSliceFile(base, mid, sid, "ASSESSMENT")} (roadmap reassessment)`;
-        case "run-uat":
-            return `${relSliceFile(base, mid, sid, "ASSESSMENT")} (UAT assessment result)`;
-        case "validate-milestone":
-            return `${relMilestoneFile(base, mid, "VALIDATION")} (milestone validation report)`;
-        case "complete-milestone":
-            return `${relMilestoneFile(base, mid, "SUMMARY")} (milestone summary)`;
-        default:
-            return null;
-    }
-}

package/dist/resources/extensions/gsd/auto-budget.js

@@ -1,30 +0,0 @@
-/**
- * Budget alert level tracking and enforcement for auto-mode.
- * Pure functions — no module state or side effects.
- */
-export function getBudgetAlertLevel(budgetPct) {
-    if (budgetPct >= 1.0)
-        return 100;
-    if (budgetPct >= 0.90)
-        return 90;
-    if (budgetPct >= 0.80)
-        return 80;
-    if (budgetPct >= 0.75)
-        return 75;
-    return 0;
-}
-export function getNewBudgetAlertLevel(previousLevel, budgetPct) {
-    const currentLevel = getBudgetAlertLevel(budgetPct);
-    if (currentLevel === 0 || currentLevel <= previousLevel)
-        return null;
-    return currentLevel;
-}
-export function getBudgetEnforcementAction(enforcement, budgetPct) {
-    if (budgetPct < 1.0)
-        return "none";
-    if (enforcement === "halt")
-        return "halt";
-    if (enforcement === "pause")
-        return "pause";
-    return "warn";
-}