fix: respect user-configured terminal integration timeout

Eric Wheeler · Eric Wheeler · commit 0701a9d57cff · 2025-05-22T22:30:49.000-07:00
The terminal integration timeout was being capped at the default value of 5 seconds due to a Math.min call in getShellIntegrationTimeout(), preventing users from setting longer timeouts. This caused shell integration to fail prematurely when initialization took longer than 5 seconds. This change removes the Math.min cap, allowing the user-configured timeout value to be respected. Fixes: #3885 Fixes: #3829 Regression from: #2820 Signed-off-by: Eric Wheeler <roo-code@z.ewheeler.org>
diff --git a/src/integrations/terminal/BaseTerminal.ts b/src/integrations/terminal/BaseTerminal.ts
@@ -170,7 +170,7 @@ export abstract class BaseTerminal implements RooTerminal {
 	}
 
 	public static getShellIntegrationTimeout(): number {
-		return Math.min(BaseTerminal.shellIntegrationTimeout, BaseTerminal.defaultShellIntegrationTimeout)
+		return BaseTerminal.shellIntegrationTimeout
 	}
 
 	public static setShellIntegrationDisabled(disabled: boolean): void {

Original file line number	Diff line number	Diff line change
`@@ -170,7 +170,7 @@ export abstract class BaseTerminal implements RooTerminal {`
`170`	`170`	`}`
`171`	`171`
`172`	`172`	`public static getShellIntegrationTimeout(): number {`
`173`		`- return Math.min(BaseTerminal.shellIntegrationTimeout, BaseTerminal.defaultShellIntegrationTimeout)`
	`173`	`+ return BaseTerminal.shellIntegrationTimeout`
`174`	`174`	`}`
`175`	`175`
`176`	`176`	`public static setShellIntegrationDisabled(disabled: boolean): void {`