Use regex groups (#8097)

Author: alexr00

.github/copilot-instructions.md

@@ -5,6 +5,7 @@
 - **Strictness**: Some strictness is disabled in `tsconfig.base.json` (e.g., `strictNullChecks: false`), but new code should avoid unsafe patterns.
 - **Testing**: Place tests under `src/test`. Do not include test code in production files.
 - **Localization**: Use `%key%` syntax for strings that require localization. See `package.nls.json`.
+- **Regular Expressions**: Use named capture groups for clarity when working with complex regex patterns.
 
 ## Extension-Specific Practices
 - **VS Code API**: Use the official VS Code API for all extension points. Register commands, views, and menus via `package.json`.

src/common/uri.ts

@@ -690,9 +690,11 @@ export function fromOpenOrCheckoutPullRequestWebviewUri(uri: vscode.Uri): OpenPu
 		const uriParam = queryParams.get('uri');
 		if (uriParam) {
 			// Parse the GitHub PR URL - match only exact format ending with the PR number
-			const match = uriParam.match(/^https?:\/\/github\.com\/([^\/]+)\/([^\/]+)\/pull\/(\d+)$/);
-			if (match) {
-				const [, owner, repo, pullRequestNumber] = match;
+			// Use named regex groups for clarity
+			const prUrlRegex = /^https?:\/\/github\.com\/(?<owner>[^\/]+)\/(?<repo>[^\/]+)\/pull\/(?<pullRequestNumber>\d+)$/;
+			const match = prUrlRegex.exec(uriParam);
+			if (match && match.groups) {
+				const { owner, repo, pullRequestNumber } = match.groups;
 				const params = {
 					owner,
 					repo,