@opengsd/gsd-pi 1.9.0 → 1.10.0

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/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 });