[bug] Web crashes with unexpected error "e.indexOf is not a function" if a repository contains a folder called "constructor"

[olegvodyanov]

Describe the bug

Hi all.

The Sourcebot UI crashes while rendering the VS Code-style folder icon when a folder is named "constructor"
The icon resolver returns a non-string value (object) for this folder name, which then causes the runtime exception.

To reproduce

Create (or have) a folder named constructor in the tree/file browser.
Open the page that renders the folder tree.
Expand/navigate to the folder list so that the "constructor" folder is visible and the icon is rendered.

Sourcebot deployment information

Sourcebot version: v4.10.27

Additional information

No response

[imsarang]

will take this up!
@brendan-kellam @msukkari

[brendan-kellam]

@claude open a PR with a fix

[claude]

Claude finished @brendan-kellam's task in 1m 21s —— View job • claude/issue-861-20260206-2300 • Create PR ➔

---

Fix for "constructor" folder crash completed ✅

Todo List

• Find the icon resolver code for folder icons
• Identify the bug where "constructor" returns an object
• Implement a fix
• Test the fix (build command requires additional permissions)
• Commit and push changes
• Provide PR creation link

Root Cause

Found the issue in packages/web/src/app/components/vscodeFolderIcon.tsx:17. When the folder name is "constructor", the getIconForFolder function returns an object (likely Object.prototype.constructor) instead of a string. The code then tries to call .indexOf() on this object, causing the crash.

Fix Applied

Added a type guard typeof icon === 'string' to check that icon is a string before calling string methods on it. This prevents the crash for "constructor" and any other reserved property names.

Changes

• Modified packages/web/src/app/components/vscodeFolderIcon.tsx to add string type check on line 16

--- |