Skip to content

fix(grove): respect configured base branch during source repo sync#57

Merged
bang9 merged 1 commit intobang9:mainfrom
DongGukMon:fix-sync-source-repo
Apr 2, 2026
Merged

fix(grove): respect configured base branch during source repo sync#57
bang9 merged 1 commit intobang9:mainfrom
DongGukMon:fix-sync-source-repo

Conversation

@DongGukMon
Copy link
Copy Markdown
Contributor

Summary

  • refresh_source_repo()가 항상 remote_default_branch()만 호출하여 사용자가 설정한 base_branch를 무시하던 버그 수정
  • base_branch: Option<&str> 파라미터를 추가하고, 설정된 경우 해당 브랜치를 사용 (리모트에 없으면 auto-detect fallback)
  • add_worktree(line ~1036)에서 이미 사용 중인 동일한 show-ref/verify 패턴 적용

Test plan

  • 새 테스트 refresh_project_uses_configured_base_branch_instead_of_remote_default 추가 (TDD red→green)
  • 기존 refresh_project 테스트 6개 모두 통과 확인
  • whip 코드리뷰 에이전트 LGTM

refresh_source_repo() always used remote_default_branch(), ignoring the
user's base_branch setting. Pass base_branch into the function and use
it when the branch exists on remote, falling back to auto-detect.
@bang9 bang9 merged commit ff460f8 into bang9:main Apr 2, 2026
1 check 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.

2 participants