fix(git): disable external diff for patch output

[stromseng]

What Changed

Appended --no-ext-diff to git diff --patch calls

Why

Having an external diff tool set in git breaks the diff display.
Closes #927

UI Changes

Checklist

• This PR is small and focused
• I explained what changed and why
• I included before/after screenshots for any UI changes
• I included a video for animation/interaction changes

---

Note

Low Risk
Low risk: only adjusts git diff flags for patch-generating commands, with no changes to repo state or auth/data handling. Main impact is patch output formatting/compatibility with user-configured external diff drivers.

Overview
Ensures patch output is generated with Git’s built-in diff by adding --no-ext-diff to git diff --patch invocations.

This affects checkpoint-to-checkpoint diffs (CheckpointStore.diffCheckpoints), staged patch generation in prepareCommitContext, and range diff patch generation in readRangeContext, preventing external diff tools from breaking server-side diff display.

^{Reviewed by Cursor Bugbot for commit 2e015e3. Bugbot is set up for automated code reviews on this repo. Configure here.}

Note

Disable external diff drivers for patch output in git diff calls

Adds --no-ext-diff to all git diff invocations that generate patch output, ensuring Git's internal diff is used regardless of any external diff drivers configured in the repo. Affected calls are the staged patch in prepareCommitContext, the range diff patch in readRangeContext, and checkpoint-to-checkpoint diffs in diffCheckpoints.

^{Macroscope summarized 2e015e3.}

[coderabbitai]

Important

Review skipped

Auto reviews are disabled on this repository. Please check the settings in the CodeRabbit UI or the .coderabbit.yaml file in this repository. To trigger a single review, invoke the @coderabbitai review command.

⚙️ Run configuration

Configuration used: Repository UI

Review profile: CHILL

Plan: Pro

Run ID: 61909412-17b0-4ffc-8f9a-f2b98b20efbe

You can disable this status message by setting the reviews.review_status to false in the CodeRabbit configuration file.

Use the checkbox below for a quick retry:

• 🔍 Trigger review

✨ Finishing Touches

🧪 Generate unit tests (beta)

• Create PR with unit tests

Tip

💬 Introducing Slack Agent: The best way for teams to turn conversations into code.

Slack Agent is built on CodeRabbit's deep understanding of your code, so your team can collaborate across the entire SDLC without losing context.

• Generate code and open pull requests
• Plan features and break down work
• Investigate incidents and troubleshoot customer tickets together
• Automate recurring tasks and respond to alerts with triggers
• Summarize progress and report instantly

Built for teams:

• Shared memory across your entire org—no repeating context
• Per-thread sandboxes to safely plan and execute work
• Governance built-in—scoped access, auditability, and budget controls

One agent for your entire SDLC. Right inside Slack.

👉 Get started

---

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

• X
• Mastodon
• Reddit
• LinkedIn

_{Comment @coderabbitai help to get the list of available commands and usage tips.}

[macroscopeapp]

Approvability

Verdict: Approved

Small defensive fix adding --no-ext-diff to git diff commands, ensuring consistent patch output regardless of user's git configuration. Clear intent, limited scope, and obvious resolution.

^{You can customize Macroscope's approvability policy. Learn more.}

[Copilot]

Pull request overview

Updates server-side Git diff invocations to ensure patch output remains in standard unified diff format even when users have configured an external diff tool (e.g., via diff.external), addressing issue #927.

Changes:

• Add --no-ext-diff to git diff --patch calls used for commit-context patch generation.
• Add --no-ext-diff to range diff patch generation for context reads.
• Add --no-ext-diff to checkpoint diff patch generation.

Reviewed changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated 3 comments.

File	Description
apps/server/src/vcs/GitVcsDriverCore.ts	Ensures unified patch output for staged and range diffs by disabling external diff tools.
apps/server/src/checkpointing/Layers/CheckpointStore.ts	Ensures checkpoint diff patch output stays parseable by disabling external diff tools.

---

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.