refactor(reverse_sync): 리스트 마커 공백 감지를 별도 함수로 분리하고 collapse_ws를 제거합니다#984
Open
refactor(reverse_sync): 리스트 마커 공백 감지를 별도 함수로 분리하고 collapse_ws를 제거합니다#984
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
…리합니다
_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
53b154a to
4e83359
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
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에서 marker_ws prefix를 출력에 포함하던 방식을 제거하고,_has_marker_ws_change로 별도 감지하도록 변경collapse_ws호출을replace('\n', ' ').strip()으로 교체하여 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포함Added/updated tests?
_has_marker_ws_change테스트 5건 추가🤖 Generated with Claude Code