@astryxdesign/core 0.7.3 → 0.7.4-canary.3f6ea7d

package/src/hooks/useTypeahead.test.tsx

@@ -10,7 +10,8 @@
  */
 
 import {describe, it, expect, vi, beforeEach, afterEach} from 'vitest';
-import {renderHook} from '@testing-library/react';
+import {Activity, type ReactNode} from 'react';
+import {render, renderHook} from '@testing-library/react';
 import {useTypeahead} from './useTypeahead';
 
 const LABELS = ['Apple', 'Apricot', 'Banana', 'Cherry'] as const;
@@ -86,6 +87,46 @@ describe('useTypeahead', () => {
     expect(onMatch).toHaveBeenLastCalledWith(2);
   });
 
+  it('clears the pending buffer reset when the component unmounts', () => {
+    const {result, unmount} = renderHook(() =>
+      useTypeahead({getItemLabels: () => LABELS, onMatch: vi.fn()}),
+    );
+    result.current.onKeyDown(key('a'));
+    expect(vi.getTimerCount()).toBe(1);
+    unmount();
+    expect(vi.getTimerCount()).toBe(0);
+  });
+
+  it('empties the buffer when an Activity boundary hides the collection', () => {
+    const latest: {api: ReturnType<typeof useTypeahead> | null} = {api: null};
+    function Probe(): ReactNode {
+      latest.api = useTypeahead({
+        getItemLabels: () => LABELS,
+        onMatch: vi.fn(),
+      });
+      return null;
+    }
+    const {rerender} = render(
+      <Activity mode="visible">
+        <Probe />
+      </Activity>,
+    );
+    latest.api?.onKeyDown(key('a'));
+    rerender(
+      <Activity mode="hidden">
+        <Probe />
+      </Activity>,
+    );
+    expect(vi.getTimerCount()).toBe(0);
+    rerender(
+      <Activity mode="visible">
+        <Probe />
+      </Activity>,
+    );
+    // The typed prefix went with the hide, so a bare Space is not typeahead.
+    expect(latest.api?.onKeyDown(key(' '))).toBe(false);
+  });
+
   it('skips disabled items', () => {
     const {onMatch, api} = setup({disabledIndices: [0]}); // Apple disabled
     api.onKeyDown(key('a'));

package/src/hooks/useTypeahead.ts

@@ -4,7 +4,7 @@
 
 /**
  * @file useTypeahead.ts
- * @input Uses React useCallback/useRef
+ * @input Uses React useCallback/useEffect/useRef
  * @output Exports useTypeahead hook for first-character (type-to-focus) search
  * @position Core hook; adds APG typeahead to menus, listboxes, and other
  *   collections. Composes with useListFocus/useGridFocus (or any collection)
@@ -14,7 +14,7 @@
  * - /packages/core/src/hooks/index.ts
  */
 
-import {useCallback, useRef} from 'react';
+import {useCallback, useEffect, useRef} from 'react';
 
 export interface UseTypeaheadOptions {
   /**
@@ -135,6 +135,11 @@ export function useTypeahead(options: UseTypeaheadOptions): UseTypeaheadReturn {
     }
   }, []);
 
+  // A collection that unmounts, or that an `<Activity>` boundary hides,
+  // mid-typing leaves no reset timer running and comes back with an empty
+  // buffer, as the reset would have left it.
+  useEffect(() => reset, [reset]);
+
   const scheduleReset = useCallback(() => {
     if (timeoutRef.current) {
       clearTimeout(timeoutRef.current);

package/src/i18n/generated-locales/en.generated.ts

@@ -340,6 +340,7 @@ const runtimeCatalog: Readonly<Record<string, string>> = {
   '@astryx.chatToolCalls.status.complete': 'Complete',
   '@astryx.chatToolCalls.status.error': 'Failed',
   '@astryx.chatToolCalls.groupLabel': '{count} tool calls',
+  '@astryx.chatTriggerMenu.searching': 'Searching…',
   '@astryx.chatTriggerMenu.suggestions': 'Suggestions',
   '@astryx.citation.label': 'Citation {number}: {title}',
   '@astryx.codeBlock.copied': 'Copied',

package/src/i18n/generated-locales/pseudo.generated.ts

@@ -347,6 +347,7 @@ const runtimeCatalog: Readonly<Record<string, string>> = {
   '@astryx.chatToolCalls.status.complete': '⟦Çóɱþłéţé⟧',
   '@astryx.chatToolCalls.status.error': '⟦Ƒàíłéð⟧',
   '@astryx.chatToolCalls.groupLabel': '⟦{count} ţóół çàłłš⟧',
+  '@astryx.chatTriggerMenu.searching': '⟦Šéàřçĥíñĝ…⟧',
   '@astryx.chatTriggerMenu.suggestions': '⟦Šúĝĝéšţíóñš⟧',
   '@astryx.citation.label': '⟦Çíţàţíóñ {number}: {title}⟧',
   '@astryx.codeBlock.copied': '⟦Çóþíéð⟧',