fix: strip trailing whitespace from sent messages

Author: Algorithm5838

src/lib/components/common/RichTextInput.svelte

@@ -37,6 +37,19 @@
 	});
 	turndownService.escape = (string) => string;
 
+	// Turndown renders <br> as a Markdown "  \n" hard break, leaving invisible
+	// trailing spaces on every line. Strip it, but only after real content and
+	// never inside fenced code blocks where trailing whitespace can be significant.
+	const stripTrailingWhitespace = (md: string): string =>
+		md
+			.split(/(```[\s\S]*?```|~~~[\s\S]*?~~~)/g)
+			.map((part, i) =>
+				i % 2
+					? part
+					: part.replace(/([^\n \t])[ \t]+\n/g, '$1\n').replace(/([^\n \t])[ \t]+$/g, '$1')
+			)
+			.join('');
+
 	// Produce single newlines between paragraphs instead of double.
 	// TipTap wraps every line in <p> tags; the default Turndown rule emits
 	// \n\n around each paragraph which then required a destructive
@@ -711,7 +724,7 @@
 					try {
 						// Try parsing the value
 						return marked.parse(value.replaceAll(`\n<br/>`, `<br/>`), {
-							breaks: false
+							breaks: true
 						});
 					} catch (error) {
 						// If no attempts remain, fallback to plain text
@@ -885,26 +898,30 @@
 				jsonValue = editor.getJSON();
 
 				if (richText) {
-					mdValue = turndownService
-						.turndown(
-							htmlValue
-								.replace(/<p><\/p>/g, '<br/>')
-								.replace(/ {2,}/g, (m) => m.replace(/ /g, '\u00a0'))
-						)
-						.replace(/\u00a0/g, ' ');
+					mdValue = stripTrailingWhitespace(
+						turndownService
+							.turndown(
+								htmlValue
+									.replace(/<p><\/p>/g, '<br/>')
+									.replace(/ {2,}/g, (m) => m.replace(/ /g, '\u00a0'))
+							)
+							.replace(/\u00a0/g, ' ')
+					);
 				} else {
-					mdValue = turndownService
-						.turndown(
-							htmlValue
-								// Replace empty paragraphs with line breaks
-								.replace(/<p><\/p>/g, '<br/>')
-								// Replace multiple spaces with non-breaking spaces
-								.replace(/ {2,}/g, (m) => m.replace(/ /g, '\u00a0'))
-								// Replace tabs with non-breaking spaces (preserve indentation)
-								.replace(/\t/g, '\u00a0\u00a0\u00a0\u00a0') // 1 tab = 4 spaces
-						)
-						// Convert non-breaking spaces back to regular spaces for markdown
-						.replace(/\u00a0/g, ' ');
+					mdValue = stripTrailingWhitespace(
+						turndownService
+							.turndown(
+								htmlValue
+									// Replace empty paragraphs with line breaks
+									.replace(/<p><\/p>/g, '<br/>')
+									// Replace multiple spaces with non-breaking spaces
+									.replace(/ {2,}/g, (m) => m.replace(/ /g, '\u00a0'))
+									// Replace tabs with non-breaking spaces (preserve indentation)
+									.replace(/\t/g, '\u00a0\u00a0\u00a0\u00a0') // 1 tab = 4 spaces
+							)
+							// Convert non-breaking spaces back to regular spaces for markdown
+							.replace(/\u00a0/g, ' ')
+					);
 				}
 
 				onChange({
@@ -1267,14 +1284,16 @@
 
 		const jsonValue = editor.getJSON();
 		const htmlValue = editor.getHTML();
-		let mdValue = turndownService
-			.turndown(
-				(preserveBreaks ? htmlValue.replace(/<p><\/p>/g, '<br/>') : htmlValue).replace(
-					/ {2,}/g,
-					(m) => m.replace(/ /g, '\u00a0')
+		let mdValue = stripTrailingWhitespace(
+			turndownService
+				.turndown(
+					(preserveBreaks ? htmlValue.replace(/<p><\/p>/g, '<br/>') : htmlValue).replace(
+						/ {2,}/g,
+						(m) => m.replace(/ /g, '\u00a0')
+					)
 				)
-			)
-			.replace(/\u00a0/g, ' ');
+				.replace(/\u00a0/g, ' ')
+		);
 
 		if (value === '') {
 			editor.commands.clearContent(); // Clear content if value is empty
@@ -1300,7 +1319,7 @@
 						preserveBreaks
 							? value
 							: marked.parse(value.replaceAll(`\n<br/>`, `<br/>`), {
-									breaks: false
+									breaks: true
 								})
 					);