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

Method GetDiff

internal/diff/git.go:174–230  ·  view source on GitHub ↗

GetDiff returns all changes as parsed model.Diff structs.

(ctx context.Context)

Source from the content-addressed store, hash-verified

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

Calls 7

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