Skip to content

fix: v2.2.0 회귀 수정 — 미커밋 리뷰 옵트인, Delivered 건너뛰기 복원 (v2.2.1) - #36

Merged
cskwork merged 3 commits into
mainfrom
fix/v2.2.1-regressions
Oct 13, 2026
Merged

cskwork merged 3 commits into
mainfrom
fix/v2.2.1-regressions

Conversation

@cskwork

@cskwork cskwork commented Oct 13, 2026

Copy link
Copy Markdown
Owner

요약

v2.2.0 사후 리뷰에서 재현된 v2.1.0 대비 회귀를 고치고 작은 수정을 묶은 패치(v2.2.1)입니다. 커밋 2개: a34af4a(회귀 2건 + 소규모 3건), 31993f2(사후 리뷰 비차단 4건).

회귀 수정

  1. 미커밋 tree 바인딩을 옵트인으로 — review <slug> pass --uncommitted <note>일 때만 HEAD+tree를 묶습니다. 기본은 v2.1.0처럼 HEAD만 묶고 fingerprint·object store 쓰기를 하지 않습니다. (v2.2.0: 커밋 후 리뷰 때 남은 부산물을 지우면 score가 정상 커밋을 거부)
  2. Delivered 줄 건너뛰기 복원 — 로컬에 없는 저장소의 Delivered 줄은 v2.1.0처럼 건너뜁니다. 일치하는 줄이 없을 때만 추가할 줄과 건너뛴 경로를 안내합니다. (v2.2.0: 비로컬 Delivered 줄 하나로 close 중단)
  3. 끝의 빈 셀 허용 — 헤더보다 많은 셀이 전부 빈 칸이면 GFM 유효 행으로 받습니다.

소규모

  • gc로 지워진 리뷰 tree를 score가 is gone (git gc?): review again으로 안내(ref 고정은 하지 않음: 사용자 저장소 오염·worktree 간 충돌).
  • FAIL 기록 시 저장소 검사 생략(FAIL은 score되지 않음).
  • repos: 쉼표 구분 허용, 빈 목록(repos: ,) 거부, 절대경로 오류에 repos: 키 이름.
  • amend 안내 범위 정정: 리뷰한 HEAD 자체를 amend·rebase한 경우만 새 리뷰.
  • --uncommitted가 미추적 파일도 object store에 해시한다는 문서 한 줄.

검증

  • bash tests/selftest.sh: 115 checks PASS(macOS bash 3.2, 두 세션에서 각각 확인).
  • 새 회귀 테스트(optin, paths, repos, uncommitted pruned-tree, cells trailing)는 v2.2.0(530d62b)에서 해당 사유로 실패.
  • 독립 리뷰(a34af4a): v2.1.0(6098242)과 같은 시나리오 11개를 양쪽에서 실행해 기본 경로 동등성 확인, PASS.

호환성

v2.2.0의 암묵적 tree 바인딩에 의존하던 커밋 게이트 프로젝트는 --uncommitted를 붙여야 합니다. v2.2.0 회귀에 이미 걸린 기능은 업그레이드 뒤 리뷰 PASS를 다시 기록합니다. 상세는 log/changelog-v2.2.1.md.

- v2.2.0 회귀 2건 수정: 리뷰 때 남은 부산물을 지운 뒤 score 거부, 로컬에 없는 Delivered 줄로 close 중단
- review pass --uncommitted 일 때만 HEAD+tree를 묶고, 기본은 v2.1.0처럼 HEAD만 묶음
- gc로 지워진 리뷰 tree를 score에서 원인대로 안내
- FAIL 기록 시 저장소 검사 생략, repos: 쉼표 구분 허용, 빈 repos: 목록 거부
- selftest 115 checks, v2.1.0 대비 기본 흐름 동등성 리뷰 PASS
- Results·Scenarios 행 끝에 빈 셀만 더 있으면 허용한다(GFM 유효 표, v2.1.0에서 승인되던 PLAN). 글자가 든 추가 셀은 그대로 오류.
- --uncommitted 안내문과 phases/3-build.md: 새 리뷰가 필요한 것은 리뷰한 HEAD 자체를 amend·rebase한 경우로 좁히고, 그 위 커밋의 메시지 수정은 통과함을 적는다. 리뷰 시 파일이 object store에 해시된다는 점을 한 줄 추가.
- repos:에 절대경로를 쓰면 "repos: '<path>' must be a path relative to the project root"로 키 이름을 밝힌다.
- selftest cells·repos 그룹에 회귀 검사 추가(a34af4a·v2.2.0에서 실패, 지금 115 checks 통과). changelog-v2.2.1 갱신.
@cskwork
cskwork merged commit 2e1c757 into main Oct 13, 2026
3 checks passed
@cskwork
cskwork deleted the fix/v2.2.1-regressions branch October 13, 2026 00:19
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