Skip to content

fix(reverse_sync): preserved anchor 리스트 마커 공백 변경 시 XHTML 미반영 수정#988

Open
jk-kim0 wants to merge 2 commits intomainfrom
jk/fix-reverse-sync-preserved-anchor-fragment-replace
Open

fix(reverse_sync): preserved anchor 리스트 마커 공백 변경 시 XHTML 미반영 수정#988
jk-kim0 wants to merge 2 commits intomainfrom
jk/fix-reverse-sync-preserved-anchor-fragment-replace

Conversation

@jk-kim0
Copy link
Copy Markdown
Contributor

@jk-kim0 jk-kim0 commented Apr 6, 2026

Summary

  • ac:image를 포함하는 preserved anchor 리스트에서 마커 뒤 공백만 변경(예: 4. 항목4. 항목)될 때, collapse_ws 후 old/new가 동일해져 text-level 패치가 no-op이 되는 버그를 수정합니다
  • mapping의 XHTML에서 synthetic reconstruction 데이터를 생성하여 _build_replace_fragment_patch 경로로 처리합니다
  • sidecar.pybuild_list_all_anchor_entries 함수를 추가하여 <li> 직접 자식 <ac:image>도 anchor entry로 수집합니다

Test plan

  • make test-reverse-sync 통합 테스트 43건 통과
  • pytest test_reverse_sync_patch_builder.py 단위 테스트 124건 통과
  • E2E 테스트 케이스는 별도 PR에서 추가 예정

🤖 Generated with Claude Code

ac:image를 포함하는 리스트에서 마커 뒤 공백만 변경(예: `4.  항목` → `4. 항목`)될 때,
collapse_ws 후 old/new가 동일해져 text-level 패치가 no-op이 되는 문제를 수정합니다.
mapping의 XHTML에서 synthetic reconstruction 데이터를 생성하여 replace_fragment 경로로
처리합니다.

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
@vercel
Copy link
Copy Markdown

vercel bot commented Apr 6, 2026

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

Project Deployment Actions Updated (UTC)
querypie-docs Ready Ready Preview, Comment Apr 7, 2026 2:36am

Request Review

jk-kim0 added a commit that referenced this pull request Apr 6, 2026
build_patches() → has_content_change → replace_fragment 경로를 검증하는
통합 테스트를 추가합니다. 기존 단위 테스트는 _normalize_list_for_content_compare()
반환값만 확인하여 실제 회귀를 감지하지 못했습니다.

이 테스트는 동반 PR (#988)의 수정이 적용되어야 통과합니다.

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
…트를 추가합니다

마커 공백만 변경된 이미지 preserved anchor 리스트에서:
- replace_fragment 패치가 생성되고 ac:image가 보존되는지 검증
- patch_xhtml 적용까지 전 과정 E2E 검증

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
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