fix: refine MarkdownV2 escaping to prevent excessive backslashes

Author: codebam

packages/shared/src/index.ts

@@ -193,9 +193,16 @@ export async function markdownToHtml(s: string): Promise<string> {
 
 export function sanitizeMarkdownV2(text: string, isInsideLink = false): string {
 	// Standard MarkdownV2 characters that MUST be escaped outside of code/pre
-	let escaped = text.replace(/([_*[\]()~`>#+\-=|{}.!\\])/g, '\\$1');
+	// First, we unescape any existing escapes to avoid double-escaping
+	// We keep the full list of potentially escaped characters here for robustness
+	let unescaped = text.replace(/\\([_*[\]()~`>#+\-=|{}.!\\])/g, '$1');
+	
+	// We only escape characters that are truly likely to cause parsing issues or are essential markup
+	// Reduced list: _ * [ ] ~ ` \ |
+	let escaped = unescaped.replace(/([_*[\]~`\\|])/g, '\\$1');
+	
 	if (isInsideLink) {
-		// Inside links, additional characters need escaping
+		// Inside links, additional characters need escaping to avoid breaking the link syntax
 		escaped = escaped.replace(/([()])/g, '\\$1');
 	}
 	return escaped;
@@ -223,7 +230,7 @@ export async function markdownToMarkdownV2(s: string): Promise<string> {
 		for (let i = 0; i < items.length; i++) {
 			const item = items[i];
 			const prefix = ordered
-				? `${start !== '' && start !== undefined ? Number(start) + i : i + 1}\\. `
+				? `${start !== '' && start !== undefined ? Number(start) + i : i + 1}. `
 				: '• ';
 			result += `${prefix}${renderer.listitem(item)}\n`;
 		}