@tulex-cli/tulex-code 0.1.1

package/dist/src/ui/hooks/useSlashCompletion.test.js

@@ -0,0 +1,805 @@
+/**
+ * @license
+ * Copyright 2025 Google LLC
+ * SPDX-License-Identifier: Apache-2.0
+ */
+/** @vitest-environment jsdom */
+import { describe, it, expect, vi } from 'vitest';
+import { renderHook, waitFor } from '@testing-library/react';
+import { useSlashCompletion } from './useSlashCompletion.js';
+import { CommandKind } from '../commands/types.js';
+import { useState } from 'react';
+function createTestCommand(command) {
+    return {
+        kind: CommandKind.BUILT_IN, // default for tests
+        ...command,
+    };
+}
+// Track AsyncFzf constructor calls for cache testing
+let asyncFzfConstructorCalls = 0;
+const resetConstructorCallCount = () => {
+    asyncFzfConstructorCalls = 0;
+};
+const getConstructorCallCount = () => asyncFzfConstructorCalls;
+// Centralized fuzzy matching simulation logic
+// Note: This is a simplified reimplementation that may diverge from real fzf behavior.
+// Integration tests in useSlashCompletion.integration.test.ts use the real fzf library
+// to catch any behavioral differences and serve as our "canary in a coal mine."
+function simulateFuzzyMatching(items, query) {
+    const results = [];
+    if (query) {
+        const lowerQuery = query.toLowerCase();
+        for (const item of items) {
+            const lowerItem = item.toLowerCase();
+            // Exact match gets highest score
+            if (lowerItem === lowerQuery) {
+                results.push({
+                    item,
+                    positions: [],
+                    score: 100,
+                    start: 0,
+                    end: item.length,
+                });
+                continue;
+            }
+            // Prefix match gets high score
+            if (lowerItem.startsWith(lowerQuery)) {
+                results.push({
+                    item,
+                    positions: [],
+                    score: 80,
+                    start: 0,
+                    end: query.length,
+                });
+                continue;
+            }
+            // Fuzzy matching: check if query chars appear in order
+            let queryIndex = 0;
+            let score = 0;
+            for (let i = 0; i < lowerItem.length && queryIndex < lowerQuery.length; i++) {
+                if (lowerItem[i] === lowerQuery[queryIndex]) {
+                    queryIndex++;
+                    score += 10 - i; // Earlier matches get higher scores
+                }
+            }
+            // If all query characters were found in order, include this item
+            if (queryIndex === lowerQuery.length) {
+                results.push({
+                    item,
+                    positions: [],
+                    score,
+                    start: 0,
+                    end: query.length,
+                });
+            }
+        }
+    }
+    // Sort by score descending (better matches first)
+    results.sort((a, b) => b.score - a.score);
+    return Promise.resolve(results);
+}
+// Mock the fzf module to provide a working fuzzy search implementation for tests
+vi.mock('fzf', async () => {
+    const actual = await vi.importActual('fzf');
+    return {
+        ...actual,
+        AsyncFzf: vi.fn().mockImplementation((items, _options) => {
+            asyncFzfConstructorCalls++;
+            return {
+                find: vi
+                    .fn()
+                    .mockImplementation((query) => simulateFuzzyMatching(items, query)),
+                // eslint-disable-next-line @typescript-eslint/no-explicit-any
+            };
+        }),
+    };
+});
+// Default mock behavior helper - now uses centralized logic
+const createDefaultAsyncFzfMock = () => (items, _options) => {
+    asyncFzfConstructorCalls++;
+    return {
+        find: vi
+            .fn()
+            .mockImplementation((query) => simulateFuzzyMatching(items, query)),
+        // eslint-disable-next-line @typescript-eslint/no-explicit-any
+    };
+};
+// Export test utilities
+export { resetConstructorCallCount, getConstructorCallCount, createDefaultAsyncFzfMock, };
+// Test harness to capture the state from the hook's callbacks.
+function useTestHarnessForSlashCompletion(enabled, query, slashCommands, commandContext, recentCommands) {
+    const [suggestions, setSuggestions] = useState([]);
+    const [isLoadingSuggestions, setIsLoadingSuggestions] = useState(false);
+    const [isPerfectMatch, setIsPerfectMatch] = useState(false);
+    const { completionStart, completionEnd } = useSlashCompletion({
+        enabled,
+        query,
+        slashCommands,
+        commandContext,
+        recentCommands,
+        setSuggestions,
+        setIsLoadingSuggestions,
+        setIsPerfectMatch,
+    });
+    return {
+        suggestions,
+        isLoadingSuggestions,
+        isPerfectMatch,
+        completionStart,
+        completionEnd,
+    };
+}
+describe('useSlashCompletion', () => {
+    // A minimal mock is sufficient for these tests.
+    const mockCommandContext = {};
+    describe('Top-Level Commands', () => {
+        it('should suggest all top-level commands for the root slash', async () => {
+            const slashCommands = [
+                createTestCommand({
+                    name: 'help',
+                    altNames: ['?'],
+                    description: 'Show help',
+                }),
+                createTestCommand({
+                    name: 'stats',
+                    altNames: ['usage'],
+                    description: 'check session stats. Usage: /stats [model|tools]',
+                }),
+                createTestCommand({
+                    name: 'clear',
+                    altNames: ['reset', 'new'],
+                    description: 'Clear the screen',
+                }),
+                createTestCommand({
+                    name: 'memory',
+                    description: 'Manage memory',
+                    subCommands: [
+                        createTestCommand({ name: 'show', description: 'Show memory' }),
+                    ],
+                }),
+                createTestCommand({ name: 'chat', description: 'Manage chat history' }),
+            ];
+            const { result } = renderHook(() => useTestHarnessForSlashCompletion(true, '/', slashCommands, mockCommandContext));
+            expect(result.current.suggestions.length).toBe(slashCommands.length);
+            expect(result.current.suggestions.map((s) => s.label)).toEqual(expect.arrayContaining([
+                'help (?)',
+                'clear (reset, new)',
+                'memory',
+                'chat',
+                'stats (usage)',
+            ]));
+        });
+        it('should filter commands based on partial input', async () => {
+            const slashCommands = [
+                createTestCommand({ name: 'memory', description: 'Manage memory' }),
+            ];
+            const { result } = renderHook(() => useTestHarnessForSlashCompletion(true, '/mem', slashCommands, mockCommandContext));
+            await waitFor(() => {
+                expect(result.current.suggestions).toEqual([
+                    expect.objectContaining({
+                        label: 'memory',
+                        value: 'memory',
+                        description: 'Manage memory',
+                        commandKind: CommandKind.BUILT_IN,
+                    }),
+                ]);
+            });
+        });
+        it('should not include alias noise for primary-name matches', async () => {
+            const slashCommands = [
+                createTestCommand({
+                    name: 'help',
+                    altNames: ['?'],
+                    description: 'for help on Qwen Code',
+                }),
+            ];
+            const { result } = renderHook(() => useTestHarnessForSlashCompletion(true, '/he', slashCommands, mockCommandContext));
+            await waitFor(() => {
+                expect(result.current.suggestions).toEqual([
+                    expect.objectContaining({
+                        label: 'help',
+                        value: 'help',
+                        matchedAlias: undefined,
+                    }),
+                ]);
+            });
+        });
+        it('should keep argumentHint out of command suggestion labels', async () => {
+            const slashCommands = [
+                createTestCommand({
+                    name: 'fix-issue',
+                    description: 'Fix GitHub issue',
+                    argumentHint: '[issue-number]',
+                }),
+            ];
+            const { result } = renderHook(() => useTestHarnessForSlashCompletion(true, '/fix', slashCommands, mockCommandContext));
+            await waitFor(() => {
+                expect(result.current.suggestions).toEqual([
+                    expect.objectContaining({
+                        label: 'fix-issue',
+                        value: 'fix-issue',
+                        description: 'Fix GitHub issue',
+                        commandKind: CommandKind.BUILT_IN,
+                        argumentHint: '[issue-number]',
+                    }),
+                ]);
+            });
+        });
+        it('should prefer higher completionPriority when match quality ties', async () => {
+            const slashCommands = [
+                createTestCommand({
+                    name: 'mock',
+                    description: 'Mock command',
+                }),
+                createTestCommand({
+                    name: 'model',
+                    description: 'Model command',
+                    completionPriority: 100,
+                }),
+            ];
+            const { result } = renderHook(() => useTestHarnessForSlashCompletion(true, '/mo', slashCommands, mockCommandContext));
+            await waitFor(() => {
+                expect(result.current.suggestions.map((s) => s.value)).toEqual([
+                    'model',
+                    'mock',
+                ]);
+            });
+        });
+        it('should suggest commands based on partial altNames', async () => {
+            const slashCommands = [
+                createTestCommand({
+                    name: 'stats',
+                    altNames: ['usage'],
+                    description: 'check session stats. Usage: /stats [model|tools]',
+                }),
+            ];
+            const { result } = renderHook(() => useTestHarnessForSlashCompletion(true, '/usag', slashCommands, mockCommandContext));
+            await waitFor(() => {
+                expect(result.current.suggestions).toEqual([
+                    expect.objectContaining({
+                        label: 'stats (alias: usage)',
+                        value: 'stats',
+                        description: 'check session stats. Usage: /stats [model|tools]',
+                        matchedAlias: 'usage',
+                    }),
+                ]);
+            });
+        });
+        it('should include command metadata in slash suggestions', async () => {
+            const slashCommands = [
+                createTestCommand({
+                    name: 'review',
+                    description: 'Review changed code',
+                    argumentHint: '[pr-number]',
+                    source: 'bundled-skill',
+                    sourceLabel: 'Skill',
+                    modelInvocable: true,
+                }),
+            ];
+            const { result } = renderHook(() => useTestHarnessForSlashCompletion(true, '/rev', slashCommands, mockCommandContext));
+            await waitFor(() => {
+                expect(result.current.suggestions).toEqual([
+                    expect.objectContaining({
+                        label: 'review',
+                        value: 'review',
+                        argumentHint: '[pr-number]',
+                        source: 'bundled-skill',
+                        sourceLabel: 'Skill',
+                        sourceBadge: '[Skill]',
+                        modelInvocable: true,
+                    }),
+                ]);
+            });
+        });
+        it('should boost recent commands for root slash suggestions', async () => {
+            const now = Date.now();
+            const slashCommands = [
+                createTestCommand({
+                    name: 'alpha',
+                    description: 'Alpha command',
+                    completionPriority: 100,
+                }),
+                createTestCommand({ name: 'beta', description: 'Beta command' }),
+            ];
+            const recentCommands = new Map([
+                ['beta', { name: 'beta', usedAt: now, count: 1 }],
+            ]);
+            const { result } = renderHook(() => useTestHarnessForSlashCompletion(true, '/', slashCommands, mockCommandContext, recentCommands));
+            expect(result.current.suggestions.map((suggestion) => suggestion.value)).toEqual(['beta', 'alpha']);
+        });
+        it('should boost recent help command above high-priority model for root slash suggestions', async () => {
+            const now = Date.now();
+            const slashCommands = [
+                createTestCommand({
+                    name: 'model',
+                    description: 'Model command',
+                    completionPriority: 100,
+                }),
+                createTestCommand({
+                    name: 'help',
+                    altNames: ['?'],
+                    description: 'for help on Qwen Code',
+                }),
+            ];
+            const recentCommands = new Map([
+                ['help', { name: 'help', usedAt: now, count: 1 }],
+            ]);
+            const { result } = renderHook(() => useTestHarnessForSlashCompletion(true, '/', slashCommands, mockCommandContext, recentCommands));
+            expect(result.current.suggestions.map((suggestion) => suggestion.value)).toEqual(['help', 'model']);
+        });
+        it('should boost recent commands for non-root prefix suggestions', async () => {
+            const now = Date.now();
+            const slashCommands = [
+                createTestCommand({
+                    name: 'model',
+                    description: 'Model command',
+                    completionPriority: 5,
+                }),
+                createTestCommand({
+                    name: 'memory',
+                    description: 'Memory command',
+                    completionPriority: 5,
+                }),
+            ];
+            // Both commands have equal completionPriority; 'memory' used recently
+            // should be ranked first for '/mo' via recentScore.
+            const recentCommands = new Map([
+                ['memory', { name: 'memory', usedAt: now, count: 1 }],
+            ]);
+            const { result } = renderHook(() => useTestHarnessForSlashCompletion(true, '/m', slashCommands, mockCommandContext, recentCommands));
+            await waitFor(() => {
+                const names = result.current.suggestions.map((s) => s.value);
+                expect(names).toContain('memory');
+                expect(names).toContain('model');
+                expect(names.indexOf('memory')).toBeLessThan(names.indexOf('model'));
+            });
+        });
+        it('should NOT provide suggestions for a perfectly typed command that is a leaf node', async () => {
+            const slashCommands = [
+                createTestCommand({
+                    name: 'clear',
+                    description: 'Clear the screen',
+                    action: vi.fn(),
+                }),
+            ];
+            const { result } = renderHook(() => useTestHarnessForSlashCompletion(true, '/clear', slashCommands, mockCommandContext));
+            expect(result.current.suggestions).toHaveLength(0);
+        });
+        it.each([['/?'], ['/usage']])('should not suggest commands when altNames is fully typed', async (query) => {
+            const mockSlashCommands = [
+                createTestCommand({
+                    name: 'help',
+                    altNames: ['?'],
+                    description: 'Show help',
+                    action: vi.fn(),
+                }),
+                createTestCommand({
+                    name: 'stats',
+                    altNames: ['usage'],
+                    description: 'check session stats. Usage: /stats [model|tools]',
+                    action: vi.fn(),
+                }),
+            ];
+            const { result } = renderHook(() => useTestHarnessForSlashCompletion(true, query, mockSlashCommands, mockCommandContext));
+            expect(result.current.suggestions).toHaveLength(0);
+        });
+        it('should not provide suggestions for a fully typed command that has no sub-commands or argument completion', async () => {
+            const slashCommands = [
+                createTestCommand({ name: 'clear', description: 'Clear the screen' }),
+            ];
+            const { result } = renderHook(() => useTestHarnessForSlashCompletion(true, '/clear ', slashCommands, mockCommandContext));
+            expect(result.current.suggestions).toHaveLength(0);
+        });
+        it('should not provide suggestions for an unknown command', async () => {
+            const slashCommands = [
+                createTestCommand({ name: 'help', description: 'Show help' }),
+            ];
+            const { result } = renderHook(() => useTestHarnessForSlashCompletion(true, '/unknown-command', slashCommands, mockCommandContext));
+            expect(result.current.suggestions).toHaveLength(0);
+        });
+        it('should not suggest hidden commands', async () => {
+            const slashCommands = [
+                createTestCommand({
+                    name: 'visible',
+                    description: 'A visible command',
+                }),
+                createTestCommand({
+                    name: 'hidden',
+                    description: 'A hidden command',
+                    hidden: true,
+                }),
+            ];
+            const { result } = renderHook(() => useTestHarnessForSlashCompletion(true, '/', slashCommands, mockCommandContext));
+            expect(result.current.suggestions.length).toBe(1);
+            expect(result.current.suggestions[0].label).toBe('visible');
+        });
+    });
+    describe('Sub-Commands', () => {
+        it('should suggest sub-commands for a parent command', async () => {
+            const slashCommands = [
+                createTestCommand({
+                    name: 'memory',
+                    description: 'Manage memory',
+                    subCommands: [
+                        createTestCommand({ name: 'show', description: 'Show memory' }),
+                        createTestCommand({ name: 'add', description: 'Add to memory' }),
+                    ],
+                }),
+            ];
+            const { result } = renderHook(() => useTestHarnessForSlashCompletion(true, '/memory', slashCommands, mockCommandContext));
+            expect(result.current.suggestions).toHaveLength(2);
+            expect(result.current.suggestions).toEqual(expect.arrayContaining([
+                expect.objectContaining({
+                    label: 'show',
+                    value: 'show',
+                    description: 'Show memory',
+                    commandKind: CommandKind.BUILT_IN,
+                }),
+                expect.objectContaining({
+                    label: 'add',
+                    value: 'add',
+                    description: 'Add to memory',
+                    commandKind: CommandKind.BUILT_IN,
+                }),
+            ]));
+        });
+        it('should suggest all sub-commands when the query ends with the parent command and a space', async () => {
+            const slashCommands = [
+                createTestCommand({
+                    name: 'memory',
+                    description: 'Manage memory',
+                    subCommands: [
+                        createTestCommand({ name: 'show', description: 'Show memory' }),
+                        createTestCommand({ name: 'add', description: 'Add to memory' }),
+                    ],
+                }),
+            ];
+            const { result } = renderHook(() => useTestHarnessForSlashCompletion(true, '/memory ', slashCommands, mockCommandContext));
+            expect(result.current.suggestions).toHaveLength(2);
+            expect(result.current.suggestions).toEqual(expect.arrayContaining([
+                expect.objectContaining({
+                    label: 'show',
+                    value: 'show',
+                    description: 'Show memory',
+                    commandKind: CommandKind.BUILT_IN,
+                }),
+                expect.objectContaining({
+                    label: 'add',
+                    value: 'add',
+                    description: 'Add to memory',
+                    commandKind: CommandKind.BUILT_IN,
+                }),
+            ]));
+        });
+        it('should filter sub-commands by prefix', async () => {
+            const slashCommands = [
+                createTestCommand({
+                    name: 'memory',
+                    description: 'Manage memory',
+                    subCommands: [
+                        createTestCommand({ name: 'show', description: 'Show memory' }),
+                        createTestCommand({ name: 'add', description: 'Add to memory' }),
+                    ],
+                }),
+            ];
+            const { result } = renderHook(() => useTestHarnessForSlashCompletion(true, '/memory a', slashCommands, mockCommandContext));
+            await waitFor(() => {
+                expect(result.current.suggestions).toEqual([
+                    expect.objectContaining({
+                        label: 'add',
+                        value: 'add',
+                        description: 'Add to memory',
+                        commandKind: CommandKind.BUILT_IN,
+                    }),
+                ]);
+            });
+        });
+        it('should provide no suggestions for an invalid sub-command', async () => {
+            const slashCommands = [
+                createTestCommand({
+                    name: 'memory',
+                    description: 'Manage memory',
+                    subCommands: [
+                        createTestCommand({ name: 'show', description: 'Show memory' }),
+                        createTestCommand({ name: 'add', description: 'Add to memory' }),
+                    ],
+                }),
+            ];
+            const { result } = renderHook(() => useTestHarnessForSlashCompletion(true, '/memory dothisnow', slashCommands, mockCommandContext));
+            expect(result.current.suggestions).toHaveLength(0);
+        });
+    });
+    describe('Argument Completion', () => {
+        it('should call the command.completion function for argument suggestions', async () => {
+            const availableTags = ['--project', '--global'];
+            const mockCompletionFn = vi
+                .fn()
+                .mockImplementation(async (_context, partialArg) => availableTags.filter((tag) => tag.startsWith(partialArg)));
+            const slashCommands = [
+                createTestCommand({
+                    name: 'config',
+                    description: 'Manage configuration',
+                    subCommands: [
+                        createTestCommand({
+                            name: 'set',
+                            description: 'Set configuration',
+                            completion: mockCompletionFn,
+                        }),
+                    ],
+                }),
+            ];
+            const { result } = renderHook(() => useTestHarnessForSlashCompletion(true, '/config set --project', slashCommands, mockCommandContext));
+            await waitFor(() => {
+                expect(mockCompletionFn).toHaveBeenCalledWith(expect.objectContaining({
+                    invocation: {
+                        raw: '/config set --project',
+                        name: 'set',
+                        args: '--project',
+                    },
+                }), '--project');
+            });
+            await waitFor(() => {
+                expect(result.current.suggestions).toEqual([
+                    { label: '--project', value: '--project' },
+                ]);
+            });
+        });
+        it('should map completion items with descriptions for argument suggestions', async () => {
+            const mockCompletionFn = vi.fn().mockResolvedValue([
+                { value: 'pdf', description: 'Create PDF documents' },
+                { value: 'xlsx', description: 'Work with spreadsheets' },
+            ]);
+            const slashCommands = [
+                createTestCommand({
+                    name: 'skills',
+                    description: 'List available skills',
+                    completion: mockCompletionFn,
+                }),
+            ];
+            const { result } = renderHook(() => useTestHarnessForSlashCompletion(true, '/skills ', slashCommands, mockCommandContext));
+            await waitFor(() => {
+                expect(result.current.suggestions).toEqual([
+                    {
+                        label: 'pdf',
+                        value: 'pdf',
+                        description: 'Create PDF documents',
+                    },
+                    {
+                        label: 'xlsx',
+                        value: 'xlsx',
+                        description: 'Work with spreadsheets',
+                    },
+                ]);
+            });
+        });
+        it('should call command.completion with an empty string when args start with a space', async () => {
+            const mockCompletionFn = vi
+                .fn()
+                .mockResolvedValue(['my-chat-tag-1', 'my-chat-tag-2', 'my-channel']);
+            const slashCommands = [
+                createTestCommand({
+                    name: 'workspace',
+                    description: 'Manage workspaces',
+                    subCommands: [
+                        createTestCommand({
+                            name: 'switch',
+                            description: 'Switch workspace',
+                            completion: mockCompletionFn,
+                        }),
+                    ],
+                }),
+            ];
+            const { result } = renderHook(() => useTestHarnessForSlashCompletion(true, '/workspace switch ', slashCommands, mockCommandContext));
+            await waitFor(() => {
+                expect(mockCompletionFn).toHaveBeenCalledWith(expect.objectContaining({
+                    invocation: {
+                        raw: '/workspace switch',
+                        name: 'switch',
+                        args: '',
+                    },
+                }), '');
+            });
+            await waitFor(() => {
+                expect(result.current.suggestions).toHaveLength(3);
+            });
+        });
+        it('should handle completion function that returns null', async () => {
+            const completionFn = vi.fn().mockResolvedValue(null);
+            const slashCommands = [
+                createTestCommand({
+                    name: 'workspace',
+                    description: 'Manage workspaces',
+                    subCommands: [
+                        createTestCommand({
+                            name: 'switch',
+                            description: 'Switch workspace',
+                            completion: completionFn,
+                        }),
+                    ],
+                }),
+            ];
+            const { result } = renderHook(() => useTestHarnessForSlashCompletion(true, '/workspace switch ', slashCommands, mockCommandContext));
+            await waitFor(() => {
+                expect(result.current.suggestions).toHaveLength(0);
+            });
+        });
+    });
+    describe('Command Kind Information', () => {
+        it('should include commandKind for MCP commands in suggestions', async () => {
+            const slashCommands = [
+                {
+                    name: 'summarize',
+                    description: 'Summarize content',
+                    kind: CommandKind.MCP_PROMPT,
+                    action: vi.fn(),
+                },
+                {
+                    name: 'help',
+                    description: 'Show help',
+                    kind: CommandKind.BUILT_IN,
+                    action: vi.fn(),
+                },
+            ];
+            const { result } = renderHook(() => useTestHarnessForSlashCompletion(true, '/', slashCommands, mockCommandContext));
+            expect(result.current.suggestions).toEqual(expect.arrayContaining([
+                expect.objectContaining({
+                    label: 'summarize',
+                    value: 'summarize',
+                    description: 'Summarize content',
+                    commandKind: CommandKind.MCP_PROMPT,
+                }),
+                expect.objectContaining({
+                    label: 'help',
+                    value: 'help',
+                    description: 'Show help',
+                    commandKind: CommandKind.BUILT_IN,
+                }),
+            ]));
+        });
+        it('should include commandKind when filtering MCP commands by prefix', async () => {
+            const slashCommands = [
+                {
+                    name: 'summarize',
+                    description: 'Summarize content',
+                    kind: CommandKind.MCP_PROMPT,
+                    action: vi.fn(),
+                },
+                {
+                    name: 'settings',
+                    description: 'Open settings',
+                    kind: CommandKind.BUILT_IN,
+                    action: vi.fn(),
+                },
+            ];
+            const { result } = renderHook(() => useTestHarnessForSlashCompletion(true, '/summ', slashCommands, mockCommandContext));
+            await waitFor(() => {
+                expect(result.current.suggestions).toEqual([
+                    expect.objectContaining({
+                        label: 'summarize',
+                        value: 'summarize',
+                        description: 'Summarize content',
+                        commandKind: CommandKind.MCP_PROMPT,
+                    }),
+                ]);
+            });
+        });
+        it('should include commandKind for sub-commands', async () => {
+            const slashCommands = [
+                {
+                    name: 'memory',
+                    description: 'Manage memory',
+                    kind: CommandKind.BUILT_IN,
+                    subCommands: [
+                        {
+                            name: 'show',
+                            description: 'Show memory',
+                            kind: CommandKind.BUILT_IN,
+                            action: vi.fn(),
+                        },
+                        {
+                            name: 'add',
+                            description: 'Add to memory',
+                            kind: CommandKind.MCP_PROMPT,
+                            action: vi.fn(),
+                        },
+                    ],
+                },
+            ];
+            const { result } = renderHook(() => useTestHarnessForSlashCompletion(true, '/memory', slashCommands, mockCommandContext));
+            expect(result.current.suggestions).toEqual(expect.arrayContaining([
+                expect.objectContaining({
+                    label: 'show',
+                    value: 'show',
+                    description: 'Show memory',
+                    commandKind: CommandKind.BUILT_IN,
+                }),
+                expect.objectContaining({
+                    label: 'add',
+                    value: 'add',
+                    description: 'Add to memory',
+                    commandKind: CommandKind.MCP_PROMPT,
+                }),
+            ]));
+        });
+        it('should include commandKind for file commands', async () => {
+            const slashCommands = [
+                {
+                    name: 'custom-script',
+                    description: 'Run custom script',
+                    kind: CommandKind.FILE,
+                    action: vi.fn(),
+                },
+            ];
+            const { result } = renderHook(() => useTestHarnessForSlashCompletion(true, '/custom', slashCommands, mockCommandContext));
+            await waitFor(() => {
+                expect(result.current.suggestions).toEqual([
+                    expect.objectContaining({
+                        label: 'custom-script',
+                        value: 'custom-script',
+                        description: 'Run custom script',
+                        commandKind: CommandKind.FILE,
+                    }),
+                ]);
+            });
+        });
+    });
+    it('should not call shared callbacks when disabled', () => {
+        const mockSetSuggestions = vi.fn();
+        const mockSetIsLoadingSuggestions = vi.fn();
+        const mockSetIsPerfectMatch = vi.fn();
+        const slashCommands = [
+            createTestCommand({
+                name: 'help',
+                description: 'Show help',
+            }),
+        ];
+        const { rerender } = renderHook(({ enabled, query }) => useSlashCompletion({
+            enabled,
+            query,
+            slashCommands,
+            commandContext: mockCommandContext,
+            setSuggestions: mockSetSuggestions,
+            setIsLoadingSuggestions: mockSetIsLoadingSuggestions,
+            setIsPerfectMatch: mockSetIsPerfectMatch,
+        }), {
+            initialProps: { enabled: false, query: '@src/file' },
+        });
+        // Clear any initial calls
+        mockSetSuggestions.mockClear();
+        mockSetIsLoadingSuggestions.mockClear();
+        mockSetIsPerfectMatch.mockClear();
+        // Change query while disabled (simulating @ completion typing)
+        rerender({ enabled: false, query: '@src/file.ts' });
+        rerender({ enabled: false, query: '@src/file.tsx' });
+        // Should not have called shared callbacks during @ completion typing
+        expect(mockSetSuggestions).not.toHaveBeenCalled();
+        expect(mockSetIsLoadingSuggestions).not.toHaveBeenCalled();
+        expect(mockSetIsPerfectMatch).not.toHaveBeenCalled();
+    });
+    describe('isDirectory propagation', () => {
+        it('should propagate isDirectory from CommandCompletionItem to Suggestion', async () => {
+            const mockCompletionFn = vi
+                .fn()
+                .mockResolvedValue([
+                { value: '/tmp/workspace/', isDirectory: true },
+                { value: '/tmp/file.txt' },
+            ]);
+            const slashCommands = [
+                createTestCommand({
+                    name: 'dir',
+                    description: 'test',
+                    completion: mockCompletionFn,
+                }),
+            ];
+            const { result } = renderHook(() => useTestHarnessForSlashCompletion(true, '/dir ', slashCommands, mockCommandContext));
+            await waitFor(() => {
+                expect(result.current.suggestions.length).toBe(2);
+            });
+            // First suggestion (directory) should have isDirectory: true
+            expect(result.current.suggestions[0].isDirectory).toBe(true);
+            // Second suggestion (file) should NOT have isDirectory flag
+            expect(result.current.suggestions[1].isDirectory).toBeFalsy();
+        });
+    });
+});
+//# sourceMappingURL=useSlashCompletion.test.js.map