Fix lint errors: import types, generic constructors, unused imports, nullish coalescing

Author: ammar-agent

src/browser/api.test.ts

@@ -101,7 +101,6 @@ describe("Browser API invokeIPC", () => {
 
     const invokeIPC = createInvokeIPC(mockFetch);
 
-    // eslint-disable-next-line @typescript-eslint/await-thenable
     await expect(invokeIPC("WORKSPACE_REMOVE", "test-workspace", { force: false })).rejects.toThrow(
       "HTTP error! status: 500"
     );

src/services/aiService.ts

@@ -12,7 +12,7 @@ import type { CmuxMessage, CmuxTextPart } from "@/types/message";
 import { createCmuxMessage } from "@/types/message";
 import type { Config } from "@/config";
 import { StreamManager } from "./streamManager";
-import { InitStateManager } from "./initStateManager";
+import type { InitStateManager } from "./initStateManager";
 import type { SendMessageError } from "@/types/errors";
 import { getToolsForModel } from "@/utils/tools/tools";
 import { createRuntime } from "@/runtime/runtimeFactory";

src/services/initStateManager.ts

@@ -65,10 +65,10 @@ export class InitStateManager extends EventEmitter {
    * Each running init has a promise that resolves when endInit() is called.
    * Multiple tools can await the same promise without race conditions.
    */
-  private readonly initPromises: Map<
+  private readonly initPromises = new Map<
     string,
     { promise: Promise<void>; resolve: () => void; reject: (error: Error) => void }
-  > = new Map();
+  >();
 
   constructor(config: Config) {
     super();

src/services/tools/file_edit_operation.test.ts

@@ -1,7 +1,6 @@
 import { describe, test, expect, jest } from "@jest/globals";
 import { executeFileEditOperation } from "./file_edit_operation";
 import { WRITE_DENIED_PREFIX } from "@/types/tools";
-import { createRuntime } from "@/runtime/runtimeFactory";
 import type { Runtime } from "@/runtime/Runtime";
 
 import { createTestToolConfig, getTestDeps } from "./testHelpers";

src/services/tools/testHelpers.ts

@@ -35,16 +35,12 @@ let testConfig: Config | null = null;
 let testInitStateManager: InitStateManager | null = null;
 
 function getTestConfig(): Config {
-  if (!testConfig) {
-    testConfig = new Config();
-  }
+  testConfig ??= new Config();
   return testConfig;
 }
 
 function getTestInitStateManager(): InitStateManager {
-  if (!testInitStateManager) {
-    testInitStateManager = new InitStateManager(getTestConfig());
-  }
+  testInitStateManager ??= new InitStateManager(getTestConfig());
   return testInitStateManager;
 }