7차 A-1: 마지막 충돌 저장 뒤 파일 비교 탭이 병합 커밋을 할 수 있다고 말한다
병합 중 충돌이 0 이면 두 칸 요약을 "충돌 정리 끝 · 병합 커밋을 할 수 있습니다 · 내 쪽 대비 받은 변경 N줄" 로, BASE 제목에 "병합 전 내 쪽". 예전에는 "남은 차이 6줄" 이라 덜 푼 것처럼 보였다. E-1 기록의 하네스 문장을 바로잡음(MergeSaveTest 기대값은 [Save] 로 고침). Co-Authored-By: Claude Opus 5 <noreply@anthropic.com>
This commit is contained in:
parent
26d9d042bf
commit
23fb002795
@ -2,6 +2,49 @@
|
||||
|
||||
지시서: [../작업지시/7차.md](../작업지시/7차.md). 6차 D-1 에서 새로 걸린 세 가지와 6차에서 뺀 pull · rebase · cherry-pick 충돌.
|
||||
|
||||
## A-1. 마지막 충돌을 저장하면 "남은 차이" 가 뜬다 (2026-09-14, 완료)
|
||||
|
||||
### 고치기 전에 다시 났다
|
||||
|
||||
새로 만든 저장소를 git 으로 병합하고 `Weld/Unit1.cs` 하나만 남긴 채 나머지 25개를 상대 쪽으로 풀었다(준비만 git).
|
||||
MergeHub 파일 비교 탭에서 그 파일의 충돌 6개를 Take Theirs → [Save].
|
||||
|
||||
| 때 | 칸 제목 | 요약 | 충돌 탭 |
|
||||
|---|---|---|---|
|
||||
| 충돌 파일 열림 | `(merge base) Weld/Unit1.cs` · 네 칸 | 남은 충돌 6개 | - |
|
||||
| **마지막 Save 뒤** (git 충돌 0, 병합 중) | **`90d2a8c · Weld/Unit1.cs`** · 두 칸 | **남은 차이 6줄** | 충돌 정리 끝 - 병합 커밋을 할 수 있습니다 |
|
||||
|
||||
두 화면이 다른 말을 한다. 파일 비교 탭만 보면 덜 푼 것으로 읽힌다.
|
||||
|
||||
(처음 두 번은 재현 준비가 틀렸다 - 충돌 탭에서 22개를 골라 한 번에 Accept 하려 했는데 자동 조작의 목록 찾기가 엉뚱한 목록을 잡아 한 개만 풀렸고,
|
||||
그다음은 준비 스크립트가 모든 충돌을 풀어 버렸다. 세 번째에 저장소를 새로 만들어 git 으로 준비했다.)
|
||||
|
||||
### 원인
|
||||
|
||||
- [Save] 뒤 `ReloadAfterResolveAsync` → `LoadCommitAsync` 가 충돌이 0 이면 보통 화면(HEAD ↔ 워킹트리)으로 연다. 방금 저장한 파일이 목록에 있어 그대로 골라진다.
|
||||
- 보통 화면의 요약은 `남은 차이 N줄`, BASE 제목은 `짧은해시 · 경로` 뿐이라 **병합 중이라는 사정도, 그 차이가 방금 받은 변경이라는 것도** 말하지 않았다.
|
||||
|
||||
### 고친 것
|
||||
|
||||
| 파일 | 무엇 |
|
||||
|---|---|
|
||||
| `Services/MergeSavePlan.cs` | `ResolvedMergeSummary(병합 중, 남은 충돌, 차이 줄)` — 병합 중 · 충돌 0 이면 "충돌 정리 끝 · 병합 커밋을 할 수 있습니다", 차이가 있으면 "· 내 쪽 대비 받은 변경 N줄". 아니면 null. `MergeBaseHeader` — `짧은해시 · 경로 · 병합 전 내 쪽` |
|
||||
| `ViewModels/HomeViewModel.cs` | `UpdateSummaries`: Git 모드 두 칸에서 위 요약을 쓴다. `LoadFileAsync`: 병합 중이고 BASE 가 HEAD 면 제목에 "병합 전 내 쪽" |
|
||||
| 하네스 `MergeSaveTest` | 요약 · 제목 검사 7건(병합 아님 · 충돌 남음 → 보통 요약, 차이 0 · 6 · 음수, "남은" 이 안 들어감) |
|
||||
|
||||
화면 틀(두 칸)과 파일 목록은 바꾸지 않았다. 병합 커밋 뒤에는 병합 중이 아니므로 지금처럼 보통 화면이다(6차 B-8).
|
||||
|
||||
### 확인한 것 (같은 준비로 다시)
|
||||
|
||||
| 때 | 칸 제목 | 요약 · 상태바 |
|
||||
|---|---|---|
|
||||
| 충돌 파일 열림 | `(merge base) Weld/Unit1.cs` | 남은 충돌 6개 |
|
||||
| **마지막 Save 뒤** (git 충돌 0) | **`9ab2256 · Weld/Unit1.cs · 병합 전 내 쪽`** | **충돌 정리 끝 · 병합 커밋을 할 수 있습니다 · 내 쪽 대비 받은 변경 6줄** |
|
||||
| 충돌 탭 | - | 충돌 정리 끝 - 병합 커밋을 할 수 있습니다 (말이 같다) |
|
||||
| [병합 커밋] 뒤 (`cea61d2`, 부모 둘) | `cea61d2 · Weld/Unit1.cs` | 차이 없음 · 병합 진행 중 표시 없음 |
|
||||
|
||||
MergeSaveTest 70건 통과. 빌드 경고 0 · 오류 0. 설정 파일은 바이트 그대로 되돌림.
|
||||
|
||||
## E-1. [해결로 저장] 이름을 [Save] 로 (2026-09-14, 추가 요청, 완료)
|
||||
|
||||
사용자: "해결로 저장은 LOCAL SAVE 버튼이잖아? 그냥 Save 로 이름 바꿔주고".
|
||||
@ -26,7 +69,11 @@
|
||||
| `Io/Unit12.cs` 충돌 1개 Take Theirs → [Save] | 알림 **"Save · Unit12.cs · 남은 충돌 23건"**. git 충돌 24 → 23, 파일 `M ` 스테이징, HEAD 그대로(커밋 없음) |
|
||||
| 저장 뒤 | 남은 충돌 첫 파일(`Common/Util/Unit15.cs`)이 네 칸으로 열림 |
|
||||
|
||||
하네스는 이 두 파일(`HomeViewModel.cs` · `MergeSavePlan.cs`)을 컴파일하지 않고 글자를 보는 검사도 없어 그대로(22벌 948건). 빌드 경고 0 · 오류 0.
|
||||
빌드 경고 0 · 오류 0.
|
||||
|
||||
**바로잡음 (A-1 때 발견)** — 처음에 "하네스는 이 파일들을 컴파일하지 않아 그대로" 라고 적었는데 틀렸다.
|
||||
`MergeSaveTest` 가 `MergeSavePlan.cs` 를 컴파일하고, 막힘 안내에 "해결로 저장" 이 들어 있는지 보고 있었다.
|
||||
찾을 때 하네스 폴더를 빠뜨렸고 하네스를 돌리지 않았다. A-1 에서 기대값을 `[Save]` 로 고치고 "옛 이름이 없다" 검사를 더했다.
|
||||
설정 파일은 백업 → `CSharpBraceStyle` 끔 → 시험 → 바이트 그대로 되돌림(해시 같음).
|
||||
|
||||
(자동 조작 쪽: 앱을 새로 켠 뒤에는 충돌 목록에 고른 것이 없어 [3분할로 열기] 가 아무것도 안 열었다. 목록에서 파일을 고르고 다시 했다.)
|
||||
|
||||
@ -8,6 +8,7 @@
|
||||
|
||||
| 날짜 | 항목 | 커밋 |
|
||||
|---|---|---|
|
||||
| 2026-09-14 | **7차 A-1** — 마지막 충돌을 [Save] 하면 파일 비교 탭이 "남은 차이 6줄" 로 떨어져 덜 푼 것처럼 보이던 것. 병합 중 충돌 0 이면 두 칸 요약이 "충돌 정리 끝 · 병합 커밋을 할 수 있습니다 · 내 쪽 대비 받은 변경 N줄", BASE 제목에 "병합 전 내 쪽". E-1 때 못 돌린 MergeSaveTest 기대값도 [Save] 로 | 아래 |
|
||||
| 2026-09-14 | **7차 지시서 · E-1** — 6차 D-1 에서 걸린 세 가지와 pull · rebase · cherry-pick 충돌로 7차 지시서를 씀. 병합 중 [해결로 저장] 단추를 [Save] 로(LOCAL 저장 단추라 Save & Commit 과 짝). 하는 일 그대로, 알림 · 막힘 안내 · 웹 문서 이름도 | 아래 |
|
||||
| 2026-09-14 | **6차 D-1** — 새로 만든 저장소를 MergeHub 만으로 처음부터 병합 커밋까지(충돌 26 · 자동 2 · 파일 단위 15 · 3분할 7 · 지우기/남기기 2). 줄끝 · BOM 이 바뀐 파일 0/33, 표식 0. 5차 C-1 의 23가지 중 사라짐 18. 새로 걸린 3가지는 7차로. 코드 변경 없음 | 아래 |
|
||||
| 2026-09-14 | **6차 C-1** — 병합 · revert · cherry-pick · rebase · pull 결과가 로그에 두 번 찍히던 것을 한 번으로. 충돌 탭 목록에 종류(양쪽 수정 · 상대 삭제 · 내가 삭제 · 양쪽 새 파일)와 "남은 충돌 N개 · 푼 것 M개 (M/전체)". 처음 충돌 목록을 .git 에 적어 두어 앱을 다시 켜도 이어짐 | 아래 |
|
||||
|
||||
@ -150,7 +150,7 @@
|
||||
|
||||
| 상태 | 항목 |
|
||||
|---|---|
|
||||
| 대기 | A-1 마지막 충돌을 저장하면 "남은 차이" 가 뜬다 |
|
||||
| **완료** | A-1 마지막 충돌을 저장하면 "남은 차이" 가 뜬다 — 병합 중 충돌 0 이면 "충돌 정리 끝 · 병합 커밋을 할 수 있습니다 · 내 쪽 대비 받은 변경 N줄", 제목에 "병합 전 내 쪽" |
|
||||
| 대기 | A-2 3분할에서 줄을 골라도 로그는 "3-Way 안 씀" 이라고 한다 |
|
||||
|
||||
### B. 치우기
|
||||
|
||||
@ -50,6 +50,23 @@ namespace MergeHub.Services
|
||||
.Count();
|
||||
}
|
||||
|
||||
/// <summary>
|
||||
/// 병합 중인데 충돌이 다 풀렸을 때 두 칸 비교의 요약. 병합 중이 아니거나 충돌이 남았으면 null(보통 요약을 쓴다).
|
||||
///
|
||||
/// 마지막 충돌을 저장하면 파일 비교 탭은 보통 화면(HEAD ↔ 워킹트리)으로 돌아간다. 거기서 "남은 차이 3줄" 이라고 하면
|
||||
/// 방금 받은 줄이 할 일로 읽혀 덜 푼 것처럼 보였다(7차 A-1). 충돌 탭과 같은 말을 하고, 그 줄 수는 무엇인지 적는다.
|
||||
/// </summary>
|
||||
public static string? ResolvedMergeSummary(bool mergeInProgress, int conflictsLeft, int diffLines)
|
||||
{
|
||||
if (!mergeInProgress || conflictsLeft > 0) return null;
|
||||
|
||||
const string done = "충돌 정리 끝 · 병합 커밋을 할 수 있습니다";
|
||||
return diffLines <= 0 ? done : $"{done} · 내 쪽 대비 받은 변경 {diffLines}줄";
|
||||
}
|
||||
|
||||
/// <summary>병합 중 보통 화면의 BASE 칸 제목. HEAD 는 병합 전 내 쪽이라는 것을 적는다(7차 A-1).</summary>
|
||||
public static string MergeBaseHeader(string shortSha, string path) => $"{shortSha} · {path} · 병합 전 내 쪽";
|
||||
|
||||
/// <summary>
|
||||
/// git 은 경로를 '/' 로, 대소문자를 그대로 본다. 윈도우 경로 구분자로 들어와도 같게 맞춘다.
|
||||
/// 대소문자는 맞추지 않는다 - git 이 서로 다른 파일로 볼 수 있다.
|
||||
|
||||
@ -1304,7 +1304,9 @@ namespace MergeHub.ViewModels
|
||||
? string.IsNullOrEmpty(baseText)
|
||||
? $"({baseOrigin}) 이 파일 없음 - 양쪽에서 새로 만든 파일입니다"
|
||||
: $"({baseOrigin}) {path}"
|
||||
: $"{commit.ShortSha} · {path}";
|
||||
: _workspace.IsMergeInProgress && commit.ShortSha == _workspace.HeadSha
|
||||
? MergeSavePlan.MergeBaseHeader(commit.ShortSha, path!)
|
||||
: $"{commit.ShortSha} · {path}";
|
||||
LocalHeader = _workspace.CurrentBranch;
|
||||
RemoteHeader = isConflict
|
||||
? $"REMOTE · {theirsName ?? "THEIRS"}"
|
||||
@ -3321,6 +3323,12 @@ namespace MergeHub.ViewModels
|
||||
? "충돌 정리 끝"
|
||||
: incoming == 0 ? $"남은 충돌 {conflicts}개" : $"남은 충돌 {conflicts}개 · 받을 변경 {incoming}줄";
|
||||
}
|
||||
else if (IsGitMode &&
|
||||
MergeSavePlan.ResolvedMergeSummary(_workspace.IsMergeInProgress, _workspace.Conflicts.Count, diff) is { } resolved)
|
||||
{
|
||||
// 병합 중 충돌을 다 푼 뒤의 두 칸(HEAD ↔ 워킹트리). 차이는 할 일이 아니라 방금 받은 변경이다(7차 A-1).
|
||||
work = resolved;
|
||||
}
|
||||
|
||||
EditSummary = _undo.Count == 0 && _redo.Count == 0
|
||||
? work
|
||||
|
||||
Loading…
x
Reference in New Issue
Block a user