MCPcopy Create free account
hub / github.com/alibaba/open-code-review / GetDiff

Method GetDiff

internal/diff/git.go:173–229  ·  view source on GitHub ↗

GetDiff returns all changes as parsed model.Diff structs.

(ctx context.Context)

Source from the content-addressed store, hash-verified

171
172// GetDiff returns all changes as parsed model.Diff structs.
173func (p *Provider) GetDiff(ctx context.Context) ([]model.Diff, error) {
174 var combined strings.Builder
175
176 switch p.mode {
177 case ModeRange:
178 base := p.MergeBase(ctx)
179 if base == "" {
180 return nil, fmt.Errorf("cannot find merge-base between %s and %s", p.from, p.to)
181 }
182 out, err := p.runGit(ctx, "-c", "core.quotepath=false", "diff", "--no-ext-diff", "--no-textconv", "--find-renames", "--src-prefix=a/", "--dst-prefix=b/", "--no-color", "-U"+fmt.Sprint(DiffContextLines), "--end-of-options", base, p.to, "--")
183 if err != nil {
184 return nil, fmt.Errorf("git diff failed: %w", err)
185 }
186 combined.WriteString(out)
187
188 case ModeCommit:
189 // --diff-merges=first-parent: for merge commits, plain `git show`
190 // emits a combined diff ("diff --cc"), which ParseDiffText cannot
191 // parse — the commit would silently yield zero reviewable diffs.
192 // Diffs against the first parent instead, in regular unified format.
193 out, err := p.runGit(ctx, "-c", "core.quotepath=false", "show", "--no-ext-diff", "--no-textconv", "--find-renames", "--src-prefix=a/", "--dst-prefix=b/", "--no-color", "--diff-merges=first-parent", "-U"+fmt.Sprint(DiffContextLines), "--end-of-options", p.commit)
194 if err != nil {
195 return nil, fmt.Errorf("git show failed: %w", err)
196 }
197 combined.WriteString(out)
198
199 case ModeWorkspace:
200 tracked, err := p.workspaceTrackedDiff(ctx)
201 if err != nil {
202 return nil, fmt.Errorf("workspace tracked diff failed: %w", err)
203 }
204 combined.WriteString(tracked)
205
206 untracked, err := p.untrackedFileDiffs(ctx)
207 if err != nil {
208 return nil, fmt.Errorf("untracked file diff failed: %w", err)
209 }
210 for _, ud := range untracked {
211 combined.WriteString(ud)
212 combined.WriteString("\n\n")
213 }
214 }
215
216 var ref string
217 switch p.mode {
218 case ModeRange:
219 ref = p.to
220 case ModeCommit:
221 ref = p.commit
222 }
223
224 diffs, err := ParseDiffText(ctx, combined.String(), p.repoDir, ref, p.runner)
225 if err != nil {
226 return nil, err
227 }
228 return p.filterDiffs(diffs), nil
229}
230

Calls 7

MergeBaseMethod · 0.95
runGitMethod · 0.95
workspaceTrackedDiffMethod · 0.95
untrackedFileDiffsMethod · 0.95
filterDiffsMethod · 0.95
ParseDiffTextFunction · 0.85
StringMethod · 0.45