Skip to content

improvement(compare): tighten comparison-page copy, cut self-referential hedging#5390

Merged
waleedlatif1 merged 2 commits into
stagingfrom
worktree-simplify-comparison-copy
Jul 3, 2026
Merged

improvement(compare): tighten comparison-page copy, cut self-referential hedging#5390
waleedlatif1 merged 2 commits into
stagingfrom
worktree-simplify-comparison-copy

Conversation

@waleedlatif1

Copy link
Copy Markdown
Collaborator

Summary

  • Removes self-referential hedging phrases across sim.ts and all 20 competitor profiles (e.g. "marketing claim", "marketing copy states", "as of this check", "it should be noted that") in favor of stating each fact directly.
  • Trims overlong, comma/semicolon-run-on sentences into shorter, more direct ones across value/detail/shortValue fields, standout features, and limitations.
  • No numbers, dates, comparisons, confidence levels, or sources were changed anywhere; the Yes:/No: boolean-value convention is untouched.

Type of Change

  • Copy/content improvement

Testing

  • tsc --noEmit clean on all 21 touched files
  • biome check --write clean
  • Grep-verified zero em-dashes, zero remaining "marketing claim"/"marketing copy" phrasing, zero diffs on any sources/url/asOf/confidence field across the whole diff

Checklist

  • Code follows project style guidelines
  • Self-reviewed my changes
  • Tests added/updated and passing
  • No new warnings introduced
  • I confirm that I have read and agree to the terms outlined in the Contributor License Agreement (CLA)

…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.
@vercel

vercel Bot commented Jul 3, 2026

Copy link
Copy Markdown

The latest updates on your projects. Learn more about Vercel for GitHub.

1 Skipped Deployment
Project Deployment Actions Updated (UTC)
docs Skipped Skipped Jul 3, 2026 7:48pm

Request Review

@cursor

cursor Bot commented Jul 3, 2026

Copy link
Copy Markdown

PR Summary

Low Risk
Static content strings in compare data only; no runtime logic, APIs, or auth changes.

Overview
Tightens marketing copy in the compare feature’s competitor data, including large rewrites in claude-cowork.ts, crewai.ts, and dust.ts (and the rest of the competitor set per the PR).

Wording-only edits to oneLiner, standout features, limitations, and nested facts fields (value, detail, shortValue): shorter sentences, less hedging (e.g. dropping “as of this check”, “marketing claim”, “documented” filler), and more direct statements. No changes to factual claims’ structure, confidence, sources, URLs, dates, or Yes/No fact conventions.

Reviewed by Cursor Bugbot for commit e10173c. Configure here.

Comment thread apps/sim/lib/compare/data/competitors/claude-cowork.ts Outdated
@greptile-apps

greptile-apps Bot commented Jul 3, 2026

Copy link
Copy Markdown
Contributor

Greptile Summary

This 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 levels, sources URLs, or TypeScript types were changed.

  • 21 data files updatedsim.ts (Sim's own profile) and all 20 competitor profiles receive prose tightening across value, detail, shortValue, description, and oneLiner fields.
  • sim.ts key changesintegrationCount.detail now explains the "1,000+ integrations" marketing figure relative to the 302/3,900 codebase counts; soc2.value removes the "(marketing claim)" parenthetical (appropriate since confidence: 'verified'); executionLimits.detail drops the explicit note that figures come from the codebase config rather than published docs.
  • Previously-flagged concerns remaincommunity.value now presents the self-reported 100k figure as fact without the original attribution ("Marketing copy states…"), and additionalCompliance.value/shortValue are both shortened to just 'SOC2', dropping the explicit "no HIPAA / ISO 27001 / PCI / FedRAMP" qualifier that was visible in the primary table cell.

Confidence Score: 5/5

Pure 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

Filename Overview
apps/sim/lib/compare/data/sim.ts Editorial prose tightening across 15+ fields; community.value and additionalCompliance.value/shortValue lose important hedging context flagged in previous review comments
apps/sim/lib/compare/data/competitors/openclaw.ts Run-on sentences split, hedging removed; learningCurve.value semicolon replaced with comma (minor comma-splice), no factual changes
apps/sim/lib/compare/data/competitors/zapier.ts Clean prose tightening throughout; templates.detail drops "and marketing pages" from a URL description, no factual impact
apps/sim/lib/compare/data/competitors/microsoft-copilot.ts Compliance description changed from "own admin documentation lists coverage under…" to "is certified under…" — slightly more assertive but confidence: 'verified' with source URLs supports this
apps/sim/lib/compare/data/competitors/claude-cowork.ts Consistent sentence-splitting and hedging removal; no factual changes
apps/sim/lib/compare/data/competitors/langchain.ts All changes are sentence splits and comma-to-period substitutions; no factual content altered
apps/sim/lib/compare/data/competitors/crewai.ts Prose tightening only, no factual changes
apps/sim/lib/compare/data/competitors/make.ts Hedging removed from learning-curve description; clean otherwise

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
Loading
%%{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
Loading

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.
@waleedlatif1

Copy link
Copy Markdown
Collaborator Author

@greptile

@waleedlatif1

Copy link
Copy Markdown
Collaborator Author

@cursor review

@cursor cursor Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

✅ 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.

@waleedlatif1 waleedlatif1 merged commit 8937005 into staging Jul 3, 2026
18 checks passed
@waleedlatif1 waleedlatif1 deleted the worktree-simplify-comparison-copy branch July 3, 2026 19:54
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant