Skip to content

Commit 398f068

Browse files
authored
Merge pull request #25 from githubnext/copilot/fix-stale-branch-patch-failures
[WIP] Fix stale branch causing create_pull_request patch failures
2 parents eba845f + 3faf701 commit 398f068

2 files changed

Lines changed: 12 additions & 6 deletions

File tree

.github/workflows/autoloop.md

Lines changed: 6 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -644,11 +644,11 @@ Examples:
644644
### How It Works
645645

646646
1. On the **first accepted iteration**, the branch is created from the default branch.
647-
2. On **subsequent iterations**, the agent checks out the existing branch and ensures it is up to date with the default branch (by merging the default branch into it).
647+
2. On **subsequent iterations**, the agent checks out the existing branch and ensures it is up to date with the default branch. If the branch's changes have already been merged into the default branch (i.e., `git diff origin/main..autoloop/{program-name}` is empty), the branch is **reset to `origin/main`** to avoid stale commits. Otherwise, the default branch is merged into it.
648648
3. **Accepted iterations** are committed and pushed to the branch. Each commit message references the GitHub Actions run URL.
649649
4. **Rejected or errored iterations** do not commit — changes are discarded.
650650
5. A **single draft PR** is created for the branch on the first accepted iteration. Future accepted iterations push additional commits to the same PR.
651-
6. The branch may be **merged into the default branch** at any time (by a maintainer or CI). After merging, the branch continues to be used for future iterations — it is never deleted while the program is active.
651+
6. The branch may be **merged into the default branch** at any time (by a maintainer or CI). After merging, the branch continues to be used for future iterations — it is never deleted while the program is active. On the next iteration, the branch is automatically reset to the default branch (see step 2) so that already-merged commits do not cause patch conflicts.
652652
7. A **sync workflow** automatically merges the default branch into all active `autoloop/*` branches whenever the default branch changes, keeping them up to date.
653653

654654
### Cross-Linking
@@ -691,7 +691,10 @@ Each run executes **one iteration for the single selected program**:
691691

692692
### Step 3: Implement
693693

694-
1. Check out the program's long-running branch `autoloop/{program-name}`. If the branch does not yet exist, create it from the default branch. If it does exist, ensure it is up to date with the default branch (merge the default branch into it).
694+
1. Check out the program's long-running branch `autoloop/{program-name}`. If the branch does not yet exist, create it from the default branch. If it does exist:
695+
- Fetch the default branch: `git fetch origin main`.
696+
- Check whether the branch's changes have already been merged into main. If `git diff origin/main..autoloop/{program-name}` produces no output (i.e., every change on the branch is already on main), the branch is stale — **reset it to `origin/main`**: `git reset --hard origin/main`.
697+
- Otherwise, merge the default branch into the long-running branch to pick up any upstream changes.
695698
2. Make the proposed changes to the target files only.
696699
3. **Respect the program constraints**: do not modify files outside the target list.
697700

workflows/autoloop.md

Lines changed: 6 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -759,11 +759,11 @@ Examples:
759759
### How It Works
760760

761761
1. On the **first accepted iteration**, the branch is created from the default branch.
762-
2. On **subsequent iterations**, the agent checks out the existing branch and ensures it is up to date with the default branch (by merging the default branch into it).
762+
2. On **subsequent iterations**, the agent checks out the existing branch and ensures it is up to date with the default branch. If the branch's changes have already been merged into the default branch (i.e., `git diff origin/main..autoloop/{program-name}` is empty), the branch is **reset to `origin/main`** to avoid stale commits. Otherwise, the default branch is merged into it.
763763
3. **Accepted iterations** are committed and pushed to the branch. Each commit message references the GitHub Actions run URL.
764764
4. **Rejected or errored iterations** do not commit — changes are discarded.
765765
5. A **single draft PR** is created for the branch on the first accepted iteration. Future accepted iterations push additional commits to the same PR.
766-
6. The branch may be **merged into the default branch** at any time (by a maintainer or CI). After merging, the branch continues to be used for future iterations — it is never deleted while the program is active.
766+
6. The branch may be **merged into the default branch** at any time (by a maintainer or CI). After merging, the branch continues to be used for future iterations — it is never deleted while the program is active. On the next iteration, the branch is automatically reset to the default branch (see step 2) so that already-merged commits do not cause patch conflicts.
767767
7. A **sync workflow** automatically merges the default branch into all active `autoloop/*` branches whenever the default branch changes, keeping them up to date.
768768

769769
### Cross-Linking
@@ -806,7 +806,10 @@ Each run executes **one iteration for the single selected program**:
806806

807807
### Step 3: Implement
808808

809-
1. Check out the program's long-running branch `autoloop/{program-name}`. If the branch does not yet exist, create it from the default branch. If it does exist, ensure it is up to date with the default branch (merge the default branch into it).
809+
1. Check out the program's long-running branch `autoloop/{program-name}`. If the branch does not yet exist, create it from the default branch. If it does exist:
810+
- Fetch the default branch: `git fetch origin main`.
811+
- Check whether the branch's changes have already been merged into main. If `git diff origin/main..autoloop/{program-name}` produces no output (i.e., every change on the branch is already on main), the branch is stale — **reset it to `origin/main`**: `git reset --hard origin/main`.
812+
- Otherwise, merge the default branch into the long-running branch to pick up any upstream changes.
810813
2. Make the proposed changes to the target files only.
811814
3. **Respect the program constraints**: do not modify files outside the target list.
812815

0 commit comments

Comments
 (0)