Skip to content

Add native difftastic usage and remove git diff fallback#15

Merged
martinabeleda merged 2 commits intomainfrom
difftastic-native
Mar 14, 2026
Merged

Add native difftastic usage and remove git diff fallback#15
martinabeleda merged 2 commits intomainfrom
difftastic-native

Conversation

@martinabeleda
Copy link
Owner

@martinabeleda martinabeleda commented Mar 13, 2026

This PR replaces ripdiff's external difft subprocess integration with an in-process Rust integration built on a forked difftastic dependency.

Instead of invoking git diff with GIT_EXTERNAL_DIFF=difft, ripdiff now prepares the relevant git/worktree snapshots in Rust and renders the diff through difftastic directly in-process.

My fork of difftastic was modified to a library in martinabeleda/difftastic#1

Related upstream discussion: Wilfred/difftastic#216

Changes

  • add difftastic as a Git dependency in Cargo.toml
  • replace the subprocess-based diff path in src/diff.rs
  • remove the external difft runtime requirement
  • remove the old plain git diff fallback

Notes

This uses a forked difftastic repo because upstream does not currently expose a supported library crate.

@chatgpt-codex-connector
Copy link

You have reached your Codex usage limits for code reviews. You can see your limits in the Codex usage dashboard.
To continue using code reviews, you can upgrade your account or add credits to your account and enable them for code reviews in your settings.

@martinabeleda martinabeleda merged commit 23c08db into main Mar 14, 2026
4 checks passed
@martinabeleda martinabeleda deleted the difftastic-native branch March 14, 2026 03:31
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