fix(comparison): fix two more FAQ text-mangling bugs found during final audit

waleedlatif1 · waleedlatif1 · commit d466ba02b7a2 · 2026-07-03T07:25:15.000-07:00
- lowercaseFirst only guarded against 2+ CONSECUTIVE leading capitals
  (acronyms like "AI"/"SSO"), so CamelCase brand names with a single
  leading capital (LangChain, OpenClaw, CrewAI) got their first letter
  wrongly lowercased ("langChain provides..."). Now checks for 2+
  uppercase letters anywhere in the leading word, which covers both
  acronyms and CamelCase brand names.
- parseFactValue and summarizeFact's boolean-prefix stripping only
  recognized "Yes:"/"No:" (colon), but "Yes, ..."/"No, ..." (comma) is
  an equally common phrasing already used across ~15 existing facts
  (stackai, pipedream, workato, zapier, etc.), so those facts kept
  their leading comma when stitched into an FAQ answer (e.g. "StackAI:
  , broad support..."). Both now accept either separator.

Found by systematically sweeping every "Sim vs X" FAQ answer across
all 20 competitor pages for garbled/mis-cased text, not just the 5
newly added ones.
diff --git a/apps/sim/app/(landing)/comparison/fact-status.ts b/apps/sim/app/(landing)/comparison/fact-status.ts
@@ -10,15 +10,17 @@ export interface ParsedFact {
 // The negative lookahead (?![a-zA-Z]) requires the "Yes"/"No" token to end at a word
 // boundary, so values like "Not documented" or "Not publicly documented" (which start
 // with the letters "No" but aren't the boolean token) fall through to 'neutral' instead
-// of being misread as a "No" status.
-const STATUS_PREFIX = /^(Yes|No)(?![a-zA-Z])(?::\s*)?(.*)$/s
+// of being misread as a "No" status. The separator group accepts either a colon
+// ("Yes: ...") or a comma ("Yes, but ...") since both are used throughout the dataset.
+const STATUS_PREFIX = /^(Yes|No)(?![a-zA-Z])(?:[:,]\s*)?(.*)$/s
 
 /**
  * Splits a {@link Fact.value} string into a status (for a compact icon) and
  * the remaining descriptive text. Every fact in `apps/sim/lib/compare/data`
- * that represents a yes/no capability is written as `"Yes: ..."` / `"No:
- * ..."`. This is the single place that convention is parsed, so the
- * comparison table and the key-differences strip render it identically.
+ * that represents a yes/no capability is written as `"Yes: ..."` / `"No: ..."`,
+ * or occasionally `"Yes, ..."` / `"No, ..."` as a more natural continuation.
+ * This is the single place that convention is parsed, so the comparison table
+ * and the key-differences strip render it identically.
  */
 export function parseFactValue(value: string): ParsedFact {
   const match = value.match(STATUS_PREFIX)
diff --git a/apps/sim/app/(landing)/comparison/utils.ts b/apps/sim/app/(landing)/comparison/utils.ts
@@ -204,21 +204,29 @@ export function ensurePeriod(value: string): string {
   return /[.!?]$/.test(value) ? value : `${value}.`
 }
 
-/** Lowercases the first letter of `value`, unless it starts with an acronym (e.g. "AI", "SSO", "MCP"). */
+/**
+ * Lowercases the first letter of `value`, unless its leading word is an acronym
+ * (e.g. "AI", "SSO", "MCP") or a CamelCase brand name (e.g. "LangChain",
+ * "OpenClaw", "CrewAI") - detected by 2+ uppercase letters anywhere in that
+ * word, not just consecutive at the start, since lowercasing either would
+ * mangle a proper noun ("langChain", "openClaw").
+ */
 export function lowercaseFirst(value: string): string {
   if (value.length === 0) return value
-  // Leave a leading acronym (2+ consecutive capitals, e.g. "AI", "SSO", "MCP") alone.
-  if (/^[A-Z]{2,}/.test(value)) return value
+  const leadingWord = value.match(/^[A-Za-z]+/)?.[0] ?? ''
+  const upperCaseCount = (leadingWord.match(/[A-Z]/g) ?? []).length
+  if (upperCaseCount >= 2) return value
   return value.charAt(0).toLowerCase() + value.slice(1)
 }
 
 /**
  * Composes {@link firstSentence} + {@link lowercaseFirst} + {@link ensurePeriod} for
- * stitching a fact value mid-sentence. Strips a leading "Yes:"/"No:" token first so
- * boolean facts don't produce mid-sentence "yes: ..."/"no: ..." fragments.
+ * stitching a fact value mid-sentence. Strips a leading "Yes:"/"No:" (or the
+ * comma-separated "Yes,"/"No,") token first so boolean facts don't produce
+ * mid-sentence "yes: ..."/"no: ..." fragments.
  */
 function summarizeFact(value: string): string {
-  const stripped = value.replace(/^(Yes|No)(?![a-zA-Z])(?::\s*)?/, '').trim()
+  const stripped = value.replace(/^(Yes|No)(?![a-zA-Z])(?:[:,]\s*)?/, '').trim()
   const base = stripped.length > 0 ? stripped : value
   return ensurePeriod(lowercaseFirst(firstSentence(base)))
 }
