@opengsd/gsd-pi 1.1.1-dev.3ea310e → 1.1.1-dev.595401e

package/dist/resources/.managed-resources-content-hash

@@ -1 +1 @@
-3a34fcd8ba090855
+e81bc72bf9d51027

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

@@ -256,7 +256,6 @@ export class AutoOrchestrator {
             this.status.activeUnit = { unitType: decision.unitType, unitId: decision.unitId };
             this.status.phase = "running";
             this.lastAdvanceKey = nextKey;
-            this.lastFinalizedUnitKey = null;
             this.bumpTransition();
             await this.deps.runtime.journalTransition({
                 name: "advance",

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

@@ -51,13 +51,13 @@ import { resolveManifest } from "../unit-context-manifest.js";
 import { createWorktreeSafetyModule } from "../worktree-safety.js";
 import { isSuspiciousGhostCompletion } from "../auto-unit-closeout.js";
 import { decideVerificationRetry, verificationRetryKey } from "./verification-retry-policy.js";
-import { buildPhaseHandoffOutcome, setAutoOutcomeWidget } from "../auto-dashboard.js";
+import { buildPhaseHandoffOutcome, setAutoActiveStatus, setAutoOutcomeWidget } from "../auto-dashboard.js";
 import { getConsecutiveDispatchBlocker } from "../dispatch-guard.js";
 import { captureRootDirtySnapshot, detectRootWriteLeak, formatRootWriteLeakMessage, } from "../root-write-leak-guard.js";
 import { classifyError, isTransient } from "../error-classifier.js";
 export const STUCK_WINDOW_SIZE = 6;
 const STUCK_RECOVERY_ATTEMPTS_KEY = "stuck_recovery_attempts";
-const ZERO_TOOL_PROVIDER_ERROR_PREFIX_RE = /^(?:api error(?::|$|\s*\()|provider error(?::|$|\s*\()|request failed\b|(?:http\s*)?(?:429|500|502|503)\b|\b(?:econnreset|etimedout|econnrefused|epipe)\b|socket hang up\b|fetch failed\b|(?:network|connection|server) error(?::|$)|connection (?:reset|refused)(?::|$|\s+by\b)|dns\b.*(?:fail|error|timeout)|unexpected eof\b|stream idle timeout\b|partial response received\b|stream_exhausted\b|terminated(?::|$)|(?:connection|stream|request)\b.{0,40}\bterminated\b|other side closed\b|rate.?limit(?:ed| exceeded| reached| error)|too many requests\b|you(?:'ve| have) hit your limit\b|usage limit\b|out of extra usage\b|service.?unavailable\b|internal(?: server)? error(?::|$)|internal(?:[_-]server)?[_-]error\b|server[_-]error\b|(?:provider|server|api|model|codex|claude|openai|anthropic|gemini)\b.{0,80}\boverloaded\b|overloaded\b.{0,80}\b(?:provider|server|api|model)\b|context (?:window|length) exceed|context window exceed)/i;
+const ZERO_TOOL_PROVIDER_ERROR_PREFIX_RE = /^(?:api error(?::|$|\s*\()|provider error(?::|$|\s*\()|request failed\b|(?:http\s*)?(?:429|500|502|503)\b|\b(?:econnreset|etimedout|econnrefused|epipe)\b|socket hang up\b|fetch failed\b|(?:network|connection|server) error(?::|$)|connection (?:reset|refused)(?::|$|\s+by\b)|dns\b.*(?:fail|error|timeout)|unexpected eof\b|stream idle timeout\b|partial response received\b|stream_exhausted\b|terminated(?::|$)|(?:connection|stream|request)\b.{0,40}\bterminated\b|other side closed\b|rate.?limit(?:ed| exceeded| reached| error)|too many requests\b|you(?:'ve| have) (?:hit|reached) your (?:\w+ )?limit\b|.*\b(?:usage|session|weekly|daily|monthly|quota) limit\b|limit\b.{0,40}\bresets?\b|out of extra usage\b|service.?unavailable\b|internal(?: server)? error(?::|$)|internal(?:[_-]server)?[_-]error\b|server[_-]error\b|(?:provider|server|api|model|codex|claude|openai|anthropic|gemini)\b.{0,80}\boverloaded\b|overloaded\b.{0,80}\b(?:provider|server|api|model)\b|context (?:window|length) exceed|context window exceed)/i;
 const ZERO_TOOL_PROVIDER_ERROR_SIGNAL_RE = /(?:\b(?:http|status(?: code)?|code|error:)\s*(?:429|500|502|503)\b|\b(?:api|provider) error\s*[:(]?\s*(?:429|500|502|503)\b|\b(?:typeerror|error):\s*(?:fetch failed\b|socket hang up\b|terminated(?::|$)|connection (?:reset|refused)(?::|$|\s+by\b)|(?:network|connection|server) error(?::|$)|stream idle timeout\b|partial response received\b|unexpected eof\b)|\b(?:server_error|api_error|stream_exhausted(?:_without_result)?)\b|\b(?:econnreset|etimedout|econnrefused|epipe)\b|context (?:window|length) exceed|context window exceed)/i;
 function classifyZeroToolProviderMessage(message) {
     const firstLine = message.trim().split(/\r?\n/, 1)[0]?.trim() ?? "";
@@ -67,6 +67,7 @@ function classifyZeroToolProviderMessage(message) {
         return null;
     return classifyError(firstLine);
 }
+export const _classifyZeroToolProviderMessageForTest = classifyZeroToolProviderMessage;
 export function resolveDispatchRecoveryAttempts(unitRecoveryCount, unitType, unitId) {
     return (unitRecoveryCount.get(`${unitType}/${unitId}`) ?? 0) > 0
         ? 0
@@ -1582,7 +1583,7 @@ export async function runUnitPhase(ic, iterData, loopState, sidecarItem) {
     const dispatchKey = `${unitType}/${unitId}`;
     const nextDispatchCount = (s.unitDispatchCount.get(dispatchKey) ?? 0) + 1;
     // Status bar (widget + preconditions deferred until after model selection — see #2899)
-    ctx.ui.setStatus("gsd-auto", s.stepMode ? "next" : "auto");
+    setAutoActiveStatus(ctx, s.stepMode ? "next" : "auto");
     if (mid)
         deps.updateSliceProgressCache(s.basePath, mid, state.activeSlice?.id);
     // ── Safety harness: reset evidence + create checkpoint ──

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

@@ -1,6 +1,6 @@
 // gsd-pi + src/resources/extensions/gsd/auto-dashboard.ts - Auto-mode progress widget rendering and dashboard helpers.
 import { getActiveHook } from "./post-unit-hooks.js";
-import { getLedger } from "./metrics.js";
+import { getLedger, getProjectTotals } from "./metrics.js";
 import { getErrorMessage } from "./error-utils.js";
 import { nativeIsRepo } from "./native-git-bridge.js";
 import { isDbAvailable, getMilestoneSlices, getSliceTasks } from "./gsd-db.js";
@@ -196,6 +196,39 @@ export function formatRuntimeHealthSignal(record, now = Date.now()) {
 export function shouldRenderRoadmapProgress(progress) {
     return !!progress && progress.total > 0;
 }
+function widgetGridLabel(theme, text, color = "borderAccent") {
+    return theme.fg(color, theme.bold(text.toUpperCase()));
+}
+function widgetGridColumn(content, width) {
+    return padRightVisible(truncateToWidth(content, width, "…"), width);
+}
+function widgetGridColumns(theme, width, parts) {
+    if (parts.length === 0)
+        return "";
+    const gap = theme.fg("dim", " │ ");
+    const gapWidth = visibleWidth(gap) * (parts.length - 1);
+    const available = Math.max(parts.length * 8, width - gapWidth);
+    const base = Math.floor(available / parts.length);
+    let remaining = available - base * parts.length;
+    const columns = parts.map((part) => {
+        const columnWidth = base + (remaining > 0 ? 1 : 0);
+        remaining--;
+        return widgetGridColumn(part, columnWidth);
+    });
+    return truncateToWidth(columns.join(gap), width, "…");
+}
+function formatSmallWidgetSpend() {
+    const ledger = getLedger();
+    if (!ledger || ledger.units.length === 0)
+        return "--";
+    const totals = getProjectTotals(ledger.units);
+    const parts = [];
+    if (totals.tokens.total > 0)
+        parts.push(formatWidgetTokens(totals.tokens.total));
+    if (totals.cost > 0)
+        parts.push(`$${totals.cost.toFixed(2)}`);
+    return parts.length > 0 ? parts.join(" · ") : "--";
+}
 // ─── ETA Estimation ──────────────────────────────────────────────────────────
 /**
  * Estimate remaining time based on average unit duration from the metrics ledger.
@@ -369,8 +402,9 @@ export const hideFooter = (_tui, theme, footerData) => ({
     invalidate() { },
     dispose() { },
 });
+export const DEFAULT_WIDGET_MODE = "small";
 const WIDGET_MODES = ["full", "small", "min", "off"];
-let widgetMode = "full";
+let widgetMode = DEFAULT_WIDGET_MODE;
 let widgetModeInitialized = false;
 let widgetModePreferencePath = null;
 function safeReadTextFile(path) {
@@ -473,10 +507,19 @@ export function getWidgetMode(projectPath, globalPath) {
 }
 /** Test-only reset for widget mode caching. */
 export function _resetWidgetModeForTests() {
-    widgetMode = "full";
+    widgetMode = DEFAULT_WIDGET_MODE;
     widgetModeInitialized = false;
     widgetModePreferencePath = null;
 }
+function clearAutoOutcomeWidget(ctx) {
+    if (!ctx.hasUI)
+        return;
+    ctx.ui.setWidget("gsd-outcome", undefined);
+}
+export function setAutoActiveStatus(ctx, status) {
+    ctx.ui.setStatus("gsd-auto", status);
+    clearAutoOutcomeWidget(ctx);
+}
 export function updateProgressWidget(ctx, unitType, unitId, state, accessors, tierBadge) {
     if (!ctx.hasUI)
         return;
@@ -495,7 +538,7 @@ export function updateProgressWidget(ctx, unitType, unitId, state, accessors, ti
         ctx.ui.setStatus("gsd-step", undefined);
     }
     if (!accessors.isSessionSwitching()) {
-        ctx.ui.setWidget("gsd-outcome", undefined);
+        clearAutoOutcomeWidget(ctx);
     }
     const verb = unitVerb(unitType);
     const phaseLabel = unitPhaseLabel(unitType);
@@ -548,6 +591,7 @@ export function updateProgressWidget(ctx, unitType, unitId, state, accessors, ti
                 logWarning("dashboard", `DB status update failed: ${err instanceof Error ? err.message : String(err)}`);
             }
         }, 15_000);
+        progressRefreshTimer.unref?.();
         return {
             render(width) {
                 if (cachedLines && cachedWidth === width)
@@ -631,26 +675,57 @@ export function updateProgressWidget(ctx, unitType, unitId, state, accessors, ti
                     cachedWidth = width;
                     return lines;
                 }
-                // ── Mode: small — header + active work progress ───────────────
+                // ── Mode: small — dense horizontal grid ───────────────────────
                 if (widgetMode === "small") {
-                    lines.push("");
-                    // Action line
-                    const target = task ? `${task.id}: ${task.title}` : unitId;
-                    const actionLeft = `${pad}${theme.fg("accent", "▸")} ${theme.fg("accent", verb)}  ${theme.fg("text", target)}`;
-                    lines.push(rightAlign(actionLeft, theme.fg("dim", phaseLabel), width));
-                    // Progress bar
+                    lines.length = 0;
+                    lines.push(...ui.bar());
                     const roadmapSlices = mid ? getRoadmapSlicesSync() : null;
+                    const unitLabel = unitId || [mid?.id, slice?.id, task?.id].filter(Boolean).join("/");
+                    const statusParts = [
+                        spinner,
+                        theme.fg("success", modeTag),
+                        theme.fg(stateColor, activeState),
+                    ];
+                    if (runtimeSignal?.summary) {
+                        statusParts.push(theme.fg(healthColor, healthSummary));
+                    }
+                    else if (healthLevel !== "green") {
+                        statusParts.push(`${theme.fg(healthColor, healthIcon)} ${theme.fg(healthColor, healthSummary)}`);
+                    }
+                    const timeValue = [elapsed, etaShort].filter(Boolean).join(" · ") || "--";
+                    const rowOne = widgetGridColumns(theme, width, [
+                        `${widgetGridLabel(theme, "status", "border")} ${statusParts.join(" ")}`,
+                        `${widgetGridLabel(theme, "unit")} ${theme.fg("text", unitLabel || "--")}`,
+                        `${widgetGridLabel(theme, "spend", "border")} ${theme.fg("dim", formatSmallWidgetSpend())}`,
+                        `${widgetGridLabel(theme, "time")} ${theme.fg("dim", timeValue)}`,
+                    ]);
+                    const target = task
+                        ? `${task.id}: ${task.title}`
+                        : slice
+                            ? `${slice.id}: ${slice.title}`
+                            : unitId;
+                    let taskValue = task?.id ?? "--";
+                    let sliceValue = slice?.id ?? "--";
                     if (shouldRenderRoadmapProgress(roadmapSlices)) {
                         const { done, total, activeSliceTasks } = roadmapSlices;
-                        const barWidth = Math.max(6, Math.min(18, Math.floor(width * 0.25)));
+                        const barWidth = Math.max(4, Math.min(14, Math.floor(width * 0.12)));
                         const bar = renderProgressBar(theme, done, total, barWidth);
-                        let meta = `${theme.fg("text", `${done}`)}${theme.fg("dim", `/${total} slices`)}`;
+                        sliceValue = `${bar} ${theme.fg("text", `${done}/${total}`)}`;
                         if (activeSliceTasks && activeSliceTasks.total > 0) {
-                            const tn = Math.min(activeSliceTasks.done + 1, activeSliceTasks.total);
-                            meta += `${theme.fg("dim", " · task ")}${theme.fg("accent", `${tn}`)}${theme.fg("dim", `/${activeSliceTasks.total}`)}`;
+                            const taskNum = isHook
+                                ? Math.max(activeSliceTasks.done, 1)
+                                : Math.min(activeSliceTasks.done + 1, activeSliceTasks.total);
+                            taskValue = `${theme.fg("accent", `${taskNum}`)}${theme.fg("dim", `/${activeSliceTasks.total}`)}`;
                         }
-                        lines.push(`${pad}${bar} ${meta}`);
                     }
+                    const rowTwo = widgetGridColumns(theme, width, [
+                        `${widgetGridLabel(theme, "phase", "border")} ${theme.fg("dim", unitType)}`,
+                        `${widgetGridLabel(theme, "work")} ${theme.fg("text", target || "--")}`,
+                        `${widgetGridLabel(theme, "task", "border")} ${taskValue}`,
+                        `${widgetGridLabel(theme, "slice")} ${sliceValue}`,
+                    ]);
+                    lines.push(rowOne);
+                    lines.push(rowTwo);
                     lines.push(...ui.bar());
                     cachedLines = lines;
                     cachedWidth = width;
@@ -789,7 +864,7 @@ export function setCompletionProgressWidget(ctx, snapshot) {
     if (!ctx.hasUI)
         return;
     const widgetKey = "gsd-progress";
-    ctx.ui.setWidget("gsd-outcome", undefined);
+    clearAutoOutcomeWidget(ctx);
     if (typeof ctx.ui?.setHeader === "function") {
         ctx.ui.setHeader(() => ({
             render() { return []; },

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

@@ -443,6 +443,7 @@ export const DISPATCH_RULES = [
                 unitType: "discuss-milestone",
                 unitId: mid,
                 prompt: await buildDiscussMilestonePrompt(mid, midTitle, basePath, structuredQuestionsAvailable, { headless: !!process.env.GSD_HEADLESS }),
+                pauseAfterDispatch: !process.env.GSD_HEADLESS,
             };
         },
     },
@@ -578,6 +579,7 @@ export const DISPATCH_RULES = [
                 unitType: "discuss-milestone",
                 unitId: mid,
                 prompt: await buildDiscussMilestonePrompt(mid, midTitle, basePath, structuredQuestionsAvailable, { headless: !!process.env.GSD_HEADLESS }),
+                pauseAfterDispatch: !process.env.GSD_HEADLESS,
             };
         },
     },
@@ -617,6 +619,7 @@ export const DISPATCH_RULES = [
                 unitType: "discuss-project",
                 unitId: "PROJECT",
                 prompt: await buildDiscussProjectPrompt(basePath, structuredQuestionsAvailable),
+                pauseAfterDispatch: !process.env.GSD_HEADLESS,
             };
         },
     },
@@ -642,6 +645,7 @@ export const DISPATCH_RULES = [
                 unitType: "discuss-requirements",
                 unitId: "REQUIREMENTS",
                 prompt: await buildDiscussRequirementsPrompt(basePath, structuredQuestionsAvailable),
+                pauseAfterDispatch: !process.env.GSD_HEADLESS,
             };
         },
     },
@@ -757,6 +761,7 @@ export const DISPATCH_RULES = [
                 unitType: "discuss-milestone",
                 unitId: mid,
                 prompt: await buildDiscussMilestonePrompt(mid, midTitle, basePath, structuredQuestionsAvailable, { headless: !!process.env.GSD_HEADLESS }),
+                pauseAfterDispatch: !process.env.GSD_HEADLESS,
             };
         },
     },

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

@@ -32,7 +32,7 @@ import { isDbAvailable, getDbPath, refreshOpenDatabaseFromDisk, getTask, getSlic
 import { renderPlanCheckboxes, renderRoadmapFromDb } from "./markdown-renderer.js";
 import { parseRoadmap as parseLegacyRoadmap } from "./parsers-legacy.js";
 import { consumeSignal } from "./session-status-io.js";
-import { checkPostUnitHooks, isRetryPending, consumeRetryTrigger, persistHookState, resolveHookArtifactPath, } from "./post-unit-hooks.js";
+import { checkPostUnitHooks, consumeHookFailure, isRetryPending, consumeRetryTrigger, consumeGateBlock, persistHookState, resolveHookArtifactPath, } from "./post-unit-hooks.js";
 import { hasPendingCaptures, loadPendingCaptures, revertExecutorResolvedCaptures } from "./captures.js";
 import { debugLog } from "./debug-logger.js";
 import { runSafely } from "./auto-utils.js";
@@ -323,6 +323,41 @@ function stripKnownIdPrefix(value, id) {
         return raw.slice(id.length + 1).trim() || undefined;
     return raw;
 }
+function parseReactiveBatchTaskIds(unitId) {
+    const { task: batchPart } = parseUnitId(unitId);
+    if (!batchPart?.startsWith("reactive+"))
+        return [];
+    const rawIds = batchPart
+        .slice("reactive+".length)
+        .split(",")
+        .map((taskId) => taskId.trim().toUpperCase())
+        .filter(Boolean);
+    const unique = new Set();
+    for (const taskId of rawIds) {
+        unique.add(taskId);
+    }
+    return [...unique];
+}
+function dedupePaths(values) {
+    const seen = new Set();
+    const result = [];
+    for (const value of values) {
+        if (!seen.has(value)) {
+            seen.add(value);
+            result.push(value);
+        }
+    }
+    return result;
+}
+function getPlannedKeyFiles(tasks) {
+    return dedupePaths(tasks.flatMap((taskRow) => [
+        ...(taskRow.expected_output ?? []),
+        ...(taskRow.files ?? []),
+        ...(taskRow.key_files ?? []),
+    ]));
+}
+export const _parseReactiveBatchTaskIdsForTest = parseReactiveBatchTaskIds;
+export const _getPlannedKeyFilesForTest = getPlannedKeyFiles;
 function resolveVerificationFailureMarkerPath(unitType, unitId, basePath) {
     const { milestone: mid, slice: sid, task: tid } = parseUnitId(unitId);
     switch (unitType) {
@@ -402,6 +437,34 @@ async function buildTaskCommitContextForUnit(basePath, unitId) {
         issueNumber: ghIssueNumber,
     };
 }
+async function buildReactiveTaskCommitContext(_basePath, unitId) {
+    const { milestone: mid, slice: sid } = parseUnitId(unitId);
+    if (!mid || !sid || !isDbAvailable())
+        return undefined;
+    const batchTaskIds = parseReactiveBatchTaskIds(unitId);
+    if (batchTaskIds.length === 0)
+        return undefined;
+    const milestone = getMilestone(mid);
+    const slice = getSlice(mid, sid);
+    const taskRows = batchTaskIds
+        .map((tid) => getTask(mid, sid, tid))
+        .filter((taskRow) => taskRow !== null);
+    const keyFiles = getPlannedKeyFiles(taskRows);
+    if (taskRows.length === 0 || keyFiles.length === 0)
+        return undefined;
+    const taskLabel = taskRows.map((row) => row.id).join(",");
+    return {
+        taskId: `${sid}/${taskLabel}`,
+        taskDisplayId: "reactive-batch",
+        taskTitle: `Reactive batch: ${taskLabel}`,
+        milestoneId: mid,
+        milestoneTitle: stripKnownIdPrefix(milestone?.title, mid),
+        sliceId: sid,
+        sliceTitle: stripKnownIdPrefix(slice?.title, sid),
+        oneLiner: `Reactive execute for ${taskLabel}`,
+        keyFiles,
+    };
+}
 async function runPostUnitGitHubSyncIfNeeded(basePath, unit) {
     if (unit.type === "complete-milestone")
         return;
@@ -761,6 +824,9 @@ export async function autoCommitUnit(basePath, unitType, unitId, ctx) {
         if (unitType === "execute-task") {
             taskContext = await buildTaskCommitContextForUnit(basePath, unitId);
         }
+        else if (unitType === "reactive-execute") {
+            taskContext = await buildReactiveTaskCommitContext(basePath, unitId);
+        }
         _resetHasChangesCache();
         if (LIFECYCLE_ONLY_UNITS.has(unitType)) {
             return null;
@@ -812,6 +878,22 @@ async function runCloseoutGitAction(pctx, unit, opts) {
                 targetRepositories = getTask(mid, sid, tid)?.target_repositories;
             }
         }
+        else if (turnAction === "commit" && unit.type === "reactive-execute") {
+            taskContext = await buildReactiveTaskCommitContext(s.basePath, unit.id);
+            const { milestone: mid, slice: sid } = parseUnitId(unit.id);
+            if (mid && sid && isDbAvailable()) {
+                const repositories = new Set();
+                for (const tid of parseReactiveBatchTaskIds(unit.id)) {
+                    const taskRow = getTask(mid, sid, tid);
+                    for (const repoId of taskRow?.target_repositories ?? []) {
+                        repositories.add(repoId);
+                    }
+                }
+                if (repositories.size > 0) {
+                    targetRepositories = [...repositories];
+                }
+            }
+        }
         // Invalidate the nativeHasChanges cache before auto-commit (#1853).
         // The cache has a 10-second TTL and is keyed by basePath. A stale
         // `false` result causes autoCommit to skip staging entirely.
@@ -1206,12 +1288,19 @@ export async function postUnitPreVerification(pctx, opts) {
             if (safetyConfig.enabled) {
                 const { milestone: sMid, slice: sSid, task: sTid } = parseUnitId(s.currentUnit.id);
                 // File change validation (execute-task only, after unit execution)
-                if (safetyConfig.file_change_validation && s.currentUnit.type === "execute-task" && sMid && sSid && sTid && isDbAvailable()) {
+                if (safetyConfig.file_change_validation && s.currentUnit.type === "execute-task" && sMid && sSid && sTid) {
                     try {
-                        const taskRow = getTask(sMid, sSid, sTid);
-                        if (taskRow) {
-                            const expectedOutput = taskRow.expected_output ?? [];
-                            const plannedFiles = taskRow.files ?? [];
+                        const sliceTaskRows = isDbAvailable()
+                            ? getSliceTasks(sMid, sSid).filter((t) => isClosedStatus(t.status) || t.id === sTid)
+                            : [];
+                        if (sliceTaskRows.length > 0) {
+                            const expectedOutput = getPlannedKeyFiles(sliceTaskRows.map((taskRow) => ({
+                                expected_output: taskRow.expected_output,
+                                files: taskRow.files,
+                            })));
+                            const plannedFiles = getPlannedKeyFiles(sliceTaskRows.map((taskRow) => ({
+                                files: taskRow.files,
+                            })));
                             const audit = validateFileChanges(s.basePath, expectedOutput, plannedFiles, safetyConfig.file_change_allowlist);
                             if (audit && audit.violations.length > 0) {
                                 const warnings = audit.violations.filter(v => v.severity === "warning");
@@ -1223,6 +1312,23 @@ export async function postUnitPreVerification(pctx, opts) {
                                 }
                             }
                         }
+                        else {
+                            const taskRow = getTask(sMid, sSid, sTid);
+                            if (taskRow) {
+                                const expectedOutput = taskRow.expected_output ?? [];
+                                const plannedFiles = taskRow.files ?? [];
+                                const audit = validateFileChanges(s.basePath, expectedOutput, plannedFiles, safetyConfig.file_change_allowlist);
+                                if (audit && audit.violations.length > 0) {
+                                    const warnings = audit.violations.filter(v => v.severity === "warning");
+                                    for (const v of warnings) {
+                                        logWarning("safety", `file-change: ${v.file} — ${v.reason}`);
+                                    }
+                                    if (warnings.length > 0) {
+                                        ctx.ui.notify(`Safety: ${warnings.length} unexpected file change(s) outside task plan`, "warning");
+                                    }
+                                }
+                            }
+                        }
                     }
                     catch (e) {
                         debugLog("postUnit", { phase: "safety-file-change", error: String(e) });
@@ -1754,18 +1860,25 @@ export async function postUnitPostVerification(pctx) {
     // ── Post-unit hooks ──
     if (s.currentUnit && !s.stepMode) {
         const hookUnit = checkPostUnitHooks(s.currentUnit.type, s.currentUnit.id, s.basePath);
+        persistHookState(s.basePath);
         if (hookUnit) {
             if (s.currentUnit) {
                 await closeoutUnit(ctx, s.basePath, s.currentUnit.type, s.currentUnit.id, s.currentUnit.startedAt, buildSnapshotOpts(s.currentUnit.type, s.currentUnit.id));
             }
-            persistHookState(s.basePath);
             return enqueueSidecar(s, ctx, { kind: "hook", unitType: hookUnit.unitType, unitId: hookUnit.unitId, prompt: hookUnit.prompt, model: hookUnit.model }, { hookName: hookUnit.hookName });
         }
+        const hookFailure = consumeHookFailure();
+        if (hookFailure) {
+            ctx.ui.notify(`Post-unit hook ${hookFailure.hookName} failed for ${hookFailure.unitId}: ${hookFailure.reason}. Pausing auto-mode.`, "warning");
+            await pauseAuto(ctx, pi);
+            return "stopped";
+        }
         // Check if a hook requested a retry of the trigger unit
         if (isRetryPending()) {
             const trigger = consumeRetryTrigger();
             if (trigger) {
-                ctx.ui.notify(`Hook requested retry of ${trigger.unitType} ${trigger.unitId} — resetting task state.`, "info");
+                persistHookState(s.basePath);
+                ctx.ui.notify(`Hook requested retry of ${trigger.unitType} ${trigger.unitId} — resetting trigger unit state.`, "info");
                 await s.orchestration?.retryActiveUnit({
                     unitType: trigger.unitType,
                     unitId: trigger.unitId,
@@ -1812,6 +1925,17 @@ export async function postUnitPostVerification(pctx) {
                 // Fall through to normal dispatch — deriveState will re-derive the unit
             }
         }
+        const gateBlock = consumeGateBlock();
+        if (gateBlock) {
+            persistHookState(s.basePath);
+            const verdict = gateBlock.verdict ? ` verdict=${gateBlock.verdict};` : "";
+            const artifact = gateBlock.artifact ? ` artifact=${gateBlock.artifact};` : "";
+            const message = `Post-unit gate "${gateBlock.hookName}" blocked ${gateBlock.triggerUnitType} ${gateBlock.triggerUnitId}:` +
+                `${verdict}${artifact} ${gateBlock.reason}. Run /gsd status to inspect, then /gsd auto after recovery.`;
+            ctx.ui.notify(message, "warning");
+            await pauseAuto(ctx, pi);
+            return "stopped";
+        }
     }
     // ── Fast-path stop detection (#3487) ──
     // Before waiting for triage, check if any PENDING captures contain explicit

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

@@ -30,6 +30,7 @@ import { classifyProject } from "./detection.js";
 import { hasBrowserRequiredText } from "./browser-evidence.js";
 import { debugLog } from "./debug-logger.js";
 import { buildSkillActivationBlock, buildSkillDiscoveryVars } from "./skill-activation.js";
+import { findMilestoneIds } from "./milestone-ids.js";
 export { buildSkillActivationBlock, buildSkillDiscoveryVars };
 // ─── Preamble Cap ─────────────────────────────────────────────────────────────
 /**
@@ -1270,7 +1271,7 @@ export async function checkNeedsRunUat(base, mid, state, prefs) {
                     if (hasVerdict(uatContent))
                         continue;
                     // Also check the ASSESSMENT file — the run-uat prompt writes the verdict
-                    // there (via gsd_summary_save artifact_type:"ASSESSMENT"), not into the
+                    // there (via gsd_uat_result_save), not into the
                     // UAT spec file. Without this check the unit re-dispatches indefinitely.
                     const assessmentFile = resolveSliceFile(base, mid, sid, "ASSESSMENT");
                     if (assessmentFile) {
@@ -1325,21 +1326,44 @@ export async function checkNeedsRunUat(base, mid, state, prefs) {
     }
     return null;
 }
-// ─── Prompt Builders ──────────────────────────────────────────────────────
-/**
- * Build a prompt for the discuss-milestone unit type.
- * Loads the guided-discuss-milestone template and inlines the CONTEXT-DRAFT
- * as a seed when present. The discussion agent interviews the user, writes
- * a full CONTEXT.md, and the phase transitions to pre-planning automatically.
- */
-export async function buildDiscussMilestonePrompt(mid, midTitle, base, structuredQuestionsAvailable = "false", { headless = false } = {}) {
-    const discussTemplates = inlineTemplate("context", "Context");
+export async function buildDiscussMilestoneInlinedContext(mid, base) {
+    const inlined = [];
+    const roadmapInline = await inlineFileOptional(resolveMilestoneFile(base, mid, "ROADMAP"), relMilestoneFile(base, mid, "ROADMAP"), "Milestone Roadmap");
+    if (roadmapInline)
+        inlined.push(roadmapInline);
+    const contextInline = await inlineFileOptional(resolveMilestoneFile(base, mid, "CONTEXT"), relMilestoneFile(base, mid, "CONTEXT"), "Milestone Context");
+    if (contextInline)
+        inlined.push(contextInline);
+    const researchInline = await inlineFileOptional(resolveMilestoneFile(base, mid, "RESEARCH"), relMilestoneFile(base, mid, "RESEARCH"), "Milestone Research");
+    if (researchInline)
+        inlined.push(researchInline);
+    const decisionsPath = resolveGsdRootFile(base, "DECISIONS");
+    if (existsSync(decisionsPath)) {
+        const decisionsContent = await loadFile(decisionsPath);
+        if (decisionsContent) {
+            inlined.push(`### Decisions Register\nSource: \`${relGsdRootFile("DECISIONS")}\`\n\n${decisionsContent.trim()}`);
+        }
+    }
+    const milestoneIds = findMilestoneIds(base);
+    const currentIndex = milestoneIds.indexOf(mid);
+    const priorMilestoneIds = currentIndex >= 0 ? milestoneIds.slice(0, currentIndex) : milestoneIds;
+    for (const priorMid of priorMilestoneIds) {
+        const summaryInline = await inlineFileOptional(resolveMilestoneFile(base, priorMid, "SUMMARY"), relMilestoneFile(base, priorMid, "SUMMARY"), `${priorMid} Prior Milestone Summary`);
+        if (summaryInline)
+            inlined.push(summaryInline);
+    }
+    return inlined.length > 0
+        ? `## Inlined Context (preloaded — do not re-read these files)\n\n${inlined.join("\n\n---\n\n")}`
+        : "## Inlined Context\n\n_(no milestone context files found yet — go in blind and ask broad questions)_";
+}
+export async function buildDiscussMilestonePrompt(mid, midTitle, base, structuredQuestionsAvailable = "false", { headless = false, commitInstruction = "Do not commit planning artifacts — .gsd/ is managed externally.", fastPathInstruction = "", includeDraftSeed = true, includeContextMode = true, } = {}) {
+    const contextTemplate = inlineTemplate("context", "Context");
     if (headless) {
         const roadmapPath = resolveMilestoneFile(base, mid, "ROADMAP");
         const roadmapContent = roadmapPath ? await loadFile(roadmapPath) : null;
         return loadPrompt("discuss-headless", {
             seedContext: roadmapContent ?? "",
-            inlinedTemplates: discussTemplates,
+            inlinedTemplates: contextTemplate,
             workingDirectory: base,
             milestoneId: mid,
             contextPath: relMilestoneFile(base, mid, "CONTEXT"),
@@ -1347,24 +1371,28 @@ export async function buildDiscussMilestonePrompt(mid, midTitle, base, structure
             multiMilestoneCommitInstruction: "Do not commit planning artifacts — .gsd/ is managed externally.",
         });
     }
-    const contextModeInstructions = renderContextModeForPrompt("discuss-milestone", base);
+    const rawInlinedContext = await buildDiscussMilestoneInlinedContext(mid, base);
+    const cappedInlinedContext = capPreamble(rawInlinedContext);
+    const discussTemplates = [cappedInlinedContext, contextTemplate].join("\n\n---\n\n");
     const basePrompt = loadPrompt("guided-discuss-milestone", {
         workingDirectory: base,
         milestoneId: mid,
         milestoneTitle: midTitle,
         inlinedTemplates: discussTemplates,
         structuredQuestionsAvailable,
-        commitInstruction: "Do not commit planning artifacts — .gsd/ is managed externally.",
-        fastPathInstruction: "",
+        commitInstruction,
+        fastPathInstruction,
     });
-    const promptWithContextMode = prependContextModeToBlock("discuss-milestone", base, basePrompt);
+    const promptWithContextMode = includeContextMode
+        ? prependContextModeToBlock("discuss-milestone", base, basePrompt)
+        : basePrompt;
     // If a CONTEXT-DRAFT.md exists, append it as seed material
     const draftPath = resolveMilestoneFile(base, mid, "CONTEXT-DRAFT");
     const draftContent = draftPath ? await loadFile(draftPath) : null;
-    if (draftContent) {
+    if (includeDraftSeed && draftContent) {
         return `${promptWithContextMode}\n\n## Prior Discussion (Draft Seed)\n\nThe following draft was captured from a prior multi-milestone discussion. Use it as seed material — the user has already provided this context. Start with a brief reflection on what the draft covers, then probe for any gaps or open questions before writing the full CONTEXT.md.\n\n${draftContent}`;
     }
-    return contextModeInstructions ? promptWithContextMode : basePrompt;
+    return promptWithContextMode;
 }
 /**
  * Build a prompt for the workflow-preferences unit type (deep mode).
@@ -2351,6 +2379,15 @@ export async function buildCompleteSlicePrompt(mid, midTitle, sid, sTitle, base,
         sliceSummaryPath,
         sliceUatPath,
         gatesToClose,
+        skillActivation: buildSkillActivationBlock({
+            base,
+            milestoneId: mid,
+            milestoneTitle: midTitle,
+            sliceId: sid,
+            sliceTitle: sTitle,
+            extraContext: [inlinedContext],
+            unitType: "complete-slice",
+        }),
     });
 }
 export async function buildCompleteMilestonePrompt(mid, midTitle, base, level) {
@@ -2531,17 +2568,26 @@ export async function buildValidateMilestonePrompt(mid, midTitle, base, level) {
         if (isDbAvailable()) {
             const milestone = getMilestone(mid);
             if (milestone) {
+                const escapeCell = (value) => value.replace(/[\\|]/g, (char) => `\\${char}`).replace(/\r?\n/g, " ");
                 const classes = [];
                 if (milestone.verification_contract)
-                    classes.push(`- **Contract:** ${milestone.verification_contract}`);
+                    classes.push(`| Contract | ${escapeCell(milestone.verification_contract)} |`);
                 if (milestone.verification_integration)
-                    classes.push(`- **Integration:** ${milestone.verification_integration}`);
+                    classes.push(`| Integration | ${escapeCell(milestone.verification_integration)} |`);
                 if (milestone.verification_operational)
-                    classes.push(`- **Operational:** ${milestone.verification_operational}`);
+                    classes.push(`| Operational | ${escapeCell(milestone.verification_operational)} |`);
                 if (milestone.verification_uat)
-                    classes.push(`- **UAT:** ${milestone.verification_uat}`);
+                    classes.push(`| UAT | ${escapeCell(milestone.verification_uat)} |`);
                 if (classes.length > 0) {
-                    const verificationClasses = `### Verification Classes (from planning)\n\nThese verification tiers were defined during milestone planning. Each non-empty class must be checked for evidence during validation.\n\n${classes.join("\n")}`;
+                    const verificationClasses = [
+                        "### Verification Classes (from planning)",
+                        "",
+                        "These verification tiers were defined during milestone planning. Every row in this table must appear in `verificationClasses` with the same canonical class name.",
+                        "",
+                        "| Class | Planned Check |",
+                        "| --- | --- |",
+                        ...classes,
+                    ].join("\n");
                     inlined.push(verificationClasses);
                     trackPromptContext(contextTelemetry, "verification-classes", "inline", verificationClasses);
                 }