adding comments (#6410)

Author: aiday-mar

package.json

@@ -3239,7 +3239,17 @@
 						"comments": {
 							"type": "array",
 							"items": {
-								"type": "string"
+								"type": "object",
+								"properties": {
+									"body": {
+										"type": "string",
+										"description": "The comment body"
+									},
+									"author": {
+										"type": "string",
+										"description": "The author of the comment"
+									}
+								}
 							},
 							"description": "The array of associated string comments"
 						},
@@ -3316,7 +3326,17 @@
 						"unreadComments": {
 							"type": "array",
 							"items": {
-								"type": "string"
+								"type": "object",
+								"properties": {
+									"body": {
+										"type": "string",
+										"description": "The comment body"
+									},
+									"author": {
+										"type": "string",
+										"description": "The author of the comment"
+									}
+								}
 							},
 							"description": "The array of unread comments under the issue/PR of the notification"
 						},

src/lm/tools/fetchNotificationTool.ts

@@ -26,6 +26,7 @@ export interface FetchNotificationResult {
 	title: string;
 	body: string;
 	unreadComments: {
+		author: string;
 		body: string;
 	}[];
 	owner: string;
@@ -73,13 +74,13 @@ export class FetchNotificationTool extends RepoToolBase<FetchNotificationToolPar
 		const itemType = issueOrPR instanceof PullRequestModel ? 'pr' : 'issue';
 		const notificationKey = getNotificationKey(owner, name, String(issueOrPR.number));
 		const comments = issueOrPR.item.comments ?? [];
-		let unreadComments: { body: string; }[];
+		let unreadComments: { body: string; author: string }[];
 		if (lastReadAt !== undefined && comments) {
 			unreadComments = comments.filter(comment => {
 				return comment.createdAt > lastReadAt;
-			}).map(comment => { return { body: comment.body }; });
+			}).map(comment => { return { body: comment.body, author: comment.author.login }; });
 		} else {
-			unreadComments = comments.map(comment => { return { body: comment.body }; });
+			unreadComments = comments.map(comment => { return { body: comment.body, author: comment.author.login }; });
 		}
 		const result: FetchNotificationResult = {
 			lastReadAt,

src/lm/tools/summarizeIssueTool.ts

@@ -79,7 +79,10 @@ If a comment references for example issue or PR #123, then output either of the
 [#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.
+When you summarize comments, always give a summary of each comment and always mention the author clearly before the comment. If the author is called 'joe' and the comment is 'this is a comment', then the output should be:
+
+joe: this is a comment
+
 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

@@ -53,6 +53,7 @@ The following are the unread comments of the thread:
 		for (const [index, comment] of unreadComments.entries()) {
 			notificationInfo += `
 Comment ${index} :
+Author: ${comment.author}
 Body: ${comment.body}
 `;
 		}
@@ -98,6 +99,10 @@ If a comment references for example issue or PR #123, then output either of the
 [#123](https://github.com/${owner}/${repo}/issues/123)
 [#123](https://github.com/${owner}/${repo}/pull/123)
 
+When you summarize comments, always give a summary of each comment and always mention the author clearly before the comment. If the author is called 'joe' and the comment is 'this is a comment', then the output should be:
+
+joe: this is a comment
+
 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: