test(timeout): update with new value

Author: elianiva

src/api/providers/__tests__/lm-studio-timeout.spec.ts

@@ -72,7 +72,7 @@ describe("LmStudioHandler timeout configuration", () => {
 		)
 	})
 
-	it("should handle zero timeout (no timeout)", () => {
+	it("should handle zero timeout", () => {
 		;(getApiRequestTimeout as any).mockReturnValue(0)
 
 		const options: ApiHandlerOptions = {
@@ -84,7 +84,7 @@ describe("LmStudioHandler timeout configuration", () => {
 
 		expect(mockOpenAIConstructor).toHaveBeenCalledWith(
 			expect.objectContaining({
-				timeout: 0, // No timeout
+				timeout: 0,
 			}),
 		)
 	})

src/api/providers/__tests__/ollama-timeout.spec.ts

@@ -84,7 +84,7 @@ describe("OllamaHandler timeout configuration", () => {
 
 		expect(mockOpenAIConstructor).toHaveBeenCalledWith(
 			expect.objectContaining({
-				timeout: 0, // No timeout
+				timeout: 0,
 			}),
 		)
 	})

src/api/providers/__tests__/openai-timeout.spec.ts

@@ -137,7 +137,7 @@ describe("OpenAiHandler timeout configuration", () => {
 
 		expect(mockOpenAIConstructor).toHaveBeenCalledWith(
 			expect.objectContaining({
-				timeout: 0, // No timeout
+				timeout: 0,
 			}),
 		)
 	})

src/api/providers/utils/timeout-config.ts

@@ -1,5 +1,13 @@
 import * as vscode from "vscode"
 
+// Use 2147483647 (2^31 - 1) as the maximum timeout value for setTimeout
+// JavaScript's setTimeout has a maximum delay limit of 2147483647ms (32-bit signed integer max)
+// Values larger than this may be clamped to 1ms or cause unexpected behavior
+// 2147483647 is the safe maximum value that won't cause issues
+export const MAX_TIMEOUT_MS = 2147483647
+
+const DEFAULT_TIMEOUT_MS = 600 * 1000
+
 /**
  * Gets the API request timeout from VSCode configuration with validation.
  *
@@ -11,7 +19,7 @@ export function getApiRequestTimeout(): number {
 
 	// Validate that it's actually a number and not NaN
 	if (typeof configTimeout !== "number" || isNaN(configTimeout)) {
-		return 600 * 1000 // Default to 600 seconds
+		return DEFAULT_TIMEOUT_MS
 	}
 
 	// Allow 0 (no timeout) but clamp negative values to 0
@@ -21,12 +29,8 @@ export function getApiRequestTimeout(): number {
 	const timeoutMs = timeoutSeconds * 1000
 
 	// Handle the special case where 0 means "no timeout"
-	// Use 2147483647 (2^31 - 1) as the maximum timeout value for setTimeout
-	// JavaScript's setTimeout has a maximum delay limit of 2147483647ms (32-bit signed integer max)
-	// Values larger than this may be clamped to 1ms or cause unexpected behavior
-	// 2147483647 is the safe maximum value that won't cause issues
 	if (timeoutMs === 0) {
-		return 2147483647
+		return MAX_TIMEOUT_MS
 	}
 
 	return timeoutMs