Skip to content

fix: resolve memory leaks in terminal, notebook, and chat context dis…#310216

Open
InSelfControll wants to merge 1 commit intomicrosoft:mainfrom
InSelfControll:memory-leak-fixes
Open

fix: resolve memory leaks in terminal, notebook, and chat context dis…#310216
InSelfControll wants to merge 1 commit intomicrosoft:mainfrom
InSelfControll:memory-leak-fixes

Conversation

@InSelfControll
Copy link
Copy Markdown

Splits the original PR (#307853) into two focused PRs. This PR addresses memory leak fixes only.

Changes:

Terminal: Clean up link cache and cancellation sources on process exit
Terminal: Dispose instance disposables properly in terminalGroup
Terminal: Dispose backgrounded terminal disposables in terminalService
Notebook: Use DisposableStore instead of IDisposable[] for proper lifecycle
Chat context: Clean up chat context items on dispose

…posables

- Terminal: clean up link cache and cancellation sources on process exit
- Terminal: dispose instance and backgrounded terminal disposables properly
- Notebook: use DisposableStore instead of IDisposable[] for proper lifecycle
- Chat context: clean up chat context items on dispose
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants