Skip to content

Remove sync-branches workflow#53

Merged
mrjf merged 3 commits intomainfrom
copilot/remove-sync-branches-workflow
Apr 23, 2026
Merged

Remove sync-branches workflow#53
mrjf merged 3 commits intomainfrom
copilot/remove-sync-branches-workflow

Conversation

Copy link
Copy Markdown
Contributor

Copilot AI commented Apr 23, 2026

The sync-branches workflow is redundant since #36 made each iteration's Step 3 fast-forward or merge origin/main into the program's autoloop/* branch. Leaving it in is actively harmful: it fails to compile under strict mode (#33), silently no-ops without a PAT on most consumer repos, and reports green while doing nothing.

Changes

  • Delete the workflow: workflows/sync-branches.md, .github/workflows/sync-branches.md, .github/workflows/sync-branches.lock.yml.
  • install.md: drop gh aw compile sync-branches from Step 4 and the troubleshooting reference.
  • AGENTS.md: remove from the repo tree and deploy steps; rewrite the "companion workflow" paragraph to point at per-iteration Step 3 (Branch Setup).
  • README.md: remove from the repo-structure tree.
  • workflows/autoloop.md (and its .github/workflows/ mirror): remove the "A sync workflow automatically merges..." bullet from the "How It Works" list.
  • tests/test_scheduling.py: drop TestSyncBranchesCredentialOrdering (3 tests covering a workflow that no longer exists).

Migration for existing installs

Consumers who previously installed sync-branches should delete .github/workflows/sync-branches.md and .github/workflows/sync-branches.lock.yml from their repo on upgrade. No state to clean up. Worth calling out in release notes once #46 lands.

Warning

Firewall rules blocked me from connecting to one or more addresses (expand for details)

I tried to connect to the following addresses, but was blocked by firewall rules:

  • https://api.github.com/repos/bogus.invalid/bogus/issues
    • Triggering command: /usr/bin/python /usr/bin/python /home/REDACTED/work/autoloop/autoloop/workflows/scripts/autoloop_scheduler.py -m owner --uid-owner 0 -j ACCEPT (http block)
    • Triggering command: /usr/bin/python /usr/bin/python /home/REDACTED/work/autoloop/autoloop/workflows/scripts/autoloop_scheduler.py -m conntrack --ctstate INVALID,NEW -j DROP (http block)
    • Triggering command: /usr/bin/python /usr/bin/python /home/REDACTED/work/autoloop/autoloop/workflows/scripts/autoloop_scheduler.py (http block)
  • https://api.github.com/repos/bogus.invalid/bogus/pulls
    • Triggering command: /usr/bin/python /usr/bin/python /home/REDACTED/work/autoloop/autoloop/workflows/scripts/autoloop_scheduler.py -m owner --uid-owner 0 -j ACCEPT (http block)
    • Triggering command: /usr/bin/python /usr/bin/python /home/REDACTED/work/autoloop/autoloop/workflows/scripts/autoloop_scheduler.py -m conntrack --ctstate INVALID,NEW -j DROP (http block)
    • Triggering command: /usr/bin/python /usr/bin/python /home/REDACTED/work/autoloop/autoloop/workflows/scripts/autoloop_scheduler.py (http block)

If you need me to access, download, or install something from one of these locations, you can either:

Copilot AI changed the title [WIP] Remove sync-branches workflow due to redundancy Remove sync-branches workflow Apr 23, 2026
Copilot finished work on behalf of mrjf April 23, 2026 15:03
Copilot AI requested a review from mrjf April 23, 2026 15:03
@mrjf mrjf marked this pull request as ready for review April 23, 2026 15:16
@mrjf mrjf merged commit 7063470 into main Apr 23, 2026
15 checks passed
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.

Remove sync-branches workflow — made redundant by per-iteration Step 3 ahead/behind logic

2 participants