[BUG] custom scheme is resulting with errors

[omryMen]

Problem (one or two sentences)

[ERROR]: Error loading color theme Expected property name or '}' in JSON at position 46 (line 4 column 9)
{
"stack": "SyntaxError: Expected property name or '}' in JSON at position 46 (line 4 column 9)\n\tat JSON.parse ()\n\tat Rei (/.../extension.js:665:9)\n\tat btt (/.../extension.js:665:559)"
}

Context (who is affected and when)

people who try install dracula pycharm theme

Reproduction steps

install Darcula PyCharm Theme
run the extension

Expected result

the extension should use the theme

Actual result

it throws errors

Variations tried (optional)

No response

App Version

v3.36.12

API Provider (optional)

None

Model Used (optional)

No response

Roo Code Task Links (optional)

No response

Relevant logs or errors (optional)

[ERROR]: Error loading color theme Expected property name or '}' in JSON at position 46 (line 4 column 9) 
{
  "stack": "SyntaxError: Expected property name or '}' in JSON at position 46 (line 4 column 9)\n\tat JSON.parse (<anonymous>)\n\tat Rei (/.../extension.js:665:9)\n\tat btt (/.../extension.js:665:559)"
}

[roomote]

I've analyzed this issue and identified the root cause.

Problem:
The error occurs in src/integrations/theme/getTheme.ts when parsing theme files. The current parseThemeString function only strips single-line comments (//) but does not handle:

1. Multi-line comments (/* ... */)
2. Trailing commas (e.g., { "key": "value", })

VSCode theme files use JSONC (JSON with Comments) format, which allows both of these. The Dracula PyCharm Theme likely uses trailing commas, causing the "Expected property name or '}'" JSON parse error.

Proposed Implementation Plan:

1. Modify the parseThemeString function in src/integrations/theme/getTheme.ts to:

   • Strip multi-line comments (/* ... */) in addition to single-line comments
   • Remove trailing commas before ] and } to make the JSON valid

2. Add proper error handling to gracefully fall back when a theme cannot be parsed, instead of throwing an unhandled error

3. Add unit tests to verify the fix handles:

   • Theme files with single-line comments
   • Theme files with multi-line comments
   • Theme files with trailing commas
   • Combination of all the above

This approach avoids adding new dependencies by using regex to clean the JSONC content before parsing.

Please mention @roomote in a comment saying that you approve this plan, and I'll proceed with the implementation.