Linkifying the output links in the summary for summarization tools (#6396)

* adding repo owner

* adding also the same for notifications summary

Author: aiday-mar

package.json

@@ -3213,6 +3213,14 @@
 							"type": "string",
 							"description": "The body of the issue/PR"
 						},
+						"owner": {
+							"type": "string",
+							"description": "The owner of the repo in which the issue/PR is located"
+						},
+						"repo": {
+							"type": "string",
+							"description": "The repo in which the issue/PR is located"
+						},
 						"comments": {
 							"type": "array",
 							"items": {
@@ -3241,7 +3249,9 @@
 					"required": [
 						"title",
 						"body",
-						"comments"
+						"comments",
+						"owner",
+						"repo"
 					]
 				},
 				"when": "config.githubPullRequests.experimental.chat"
@@ -3280,6 +3290,14 @@
 							"type": "string",
 							"description": "The body of the notification issue/PR"
 						},
+						"owner": {
+							"type": "string",
+							"description": "The owner of the repo in which the issue/PR is located"
+						},
+						"repo": {
+							"type": "string",
+							"description": "The repo in which the issue/PR is located"
+						},
 						"unreadComments": {
 							"type": "array",
 							"items": {
@@ -3323,7 +3341,9 @@
 						"lastUpdatedAt",
 						"unread",
 						"threadId",
-						"notificationKey"
+						"notificationKey",
+						"owner",
+						"repo"
 					]
 				},
 				"when": "config.githubPullRequests.experimental.chat"
@@ -3708,4 +3728,4 @@
 		"vsls": "^0.3.967"
 	},
 	"license": "MIT"
-}
+}

src/lm/tools/fetchIssueTool.ts

@@ -29,6 +29,8 @@ export interface FetchIssueResult {
 		author: string;
 		body: string;
 	}[];
+	owner: string;
+	repo: string;
 	fileChanges?: FileChange[];
 }
 
@@ -42,6 +44,8 @@ export class FetchIssueTool extends RepoToolBase<FetchIssueToolParameters> {
 			throw new Error(`No issue or PR found for ${owner}/${name}/${options.parameters.issueNumber}. Make sure the issue or PR exists.`);
 		}
 		const result: FetchIssueResult = {
+			owner,
+			repo: name,
 			title: issueOrPullRequest.title,
 			body: issueOrPullRequest.body,
 			comments: issueOrPullRequest.item.comments?.map(c => ({ body: c.body, author: c.author.login })) ?? []

src/lm/tools/fetchNotificationTool.ts

@@ -28,6 +28,8 @@ export interface FetchNotificationResult {
 	unreadComments: {
 		body: string;
 	}[];
+	owner: string;
+	repo: string;
 	fileChanges?: FileChange[];
 	threadId: number,
 	notificationKey: string
@@ -85,6 +87,8 @@ export class FetchNotificationTool extends RepoToolBase<FetchNotificationToolPar
 			notificationKey,
 			title: issueOrPR.title,
 			body: issueOrPR.body,
+			owner,
+			repo: name
 		};
 		if (issueOrPR instanceof PullRequestModel) {
 			const fileChanges = await issueOrPR.getFileChangesInfo();

src/lm/tools/summarizeIssueTool.ts

@@ -56,7 +56,7 @@ Body: ${comment.body}
 		const model = models[0];
 
 		if (model) {
-			const messages = [vscode.LanguageModelChatMessage.User(this.summarizeInstructions())];
+			const messages = [vscode.LanguageModelChatMessage.User(this.summarizeInstructions(options.parameters.repo, options.parameters.owner))];
 			messages.push(vscode.LanguageModelChatMessage.User(`The issue or pull request information is as follows:`));
 			messages.push(vscode.LanguageModelChatMessage.User(issueOrPullRequestInfo));
 			const response = await model.sendRequest(messages, {});
@@ -67,12 +67,18 @@ Body: ${comment.body}
 		}
 	}
 
-	private summarizeInstructions(): string {
+	private summarizeInstructions(repo: string, owner: string): string {
 		return `
 You are an AI assistant who is very proficient in summarizing issues and PRs.
 You will be given information relative to an issue or PR : the title, the body and the comments. In the case of a PR you will also be given patches of the PR changes.
 Your task is to output a summary of all this information.
 Do not output code. When you try to summarize PR changes, summarize in a textual format.
+Output references to other issues and PRs as Markdown links. The current issue has owner ${owner} and is in the repo ${repo}.
+If a comment references for example issue or PR #123, then output either of the following in the summary depending on if it is an issue or a PR:
+
+[#123](https://github.com/${owner}/${repo}/issues/123)
+[#123](https://github.com/${owner}/${repo}/pull/123)
+
 When you summarize comments, give a summary of each comment and mention the author clearly.
 Make sure the summary is at least as short or shorter than the issue or PR with the comments and the patches if there are.
 `;

src/lm/tools/summarizeNotificationsTool.ts

@@ -61,7 +61,7 @@ Body: ${comment.body}
 			}]
 		};
 		if (model) {
-			const messages = [vscode.LanguageModelChatMessage.User(this.summarizeInstructions())];
+			const messages = [vscode.LanguageModelChatMessage.User(this.summarizeInstructions(options.parameters.owner, options.parameters.repo))];
 			messages.push(vscode.LanguageModelChatMessage.User(`The notification information is as follows:`));
 			messages.push(vscode.LanguageModelChatMessage.User(notificationInfo));
 			const response = await model.sendRequest(messages, {});
@@ -77,12 +77,18 @@ Body: ${comment.body}
 		}
 	}
 
-	private summarizeInstructions(): string {
+	private summarizeInstructions(owner: string, repo: string): string {
 		return `
 You are an AI assistant who is very proficient in summarizing notification threads.
 You will be given information relative to a notification thread : the title, the body and the comments. In the case of a PR you will also be given patches of the PR changes.
 Since you are reviewing a notification thread, part of the content is by definition unread. You will be told what part of the content is yet unread. This can be the comments or it can be both the thread issue/PR as well as the comments.
 Your task is to output a summary of all this notification thread information and give an update to the user concerning the unread part of the thread.
+Output references to issues and PRs as Markdown links. The current notification is for a thread that has owner ${owner} and is in the repo ${repo}.
+If a comment references for example issue or PR #123, then output either of the following in the summary depending on if it is an issue or a PR:
+
+[#123](https://github.com/${owner}/${repo}/issues/123)
+[#123](https://github.com/${owner}/${repo}/pull/123)
+
 Always include in your output, which part of the thread is unread by prefixing that part with the markdown heading of level 1 with text "Unread Thread" or "Unread Comments".
 Make sure the summary is at least as short or shorter than the issue or PR with the comments and the patches if there are.
 Example output: