@akiojin/gwt 2.2.0 → 2.3.0

package/src/cli/ui/__tests__/components/screens/ExecutionModeSelectorScreen.test.tsx

@@ -1,13 +1,13 @@
 /**
  * @vitest-environment happy-dom
  */
-import { describe, it, expect, beforeEach, vi } from 'vitest';
-import { render } from '@testing-library/react';
-import React from 'react';
-import { ExecutionModeSelectorScreen } from '../../../components/screens/ExecutionModeSelectorScreen.js';
-import { Window } from 'happy-dom';
+import { describe, it, expect, beforeEach, vi } from "vitest";
+import { render } from "@testing-library/react";
+import React from "react";
+import { ExecutionModeSelectorScreen } from "../../../components/screens/ExecutionModeSelectorScreen.js";
+import { Window } from "happy-dom";
 
-describe('ExecutionModeSelectorScreen', () => {
+describe("ExecutionModeSelectorScreen", () => {
   beforeEach(() => {
     // Setup happy-dom
     const window = new Window();
@@ -15,21 +15,21 @@ describe('ExecutionModeSelectorScreen', () => {
     globalThis.document = window.document as any;
   });
 
-  it('should render header with title', () => {
+  it("should render header with title", () => {
     const onBack = vi.fn();
     const onSelect = vi.fn();
     const { container } = render(
-      <ExecutionModeSelectorScreen onBack={onBack} onSelect={onSelect} />
+      <ExecutionModeSelectorScreen onBack={onBack} onSelect={onSelect} />,
     );
 
     expect(container).toBeDefined();
   });
 
-  it('should render execution mode options', () => {
+  it("should render execution mode options", () => {
     const onBack = vi.fn();
     const onSelect = vi.fn();
     const { getByText } = render(
-      <ExecutionModeSelectorScreen onBack={onBack} onSelect={onSelect} />
+      <ExecutionModeSelectorScreen onBack={onBack} onSelect={onSelect} />,
     );
 
     expect(getByText(/Normal/i)).toBeDefined();
@@ -37,25 +37,25 @@ describe('ExecutionModeSelectorScreen', () => {
     expect(getByText(/Resume/i)).toBeDefined();
   });
 
-  it('should render footer with actions', () => {
+  it("should render footer with actions", () => {
     const onBack = vi.fn();
     const onSelect = vi.fn();
     const { getAllByText } = render(
-      <ExecutionModeSelectorScreen onBack={onBack} onSelect={onSelect} />
+      <ExecutionModeSelectorScreen onBack={onBack} onSelect={onSelect} />,
     );
 
     expect(getAllByText(/enter/i).length).toBeGreaterThan(0);
     expect(getAllByText(/esc/i).length).toBeGreaterThan(0);
   });
 
-  it('should use terminal height for layout calculation', () => {
+  it("should use terminal height for layout calculation", () => {
     const originalRows = process.stdout.rows;
     process.stdout.rows = 30;
 
     const onBack = vi.fn();
     const onSelect = vi.fn();
     const { container } = render(
-      <ExecutionModeSelectorScreen onBack={onBack} onSelect={onSelect} />
+      <ExecutionModeSelectorScreen onBack={onBack} onSelect={onSelect} />,
     );
 
     expect(container).toBeDefined();
@@ -63,22 +63,22 @@ describe('ExecutionModeSelectorScreen', () => {
     process.stdout.rows = originalRows;
   });
 
-  it('should handle back navigation with ESC key', () => {
+  it("should handle back navigation with ESC key", () => {
     const onBack = vi.fn();
     const onSelect = vi.fn();
     const { container } = render(
-      <ExecutionModeSelectorScreen onBack={onBack} onSelect={onSelect} />
+      <ExecutionModeSelectorScreen onBack={onBack} onSelect={onSelect} />,
     );
 
     // Test will verify onBack is called when ESC is pressed
     expect(container).toBeDefined();
   });
 
-  it('should handle mode selection', () => {
+  it("should handle mode selection", () => {
     const onBack = vi.fn();
     const onSelect = vi.fn();
     const { container } = render(
-      <ExecutionModeSelectorScreen onBack={onBack} onSelect={onSelect} />
+      <ExecutionModeSelectorScreen onBack={onBack} onSelect={onSelect} />,
     );
 
     // Test will verify onSelect is called with correct mode
@@ -87,12 +87,12 @@ describe('ExecutionModeSelectorScreen', () => {
 
   // TDD: Tests for 2-step selection (mode + skipPermissions)
   // TODO: Implement integration tests with user interaction simulation
-  describe.skip('Skip Permissions Selection', () => {
-    it('should render skip permissions prompt after mode selection', () => {
+  describe.skip("Skip Permissions Selection", () => {
+    it("should render skip permissions prompt after mode selection", () => {
       const onBack = vi.fn();
       const onSelect = vi.fn();
       const { getByText } = render(
-        <ExecutionModeSelectorScreen onBack={onBack} onSelect={onSelect} />
+        <ExecutionModeSelectorScreen onBack={onBack} onSelect={onSelect} />,
       );
 
       // After selecting a mode, should show skip permissions prompt
@@ -100,11 +100,11 @@ describe('ExecutionModeSelectorScreen', () => {
       expect(getByText(/Skip permission checks/i)).toBeDefined();
     });
 
-    it('should show correct flag hints for skipPermissions prompt', () => {
+    it("should show correct flag hints for skipPermissions prompt", () => {
       const onBack = vi.fn();
       const onSelect = vi.fn();
       const { getByText } = render(
-        <ExecutionModeSelectorScreen onBack={onBack} onSelect={onSelect} />
+        <ExecutionModeSelectorScreen onBack={onBack} onSelect={onSelect} />,
       );
 
       // Should show both --dangerously-skip-permissions and --yolo hints
@@ -112,10 +112,12 @@ describe('ExecutionModeSelectorScreen', () => {
       expect(getByText(/--yolo/i)).toBeDefined();
     });
 
-    it('should call onSelect with mode and skipPermissions=true when Yes is selected', () => {
+    it("should call onSelect with mode and skipPermissions=true when Yes is selected", () => {
       const onBack = vi.fn();
       const onSelect = vi.fn();
-      render(<ExecutionModeSelectorScreen onBack={onBack} onSelect={onSelect} />);
+      render(
+        <ExecutionModeSelectorScreen onBack={onBack} onSelect={onSelect} />,
+      );
 
       // After selecting mode and Yes for skipPermissions
       // onSelect should be called with { mode: 'normal', skipPermissions: true }
@@ -124,14 +126,16 @@ describe('ExecutionModeSelectorScreen', () => {
         expect.objectContaining({
           mode: expect.any(String),
           skipPermissions: true,
-        })
+        }),
       );
     });
 
-    it('should call onSelect with mode and skipPermissions=false when No is selected', () => {
+    it("should call onSelect with mode and skipPermissions=false when No is selected", () => {
       const onBack = vi.fn();
       const onSelect = vi.fn();
-      render(<ExecutionModeSelectorScreen onBack={onBack} onSelect={onSelect} />);
+      render(
+        <ExecutionModeSelectorScreen onBack={onBack} onSelect={onSelect} />,
+      );
 
       // After selecting mode and No for skipPermissions
       // onSelect should be called with { mode: 'normal', skipPermissions: false }
@@ -140,21 +144,23 @@ describe('ExecutionModeSelectorScreen', () => {
         expect.objectContaining({
           mode: expect.any(String),
           skipPermissions: false,
-        })
+        }),
       );
     });
 
-    it('should default skipPermissions to false', () => {
+    it("should default skipPermissions to false", () => {
       const onBack = vi.fn();
       const onSelect = vi.fn();
-      render(<ExecutionModeSelectorScreen onBack={onBack} onSelect={onSelect} />);
+      render(
+        <ExecutionModeSelectorScreen onBack={onBack} onSelect={onSelect} />,
+      );
 
       // Default should be No (skipPermissions: false)
       // This test will fail until implementation
       expect(onSelect).toHaveBeenCalledWith(
         expect.objectContaining({
           skipPermissions: false,
-        })
+        }),
       );
     });
   });

package/src/cli/ui/__tests__/components/screens/PRCleanupScreen.test.tsx

@@ -1,14 +1,14 @@
 /**
  * @vitest-environment happy-dom
  */
-import { describe, it, expect, beforeEach, vi } from 'vitest';
-import { render } from '@testing-library/react';
-import React from 'react';
-import { PRCleanupScreen } from '../../../components/screens/PRCleanupScreen.js';
-import { Window } from 'happy-dom';
-import type { CleanupTarget } from '../../../types.js';
-
-describe('PRCleanupScreen', () => {
+import { describe, it, expect, beforeEach, vi } from "vitest";
+import { render } from "@testing-library/react";
+import React from "react";
+import { PRCleanupScreen } from "../../../components/screens/PRCleanupScreen.js";
+import { Window } from "happy-dom";
+import type { CleanupTarget } from "../../../types.js";
+
+describe("PRCleanupScreen", () => {
   beforeEach(() => {
     // Setup happy-dom
     const window = new Window();
@@ -18,30 +18,30 @@ describe('PRCleanupScreen', () => {
 
   const mockTargets: CleanupTarget[] = [
     {
-      branch: 'feature/add-new-feature',
-      cleanupType: 'worktree-and-branch',
+      branch: "feature/add-new-feature",
+      cleanupType: "worktree-and-branch",
       pullRequest: {
         number: 123,
-        title: 'Add new feature',
-        branch: 'feature/add-new-feature',
-        mergedAt: '2025-01-20T10:00:00Z',
-        author: 'user1',
+        title: "Add new feature",
+        branch: "feature/add-new-feature",
+        mergedAt: "2025-01-20T10:00:00Z",
+        author: "user1",
       },
-      worktreePath: '/workspace/feature-add-new-feature',
+      worktreePath: "/workspace/feature-add-new-feature",
       hasUncommittedChanges: false,
       hasUnpushedCommits: false,
       hasRemoteBranch: true,
       isAccessible: true,
     },
     {
-      branch: 'hotfix/fix-bug',
-      cleanupType: 'branch-only',
+      branch: "hotfix/fix-bug",
+      cleanupType: "branch-only",
       pullRequest: {
         number: 124,
-        title: 'Fix bug',
-        branch: 'hotfix/fix-bug',
-        mergedAt: '2025-01-21T15:30:00Z',
-        author: 'user2',
+        title: "Fix bug",
+        branch: "hotfix/fix-bug",
+        mergedAt: "2025-01-21T15:30:00Z",
+        author: "user2",
       },
       worktreePath: null,
       hasUncommittedChanges: false,
@@ -50,7 +50,7 @@ describe('PRCleanupScreen', () => {
     },
   ];
 
-  it('should render header with title', () => {
+  it("should render header with title", () => {
     const onBack = vi.fn();
     const onCleanup = vi.fn();
     const { getByText } = render(
@@ -61,13 +61,13 @@ describe('PRCleanupScreen', () => {
         onBack={onBack}
         onRefresh={vi.fn()}
         onCleanup={onCleanup}
-      />
+      />,
     );
 
     expect(getByText(/Branch Cleanup/i)).toBeDefined();
   });
 
-  it('should render PR list', () => {
+  it("should render PR list", () => {
     const onBack = vi.fn();
     const onCleanup = vi.fn();
     const { getByText } = render(
@@ -78,14 +78,14 @@ describe('PRCleanupScreen', () => {
         onBack={onBack}
         onRefresh={vi.fn()}
         onCleanup={onCleanup}
-      />
+      />,
     );
 
     expect(getByText(/feature\/add-new-feature/i)).toBeDefined();
     expect(getByText(/hotfix\/fix-bug/i)).toBeDefined();
   });
 
-  it('should render footer with actions', () => {
+  it("should render footer with actions", () => {
     const onBack = vi.fn();
     const onCleanup = vi.fn();
     const { getAllByText } = render(
@@ -96,14 +96,14 @@ describe('PRCleanupScreen', () => {
         onBack={onBack}
         onRefresh={vi.fn()}
         onCleanup={onCleanup}
-      />
+      />,
     );
 
     expect(getAllByText(/enter/i).length).toBeGreaterThan(0);
     expect(getAllByText(/esc/i).length).toBeGreaterThan(0);
   });
 
-  it('should handle empty PR list', () => {
+  it("should handle empty PR list", () => {
     const onBack = vi.fn();
     const onCleanup = vi.fn();
     const { getByText } = render(
@@ -114,13 +114,13 @@ describe('PRCleanupScreen', () => {
         onBack={onBack}
         onRefresh={vi.fn()}
         onCleanup={onCleanup}
-      />
+      />,
     );
 
     expect(getByText(/No cleanup targets found/i)).toBeDefined();
   });
 
-  it('should display PR count in stats', () => {
+  it("should display PR count in stats", () => {
     const onBack = vi.fn();
     const onCleanup = vi.fn();
     const { getByText, getAllByText } = render(
@@ -131,14 +131,14 @@ describe('PRCleanupScreen', () => {
         onBack={onBack}
         onRefresh={vi.fn()}
         onCleanup={onCleanup}
-      />
+      />,
     );
 
     expect(getByText(/Total:/i)).toBeDefined();
     expect(getAllByText(/^2$/).length).toBeGreaterThan(0);
   });
 
-  it('should use terminal height for layout calculation', () => {
+  it("should use terminal height for layout calculation", () => {
     const originalRows = process.stdout.rows;
     process.stdout.rows = 30;
 
@@ -152,7 +152,7 @@ describe('PRCleanupScreen', () => {
         onBack={onBack}
         onRefresh={vi.fn()}
         onCleanup={onCleanup}
-      />
+      />,
     );
 
     expect(container).toBeDefined();
@@ -160,7 +160,7 @@ describe('PRCleanupScreen', () => {
     process.stdout.rows = originalRows;
   });
 
-  it('should handle back navigation with ESC key', () => {
+  it("should handle back navigation with ESC key", () => {
     const onBack = vi.fn();
     const onCleanup = vi.fn();
     const { container } = render(
@@ -171,14 +171,14 @@ describe('PRCleanupScreen', () => {
         onBack={onBack}
         onRefresh={vi.fn()}
         onCleanup={onCleanup}
-      />
+      />,
     );
 
     // Test will verify onBack is called when ESC is pressed
     expect(container).toBeDefined();
   });
 
-  it('should render status message when provided', () => {
+  it("should render status message when provided", () => {
     const onBack = vi.fn();
     const onCleanup = vi.fn();
     const { getByText } = render(
@@ -190,13 +190,13 @@ describe('PRCleanupScreen', () => {
         onBack={onBack}
         onRefresh={vi.fn()}
         onCleanup={onCleanup}
-      />
+      />,
     );
 
     expect(getByText(/Cleanup completed/i)).toBeDefined();
   });
 
-  it('should render loading message when loading', () => {
+  it("should render loading message when loading", () => {
     const onBack = vi.fn();
     const onCleanup = vi.fn();
     const { getByText } = render(
@@ -207,7 +207,7 @@ describe('PRCleanupScreen', () => {
         onBack={onBack}
         onRefresh={vi.fn()}
         onCleanup={onCleanup}
-      />
+      />,
     );
 
     expect(getByText(/Loading cleanup targets/i)).toBeDefined();

package/src/cli/ui/__tests__/components/screens/SessionSelectorScreen.test.tsx

@@ -1,13 +1,13 @@
 /**
  * @vitest-environment happy-dom
  */
-import { describe, it, expect, beforeEach, vi } from 'vitest';
-import { render } from '@testing-library/react';
-import React from 'react';
-import { SessionSelectorScreen } from '../../../components/screens/SessionSelectorScreen.js';
-import { Window } from 'happy-dom';
+import { describe, it, expect, beforeEach, vi } from "vitest";
+import { render } from "@testing-library/react";
+import React from "react";
+import { SessionSelectorScreen } from "../../../components/screens/SessionSelectorScreen.js";
+import { Window } from "happy-dom";
 
-describe('SessionSelectorScreen', () => {
+describe("SessionSelectorScreen", () => {
   beforeEach(() => {
     // Setup happy-dom
     const window = new Window();
@@ -15,23 +15,31 @@ describe('SessionSelectorScreen', () => {
     globalThis.document = window.document as any;
   });
 
-  const mockSessions = ['session-1', 'session-2', 'session-3'];
+  const mockSessions = ["session-1", "session-2", "session-3"];
 
-  it('should render header with title', () => {
+  it("should render header with title", () => {
     const onBack = vi.fn();
     const onSelect = vi.fn();
     const { getByText } = render(
-      <SessionSelectorScreen sessions={mockSessions} onBack={onBack} onSelect={onSelect} />
+      <SessionSelectorScreen
+        sessions={mockSessions}
+        onBack={onBack}
+        onSelect={onSelect}
+      />,
     );
 
     expect(getByText(/Session Selection/i)).toBeDefined();
   });
 
-  it('should render session list', () => {
+  it("should render session list", () => {
     const onBack = vi.fn();
     const onSelect = vi.fn();
     const { getByText } = render(
-      <SessionSelectorScreen sessions={mockSessions} onBack={onBack} onSelect={onSelect} />
+      <SessionSelectorScreen
+        sessions={mockSessions}
+        onBack={onBack}
+        onSelect={onSelect}
+      />,
     );
 
     expect(getByText(/session-1/i)).toBeDefined();
@@ -39,46 +47,62 @@ describe('SessionSelectorScreen', () => {
     expect(getByText(/session-3/i)).toBeDefined();
   });
 
-  it('should render footer with actions', () => {
+  it("should render footer with actions", () => {
     const onBack = vi.fn();
     const onSelect = vi.fn();
     const { getAllByText } = render(
-      <SessionSelectorScreen sessions={mockSessions} onBack={onBack} onSelect={onSelect} />
+      <SessionSelectorScreen
+        sessions={mockSessions}
+        onBack={onBack}
+        onSelect={onSelect}
+      />,
     );
 
     expect(getAllByText(/enter/i).length).toBeGreaterThan(0);
     expect(getAllByText(/esc/i).length).toBeGreaterThan(0);
   });
 
-  it('should handle empty session list', () => {
+  it("should handle empty session list", () => {
     const onBack = vi.fn();
     const onSelect = vi.fn();
     const { getByText } = render(
-      <SessionSelectorScreen sessions={[]} onBack={onBack} onSelect={onSelect} />
+      <SessionSelectorScreen
+        sessions={[]}
+        onBack={onBack}
+        onSelect={onSelect}
+      />,
     );
 
     expect(getByText(/No sessions found/i)).toBeDefined();
   });
 
-  it('should display session count in stats', () => {
+  it("should display session count in stats", () => {
     const onBack = vi.fn();
     const onSelect = vi.fn();
     const { getByText, getAllByText } = render(
-      <SessionSelectorScreen sessions={mockSessions} onBack={onBack} onSelect={onSelect} />
+      <SessionSelectorScreen
+        sessions={mockSessions}
+        onBack={onBack}
+        onSelect={onSelect}
+      />,
     );
 
     expect(getByText(/Total:/i)).toBeDefined();
     expect(getAllByText(/3/).length).toBeGreaterThan(0);
   });
 
-  it('should use terminal height for layout calculation', () => {
+  it("should use terminal height for layout calculation", () => {
     const originalRows = process.stdout.rows;
     process.stdout.rows = 30;
 
     const onBack = vi.fn();
     const onSelect = vi.fn();
     const { container } = render(
-      <SessionSelectorScreen sessions={mockSessions} onBack={onBack} onSelect={onSelect} />
+      <SessionSelectorScreen
+        sessions={mockSessions}
+        onBack={onBack}
+        onSelect={onSelect}
+      />,
     );
 
     expect(container).toBeDefined();
@@ -86,11 +110,15 @@ describe('SessionSelectorScreen', () => {
     process.stdout.rows = originalRows;
   });
 
-  it('should handle back navigation with ESC key', () => {
+  it("should handle back navigation with ESC key", () => {
     const onBack = vi.fn();
     const onSelect = vi.fn();
     const { container } = render(
-      <SessionSelectorScreen sessions={mockSessions} onBack={onBack} onSelect={onSelect} />
+      <SessionSelectorScreen
+        sessions={mockSessions}
+        onBack={onBack}
+        onSelect={onSelect}
+      />,
     );
 
     // Test will verify onBack is called when ESC is pressed