Improve a725579

Author: haydenbleasel

packages/remend/__tests__/inline-code.test.ts

@@ -64,6 +64,17 @@ describe("inline code formatting (`)", () => {
   });
 });
 
+describe("escaped backticks in inline code", () => {
+  it("should not treat escaped backticks as code delimiters", () => {
+    // \` is not a real backtick delimiter, so **bold should still be completed
+    expect(remend("\\`not code\\` **bold")).toBe("\\`not code\\` **bold**");
+  });
+
+  it("should complete emphasis when only escaped backticks are present", () => {
+    expect(remend("\\` *italic")).toBe("\\` *italic*");
+  });
+});
+
 describe("emphasis markers inside inline code spans should not leak", () => {
   it("should not complete bold/italic/strikethrough if they are inside inline code", () => {
     expect(remend("`**bold`")).toBe("`**bold`");

packages/remend/__tests__/mixed-formatting.test.ts

@@ -48,7 +48,9 @@ describe("mixed formatting", () => {
   });
 
   it("should handle deeply nested incomplete formatting", () => {
-    // Formats are closed in the order they're processed
+    // Formats are closed in the order they're processed.
+    // The ~~ isn't closed because it ends up inside the completed inline code span (`code ~~strike`),
+    // so remend correctly leaves it alone.
     expect(remend("**bold *italic `code ~~strike")).toBe(
       "**bold *italic `code ~~strike*`"
     );

packages/remend/src/code-block-utils.ts

@@ -5,6 +5,12 @@ export const isInsideCodeBlock = (text: string, position: number): boolean => {
   let inMultilineCode = false;
 
   for (let i = 0; i < position; i += 1) {
+    // Skip escaped backticks
+    if (text[i] === "\\" && i + 1 < text.length && text[i + 1] === "`") {
+      i += 1;
+      continue;
+    }
+
     // Check for triple backticks (multiline code blocks)
     if (text.substring(i, i + 3) === "```") {
       inMultilineCode = !inMultilineCode;
@@ -30,10 +36,15 @@ export const isPartOfTripleBacktick = (text: string, i: number): boolean => {
   return isTripleStart || isTripleMiddle || isTripleEnd;
 };
 
-// Counts single backticks that are not part of triple backticks
+// Counts single backticks that are not part of triple backticks or escaped
 export const countSingleBackticks = (text: string): number => {
   let count = 0;
   for (let i = 0; i < text.length; i += 1) {
+    // Skip escaped backticks
+    if (text[i] === "\\" && i + 1 < text.length && text[i + 1] === "`") {
+      i += 1;
+      continue;
+    }
     if (text[i] === "`" && !isPartOfTripleBacktick(text, i)) {
       count += 1;
     }
@@ -52,6 +63,12 @@ export const isWithinCompleteInlineCode = (
   let inlineCodeStart = -1;
 
   for (let i = 0; i < text.length; i += 1) {
+    // Skip escaped backticks
+    if (text[i] === "\\" && i + 1 < text.length && text[i + 1] === "`") {
+      i += 1;
+      continue;
+    }
+
     // Check for triple backticks (multiline code blocks)
     if (text.substring(i, i + 3) === "```") {
       inMultilineCode = !inMultilineCode;

packages/remend/src/comparison-operator-handler.ts

@@ -2,7 +2,7 @@
 // AI models frequently generate list items like "- > 25: expensive" where
 // the > is a "greater than" operator, not a blockquote marker.
 
-import { isWithinCodeBlock } from "./utils";
+import { isInsideCodeBlock } from "./code-block-utils";
 
 // Match list items where > appears as a comparison operator followed by a digit
 // Pattern: list marker (-, *, +, or 1.) followed by > then optional = and a digit
@@ -23,7 +23,7 @@ export const handleComparisonOperators = (text: string): string => {
     LIST_COMPARISON_PATTERN,
     (match, prefix, suffix, offset) => {
       // Don't escape inside code blocks
-      if (isWithinCodeBlock(text, offset)) {
+      if (isInsideCodeBlock(text, offset)) {
         return match;
       }
 

packages/remend/src/emphasis-handlers.ts

@@ -677,7 +677,10 @@ export const handleIncompleteSingleUnderscoreItalic = (
     return text;
   }
 
-  if (isWithinCompleteInlineCode(text, firstSingleUnderscoreIndex)) {
+  if (
+    isInsideCodeBlock(text, firstSingleUnderscoreIndex) ||
+    isWithinCompleteInlineCode(text, firstSingleUnderscoreIndex)
+  ) {
     return text;
   }