Skip to content

Commit b2ab5be

Browse files
SimplyLizclaude
andcommitted
fix(query): extend auto-fetch fallback to summarizePr and summarizeDiff
Apply EnsureRef to the two other user-facing base-ref callers so they work in shallow CI clones just like review. summarizeDiff uses best-effort resolution since its inputs can be raw SHAs, which should not go through branch-name fetch. Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
1 parent 6021b0d commit b2ab5be

2 files changed

Lines changed: 12 additions & 1 deletion

File tree

internal/query/navigation.go

Lines changed: 6 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -2026,6 +2026,12 @@ func (e *Engine) SummarizeDiff(ctx context.Context, opts SummarizeDiffOptions) (
20262026
selector = DiffSelector{Type: "commitRange", Value: opts.CommitRange.Base + ".." + opts.CommitRange.Head}
20272027
base = opts.CommitRange.Base
20282028
head = opts.CommitRange.Head
2029+
if resolved, rerr := e.gitAdapter.EnsureRef(base); rerr == nil {
2030+
base = resolved
2031+
}
2032+
if resolved, rerr := e.gitAdapter.EnsureRef(head); rerr == nil {
2033+
head = resolved
2034+
}
20292035
diffStats, err = e.gitAdapter.GetCommitRangeDiff(base, head)
20302036
if err != nil {
20312037
return nil, fmt.Errorf("failed to get commit range diff: %w", err)

internal/query/pr.go

Lines changed: 6 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -102,7 +102,12 @@ func (e *Engine) SummarizePR(ctx context.Context, opts SummarizePROptions) (*Sum
102102
if headRef == "" {
103103
headRef = "HEAD"
104104
}
105-
diffStats, err := e.gitAdapter.GetCommitRangeDiff(opts.BaseBranch, headRef)
105+
baseRef, err := e.gitAdapter.EnsureRef(opts.BaseBranch)
106+
if err != nil {
107+
return nil, fmt.Errorf("failed to resolve base ref %q: %w", opts.BaseBranch, err)
108+
}
109+
opts.BaseBranch = baseRef
110+
diffStats, err := e.gitAdapter.GetCommitRangeDiff(baseRef, headRef)
106111
if err != nil {
107112
return nil, fmt.Errorf("failed to get diff: %w", err)
108113
}

0 commit comments

Comments
 (0)