Skip to content

confluence-mdx: reverse_sync 리스트 항목 수 불일치 시 전체 inner XHTML 재생성#850

Merged
jk-kim0 merged 1 commit intomainfrom
fix/reverse-sync-list-item-count-mismatch
Feb 25, 2026
Merged

confluence-mdx: reverse_sync 리스트 항목 수 불일치 시 전체 inner XHTML 재생성#850
jk-kim0 merged 1 commit intomainfrom
fix/reverse-sync-list-item-count-mismatch

Conversation

@jk-kim0
Copy link
Contributor

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

Summary

  • build_list_item_patches에서 리스트 항목 수가 달라질 때(삭제/추가) 빈 패치를 반환하여 XHTML에 반영되지 않는 버그 수정
  • parent mapping이 존재하면 전체 리스트를 new_inner_xhtml로 재생성하도록 변경
  • release-notes/10.3.0-10.3.4.mdx의 중복 리스트 항목 삭제 시 verify 실패 해결

Test plan

  • 새 테스트 test_item_count_mismatch_with_parent_generates_inner_xhtml 추가 및 통과
  • 기존 테스트 test_path2_sidecar_match_child_fail_list_split 새 동작에 맞게 업데이트
  • Python 단위 테스트 전체 통과
  • 셸 기반 테스트 (convert, skeleton, reverse-sync, image-copy) 전체 통과
  • bin/reverse_sync_cli.py verify 로 10.3.0-10.3.4.mdx PASS 확인

🤖 Generated with Claude Code

## Description
- `build_list_item_patches`에서 `len(old_items) != len(new_items)`일 때 빈 패치를 반환하여 리스트 항목 삭제/추가가 XHTML에 반영되지 않는 버그를 수정합니다.
- parent mapping이 존재하는 경우, 전체 리스트를 `new_inner_xhtml`로 재생성하도록 변경합니다.

### Background
release-notes/10.3.0-10.3.4.mdx에서 중복된 리스트 항목을 삭제했으나, reverse_sync verify 시 XHTML에서 해당 항목이 제거되지 않아 verify가 실패했습니다.

## Related tickets & links
- split/ko-proofread-20260221-release-notes 브랜치의 verify 실패 건

## Added/updated tests?
- [x] Yes

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

vercel bot commented Feb 25, 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 25, 2026 1:02pm

Request Review

@jk-kim0 jk-kim0 self-assigned this Feb 25, 2026
@jk-kim0 jk-kim0 merged commit f5f307f into main Feb 25, 2026
7 checks passed
@jk-kim0 jk-kim0 deleted the fix/reverse-sync-list-item-count-mismatch branch February 25, 2026 16:29
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