gsd-pi 2.23.0 → 2.25.0

package/src/resources/extensions/gsd/tests/visualizer-data.test.ts

@@ -56,6 +56,51 @@ assertTrue(
   "exports ChangelogInfo interface",
 );
 
+assertTrue(
+  dataSrc.includes("export interface SliceVerification"),
+  "exports SliceVerification interface",
+);
+
+assertTrue(
+  dataSrc.includes("export interface KnowledgeInfo"),
+  "exports KnowledgeInfo interface",
+);
+
+assertTrue(
+  dataSrc.includes("export interface CapturesInfo"),
+  "exports CapturesInfo interface",
+);
+
+assertTrue(
+  dataSrc.includes("export interface HealthInfo"),
+  "exports HealthInfo interface",
+);
+
+assertTrue(
+  dataSrc.includes("export interface VisualizerDiscussionState"),
+  "exports VisualizerDiscussionState interface",
+);
+
+assertTrue(
+  dataSrc.includes("export type DiscussionState"),
+  "exports DiscussionState type",
+);
+
+assertTrue(
+  dataSrc.includes("export interface VisualizerSliceRef"),
+  "exports VisualizerSliceRef interface",
+);
+
+assertTrue(
+  dataSrc.includes("export interface VisualizerSliceActivity"),
+  "exports VisualizerSliceActivity interface",
+);
+
+assertTrue(
+  dataSrc.includes("export interface VisualizerStats"),
+  "exports VisualizerStats interface",
+);
+
 // Function export
 assertTrue(
   dataSrc.includes("export async function loadVisualizerData"),
@@ -123,6 +168,36 @@ assertTrue(
   "uses aggregateByModel",
 );
 
+assertTrue(
+  dataSrc.includes("aggregateByTier"),
+  "uses aggregateByTier",
+);
+
+assertTrue(
+  dataSrc.includes("formatTierSavings"),
+  "uses formatTierSavings",
+);
+
+assertTrue(
+  dataSrc.includes("loadAllCaptures"),
+  "uses loadAllCaptures",
+);
+
+assertTrue(
+  dataSrc.includes("countPendingCaptures"),
+  "uses countPendingCaptures",
+);
+
+assertTrue(
+  dataSrc.includes("loadEffectiveGSDPreferences"),
+  "uses loadEffectiveGSDPreferences",
+);
+
+assertTrue(
+  dataSrc.includes("resolveGsdRootFile"),
+  "uses resolveGsdRootFile for KNOWLEDGE path",
+);
+
 // Interface fields
 assertTrue(
   dataSrc.includes("dependsOn: string[]"),
@@ -144,6 +219,11 @@ assertTrue(
   "VisualizerData has units array",
 );
 
+assertTrue(
+  dataSrc.includes("estimate?: string"),
+  "VisualizerTask has optional estimate field",
+);
+
 // New data model fields
 assertTrue(
   dataSrc.includes("criticalPath: CriticalPathInfo"),
@@ -165,6 +245,56 @@ assertTrue(
   "VisualizerData has changelog field",
 );
 
+assertTrue(
+  dataSrc.includes("sliceVerifications: SliceVerification[]"),
+  "VisualizerData has sliceVerifications field",
+);
+
+assertTrue(
+  dataSrc.includes("knowledge: KnowledgeInfo"),
+  "VisualizerData has knowledge field",
+);
+
+assertTrue(
+  dataSrc.includes("captures: CapturesInfo"),
+  "VisualizerData has captures field",
+);
+
+assertTrue(
+  dataSrc.includes("health: HealthInfo"),
+  "VisualizerData has health field",
+);
+
+assertTrue(
+  dataSrc.includes("stats: VisualizerStats"),
+  "VisualizerData has stats field",
+);
+
+assertTrue(
+  dataSrc.includes("discussion: VisualizerDiscussionState[]"),
+  "VisualizerData has discussion field",
+);
+
+assertTrue(
+  dataSrc.includes("loadDiscussionState"),
+  "uses loadDiscussionState helper",
+);
+
+assertTrue(
+  dataSrc.includes("buildVisualizerStats"),
+  "uses buildVisualizerStats helper",
+);
+
+assertTrue(
+  dataSrc.includes("byTier: TierAggregate[]"),
+  "VisualizerData has byTier field",
+);
+
+assertTrue(
+  dataSrc.includes("tierSavingsLine: string"),
+  "VisualizerData has tierSavingsLine field",
+);
+
 // completedAt must be coerced to String() to handle YAML Date objects (issue #644)
 assertTrue(
   dataSrc.includes("String(summary.frontmatter.completed_at"),
@@ -227,6 +357,21 @@ assertTrue(
   "overlay delegates to renderExportView",
 );
 
+assertTrue(
+  overlaySrc.includes("renderKnowledgeView"),
+  "overlay delegates to renderKnowledgeView",
+);
+
+assertTrue(
+  overlaySrc.includes("renderCapturesView"),
+  "overlay delegates to renderCapturesView",
+);
+
+assertTrue(
+  overlaySrc.includes("renderHealthView"),
+  "overlay delegates to renderHealthView",
+);
+
 assertTrue(
   overlaySrc.includes("handleInput"),
   "overlay has handleInput method",
@@ -273,8 +418,8 @@ assertTrue(
 );
 
 assertTrue(
-  overlaySrc.includes("7 Export"),
-  "overlay has 7 tab labels",
+  overlaySrc.includes("0 Health"),
+  "overlay has 10 tab labels",
 );
 
 // Verify commands.ts integration

package/src/resources/extensions/gsd/tests/visualizer-overlay.test.ts

@@ -1,5 +1,5 @@
 // Tests for GSD visualizer overlay.
-// Verifies filter mode, tab switching, including reverse tab navigation, and export key handling.
+// Verifies filter mode, tab switching, mouse support, page scroll, help overlay, and 10-tab config.
 
 import { readFileSync } from "node:fs";
 import { join, dirname } from "node:path";
@@ -14,8 +14,8 @@ const overlaySrc = readFileSync(join(__dirname, "..", "visualizer-overlay.ts"),
 console.log("\n=== Overlay: Tab Configuration ===");
 
 assertTrue(
-  overlaySrc.includes("TAB_COUNT = 7"),
-  "TAB_COUNT is 7",
+  overlaySrc.includes("TAB_COUNT = 10"),
+  "TAB_COUNT is 10",
 );
 
 assertTrue(
@@ -38,6 +38,21 @@ assertTrue(
   "has Export tab label",
 );
 
+assertTrue(
+  overlaySrc.includes('"8 Knowledge"'),
+  "has Knowledge tab label",
+);
+
+assertTrue(
+  overlaySrc.includes('"9 Captures"'),
+  "has Captures tab label",
+);
+
+assertTrue(
+  overlaySrc.includes('"0 Health"'),
+  "has Health tab label",
+);
+
 console.log("\n=== Overlay: Filter Mode ===");
 
 assertTrue(
@@ -69,10 +84,10 @@ assertTrue(
 
 console.log("\n=== Overlay: Tab Switching ===");
 
-// Supports 1-7 keys
+// Supports 1-9,0 keys
 assertTrue(
-  overlaySrc.includes('"1234567"'),
-  "supports keys 1-7 for tab switching",
+  overlaySrc.includes('"1234567890"'),
+  "supports keys 1-9,0 for tab switching",
 );
 
 // Tab wraps with TAB_COUNT
@@ -86,6 +101,64 @@ assertTrue(
   "supports Shift+Tab for reverse tab switching",
 );
 
+console.log("\n=== Overlay: Page/Half-Page Scroll ===");
+
+assertTrue(
+  overlaySrc.includes("Key.pageUp"),
+  "has Key.pageUp handler",
+);
+
+assertTrue(
+  overlaySrc.includes("Key.pageDown"),
+  "has Key.pageDown handler",
+);
+
+assertTrue(
+  overlaySrc.includes('Key.ctrl("u")'),
+  "has Ctrl+U half-page scroll",
+);
+
+assertTrue(
+  overlaySrc.includes('Key.ctrl("d")'),
+  "has Ctrl+D half-page scroll",
+);
+
+console.log("\n=== Overlay: Mouse Support ===");
+
+assertTrue(
+  overlaySrc.includes("parseSGRMouse"),
+  "has parseSGRMouse method",
+);
+
+assertTrue(
+  overlaySrc.includes("?1003h"),
+  "enables mouse tracking in constructor",
+);
+
+assertTrue(
+  overlaySrc.includes("?1003l"),
+  "disables mouse tracking in dispose",
+);
+
+console.log("\n=== Overlay: Collapsible Milestones ===");
+
+assertTrue(
+  overlaySrc.includes("collapsedMilestones"),
+  "has collapsedMilestones state",
+);
+
+console.log("\n=== Overlay: Help Overlay ===");
+
+assertTrue(
+  overlaySrc.includes("showHelp"),
+  "has showHelp state",
+);
+
+assertTrue(
+  overlaySrc.includes('data === "?"'),
+  "? key toggles help",
+);
+
 console.log("\n=== Overlay: Export Key Interception ===");
 
 assertTrue(
@@ -106,13 +179,18 @@ assertTrue(
 console.log("\n=== Overlay: Footer ===");
 
 assertTrue(
-  overlaySrc.includes("Tab/Shift+Tab/1-7"),
-  "footer hint shows Tab, Shift+Tab, and 1-7 tab range",
+  overlaySrc.includes("1-9,0"),
+  "footer hint shows 1-9,0 tab range",
+);
+
+assertTrue(
+  overlaySrc.includes("PgUp/PgDn"),
+  "footer hint mentions PgUp/PgDn",
 );
 
 assertTrue(
-  overlaySrc.includes("/ filter"),
-  "footer hint mentions filter",
+  overlaySrc.includes("? help"),
+  "footer hint mentions ? for help",
 );
 
 console.log("\n=== Overlay: Scroll Offsets ===");