From 081380f4702a2b7dca900dc6f240fdd8f4533091 Mon Sep 17 00:00:00 2001 From: Roo Code Date: Fri, 11 Jul 2025 06:54:01 +0000 Subject: [PATCH] fix: resolve MCP server connection race condition (#5591) - Fix 'Cannot assign to read only property all' error during extension startup - Create local copy of vscode.extensions.all to avoid read-only property conflicts - Add defensive null checking for better error handling - Resolves race condition between MCP initialization and theme loading Fixes #5591 --- src/integrations/theme/getTheme.ts | 10 +++++++--- 1 file changed, 7 insertions(+), 3 deletions(-) diff --git a/src/integrations/theme/getTheme.ts b/src/integrations/theme/getTheme.ts index 20171cc3045..7270f284aca 100644 --- a/src/integrations/theme/getTheme.ts +++ b/src/integrations/theme/getTheme.ts @@ -37,12 +37,16 @@ export async function getTheme() { const colorTheme = vscode.workspace.getConfiguration("workbench").get("colorTheme") || "Default Dark Modern" try { - for (let i = vscode.extensions.all.length - 1; i >= 0; i--) { + // Create a local copy of the extensions array to avoid potential conflicts with the read-only property + const extensions = [...vscode.extensions.all] + + for (let i = extensions.length - 1; i >= 0; i--) { if (currentTheme) { break } - const extension = vscode.extensions.all[i] - if (extension.packageJSON?.contributes?.themes?.length > 0) { + const extension = extensions[i] + // Add null check for extension and packageJSON + if (extension?.packageJSON?.contributes?.themes?.length > 0) { for (const theme of extension.packageJSON.contributes.themes) { if (theme.label === colorTheme) { const themePath = path.join(extension.extensionPath, theme.path)