Add function to convert issue/PR references to clickable links in chat output

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

Author: Copilot

src/common/uri.ts

@@ -793,3 +793,25 @@ export function resolvePath(from: vscode.Uri, to: string) {
 		return pathUtils.posix.resolve(from.path, to);
 	}
 }
+
+/**
+ * Converts issue and PR number references in text to clickable markdown links.
+ * Matches patterns like "#123", "issue 123", "issue #123", "PR 123", "PR #123"
+ * @param text The text to process
+ * @param owner The repository owner
+ * @param repo The repository name
+ * @returns The text with issue/PR references converted to markdown links
+ */
+export function convertIssuePRReferencesToLinks(text: string, owner: string, repo: string): string {
+	// Pattern matches:
+	// - #123 (standalone hash with number)
+	// - issue 123 or issue #123 (case-insensitive)
+	// - PR 123 or PR #123 (case-insensitive)
+	// Uses word boundaries to avoid matching in the middle of words
+	const pattern = /\b(?:issue\s+#?|PR\s+#?|#)(\d+)\b/gi;
+
+	return text.replace(pattern, (match, number) => {
+		const url = `https://github.com/${owner}/${repo}/issues/${number}`;
+		return `[${match}](${url})`;
+	});
+}

src/github/copilotRemoteAgent.ts

@@ -21,7 +21,7 @@ import Logger from '../common/logger';
 import { GitHubRemote } from '../common/remote';
 import { CODING_AGENT, CODING_AGENT_AUTO_COMMIT_AND_PUSH } from '../common/settingKeys';
 import { ITelemetry } from '../common/telemetry';
-import { toOpenPullRequestWebviewUri } from '../common/uri';
+import { convertIssuePRReferencesToLinks, toOpenPullRequestWebviewUri } from '../common/uri';
 import { ChatSessionContentBuilder } from './copilotRemoteAgent/chatSessionContentBuilder';
 import { GitOperationsManager } from './copilotRemoteAgent/gitOperationsManager';
 import { extractTitle, formatBodyPlaceholder, truncatePrompt } from './copilotRemoteAgentUtils';
@@ -1246,7 +1246,12 @@ export class CopilotRemoteAgentManager extends Disposable {
 						} else {
 							if (delta.content) {
 								if (!delta.content.startsWith('<pr_title>')) {
-									stream.markdown(delta.content);
+									const convertedContent = convertIssuePRReferencesToLinks(
+										delta.content,
+										pullRequest.remote.owner,
+										pullRequest.remote.repositoryName
+									);
+									stream.markdown(convertedContent);
 									hasStreamedContent = true;
 								}
 							}

src/github/copilotRemoteAgent/chatSessionContentBuilder.ts

@@ -10,7 +10,7 @@ import { parseSessionLogs, parseToolCallDetails, StrReplaceEditorToolData } from
 import { COPILOT_SWE_AGENT } from '../../common/copilot';
 import Logger from '../../common/logger';
 import { CommentEvent, CopilotFinishedEvent, CopilotStartedEvent, EventType, ReviewEvent, TimelineEvent } from '../../common/timelineEvent';
-import { toOpenPullRequestWebviewUri } from '../../common/uri';
+import { convertIssuePRReferencesToLinks, toOpenPullRequestWebviewUri } from '../../common/uri';
 import { InMemFileChangeModel, RemoteFileChangeModel } from '../../view/fileChangeModel';
 import { AssistantDelta, Choice, ToolCall } from '../common';
 import { CopilotApi, SessionInfo } from '../copilotApi';
@@ -288,7 +288,12 @@ export class ChatSessionContentBuilder {
 			}
 
 			if (currentResponseContent.trim()) {
-				responseParts.push(new vscode.ChatResponseMarkdownPart(currentResponseContent.trim()));
+				const convertedContent = convertIssuePRReferencesToLinks(
+					currentResponseContent.trim(),
+					pullRequest.remote.owner,
+					pullRequest.remote.repositoryName
+				);
+				responseParts.push(new vscode.ChatResponseMarkdownPart(convertedContent));
 			}
 
 			if (session.state === 'completed' || session.state === 'failed' /** session can fail with proposed changes */) {
@@ -336,7 +341,12 @@ export class ChatSessionContentBuilder {
 				if (delta.content && delta.content.trim()) {
 					// Add any accumulated content as markdown first
 					if (currentResponseContent.trim()) {
-						responseParts.push(new vscode.ChatResponseMarkdownPart(currentResponseContent.trim()));
+						const convertedContent = convertIssuePRReferencesToLinks(
+							currentResponseContent.trim(),
+							pullRequest.remote.owner,
+							pullRequest.remote.repositoryName
+						);
+						responseParts.push(new vscode.ChatResponseMarkdownPart(convertedContent));
 						currentResponseContent = '';
 					}
 
@@ -357,7 +367,12 @@ export class ChatSessionContentBuilder {
 				if (delta.tool_calls) {
 					// Add any accumulated content as markdown first
 					if (currentResponseContent.trim()) {
-						responseParts.push(new vscode.ChatResponseMarkdownPart(currentResponseContent.trim()));
+						const convertedContent = convertIssuePRReferencesToLinks(
+							currentResponseContent.trim(),
+							pullRequest.remote.owner,
+							pullRequest.remote.repositoryName
+						);
+						responseParts.push(new vscode.ChatResponseMarkdownPart(convertedContent));
 						currentResponseContent = '';
 					}
 

src/test/common/uri.test.ts

@@ -5,7 +5,7 @@
 
 import { default as assert } from 'assert';
 import * as vscode from 'vscode';
-import { fromOpenOrCheckoutPullRequestWebviewUri } from '../../common/uri';
+import { convertIssuePRReferencesToLinks, fromOpenOrCheckoutPullRequestWebviewUri } from '../../common/uri';
 
 describe('uri', () => {
 	describe('fromOpenOrCheckoutPullRequestWebviewUri', () => {
@@ -110,4 +110,75 @@ describe('uri', () => {
 			assert.strictEqual(result2, undefined);
 		});
 	});
+
+	describe('convertIssuePRReferencesToLinks', () => {
+		const owner = 'microsoft';
+		const repo = 'vscode-pull-request-github';
+
+		it('should convert standalone issue numbers with # prefix', () => {
+			const text = 'This PR addresses issue #7280.';
+			const result = convertIssuePRReferencesToLinks(text, owner, repo);
+			assert.strictEqual(result, 'This PR addresses issue [#7280](https://github.com/microsoft/vscode-pull-request-github/issues/7280).');
+		});
+
+		it('should convert issue references without # prefix', () => {
+			const text = 'This fixes issue 123.';
+			const result = convertIssuePRReferencesToLinks(text, owner, repo);
+			assert.strictEqual(result, 'This fixes [issue 123](https://github.com/microsoft/vscode-pull-request-github/issues/123).');
+		});
+
+		it('should convert PR references with # prefix', () => {
+			const text = 'See PR #456 for details.';
+			const result = convertIssuePRReferencesToLinks(text, owner, repo);
+			assert.strictEqual(result, 'See [PR #456](https://github.com/microsoft/vscode-pull-request-github/issues/456) for details.');
+		});
+
+		it('should convert PR references without # prefix', () => {
+			const text = 'Related to PR 789.';
+			const result = convertIssuePRReferencesToLinks(text, owner, repo);
+			assert.strictEqual(result, 'Related to [PR 789](https://github.com/microsoft/vscode-pull-request-github/issues/789).');
+		});
+
+		it('should convert multiple issue/PR references in the same text', () => {
+			const text = 'This fixes issue #123 and PR #456.';
+			const result = convertIssuePRReferencesToLinks(text, owner, repo);
+			assert.strictEqual(result, 'This fixes [issue #123](https://github.com/microsoft/vscode-pull-request-github/issues/123) and [PR #456](https://github.com/microsoft/vscode-pull-request-github/issues/456).');
+		});
+
+		it('should handle case-insensitive issue/PR keywords', () => {
+			const text = 'See Issue #100 and pr #200.';
+			const result = convertIssuePRReferencesToLinks(text, owner, repo);
+			assert.strictEqual(result, 'See [Issue #100](https://github.com/microsoft/vscode-pull-request-github/issues/100) and [pr #200](https://github.com/microsoft/vscode-pull-request-github/issues/200).');
+		});
+
+		it('should not convert issue/PR references in the middle of words', () => {
+			const text = 'This is not#123 an issue.';
+			const result = convertIssuePRReferencesToLinks(text, owner, repo);
+			assert.strictEqual(result, 'This is not#123 an issue.');
+		});
+
+		it('should not convert # followed by non-numeric characters', () => {
+			const text = 'This is #notanissue.';
+			const result = convertIssuePRReferencesToLinks(text, owner, repo);
+			assert.strictEqual(result, 'This is #notanissue.');
+		});
+
+		it('should convert standalone # followed by number', () => {
+			const text = 'See #42 for more info.';
+			const result = convertIssuePRReferencesToLinks(text, owner, repo);
+			assert.strictEqual(result, 'See [#42](https://github.com/microsoft/vscode-pull-request-github/issues/42) for more info.');
+		});
+
+		it('should handle text with no issue/PR references', () => {
+			const text = 'This is just regular text.';
+			const result = convertIssuePRReferencesToLinks(text, owner, repo);
+			assert.strictEqual(result, 'This is just regular text.');
+		});
+
+		it('should handle empty text', () => {
+			const text = '';
+			const result = convertIssuePRReferencesToLinks(text, owner, repo);
+			assert.strictEqual(result, '');
+		});
+	});
 });