fix(consult): correct diff scope on non-default integration branches#821

Merged

waleedkadous merged 18 commits into

mainfrom

fix/777-784-consult-default-branch-three-dot-and-defect-a

May 22, 2026

Commits on May 22, 2026

feat(lib): add resolveDefaultBranch helper
amrmelsayed
committed
[Bugfix #777][Bugfix #784] fix(consult): resolve default branch + use three-dot diff semantics
amrmelsayed
committed
[Bugfix #777] feat(consult): --branch flag and PR-default ref resolver for spec/plan artifacts
amrmelsayed
committed
[Bugfix #777] docs(protocols): use integration branch instead of literal "main" in non-PIR protocols
amrmelsayed
committed
test(consult): cover non-main default branch, three-dot scope, and ref-based artifact resolution
amrmelsayed
committed
[Bugfix #777] test(baked-decisions): refresh AIR pr-review baseline after main → integration-branch rename
amrmelsayed
committed
[Bugfix #777] fix(bugfix-protocol): default-branch shell substitution falls back to main when origin/HEAD unset
amrmelsayed
committed
[Bugfix #777] fix(consult): scope architect impl diff against PR.baseRefName, not repo default branch
amrmelsayed
committed
test(consult): cover PR.baseRefName != repo default as merge-base anchor
amrmelsayed
committed
[Bugfix #777] fix(consult): surface fetch failures instead of silently using stale refs
amrmelsayed
committed
[Bugfix #777] refactor(consult): collapse redundant merge-base, use execFileSync for ref interpolation
amrmelsayed
committed
[Bugfix #777] fix(protocols): default-branch resolution falls through to "main" correctly
amrmelsayed
committed
[Bugfix #777] fix(consult): tolerate stale pr-search.sh overrides without baseRefName
amrmelsayed
committed
[Bugfix #777] fix(consult): verify both PR refs up front; convert last ref-interpolated execSync; refresh GitRefResolver docstring
amrmelsayed
committed
[Bugfix #777] fix(protocols): complete the || echo main → ${VAR:-main} fix in PIR (sites missed in f68f24e7)
amrmelsayed
committed
[Bugfix #777][Bugfix #784] fix(protocols): anchor diff invocations at merge-base, not the moving base branch tip
amrmelsayed
committed
test(protocols): add regression-guard audit for #777 / #784 prompt patterns
amrmelsayed
committed
docs(consult-types/pr-review): add reviewer-scope rule against flagging quoted git diff prose
amrmelsayed
committed

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

fix(consult): correct diff scope on non-default integration branches#821

fix(consult): correct diff scope on non-default integration branches#821
waleedkadous merged 18 commits into
mainfrom
fix/777-784-consult-default-branch-three-dot-and-defect-a

Commits on May 22, 2026

feat(lib): add resolveDefaultBranch helper

[Bugfix #777][Bugfix #784] fix(consult): resolve default branch + use three-dot diff semantics

[Bugfix #777] feat(consult): --branch flag and PR-default ref resolver for spec/plan artifacts

[Bugfix #777] docs(protocols): use integration branch instead of literal "main" in non-PIR protocols

test(consult): cover non-main default branch, three-dot scope, and ref-based artifact resolution

[Bugfix #777] test(baked-decisions): refresh AIR pr-review baseline after main → integration-branch rename

[Bugfix #777] fix(bugfix-protocol): default-branch shell substitution falls back to main when origin/HEAD unset

[Bugfix #777] fix(consult): scope architect impl diff against PR.baseRefName, not repo default branch

test(consult): cover PR.baseRefName != repo default as merge-base anchor

[Bugfix #777] fix(consult): surface fetch failures instead of silently using stale refs

[Bugfix #777] refactor(consult): collapse redundant merge-base, use execFileSync for ref interpolation

[Bugfix #777] fix(protocols): default-branch resolution falls through to "main" correctly

[Bugfix #777] fix(consult): tolerate stale pr-search.sh overrides without baseRefName

[Bugfix #777] fix(consult): verify both PR refs up front; convert last ref-interpolated execSync; refresh GitRefResolver docstring

[Bugfix #777] fix(protocols): complete the || echo main → ${VAR:-main} fix in PIR (sites missed in f68f24e7)

[Bugfix #777][Bugfix #784] fix(protocols): anchor diff invocations at merge-base, not the moving base branch tip

test(protocols): add regression-guard audit for #777 / #784 prompt patterns

docs(consult-types/pr-review): add reviewer-scope rule against flagging quoted git diff prose