Update max retry delay to 600 since this is what's set by MAX_EXPONENTIAL_BACKOFF_SECONDS

Author: nonsleepr

packages/types/src/global-settings.ts

@@ -190,7 +190,7 @@ export const EVALS_SETTINGS: RooCodeSettings = {
 	alwaysAllowBrowser: true,
 	alwaysApproveResubmit: true,
 	requestDelaySeconds: 5,
-	maxRequestDelaySeconds: 100,
+	maxRequestDelaySeconds: 600,
 	alwaysAllowMcp: true,
 	alwaysAllowModeSwitch: true,
 	alwaysAllowSubtasks: true,

src/core/task/Task.ts

@@ -1807,10 +1807,8 @@ export class Task extends EventEmitter<ClineEvents> {
 				const minDelay = requestDelaySeconds ?? 5
 				const maxDelay = maxRequestDelaySeconds ?? MAX_EXPONENTIAL_BACKOFF_SECONDS
 
-				// Use the minimum delay as the base for exponential backoff
 				let exponentialDelay = Math.ceil(minDelay * Math.pow(2, retryAttempt))
 
-				// Clamp exponential delay to the configured range
 				exponentialDelay = Math.max(minDelay, Math.min(exponentialDelay, maxDelay))
 
 				// If the error is a 429, and the error details contain a retry delay, use that delay instead of exponential backoff

src/core/webview/ClineProvider.ts

@@ -1479,7 +1479,7 @@ export class ClineProvider
 			enableMcpServerCreation: enableMcpServerCreation ?? true,
 			alwaysApproveResubmit: alwaysApproveResubmit ?? false,
 			requestDelaySeconds: requestDelaySeconds ?? 5,
-			maxRequestDelaySeconds: maxRequestDelaySeconds ?? 100,
+			maxRequestDelaySeconds: maxRequestDelaySeconds ?? 600,
 			currentApiConfigName: currentApiConfigName ?? "default",
 			listApiConfigMeta: listApiConfigMeta ?? [],
 			pinnedApiConfigs: pinnedApiConfigs ?? {},
@@ -1644,7 +1644,7 @@ export class ClineProvider
 			enableMcpServerCreation: stateValues.enableMcpServerCreation ?? true,
 			alwaysApproveResubmit: stateValues.alwaysApproveResubmit ?? false,
 			requestDelaySeconds: Math.max(1, stateValues.requestDelaySeconds ?? 5),
-			maxRequestDelaySeconds: Math.min(100, stateValues.maxRequestDelaySeconds ?? 100),
+			maxRequestDelaySeconds: Math.min(600, stateValues.maxRequestDelaySeconds ?? 600),
 			currentApiConfigName: stateValues.currentApiConfigName ?? "default",
 			listApiConfigMeta: stateValues.listApiConfigMeta ?? [],
 			pinnedApiConfigs: stateValues.pinnedApiConfigs ?? {},

src/core/webview/__tests__/ClineProvider.spec.ts

@@ -521,7 +521,7 @@ describe("ClineProvider", () => {
 			mcpEnabled: true,
 			enableMcpServerCreation: false,
 			requestDelaySeconds: 5,
-			maxRequestDelaySeconds: 100,
+			maxRequestDelaySeconds: 600,
 			mode: defaultModeSlug,
 			customModes: [],
 			experiments: experimentDefault,
@@ -814,7 +814,7 @@ describe("ClineProvider", () => {
 		expect(state.requestDelaySeconds).toBe(5)
 	})
 
-	test("maxRequestDelaySeconds defaults to 100 seconds", async () => {
+	test("maxRequestDelaySeconds defaults to 600 seconds", async () => {
 		// Mock globalState.get to return undefined for requestDelaySeconds
 		;(mockContext.globalState.get as any).mockImplementation((key: string) => {
 			if (key === "maxRequestDelaySeconds") {
@@ -824,7 +824,7 @@ describe("ClineProvider", () => {
 		})
 
 		const state = await provider.getState()
-		expect(state.maxRequestDelaySeconds).toBe(100)
+		expect(state.maxRequestDelaySeconds).toBe(600)
 	})
 
 	test("alwaysApproveResubmit defaults to false", async () => {
@@ -1021,8 +1021,8 @@ describe("ClineProvider", () => {
 		expect(mockPostMessage).toHaveBeenCalled()
 
 		// Test maxRequestDelaySeconds
-		await messageHandler({ type: "maxRequestDelaySeconds", value: 100 })
-		expect(mockContext.globalState.update).toHaveBeenCalledWith("maxRequestDelaySeconds", 100)
+		await messageHandler({ type: "maxRequestDelaySeconds", value: 600 })
+		expect(mockContext.globalState.update).toHaveBeenCalledWith("maxRequestDelaySeconds", 600)
 		expect(mockPostMessage).toHaveBeenCalled()
 	})
 

src/core/webview/webviewMessageHandler.ts

@@ -863,7 +863,7 @@ export const webviewMessageHandler = async (
 			await provider.postStateToWebview()
 			break
 		case "maxRequestDelaySeconds":
-			await updateGlobalState("maxRequestDelaySeconds", Math.max(1, message.value ?? 100))
+			await updateGlobalState("maxRequestDelaySeconds", Math.max(1, message.value ?? 600))
 			await provider.postStateToWebview()
 			break
 		case "writeDelayMs":

webview-ui/src/components/settings/AutoApproveSettings.tsx

@@ -197,7 +197,7 @@ export const AutoApproveSettings = ({
 							<div className="flex items-center gap-2">
 								<Slider
 									min={1}
-									max={100}
+									max={600}
 									step={1}
 									value={[requestDelaySeconds, maxRequestDelaySeconds]}
 									onValueChange={([min, max]) => {

webview-ui/src/context/ExtensionStateContext.tsx

@@ -179,7 +179,7 @@ export const ExtensionStateContextProvider: React.FC<{ children: React.ReactNode
 		enableMcpServerCreation: false,
 		alwaysApproveResubmit: false,
 		requestDelaySeconds: 5,
-		maxRequestDelaySeconds: 100,
+		maxRequestDelaySeconds: 600,
 		currentApiConfigName: "default",
 		listApiConfigMeta: [],
 		mode: defaultModeSlug,

webview-ui/src/context/__tests__/ExtensionStateContext.spec.tsx

@@ -191,7 +191,7 @@ describe("mergeExtensionState", () => {
 			enableCheckpoints: true,
 			writeDelayMs: 1000,
 			requestDelaySeconds: 5,
-			maxRequestDelaySeconds: 100,
+			maxRequestDelaySeconds: 600,
 			mode: "default",
 			experiments: {} as Record<ExperimentId, boolean>,
 			customModes: [],