gtx-cli 2.5.17 → 2.5.18

package/CHANGELOG.md

@@ -1,5 +1,11 @@
 # gtx-cli
 
+## 2.5.18
+
+### Patch Changes
+
+- [#851](https://github.com/generaltranslation/gt/pull/851) [`cf5f0e3`](https://github.com/generaltranslation/gt/commit/cf5f0e3f1537c304b7ea5703714ffb4956a7f6f4) Thanks [@fernando-aviles](https://github.com/fernando-aviles)! - Skip anchor ID fallback in codeblocks to avoid adding anchors to comments
+
 ## 2.5.17
 
 ### Patch Changes

package/dist/utils/addExplicitAnchorIds.js

@@ -30,6 +30,53 @@ function extractHeadingText(heading) {
     });
     return text;
 }
+/**
+ * Simple line-by-line heading extractor that skips fenced code blocks.
+ * Used as a fallback when MDX parsing fails.
+ */
+function extractHeadingsWithFallback(mdxContent) {
+    const headings = [];
+    const lines = mdxContent.split('\n');
+    let position = 0;
+    let inFence = false;
+    let fenceChar = null;
+    for (const line of lines) {
+        const fenceMatch = line.match(/^(\s*)(`{3,}|~{3,})/);
+        if (fenceMatch) {
+            const fenceString = fenceMatch[2];
+            if (!inFence) {
+                inFence = true;
+                fenceChar = fenceString;
+            }
+            else if (fenceChar &&
+                fenceString[0] === fenceChar[0] &&
+                fenceString.length >= fenceChar.length) {
+                inFence = false;
+                fenceChar = null;
+            }
+            continue;
+        }
+        if (inFence) {
+            continue;
+        }
+        const headingMatch = line.match(/^(#{1,6})\s+(.*)$/);
+        if (!headingMatch) {
+            continue;
+        }
+        const hashes = headingMatch[1];
+        const rawText = headingMatch[2];
+        const { cleanedText, explicitId } = parseHeadingContent(rawText);
+        if (cleanedText || explicitId) {
+            headings.push({
+                text: cleanedText,
+                level: hashes.length,
+                slug: explicitId ?? generateSlug(cleanedText),
+                position: position++,
+            });
+        }
+    }
+    return headings;
+}
 function parseHeadingContent(text) {
     // Support both {#id} and escaped \{#id\} forms
     const anchorMatch = text.match(/(\\\{#([^}]+)\\\}|\{#([^}]+)\})\s*$/);
@@ -67,25 +114,8 @@ export function extractHeadingInfo(mdxContent) {
     }
     catch (error) {
         console.warn(`Failed to parse MDX content: ${error instanceof Error ? error.message : String(error)}`);
-        // Fallback: simple regex-based extraction to keep IDs usable
-        const fallbackHeadings = [];
-        const headingRegex = /^(#{1,6})\s+(.*)$/gm;
-        let position = 0;
-        let match;
-        while ((match = headingRegex.exec(mdxContent)) !== null) {
-            const hashes = match[1];
-            const rawText = match[2];
-            const { cleanedText, explicitId } = parseHeadingContent(rawText);
-            if (cleanedText || explicitId) {
-                fallbackHeadings.push({
-                    text: cleanedText,
-                    level: hashes.length,
-                    slug: explicitId ?? generateSlug(cleanedText),
-                    position: position++,
-                });
-            }
-        }
-        return fallbackHeadings;
+        // Fallback: line-by-line extraction skipping fenced code blocks
+        return extractHeadingsWithFallback(mdxContent);
     }
     let position = 0;
     visit(processedAst, 'heading', (heading) => {
@@ -270,25 +300,52 @@ function applyInlineIds(translatedContent, idMappings, escapeAnchors) {
  */
 function applyInlineIdsStringFallback(translatedContent, idMappings, escapeAnchors) {
     let headingIndex = 0;
-    return translatedContent.replace(/^(#{1,6}\s+)(.*)$/gm, (match, prefix, text) => {
+    let inFence = false;
+    let fenceChar = null;
+    const processedLines = translatedContent.split('\n').map((line) => {
+        const fenceMatch = line.match(/^(\s*)(`{3,}|~{3,})/);
+        if (fenceMatch) {
+            const fenceString = fenceMatch[2];
+            if (!inFence) {
+                inFence = true;
+                fenceChar = fenceString;
+            }
+            else if (fenceChar &&
+                fenceString[0] === fenceChar[0] &&
+                fenceString.length >= fenceChar.length) {
+                inFence = false;
+                fenceChar = null;
+            }
+            return line;
+        }
+        if (inFence) {
+            return line;
+        }
+        const headingMatch = line.match(/^(#{1,6}\s+)(.*)$/);
+        if (!headingMatch) {
+            return line;
+        }
+        const prefix = headingMatch[1];
+        const text = headingMatch[2];
         const id = idMappings.get(headingIndex++);
         if (!id) {
-            return match;
+            return line;
         }
         const hasEscaped = /\\\{#[^}]+\\\}\s*$/.test(text);
         const hasUnescaped = /\{#[^}]+\}\s*$/.test(text);
         if (hasEscaped) {
-            return match;
+            return line;
         }
         if (hasUnescaped) {
             if (!escapeAnchors) {
-                return match;
+                return line;
             }
             return `${prefix}${text.replace(/\{#([^}]+)\}\s*$/, '\\\\{#$1\\\\}')}`;
         }
         const suffix = escapeAnchors ? ` \\{#${id}\\}` : ` {#${id}}`;
         return `${prefix}${text}${suffix}`;
     });
+    return processedLines.join('\n');
 }
 /**
  * Wraps headings in divs with IDs (Mintlify approach)

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "gtx-cli",
-  "version": "2.5.17",
+  "version": "2.5.18",
   "main": "dist/index.js",
   "bin": "dist/main.js",
   "files": [