ClineProvider Instance cleanup

[xyOz-dev]

Description

significantly improves resource management by separating webview-specific disposables, preventing memory leaks and allowing for more efficient cleanup and reuse of ClineProvider instances

Type of Change

• 🐛 Bug Fix: Non-breaking change that fixes an issue.

---

Important

Improved resource management in ClineProvider by separating webview-specific disposables to prevent memory leaks and enhance cleanup efficiency.

• Resource Management:
  • Introduced webviewDisposables array in ClineProvider to manage webview-specific disposables separately from general disposables.
  • Added clearWebviewResources() method to dispose of all webview-specific disposables.
  • Updated dispose() method to call clearWebviewResources() for efficient cleanup.
• Code Changes:
  • Moved codeIndexStatusSubscription and other webview-related disposables to webviewDisposables in resolveWebviewView().
  • Updated event listeners in resolveWebviewView() to use webviewDisposables.
  • Adjusted onDidDispose event to clear webview resources for sidebar views without disposing the entire ClineProvider instance.

^{This description was created by for d9d40e8. You can customize this summary. It will automatically update as commits are pushed.}

[daniel-lxs]

I like this change, we still keep the recent fix of keeping the ClineProvider instance when moving Roo from one sidebar to the other, but the resources of the webview are properly disposed as well.

I tested it and everything seems to work and the resources are disposed correctly.

LGTM