improvement(compare): tighten comparison-page copy, cut self-referential hedging#5390
Conversation
…ial hedging Across sim.ts and all 20 competitor profiles, remove phrasing that describes a claim's own provenance/reliability (e.g. 'marketing claim', 'marketing copy states', 'as of this check') in favor of stating the fact directly. Also trims overlong run-on sentences into shorter, more direct ones. No numbers, dates, comparisons, confidence levels, or sources changed.
|
The latest updates on your projects. Learn more about Vercel for GitHub. |
PR SummaryLow Risk Overview Wording-only edits to Reviewed by Cursor Bugbot for commit e10173c. Configure here. |
Greptile SummaryThis PR is a copy-only editorial pass across 21 TypeScript compare-page data files, removing hedging phrases ("marketing claim," "marketing copy states," "as of this check") and splitting comma/semicolon run-on sentences into shorter, direct statements. No numeric values, dates,
Confidence Score: 5/5Pure copy edits to static TypeScript data files — no logic, no types, no schema, no sources changed. Safe to merge. All 21 files are static data objects containing comparison-page copy. No computation, routing, or rendering logic was touched. Two previously-flagged concerns about factual framing in sim.ts (community size presented as fact, compliance shortValue shortened to just 'SOC2') exist in the diff but were already raised in prior review comments and are content judgement calls rather than code defects. apps/sim/lib/compare/data/sim.ts — the community and additionalCompliance fields carry open questions from previous review comments about how much hedging to preserve for self-reported figures. Important Files Changed
Flowchart%%{init: {'theme': 'neutral'}}%%
flowchart TD
A[sim.ts\nSim's own profile] -->|left-hand column| P[Compare Page Renderer]
B[competitors/\n20 x *.ts profiles] -->|right-hand column| P
P --> C{Field rendered}
C -->|primary cell| V[value / shortValue]
C -->|expandable detail| D[detail]
C -->|standout cards| SF[standoutFeatures]
C -->|limitation cards| LF[limitations]
V --> R[Visitor sees claim]
D --> R2[Visitor expands detail]
SF --> R3[Sim advantages shown]
LF --> R4[Competitor limitations shown]
style A fill:#f9f,stroke:#333
style B fill:#bbf,stroke:#333
%%{init: {'theme': 'base', 'themeVariables': {"darkMode": true, "background": "#0d1117", "primaryColor": "#21262d", "primaryTextColor": "#e6edf3", "primaryBorderColor": "#8b949e", "lineColor": "#8b949e", "textColor": "#e6edf3", "edgeLabelBackground": "#161b22", "actorBkg": "#21262d", "actorBorder": "#8b949e", "actorTextColor": "#e6edf3", "actorLineColor": "#8b949e", "signalColor": "#8b949e", "signalTextColor": "#e6edf3", "noteBkgColor": "#373320", "noteBorderColor": "#d4a72c", "noteTextColor": "#f0e6c0", "labelBoxBkgColor": "#21262d", "labelBoxBorderColor": "#8b949e", "labelTextColor": "#e6edf3", "loopTextColor": "#e6edf3", "activationBkgColor": "#30363d", "activationBorderColor": "#8b949e"}}}%%
flowchart TD
A[sim.ts\nSim's own profile] -->|left-hand column| P[Compare Page Renderer]
B[competitors/\n20 x *.ts profiles] -->|right-hand column| P
P --> C{Field rendered}
C -->|primary cell| V[value / shortValue]
C -->|expandable detail| D[detail]
C -->|standout cards| SF[standoutFeatures]
C -->|limitation cards| LF[limitations]
V --> R[Visitor sees claim]
D --> R2[Visitor expands detail]
SF --> R3[Sim advantages shown]
LF --> R4[Competitor limitations shown]
style A fill:#f9f,stroke:#333
style B fill:#bbf,stroke:#333
Reviews (2): Last reviewed commit: "fix(compare): avoid accidental boolean-i..." | Re-trigger Greptile |
…ning Three facts whose tightened wording started with a bare 'No ' now matched the Yes/No boolean-icon convention (parseFactValue), collapsing a nuanced 'Partial' or plain descriptive sentence into a bare X icon. Rephrase with 'Not'/'not' so these render as full text again, matching pre-edit behavior.
|
@cursor review |
There was a problem hiding this comment.
✅ Bugbot reviewed your changes and found no new issues!
Comment @cursor review or bugbot run to trigger another review on this PR
Reviewed by Cursor Bugbot for commit e10173c. Configure here.
Summary
value/detail/shortValuefields, standout features, and limitations.confidencelevels, orsourceswere changed anywhere; theYes:/No:boolean-value convention is untouched.Type of Change
Testing
tsc --noEmitclean on all 21 touched filesbiome check --writecleansources/url/asOf/confidencefield across the whole diffChecklist