@seanshin 님, PR #282 검토 요청 감사합니다.

현재 이 PR은 devel과 충돌(CONFLICTING) 상태이며, #279와 무관한 커밋/파일이 다수 포함되어 있어 리베이스 정리 후 재요청 부탁드립니다.

확인된 이슈

1. Merge Conflict

mergeStateStatus: DIRTY — devel에 이후 머지된 변경사항과 충돌이 발생했습니다.

2. #279와 무관한 파일이 포함됨

PR diff에서 아래 파일들이 #279 작업 범위를 벗어납니다:

• mydocs/pr/pr_278_report.md / pr_278_review.md — 다른 PR(Task #275: WASM canvas OLE RawSvg/Placeholder 처리 복구 (bitmap.hwp/한셀OLE.hwp 빈 페이지) #278)의 검토 문서
• mydocs/eng/plans/task_right_tab_align.md / task_visual_diff.md — 별도 기획 문서
• src/parser/hwpx/header.rs (+97줄) / reader.rs (+117줄) — 목차 right tab 리더 점 렌더링 + 소제목 페이지 번호 정렬 불일치 #279 범위 밖의 파서 변경
• src/parser/cfb_reader.rs / src/renderer/html.rs / layout.rs / table_layout.rs / paragraph_layout.rs — 표 외곽 테두리, PUA 심볼 등 이전 작업 잔재
• CHANGELOG.md, Cargo.toml, README.md, README_EN.md — v0.7.3 릴리즈 관련 (메인테이너 관리 영역)

요청 사항

feature/task279-toc-leader-tab 브랜치를 최신 devel에서 깨끗하게 재분기해주시기 바랍니다.

권장 절차:

# 1. 최신 devel fetch
git fetch upstream devel   # (또는 origin devel)

# 2. 새 브랜치를 devel 기준으로 생성
git checkout -b feature/task279-toc-leader-tab-v2 upstream/devel

# 3. #279 핵심 커밋만 cherry-pick (리더 도트 + 탭 정렬)
git cherry-pick d48af5c   # fix: Task #279 Stage 2+3
git cherry-pick 76436df   # docs: Task #279 Stage 3+4 보고서
git cherry-pick 5d1c80f   # docs: Task #279 수행계획서

# 4. 강제 push 후 PR 갱신 (혹은 새 PR 생성)
git push --force-with-lease origin feature/task279-toc-leader-tab

#279의 최종 변경 범위 (이것만 있으면 됩니다)

• src/renderer/svg.rs, src/renderer/web_canvas.rs — fill_type=3 리더 도트 렌더링
• src/renderer/layout/text_measurement.rs — right tab 클램핑 제외
• mydocs/plans/task_m100_279.md — 수행 계획서
• mydocs/working/task_m100_279_stage3.md — 단계별 보고서

참고

• Task #275: WASM canvas OLE RawSvg/Placeholder 처리 복구 (bitmap.hwp/한셀OLE.hwp 빈 페이지) #278 검토 문서는 이미 별도로 다뤄지고 있습니다 — 이 PR에서 제거해주세요.
• README/CHANGELOG/Cargo.toml의 릴리즈 관련 변경은 메인테이너가 따로 관리합니다.
• 표 외곽 테두리/PUA 심볼 수정은 별도 PR로 분리하시거나, 이미 devel에 머지됐다면 리베이스로 자동 해소될 것입니다.

정리 후 다시 알려주시면 우선순위로 검토하겠습니다. 감사합니다!

@seanshin 님, 검토 코멘트 후 응답이 없으셔서 본 PR 은 close 처리하고, 메인테이너 측에서 #279 를 직접 마무리하기로 결정했습니다.

처음 제출하신 두 핵심 수정 (리더 도트 stroke-dasharray="0.1 3" stroke-linecap="round" + right tab 클램핑 제외) 자체는 좋은 분석이었습니다. 다만 PR 정리(범위 분리, devel rebase) 작업이 길어지는 동안 본 이슈를 더 두면 다른 의존 작업이 막힙니다.

기여 의사 감사드립니다. 다음 PR 에서 다시 뵙겠습니다.

@seanshin 님, close 처리는 정정합니다 — 본 PR 을 다시 열고 메인테이너가 직접 정리해서 마무리하겠습니다.

PR 자체를 close 하면 기여 인정이 어려워지고, 대신 새 PR 을 여는 것보다 본 PR 을 정리해서 머지하는 쪽이 깔끔합니다. maintainerCanModify=true 설정 덕분에 메인테이너가 작성자 브랜치에 직접 push 가능하므로 다음과 같이 진행합니다.

진행 계획

1. 본 PR 의 목차 right tab 리더 점 렌더링 + 소제목 페이지 번호 정렬 불일치 #279 핵심 3 커밋 만 origin/devel 기반 새 head 로 정리
   • 5d1c80f 수행계획서
   • d48af5c Stage 2+3 (svg.rs / web_canvas.rs / text_measurement.rs)
   • 76436df Stage 3+4 보고서
2. 메인테이너가 구현계획서 + 회귀 검증 + 최종 보고서 보강
3. force-push 로 본 PR 갱신
4. CI + 시각 검증 후 admin merge

기여 인정

3 커밋의 author 정보 (seanshin) 를 보존합니다. 머지 후에도 commit log 에 본인 이름으로 남게 됩니다.

분석 자체 (리더 도트 dasharray + right tab 클램핑 제외) 가 정확하셨던 부분을 그대로 살립니다. 마무리 후 알려드리겠습니다.

기여 감사합니다.

@seanshin 님 — PR 정리 + 메인테이너 보강이 완료되었습니다. CI 통과 후 admin merge 진행하겠습니다.

본 PR 의 최종 변경 (8 commits)

메인테이너 추가 보강 (6가지 추가 결함)

본인 핵심 분석 (리더 도트 dasharray + RIGHT 탭 클램핑 제외) 위에 시각 검증 사이클로 6가지 추가 결함 식별·보강:

1. trailing 공백 \t 케이스 (\t ) 가드 정밀화 — cross-run RIGHT 진입 활성화
2. resolve_last_tab_pending 시그니처 확장 (fill_type 추가)
3. 리더 시멘틱 — leader 있는 RIGHT 탭은 cell inner 우측 끝으로 강제 (셀 padding 인지)
4. 페이지번호 폭별 leader 길이 차등화 — leader-bearing TextRun 검색 + leader.end_x 단축
5. 공백 only run carry-over — 정렬 단위 아님
6. trim_start 제거 — 선행 공백 시각 보정 (장제목 두 자리 케이스)

KTX 목차 모든 페이지번호 right edge ≈ 700.0 으로 정렬 통일 (한컴 결과와 동등).

기여 인정 산출물 (7가지)

1. Cherry-pick author 보존 (3 커밋, hyoun mouk shin)
2. Co-Authored-By 체인 (메인테이너 신규 5 커밋)
3. CHANGELOG.md "분석·구현 by @seanshin"
4. HWP Spec Errata entry 30 (TabDef.position 시멘틱 — 발견·구현 본인 명기)
5. 위키 페이지 신규: HWP Tab Leader Rendering (본인 크레딧 머리말)
6. 트러블슈팅 문서 (외부 기여 인정 섹션) — mydocs/troubleshootings/toc_leader_right_tab_alignment.md
7. 최종 보고서 머리말 + 본인 stage3 보고서 인용 보존 — mydocs/report/task_m100_279_report.md

핵심 교훈 (errata 30 + 보고서 § 학습)

HWP 스펙은 데이터 포맷 정의일 뿐, 한컴 조판 알고리즘은 비공개다. 스펙 충실 구현 ≠ 한컴 호환. rhwp 는 "한컴 조판 결과를 재현하는 엔진" 이어야 한다.

본인의 분석이 이 원칙의 정확한 적용 (tab_type != 1 클램핑 제외 + 리더 도트 round cap 시멘틱) 이라 본 task 의 본질적 통찰을 제공해 주셨습니다. 감사합니다.

CI 통과 직후 admin merge 진행하겠습니다.