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 marketing/compare copy in TypeScript data files only; no runtime logic, auth, or API behavior changes.

Overview
Editorial pass on compare-page competitor data (claude-cowork, crewai, dust, and the rest of the profile set per the PR): long oneLiner, standout feature, limitation, and facts.* strings are rewritten into shorter, direct sentences.

The diff drops meta/hedging phrasing (e.g. “no documented”, “as of GA”, “no public evidence”, third-party-review caveats framed as distance from the vendor) in favor of plain statements, while keeping the same factual claims and structure (value / detail / shortValue, Yes:/No: labels).

No research metadata changes in the touched fields: sources, confidence, dates, and numeric claims are left as-is; this is presentation/copy only for the comparison UI.

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 pure copy/editorial pass across 21 comparison-page data files, removing self-referential hedging phrases ("marketing claim", "as of this check", "it should be noted that") and splitting run-on sentences into shorter, direct ones. No numbers, dates, confidence levels, sources, or boolean Yes:/No: values were changed.

  • sim.ts: Hedging stripped from ~15 fields; one inconsistency introduced where integrationCount.detail now describes the landing page's "1,000+" as a "broader figure counting individual API actions" even though the same file already reports ~3,900 tool actions — making the smaller figure appear "broader" contradicts the data.
  • 20 competitor files: Uniformly tightened — passive constructions made direct, comma/semicolon run-ons split at sentence boundaries, no factual content altered.

Confidence Score: 5/5

Safe to merge — all changes are editorial copy rewrites with no logic, schema, or data alterations.

The diff touches only string literals inside static data files. No executable logic, type definitions, API contracts, or test fixtures were modified. The one inconsistency flagged in integrationCount.detail is a copy-level framing issue, not a broken data field.

apps/sim/lib/compare/data/sim.ts — the integrationCount.detail wording warrants a second look before the page goes live.

Important Files Changed

Filename Overview
apps/sim/lib/compare/data/sim.ts Copy edits remove hedging throughout; one logical inconsistency introduced in integrationCount.detail (1,000+ described as "broader" than ~3,900), and two previously flagged accuracy regressions (community.value and additionalCompliance.value) remain unresolved.
apps/sim/lib/compare/data/competitors/openclaw.ts Editorial copy pass: hedging phrases removed, run-on sentences split. No factual values or confidence/source fields changed.
apps/sim/lib/compare/data/competitors/claude-cowork.ts Minor phrasing tightened; no factual changes.
apps/sim/lib/compare/data/competitors/make.ts Long detail strings trimmed and hedging removed. No figures or sources altered.
apps/sim/lib/compare/data/competitors/microsoft-copilot.ts Prose tightened across standout features and comparison fields; no numbers, URLs, or confidence values changed.
apps/sim/lib/compare/data/competitors/zapier.ts Hedging phrases removed and sentences shortened; no factual data altered.
apps/sim/lib/compare/data/competitors/pipedream.ts Copy edits only — passive constructions made direct, no factual values changed.
apps/sim/lib/compare/data/competitors/crewai.ts Editorial tightening throughout; no numbers or confidence levels changed.
apps/sim/lib/compare/data/competitors/n8n.ts Hedging and self-referential phrases removed; data unchanged.

Flowchart

%%{init: {'theme': 'neutral'}}%%
flowchart TD
    A[sim.ts\nSim profile] --> C[Compare Page]
    B1[zapier.ts] --> C
    B2[n8n.ts] --> C
    B3[make.ts] --> C
    B4[openclaw.ts] --> C
    B5["...17 more competitor files"] --> C
    C --> D[Rendered comparison table\nvalue / shortValue / detail fields]
    style A fill:#f9f,stroke:#333
    style C 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 profile] --> C[Compare Page]
    B1[zapier.ts] --> C
    B2[n8n.ts] --> C
    B3[make.ts] --> C
    B4[openclaw.ts] --> C
    B5["...17 more competitor files"] --> C
    C --> D[Rendered comparison table\nvalue / shortValue / detail fields]
    style A fill:#f9f,stroke:#333
    style C fill:#bbf,stroke:#333
Loading

Reviews (6): 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
@waleedlatif1

Copy link
Copy Markdown
Collaborator Author

@greptile

@waleedlatif1

Copy link
Copy Markdown
Collaborator Author

@cursor review

Comment thread apps/sim/lib/compare/data/competitors/dust.ts
@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!

1 issue from previous review remains unresolved.

Fix All in Cursor

Comment @cursor review or bugbot run to trigger another review on this PR

Reviewed by Cursor Bugbot for commit e10173c. Configure here.

@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!

1 issue from previous review remains unresolved.

Fix All in Cursor

Comment @cursor review or bugbot run to trigger another review on this PR

Reviewed by Cursor Bugbot for commit e10173c. Configure here.

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

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