From c313c3dee1872a0edc943ad096eab68a03a3dda5 Mon Sep 17 00:00:00 2001
From: haroldmciver-go <haroldmciver@google.com>
Date: Mon, 14 Jul 2025 12:22:37 -0400
Subject: updated '/auth' to use new slash command arch (#3797)

Co-authored-by: Abhi <43648792+abhipatel12@users.noreply.github.com>
---
 .../cli/src/ui/hooks/slashCommandProcessor.test.ts | 27 ++++++++++++++++++++++
 packages/cli/src/ui/hooks/slashCommandProcessor.ts | 10 ++++----
 2 files changed, 31 insertions(+), 6 deletions(-)

(limited to 'packages/cli/src/ui/hooks')

diff --git a/packages/cli/src/ui/hooks/slashCommandProcessor.test.ts b/packages/cli/src/ui/hooks/slashCommandProcessor.test.ts
index 563092a2..d920117d 100644
--- a/packages/cli/src/ui/hooks/slashCommandProcessor.test.ts
+++ b/packages/cli/src/ui/hooks/slashCommandProcessor.test.ts
@@ -507,6 +507,33 @@ describe('useSlashCommandProcessor', () => {
       expect(commandResult).toEqual({ type: 'handled' });
     });
 
+    it('should open the auth dialog when a new command returns an auth dialog action', async () => {
+      const mockAction = vi.fn().mockResolvedValue({
+        type: 'dialog',
+        dialog: 'auth',
+      });
+      const newAuthCommand: SlashCommand = { name: 'auth', action: mockAction };
+
+      const mockLoader = async () => [newAuthCommand];
+      const commandServiceInstance = new ActualCommandService(mockLoader);
+      vi.mocked(CommandService).mockImplementation(
+        () => commandServiceInstance,
+      );
+
+      const { result } = getProcessorHook();
+      await vi.waitFor(() => {
+        expect(
+          result.current.slashCommands.some((c) => c.name === 'auth'),
+        ).toBe(true);
+      });
+
+      const commandResult = await result.current.handleSlashCommand('/auth');
+
+      expect(mockAction).toHaveBeenCalledTimes(1);
+      expect(mockOpenAuthDialog).toHaveBeenCalledWith();
+      expect(commandResult).toEqual({ type: 'handled' });
+    });
+
     it('should open the theme dialog when a new command returns a theme dialog action', async () => {
       const mockAction = vi.fn().mockResolvedValue({
         type: 'dialog',
diff --git a/packages/cli/src/ui/hooks/slashCommandProcessor.ts b/packages/cli/src/ui/hooks/slashCommandProcessor.ts
index 59f748bf..3a071fab 100644
--- a/packages/cli/src/ui/hooks/slashCommandProcessor.ts
+++ b/packages/cli/src/ui/hooks/slashCommandProcessor.ts
@@ -242,11 +242,6 @@ export const useSlashCommandProcessor = (
           }
         },
       },
-      {
-        name: 'auth',
-        description: 'change the auth method',
-        action: (_mainCommand, _subCommand, _args) => openAuthDialog(),
-      },
       {
         name: 'editor',
         description: 'set external editor preference',
@@ -1027,7 +1022,6 @@ export const useSlashCommandProcessor = (
     return commands;
   }, [
     addMessage,
-    openAuthDialog,
     openEditorDialog,
     openPrivacyNotice,
     toggleCorgiMode,
@@ -1125,6 +1119,9 @@ export const useSlashCommandProcessor = (
                   case 'help':
                     setShowHelp(true);
                     return { type: 'handled' };
+                  case 'auth':
+                    openAuthDialog();
+                    return { type: 'handled' };
                   case 'theme':
                     openThemeDialog();
                     return { type: 'handled' };
@@ -1205,6 +1202,7 @@ export const useSlashCommandProcessor = (
     [
       addItem,
       setShowHelp,
+      openAuthDialog,
       commands,
       legacyCommands,
       commandContext,
-- 
cgit v1.2.3