cleanup

Author: Eric Oliver

src/api/providers/human-relay.ts

@@ -139,28 +139,33 @@ async function showHumanRelayDialog(promptText: string): Promise<string | undefi
 	}
 
 	// VSCode mode - use command-based dialog
-	return new Promise<string | undefined>(async (resolve) => {
+	return new Promise<string | undefined>((resolve) => {
 		// Create a unique request ID.
 		const requestId = Date.now().toString()
 
-		try {
-			const platformServices = await getPlatformServices()
-
-			// Register a global callback function.
-			await platformServices.commandExecutor.executeCommand(
-				getCommand("registerHumanRelayCallback"),
-				requestId,
-				(response: string | undefined) => resolve(response),
-			)
-
-			// Open the dialog box directly using the current panel.
-			await platformServices.commandExecutor.executeCommand(getCommand("showHumanRelayDialog"), {
-				requestId,
-				promptText,
-			})
-		} catch (error) {
-			console.error("Failed to show human relay dialog:", error)
-			resolve(undefined)
+		const setupPromise = async () => {
+			try {
+				const platformServices = await getPlatformServices()
+
+				// Register a global callback function.
+				await platformServices.commandExecutor.executeCommand(
+					getCommand("registerHumanRelayCallback"),
+					requestId,
+					(response: string | undefined) => resolve(response),
+				)
+
+				// Open the dialog box directly using the current panel.
+				await platformServices.commandExecutor.executeCommand(getCommand("showHumanRelayDialog"), {
+					requestId,
+					promptText,
+				})
+			} catch (error) {
+				console.error("Failed to show human relay dialog:", error)
+				resolve(undefined)
+			}
 		}
+
+		// Call the async setup function
+		setupPromise()
 	})
 }

src/cli/__mocks__/@roo-code/telemetry.js

@@ -1,3 +1,4 @@
+/* eslint-env node */
 // Mock telemetry module for CLI context
 
 class BaseTelemetryClient {
@@ -20,6 +21,7 @@ class MockTelemetryService {
 	captureSchemaValidationError() {}
 }
 
+// eslint-disable-next-line no-undef
 module.exports = {
 	BaseTelemetryClient,
 	TelemetryService: MockTelemetryService,

src/cli/__mocks__/vscode.js

@@ -1,4 +1,6 @@
+/* eslint-env node */
 // Mock VSCode module for CLI context
+// eslint-disable-next-line no-undef
 module.exports = {
 	workspace: {
 		getConfiguration: () => ({

src/core/adapters/PlatformServiceFactory.ts

@@ -21,14 +21,16 @@ export class PlatformServiceFactory {
 		this.context = context
 
 		switch (context) {
-			case PlatformContext.VSCode:
+			case PlatformContext.VSCode: {
 				const { VsCodePlatformServices } = await import("./vscode/VsCodePlatformServices")
 				this.instance = new VsCodePlatformServices(extensionName)
 				break
-			case PlatformContext.CLI:
+			}
+			case PlatformContext.CLI: {
 				const { CliPlatformServices } = await import("./cli/CliPlatformServices")
 				this.instance = new CliPlatformServices(extensionName, configPath)
 				break
+			}
 			default:
 				throw new Error(`Unsupported platform context: ${context}`)
 		}