From ad26adcae58a978b76df0f49ba0733512fe4cb17 Mon Sep 17 00:00:00 2001 From: Roo Code Date: Thu, 31 Jul 2025 02:43:04 +0000 Subject: [PATCH] fix: make connection properties optional in provider settings schema - Remove default values from connection properties to make them truly optional - Fixes TypeScript compilation errors in e2e tests that use partial RooCodeSettings objects - Connection properties will use fallback defaults in Task.ts implementation --- packages/types/src/provider-settings.ts | 10 +++++----- 1 file changed, 5 insertions(+), 5 deletions(-) diff --git a/packages/types/src/provider-settings.ts b/packages/types/src/provider-settings.ts index 2b904196e0..d8df33ed1a 100644 --- a/packages/types/src/provider-settings.ts +++ b/packages/types/src/provider-settings.ts @@ -78,11 +78,11 @@ const baseProviderSettingsSchema = z.object({ modelMaxThinkingTokens: z.number().optional(), // Enterprise network configuration for connection reliability - connectionKeepAliveEnabled: z.boolean().optional().default(true), - connectionKeepAliveInterval: z.number().min(5000).max(300000).optional().default(30000), // 5s to 5min, default 30s - connectionRetryEnabled: z.boolean().optional().default(true), - connectionMaxRetries: z.number().min(0).max(10).optional().default(3), - connectionRetryBaseDelay: z.number().min(1000).max(30000).optional().default(2000), // 1s to 30s, default 2s + connectionKeepAliveEnabled: z.boolean().optional(), + connectionKeepAliveInterval: z.number().min(5000).max(300000).optional(), + connectionRetryEnabled: z.boolean().optional(), + connectionMaxRetries: z.number().min(0).max(10).optional(), + connectionRetryBaseDelay: z.number().min(1000).max(30000).optional(), }) // Several of the providers share common model config properties.