Skip to content

fix(reverse_sync): 순서 목록 시작 번호(ol start) 변경을 XHTML에 반영합니다#905

Merged
jk-kim0 merged 1 commit intomainfrom
jk/fix-reverse-sync-accuracy-tests
Mar 11, 2026
Merged

fix(reverse_sync): 순서 목록 시작 번호(ol start) 변경을 XHTML에 반영합니다#905
jk-kim0 merged 1 commit intomainfrom
jk/fix-reverse-sync-accuracy-tests

Conversation

@jk-kim0
Copy link
Contributor

@jk-kim0 jk-kim0 commented Mar 11, 2026

Description

MDX 순서 목록의 시작 번호(예: 3. first)가 변경될 때 Confluence XHTML의
<ol start="N"> 속성이 업데이트되지 않는 버그를 수정합니다.

변경 전: MDX 3. first\n4. second → XHTML <ol><li>...</li></ol> (start 속성 없음) ❌
변경 후: MDX 3. first\n4. second → XHTML <ol start="3"><li>...</li></ol>

변경 파일

  • bin/reverse_sync/list_patcher.py: _get_ordered_list_start() 헬퍼 추가, _regenerate_list_from_parent()에서 시작 번호 변경 감지 시 ol_start 필드 포함
  • bin/reverse_sync/xhtml_patcher.py: ol_start 패치 필드로 <ol start> 속성 갱신(또는 start=1이면 제거)
  • tests/test_reverse_sync_xhtml_patcher.py: TestOlStartPatch 클래스 추가 (new_inner_xhtml 경로, 텍스트 전이 경로, start 제거 케이스)
  • tests/test_reverse_sync_list_patcher_ol_start.py: _get_ordered_list_start 단위 테스트 추가

Added/updated tests?

  • Yes — TestOlStartPatch 3개, TestGetOrderedListStart 6개 추가

🤖 Generated with Claude Code

@vercel
Copy link

vercel bot commented Mar 11, 2026

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

Project Deployment Actions Updated (UTC)
querypie-docs Ready Ready Preview, Comment Mar 11, 2026 11:55am

Request Review

MDX 순서 목록의 시작 번호(예: \`3. first\`)가 변경될 때 Confluence XHTML의
<ol start="N"> 속성이 업데이트되지 않는 버그를 수정합니다.

- bin/reverse_sync/list_patcher.py: _get_ordered_list_start() 헬퍼 추가,
  _regenerate_list_from_parent()에서 시작 번호 변경 감지 시 ol_start 필드 포함
- bin/reverse_sync/xhtml_patcher.py: ol_start 패치 필드로 <ol start> 속성 갱신/제거
- tests/test_reverse_sync_xhtml_patcher.py: ol_start 패치 테스트 추가
- tests/test_reverse_sync_list_patcher_ol_start.py: _get_ordered_list_start 단위 테스트 추가

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
@jk-kim0 jk-kim0 force-pushed the jk/fix-reverse-sync-accuracy-tests branch from 5ecf849 to ba028ca Compare March 11, 2026 11:52
@jk-kim0 jk-kim0 changed the title test(confluence-mdx): reverse-sync 정확도 개선 단위 테스트 추가 fix(confluence-mdx): reverse-sync에서 <ol start="N"> 속성 변경을 지원합니다 Mar 11, 2026
@jk-kim0 jk-kim0 changed the title fix(confluence-mdx): reverse-sync에서 <ol start="N"> 속성 변경을 지원합니다 fix(confluence-mdx): reverse-sync 순서 목록 시작 번호(ol start) 변경을 XHTML에 반영합니다 Mar 11, 2026
@jk-kim0 jk-kim0 changed the title fix(confluence-mdx): reverse-sync 순서 목록 시작 번호(ol start) 변경을 XHTML에 반영합니다 fix(reverse_sync): 순서 목록 시작 번호(ol start) 변경을 XHTML에 반영합니다 Mar 11, 2026
@jk-kim0 jk-kim0 self-assigned this Mar 11, 2026
@jk-kim0 jk-kim0 merged commit de9b364 into main Mar 11, 2026
7 checks passed
@jk-kim0 jk-kim0 deleted the jk/fix-reverse-sync-accuracy-tests branch March 11, 2026 12:14
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