Skip to content

fix(reverse_sync): 리스트 항목 선행 공백 축소가 XHTML에 반영되지 않는 문제를 수정합니다#977

Merged
jk-kim0 merged 2 commits intomainfrom
jk/fix-fc-li-leading-space
Apr 3, 2026
Merged

fix(reverse_sync): 리스트 항목 선행 공백 축소가 XHTML에 반영되지 않는 문제를 수정합니다#977
jk-kim0 merged 2 commits intomainfrom
jk/fix-fc-li-leading-space

Conversation

@jk-kim0
Copy link
Copy Markdown
Contributor

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

Summary

  • _apply_text_changes에서 텍스트 노드 사이 gap이 축소될 때(예: 2공백→1공백) leading whitespace가 보존되는 버그를 수정합니다
  • 이로 인해 <p> Admin → FC → * Admin (이중 공백)이 발생하던 문제가 해결됩니다
  • gap이 완전 삭제(not gap_new)뿐 아니라 공백으로만 축소된 경우에도 leading을 제거합니다

Test plan

  • make test-reverse-sync 전체 통과 (42 passed, 0 failed)
  • make test-reverse-sync-bugs-one TEST_ID=544377652 통과
  • convert 테스트 전체 통과 (21 passed)
  • pytest 단위 테스트 통과 (929 passed, e2e 2건은 worktree 환경 문제로 제외)

🤖 Generated with Claude Code

…거합니다

텍스트 노드 사이 gap이 축소될 때(예: 2공백→1공백) leading whitespace가
보존되어 `<p> Admin` → FC → `*  Admin` (이중 공백)이 되는 문제를 수정합니다.

gap이 삭제(not gap_new)뿐 아니라 공백만으로 축소된 경우에도
leading을 제거하여 원래 diff 의도를 반영합니다.

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

vercel bot commented Apr 3, 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 3, 2026 2:00pm

Request Review

_apply_text_changes의 gap 축소 로직을 검증하는 3건의 테스트를 추가합니다:
- gap 2공백→1공백 축소 시 <p> leading space 제거 (버그 재현)
- gap 완전 삭제 시 leading 제거 (기존 동작)
- gap 미축소 시 leading 보존 (회귀 방지)

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
@jk-kim0 jk-kim0 self-assigned this Apr 3, 2026
@jk-kim0 jk-kim0 merged commit fb307bc into main Apr 3, 2026
7 checks passed
@jk-kim0 jk-kim0 deleted the jk/fix-fc-li-leading-space branch April 3, 2026 14:59
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