nodebb-plugin-mentions 4.6.0 → 4.6.2

package/library.js

@@ -5,7 +5,6 @@
 const _ = require('lodash');
 const validator = require('validator');
 const entitiesDecode = require('html-entities').decode;
-const cheerio = require('cheerio');
 
 const nconf = require.main.require('nconf');
 const winston = require.main.require('winston');
@@ -37,6 +36,7 @@ const parts = {
 };
 const regex = RegExp(`${parts.before}${parts.main}`, 'gu');
 const isLatinMention = /@[\w\d\-_.@]+$/;
+const anchorRegex = /<a.*?href=['"](.+?)['"].*?>(.*?)<\/a>/ig;
 
 const Mentions = module.exports;
 
@@ -356,31 +356,36 @@ async function getMatches(content, isMarkdown = false) {
 		}
 	});
 
-	const $ = cheerio.load(splitContent.join(''));
-	const anchors = $('a');
-	const urls = new Set();
-	Array.from(anchors).forEach((anchor) => {
-		const text = $(anchor).prop('innerText');
-		const match = text.match(regex);
-		if (match) {
-			urls.add($(anchor).attr('href'));
-		}
-	});
+	// Early return
+	if (isMarkdown) {
+		return { splitContent, matches, urlMap: new Map() };
+	}
 
-	// Filter out urls that don't backreference to a remote id
-	const backrefs = await db.getObjectFields('remoteUrl:uid', Array.from(urls));
-	const urlAsIdExists = await db.isSortedSetMembers('usersRemote:lastCrawled', Array.from(urls));
+	// Find matches via backreferenced href
+	const joined = splitContent.join('');
+	const anchors = new Set(joined.matchAll(anchorRegex));
 	const urlMap = new Map();
-	Array.from(urls).map(async (url, index) => {
-		if (backrefs[url] || urlAsIdExists[index]) {
-			urlMap.set(url, backrefs[url] || url);
-		}
-	});
-	let slugs = await User.getUsersFields(Array.from(urlMap.values()), ['userslug']);
-	slugs = slugs.map(({ userslug }) => userslug);
-	Array.from(urlMap.keys()).forEach((url, idx) => {
-		urlMap.set(url, `/user/${encodeURIComponent(slugs[idx])}`);
-	});
+	const urls = new Set();
+	if (!isMarkdown && anchors.size) {
+		anchors.forEach((match) => {
+			const [, url] = match;
+			urls.add(url);
+		});
+
+		// Filter out urls that don't backreference to a remote id
+		const backrefs = urls.size ? await db.getObjectFields('remoteUrl:uid', Array.from(urls)) : {};
+		const urlAsIdExists = await db.isSortedSetMembers('usersRemote:lastCrawled', Array.from(urls));
+		Array.from(urls).map(async (url, index) => {
+			if (backrefs[url] || urlAsIdExists[index]) {
+				urlMap.set(url, backrefs[url] || url);
+			}
+		});
+		let slugs = await User.getUsersFields(Array.from(urlMap.values()), ['userslug']);
+		slugs = slugs.map(({ userslug }) => userslug);
+		Array.from(urlMap.keys()).forEach((url, idx) => {
+			urlMap.set(url, `/user/${encodeURIComponent(slugs[idx])}`);
+		});
+	}
 
 	return { splitContent, matches, urlMap };
 }
@@ -497,20 +502,25 @@ Mentions.parseRaw = async (content, type = 'default') => {
 		}
 	}));
 
-	const parsed = splitContent.join('');
+	let parsed = splitContent.join('');
+
+	// Early return if no urls to modify
+	if (!urlMap.size) {
+		return parsed;
+	}
 
 	// Modify existing anchors to local profile
-	const $ = cheerio.load(parsed);
-	const anchors = $('a');
-	Array.from(anchors).forEach((anchor) => {
-		const $anchor = $(anchor);
-		const url = $anchor.attr('href');
-		if (urlMap.has(url)) {
-			$anchor.attr('href', urlMap.get(url));
-		}
+	const anchors = new Set(Array.from(parsed.matchAll(anchorRegex)).reverse());
+	if (!anchors.size) {
+		return parsed;
+	}
+
+	anchors.forEach(([match, href]) => {
+		const replacement = match.replace(href, urlMap.get(href));
+		parsed = parsed.split(match).join(replacement);
 	});
 
-	return $.html();
+	return parsed;
 };
 
 Mentions.clean = function (input, isMarkdown, stripBlockquote, stripCode) {

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "nodebb-plugin-mentions",
-  "version": "4.6.0",
+  "version": "4.6.2",
   "description": "NodeBB Plugin that allows users to mention other users by prepending an '@' sign to their username",
   "main": "library.js",
   "repository": {
@@ -22,16 +22,14 @@
     "compatibility": "^4.0.0"
   },
   "dependencies": {
-    "cheerio": "^1.0.0-rc.12",
     "html-entities": "^2.3.2",
     "lodash": "4.17.21",
-    "sanitize-html": "^2.13.0",
     "validator": "^13.0.0"
   },
   "devDependencies": {
-    "mocha": "10.4.0",
-    "eslint": "9.3.0",
+    "eslint": "^8.0.0",
     "eslint-config-nodebb": "0.2.1",
-    "eslint-plugin-import": "2.29.1"
+    "eslint-plugin-import": "2.29.1",
+    "mocha": "10.4.0"
   }
 }