Fix: Filter out invalid workspace URIs in URI handler to prevent error on second PR open

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

Author: Copilot

src/test/common/uri.test.ts

@@ -6,8 +6,41 @@
 import { default as assert } from 'assert';
 import * as vscode from 'vscode';
 import { fromOpenOrCheckoutPullRequestWebviewUri } from '../../common/uri';
+import { isValidWorkspaceUri } from '../../uriHandler';
 
 describe('uri', () => {
+	describe('isValidWorkspaceUri', () => {
+		it('should return true for valid Unix file URIs', () => {
+			const uri = vscode.Uri.parse('file:///home/user/repos/vscode');
+			assert.strictEqual(isValidWorkspaceUri(uri), true);
+		});
+
+		it('should return true for valid Windows file URIs', () => {
+			const uri = vscode.Uri.parse('file:///c%3A/Users/dmitriv/repos/vscode');
+			assert.strictEqual(isValidWorkspaceUri(uri), true);
+		});
+
+		it('should return false for numeric-only path URIs (timestamps)', () => {
+			const uri = vscode.Uri.parse('file:///1761808101585');
+			assert.strictEqual(isValidWorkspaceUri(uri), false);
+		});
+
+		it('should return false for non-file scheme URIs', () => {
+			const uri = vscode.Uri.parse('https://github.com/microsoft/vscode');
+			assert.strictEqual(isValidWorkspaceUri(uri), false);
+		});
+
+		it('should return true for paths that contain numbers mixed with letters', () => {
+			const uri = vscode.Uri.parse('file:///home/user123/repos');
+			assert.strictEqual(isValidWorkspaceUri(uri), true);
+		});
+
+		it('should return true for paths that end with numbers', () => {
+			const uri = vscode.Uri.parse('file:///home/project/version2');
+			assert.strictEqual(isValidWorkspaceUri(uri), true);
+		});
+	});
+
 	describe('fromOpenOrCheckoutPullRequestWebviewUri', () => {
 		it('should parse the new simplified format with uri parameter', () => {
 			const uri = vscode.Uri.parse('vscode://github.vscode-pull-request-github/checkout-pull-request?uri=https://github.com/microsoft/vscode-css-languageservice/pull/460');

src/uriHandler.ts

@@ -19,6 +19,25 @@ import { ReviewsManager } from './view/reviewsManager';
 
 export const PENDING_CHECKOUT_PULL_REQUEST_KEY = 'pendingCheckoutPullRequest';
 
+/**
+ * Validates that a workspace URI has a valid file path.
+ * Filters out invalid URIs like file:///1761808101585 (which are numeric timestamps).
+ */
+export function isValidWorkspaceUri(uri: vscode.Uri): boolean {
+	if (uri.scheme !== 'file') {
+		return false;
+	}
+	// The fsPath should be a proper file system path, not just a number
+	// On Windows, valid paths have a drive letter (e.g., C:\...)
+	// On Unix, valid paths start with / followed by directory names
+	const fsPath = uri.fsPath;
+	// Check if the path is just a number (invalid)
+	if (/^[/\\]?\d+$/.test(fsPath)) {
+		return false;
+	}
+	return true;
+}
+
 interface PendingCheckoutPayload {
 	owner: string;
 	repo: string;
@@ -199,10 +218,12 @@ export class UriHandler implements vscode.UriHandler {
 			try {
 				progress.report({ message: vscode.l10n.t('Locating workspace') });
 				const remoteUri = vscode.Uri.parse(`https://github.com/${params.owner}/${params.repo}`);
-				const workspaces = await this._git.getRepositoryWorkspace(remoteUri);
+				const allWorkspaces = await this._git.getRepositoryWorkspace(remoteUri);
 				if (token.isCancellationRequested) {
 					return;
 				}
+				// Filter out invalid workspace URIs (e.g., URIs with numeric-only paths)
+				const workspaces = allWorkspaces?.filter(isValidWorkspaceUri);
 				if (workspaces && workspaces.length) {
 					Logger.appendLine(`Found workspaces for ${remoteUri.toString()}: ${workspaces.map(w => w.toString()).join(', ')}`, UriHandler.ID);
 					progress.report({ message: vscode.l10n.t('Opening workspace') });