fix(cli): fix ESLint unused variable and duplicate import warnings

- Remove unused variables across release files, components, hooks, and utils
- Consolidate duplicate imports in e2e test files
- Add proper type annotations to mock functions in test files
- Remove unused props from component interfaces (textAttributes, width, etc.)

Author: brandonkachen

cli/release-staging/index.js

@@ -114,21 +114,20 @@ function getCurrentVersion() {
   if (!fs.existsSync(CONFIG.binaryPath)) return null
 
   try {
-    return new Promise((resolve, reject) => {
+    return new Promise((resolve) => {
       const child = spawn(CONFIG.binaryPath, ['--version'], {
         cwd: CONFIG.packageDir,
         stdio: 'pipe',
       })
 
       let output = ''
-      let errorOutput = ''
 
       child.stdout.on('data', (data) => {
         output += data.toString()
       })
 
-      child.stderr.on('data', (data) => {
-        errorOutput += data.toString()
+      child.stderr.on('data', () => {
+        // Ignore stderr output
       })
 
       const timeout = setTimeout(() => {

cli/release/index.js

@@ -114,21 +114,20 @@ function getCurrentVersion() {
   if (!fs.existsSync(CONFIG.binaryPath)) return null
 
   try {
-    return new Promise((resolve, reject) => {
+    return new Promise((resolve) => {
       const child = spawn(CONFIG.binaryPath, ['--version'], {
         cwd: os.homedir(),
         stdio: 'pipe',
       })
 
       let output = ''
-      let errorOutput = ''
 
       child.stdout.on('data', (data) => {
         output += data.toString()
       })
 
-      child.stderr.on('data', (data) => {
-        errorOutput += data.toString()
+      child.stderr.on('data', () => {
+        // Ignore stderr output
       })
 
       const timeout = setTimeout(() => {

cli/release/postinstall.js

@@ -1,8 +1,8 @@
 #!/usr/bin/env node
 
 const fs = require('fs');
-const path = require('path');
 const os = require('os');
+const path = require('path');
 
 // Clean up old binary
 const binaryPath = path.join(

cli/scripts/build-binary.ts

@@ -11,9 +11,9 @@ import {
   rmSync,
   writeFileSync,
 } from 'fs'
+import { tmpdir } from 'os'
 import { dirname, join } from 'path'
 import { fileURLToPath } from 'url'
-import { tmpdir } from 'os'
 
 type TargetInfo = {
   bunTarget: string

cli/src/__tests__/bash-mode.test.ts

@@ -21,8 +21,8 @@ import type { InputMode } from '../utils/input-modes'
 describe('bash-mode', () => {
   describe('entering bash mode', () => {
     test('typing exactly "!" enters bash mode and clears input', () => {
-      const setInputMode = mock((_mode: InputMode) => {})
-      const setInputValue = mock((_value: any) => {})
+      const setInputMode = mock(() => {})
+      const setInputValue = mock(() => {})
 
       // Simulate user typing '!'
       const inputValue = {
@@ -50,8 +50,8 @@ describe('bash-mode', () => {
     })
 
     test('typing "!ls" does NOT enter bash mode (not exactly "!")', () => {
-      const setInputMode = mock((_mode: InputMode) => {})
-      const setInputValue = mock((_value: any) => {})
+      const setInputMode = mock(() => {})
+      const setInputValue = mock(() => {})
 
       // Simulate user typing '!ls'
       const inputValue = {
@@ -78,15 +78,15 @@ describe('bash-mode', () => {
     })
 
     test('typing "!" when already in bash mode does nothing special', () => {
-      const setInputMode = mock((_mode: InputMode) => {})
-      const setInputValue = mock((_value: any) => {})
+      const setInputMode = mock(() => {})
+      const setInputValue = mock(() => {})
 
       const inputValue = {
         text: '!',
         cursorPosition: 1,
         lastEditDueToNav: false,
       }
-      let inputMode = 'bash' as InputMode
+      const inputMode = 'bash' as InputMode
 
       const userTypedBang = inputMode === ('default' as InputMode) && inputValue.text === '!'
 
@@ -108,7 +108,7 @@ describe('bash-mode', () => {
 
   describe('exiting bash mode', () => {
     test('backspace at cursor position 0 exits bash mode', () => {
-      const setInputMode = mock((_mode: InputMode) => {})
+      const setInputMode = mock(() => {})
 
       // Simulate backspace key press in bash mode at cursor position 0
       const inputMode: InputMode = 'bash'
@@ -128,7 +128,7 @@ describe('bash-mode', () => {
     })
 
     test('backspace at cursor position 0 with non-empty input DOES exit bash mode', () => {
-      const setInputMode = mock((_mode: InputMode) => {})
+      const setInputMode = mock(() => {})
 
       const inputMode: InputMode = 'bash'
       const cursorPosition = 0
@@ -147,7 +147,7 @@ describe('bash-mode', () => {
     })
 
     test('backspace at cursor position > 0 does NOT exit bash mode', () => {
-      const setInputMode = mock((_mode: InputMode) => {})
+      const setInputMode = mock(() => {})
 
       const inputMode: InputMode = 'bash'
       const cursorPosition: number = 2
@@ -166,7 +166,7 @@ describe('bash-mode', () => {
     })
 
     test('other keys at cursor position 0 do NOT exit bash mode', () => {
-      const setInputMode = mock((_mode: InputMode) => {})
+      const setInputMode = mock(() => {})
 
       const inputMode: InputMode = 'bash'
       const cursorPosition = 0
@@ -185,9 +185,9 @@ describe('bash-mode', () => {
     })
 
     test('backspace when NOT in bash mode does nothing to bash mode', () => {
-      const setInputMode = mock((_mode: InputMode) => {})
+      const setInputMode = mock(() => {})
 
-      let inputMode = 'default' as InputMode
+      const inputMode = 'default' as InputMode
       const cursorPosition = 0
       const key = { name: 'backspace' }
 
@@ -217,12 +217,10 @@ describe('bash-mode', () => {
     })
 
     test('normal mode input can contain "!" anywhere', () => {
-      const inputMode: InputMode = 'default'
       const inputValue = 'fix this bug!'
 
       // In normal mode, '!' is just a regular character
       expect(inputValue).toContain('!')
-      expect(inputMode).toBe('default')
     })
   })
 
@@ -249,8 +247,7 @@ describe('bash-mode', () => {
     })
 
     test('submission saves command WITH "!" to history', () => {
-      const saveToHistory = mock((_cmd: string) => {})
-      const inputMode: InputMode = 'bash'
+      const saveToHistory = mock(() => {})
       const trimmedInput = 'git status'
       const commandWithBang = '!' + trimmedInput
 
@@ -261,7 +258,7 @@ describe('bash-mode', () => {
     })
 
     test('submission exits bash mode after running command', () => {
-      const setInputMode = mock((_mode: InputMode) => {})
+      const setInputMode = mock(() => {})
 
       // After submission, bash mode should be exited
       setInputMode('default')
@@ -270,7 +267,7 @@ describe('bash-mode', () => {
     })
 
     test('terminal command receives value WITHOUT "!" prefix', () => {
-      const runTerminalCommand = mock((_params: any) =>
+      const runTerminalCommand = mock(() =>
         Promise.resolve([{ value: { stdout: 'output' } }]),
       )
       const trimmedInput = 'echo hello'
@@ -293,8 +290,8 @@ describe('bash-mode', () => {
   describe('bash mode UI state', () => {
     test('input mode is stored separately from input value', () => {
       // The inputMode is independent of the input text
-      const state1 = { inputMode: 'bash' as InputMode, inputValue: 'ls' }
-      const state2 = { inputMode: 'default' as InputMode, inputValue: 'hello' }
+      const state1: { inputMode: InputMode; inputValue: string } = { inputMode: 'bash', inputValue: 'ls' }
+      const state2: { inputMode: InputMode; inputValue: string } = { inputMode: 'default', inputValue: 'hello' }
 
       expect(state1.inputMode).toBe('bash')
       expect(state1.inputValue).not.toContain('!')
@@ -305,21 +302,21 @@ describe('bash-mode', () => {
 
     test('input width is adjusted in bash mode for "!" column', () => {
       const baseInputWidth = 100
-      const inputMode: InputMode = 'bash'
+      const inputModeValue: InputMode = 'bash'
 
       // Width should be reduced by 2 to account for '!' and spacing
       const adjustedInputWidth =
-        inputMode === 'bash' ? baseInputWidth - 2 : baseInputWidth
+        inputModeValue === 'bash' ? baseInputWidth - 2 : baseInputWidth
 
       expect(adjustedInputWidth).toBe(98)
     })
 
     test('input width is NOT adjusted when not in bash mode', () => {
       const baseInputWidth = 100
-      let inputMode = 'default' as InputMode
+      const inputModeValue = 'default' as InputMode
 
       const adjustedInputWidth =
-        inputMode === ('bash' as InputMode) ? baseInputWidth - 2 : baseInputWidth
+        inputModeValue === ('bash' as InputMode) ? baseInputWidth - 2 : baseInputWidth
 
       expect(adjustedInputWidth).toBe(100)
     })
@@ -338,7 +335,7 @@ describe('bash-mode', () => {
     test('placeholder is normal when not in bash mode', () => {
       const normalPlaceholder = 'Ask Buffy anything...'
       const bashPlaceholder = 'enter bash command...'
-      let inputMode = 'default' as InputMode
+      const inputMode = 'default' as InputMode
 
       const effectivePlaceholder =
         inputMode === ('bash' as InputMode) ? bashPlaceholder : normalPlaceholder
@@ -408,7 +405,7 @@ describe('bash-mode', () => {
     })
 
     test('normal commands starting with "!" are NOT bash commands', () => {
-      let inputMode = 'default' as InputMode
+      const inputMode = 'default' as InputMode
       const inputValue = '!ls' // User typed this in normal mode
 
       // This should be treated as a normal prompt, not a bash command
@@ -418,7 +415,7 @@ describe('bash-mode', () => {
     })
 
     test('bash mode takes precedence over slash commands', () => {
-      let inputMode = 'bash' as InputMode
+      const inputMode = 'bash' as InputMode
       const trimmedInput = '/help' // Looks like a slash command
 
       // But in bash mode, it's just a bash command

cli/src/__tests__/e2e-cli.test.ts

@@ -1,7 +1,10 @@
-import { describe, test, expect } from 'bun:test'
 import { spawn } from 'child_process'
-import stripAnsi from 'strip-ansi'
 import path from 'path'
+
+import { describe, test, expect } from 'bun:test'
+import stripAnsi from 'strip-ansi'
+
+
 import { isSDKBuilt, ensureCliTestEnv } from './test-utils'
 
 const CLI_PATH = path.join(__dirname, '../index.tsx')

cli/src/__tests__/e2e/logout-relogin-flow.test.ts

@@ -1,3 +1,7 @@
+import fs from 'fs'
+import os from 'os'
+import path from 'path'
+
 import {
   describe,
   test,
@@ -7,17 +11,17 @@ import {
   mock,
   spyOn,
 } from 'bun:test'
-import fs from 'fs'
-import os from 'os'
-import path from 'path'
 
 import {
   saveUserCredentials,
   getUserCredentials,
   logoutUser,
-  type User,
 } from '../../utils/auth'
 
+import type * as AuthModule from '../../utils/auth'
+
+type User = AuthModule.User
+
 const ORIGINAL_USER: User = {
   id: 'user-001',
   name: 'CLI Tester',
@@ -49,8 +53,7 @@ describe('Logout and Re-login helpers', () => {
   })
 
   const mockConfigPaths = () => {
-    const authModule =
-      require('../../utils/auth') as typeof import('../../utils/auth')
+    const authModule = require('../../utils/auth') as typeof AuthModule
     spyOn(authModule, 'getConfigDir').mockReturnValue(tempConfigDir)
     spyOn(authModule, 'getCredentialsPath').mockReturnValue(
       path.join(tempConfigDir, 'credentials.json'),

cli/src/__tests__/e2e/returning-user-auth.test.ts

@@ -13,16 +13,19 @@ import {
   spyOn,
 } from 'bun:test'
 
+
 import { validateApiKey } from '../../hooks/use-auth-query'
 import {
   getAuthTokenDetails,
   saveUserCredentials,
-  type User,
 } from '../../utils/auth'
 
+import type * as AuthModule from '../../utils/auth'
 import type { GetUserInfoFromApiKeyFn } from '@codebuff/common/types/contracts/database'
 import type { Logger } from '@codebuff/common/types/contracts/logger'
 
+type User = AuthModule.User
+
 const RETURNING_USER: User = {
   id: 'returning-user-456',
   name: 'Returning User',
@@ -59,8 +62,7 @@ describe('Returning User Authentication helpers', () => {
   })
 
   test('should load auth token from credentials file for returning user', () => {
-    const authModule =
-      require('../../utils/auth') as typeof import('../../utils/auth')
+    const authModule = require('../../utils/auth') as typeof AuthModule
 
     spyOn(authModule, 'getConfigDir').mockReturnValue(tempConfigDir)
     spyOn(authModule, 'getCredentialsPath').mockReturnValue(
@@ -75,8 +77,7 @@ describe('Returning User Authentication helpers', () => {
   })
 
   test('should fall back to CODEBUFF_API_KEY when credentials are missing', () => {
-    const authModule =
-      require('../../utils/auth') as typeof import('../../utils/auth')
+    const authModule = require('../../utils/auth') as typeof AuthModule
 
     spyOn(authModule, 'getConfigDir').mockReturnValue(tempConfigDir)
     spyOn(authModule, 'getCredentialsPath').mockReturnValue(
@@ -91,8 +92,7 @@ describe('Returning User Authentication helpers', () => {
   })
 
   test('should validate stored credentials without blocking the UI thread', async () => {
-    const authModule =
-      require('../../utils/auth') as typeof import('../../utils/auth')
+    const authModule = require('../../utils/auth') as typeof AuthModule
 
     spyOn(authModule, 'getConfigDir').mockReturnValue(tempConfigDir)
     spyOn(authModule, 'getCredentialsPath').mockReturnValue(

cli/src/__tests__/helpers/mock-api-client.ts

@@ -1,6 +1,6 @@
 import { mock } from 'bun:test'
 
-import type { CodebuffApiClient, ApiResponse } from '../../utils/codebuff-api'
+import type { CodebuffApiClient } from '../../utils/codebuff-api'
 
 export interface MockApiClientOverrides {
   get?: ReturnType<typeof mock>

cli/src/__tests__/integration-tmux.test.ts

@@ -1,7 +1,10 @@
-import { describe, test, expect, beforeAll } from 'bun:test'
 import { spawn } from 'child_process'
-import stripAnsi from 'strip-ansi'
 import path from 'path'
+
+import { describe, test, expect, beforeAll } from 'bun:test'
+import stripAnsi from 'strip-ansi'
+
+
 import {
   isTmuxAvailable,
   isSDKBuilt,