fix: escape only pipes

ubiquity-os · Jun 18, 2024 · 6e24973 · 6e24973
1 parent 05c505c
commit 6e24973
Show file tree

Hide file tree

Showing 2 changed files with 4 additions and 28 deletions.
diff --git a/src/github/handlers/issue-comment-created.ts b/src/github/handlers/issue-comment-created.ts
@@ -31,32 +31,6 @@ export default async function issueCommentCreated(context: GitHubContext<"issue_
 /**
  * Ensures that passed content does not break MD display within the table.
  */
-function escapeMarkdown(text: string) {
-  const replacements = {
-    "\\": "\\\\",
-    "`": "\\`",
-    "*": "\\*",
-    _: "\\_",
-    "{": "\\{",
-    "}": "\\}",
-    "[": "\\[",
-    "]": "\\]",
-    "(": "\\(",
-    ")": "\\)",
-    "#": "\\#",
-    "+": "\\+",
-    "-": "\\-",
-    ".": "\\.",
-    "!": "\\!",
-    "|": "\\|",
-    "~": "\\~",
-    ">": "\\>",
-    "<": "\\<",
-  };
-
-  return text.replace(/[\\`*_{}[\]()#+\-.!|~><]/g, (match) => replacements[match as keyof typeof replacements]);
-}
-
 function getContent(content: string | undefined) {
-  return content ? escapeMarkdown(content) : "-";
+  return content ? content.replace("|", "\\|") : "-";
 }
diff --git a/tests/events.test.ts b/tests/events.test.ts
@@ -72,7 +72,9 @@ describe("Event related tests", () => {
     expect(spy.mock.calls).toEqual([
       [
         {
-          body: "### Available Commands\n\n\n| Command | Description | Example |\n|---|---|---|\n| `/help` | List all available commands. | `/help` |\n| `/command` | Plugin A | `/command \\[foo \\| bar\\]` |",
+          body:
+            "### Available Commands\n\n\n| Command | Description | Example |\n|---|---|---|\n| `/help` | List" +
+            " all available commands. | `/help` |\n| `/command` | Plugin A | `/command [foo \\| bar]` |",
           issue_number: 1,
           owner: "ubiquity",
           repo: "ubiquibot-kernel",