Repository navigation
fix(comparisons): lead Flowise copy with its end of life, qualify Retool self-hosting, and note the Zapier Agents migration - #9267
Conversation
…ool self-hosting, and note the Zapier Agents migration
[Medium impact] The PR appears safe to merge; no new actionable issues remain. SummaryUpdates comparison copy, fixes shortened FAQ answers, and keeps
Reviews (4) · Last reviewed commit: "fix(comparisons): don't add a period aft..." · Reviewed by Greptile |
|
@greptile |
|
@cubic-dev-ai review this PR |
@waleedlatif1 I have started the AI code review. It will take a few minutes to complete. |
There was a problem hiding this comment.
All reported issues were addressed across 6 files
Reply with feedback, questions, or to request a fix.
Turn on auto-fix | Re-trigger cubic
… Flowise's end of life to the open-source project
|
@greptile |
|
@cubic-dev-ai review this PR |
@waleedlatif1 I have started the AI code review. It will take a few minutes to complete. |
|
@greptile |
|
@cubic-dev-ai review this PR |

Summary
firstSentencein the comparisons utils cut at any period. It feeds the generated FAQ answers and FAQPage JSON-LD, so live pages were publishing fragments like "usage-based credits indexed at $0." (Gumloop pricing), "seat-based (separate builder vs." (Retool pricing), "...EU-U." (Gumloop compliance), "PCI DSS v4." (OpenAI AgentKit, Workato), and "...Opus 4." (Microsoft Copilot, Power Automate). It now ends a sentence only at.,!, or?followed by whitespace and a capital, quote, or paren, or at the end of the string. Decimals and abbreviations (vs.,incl.,etc.,e.g.,U.S.) no longer end a sentence. This changes 13 FAQ answers across 11 comparison pages, and each one is now a complete sentence.lib/landing/routes.ts: the comment now says three consumers instead of two, and/index.md(the root Markdown twin) joins the marketing documents so self-hosted installs sendnoindexfor it. The proxy matcher already runs for/index.md.Type of Change
Testing
bunx vitest run proxy.test.ts: 31 passed. I added/index.mdto the existing indexing-policy and matcher assertions. With the oldroutes.ts, the self-hosted noindex assertion fails (expected null to be 'noindex, nofollow').buildComparisonFaqsfor every competitor before and after the splitter change. It also prints where the newfirstSentencecuts the self-hosting, pricing model, compliance, and multi-LLM facts for Sim and every competitor. 13 answers changed, all from truncated fragments to full sentences. Every cut now ends at a real sentence end. The first run found a cut at "(incl.", which is whyincl.is on the abbreviation list.bunx biome checkon the changed files: clean.Checklist
test-auditauthoring gate)