Implement commit message line unwrapping for PR creation

- Add unwrapCommitMessageBody function to join wrapped lines
- Preserve blank lines, list items, quotes, and indented content
- Add comprehensive tests for various unwrapping scenarios
- Matches GitHub's behavior when converting commit messages to PR descriptions

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

Author: Copilot

src/github/folderRepositoryManager.ts

@@ -2877,15 +2877,83 @@ const ownedByMe: AsyncPredicate<GitHubRepository> = async repo => {
 export const byRemoteName = (name: string): Predicate<GitHubRepository> => ({ remote: { remoteName } }) =>
 	remoteName === name;
 
+/**
+ * Unwraps lines that were wrapped for conventional commit message formatting (typically at 72 characters).
+ * Similar to GitHub's behavior when converting commit messages to PR descriptions.
+ * 
+ * Rules:
+ * - Preserves blank lines as paragraph breaks
+ * - Preserves lines that start with special characters (list items, quotes, indentation)
+ * - Joins consecutive lines that appear to be wrapped mid-sentence
+ */
+function unwrapCommitMessageBody(body: string): string {
+	if (!body) {
+		return body;
+	}
+
+	const lines = body.split('\n');
+	const result: string[] = [];
+	let i = 0;
+
+	while (i < lines.length) {
+		const line = lines[i];
+
+		// Preserve blank lines
+		if (line.trim() === '') {
+			result.push(line);
+			i++;
+			continue;
+		}
+
+		// Check if this line should NOT be joined with the next
+		// Lines that start with special formatting characters should be preserved
+		const shouldPreserveLine = /^[\s*\-+>]|^\d+\./.test(line);
+
+		if (shouldPreserveLine) {
+			result.push(line);
+			i++;
+			continue;
+		}
+
+		// Start accumulating lines that should be joined
+		let joinedLine = line;
+		i++;
+
+		// Keep joining lines until we hit a blank line or a line that shouldn't be joined
+		while (i < lines.length) {
+			const nextLine = lines[i];
+
+			// Stop at blank lines
+			if (nextLine.trim() === '') {
+				break;
+			}
+
+			// Stop at lines that start with special formatting
+			if (/^[\s*\-+>]|^\d+\./.test(nextLine)) {
+				break;
+			}
+
+			// Join this line with a space
+			joinedLine += ' ' + nextLine;
+			i++;
+		}
+
+		result.push(joinedLine);
+	}
+
+	return result.join('\n');
+}
+
 export const titleAndBodyFrom = async (promise: Promise<string | undefined>): Promise<{ title: string; body: string } | undefined> => {
 	const message = await promise;
 	if (!message) {
 		return;
 	}
 	const idxLineBreak = message.indexOf('\n');
+	const rawBody = idxLineBreak === -1 ? '' : message.slice(idxLineBreak + 1).trim();
 	return {
 		title: idxLineBreak === -1 ? message : message.substr(0, idxLineBreak),
 
-		body: idxLineBreak === -1 ? '' : message.slice(idxLineBreak + 1).trim(),
+		body: unwrapCommitMessageBody(rawBody),
 	};
 };

src/test/github/folderRepositoryManager.test.ts

@@ -94,4 +94,76 @@ describe('titleAndBodyFrom', function () {
 		assert.strictEqual(result?.title, 'title');
 		assert.strictEqual(result?.body, '');
 	});
+
+	it('unwraps wrapped lines in body', async function () {
+		const message = Promise.resolve('title\n\nThis is a long line that has been wrapped at 72 characters\nto fit the conventional commit message format.');
+
+		const result = await titleAndBodyFrom(message);
+		assert.strictEqual(result?.title, 'title');
+		assert.strictEqual(result?.body, 'This is a long line that has been wrapped at 72 characters to fit the conventional commit message format.');
+	});
+
+	it('preserves blank lines as paragraph breaks', async function () {
+		const message = Promise.resolve('title\n\nFirst paragraph that is wrapped\nacross multiple lines.\n\nSecond paragraph that is also wrapped\nacross multiple lines.');
+
+		const result = await titleAndBodyFrom(message);
+		assert.strictEqual(result?.title, 'title');
+		assert.strictEqual(result?.body, 'First paragraph that is wrapped across multiple lines.\n\nSecond paragraph that is also wrapped across multiple lines.');
+	});
+
+	it('preserves list items', async function () {
+		const message = Promise.resolve('title\n\n- First item\n- Second item\n- Third item');
+
+		const result = await titleAndBodyFrom(message);
+		assert.strictEqual(result?.title, 'title');
+		assert.strictEqual(result?.body, '- First item\n- Second item\n- Third item');
+	});
+
+	it('preserves numbered list items', async function () {
+		const message = Promise.resolve('title\n\n1. First item\n2. Second item\n3. Third item');
+
+		const result = await titleAndBodyFrom(message);
+		assert.strictEqual(result?.title, 'title');
+		assert.strictEqual(result?.body, '1. First item\n2. Second item\n3. Third item');
+	});
+
+	it('preserves indented lines', async function () {
+		const message = Promise.resolve('title\n\nNormal paragraph.\n\n    Indented code block\n    More code');
+
+		const result = await titleAndBodyFrom(message);
+		assert.strictEqual(result?.title, 'title');
+		assert.strictEqual(result?.body, 'Normal paragraph.\n\n    Indented code block\n    More code');
+	});
+
+	it('unwraps but preserves asterisk list items', async function () {
+		const message = Promise.resolve('title\n\n* First item\n* Second item');
+
+		const result = await titleAndBodyFrom(message);
+		assert.strictEqual(result?.title, 'title');
+		assert.strictEqual(result?.body, '* First item\n* Second item');
+	});
+
+	it('handles mixed content with wrapped paragraphs and lists', async function () {
+		const message = Promise.resolve('title\n\nThis is a paragraph that has been wrapped\nat 72 characters.\n\n- Item 1\n- Item 2\n\nAnother wrapped paragraph\nthat continues here.');
+
+		const result = await titleAndBodyFrom(message);
+		assert.strictEqual(result?.title, 'title');
+		assert.strictEqual(result?.body, 'This is a paragraph that has been wrapped at 72 characters.\n\n- Item 1\n- Item 2\n\nAnother wrapped paragraph that continues here.');
+	});
+
+	it('preserves lines with special characters at the start', async function () {
+		const message = Promise.resolve('title\n\n> Quote line 1\n> Quote line 2');
+
+		const result = await titleAndBodyFrom(message);
+		assert.strictEqual(result?.title, 'title');
+		assert.strictEqual(result?.body, '> Quote line 1\n> Quote line 2');
+	});
+
+	it('handles wrapped lines with punctuation', async function () {
+		const message = Promise.resolve('title\n\nThis is a sentence.\nThis is another sentence on a new line.');
+
+		const result = await titleAndBodyFrom(message);
+		assert.strictEqual(result?.title, 'title');
+		assert.strictEqual(result?.body, 'This is a sentence. This is another sentence on a new line.');
+	});
 });