scriptoriumadmin/Roo-Code
1
0
Fork 0
mirror of https://github.com/RooVetGit/Roo-Code.git synced 2026-09-11 22:51:26 +00:00
Code Issues Projects Releases Packages Wiki Activity Actions

fix: resolve MCP server connection race condition in getTheme.ts

Browse source 
- Create local copy of vscode.extensions.all to avoid read-only property conflicts
- Add defensive null checking for extension and packageJSON
- Resolves race condition between MCP initialization and theme loading
- Reapplies fix from automatically closed PR #5592

Fixes #5591
This commit is contained in:
Roo Code 2025-07-18 17:31:49 +00:00
parent a6e16e80d9
commit 897a11ac36
1 changed files with 7 additions and 3 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

10
src/integrations/theme/getTheme.ts
View file

@ -37,12 +37,16 @@ export async function getTheme() {
const colorTheme = vscode.workspace.getConfiguration("workbench").get<string>("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)