🪲 BUG-#8: Kill child process on extension deactivate

[FernandoCelmer]

Description

• src/extension.ts: Added module-level activeProvider reference, assigned in activate(), and wired into deactivate() to call ChatViewProvider.dispose().
• src/vscode/sidebar/chatSidebarProvider.ts: Added dispose() method that forwards to ChatController.dispose() (already idempotent).

Motivation and Context

deactivate() was a no-op. The pycodeloop serve child process was only killed via webviewView.onDidDispose(), which does not fire (or fires too late) when the webview is hidden at the moment the extension host tears down (window close, reload, extension disable), leaving the process orphaned.

Closes #8

Types of changes

• Bug fix (change that fixes an issue)
• New feature (change which adds functionality)
• Documentation

Checklist

• I have performed a self-review of my own code
• I have added tests that prove my fix is effective or that my feature works
• I have updated the CHANGELOG
• I have updated the documentation accordingly

[FernandoCelmer]

🔍 Code Review

Code issues found: 1

#	Severity	Comment
1	[Suggestion]	Use context.subscriptions instead of module-level variable