Fix memory leak with event listeners

- Properly dispose event listeners after promise resolves
- Remove unrelated changes to proposed API file

Co-authored-by: alexr00 <38270282+alexr00@users.noreply.github.com>

Author: Copilot

src/commands.ts

@@ -1629,18 +1629,24 @@ ${contents}
 				quickPick.busy = false;
 
 				// Handle selection
+				let acceptDisposable: vscode.Disposable;
+				let hideDisposable: vscode.Disposable;
 				const selected = await new Promise<(vscode.QuickPickItem & { pr?: PullRequestModel }) | string | undefined>((resolve) => {
-					quickPick.onDidAccept(() => {
+					acceptDisposable = quickPick.onDidAccept(() => {
 						if (quickPick.selectedItems.length > 0) {
 							resolve(quickPick.selectedItems[0]);
 						} else if (quickPick.value) {
 							// User typed something but didn't select from list
 							resolve(quickPick.value);
 						}
 					});
-					quickPick.onDidHide(() => resolve(undefined));
+					hideDisposable = quickPick.onDidHide(() => resolve(undefined));
 				});
 
+				// Clean up event listeners
+				acceptDisposable.dispose();
+				hideDisposable.dispose();
+
 				quickPick.hide();
 				quickPick.dispose();