fix(chunkers): accept single-quoted FAQ items with trailing commas

session-policies.mdx and verified-domains.mdx write FAQ items with
single-quoted multiline values and trailing commas; the double-quote-only
item pattern matched nothing there, so the component consumer replaced
those whole FAQ blocks with a space. Capture either quote style
escape-aware (quotes of the other style inside a value are fine) and
allow the trailing comma; captured values keep their quotes and are
unquoted before unescaping.

Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>

Author: j15z

apps/sim/lib/chunkers/docs-chunker.test.ts

@@ -74,6 +74,32 @@ describe('cleanContent FAQ extraction', () => {
     expect(cleaned).toContain('Yes, on enterprise plans.')
   })
 
+  it('extracts single-quoted multiline items with trailing commas (session-policies shape)', () => {
+    const cleaned = cleanContent(
+      [
+        '<FAQ',
+        '  items={[',
+        '    {',
+        "      question: 'Do session policies apply to SSO sign-ins?',",
+        '      answer:',
+        "        'Yes. Sessions created through SSO follow the same limits.',",
+        '    },',
+        '    {',
+        '      question: \'Does "Sign out all members" affect API keys?\',',
+        "      answer: 'No. API keys are unaffected.',",
+        '    },',
+        '  ]}',
+        '/>',
+      ].join('\n')
+    )
+
+    expect(cleaned).toContain('Do session policies apply to SSO sign-ins?')
+    expect(cleaned).toContain('Yes. Sessions created through SSO follow the same limits.')
+    expect(cleaned).toContain('Does "Sign out all members" affect API keys?')
+    expect(cleaned).toContain('No. API keys are unaffected.')
+    expect(cleaned).not.toContain('items=')
+  })
+
   it('unescapes escaped quotes in extracted strings', () => {
     const cleaned = cleanContent(
       '<FAQ items={[ { question: "What does \\"draft\\" mean?", answer: "An unsaved workflow." } ]} />'

apps/sim/lib/chunkers/docs-chunker.ts

@@ -22,12 +22,19 @@ interface Frontmatter {
 const logger = createLogger('DocsChunker')
 
 /**
- * One `{ question: "...", answer: "..." }` FAQ item. Quoted strings are
- * consumed escape-aware, so braces or quotes inside an answer never end a
- * match early.
+ * One `{ question: "...", answer: "..." }` FAQ item, in either quote style and
+ * with an optional trailing comma (`session-policies.mdx` uses single-quoted
+ * multiline items). Each captured value keeps its surrounding quotes — the
+ * quoted strings are consumed escape-aware per style, so quotes of the other
+ * style, braces, or escapes inside an answer never end a match early.
  */
 const FAQ_ITEM_PATTERN =
-  /\{\s*question:\s*"((?:[^"\\]|\\.)*)"\s*,\s*answer:\s*"((?:[^"\\]|\\.)*)"\s*\}/g
+  /\{\s*question:\s*("(?:[^"\\]|\\.)*"|'(?:[^'\\]|\\.)*')\s*,\s*answer:\s*("(?:[^"\\]|\\.)*"|'(?:[^'\\]|\\.)*')\s*,?\s*\}/g
+
+/** Strip a captured value's surrounding quotes (either style), then unescape. */
+function unquoteJsxString(value: string): string {
+  return unescapeJsxString(value.slice(1, -1))
+}
 
 function unescapeJsxString(value: string): string {
   return value.replace(/\\(.)/g, (_, char: string) =>
@@ -47,7 +54,7 @@ function unescapeJsxString(value: string): string {
 function extractFaqProse(items: string): string {
   const lines: string[] = []
   for (const match of items.matchAll(FAQ_ITEM_PATTERN)) {
-    lines.push(unescapeJsxString(match[1]), unescapeJsxString(match[2]))
+    lines.push(unquoteJsxString(match[1]), unquoteJsxString(match[2]))
   }
   if (lines.length === 0) return ' '
   return `\n${lines.join('\n').replace(/[<>{}]/g, '')}\n`