feat(plan): update cycling order of approval modes (#17622)

Author: Adib234

packages/cli/src/ui/constants/tips.ts

@@ -90,7 +90,7 @@ export const INFORMATIVE_TIPS = [
   'Toggle the todo list display with Ctrl+T…',
   'See full, untruncated responses with Ctrl+O…',
   'Toggle auto-approval (YOLO mode) for all tools with Ctrl+Y…',
-  'Cycle through approval modes (Default, Plan, Auto-Edit) with Shift+Tab…',
+  'Cycle through approval modes (Default, Auto-Edit, Plan) with Shift+Tab…',
   'Toggle Markdown rendering (raw markdown mode) with Alt+M…',
   'Toggle shell mode by typing ! in an empty prompt…',
   'Insert a newline with a backslash (\\) followed by Enter…',

packages/cli/src/ui/hooks/useApprovalModeIndicator.test.ts

@@ -237,23 +237,23 @@ describe('useApprovalModeIndicator', () => {
       }),
     );
 
-    // DEFAULT -> PLAN
+    // DEFAULT -> AUTO_EDIT
     act(() => {
       capturedUseKeypressHandler({ name: 'tab', shift: true } as Key);
     });
     expect(mockConfigInstance.setApprovalMode).toHaveBeenCalledWith(
-      ApprovalMode.PLAN,
+      ApprovalMode.AUTO_EDIT,
     );
 
-    // PLAN -> AUTO_EDIT
+    // AUTO_EDIT -> PLAN
     act(() => {
       capturedUseKeypressHandler({ name: 'tab', shift: true } as Key);
     });
     expect(mockConfigInstance.setApprovalMode).toHaveBeenCalledWith(
-      ApprovalMode.AUTO_EDIT,
+      ApprovalMode.PLAN,
     );
 
-    // AUTO_EDIT -> DEFAULT
+    // PLAN -> DEFAULT
     act(() => {
       capturedUseKeypressHandler({ name: 'tab', shift: true } as Key);
     });

packages/cli/src/ui/hooks/useApprovalModeIndicator.ts

@@ -59,14 +59,14 @@ export function useApprovalModeIndicator({
         const currentMode = config.getApprovalMode();
         switch (currentMode) {
           case ApprovalMode.DEFAULT:
+            nextApprovalMode = ApprovalMode.AUTO_EDIT;
+            break;
+          case ApprovalMode.AUTO_EDIT:
             nextApprovalMode = config.isPlanEnabled()
               ? ApprovalMode.PLAN
-              : ApprovalMode.AUTO_EDIT;
+              : ApprovalMode.DEFAULT;
             break;
           case ApprovalMode.PLAN:
-            nextApprovalMode = ApprovalMode.AUTO_EDIT;
-            break;
-          case ApprovalMode.AUTO_EDIT:
             nextApprovalMode = ApprovalMode.DEFAULT;
             break;
           case ApprovalMode.YOLO: