Only showing n files where n is the number of files changed in turn (microsoft#258180)

only showing n files

Author: aiday-mar

src/vs/workbench/contrib/chat/browser/chatListRenderer.ts

@@ -696,21 +696,9 @@ export class ChatListItemRenderer extends Disposable implements ITreeRenderer<Ch
 		if (element.model.response === element.model.entireResponse && element.errorDetails?.message && element.errorDetails.message !== canceledName) {
 			content.push({ kind: 'errorDetails', errorDetails: element.errorDetails, isLast: index === this.delegate.getListLength() - 1 });
 		}
-		if (this.shouldShowFileChangesSummary(element)) {
-			const fileChanges: IChatChangesSummary[] = [];
-			for (const part of element.model.entireResponse.value) {
-				if (part.kind === 'textEditGroup') {
-					fileChanges.push({
-						kind: 'changesSummary',
-						reference: part.uri,
-						sessionId: element.sessionId,
-						requestId: element.requestId,
-					});
-				}
-			}
-			if (fileChanges.length) {
-				content.push({ kind: 'changesSummary', fileChanges });
-			}
+		const fileChangesSummaryPart = this.getChatFileChangesSummaryPart(element);
+		if (fileChangesSummaryPart) {
+			content.push(fileChangesSummaryPart);
 		}
 
 		const diff = this.diff(templateData.renderedParts ?? [], content, element);
@@ -719,6 +707,29 @@ export class ChatListItemRenderer extends Disposable implements ITreeRenderer<Ch
 		this.updateItemHeightOnRender(element, templateData);
 	}
 
+	private getChatFileChangesSummaryPart(element: IChatResponseViewModel): IChatChangesSummaryPart | undefined {
+		if (!this.shouldShowFileChangesSummary(element)) {
+			return undefined;
+		}
+		const consideredFiles: Set<string> = new Set();
+		const fileChanges: IChatChangesSummary[] = [];
+		for (const part of element.model.entireResponse.value) {
+			if (part.kind === 'textEditGroup' && !consideredFiles.has(part.uri.toString(true))) {
+				fileChanges.push({
+					kind: 'changesSummary',
+					reference: part.uri,
+					sessionId: element.sessionId,
+					requestId: element.requestId,
+				});
+				consideredFiles.add(part.uri.toString(true));
+			}
+		}
+		if (!fileChanges.length) {
+			return undefined;
+		}
+		return { kind: 'changesSummary', fileChanges };
+	}
+
 	private renderChatRequest(element: IChatRequestViewModel, index: number, templateData: IChatListItemTemplate) {
 		templateData.rowContainer.classList.toggle('chat-response-loading', false);
 		if (element.id === this.viewModel?.editing?.id) {
@@ -1019,21 +1030,9 @@ export class ChatListItemRenderer extends Disposable implements ITreeRenderer<Ch
 			const isPaused = element.model.isPaused.get();
 			partsToRender.push({ kind: 'working', isPaused, setPaused: p => element.model.setPaused(p) });
 		}
-		if (this.shouldShowFileChangesSummary(element)) {
-			const fileChanges: IChatChangesSummary[] = [];
-			for (const part of element.model.entireResponse.value) {
-				if (part.kind === 'textEditGroup') {
-					fileChanges.push({
-						kind: 'changesSummary',
-						reference: part.uri,
-						sessionId: element.sessionId,
-						requestId: element.requestId,
-					});
-				}
-			}
-			if (fileChanges.length > 0) {
-				partsToRender.push({ kind: 'changesSummary', fileChanges });
-			}
+		const fileChangesSummaryPart = this.getChatFileChangesSummaryPart(element);
+		if (fileChangesSummaryPart) {
+			partsToRender.push(fileChangesSummaryPart);
 		}
 
 		return { content: partsToRender, moreContentAvailable };