fix(reverse_sync): code span 뒤 공백이 소실되는 문제를 수정합니다

[jk-kim0]

Summary

_apply_text_changes의 노드 간 gap whitespace 처리 로직을 전면 개편합니다.

기존에는 gap 축소 시 현재 노드의 leading whitespace를 전부 제거하는 단순 로직이었으나, 이로 인해 두 가지 버그가 발생했습니다:

1. code span 뒤 공백 소실: <code>{{..}}</code><span> 안의</span> 구조에서 gap이 2→1로 축소될 때, <span>의 leading이 전부 제거되어 <span>안의</span>가 됨
2. 마크업 경계 포함 gap 미처리: <strong>Secure </strong> : 구조에서 gap에 </strong> 경계가 포함되어 gap_old.isspace() 조건에 진입하지 못하고, trailing space가 잔존

변경 사항

• _compute_gap_whitespace_delta: gap 변경에서 공통 토큰(마크업 등)을 제외한 공백 길이 변화량만 분리하는 헬퍼 추출
• _redistribute_gap_whitespace: gap 축소/확장/삭제 시 current leading ↔ previous trailing 간 재분배 헬퍼 추출
• prev_replaced / prev_eff_end 추적으로 다중 노드 간 gap 분배 지원
• trailing_in_range로 이미 diff 처리된 영역을 gap 계산에서 제외

테스트

• 테스트케이스 883654669 (Slack DM 연동) 추가
• 798064641 golden test (bold 라벨 뒤 colon spacing)
• gap 축소/확장/삭제/mixed 시나리오 단위 테스트 6건

Test plan

• make test-reverse-sync-bugs-one TEST_ID=883654669 PASS
• make test-reverse-sync-bugs — 43 passed, 0 failed
• make test-convert — 21 passed
• make test-reverse-sync — 43 passed
• make test-skeleton — 18 passed
• make test-image-copy — 1 passed
• make test-xhtml-diff — 15 passed
• make test-byte-verify — 21 passed
• make test-render — 21 passed
• pytest — 959 passed (1 pre-existing failure)

🤖 Generated with Claude Code

[vercel]

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

Project	Deployment	Actions	Updated (UTC)
querypie-docs	Ready	Preview, Comment	Apr 4, 2026 5:09pm

[jk-kim0]

[debate-review][sha:bee8454fbfba5b34451cf65f94a769e9601feb34] Consensus reached after 2 rounds.

No actionable issues remain.

[jk-kim0]

[debate-review][sha:0f41c45496e5bb8f5999cf30282f4e719a8c8ac8] Consensus reached after 3 rounds.

Debate Summary

• isu_001 [withdrawn] gap이 이전 노드의 trailing 공백과 현재 노드의 leading 공백에 걸쳐 있을 때, len(gap_old) > len(leading) 조건으로 leading을 전부 제거하면 trailing_in_range가 trailing도 흡수하여 총 공백이 0이 되는 엣지 케이스가 존재합니다. 예: X 안의에서 gap 2→1 축소 시 두 노드 모두 공백이 제거됩니다.

Withdrawn Findings

• confluence-mdx/bin/reverse_sync/xhtml_patcher.py:684 - gap이 이전 노드의 trailing 공백과 현재 노드의 leading 공백에 걸쳐 있을 때, len(gap_old) > len(leading) 조건으로 leading을 전부 제거하면 trailing_in_range가 trailing도 흡수하여 총 공백이 0이 되는 엣지 케이스가 존재합니다. 예: X 안의에서 gap 2→1 축소 시 두 노드 모두 공백이 제거됩니다.
  Reason: 현재 코드와 base 코드에서 동일하게 재현되는 기존 문제라서 이번 PR diff가 도입한 신규 이슈로 볼 수 없습니다.

[jk-kim0]

[debate-review][sha:5abda6e94b92c4b0e025ab504a0380d60d87a731] Consensus reached after 2 rounds.

No actionable issues remain.

[jk-kim0]

[debate-review][sha:ded30dfddef259e92723ebe53fba706e2b57511f] Consensus reached after 4 rounds.

Debate Summary

• isu_001 [accepted] 새 gap 분배 로직은 gap_old/gap_new가 공백만으로 이루어진 경우에만 leading을 줄입니다. 그래서 Secure :처럼 마크업 토큰과 공백이 함께 바뀌는 케이스에서는 앞 공백이 그대로 남아 reverse-sync/798064641가 Secure:대신 Secure :로 round-trip 되며 현재 CI의 Test Confluence MDX Converter가 실제로 이 회귀로 실패합니다.
• isu_002 [accepted] gap 삭제(line 689-701)와 gap 축소(line 706-717) 분기에 leading 줄이기→prev_replaced trailing 흡수 로직이 완전히 동일하게 중복됩니다. 헬퍼 함수로 추출하면 CI 실패(isu_001)와 같은 분기별 불일치 버그를 예방하고, 향후 한쪽만 수정되는 위험을 제거할 수 있습니다.

Applied Fixes

• confluence-mdx/bin/reverse_sync/xhtml_patcher.py:704 - (reported by codex, applied by codex) 새 gap 분배 로직은 gap_old/gap_new가 공백만으로 이루어진 경우에만 leading을 줄입니다. 그래서 Secure :처럼 마크업 토큰과 공백이 함께 바뀌는 케이스에서는 앞 공백이 그대로 남아 reverse-sync/798064641가 Secure:대신 Secure :로 round-trip 되며 현재 CI의 Test Confluence MDX Converter가 실제로 이 회귀로 실패합니다.
• confluence-mdx/bin/reverse_sync/xhtml_patcher.py:706 - (reported by cc, applied by None) gap 삭제(line 689-701)와 gap 축소(line 706-717) 분기에 leading 줄이기→prev_replaced trailing 흡수 로직이 완전히 동일하게 중복됩니다. 헬퍼 함수로 추출하면 CI 실패(isu_001)와 같은 분기별 불일치 버그를 예방하고, 향후 한쪽만 수정되는 위험을 제거할 수 있습니다.