From ec5e9d1025fee66938153140b5f37e35b50c67ed Mon Sep 17 00:00:00 2001 From: Lee Won Jun Date: Wed, 16 Jul 2025 08:45:10 +0900 Subject: Improve altName completion behavior in slash commands (#4227) Co-authored-by: Jacob Richman --- .../src/ui/hooks/useCompletion.integration.test.ts | 35 +++++++++++++++++++--- 1 file changed, 31 insertions(+), 4 deletions(-) (limited to 'packages/cli/src/ui/hooks/useCompletion.integration.test.ts') diff --git a/packages/cli/src/ui/hooks/useCompletion.integration.test.ts b/packages/cli/src/ui/hooks/useCompletion.integration.test.ts index 705b2735..37075e3c 100644 --- a/packages/cli/src/ui/hooks/useCompletion.integration.test.ts +++ b/packages/cli/src/ui/hooks/useCompletion.integration.test.ts @@ -54,6 +54,12 @@ describe('useCompletion git-aware filtering integration', () => { description: 'Show help', action: vi.fn(), }, + { + name: 'stats', + altName: 'usage', + description: 'check session stats. Usage: /stats [model|tools]', + action: vi.fn(), + }, { name: 'clear', description: 'Clear the screen', @@ -511,10 +517,27 @@ describe('useCompletion git-aware filtering integration', () => { expect(result.current.showSuggestions).toBe(true); }); - it('should suggest commands based on altName', async () => { + it.each([['/?'], ['/usage']])( + 'should not suggest commands when altName is fully typed', + async (altName) => { + const { result } = renderHook(() => + useCompletion( + altName, + '/test/cwd', + true, + mockSlashCommands, + mockCommandContext, + ), + ); + + expect(result.current.suggestions).toHaveLength(0); + }, + ); + + it('should suggest commands based on partial altName matches', async () => { const { result } = renderHook(() => useCompletion( - '/?', + '/usag', // part of the word "usage" '/test/cwd', true, mockSlashCommands, @@ -523,7 +546,11 @@ describe('useCompletion git-aware filtering integration', () => { ); expect(result.current.suggestions).toEqual([ - { label: 'help', value: 'help', description: 'Show help' }, + { + label: 'stats', + value: 'stats', + description: 'check session stats. Usage: /stats [model|tools]', + }, ]); }); @@ -734,7 +761,7 @@ describe('useCompletion git-aware filtering integration', () => { expect(result.current.suggestions.length).toBe(mockSlashCommands.length); expect(result.current.suggestions.map((s) => s.label)).toEqual( - expect.arrayContaining(['help', 'clear', 'memory', 'chat']), + expect.arrayContaining(['help', 'clear', 'memory', 'chat', 'stats']), ); }); -- cgit v1.2.3