fix: lint and type errors

packages/cli/src/ui/App.tsx

@@ -56,12 +56,12 @@ import { RadioButtonSelect } from './components/shared/RadioButtonSelect.js';
 import { ModelSelectionDialog } from './components/ModelSelectionDialog.js';
 import {
   ModelSwitchDialog,
-  VisionSwitchOutcome,
+  type VisionSwitchOutcome,
 } from './components/ModelSwitchDialog.js';
 import {
   AVAILABLE_MODELS_QWEN,
   getOpenAIAvailableModelFromEnv,
-  AvailableModel,
+  type AvailableModel,
 } from './models/availableModels.js';
 import { processVisionSwitchOutcome } from './hooks/useVisionAutoSwitch.js';
 import {

packages/cli/src/ui/commands/modelCommand.ts

@@ -5,17 +5,17 @@
  */
 
 import { AuthType } from '@qwen-code/qwen-code-core';
-import {
+import type {
   SlashCommand,
   CommandContext,
-  CommandKind,
   OpenDialogActionReturn,
   MessageActionReturn,
 } from './types.js';
+import { CommandKind } from './types.js';
 import {
   AVAILABLE_MODELS_QWEN,
   getOpenAIAvailableModelFromEnv,
-  AvailableModel,
+  type AvailableModel,
 } from '../models/availableModels.js';
 
 function getAvailableModelsForAuthType(authType: AuthType): AvailableModel[] {

packages/cli/src/ui/components/ModelSelectionDialog.test.tsx

@@ -8,8 +8,8 @@ import React from 'react';
 import { render } from 'ink-testing-library';
 import { describe, it, expect, vi, beforeEach } from 'vitest';
 import { ModelSelectionDialog } from './ModelSelectionDialog.js';
-import { AvailableModel } from '../models/availableModels.js';
+import type { AvailableModel } from '../models/availableModels.js';
-import { RadioSelectItem } from './shared/RadioButtonSelect.js';
+import type { RadioSelectItem } from './shared/RadioButtonSelect.js';
 
 // Mock the useKeypress hook
 const mockUseKeypress = vi.hoisted(() => vi.fn());

packages/cli/src/ui/components/ModelSelectionDialog.tsx

@@ -4,7 +4,7 @@
  * SPDX-License-Identifier: Apache-2.0
  */
 
-import React from 'react';
+import type React from 'react';
 import { Box, Text } from 'ink';
 import { Colors } from '../colors.js';
 import {
@@ -12,7 +12,7 @@ import {
   type RadioSelectItem,
 } from './shared/RadioButtonSelect.js';
 import { useKeypress } from '../hooks/useKeypress.js';
-import { AvailableModel } from '../models/availableModels.js';
+import type { AvailableModel } from '../models/availableModels.js';
 
 export interface ModelSelectionDialogProps {
   availableModels: AvailableModel[];

packages/cli/src/ui/components/ModelSwitchDialog.tsx

@@ -4,7 +4,7 @@
  * SPDX-License-Identifier: Apache-2.0
  */
 
-import React from 'react';
+import type React from 'react';
 import { Box, Text } from 'ink';
 import { Colors } from '../colors.js';
 import {

packages/cli/src/ui/hooks/useGeminiStream.test.tsx

@@ -39,7 +39,7 @@ import type { LoadedSettings } from '../../config/settings.js';
 // --- MOCKS ---
 const mockSendMessageStream = vi
   .fn()
-  .mockReturnValue((async function* () { })());
+  .mockReturnValue((async function* () {})());
 const mockStartChat = vi.fn();
 
 const MockedGeminiClientClass = vi.hoisted(() =>
@@ -52,7 +52,7 @@ const MockedGeminiClientClass = vi.hoisted(() =>
 );
 
 const MockedUserPromptEvent = vi.hoisted(() =>
-  vi.fn().mockImplementation(() => { }),
+  vi.fn().mockImplementation(() => {}),
 );
 const mockParseAndFormatApiError = vi.hoisted(() => vi.fn());
 
@@ -237,7 +237,7 @@ describe('useGeminiStream', () => {
     } as unknown as any); // GeminiChat -> any
     mockSendMessageStream
       .mockClear()
-      .mockReturnValue((async function* () { })());
+      .mockReturnValue((async function* () {})());
     handleAtCommandSpy = vi.spyOn(atCommandProcessor, 'handleAtCommand');
   });
 
@@ -295,12 +295,12 @@ describe('useGeminiStream', () => {
           props.handleSlashCommand,
           props.shellModeActive,
           () => 'vscode' as EditorType,
-          () => { },
+          () => {},
           () => Promise.resolve(),
           false,
-          () => { },
+          () => {},
-          () => { },
+          () => {},
-          () => { },
+          () => {},
         );
       },
       {
@@ -456,12 +456,12 @@ describe('useGeminiStream', () => {
         mockHandleSlashCommand,
         false,
         () => 'vscode' as EditorType,
-        () => { },
+        () => {},
         () => Promise.resolve(),
         false,
-        () => { },
+        () => {},
-        () => { },
+        () => {},
-        () => { },
+        () => {},
       ),
     );
 
@@ -535,12 +535,12 @@ describe('useGeminiStream', () => {
         mockHandleSlashCommand,
         false,
         () => 'vscode' as EditorType,
-        () => { },
+        () => {},
         () => Promise.resolve(),
         false,
-        () => { },
+        () => {},
-        () => { },
+        () => {},
-        () => { },
+        () => {},
       ),
     );
 
@@ -643,12 +643,12 @@ describe('useGeminiStream', () => {
         mockHandleSlashCommand,
         false,
         () => 'vscode' as EditorType,
-        () => { },
+        () => {},
         () => Promise.resolve(),
         false,
-        () => { },
+        () => {},
-        () => { },
+        () => {},
-        () => { },
+        () => {},
       ),
     );
 
@@ -752,12 +752,12 @@ describe('useGeminiStream', () => {
         mockHandleSlashCommand,
         false,
         () => 'vscode' as EditorType,
-        () => { },
+        () => {},
         () => Promise.resolve(),
         false,
-        () => { },
+        () => {},
-        () => { },
+        () => {},
-        () => { },
+        () => {},
       ),
     );
 
@@ -813,7 +813,7 @@ describe('useGeminiStream', () => {
         if (options.isActive) {
           keypressCallback = callback;
         } else {
-          keypressCallback = () => { };
+          keypressCallback = () => {};
         }
       });
     });
@@ -828,7 +828,7 @@ describe('useGeminiStream', () => {
       const mockStream = (async function* () {
         yield { type: 'content', value: 'Part 1' };
         // Keep the stream open
-        await new Promise(() => { });
+        await new Promise(() => {});
       })();
       mockSendMessageStream.mockReturnValue(mockStream);
 
@@ -867,7 +867,7 @@ describe('useGeminiStream', () => {
       const mockStream = (async function* () {
         yield { type: 'content', value: 'Part 1' };
         // Keep the stream open
-        await new Promise(() => { });
+        await new Promise(() => {});
       })();
       mockSendMessageStream.mockReturnValue(mockStream);
 
@@ -881,11 +881,11 @@ describe('useGeminiStream', () => {
           mockHandleSlashCommand,
           false,
           () => 'vscode' as EditorType,
-          () => { },
+          () => {},
           () => Promise.resolve(),
           false,
-          () => { },
+          () => {},
-          () => { },
+          () => {},
           cancelSubmitSpy,
         ),
       );
@@ -1192,12 +1192,12 @@ describe('useGeminiStream', () => {
           mockHandleSlashCommand,
           false,
           () => 'vscode' as EditorType,
-          () => { },
+          () => {},
           mockPerformMemoryRefresh,
           false,
-          () => { },
+          () => {},
-          () => { },
+          () => {},
-          () => { },
+          () => {},
         ),
       );
 
@@ -1245,12 +1245,12 @@ describe('useGeminiStream', () => {
           mockHandleSlashCommand,
           false,
           () => 'vscode' as EditorType,
-          () => { },
+          () => {},
           () => Promise.resolve(),
           false,
-          () => { },
+          () => {},
-          () => { },
+          () => {},
-          () => { },
+          () => {},
         ),
       );
 
@@ -1295,12 +1295,12 @@ describe('useGeminiStream', () => {
           mockHandleSlashCommand,
           false,
           () => 'vscode' as EditorType,
-          () => { },
+          () => {},
           () => Promise.resolve(),
           false,
-          () => { },
+          () => {},
-          () => { },
+          () => {},
-          () => { },
+          () => {},
         ),
       );
 
@@ -1343,12 +1343,12 @@ describe('useGeminiStream', () => {
           mockHandleSlashCommand,
           false,
           () => 'vscode' as EditorType,
-          () => { },
+          () => {},
           () => Promise.resolve(),
           false,
-          () => { },
+          () => {},
-          () => { },
+          () => {},
-          () => { },
+          () => {},
         ),
       );
 
@@ -1392,12 +1392,12 @@ describe('useGeminiStream', () => {
           mockHandleSlashCommand,
           false,
           () => 'vscode' as EditorType,
-          () => { },
+          () => {},
           () => Promise.resolve(),
           false,
-          () => { },
+          () => {},
-          () => { },
+          () => {},
-          () => { },
+          () => {},
         ),
       );
 
@@ -1481,12 +1481,12 @@ describe('useGeminiStream', () => {
             mockHandleSlashCommand,
             false,
             () => 'vscode' as EditorType,
-            () => { },
+            () => {},
             () => Promise.resolve(),
             false,
-            () => { },
+            () => {},
-            () => { },
+            () => {},
-            () => { },
+            () => {},
           ),
         );
 
@@ -1596,12 +1596,12 @@ describe('useGeminiStream', () => {
           mockHandleSlashCommand,
           false,
           () => 'vscode' as EditorType,
-          () => { },
+          () => {},
           () => Promise.resolve(),
           false,
-          () => { },
+          () => {},
-          () => { },
+          () => {},
-          () => { },
+          () => {},
         ),
       );
 
@@ -1674,12 +1674,12 @@ describe('useGeminiStream', () => {
           mockHandleSlashCommand,
           false,
           () => 'vscode' as EditorType,
-          () => { },
+          () => {},
           () => Promise.resolve(),
           false,
-          () => { },
+          () => {},
-          () => { },
+          () => {},
-          () => { },
+          () => {},
         ),
       );
 
@@ -1728,12 +1728,12 @@ describe('useGeminiStream', () => {
           mockHandleSlashCommand,
           false,
           () => 'vscode' as EditorType,
-          () => { },
+          () => {},
           () => Promise.resolve(),
           false,
-          () => { },
+          () => {},
-          () => { },
+          () => {},
-          () => { },
+          () => {},
         ),
       );
 
@@ -1937,12 +1937,12 @@ describe('useGeminiStream', () => {
           mockHandleSlashCommand,
           false,
           () => 'vscode' as EditorType,
-          () => { },
+          () => {},
           () => Promise.resolve(),
           false,
-          () => { },
+          () => {},
-          () => { },
+          () => {},
-          () => { },
+          () => {},
         ),
       );
 
@@ -1969,12 +1969,12 @@ describe('useGeminiStream', () => {
           mockHandleSlashCommand,
           false,
           () => 'vscode' as EditorType,
-          () => { },
+          () => {},
           () => Promise.resolve(),
           false,
-          () => { },
+          () => {},
-          () => { },
+          () => {},
-          () => { },
+          () => {},
         ),
       );
 
@@ -2022,12 +2022,12 @@ describe('useGeminiStream', () => {
           mockHandleSlashCommand,
           false,
           () => 'vscode' as EditorType,
-          () => { },
+          () => {},
           () => Promise.resolve(),
           false,
-          () => { },
+          () => {},
-          () => { },
+          () => {},
-          () => { },
+          () => {},
         ),
       );
 
@@ -2059,12 +2059,12 @@ describe('useGeminiStream', () => {
           mockHandleSlashCommand,
           false,
           () => 'vscode' as EditorType,
-          () => { },
+          () => {},
           () => Promise.resolve(),
           false,
-          () => { },
+          () => {},
-          () => { },
+          () => {},
-          () => { },
+          () => {},
         ),
       );
 

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

@@ -7,8 +7,8 @@
 /* eslint-disable @typescript-eslint/no-explicit-any */
 import { describe, it, expect, vi, beforeEach } from 'vitest';
 import { renderHook, act } from '@testing-library/react';
-import { Part, PartListUnion } from '@google/genai';
+import type { Part, PartListUnion } from '@google/genai';
-import { AuthType, Config } from '@qwen-code/qwen-code-core';
+import { AuthType, type Config } from '@qwen-code/qwen-code-core';
 import {
   shouldOfferVisionSwitch,
   processVisionSwitchOutcome,

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

@@ -5,7 +5,7 @@
  */
 
 import { type PartListUnion, type Part } from '@google/genai';
-import { AuthType, Config } from '@qwen-code/qwen-code-core';
+import { AuthType, type Config } from '@qwen-code/qwen-code-core';
 import { useCallback, useRef } from 'react';
 import { VisionSwitchOutcome } from '../components/ModelSwitchDialog.js';
 import {
@@ -13,7 +13,7 @@ import {
   isVisionModel,
 } from '../models/availableModels.js';
 import { MessageType } from '../types.js';
-import { UseHistoryManagerReturn } from './useHistoryManager.js';
+import type { UseHistoryManagerReturn } from './useHistoryManager.js';
 
 /**
  * Checks if a PartListUnion contains image parts

packages/core/src/core/openaiContentGenerator/pipeline.ts

@@ -9,10 +9,10 @@ import {
   type GenerateContentParameters,
   GenerateContentResponse,
 } from '@google/genai';
-import { Config } from '../../config/config.js';
+import type { Config } from '../../config/config.js';
 import type { ContentGeneratorConfig } from '../contentGenerator.js';
 import type { OpenAICompatibleProvider } from './provider/index.js';
-import { OpenAIContentConverter } from './converter.js';
+import type { OpenAIContentConverter } from './converter.js';
 import type { TelemetryService, RequestContext } from './telemetryService.js';
 import type { ErrorHandler } from './errorHandler.js';
 

packages/core/src/utils/request-tokenizer/imageTokenizer.ts

@@ -4,7 +4,7 @@
  * SPDX-License-Identifier: Apache-2.0
  */
 
-import { ImageMetadata } from './types.js';
+import type { ImageMetadata } from './types.js';
 
 /**
  * Image tokenizer for calculating image tokens based on dimensions

packages/core/src/utils/request-tokenizer/requestTokenizer.test.ts

@@ -6,7 +6,7 @@
 
 import { describe, it, expect, beforeEach, afterEach } from 'vitest';
 import { DefaultRequestTokenizer } from './requestTokenizer.js';
-import { CountTokensParameters } from '@google/genai';
+import type { CountTokensParameters } from '@google/genai';
 
 describe('DefaultRequestTokenizer', () => {
   let tokenizer: DefaultRequestTokenizer;

packages/core/src/utils/request-tokenizer/requestTokenizer.ts

@@ -4,8 +4,13 @@
  * SPDX-License-Identifier: Apache-2.0
  */
 
-import { CountTokensParameters, Content, Part, PartUnion } from '@google/genai';
+import type {
-import {
+  CountTokensParameters,
+  Content,
+  Part,
+  PartUnion,
+} from '@google/genai';
+import type {
   RequestTokenizer,
   TokenizerConfig,
   TokenCalculationResult,

packages/core/src/utils/request-tokenizer/types.ts

@@ -4,7 +4,7 @@
  * SPDX-License-Identifier: Apache-2.0
  */
 
-import { CountTokensParameters } from '@google/genai';
+import type { CountTokensParameters } from '@google/genai';
 
 /**
  * Token calculation result for different content types