Skip to content

confluence-mdx: reverse_sync <span style> 가드 및 <ol start="1"> 보존 수정#862

Merged
jk-kim0 merged 2 commits intomainfrom
fix/r2-followup-span-style-ol-start
Feb 26, 2026
Merged

confluence-mdx: reverse_sync <span style> 가드 및 <ol start="1"> 보존 수정#862
jk-kim0 merged 2 commits intomainfrom
fix/r2-followup-span-style-ol-start

Conversation

@jk-kim0
Copy link
Contributor

@jk-kim0 jk-kim0 commented Feb 26, 2026

Summary

R2 (#859) 후속 수정으로, reverse_sync 리스트 패칭 시 발생하던 2가지 데이터 손실/차이를 해결합니다.

  • F1: <span style> 가드 추가list_patcher.py에서 <span style=...> 포함 리스트 항목을 재생성 대신 텍스트 전이(transfer_text_changes)로 폴백하여 인라인 스타일 소실 방지
  • F2: <ol start="1"> 보존mdx_to_xhtml_inline.py _render_nested_list에서 순서형 리스트를 <ol start="1">으로 렌더링하여 원본 XHTML과 동일한 속성 유지

결과

  • 544379140: XHTML diff에서 <span style> 소실 4건 + <ol start="1"><ol> 변환 4건 모두 해소 → 텍스트 변경만 남음
  • 544384417: <ol start="1"><ol> 변환 8건 해소 → 텍스트 변경 1건만 남음

테스트케이스 expected 데이터 업데이트

original.mdximproved.mdx를 현재 forward converter의 expected.mdx 기반으로 재생성하여, 라운드트립 검증 시 converter 포맷 차이로 인한 status: fail을 해소했습니다.

테스트케이스 변경 전 변경 후 텍스트 교정 내용
544384417 fail pass 제외할 지제외할지 (1건)
544379140 fail pass 완료시완료 시, 클릭하시면클릭하면 등 (9건)
544112828 fail pass 여러개여러 개, 필요시필요 시 (2건)

원인: 기존 original.mdx가 이전 버전 converter 포맷(예: 1. **bold** :)으로 작성되어 있어, 현재 converter(예: 1. **bold** :)와의 포맷 차이로 라운드트립 exact_match가 실패했습니다. expected.mdx(현재 converter 출력)를 기준으로 original.mdx/improved.mdx를 재생성하고, 텍스트 교정만 improved.mdx에 적용하여 라운드트립이 정확히 일치하도록 했습니다.

Test plan

  • Python unit tests (773건 통과)
  • 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

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>
@vercel
Copy link

vercel bot commented Feb 26, 2026

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated (UTC)
querypie-docs Ready Ready Preview, Comment Feb 26, 2026 9:08am

Request Review

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>
@jk-kim0 jk-kim0 self-assigned this Feb 26, 2026
@jk-kim0 jk-kim0 merged commit 5f616e2 into main Feb 26, 2026
7 checks passed
@jk-kim0 jk-kim0 deleted the fix/r2-followup-span-style-ol-start branch February 26, 2026 09:16
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