refactor(reverse_sync): 리스트 마커 공백 감지를 별도 함수로 분리합니다#984
Open
Conversation
|
The latest updates on your projects. Learn more about Vercel for GitHub.
|
jk-kim0
force-pushed
the
jk/fix-reverse-sync-inline-boundary-whitespace
branch
from
8ff1ab0 to
21b9b91
Compare
jk-kim0
force-pushed
the
jk/fix-reverse-sync-inline-boundary-whitespace
branch
from
21b9b91 to
53b154a
Compare
jk-kim0
changed the title
jk-kim0
force-pushed
the
jk/fix-reverse-sync-inline-boundary-whitespace
branch
from
53b154a to
4e83359
Compare
…리합니다
_normalize_list_for_content_compare에서 marker_ws prefix를 출력에 포함하던 방식을
제거하고, _has_marker_ws_change로 별도 감지하도록 변경합니다.
이로써 collapse_ws 호출 없이 replace('\n', ' ').strip()만으로 XHTML 정렬이
가능해지며, preserved anchor 리스트에서 marker-ws-only 변경이 no-op 패치를
생성하는 문제가 해소됩니다.
- _normalize_list_for_content_compare: current_marker_ws 제거, 항목별 strip 추가
- _has_marker_ws_change: 마커 뒤 공백 변경 별도 감지 함수 추가
- collapse_ws(_old_plain_raw) → _old_plain_raw.replace('\n', ' ').strip()
- has_any_change에 has_marker_ws_change 포함
- 테스트: 기존 marker ws 단위 테스트 갱신 + _has_marker_ws_change 테스트 5건 추가
Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
jk-kim0
force-pushed
the
jk/fix-reverse-sync-inline-boundary-whitespace
branch
from
4e83359 to
9387539
Compare
jk-kim0
changed the title
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
1 participant
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.
Summary
_normalize_list_for_content_compare에서 마커 뒤 공백(current_marker_ws)을 텍스트 접두어로 포함하던 방식을 제거합니다_has_marker_ws_change함수로 별도 감지하도록 분리합니다collapse_ws호출을replace('\n', ' ').strip()으로 교체하여, 내부 가시 공백을 보존하면서 XHTML과 정렬합니다_apply_mdx_diff_to_xhtml)을 건너뛰도록 개선합니다변경 파일
patch_builder.py_normalize_list_for_content_compare:current_marker_ws제거, 항목별.strip()추가patch_builder.py_has_marker_ws_change: 마커 뒤 공백 변경 별도 감지 함수 신규patch_builder.pycollapse_ws(_old_plain_raw)→_old_plain_raw.replace('\n', ' ').strip()patch_builder.pyhas_any_change에has_marker_ws_change플래그 포함Test plan
_normalize_list_for_content_compare마커 공백 무시 단위 테스트 6건 통과_has_marker_ws_change단위 테스트 5건 추가 및 통과🤖 Generated with Claude Code