gsd-pi 2.24.0 → 2.25.0

package/dist/resources/extensions/gsd/tests/memory-store.test.ts

@@ -0,0 +1,345 @@
+import { createTestContext } from './test-helpers.ts';
+import {
+  openDatabase,
+  closeDatabase,
+  isDbAvailable,
+  _getAdapter,
+} from '../gsd-db.ts';
+import {
+  getActiveMemories,
+  getActiveMemoriesRanked,
+  nextMemoryId,
+  createMemory,
+  updateMemoryContent,
+  reinforceMemory,
+  supersedeMemory,
+  isUnitProcessed,
+  markUnitProcessed,
+  decayStaleMemories,
+  enforceMemoryCap,
+  applyMemoryActions,
+  formatMemoriesForPrompt,
+} from '../memory-store.ts';
+import type { MemoryAction } from '../memory-store.ts';
+
+const { assertEq, assertTrue, assertMatch, report } = createTestContext();
+
+// ═══════════════════════════════════════════════════════════════════════════
+// memory-store: fallback when DB not open
+// ═══════════════════════════════════════════════════════════════════════════
+
+console.log('\n=== memory-store: fallback returns empty when DB not open ===');
+{
+  closeDatabase();
+  assertTrue(!isDbAvailable(), 'DB should not be available');
+
+  assertEq(getActiveMemories(), [], 'getActiveMemories returns [] when DB closed');
+  assertEq(getActiveMemoriesRanked(), [], 'getActiveMemoriesRanked returns [] when DB closed');
+  assertEq(nextMemoryId(), 'MEM001', 'nextMemoryId returns MEM001 when DB closed');
+  assertEq(createMemory({ category: 'test', content: 'test' }), null, 'createMemory returns null when DB closed');
+  assertTrue(!reinforceMemory('MEM001'), 'reinforceMemory returns false when DB closed');
+  assertTrue(!isUnitProcessed('test/key'), 'isUnitProcessed returns false when DB closed');
+}
+
+// ═══════════════════════════════════════════════════════════════════════════
+// memory-store: CRUD operations
+// ═══════════════════════════════════════════════════════════════════════════
+
+console.log('\n=== memory-store: create and query memories ===');
+{
+  openDatabase(':memory:');
+
+  // Create memories
+  const id1 = createMemory({ category: 'gotcha', content: 'esbuild drops .node binaries' });
+  assertTrue(id1 !== null, 'createMemory should return an ID');
+  assertEq(id1, 'MEM001', 'first memory ID should be MEM001');
+
+  const id2 = createMemory({ category: 'convention', content: 'use :memory: for tests', confidence: 0.9 });
+  assertEq(id2, 'MEM002', 'second memory ID should be MEM002');
+
+  const id3 = createMemory({ category: 'architecture', content: 'extensions discovered from src/resources/' });
+  assertEq(id3, 'MEM003', 'third memory ID should be MEM003');
+
+  // Query all active
+  const active = getActiveMemories();
+  assertEq(active.length, 3, 'should have 3 active memories');
+  assertEq(active[0].category, 'gotcha', 'first memory category');
+  assertEq(active[0].content, 'esbuild drops .node binaries', 'first memory content');
+  assertEq(active[1].confidence, 0.9, 'second memory confidence');
+
+  closeDatabase();
+}
+
+// ═══════════════════════════════════════════════════════════════════════════
+// memory-store: update and reinforce
+// ═══════════════════════════════════════════════════════════════════════════
+
+console.log('\n=== memory-store: update and reinforce ===');
+{
+  openDatabase(':memory:');
+
+  createMemory({ category: 'gotcha', content: 'original content' });
+
+  // Update content
+  const updated = updateMemoryContent('MEM001', 'revised content', 0.95);
+  assertTrue(updated, 'updateMemoryContent should return true');
+
+  const active = getActiveMemories();
+  assertEq(active[0].content, 'revised content', 'content should be updated');
+  assertEq(active[0].confidence, 0.95, 'confidence should be updated');
+
+  // Reinforce
+  const reinforced = reinforceMemory('MEM001');
+  assertTrue(reinforced, 'reinforceMemory should return true');
+
+  const after = getActiveMemories();
+  assertEq(after[0].hit_count, 1, 'hit_count should be 1 after reinforce');
+
+  // Reinforce again
+  reinforceMemory('MEM001');
+  const after2 = getActiveMemories();
+  assertEq(after2[0].hit_count, 2, 'hit_count should be 2 after second reinforce');
+
+  closeDatabase();
+}
+
+// ═══════════════════════════════════════════════════════════════════════════
+// memory-store: supersede
+// ═══════════════════════════════════════════════════════════════════════════
+
+console.log('\n=== memory-store: supersede ===');
+{
+  openDatabase(':memory:');
+
+  createMemory({ category: 'convention', content: 'old convention' });
+  createMemory({ category: 'convention', content: 'new convention' });
+
+  supersedeMemory('MEM001', 'MEM002');
+
+  const active = getActiveMemories();
+  assertEq(active.length, 1, 'should have 1 active memory after supersede');
+  assertEq(active[0].id, 'MEM002', 'active memory should be MEM002');
+
+  closeDatabase();
+}
+
+// ═══════════════════════════════════════════════════════════════════════════
+// memory-store: ranked query ordering
+// ═══════════════════════════════════════════════════════════════════════════
+
+console.log('\n=== memory-store: ranked query ordering ===');
+{
+  openDatabase(':memory:');
+
+  // Low confidence, no hits
+  createMemory({ category: 'pattern', content: 'low ranking', confidence: 0.5 });
+  // High confidence, no hits
+  createMemory({ category: 'gotcha', content: 'high confidence', confidence: 0.95 });
+  // Medium confidence, many hits
+  createMemory({ category: 'convention', content: 'frequently used', confidence: 0.7 });
+
+  // Reinforce MEM003 multiple times to boost its ranking
+  for (let i = 0; i < 10; i++) reinforceMemory('MEM003');
+
+  const ranked = getActiveMemoriesRanked(10);
+  assertEq(ranked.length, 3, 'should have 3 ranked memories');
+  // MEM003: 0.7 * (1 + 10*0.1) = 0.7 * 2.0 = 1.4
+  // MEM002: 0.95 * (1 + 0*0.1) = 0.95
+  // MEM001: 0.5 * (1 + 0*0.1) = 0.5
+  assertEq(ranked[0].id, 'MEM003', 'highest ranked should be MEM003 (reinforced)');
+  assertEq(ranked[1].id, 'MEM002', 'second ranked should be MEM002 (high confidence)');
+  assertEq(ranked[2].id, 'MEM001', 'lowest ranked should be MEM001');
+
+  // Test limit
+  const limited = getActiveMemoriesRanked(2);
+  assertEq(limited.length, 2, 'limit should cap results');
+
+  closeDatabase();
+}
+
+// ═══════════════════════════════════════════════════════════════════════════
+// memory-store: processed unit tracking
+// ═══════════════════════════════════════════════════════════════════════════
+
+console.log('\n=== memory-store: processed unit tracking ===');
+{
+  openDatabase(':memory:');
+
+  assertTrue(!isUnitProcessed('execute-task/M001/S01/T01'), 'should not be processed initially');
+
+  markUnitProcessed('execute-task/M001/S01/T01', '/path/to/activity.jsonl');
+
+  assertTrue(isUnitProcessed('execute-task/M001/S01/T01'), 'should be processed after marking');
+  assertTrue(!isUnitProcessed('execute-task/M001/S01/T02'), 'different key should not be processed');
+
+  closeDatabase();
+}
+
+// ═══════════════════════════════════════════════════════════════════════════
+// memory-store: enforce memory cap
+// ═══════════════════════════════════════════════════════════════════════════
+
+console.log('\n=== memory-store: enforce memory cap ===');
+{
+  openDatabase(':memory:');
+
+  // Create 5 memories with varying confidence
+  createMemory({ category: 'gotcha', content: 'mem 1', confidence: 0.9 });
+  createMemory({ category: 'gotcha', content: 'mem 2', confidence: 0.5 });
+  createMemory({ category: 'gotcha', content: 'mem 3', confidence: 0.3 });
+  createMemory({ category: 'gotcha', content: 'mem 4', confidence: 0.95 });
+  createMemory({ category: 'gotcha', content: 'mem 5', confidence: 0.7 });
+
+  // Enforce cap of 3
+  enforceMemoryCap(3);
+
+  const active = getActiveMemories();
+  assertEq(active.length, 3, 'should have 3 active memories after cap enforcement');
+
+  // The 2 lowest-ranked (MEM003=0.3 and MEM002=0.5) should be superseded
+  const ids = active.map(m => m.id).sort();
+  assertTrue(ids.includes('MEM001'), 'MEM001 (0.9) should survive');
+  assertTrue(ids.includes('MEM004'), 'MEM004 (0.95) should survive');
+  assertTrue(ids.includes('MEM005'), 'MEM005 (0.7) should survive');
+
+  closeDatabase();
+}
+
+// ═══════════════════════════════════════════════════════════════════════════
+// memory-store: applyMemoryActions transaction
+// ═══════════════════════════════════════════════════════════════════════════
+
+console.log('\n=== memory-store: applyMemoryActions ===');
+{
+  openDatabase(':memory:');
+
+  const actions: MemoryAction[] = [
+    { action: 'CREATE', category: 'gotcha', content: 'first gotcha', confidence: 0.8 },
+    { action: 'CREATE', category: 'convention', content: 'first convention', confidence: 0.9 },
+  ];
+
+  applyMemoryActions(actions, 'execute-task', 'M001/S01/T01');
+
+  let active = getActiveMemories();
+  assertEq(active.length, 2, 'should have 2 memories after CREATE actions');
+
+  // Now apply UPDATE + REINFORCE
+  const updateActions: MemoryAction[] = [
+    { action: 'UPDATE', id: 'MEM001', content: 'updated gotcha' },
+    { action: 'REINFORCE', id: 'MEM002' },
+  ];
+
+  applyMemoryActions(updateActions, 'execute-task', 'M001/S01/T02');
+
+  active = getActiveMemories();
+  assertEq(active.find(m => m.id === 'MEM001')?.content, 'updated gotcha', 'MEM001 should be updated');
+  assertEq(active.find(m => m.id === 'MEM002')?.hit_count, 1, 'MEM002 should be reinforced');
+
+  // SUPERSEDE
+  const supersedeActions: MemoryAction[] = [
+    { action: 'CREATE', category: 'gotcha', content: 'better gotcha', confidence: 0.95 },
+    { action: 'SUPERSEDE', id: 'MEM001', superseded_by: 'MEM003' },
+  ];
+
+  applyMemoryActions(supersedeActions, 'execute-task', 'M001/S01/T03');
+
+  active = getActiveMemories();
+  assertEq(active.length, 2, 'should have 2 active after supersede');
+  assertTrue(!active.find(m => m.id === 'MEM001'), 'MEM001 should be superseded');
+  assertTrue(!!active.find(m => m.id === 'MEM003'), 'MEM003 should be active');
+
+  closeDatabase();
+}
+
+// ═══════════════════════════════════════════════════════════════════════════
+// memory-store: formatMemoriesForPrompt
+// ═══════════════════════════════════════════════════════════════════════════
+
+console.log('\n=== memory-store: formatMemoriesForPrompt ===');
+{
+  openDatabase(':memory:');
+
+  createMemory({ category: 'gotcha', content: 'esbuild drops .node binaries' });
+  createMemory({ category: 'convention', content: 'use :memory: for tests' });
+  createMemory({ category: 'architecture', content: 'extensions in src/resources/' });
+  createMemory({ category: 'gotcha', content: 'TypeScript path aliases need .js' });
+
+  const memories = getActiveMemoriesRanked(30);
+  const formatted = formatMemoriesForPrompt(memories);
+
+  assertTrue(formatted.includes('## Project Memory (auto-learned)'), 'should have header');
+  assertTrue(formatted.includes('### Gotcha'), 'should have gotcha category');
+  assertTrue(formatted.includes('### Convention'), 'should have convention category');
+  assertTrue(formatted.includes('### Architecture'), 'should have architecture category');
+  assertTrue(formatted.includes('- esbuild drops .node binaries'), 'should have gotcha content');
+  assertTrue(formatted.includes('- use :memory: for tests'), 'should have convention content');
+
+  // Test empty memories
+  closeDatabase();
+  openDatabase(':memory:');
+  const emptyFormatted = formatMemoriesForPrompt([]);
+  assertEq(emptyFormatted, '', 'empty memories should return empty string');
+
+  // Test token budget truncation
+  closeDatabase();
+  openDatabase(':memory:');
+  for (let i = 0; i < 20; i++) {
+    createMemory({ category: 'pattern', content: `A very long memory entry that takes up space #${i}: ${'x'.repeat(200)}` });
+  }
+  const budgetMemories = getActiveMemoriesRanked(30);
+  const truncated = formatMemoriesForPrompt(budgetMemories, 500);
+  assertTrue(truncated.length < 2500, `formatted length ${truncated.length} should be under budget`);
+
+  closeDatabase();
+}
+
+// ═══════════════════════════════════════════════════════════════════════════
+// memory-store: ID generation
+// ═══════════════════════════════════════════════════════════════════════════
+
+console.log('\n=== memory-store: ID generation ===');
+{
+  openDatabase(':memory:');
+
+  assertEq(nextMemoryId(), 'MEM001', 'first ID should be MEM001');
+
+  createMemory({ category: 'test', content: 'test' });
+  assertEq(nextMemoryId(), 'MEM002', 'after first create, next should be MEM002');
+
+  // Create several more
+  for (let i = 0; i < 98; i++) createMemory({ category: 'test', content: `test ${i}` });
+  assertEq(nextMemoryId(), 'MEM100', 'after 99 creates, next should be MEM100');
+
+  closeDatabase();
+}
+
+// ═══════════════════════════════════════════════════════════════════════════
+// memory-store: schema migration (v2 → v3)
+// ═══════════════════════════════════════════════════════════════════════════
+
+console.log('\n=== memory-store: schema includes memories table ===');
+{
+  openDatabase(':memory:');
+
+  const adapter = _getAdapter()!;
+
+  // Verify memories table exists
+  const memCount = adapter.prepare('SELECT count(*) as cnt FROM memories').get();
+  assertEq(memCount?.['cnt'], 0, 'memories table should exist and be empty');
+
+  // Verify memory_processed_units table exists
+  const procCount = adapter.prepare('SELECT count(*) as cnt FROM memory_processed_units').get();
+  assertEq(procCount?.['cnt'], 0, 'memory_processed_units table should exist and be empty');
+
+  // Verify active_memories view exists
+  const viewCount = adapter.prepare('SELECT count(*) as cnt FROM active_memories').get();
+  assertEq(viewCount?.['cnt'], 0, 'active_memories view should exist');
+
+  // Verify schema version is 3
+  const version = adapter.prepare('SELECT MAX(version) as v FROM schema_version').get();
+  assertEq(version?.['v'], 3, 'schema version should be 3');
+
+  closeDatabase();
+}
+
+report();

package/dist/resources/extensions/gsd/tests/smart-entry-draft.test.ts

@@ -81,7 +81,7 @@ assert(
 
 // Check the branch has draft-aware menu options
 const branchIdx = guidedFlowSource.indexOf('state.phase === "needs-discussion"');
-const branchChunk = guidedFlowSource.slice(branchIdx, branchIdx + 3000);
+const branchChunk = guidedFlowSource.slice(branchIdx, branchIdx + 4000);
 
 assert(
   branchChunk.includes("discuss_draft"),

package/dist/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/dist/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 ===");