Fix promise resolution edge cases

- Handle case when user presses Enter with no selection
- Make disposables optional to prevent undefined access

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

Author: Copilot

src/commands.ts

@@ -1629,23 +1629,26 @@ ${contents}
 				quickPick.busy = false;
 
 				// Handle selection
-				let acceptDisposable: vscode.Disposable;
-				let hideDisposable: vscode.Disposable;
+				let acceptDisposable: vscode.Disposable | undefined;
+				let hideDisposable: vscode.Disposable | undefined;
 				const selected = await new Promise<(vscode.QuickPickItem & { pr?: PullRequestModel }) | string | undefined>((resolve) => {
 					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);
+						} else {
+							// User pressed Enter with no selection and no input
+							resolve(undefined);
 						}
 					});
 					hideDisposable = quickPick.onDidHide(() => resolve(undefined));
 				});
 
 				// Clean up event listeners
-				acceptDisposable.dispose();
-				hideDisposable.dispose();
+				acceptDisposable?.dispose();
+				hideDisposable?.dispose();
 
 				quickPick.hide();
 				quickPick.dispose();