@opengsd/gsd-pi 1.9.0 → 1.10.0-dev.a5e42e99

package/dist/resources/extensions/gsd/tools/complete-task.js

@@ -15,7 +15,7 @@ import { isClosedStatus } from "../status-guards.js";
 import { transaction, insertMilestone, insertSlice, insertTask, insertVerificationEvidence, getMilestone, getSlice, getTask, updateTaskStatus, deleteVerificationEvidence, saveGateResult, getPendingGatesForTurn, getUnresolvedBlockingReworkFindingsForTask, getBlockingReworkFindingsForTask, applyReworkResolutions, } from "../gsd-db.js";
 import { getWorkflowDatabasePath, ensureWorkflowDbAtPath } from "../db-workspace.js";
 import { getGatesForTurn } from "../gate-registry.js";
-import { gsdProjectionRoot, clearPathCache, resolveMilestonePath, resolveSlicePath } from "../paths.js";
+import { buildFlatTaskFileName, buildTaskFileName, gsdProjectionRoot, clearPathCache, legacyMilestonesDir, resolveMilestonePath, resolveSlicePath, } from "../paths.js";
 import { resolveCanonicalMilestoneRoot } from "../worktree-manager.js";
 import { checkOwnership, taskUnitKey } from "../unit-ownership.js";
 import { saveFile, clearParseCache } from "../files.js";
@@ -34,13 +34,20 @@ function taskSummaryPath(basePath, milestoneId, sliceId, taskId) {
     // a legacy layout and breaks all subsequent path resolution for the session.
     const slicePath = resolveSlicePath(basePath, milestoneId, sliceId);
     const phaseDir = resolveMilestonePath(basePath, milestoneId);
-    if (slicePath && phaseDir && slicePath !== phaseDir) {
-        // Legacy layout: the slice has its own slices/SID/ subdir → tasks/ subdir
-        return join(slicePath, "tasks", `${taskId}-SUMMARY.md`);
+    const legacyBase = legacyMilestonesDir(basePath);
+    const isLegacy = phaseDir
+        ? phaseDir.startsWith(legacyBase + "/") || phaseDir.startsWith(legacyBase + "\\")
+        : false;
+    if (isLegacy && phaseDir) {
+        // Legacy layout: the slice has its own slices/SID/ subdir → tasks/ subdir.
+        const legacySlicePath = slicePath && slicePath !== phaseDir
+            ? slicePath
+            : join(phaseDir, "slices", sliceId);
+        return join(legacySlicePath, "tasks", buildTaskFileName(taskId, "SUMMARY"));
     }
     if (phaseDir) {
         // Flat-phase: task summaries go in the phase dir (no tasks/ subdir)
-        return join(phaseDir, `${taskId}-SUMMARY.md`);
+        return join(phaseDir, buildFlatTaskFileName(sliceId, taskId, "SUMMARY"));
     }
     // Fallback: legacy hardcoded path (milestone/slice dir not on disk yet)
     return join(gsdProjectionRoot(basePath), "milestones", milestoneId, "slices", sliceId, "tasks", `${taskId}-SUMMARY.md`);

package/dist/resources/extensions/gsd/tools/plan-slice.js

@@ -6,7 +6,7 @@ import { isNonEmptyString, validateStringArray } from "../validation.js";
 import { getGateIdsForTurn } from "../gate-registry.js";
 import { transaction, getMilestone, getSlice, getSliceTasks, insertTask, upsertSlicePlanning, upsertTaskPlanning, insertGateRow, updateSliceStatus, setSliceSketchFlag, deleteTask, deleteArtifactByPath, } from "../gsd-db.js";
 import { invalidateStateCache } from "../state.js";
-import { renderPlanFromDb } from "../markdown-renderer.js";
+import { renderPlanCheckboxes, renderPlanFromDb } from "../markdown-renderer.js";
 import { writeManifest } from "../workflow-manifest.js";
 import { appendEvent } from "../workflow-events.js";
 import { logWarning } from "../workflow-logger.js";
@@ -371,9 +371,13 @@ export async function handlePlanSlice(rawParams, basePath) {
             deleteArtifactByPath(artifactPath);
         }
         const sliceTasks = getSliceTasks(params.milestoneId, params.sliceId);
+        const hasClosedTasks = sliceTasks.some((task) => isClosedStatus(task.status));
         const renderResult = sliceTasks.length === 0
             ? { planPath: "", taskPlanPaths: [] }
             : await renderPlanFromDb(basePath, params.milestoneId, params.sliceId);
+        if (sliceTasks.length > 0 && hasClosedTasks) {
+            await renderPlanCheckboxes(basePath, params.milestoneId, params.sliceId);
+        }
         if (sliceTasks.length > 0) {
             setSliceSketchFlag(params.milestoneId, params.sliceId, false);
         }

package/dist/resources/extensions/gsd/tools/reopen-milestone.js

@@ -16,7 +16,7 @@ import { logWarning } from "../workflow-logger.js";
 import { debugLog } from "../debug-logger.js";
 import { existsSync, unlinkSync } from "node:fs";
 import { join } from "node:path";
-import { resolveMilestonePath, resolveSlicePath, resolveTasksDir, clearPathCache } from "../paths.js";
+import { buildFlatTaskFileName, buildTaskFileName, legacyMilestonesDir, resolveMilestonePath, resolveSlicePath, resolveTasksDir, clearPathCache, } from "../paths.js";
 export async function handleReopenMilestone(params, basePath) {
     // ── Validate required fields ────────────────────────────────────────────
     if (!params.milestoneId || typeof params.milestoneId !== "string" || params.milestoneId.trim() === "") {
@@ -41,6 +41,8 @@ export async function handleReopenMilestone(params, basePath) {
     // auto-corrects entities back to "complete", making reopen a no-op (#3161).
     try {
         const milestoneDir = resolveMilestonePath(basePath, params.milestoneId);
+        const legacyBase = legacyMilestonesDir(basePath);
+        const isLegacy = !!milestoneDir && (milestoneDir.startsWith(legacyBase + "/") || milestoneDir.startsWith(legacyBase + "\\"));
         if (milestoneDir) {
             const milestoneSummary = join(milestoneDir, `${params.milestoneId}-SUMMARY.md`);
             if (existsSync(milestoneSummary))
@@ -58,10 +60,17 @@ export async function handleReopenMilestone(params, basePath) {
                     unlinkSync(sliceUat);
             }
             const tasksDir = resolveTasksDir(basePath, params.milestoneId, slice.id);
-            if (tasksDir) {
-                const tasks = getSliceTasks(params.milestoneId, slice.id);
-                for (const task of tasks) {
-                    const taskSummary = join(tasksDir, `${task.id}-SUMMARY.md`);
+            const tasks = getSliceTasks(params.milestoneId, slice.id);
+            for (const task of tasks) {
+                const taskSummaries = isLegacy
+                    ? (tasksDir ? [join(tasksDir, buildTaskFileName(task.id, "SUMMARY"))] : [])
+                    : milestoneDir
+                        ? [
+                            join(milestoneDir, buildFlatTaskFileName(slice.id, task.id, "SUMMARY")),
+                            join(milestoneDir, buildTaskFileName(task.id, "SUMMARY")),
+                        ]
+                        : [];
+                for (const taskSummary of taskSummaries) {
                     if (existsSync(taskSummary))
                         unlinkSync(taskSummary);
                 }

package/dist/resources/extensions/gsd/tools/reopen-slice.js

@@ -22,7 +22,7 @@ import { appendEvent } from "../workflow-events.js";
 import { logWarning } from "../workflow-logger.js";
 import { existsSync, unlinkSync } from "node:fs";
 import { join } from "node:path";
-import { resolveTasksDir, resolveSlicePath, clearPathCache } from "../paths.js";
+import { buildFlatTaskFileName, buildTaskFileName, legacyMilestonesDir, resolveMilestonePath, resolveTasksDir, resolveSlicePath, clearPathCache, } from "../paths.js";
 export async function handleReopenSlice(params, basePath) {
     // ── Validate required fields ────────────────────────────────────────────
     if (!params.sliceId || typeof params.sliceId !== "string" || params.sliceId.trim() === "") {
@@ -52,11 +52,21 @@ export async function handleReopenSlice(params, basePath) {
     // Without this, the DB-filesystem reconciler sees SUMMARY.md files and
     // auto-corrects tasks back to "complete", making reopen a no-op (#3161).
     try {
+        const milestoneDir = resolveMilestonePath(basePath, params.milestoneId);
+        const legacyBase = legacyMilestonesDir(basePath);
+        const isLegacy = !!milestoneDir && (milestoneDir.startsWith(legacyBase + "/") || milestoneDir.startsWith(legacyBase + "\\"));
         const tasksDir = resolveTasksDir(basePath, params.milestoneId, params.sliceId);
-        if (tasksDir) {
-            const tasks = getSliceTasks(params.milestoneId, params.sliceId);
-            for (const task of tasks) {
-                const summaryPath = join(tasksDir, `${task.id}-SUMMARY.md`);
+        const tasks = getSliceTasks(params.milestoneId, params.sliceId);
+        for (const task of tasks) {
+            const summaryPaths = isLegacy
+                ? (tasksDir ? [join(tasksDir, buildTaskFileName(task.id, "SUMMARY"))] : [])
+                : milestoneDir
+                    ? [
+                        join(milestoneDir, buildFlatTaskFileName(params.sliceId, task.id, "SUMMARY")),
+                        join(milestoneDir, buildTaskFileName(task.id, "SUMMARY")),
+                    ]
+                    : [];
+            for (const summaryPath of summaryPaths) {
                 if (existsSync(summaryPath))
                     unlinkSync(summaryPath);
             }

package/dist/resources/extensions/gsd/tools/reopen-task.js

@@ -17,8 +17,8 @@ import { appendEvent } from "../workflow-events.js";
 import { logWarning } from "../workflow-logger.js";
 import { writeReopenReason } from "../reopen-reason.js";
 import { existsSync, unlinkSync } from "node:fs";
-import { join, basename, dirname } from "node:path";
-import { resolveTasksDir, resolveSlicePath, clearPathCache } from "../paths.js";
+import { join } from "node:path";
+import { buildFlatTaskFileName, buildTaskFileName, legacyMilestonesDir, resolveMilestonePath, resolveTasksDir, resolveSlicePath, clearPathCache, } from "../paths.js";
 export async function handleReopenTask(params, basePath) {
     // ── Validate required fields ────────────────────────────────────────────
     if (!params.taskId || typeof params.taskId !== "string" || params.taskId.trim() === "") {
@@ -76,14 +76,20 @@ export async function handleReopenTask(params, basePath) {
     // summary cleanup into tasks/ (#1208).
     try {
         const slicePath = resolveSlicePath(basePath, params.milestoneId, params.sliceId);
-        if (slicePath) {
-            const isLegacySlice = basename(dirname(slicePath)) === "slices";
-            const summaryDir = isLegacySlice
-                ? (resolveTasksDir(basePath, params.milestoneId, params.sliceId) ?? slicePath)
-                : slicePath;
-            const summaryPath = join(summaryDir, `${params.taskId}-SUMMARY.md`);
-            if (existsSync(summaryPath))
-                unlinkSync(summaryPath);
+        const milestonePath = resolveMilestonePath(basePath, params.milestoneId);
+        if (milestonePath) {
+            const legacyBase = legacyMilestonesDir(basePath);
+            const isLegacy = milestonePath.startsWith(legacyBase + "/") || milestonePath.startsWith(legacyBase + "\\");
+            const summaryPaths = isLegacy
+                ? [join(resolveTasksDir(basePath, params.milestoneId, params.sliceId) ?? slicePath ?? join(milestonePath, "slices", params.sliceId, "tasks"), buildTaskFileName(params.taskId, "SUMMARY"))]
+                : [
+                    join(milestonePath, buildFlatTaskFileName(params.sliceId, params.taskId, "SUMMARY")),
+                    join(milestonePath, buildTaskFileName(params.taskId, "SUMMARY")),
+                ];
+            for (const summaryPath of summaryPaths) {
+                if (existsSync(summaryPath))
+                    unlinkSync(summaryPath);
+            }
         }
     }
     catch (cleanupErr) {

package/dist/resources/extensions/gsd/unit-context-composer.js

@@ -36,6 +36,7 @@
 // owned by the unit, not in the composer. Reviews must enforce this — it
 // is the difference between an orchestrator and a runaway DSL.
 import { resolveManifest, } from "./unit-context-manifest.js";
+import { resolveEffectivePlanningToolsPolicy } from "./planning-subagent-policy.js";
 import { getUnitToolSurfaceContract } from "./unit-tool-contracts.js";
 /**
  * Produce the inlined-context portion of a unit's system prompt by
@@ -148,11 +149,32 @@ const TOOL_SURFACE_GUIDANCE_BY_UNIT = {
     "validate-milestone": "Dispatch reviewer subagents in parallel, then persist the verdict via `gsd_validate_milestone`. Do not query `.gsd/gsd.db` directly — use `gsd_milestone_status` and inlined context.",
     "complete-milestone": "Persist completion only through `gsd_complete_milestone` after verification passes. Do not query `.gsd/gsd.db` directly. Do not write `.gsd/PROJECT.md` or `.gsd/REQUIREMENTS.md` by hand — use `gsd_summary_save` and `gsd_requirement_update`.",
     "replan-slice": "Persist replans through `gsd_replan_slice` only. Do not edit `PLAN.md` or task plans directly.",
-    "plan-slice": "Persist planning through `gsd_plan_slice` only. Dispatch subagents only to **scout** or **planner** for reconnaissance — not implementation agents. Do not edit user source files outside `.gsd/**`.",
-    "refine-slice": "Persist refinements through `gsd_plan_slice` only. Dispatch subagents only to **scout** or **planner**. Do not edit user source files outside `.gsd/**`.",
-    "plan-milestone": "Persist milestone planning through `gsd_plan_milestone` / `gsd_plan_slice`. Do not edit user source files outside `.gsd/**`.",
     "research-slice": "Dispatch subagents only to **scout** or **planner** for reconnaissance. Do not edit user source files outside `.gsd/**`.",
 };
+function formatAllowedAgents(agents) {
+    return agents.map((agent) => `**${agent}**`).join(", ");
+}
+function guidanceForUnitToolsPolicy(unitType, policy) {
+    if (unitType === "plan-slice") {
+        const dispatch = policy.mode === "planning-dispatch"
+            ? ` Dispatch subagents only to ${formatAllowedAgents(policy.allowedSubagents)} for reconnaissance — not implementation agents.`
+            : " Do not dispatch subagents.";
+        return `Persist planning through \`gsd_plan_slice\` only.${dispatch} Do not edit user source files outside \`.gsd/**\`.`;
+    }
+    if (unitType === "refine-slice") {
+        const dispatch = policy.mode === "planning-dispatch"
+            ? ` Dispatch subagents only to ${formatAllowedAgents(policy.allowedSubagents)}.`
+            : " Do not dispatch subagents.";
+        return `Persist refinements through \`gsd_plan_slice\` only.${dispatch} Do not edit user source files outside \`.gsd/**\`.`;
+    }
+    if (unitType === "plan-milestone") {
+        const dispatch = policy.mode === "planning-dispatch"
+            ? ` Dispatch subagents only to ${formatAllowedAgents(policy.allowedSubagents)}.`
+            : "";
+        return `Persist milestone planning through \`gsd_plan_milestone\` / \`gsd_plan_slice\`.${dispatch} Do not edit user source files outside \`.gsd/**\`.`;
+    }
+    return undefined;
+}
 function guidanceForToolsPolicy(policy) {
     switch (policy.mode) {
         case "planning":
@@ -191,8 +213,9 @@ export function composeToolSurfaceInstructions(unitType, opts) {
     const manifest = resolveManifest(unitType);
     if (!manifest)
         return "";
-    const unitGuidance = TOOL_SURFACE_GUIDANCE_BY_UNIT[unitType];
-    const policyGuidance = unitGuidance ? null : guidanceForToolsPolicy(manifest.tools);
+    const effectiveTools = resolveEffectivePlanningToolsPolicy(unitType, manifest.tools, opts.basePath) ?? manifest.tools;
+    const unitGuidance = guidanceForUnitToolsPolicy(unitType, effectiveTools) ?? TOOL_SURFACE_GUIDANCE_BY_UNIT[unitType];
+    const policyGuidance = unitGuidance ? null : guidanceForToolsPolicy(effectiveTools);
     const forbiddenLine = formatForbiddenWorkflowToolsLine(unitType, unitGuidance);
     const parts = [unitGuidance, policyGuidance, forbiddenLine].filter((part) => typeof part === "string" && part.length > 0);
     if (parts.length === 0)

package/dist/resources/extensions/gsd/verification-evidence.js

@@ -11,13 +11,14 @@
 import { mkdirSync, writeFileSync } from "node:fs";
 import { join } from "node:path";
 /**
- * Write a T##-VERIFY.json artifact to the tasks directory.
+ * Write a T##-VERIFY.json artifact to the evidence directory.
  * Creates the directory with mkdirSync({ recursive: true }) if it doesn't exist.
+ * Flat-phase callers can pass sliceId to write S##-T##-VERIFY.json.
  *
  * stdout/stderr are excluded from the JSON — the full output lives in VerificationResult
  * in memory and is logged to stderr during the gate run.
  */
-export function writeVerificationJSON(result, tasksDir, taskId, unitId, retryAttempt, maxRetries) {
+export function writeVerificationJSON(result, tasksDir, taskId, unitId, retryAttempt, maxRetries, sliceId) {
     mkdirSync(tasksDir, { recursive: true });
     const evidence = {
         schemaVersion: 1,
@@ -52,7 +53,8 @@ export function writeVerificationJSON(result, tasksDir, taskId, unitId, retryAtt
             fixAvailable: w.fixAvailable,
         }));
     }
-    const filePath = join(tasksDir, `${taskId}-VERIFY.json`);
+    const fileName = sliceId ? `${sliceId}-${taskId}-VERIFY.json` : `${taskId}-VERIFY.json`;
+    const filePath = join(tasksDir, fileName);
     writeFileSync(filePath, JSON.stringify(evidence, null, 2) + "\n", "utf-8");
 }
 /**

package/dist/resources/extensions/gsd/workflow-projections.js

@@ -13,7 +13,7 @@ import { isClosedStatus } from "./status-guards.js";
 import { deriveState } from "./state.js";
 import { renderPlanFromDb, renderRoadmapFromDb } from "./markdown-renderer.js";
 import { readManifest } from "./workflow-manifest.js";
-import { gsdRoot, resolveMilestoneFile, resolveSliceFile, resolveTaskFile, relTaskFile } from "./paths.js";
+import { gsdRoot, resolveMilestoneFile, resolveSliceFile, resolveTaskFile, targetTaskFile } from "./paths.js";
 // ─── Helpers ─────────────────────────────────────────────────────────────
 /**
  * Strip a leading ID prefix (e.g. "M001: " or "S04: ") from a title
@@ -276,8 +276,7 @@ export function renderSummaryProjection(basePath, milestoneId, sliceId, taskId)
         return;
     const evidenceRows = getVerificationEvidence(milestoneId, sliceId, taskId);
     const content = renderSummaryContent(taskRow, sliceId, milestoneId, evidenceRows);
-    const summaryPath = resolveTaskFile(basePath, milestoneId, sliceId, taskId, "SUMMARY")
-        ?? join(basePath, relTaskFile(basePath, milestoneId, sliceId, taskId, "SUMMARY", getMilestone(milestoneId)?.title));
+    const summaryPath = targetTaskFile(basePath, milestoneId, sliceId, taskId, "SUMMARY", getMilestone(milestoneId)?.title);
     mkdirSync(dirname(summaryPath), { recursive: true });
     atomicWriteSync(summaryPath, content);
 }

package/dist/resources/extensions/gsd/workspace.js

@@ -2,6 +2,7 @@
 import { dirname, join, resolve } from "node:path";
 import { resolveGsdPathContract, resolveMilestoneFile, resolveMilestonePath, targetMilestoneFile, normalizeRealPath, } from "./paths.js";
 import { isGsdWorktreePath, resolveWorktreeProjectRoot } from "./worktree-root.js";
+import { milestoneMetaPath } from "./git-service.js";
 function tryRealpath(p) {
     return normalizeRealPath(p);
 }
@@ -63,7 +64,7 @@ export function scopeMilestone(workspace, milestoneId) {
         stateFile: () => join(gsd, "STATE.md"),
         dbPath: () => contract.projectDb,
         milestoneDir: () => scopeMilestoneDir(basePath, milestoneId),
-        metaJson: () => join(gsd, `${milestoneId}-META.json`),
+        metaJson: () => milestoneMetaPath(basePath, milestoneId),
     });
     return scope;
 }

package/dist/resources/extensions/gsd/worktree-state-projection.js

@@ -21,7 +21,7 @@
 import { cpSync, existsSync, mkdirSync, readdirSync, readFileSync, realpathSync, statSync, unlinkSync, } from "node:fs";
 import { join } from "node:path";
 import { reconcileWorktreeDb } from "./gsd-db.js";
-import { resolveGsdPathContract } from "./paths.js";
+import { dirIsContentBearingLegacyMilestone, resolveGsdPathContract } from "./paths.js";
 import { safeCopy, safeCopyRecursive } from "./safe-fs.js";
 import { logError, logWarning } from "./workflow-logger.js";
 // ─── Private helpers ──────────────────────────────────────────────────────
@@ -129,6 +129,8 @@ function syncOtherMilestoneArtifacts(srcMilestonesDir, dstMilestonesDir, current
             if (milestoneEntry.name === currentMilestoneId)
                 continue;
             const srcMilestoneDir = join(srcMilestonesDir, milestoneEntry.name);
+            if (!dirIsContentBearingLegacyMilestone(srcMilestoneDir))
+                continue;
             const dstMilestoneDir = join(dstMilestonesDir, milestoneEntry.name);
             // Additively project the entire milestone subtree (force:false), not just
             // top-level files. Prior completed milestones keep their per-slice and
@@ -219,25 +221,25 @@ export function _projectRootToWorktreeImpl(projectRoot, worktreePath_, milestone
     // Without this, worktree-local files (e.g. VALIDATION.md written
     // by validate-milestone) get clobbered by stale project root copies,
     // causing an infinite re-validation loop (#1886).
-    safeCopyRecursive(join(prGsd, "milestones", milestoneId), join(wtGsd, "milestones", milestoneId), { force: false });
-    // Additively project the full subtree of every OTHER milestone so
-    // worktree-bound units can read prior-milestone context artifacts and so
-    // completed milestones retain their per-slice/per-task SUMMARY.md and
-    // UAT.md on the worktree filesystem (otherwise the stale-render detector
-    // flags them as "complete in DB but missing on disk").
-    syncOtherMilestoneArtifacts(join(prGsd, "milestones"), join(wtGsd, "milestones"), milestoneId);
-    // Force-sync ASSESSMENT files that have a verdict from project root (#2821).
-    // The additive-only copy above preserves worktree-local files, but
-    // ASSESSMENT files are special: after run-uat writes a verdict and post-unit
-    // syncs it to the project root, the worktree may retain a stale copy (e.g.
-    // verdict:fail while the project root has verdict:pass from a retry). On
-    // session resume the DB is rebuilt from disk, and if the stale ASSESSMENT
-    // persists, checkNeedsRunUat finds no passing verdict → re-dispatches
-    // run-uat indefinitely (stuck-loop ×9).
     const prMilestoneDir = join(prGsd, "milestones", milestoneId);
     const wtMilestoneDir = join(wtGsd, "milestones", milestoneId);
-    forceOverwriteValidationWithVerdict(prMilestoneDir, wtMilestoneDir, milestoneId);
-    forceOverwriteAssessmentsWithVerdict(prMilestoneDir, wtMilestoneDir);
+    if (dirIsContentBearingLegacyMilestone(prMilestoneDir)) {
+        safeCopyRecursive(prMilestoneDir, wtMilestoneDir, { force: false });
+        // Force-sync ASSESSMENT files that have a verdict from project root (#2821).
+        // The additive-only copy above preserves worktree-local files, but
+        // ASSESSMENT files are special: after run-uat writes a verdict and post-unit
+        // syncs it to the project root, the worktree may retain a stale copy (e.g.
+        // verdict:fail while the project root has verdict:pass from a retry). On
+        // session resume the DB is rebuilt from disk, and if the stale ASSESSMENT
+        // persists, checkNeedsRunUat finds no passing verdict → re-dispatches
+        // run-uat indefinitely (stuck-loop ×9).
+        forceOverwriteValidationWithVerdict(prMilestoneDir, wtMilestoneDir, milestoneId);
+        forceOverwriteAssessmentsWithVerdict(prMilestoneDir, wtMilestoneDir);
+    }
+    // Additively project the full subtree of every OTHER content-bearing legacy
+    // milestone so worktree-bound units can read prior-milestone context artifacts
+    // without recreating empty/meta-only milestones/ scaffolds in flat-phase projects.
+    syncOtherMilestoneArtifacts(join(prGsd, "milestones"), join(wtGsd, "milestones"), milestoneId);
     // Forward-sync completed-units.json from project root to worktree.
     // Project root is authoritative for completion state after crash recovery;
     // without this, the worktree re-dispatches already-completed units (#1886).

package/dist/resources/extensions/ollama/ollama-chat-provider.js

@@ -160,7 +160,7 @@ export function streamOllamaChat(model, context, options) {
  * - `"low" | "medium" | "high"` are passed through as strings; gpt-oss honors
  *   them as strength levels, other thinking models treat any present value
  *   as "thinking enabled".
- * - `"xhigh"` is collapsed to `"high"` (ollama caps strength at high).
+ * - `"xhigh"` and `"max"` are collapsed to `"high"` (ollama caps strength at high).
  */
 export function buildThinkParam(model, options) {
     if (!model.reasoning)
@@ -170,7 +170,7 @@ export function buildThinkParam(model, options) {
         return undefined;
     if (level === "minimal")
         return false;
-    if (level === "xhigh")
+    if (level === "xhigh" || level === "max")
         return "high";
     return level;
 }

package/dist/resources/extensions/shared/interview-ui.js

@@ -144,6 +144,10 @@ export async function showInterviewRound(questions, opts, ctx) {
         let previewViewport = 0;
         let previewTotal = 0;
         function resetPreviewScroll() { previewScroll = 0; }
+        let optionsScroll = 0;
+        let optionsViewport = 0;
+        let optionsTotal = 0;
+        function resetOptionsScroll() { optionsScroll = 0; }
         function finish(result) {
             if (completed)
                 return;
@@ -203,6 +207,7 @@ export async function showInterviewRound(questions, opts, ctx) {
             saveEditorToState();
             currentIdx = newIdx;
             resetPreviewScroll();
+            resetOptionsScroll();
             loadStateToEditor();
             focusNotes = states[currentIdx].notesVisible && states[currentIdx].notes.length > 0;
             refresh();
@@ -376,32 +381,49 @@ export async function showInterviewRound(questions, opts, ctx) {
                     return;
                 }
             }
-            // ── Preview scrolling ────────────────────────────────────────
+            // ── Side-by-side panel scrolling ─────────────────────────────
             if (matchesKey(data, Key.pageUp)) {
+                let didScroll = false;
+                if (optionsScroll > 0) {
+                    optionsScroll = Math.max(0, optionsScroll - optionsViewport);
+                    didScroll = true;
+                }
                 if (previewScroll > 0) {
                     previewScroll = Math.max(0, previewScroll - previewViewport);
-                    refresh();
+                    didScroll = true;
                 }
+                if (didScroll)
+                    refresh();
                 return;
             }
             if (matchesKey(data, Key.pageDown)) {
+                let didScroll = false;
+                const maxOptionsScroll = Math.max(0, optionsTotal - optionsViewport);
+                if (optionsScroll < maxOptionsScroll) {
+                    optionsScroll = Math.min(maxOptionsScroll, optionsScroll + optionsViewport);
+                    didScroll = true;
+                }
                 const maxScroll = Math.max(0, previewTotal - previewViewport);
                 if (previewScroll < maxScroll) {
                     previewScroll = Math.min(maxScroll, previewScroll + previewViewport);
-                    refresh();
+                    didScroll = true;
                 }
+                if (didScroll)
+                    refresh();
                 return;
             }
             // ── Cursor navigation ────────────────────────────────────────
             if (matchesKey(data, Key.up)) {
                 st.cursorIndex = (st.cursorIndex - 1 + optCount) % optCount;
                 resetPreviewScroll();
+                resetOptionsScroll();
                 refresh();
                 return;
             }
             if (matchesKey(data, Key.down)) {
                 st.cursorIndex = (st.cursorIndex + 1) % optCount;
                 resetPreviewScroll();
+                resetOptionsScroll();
                 refresh();
                 return;
             }
@@ -637,6 +659,8 @@ export async function showInterviewRound(questions, opts, ctx) {
             // No scrollable preview unless the side-by-side path sets these below.
             previewTotal = 0;
             previewViewport = 0;
+            optionsTotal = 0;
+            optionsViewport = 0;
             if (useSideBySide) {
                 // ── Preview path ──────────────────────────────────────
                 const ui = makeUI(theme, contentWidth);
@@ -672,10 +696,21 @@ export async function showInterviewRound(questions, opts, ctx) {
                 const previewWidth = Math.max(MIN_PREVIEW_WIDTH, Math.min(contentWidth - MIN_OPTIONS_WIDTH - DIVIDER_WIDTH, Math.floor(contentWidth * PREVIEW_RATIO)));
                 const leftWidth = Math.max(MIN_OPTIONS_WIDTH, contentWidth - previewWidth - DIVIDER_WIDTH);
                 const fullLeft = renderOptionsColumn(leftWidth);
-                const leftLines = fullLeft.slice(0, maxBody);
+                optionsTotal = fullLeft.length;
+                optionsViewport = maxBody;
+                const maxOptionsScroll = Math.max(0, fullLeft.length - maxBody);
+                if (optionsScroll > maxOptionsScroll)
+                    optionsScroll = maxOptionsScroll;
+                const leftLines = fullLeft.slice(optionsScroll, optionsScroll + maxBody);
                 if (fullLeft.length > maxBody) {
-                    const n = fullLeft.length - maxBody + 1;
-                    leftLines[maxBody - 1] = scrollIndicator(`+${n} lines hidden`, leftWidth);
+                    const hiddenAbove = optionsScroll;
+                    const hiddenBelow = fullLeft.length - (optionsScroll + maxBody);
+                    if (hiddenAbove > 0) {
+                        leftLines[0] = scrollIndicator(`▲ ${hiddenAbove} more`, leftWidth);
+                    }
+                    if (hiddenBelow > 0) {
+                        leftLines[leftLines.length - 1] = scrollIndicator(`▼ ${hiddenBelow} more · PgUp/PgDn`, leftWidth);
+                    }
                 }
                 const preview = getCurrentPreview();
                 const fullRight = preview ? renderPreviewColumn(preview, previewWidth) : [];
@@ -734,6 +769,8 @@ export async function showInterviewRound(questions, opts, ctx) {
                 }
                 if (previewTotal > previewViewport)
                     hints.push("pgup/pgdn scroll preview");
+                if (optionsTotal > optionsViewport)
+                    hints.push("pgup/pgdn scroll options");
                 hints.push("esc to exit");
                 const footer = ui.hints(hints)[0] ?? "";
                 cachedLines = renderSharedDialogFrame(theme, title, lines, width, { footer });

package/dist/resources/extensions/subagent/index.js

@@ -591,7 +591,7 @@ thinkingOverride) {
             }
     }
 }
-const ThinkingLevelSchema = StringEnum(["off", "minimal", "low", "medium", "high", "xhigh"], {
+const ThinkingLevelSchema = StringEnum(["off", "minimal", "low", "medium", "high", "xhigh", "max"], {
     description: "Reasoning effort override for the subagent (forwarded as --thinking). " +
         "Independent of model choice; unsupported levels are clamped to the model at dispatch.",
 });

package/dist/startup-model-validation.d.ts

@@ -15,7 +15,7 @@ interface MinimalModel {
 interface MinimalModelRegistry {
     getAvailable(): MinimalModel[];
 }
-type ThinkingLevel = 'off' | 'minimal' | 'low' | 'medium' | 'high' | 'xhigh';
+type ThinkingLevel = 'off' | 'minimal' | 'low' | 'medium' | 'high' | 'xhigh' | 'max';
 interface MinimalSettingsManager {
     getDefaultProvider(): string | undefined;
     getDefaultModel(): string | undefined;