improvement(diff): refine embedded diff presentation by waleedlatif1 · Pull Request #8862 · simstudioai/sim

waleedlatif1 · 2026-10-09T20:42:16Z

Summary

Give file diffs quieter change colors, precise word highlights, clearer gutters, and expandable context.
Align document comparisons, including reversed and interleaved changes and heading levels, and keep source headings fixed while each document scrolls. Use the platform's editor controls, typography, tokens, tooltips, and touch targets.

Type of Change

Enhancement

Testing

Ran lint, all workspace type checks, 58 audits, the block-registry and docs-manifest checks, and checksum-verified actionlint. Ran 95 existing diff/editor checks and the full repository test suite with four workers (36,890 app tests passed). Browser-verified read-only and editable source switching, independent keyboard scrolling, wrapping, shared context expansion, unequal added rows, reversed and interleaved changes, all six heading levels, heading/paragraph conversions, named sources, and responsive layouts in the real editor styles. Retained screenshots and before/after pairing and heading reports with the task.

Checklist

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

vercel · 2026-10-09T20:42:23Z

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

1 Skipped Deployment

Project	Deployment	Actions	Updated
docs	Skipped		Oct 9, 2026 10:29pm UTC

cubic-dev-ai

No issues found across 3 files

Confidence score: 5/5

Automated review surfaced no issues in the provided summaries.
No files require special attention.

_{Turn on auto-fix | Re-trigger cubic}

greptile-apps · 2026-10-09T20:48:11Z

Confidence Score: 5/5

[Medium impact] The PR appears safe to merge; no actionable issues remain.

Summary

Refines embedded diffs with quieter colors, word highlights, clearer gutters, shared context expansion, and editor controls.

Code diffs make edits easier to scan and control.
Document diffs line up the Before and After text.

_{Reviews (5) · Last reviewed commit: "fix(diff): preserve heading-level change..." · Reviewed by Greptile}

waleedlatif1 · 2026-10-09T21:06:03Z

@greptile

waleedlatif1 · 2026-10-09T21:06:04Z

@cubic-dev-ai review this PR

cubic-dev-ai · 2026-10-09T21:06:11Z

@cubic-dev-ai review this PR

@waleedlatif1 I have started the AI code review. It will take a few minutes to complete.

waleedlatif1 · 2026-10-09T21:08:01Z

@cubic-dev-ai review this PR

cubic-dev-ai · 2026-10-09T21:08:07Z

@cubic-dev-ai review this PR

@waleedlatif1 I have started the AI code review. It will take a few minutes to complete.

waleedlatif1 · 2026-10-09T21:28:52Z

@greptile

waleedlatif1 · 2026-10-09T21:28:53Z

@cubic-dev-ai review this PR

cubic-dev-ai · 2026-10-09T21:29:00Z

@cubic-dev-ai review this PR

@waleedlatif1 I have started the AI code review. It will take a few minutes to complete.

cubic-dev-ai

All reported issues were addressed across 3 files

_{Reply with feedback, questions, or to request a fix.

Turn on auto-fix | Re-trigger cubic}

waleedlatif1 · 2026-10-09T22:10:19Z

@greptile

waleedlatif1 · 2026-10-09T22:10:21Z

@cubic-dev-ai review this PR

cubic-dev-ai · 2026-10-09T22:10:28Z

@cubic-dev-ai review this PR

@waleedlatif1 I have started the AI code review. It will take a few minutes to complete.

cubic-dev-ai

All reported issues were addressed across 3 files

_{Reply with feedback, questions, or to request a fix.

Turn on auto-fix | Re-trigger cubic}

waleedlatif1 · 2026-10-09T22:29:22Z

@greptile

waleedlatif1 · 2026-10-09T22:29:23Z

@cubic-dev-ai review this PR

cubic-dev-ai · 2026-10-09T22:29:29Z

@cubic-dev-ai review this PR

@waleedlatif1 I have started the AI code review. It will take a few minutes to complete.

cubic-dev-ai

No issues found across 3 files

Confidence score: 5/5

Automated review surfaced no issues in the provided summaries.
No files require special attention.

_{Turn on auto-fix | Re-trigger cubic}

improvement(diff): refine embedded diff presentation

c126c24

cubic-dev-ai Bot reviewed Oct 9, 2026

View reviewed changes

greptile-apps Bot reviewed Oct 9, 2026

View reviewed changes

Comment thread apps/sim/components/diff/diff-view.tsx Outdated

fix(diff): keep comparison headings aligned while scrolling

6164ea0

vercel Bot temporarily deployed to Preview October 9, 2026 21:05 Inactive

greptile-apps Bot reviewed Oct 9, 2026

View reviewed changes

Comment thread apps/sim/components/diff/diff-view.tsx Outdated

fix(diff): keep the comparison divider outside scroll fades

4037b96

vercel Bot temporarily deployed to Preview October 9, 2026 21:28 Inactive

cubic-dev-ai Bot reviewed Oct 9, 2026

View reviewed changes

Comment thread apps/sim/components/diff/diff-view.tsx Outdated

fix(diff): pair complete change blocks in document comparisons

a76dfe0

vercel Bot temporarily deployed to Preview October 9, 2026 22:10 Inactive

cubic-dev-ai Bot reviewed Oct 9, 2026

View reviewed changes

Comment thread apps/sim/components/diff/diff-view.tsx

fix(diff): preserve heading-level changes in comparisons

5b85bab

vercel Bot temporarily deployed to Preview October 9, 2026 22:29 Inactive

cubic-dev-ai Bot reviewed Oct 9, 2026

View reviewed changes

waleedlatif1 merged commit 9d64913 into staging Oct 9, 2026
38 checks passed

waleedlatif1 deleted the codex/refine-document-diffs branch October 9, 2026 22:38

waleedlatif1 mentioned this pull request Oct 9, 2026

v0.9.17: tables ledger, oracle integrations, mship improvements #8853

Open

Conversation

waleedlatif1 commented Oct 9, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Summary

Type of Change

Testing

Checklist

Uh oh!

vercel Bot commented Oct 9, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Uh oh!

cubic-dev-ai Bot left a comment

Choose a reason for hiding this comment

Uh oh!

greptile-apps Bot commented Oct 9, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Confidence Score: 5/5

Summary

Uh oh!

Uh oh!

waleedlatif1 commented Oct 9, 2026

Uh oh!

waleedlatif1 commented Oct 9, 2026

Uh oh!

cubic-dev-ai Bot commented Oct 9, 2026

Uh oh!

waleedlatif1 commented Oct 9, 2026

Uh oh!

cubic-dev-ai Bot commented Oct 9, 2026

Uh oh!

Uh oh!

waleedlatif1 commented Oct 9, 2026

Uh oh!

waleedlatif1 commented Oct 9, 2026

Uh oh!

cubic-dev-ai Bot commented Oct 9, 2026

Uh oh!

cubic-dev-ai Bot left a comment • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Uh oh!

waleedlatif1 commented Oct 9, 2026

Uh oh!

waleedlatif1 commented Oct 9, 2026

Uh oh!

cubic-dev-ai Bot commented Oct 9, 2026

Uh oh!

cubic-dev-ai Bot left a comment • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Uh oh!

waleedlatif1 commented Oct 9, 2026

Uh oh!

waleedlatif1 commented Oct 9, 2026

Uh oh!

cubic-dev-ai Bot commented Oct 9, 2026

Uh oh!

cubic-dev-ai Bot left a comment

Choose a reason for hiding this comment

Uh oh!

Uh oh!

This branch was previously deployed

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

1 participant

waleedlatif1 commented Oct 9, 2026 •

edited

Loading

vercel Bot commented Oct 9, 2026 •

edited

Loading

greptile-apps Bot commented Oct 9, 2026 •

edited

Loading

cubic-dev-ai Bot left a comment •

edited

Loading

cubic-dev-ai Bot left a comment •

edited

Loading