From 05842b5143072e7334f205850848718e565d8b96 Mon Sep 17 00:00:00 2001
From: Rex <rex.garland@gmail.com>
Date: Mon, 7 Aug 2023 11:43:24 -0700
Subject: [PATCH] Only use regexes supported by Safari (#38)

---
 __test__/slackify-markdown.test.js |  8 ++++++++
 package.json                       |  2 +-
 src/slackify.js                    | 19 +++++++++++++++----
 3 files changed, 24 insertions(+), 5 deletions(-)

diff --git a/__test__/slackify-markdown.test.js b/__test__/slackify-markdown.test.js
index ea69d36..21ce784 100644
--- a/__test__/slackify-markdown.test.js
+++ b/__test__/slackify-markdown.test.js
@@ -1,3 +1,5 @@
+const fsPromises = require('fs/promises');
+
 const slackifyMarkdown = require('..');
 
 const zws = String.fromCharCode(0x200B); // zero-width-space
@@ -242,3 +244,9 @@ test('User mention', () => {
 
   expect(slackifyMarkdown(mrkdown)).toBe(slack);
 });
+
+test('code does not use negative lookarounds', async () => {
+  const sourceCode = await fsPromises.readFile('./src/slackify.js', 'utf8');
+  const usesNegativeLookaround = /\/\(\?<?!/;
+  expect(sourceCode).not.toMatch(usesNegativeLookaround);
+});
diff --git a/package.json b/package.json
index e3f8cf0..3fa258f 100644
--- a/package.json
+++ b/package.json
@@ -1,6 +1,6 @@
 {
   "name": "slackify-markdown",
-  "version": "4.3.1",
+  "version": "4.3.2",
   "description": "Convert markdown into Slack-specific markdown",
   "license": "MIT",
   "main": "index.js",
diff --git a/src/slackify.js b/src/slackify.js
index 3914622..6e50c5b 100644
--- a/src/slackify.js
+++ b/src/slackify.js
@@ -6,6 +6,20 @@ const { wrap, isURL, isPotentiallyEncoded } = require('./utils');
 // fixes slack in-word formatting (e.g. hel*l*o)
 const zeroWidthSpace = String.fromCharCode(0x200B);
 
+const escapeSpecials = text => {
+  const escaped = text
+    .replace('&', '&amp;')
+    .replace(/<([^@]|$)/g, (_, m) => `&lt;${m}`)
+    .replace(/^(.*)>/g, (_, m) => {
+      const isEndOfUserMention = Boolean(m.match(/<@[A-Z0-9]+$/));
+      if (isEndOfUserMention) {
+        return `${m}>`;
+      }
+      return `${m}&gt;`;
+    });
+  return escaped;
+};
+
 /**
  * Creates custom `mdast-util-to-markdown` handlers that tailor the output for
  * Slack Markdown.
@@ -120,10 +134,7 @@ const createHandlers = definitions => ({
   text: (node, _parent, context) => {
     const exit = context.enter('text');
     // https://api.slack.com/reference/surfaces/formatting#escaping
-    const text = node.value
-      .replace(/&/g, '&amp;')
-      .replace(/<(?!@)/g, '&lt;')
-      .replace(/(?<!@[A-Z0-9]+)>/g, '&gt;');
+    const text = escapeSpecials(node.value);
     exit();
 
     // Do we need more escaping like the default handler uses?