Skip to content

confluence-mdx: callout 블록 reverse sync 패치를 수정합니다#866

Merged
jk-kim0 merged 2 commits intomainfrom
fix/reverse-sync-callout-list-patch
Feb 27, 2026
Merged

confluence-mdx: callout 블록 reverse sync 패치를 수정합니다#866
jk-kim0 merged 2 commits intomainfrom
fix/reverse-sync-callout-list-patch

Conversation

@jk-kim0
Copy link
Contributor

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

Summary

  • callout 블록이 child 매핑에 direct 전략으로 해석될 때 _convert_callout_inner<li><p> 구조를 생성하지 못하는 문제를 containing 전략으로 전환하여 수정합니다. (544380354)
  • _apply_text_changes에서 텍스트 노드가 old_text 전체를 커버하지 않을 때 effective_end +1이 비텍스트 영역(ac:plain-text-body)에 침범하여 잘못된 문자가 포함되는 문제를 수정합니다. (544112828)

Test plan

  • make test-reverse-sync-bugs-one TEST_ID=544380354 — PASS
  • make test-reverse-sync-bugs-one TEST_ID=544112828 — PASS
  • make test-reverse-sync — 16 passed, 0 failed
  • pytest tests/ --ignore=tests/test_unused_attachments.py — 753 passed

🤖 Generated with Claude Code

jk-kim0 and others added 2 commits February 27, 2026 01:30
- 544380354: callout 내부 리스트 패치 (direct 전략 DOM 파괴)
- 544112828: callout 내부 코드 블록 포함 시 텍스트 누수

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
- patch_builder: callout 블록이 child 매핑에 direct 전략으로 해석될 때
  _convert_callout_inner가 <li><p> 구조를 생성하지 못하는 문제를 containing
  전략으로 전환하여 수정합니다.
- xhtml_patcher: _apply_text_changes에서 텍스트 노드가 old_text 전체를
  커버하지 않을 때 effective_end +1이 비텍스트 영역(ac:plain-text-body)에
  침범하여 잘못된 문자가 포함되는 문제를 수정합니다.

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 4:34pm

Request Review

@jk-kim0 jk-kim0 self-assigned this Feb 27, 2026
@jk-kim0 jk-kim0 merged commit b9dae82 into main Feb 27, 2026
7 checks passed
@jk-kim0 jk-kim0 deleted the fix/reverse-sync-callout-list-patch branch February 27, 2026 01:27
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