[danialsamiei]

Fix this-binding: use arrow functions in web LLM service

Summary

Converts 4 methods in LLMMessageServiceWeb from regular class methods to arrow function properties. RefreshModelService.startRefreshingModels extracts these methods into a standalone variable (const listFn = this.llmMessageService.openAICompatibleList) and calls them directly, which loses the this context. This caused a runtime crash:

TypeError: Cannot read properties of undefined (reading 'voidSettingsService')
    at openAICompatibleList (sendLLMMessageService.js:268:49)
    at RefreshModelService.startRefreshingModels (refreshModelService.js:66:13)

The fix is purely mechanical — method(args) {} → method = (args) => {} — for sendLLMMessage, abort, ollamaList, and openAICompatibleList.

Review & Testing Checklist for Human

• Deploy and verify no voidSettingsService TypeError in browser console. Open https://ide.orcest.ai/, open DevTools console, and confirm the Cannot read properties of undefined error is gone.
• Test that the Settings panel opens. Click the gear icon or "Void's Settings" link and confirm it renders without errors.

Notes

• The caller pattern in refreshModelService.ts:164-167 is pre-existing and used in Electron mode too, but Electron's LLMMessageService methods use IPC channels that don't reference this, so the bug only manifests in the web stub.
• Arrow function properties are per-instance rather than on the prototype, which is fine for a singleton service.

Link to Devin run: https://app.devin.ai/sessions/9a3910ef8208402b93e14fa38bb90f2b
Requested by: @danialsamiei

[devin-ai-integration]

🤖 Devin AI Engineer

I'll be helping with this pull request! Here's what you should know:

✅ I will automatically:

• Address comments on this PR. Add '(aside)' to your comment to have me ignore it.
• Look at CI failures and help fix them

Note: I can only respond to comments from users who have write access to this repository.

⚙️ Control Options:

• Disable automatic comment and CI monitoring