Skip to content

fix(runway): reconcile stale merge retries - #675

Draft
sbalabanov wants to merge 1 commit into
mainfrom
sergeyb/fix-runway-stale-retry
Draft

fix(runway): reconcile stale merge retries#675
sbalabanov wants to merge 1 commit into
mainfrom
sergeyb/fix-runway-stale-retry

Conversation

@sbalabanov

Copy link
Copy Markdown
Contributor

Summary

Intent:

  • Let at-least-once merge deliveries converge when a prior attempt already updated Git but failed to publish its result.
  • Preserve staleness protection for superseded requests that would still modify the target.

Changes:

  • Apply transforming merge strategies locally before staleness validation and return success when the target already satisfies every step.
  • Check PROMOTE containment before freshness and account for head refs moved by a prior contention attempt.
  • Add real-Git fail-once end-to-end coverage, strategy-level regression tests, and adjacent reproduction documentation.

Generated by the 🪄 pr-create skill in devexp-agent-marketplace

Test Plan

AI Verification

Validated at 935e871 on Sep 4 22:05 UTC · 5 files analyzed · 1s

Validator Status Issues
android-lint not_applicable 0
java-coverage not_applicable 0
ios-lint not_applicable 0
web-repocheck not_applicable 0
java-lint not_applicable 0
go-coverage not_applicable 0
go-gazelle not_applicable 0
arc-lint not_applicable 0
visual-ios not_applicable 0
visual-android not_applicable 0
go-proto-lint not_applicable 0
android-coverage not_applicable 0
arc-unit not_applicable 0
go-thrift-lint not_applicable 0
ios-test not_applicable 0
uber-one not_applicable 0
web-lint not_applicable 0
web-coverage not_applicable 0
diff-template not_applicable 0
fix-disclosure not_applicable 0
go-lint not_applicable 0
web-typecheck not_applicable 0
web-unit not_applicable 0
merge-conflict not_applicable 0
visual-web not_applicable 0
ureview completed 0
custom not_applicable 0

0 issues detected

Skipped validators: claude · EngWiki

Prior runs

Run at c9ee7a8 on Sep 4 22:04 UTC · 5 files · 2s · 0 issues detected

Issues

Summary:
Intent:
- Let at-least-once merge deliveries converge when a prior attempt already updated Git but failed to publish its result.
- Preserve staleness protection for superseded requests that would still modify the target.

Changes:
- Apply transforming merge strategies locally before staleness validation and return success when the target already satisfies every step.
- Check PROMOTE containment before freshness and account for head refs moved by a prior contention attempt.
- Add real-Git fail-once end-to-end coverage, strategy-level regression tests, and adjacent reproduction documentation.

---

<sub>Generated by the 🪄 pr-create skill in devexp-agent-marketplace</sub>
@CLAassistant

Copy link
Copy Markdown

CLA assistant check
Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.
You have signed the CLA already but the status is still pending? Let us recheck it.

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.

3 participants