Fix @ in urls (#6027)

alexr00 · web-flow · commit f8f644659c13 · 2024-05-30T06:06:37.000-07:00
Fixes #5924
diff --git a/src/github/prComment.ts b/src/github/prComment.ts
@@ -376,7 +376,7 @@ ${lineContents}
 		}
 		const newLinesReplaced = this.replaceNewlines(body);
 		const documentLanguage = (await vscode.workspace.openTextDocument(this.parent.uri)).languageId;
-		const replacerRegex = new RegExp(`([^\[\`]|^)@(${ALLOWED_USERS})`, 'g');
+		const replacerRegex = new RegExp(`([^/\[\`]|^)@(${ALLOWED_USERS})`, 'g');
 		// Replace user
 		const linkified = newLinesReplaced.replace(replacerRegex, (substring, _1, _2, offset) => {
 			// Do not try to replace user if there's a code block.

Original file line number	Diff line number	Diff line change
`@@ -376,7 +376,7 @@ ${lineContents}`
`376`	`376`	`}`
`377`	`377`	`const newLinesReplaced = this.replaceNewlines(body);`
`378`	`378`	`const documentLanguage = (await vscode.workspace.openTextDocument(this.parent.uri)).languageId;`
`379`		- const replacerRegex = new RegExp(`([^\[\`]\|^)@(${ALLOWED_USERS})`, 'g');
	`379`	+ const replacerRegex = new RegExp(`([^/\[\`]\|^)@(${ALLOWED_USERS})`, 'g');
`380`	`380`	`// Replace user`
`381`	`381`	`const linkified = newLinesReplaced.replace(replacerRegex, (substring, _1, _2, offset) => {`
`382`	`382`	`// Do not try to replace user if there's a code block.`