confluence-mdx: reverse_sync <span style> 가드 및 <ol start="1"> 보존 수정#862
Merged
confluence-mdx: reverse_sync <span style> 가드 및 <ol start="1"> 보존 수정#862
Conversation
F1: list_patcher에서 <span style=...> 포함 리스트 항목을 재생성 대신 텍스트 전이(transfer_text_changes)로 폴백하여 인라인 스타일 소실 방지 F2: mdx_to_xhtml_inline _render_nested_list에서 순서형 리스트를 <ol start="1">으로 렌더링하여 원본 XHTML과 동일한 속성 유지 Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
|
The latest updates on your projects. Learn more about Vercel for GitHub.
|
original.mdx와 improved.mdx를 현재 forward converter의 expected.mdx 기반으로 재생성하여 라운드트립 검증 시 converter 포맷 차이로 인한 fail을 해소합니다. - 544384417: fail → pass (텍스트 변경 1건: 제외할 지→제외할지) - 544379140: fail → pass (텍스트 변경 9건: 완료시→완료 시, 클릭하시면→클릭하면 등) - 544112828: fail → pass (텍스트 변경 2건: 여러개→여러 개, 필요시→필요 시) Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
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.
Summary
R2 (#859) 후속 수정으로, reverse_sync 리스트 패칭 시 발생하던 2가지 데이터 손실/차이를 해결합니다.
<span style>가드 추가 —list_patcher.py에서<span style=...>포함 리스트 항목을 재생성 대신 텍스트 전이(transfer_text_changes)로 폴백하여 인라인 스타일 소실 방지<ol start="1">보존 —mdx_to_xhtml_inline.py_render_nested_list에서 순서형 리스트를<ol start="1">으로 렌더링하여 원본 XHTML과 동일한 속성 유지결과
<span style>소실 4건 +<ol start="1">→<ol>변환 4건 모두 해소 → 텍스트 변경만 남음<ol start="1">→<ol>변환 8건 해소 → 텍스트 변경 1건만 남음테스트케이스 expected 데이터 업데이트
original.mdx와improved.mdx를 현재 forward converter의expected.mdx기반으로 재생성하여, 라운드트립 검증 시 converter 포맷 차이로 인한status: fail을 해소했습니다.제외할 지→제외할지(1건)완료시→완료 시,클릭하시면→클릭하면등 (9건)여러개→여러 개,필요시→필요 시(2건)원인: 기존
original.mdx가 이전 버전 converter 포맷(예:1. **bold** :)으로 작성되어 있어, 현재 converter(예:1. **bold** :)와의 포맷 차이로 라운드트립exact_match가 실패했습니다.expected.mdx(현재 converter 출력)를 기준으로original.mdx/improved.mdx를 재생성하고, 텍스트 교정만improved.mdx에 적용하여 라운드트립이 정확히 일치하도록 했습니다.Test plan
make test-convert(21건 통과)make test-skeleton(18건 통과)make test-reverse-sync(16건 통과)make test-image-copy(1건 통과)make test-xhtml-diff(14건 통과)make test-byte-verify(42건 통과: fast-path 21 + splice 21)make test-render(21건 통과)🤖 Generated with Claude Code