@akiojin/gwt 2.2.0 → 2.3.0

package/src/cli/ui/__tests__/components/parts/Footer.test.tsx

@@ -1,13 +1,13 @@
 /**
  * @vitest-environment happy-dom
  */
-import { describe, it, expect, beforeEach } from 'vitest';
-import { render } from '@testing-library/react';
-import React from 'react';
-import { Footer } from '../../../components/parts/Footer.js';
-import { Window } from 'happy-dom';
+import { describe, it, expect, beforeEach } from "vitest";
+import { render } from "@testing-library/react";
+import React from "react";
+import { Footer } from "../../../components/parts/Footer.js";
+import { Window } from "happy-dom";
 
-describe('Footer', () => {
+describe("Footer", () => {
   beforeEach(() => {
     // Setup happy-dom
     const window = new Window();
@@ -16,12 +16,12 @@ describe('Footer', () => {
   });
 
   const mockActions = [
-    { key: 'enter', description: 'Select' },
-    { key: 'esc', description: 'Back' },
-    { key: 'h', description: 'Help' },
+    { key: "enter", description: "Select" },
+    { key: "esc", description: "Back" },
+    { key: "h", description: "Help" },
   ];
 
-  it('should render all actions', () => {
+  it("should render all actions", () => {
     const { getByText } = render(<Footer actions={mockActions} />);
 
     expect(getByText(/enter/)).toBeDefined();
@@ -32,29 +32,31 @@ describe('Footer', () => {
     expect(getByText(/Help/)).toBeDefined();
   });
 
-  it('should render with empty actions array', () => {
+  it("should render with empty actions array", () => {
     const { container } = render(<Footer actions={[]} />);
 
     expect(container).toBeDefined();
   });
 
-  it('should render single action', () => {
-    const singleAction = [{ key: 'esc', description: 'Exit' }];
+  it("should render single action", () => {
+    const singleAction = [{ key: "esc", description: "Exit" }];
     const { getByText } = render(<Footer actions={singleAction} />);
 
     expect(getByText(/esc/)).toBeDefined();
     expect(getByText(/Exit/)).toBeDefined();
   });
 
-  it('should render actions in a horizontal layout', () => {
+  it("should render actions in a horizontal layout", () => {
     const { container } = render(<Footer actions={mockActions} />);
 
     // Verify component renders without error
     expect(container).toBeDefined();
   });
 
-  it('should accept custom separator', () => {
-    const { getAllByText } = render(<Footer actions={mockActions} separator=" | " />);
+  it("should accept custom separator", () => {
+    const { getAllByText } = render(
+      <Footer actions={mockActions} separator=" | " />,
+    );
 
     const separators = getAllByText(/\|/);
     expect(separators.length).toBeGreaterThan(0);

package/src/cli/ui/__tests__/components/parts/Header.test.tsx

@@ -1,13 +1,13 @@
 /**
  * @vitest-environment happy-dom
  */
-import { describe, it, expect, beforeEach } from 'vitest';
-import { render } from '@testing-library/react';
-import React from 'react';
-import { Header } from '../../../components/parts/Header.js';
-import { Window } from 'happy-dom';
+import { describe, it, expect, beforeEach } from "vitest";
+import { render } from "@testing-library/react";
+import React from "react";
+import { Header } from "../../../components/parts/Header.js";
+import { Window } from "happy-dom";
 
-describe('Header', () => {
+describe("Header", () => {
   beforeEach(() => {
     // Setup happy-dom
     const window = new Window();
@@ -15,38 +15,38 @@ describe('Header', () => {
     globalThis.document = window.document as any;
   });
 
-  it('should render title', () => {
+  it("should render title", () => {
     const { getByText } = render(<Header title="gwt" />);
 
-    expect(getByText('gwt')).toBeDefined();
+    expect(getByText("gwt")).toBeDefined();
   });
 
-  it('should render divider', () => {
+  it("should render divider", () => {
     const { getByText } = render(<Header title="Test" />);
 
     // Check for a line of dashes (divider)
     expect(getByText(/─+/)).toBeDefined();
   });
 
-  it('should render title in bold and cyan by default', () => {
+  it("should render title in bold and cyan by default", () => {
     const { container } = render(<Header title="Test Title" />);
 
     expect(container).toBeDefined();
   });
 
-  it('should accept custom title color', () => {
+  it("should accept custom title color", () => {
     const { container } = render(<Header title="Test" titleColor="green" />);
 
     expect(container).toBeDefined();
   });
 
-  it('should accept custom divider character', () => {
+  it("should accept custom divider character", () => {
     const { getByText } = render(<Header title="Test" dividerChar="=" />);
 
     expect(getByText(/=+/)).toBeDefined();
   });
 
-  it('should render without divider when showDivider is false', () => {
+  it("should render without divider when showDivider is false", () => {
     const { queryByText } = render(<Header title="Test" showDivider={false} />);
 
     expect(queryByText(/─+/)).toBeNull();

package/src/cli/ui/__tests__/components/parts/ScrollableList.test.tsx

@@ -1,14 +1,14 @@
 /**
  * @vitest-environment happy-dom
  */
-import { describe, it, expect, beforeEach } from 'vitest';
-import { render } from '@testing-library/react';
-import React from 'react';
-import { ScrollableList } from '../../../components/parts/ScrollableList.js';
-import { Text } from 'ink';
-import { Window } from 'happy-dom';
+import { describe, it, expect, beforeEach } from "vitest";
+import { render } from "@testing-library/react";
+import React from "react";
+import { ScrollableList } from "../../../components/parts/ScrollableList.js";
+import { Text } from "ink";
+import { Window } from "happy-dom";
 
-describe('ScrollableList', () => {
+describe("ScrollableList", () => {
   beforeEach(() => {
     // Setup happy-dom
     const window = new Window();
@@ -16,53 +16,53 @@ describe('ScrollableList', () => {
     globalThis.document = window.document as any;
   });
 
-  it('should render children', () => {
+  it("should render children", () => {
     const { getByText } = render(
       <ScrollableList>
         <Text>Item 1</Text>
         <Text>Item 2</Text>
         <Text>Item 3</Text>
-      </ScrollableList>
+      </ScrollableList>,
     );
 
-    expect(getByText('Item 1')).toBeDefined();
-    expect(getByText('Item 2')).toBeDefined();
-    expect(getByText('Item 3')).toBeDefined();
+    expect(getByText("Item 1")).toBeDefined();
+    expect(getByText("Item 2")).toBeDefined();
+    expect(getByText("Item 3")).toBeDefined();
   });
 
-  it('should render with no children', () => {
+  it("should render with no children", () => {
     const { container } = render(<ScrollableList>{null}</ScrollableList>);
 
     expect(container).toBeDefined();
   });
 
-  it('should accept maxHeight prop', () => {
+  it("should accept maxHeight prop", () => {
     const { container } = render(
       <ScrollableList maxHeight={10}>
         <Text>Content</Text>
-      </ScrollableList>
+      </ScrollableList>,
     );
 
     expect(container).toBeDefined();
   });
 
-  it('should render in a vertical layout', () => {
+  it("should render in a vertical layout", () => {
     const { container } = render(
       <ScrollableList>
         <Text>Content</Text>
-      </ScrollableList>
+      </ScrollableList>,
     );
 
     expect(container).toBeDefined();
   });
 
-  it('should handle single child', () => {
+  it("should handle single child", () => {
     const { getByText } = render(
       <ScrollableList>
         <Text>Single Item</Text>
-      </ScrollableList>
+      </ScrollableList>,
     );
 
-    expect(getByText('Single Item')).toBeDefined();
+    expect(getByText("Single Item")).toBeDefined();
   });
 });

package/src/cli/ui/__tests__/components/parts/Stats.test.tsx

@@ -1,14 +1,14 @@
 /**
  * @vitest-environment happy-dom
  */
-import { describe, it, expect, beforeEach } from 'vitest';
-import { render } from '@testing-library/react';
-import React from 'react';
-import { Stats } from '../../../components/parts/Stats.js';
-import type { Statistics } from '../../../types.js';
-import { Window } from 'happy-dom';
-
-describe('Stats', () => {
+import { describe, it, expect, beforeEach } from "vitest";
+import { render } from "@testing-library/react";
+import React from "react";
+import { Stats } from "../../../components/parts/Stats.js";
+import type { Statistics } from "../../../types.js";
+import { Window } from "happy-dom";
+
+describe("Stats", () => {
   beforeEach(() => {
     // Setup happy-dom
     const window = new Window();
@@ -21,10 +21,10 @@ describe('Stats', () => {
     remoteCount: 8,
     worktreeCount: 3,
     changesCount: 2,
-    lastUpdated: new Date('2025-01-25T12:00:00Z'),
+    lastUpdated: new Date("2025-01-25T12:00:00Z"),
   };
 
-  it('should render all statistics', () => {
+  it("should render all statistics", () => {
     const { getByText } = render(<Stats stats={mockStats} />);
 
     expect(getByText(/Local:/)).toBeDefined();
@@ -37,7 +37,7 @@ describe('Stats', () => {
     expect(getByText(/2/)).toBeDefined();
   });
 
-  it('should render with zero counts', () => {
+  it("should render with zero counts", () => {
     const zeroStats: Statistics = {
       localCount: 0,
       remoteCount: 0,
@@ -53,21 +53,23 @@ describe('Stats', () => {
     expect(zeros.length).toBe(4); // All 4 counts are 0
   });
 
-  it('should render in a horizontal layout', () => {
+  it("should render in a horizontal layout", () => {
     const { container } = render(<Stats stats={mockStats} />);
 
     // Verify component renders without error
     expect(container).toBeDefined();
   });
 
-  it('should accept custom separator', () => {
-    const { getAllByText } = render(<Stats stats={mockStats} separator=" | " />);
+  it("should accept custom separator", () => {
+    const { getAllByText } = render(
+      <Stats stats={mockStats} separator=" | " />,
+    );
 
     const separators = getAllByText(/\|/);
     expect(separators.length).toBeGreaterThan(0);
   });
 
-  it('should handle large numbers', () => {
+  it("should handle large numbers", () => {
     const largeStats: Statistics = {
       localCount: 999,
       remoteCount: 888,
@@ -84,51 +86,61 @@ describe('Stats', () => {
     expect(getByText(/666/)).toBeDefined();
   });
 
-  it('should display lastUpdated when provided', () => {
+  it("should display lastUpdated when provided", () => {
     const now = new Date();
     const lastUpdated = new Date(now.getTime() - 5000); // 5 seconds ago
 
-    const { getByText } = render(<Stats stats={mockStats} lastUpdated={lastUpdated} />);
+    const { getByText } = render(
+      <Stats stats={mockStats} lastUpdated={lastUpdated} />,
+    );
 
     expect(getByText(/Updated:/)).toBeDefined();
     expect(getByText(/ago/)).toBeDefined();
   });
 
-  it('should not display lastUpdated when null', () => {
-    const { queryByText } = render(<Stats stats={mockStats} lastUpdated={null} />);
+  it("should not display lastUpdated when null", () => {
+    const { queryByText } = render(
+      <Stats stats={mockStats} lastUpdated={null} />,
+    );
 
     expect(queryByText(/Updated:/)).toBeNull();
   });
 
-  it('should not display lastUpdated when not provided', () => {
+  it("should not display lastUpdated when not provided", () => {
     const { queryByText } = render(<Stats stats={mockStats} />);
 
     expect(queryByText(/Updated:/)).toBeNull();
   });
 
-  it('should format relative time correctly (seconds)', () => {
+  it("should format relative time correctly (seconds)", () => {
     const now = new Date();
     const lastUpdated = new Date(now.getTime() - 30000); // 30 seconds ago
 
-    const { getByText } = render(<Stats stats={mockStats} lastUpdated={lastUpdated} />);
+    const { getByText } = render(
+      <Stats stats={mockStats} lastUpdated={lastUpdated} />,
+    );
 
     expect(getByText(/30s ago/)).toBeDefined();
   });
 
-  it('should format relative time correctly (minutes)', () => {
+  it("should format relative time correctly (minutes)", () => {
     const now = new Date();
     const lastUpdated = new Date(now.getTime() - 120000); // 2 minutes ago
 
-    const { getByText } = render(<Stats stats={mockStats} lastUpdated={lastUpdated} />);
+    const { getByText } = render(
+      <Stats stats={mockStats} lastUpdated={lastUpdated} />,
+    );
 
     expect(getByText(/2m ago/)).toBeDefined();
   });
 
-  it('should format relative time correctly (hours)', () => {
+  it("should format relative time correctly (hours)", () => {
     const now = new Date();
     const lastUpdated = new Date(now.getTime() - 7200000); // 2 hours ago
 
-    const { getByText } = render(<Stats stats={mockStats} lastUpdated={lastUpdated} />);
+    const { getByText } = render(
+      <Stats stats={mockStats} lastUpdated={lastUpdated} />,
+    );
 
     expect(getByText(/2h ago/)).toBeDefined();
   });

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

@@ -1,29 +1,29 @@
 /**
  * @vitest-environment happy-dom
  */
-import { describe, it, expect, beforeEach, vi } from 'vitest';
-import { render, waitFor } from '@testing-library/react';
-import React from 'react';
-import { AIToolSelectorScreen } from '../../../components/screens/AIToolSelectorScreen.js';
-import { Window } from 'happy-dom';
+import { describe, it, expect, beforeEach, vi } from "vitest";
+import { render, waitFor } from "@testing-library/react";
+import React from "react";
+import { AIToolSelectorScreen } from "../../../components/screens/AIToolSelectorScreen.js";
+import { Window } from "happy-dom";
 
 // Mock getAllTools
-vi.mock('../../../config/tools.js', () => ({
+vi.mock("../../../config/tools.js", () => ({
   getAllTools: vi.fn().mockResolvedValue([
     {
-      id: 'claude-code',
-      displayName: 'Claude Code',
+      id: "claude-code",
+      displayName: "Claude Code",
       isBuiltin: true,
     },
     {
-      id: 'codex-cli',
-      displayName: 'Codex CLI',
+      id: "codex-cli",
+      displayName: "Codex",
       isBuiltin: true,
     },
   ]),
 }));
 
-describe('AIToolSelectorScreen', () => {
+describe("AIToolSelectorScreen", () => {
   beforeEach(() => {
     // Setup happy-dom
     const window = new Window();
@@ -31,49 +31,49 @@ describe('AIToolSelectorScreen', () => {
     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 { getByText } = render(
-      <AIToolSelectorScreen onBack={onBack} onSelect={onSelect} />
+      <AIToolSelectorScreen onBack={onBack} onSelect={onSelect} />,
     );
 
     expect(getByText(/AI Tool Selection/i)).toBeDefined();
   });
 
-  it('should render AI tool options', async () => {
+  it("should render AI tool options", async () => {
     const onBack = vi.fn();
     const onSelect = vi.fn();
     const { getByText } = render(
-      <AIToolSelectorScreen onBack={onBack} onSelect={onSelect} />
+      <AIToolSelectorScreen onBack={onBack} onSelect={onSelect} />,
     );
 
     // Wait for tools to load
     await waitFor(() => {
       expect(getByText(/Claude Code/i)).toBeDefined();
-      expect(getByText(/Codex CLI/i)).toBeDefined();
+      expect(getByText(/Codex/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(
-      <AIToolSelectorScreen onBack={onBack} onSelect={onSelect} />
+      <AIToolSelectorScreen 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(
-      <AIToolSelectorScreen onBack={onBack} onSelect={onSelect} />
+      <AIToolSelectorScreen onBack={onBack} onSelect={onSelect} />,
     );
 
     expect(container).toBeDefined();
@@ -81,22 +81,22 @@ describe('AIToolSelectorScreen', () => {
     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(
-      <AIToolSelectorScreen onBack={onBack} onSelect={onSelect} />
+      <AIToolSelectorScreen onBack={onBack} onSelect={onSelect} />,
     );
 
     // Test will verify onBack is called when ESC is pressed
     expect(container).toBeDefined();
   });
 
-  it('should handle tool selection', () => {
+  it("should handle tool selection", () => {
     const onBack = vi.fn();
     const onSelect = vi.fn();
     const { container } = render(
-      <AIToolSelectorScreen onBack={onBack} onSelect={onSelect} />
+      <AIToolSelectorScreen onBack={onBack} onSelect={onSelect} />,
     );
 
     // Test will verify onSelect is called with correct tool
@@ -106,15 +106,15 @@ describe('AIToolSelectorScreen', () => {
   /**
    * T210: カスタムツール表示のテスト
    */
-  describe('Custom tool display', () => {
-    it('should load tools from getAllTools() dynamically', async () => {
+  describe("Custom tool display", () => {
+    it("should load tools from getAllTools() dynamically", async () => {
       // TODO: 実装後にテストを記述
       // getAllTools()がモックされ、呼び出されることを確認
       // モックの戻り値がツールアイテムとして表示されることを確認
       expect(true).toBe(true);
     });
 
-    it('should display both builtin and custom tools', async () => {
+    it("should display both builtin and custom tools", async () => {
       // TODO: 実装後にテストを記述
       // getAllTools()がビルトインツール（claude-code, codex-cli）と
       // カスタムツール（例: aider）を返す場合、
@@ -122,28 +122,28 @@ describe('AIToolSelectorScreen', () => {
       expect(true).toBe(true);
     });
 
-    it('should display custom tool with icon if defined', async () => {
+    it("should display custom tool with icon if defined", async () => {
       // TODO: 実装後にテストを記述
       // カスタムツールにiconフィールドがある場合、
       // それが表示されることを確認
       expect(true).toBe(true);
     });
 
-    it('should display custom tool without icon if not defined', async () => {
+    it("should display custom tool without icon if not defined", async () => {
       // TODO: 実装後にテストを記述
       // カスタムツールにiconフィールドがない場合、
       // ツール名のみが表示されることを確認
       expect(true).toBe(true);
     });
 
-    it('should handle custom tool selection', async () => {
+    it("should handle custom tool selection", async () => {
       // TODO: 実装後にテストを記述
       // カスタムツールを選択した場合、
       // onSelect()がカスタムツールのIDで呼び出されることを確認
       expect(true).toBe(true);
     });
 
-    it('should display only builtin tools if no custom tools exist', async () => {
+    it("should display only builtin tools if no custom tools exist", async () => {
       // TODO: 実装後にテストを記述
       // getAllTools()がビルトインツールのみを返す場合、
       // ビルトインツールのみが表示されることを確認