fix: Resolve remaining CodeRabbit issues in adocToMarkdown converter

- Fix missing blank line after table: return '\n\n' from convertAdocTable
- Move cross-reference resolution before def-list conversion so terms
  like <<spc,SPC>>:: are correctly expanded before matching
- Use non-greedy term matching in def-list regex ([^\n|#`>]+?) to handle
  terms containing colons (e.g. "Anti-pattern: Ice cream cone::")

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>

Author: raifdmueller

scripts/generate-llms-txt.js

@@ -59,7 +59,7 @@ function convertAdocTable(body) {
   for (const row of rows.slice(1)) {
     if (row.length > 0) out.push('| ' + row.join(' | ') + ' |')
   }
-  return out.join('\n')
+  return out.join('\n') + '\n\n'
 }
 
 // ─── AsciiDoc → Markdown converter ──────────────────────────────────────────
@@ -100,17 +100,6 @@ function adocToMarkdown(adoc) {
   // AsciiDoc line continuation (+) → remove
   md = md.replace(/^\+\s*$/gm, '')
 
-  // Nested definition lists: term::: description → - **term**: description
-  md = md.replace(/^([^:\n|#`>]+):::[^\S\n]*(.*)$/gm, (_, term, desc) =>
-    desc.trim() ? `- **${term.trim()}**: ${desc.trim()}` : `- **${term.trim()}**`
-  )
-
-  // Definition lists: term:: description → **term**: description
-  // Use [^\S\n]* (horizontal whitespace only) to avoid matching across newlines
-  md = md.replace(/^([^:\n|#`>]+)::[^\S\n]*(.*)$/gm, (_, term, desc) =>
-    desc.trim() ? `**${term.trim()}**: ${desc.trim()}` : `**${term.trim()}**`
-  )
-
   // Links: link:url[text] → [text](url), resolve relative .adoc paths to GitHub URLs
   md = md.replace(/link:([^\[]+)\[([^\]]*)\]/g, (_, url, text) => {
     if (/^\.\.\/.*\.adoc$/.test(url)) {
@@ -120,9 +109,22 @@ function adocToMarkdown(adoc) {
   })
 
   // Cross-references: <<id,text>> → text, <<id>> → `id`
+  // Must run before def-list conversion so terms like <<spc,SPC>>:: are resolved first
   md = md.replace(/<<([^,>]+),([^>]+)>>/g, '$2')
   md = md.replace(/<<([^>]+)>>/g, '`$1`')
 
+  // Nested definition lists: term::: description → - **term**: description
+  // Non-greedy term match allows colons in term (e.g. "Anti-pattern: X:::")
+  md = md.replace(/^([^\n|#`>]+?):::(?!:)[^\S\n]*(.*)$/gm, (_, term, desc) =>
+    desc.trim() ? `- **${term.trim()}**: ${desc.trim()}` : `- **${term.trim()}**`
+  )
+
+  // Definition lists: term:: description → **term**: description
+  // Non-greedy term match allows colons in term (e.g. "Anti-pattern: Ice cream cone::")
+  md = md.replace(/^([^\n|#`>]+?)::[^\S\n]*(.*)$/gm, (_, term, desc) =>
+    desc.trim() ? `**${term.trim()}**: ${desc.trim()}` : `**${term.trim()}**`
+  )
+
   // Bold: **text** stays, *text* → **text**
   md = md.replace(/(?<![*\w])\*([^*\n]+)\*(?![*\w])/g, '**$1**')
 

website/public/llms.txt

@@ -92,7 +92,9 @@ These terms are frequently used but **lack the depth and definition** required f
 | **"Keep it short"** | Pure directive with no conceptual depth. It's an instruction, not a methodology. |
 | **"Best practices"** | Too vague and not attributable. Whose best practices? Based on what research or authority? |
 | **"Modern approach"** | Ambiguous and inconsistent. "Modern" means different things to different people and changes over time. |
-| **"Make it simple"** | No reference to specific simplification frameworks (unlike KISS principle or Occam's Razor which **are** semantic anchors). |### Comparison: Good vs. Bad
+| **"Make it simple"** | No reference to specific simplification frameworks (unlike KISS principle or Occam's Razor which **are** semantic anchors). |
+
+### Comparison: Good vs. Bad
 
 **Example 1: Testing Instructions**
 
@@ -1895,7 +1897,7 @@ Filter by constraints (e.g., "AsciiDoc + Hugo not well-supported")
 
 **Nelson Rules**: 8 rules for pattern recognition on Control Charts
 
-SPC:: Control Charts are the central tool of Statistical Process Control
+**SPC**: Control Charts are the central tool of Statistical Process Control
 
 **Six Sigma**: Control Charts are used in the Control phase of DMAIC
 
@@ -1942,7 +1944,7 @@ SPC:: Control Charts are the central tool of Statistical Process Control
 
 **Control Chart (Shewhart)**: Nelson Rules are applied to Control Charts
 
-SPC:: Nelson Rules are a tool within Statistical Process Control
+**SPC**: Nelson Rules are a tool within Statistical Process Control
 
 **Western Electric Rules**: Predecessor; Nelson Rules extend these with Rules 5-8
 
@@ -2395,7 +2397,7 @@ SPC:: Nelson Rules are a tool within Statistical Process Control
 
 **Feedback loops**: Faster feedback from lower levels
 
-Anti-pattern: Ice cream cone:: Too many E2E tests, too few unit tests
+**Anti-pattern: Ice cream cone**: Too many E2E tests, too few unit tests
 
 **Test at the right level**: Don't test through UI what can be tested in isolation