Skip to content

zbobr_fix-61-sync-fork#344

Open
milyin wants to merge 7 commits into
mainfrom
zbobr_fix-61-sync-fork
Open

zbobr_fix-61-sync-fork#344
milyin wants to merge 7 commits into
mainfrom
zbobr_fix-61-sync-fork

Conversation

@milyin
Copy link
Copy Markdown
Owner

@milyin milyin commented Apr 8, 2026

@milyin milyin mentioned this pull request Apr 8, 2026
7 tasks
milyin and others added 6 commits April 8, 2026 23:43
Adds `auto_sync_fork: bool` (default: true) to ZbobrRepoBackendGithubConfig
and a `maybe_sync_fork` method that calls GitHub's merge-upstream API to
bring a fork's branch in sync with upstream before each `git fetch origin`.

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
When auto_sync_fork is enabled and merge-upstream fails, the error is
now returned to the caller instead of being downgraded to a warning.
This ensures the backend does not silently continue with stale fork refs.

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
…gration test env

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
Tests cover:
1. auto_sync_fork=false skips all HTTP calls
2. Non-fork repos skip merge-upstream API calls
3. Fork repos call merge-upstream and return Ok
4. Merge-upstream HTTP errors are properly propagated

All tests use wiremock to mock the GitHub API and verify correct behavior.

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
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.

sync fork

1 participant