Skip to content

fix(sync): restore v1.0 features lost by -X theirs auto-resolution#329

Merged
erishforG merged 1 commit into
release/1.0from
fix/revert-bad-sync-restore-v1
May 30, 2026
Merged

fix(sync): restore v1.0 features lost by -X theirs auto-resolution#329
erishforG merged 1 commit into
release/1.0from
fix/revert-bad-sync-restore-v1

Conversation

@erishforG
Copy link
Copy Markdown
Owner

무엇이 잘못됐나

오늘 (2026-05-30) parsec-v1-sync-daily cron 첫 실행에서 `-X theirs` 정책이 v1.0 작업을 덮어씀:

src/cli/mod.rs · src/cli/commands/mod.rs · src/config/mod.rs 3개 파일이 충돌했는데 `-X theirs` 가 develop 쪽 (Smartlog/Complete/CacheStrategy) 으로 통일하면서 release/1.0 의 Commit/AiProvider variant 가 사라짐.

복원 방법

  1. `git revert -m 1 9b0f29f` → `git reset --hard 2d023aa` (sync 직전 release/1.0 상태로)
  2. `git merge origin/develop --no-ff` 재실행
  3. 3개 파일 충돌을 수동으로 해결 — Commit + Smartlog + Complete enum variant 모두 보존
  4. cargo build / clippy / fmt / test 67 전부 clean

영향 PR

sync 정책 변경 필요 (별도 작업)

이 PR 머지 후 parsec-v1-sync-daily cron 정책 재검토:

  • `-X theirs` 는 release/1.0 의 v1.0-specific 코드를 silently 덮음
  • 안전한 대안: 충돌 시 자동 중단 + Eric 호출 (원래 옵션 A)
  • 또는 `-X union` (양쪽 모두 보존) — enum variant 같은 경우 잘 동작하지만 일반적으론 무리
  • 가장 안전: 자동 머지 시도 → 충돌 시 abort + 알림

Eric 결정 부탁:

  1. sync cron 충돌 정책을 옵션 A (자동 중단) 로 복귀?
  2. 아니면 이런 케이스 (양쪽 add) 만 union 시도?

Test plan

  • cargo build clean
  • cargo clippy --all-targets -- -D warnings clean
  • cargo fmt --check
  • cargo test 67 pass (5 + 62)
  • grep 확인: Commit subcommand, AiProvider variant, Smartlog/Complete enum 모두 src/cli/mod.rs 에 존재

@erishforG

🤖 Generated with Claude Code

…tures)

# Conflicts:
#	src/cli/commands/mod.rs
#	src/cli/mod.rs
#	src/config/mod.rs
@erishforG erishforG merged commit 022332a into release/1.0 May 30, 2026
12 checks passed
@erishforG erishforG deleted the fix/revert-bad-sync-restore-v1 branch May 30, 2026 05:33
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.

1 participant