Skip to content

fix: memory leak in ChatInputPart#257082

Merged
deepak1556 merged 1 commit intomicrosoft:mainfrom
SimonSiefke:fix/memory-leak-chat-input
Jul 24, 2025
Merged

fix: memory leak in ChatInputPart#257082
deepak1556 merged 1 commit intomicrosoft:mainfrom
SimonSiefke:fix/memory-leak-chat-input

Conversation

@SimonSiefke
Copy link
Contributor

Registers the _contextResourceLabels disposable to dispose it when chatInputPart is disposed.

@vs-code-engineering vs-code-engineering bot added this to the July 2025 milestone Jul 21, 2025
Copy link
Member

@roblourens roblourens left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks!

@deepak1556 deepak1556 enabled auto-merge (squash) July 24, 2025 22:23
@deepak1556 deepak1556 merged commit e50b55c into microsoft:main Jul 24, 2025
17 checks passed
@vs-code-engineering vs-code-engineering bot locked and limited conversation to collaborators Sep 7, 2025
@SimonSiefke SimonSiefke deleted the fix/memory-leak-chat-input branch January 15, 2026 15:59
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants