Final implementation with proper formatting

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

Author: Copilot

src/test/view/compareChangesTreeDataProvider.test.ts

@@ -8,9 +8,9 @@ import { default as assert } from 'assert';
 // Test the text truncation logic that should be applied to tree view button text
 describe('CompareChangesTreeDataProvider text truncation', () => {
 	const MAX_BUTTON_TEXT_LENGTH = 40; // Same constant used in implementation
-	
+
 	function truncateButtonText(buttonText: string): string {
-		return buttonText.length > MAX_BUTTON_TEXT_LENGTH 
+		return buttonText.length > MAX_BUTTON_TEXT_LENGTH
 			? buttonText.substring(0, MAX_BUTTON_TEXT_LENGTH - 3) + '...'
 			: buttonText;
 	}
@@ -19,7 +19,7 @@ describe('CompareChangesTreeDataProvider text truncation', () => {
 		it('should truncate long button text with ellipsis', () => {
 			const longButtonText = '$(sparkle) Very Long Copilot Reviewer Name That Would Cause Overflow Issues Code Review';
 			const result = truncateButtonText(longButtonText);
-			
+
 			assert.strictEqual(result.length, MAX_BUTTON_TEXT_LENGTH);
 			assert.ok(result.endsWith('...'));
 			assert.ok(result.includes('$(sparkle)'));
@@ -29,23 +29,23 @@ describe('CompareChangesTreeDataProvider text truncation', () => {
 		it('should not truncate short button text', () => {
 			const shortButtonText = '$(sparkle) Copilot Code Review';
 			const result = truncateButtonText(shortButtonText);
-			
+
 			assert.strictEqual(result, shortButtonText);
 			assert.ok(!result.includes('...'));
 		});
 
 		it('should handle exactly max length text', () => {
 			const exactLengthText = 'A'.repeat(MAX_BUTTON_TEXT_LENGTH);
 			const result = truncateButtonText(exactLengthText);
-			
+
 			assert.strictEqual(result, exactLengthText);
 			assert.ok(!result.includes('...'));
 		});
 
 		it('should handle text that is one character over the limit', () => {
 			const overLimitByOne = 'A'.repeat(MAX_BUTTON_TEXT_LENGTH + 1);
 			const result = truncateButtonText(overLimitByOne);
-			
+
 			assert.strictEqual(result.length, MAX_BUTTON_TEXT_LENGTH);
 			assert.ok(result.endsWith('...'));
 			assert.strictEqual(result, 'A'.repeat(MAX_BUTTON_TEXT_LENGTH - 3) + '...');
@@ -54,7 +54,7 @@ describe('CompareChangesTreeDataProvider text truncation', () => {
 		it('should preserve sparkle icon in truncated text', () => {
 			const longTextWithIcon = '$(sparkle) ' + 'Very '.repeat(20) + 'Long Code Review';
 			const result = truncateButtonText(longTextWithIcon);
-			
+
 			assert.ok(result.startsWith('$(sparkle)'));
 			assert.ok(result.endsWith('...'));
 			assert.strictEqual(result.length, MAX_BUTTON_TEXT_LENGTH);

src/view/compareChangesTreeDataProvider.ts

@@ -264,14 +264,14 @@ class CompareChangesFilesTreeProvider extends CompareChangesTreeProvider {
 			markdown.appendMarkdown('\n\n');
 		}
 		markdown.supportThemeIcons = true;
-		
+
 		// Create the button text with potential truncation for better UI in smaller panels
 		const buttonText = vscode.l10n.t('$(sparkle) {0} Code Review', preReviewer.title);
 		const MAX_BUTTON_TEXT_LENGTH = 40; // Reasonable length for tree view messages
-		const truncatedButtonText = buttonText.length > MAX_BUTTON_TEXT_LENGTH 
+		const truncatedButtonText = buttonText.length > MAX_BUTTON_TEXT_LENGTH
 			? buttonText.substring(0, MAX_BUTTON_TEXT_LENGTH - 3) + '...'
 			: buttonText;
-			
+
 		markdown.appendMarkdown(`[${truncatedButtonText}](command:pr.preReview)`);
 		return markdown;
 	}