Open
Conversation
|
The latest updates on your projects. Learn more about Vercel for GitHub.
|
c350d4c to
665a5bd
Compare
fd1d4e8 to
3273a5a
Compare
3273a5a to
a705a9c
Compare
a705a9c to
3db3af3
Compare
- _normalize_empty_list_items: 빈 줄 치환 → 줄째 제거로 변경 - _normalize_consecutive_blank_lines: 연속 빈 줄(3+) → 단일 빈 줄 정규화 추가 - _normalize_blank_line_after_br: <br/> 뒤 빈 줄 제거 정규화 추가 Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
- _normalize_empty_list_items: 빈 번호 줄 제거 단위 테스트 3건 - _normalize_consecutive_blank_lines: 연속 빈 줄 정규화 단위 테스트 3건 - _normalize_blank_line_after_br: <br/> 뒤 빈 줄 제거 단위 테스트 3건 - verify_roundtrip 통합 테스트 3건 (각 정규화가 PASS 처리되는지 검증) Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
3db3af3 to
5d248f4
Compare
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Description
bin/reverse-sync verify --branch=split/ko-proofread-20260221-administrator-manual-general실행 시 발생하는 verify diff의 원인을 조사하고, roundtrip verifier 정규화를 정비합니다.목표
verifier 정규화를 정비하여, MDX 교정이 실질적으로 Confluence XHTML 수정으로 이어지는지 검증 가능하게 합니다. 장기적으로는 패처/변환기의 근본 원인을 수정하여 정규화를 줄여나가는 것이 목표입니다.
불일치 현상 및 원인
--no-normalize모드(PR #983)로 정규화를 전부 비활성화하면 45건 중 6건 FAIL (총 diff 11쌍)이 발생합니다. 유형별 분석:유형 A:
<li><p>선행 공백 (3건, diff 3쌍)Confluence XHTML 원본의
<li><p>안에 선행 공백이 있어 FC가 이중 공백을 출력하는 패턴입니다.<li><p> Admin >* Admin→* Admin<p>내 선행 공백<p>7. 생성이7. 생성이→7. 생성이<p>내 이중 공백<li><p> Workflow* Workflow→* Workflow<p>내 선행 공백기본 모드 처리:
_normalize_consecutive_spaces_in_text(연속 공백 → 단일 공백)근본 해결: 패처가 XHTML
<p>내 선행/연속 공백을 정규화하여 패치, 또는 FC가<p>내 leading whitespace를 trim유형 B:
<br/>인접 공백 (2건, diff 3쌍)FC가 XHTML의
<br/>태그 전후 텍스트 노드를 결합할 때 공백이 추가/변형되는 패턴입니다.<code>Delete</code>버튼을 클릭합니다</p><ac:image(XHTML에<br/>없음)클릭합니다.<br/>→클릭합니다. <br/><br/>앞에 공백 추가인 경우<br /> </p>,경우<br /></p><br/> <br/>→<br/> <br/><br/>사이 공백 노드를 이중 공백으로 출력기본 모드 처리:
_normalize_br_space(<br/>앞 공백 제거)근본 해결: FC의
<br/>인접 공백 처리 개선, 또는 패처가<br/>인접 공백을 정규화유형 C: 테이블 셀 패딩 (1건, diff 5쌍)
FC의 컬럼 폭 계산(
_display_width)과 improved.mdx의 수동 패딩이 수 글자 차이나는 패턴입니다.기본 모드 처리:
_normalize_table_cell_padding(셀 패딩 공백 정규화)근본 해결: 테이블 패딩은 시각적 서식이므로 정규화가 올바른 접근 (해결 불필요)
유형별 요약
<li><p>선행 공백_normalize_consecutive_spaces_in_text<br/>인접 공백_normalize_br_space_normalize_table_cell_padding변경 내용
roundtrip_verifier.py:_normalize_empty_list_items: 빈 줄 치환(^([ \t]+)\d+\.\s*$→'') → 줄째 제거(^[ \t]+\d+\.\s*\n→'')로 개선 — 기존 정규식은 빈 줄을 남겨서 연속 빈 줄 문제를 유발_normalize_consecutive_blank_lines추가: 3개 이상 연속 개행 → 2개로 정규화_normalize_blank_line_after_br추가:<br/>뒤 빈 줄 제거test_reverse_sync_roundtrip_verifier.py:_normalize_empty_list_items단위 테스트 3건_normalize_consecutive_blank_lines단위 테스트 3건_normalize_blank_line_after_br단위 테스트 3건verify_roundtrip통합 테스트 3건 (각 패턴이 정규화로 PASS되는지 검증)검증
Added/updated tests?
Additional notes
len()→_display_width()수정은 별도 PR #975로 머지 완료--no-normalize옵션은 PR #983에서 별도로 추가🤖 Generated with Claude Code