PR 검토 결과 안내드립니다. 코드 자체는 합리적입니다 — `render_paragraph_parts` 가 controls 무시하던 정황을 정확히 짚으셨고, `render_run_content` / `render_equation` 분리 + 모호한 control gap 보수적 처리 + parser 측 XML entity 복원 모두 양호합니다. dry-run merge 도 자동 성공했고 cargo test --lib 1034 passed 검증도 통과했습니다.

다만 본 저장소의 PR 처리 정책상 다음 항목 보강이 필요합니다.

1. 회귀 테스트 보강 — 자기검증 한계

현재 PR 의 회귀 테스트 3개는 모두 자기검증 패턴 입니다:

```rust
let mut doc = Document::default();
para.controls.push(Equation { script: "x < y & z", ... }); // 작성자가 IR 생성
serialize_hwpx(&doc) → bytes;
parse_hwpx(&bytes) → parsed_eq;
assert_eq!(parsed_eq.script, "x < y & z"); // 자기 데이터 검증
```

이 패턴은 rhwp 자체 roundtrip 항등성 만 검증합니다. 다음이 검증 안 됩니다:

• 한컴 spec 일치성 — `hp:equation` 속성 / 자식 요소 / 네임스페이스가 한컴 spec 과 정확히 일치하는지
• 한컴 호환성 — 본 직렬화 결과를 한컴 편집기가 정상 열람하는지
• 실제 한컴 hwpx 의 회귀 — 한컴이 만든 hwpx 의 의미 보존

본 저장소 메모리 (`feedback_self_verification_not_hancom.md`):

자기 라운드트립 통과해도 한컴 거부 가능. HWP 저장 작업은 한컴 수동 검증 게이트 필수

보강 권장 — 실제 한컴 hwpx 샘플 활용

`samples/hwpx/` 에 한컴 생성 hwpx 가 있습니다. 자체 IR 대신 실제 한컴 hwpx 를 활용한 테스트가 더 견고합니다:

```rust
let bytes = std::fs::read("samples/hwpx/<수식 포함 샘플>")?;
let doc = parse_hwpx(&bytes)?;
let serialized = serialize_hwpx(&doc)?;
let reparsed = parse_hwpx(&serialized)?;
// 원본 doc 과 reparsed doc 의 수식 정보가 일치하는지 (rhwp 자체가 만든 데이터 아님)
```

수식 포함 한컴 hwpx 를 본 저장소에서 찾을 수 없으면 메인테이너에게 요청해 주세요.

2. 한컴 호환 검증 자료

본 PR 의 직렬화 결과 hwpx 를 한컴 편집기 (한컴 2010 또는 2022) 에서 정상 열람하는 증빙 (스크린샷 / PDF) 을 PR 본문 또는 댓글에 첨부해 주세요. 이는 본 저장소의 외부 컨트리뷰터 PR 머지 게이트입니다.

테스트 시나리오:

1. 수식 포함 hwpx 샘플 로드
2. 본 PR 의 `serialize_hwpx` 호출 → 새 hwpx 생성
3. 결과 hwpx 를 한컴 편집기에서 열람 → 수식이 정상 보이는지
4. 결과 캡처 첨부

3. devel 기반 rebase 필요

현재 PR 의 base 가 `94d9347` 입니다. 그 이후 devel 에 PR #395 (그림 밝기/대비) + PR #396 (수식 렌더링) 이 머지되어 BEHIND 상태입니다 (다행히 직접 충돌은 없으나 자동 머지 후 rebase 필요).

```bash
git fetch upstream # 또는 origin
git rebase upstream/devel
git push --force-with-lease origin feature/hwpx-equation-serialization
```

4. CI 실행 확인

현재 PR 의 CI status 가 비어있습니다. rebase + push 시 자동으로 실행될 것입니다. 머지 전 모든 check 가 SUCCESS 인지 확인 부탁드립니다.

5. 이슈 #286 milestone

이슈 #286 의 milestone 을 v1.0.0 으로 갱신했습니다. 작성자가 보고하신 내용대로 본 PR 은 v1.0.0 범위에 포함됩니다 — 시기상 적합한 작업입니다.

정리

본 PR 은 OPEN 유지합니다. 위 항목 보강 후 알려주시면 다시 검토 후 머지 진행하겠습니다.

좋은 기여 감사합니다.

검토 의견 감사합니다. 5개 항목 처리 결과를 보고합니다.

1. 회귀 테스트 보강 — 한컴 origin 데이터 활용

자기검증 한계 지적에 동의합니다. samples/equation-lim.hwp(한컴 편집기 origin) 의 수식 IR을 입력으로 사용하는 회귀 테스트를 추가했습니다.

parse_hwp(samples/equation-lim.hwp) → Document IR  (한컴 origin 데이터)
  → serialize_hwpx → bytes
  → parse_hwpx → IR'
  → Equation.{script, font_size, baseline, font_name, color,
              common.{width, height, treat_as_char}}
    모든 필드가 origin 과 일치하는지 단언

자체 IR 생성 패턴(Document::default() + 수동 controls.push()) 을 회피하고, 한컴 편집기가 만든 hwp 파일에서 추출한 Equation 데이터로 직렬화 항등성을 검증합니다.

추가 커밋: ecd7d9a test: 한컴 origin hwp 기반 수식 HWPX 라운드트립 회귀 추가

다만 samples/hwpx/ 폴더에는 <hp:equation> 포함 hwpx 가 부재하여 hwp 샘플로 대체했습니다. 한컴 origin hwpx 가 별도 확보되면 동일 패턴으로 회귀를 한 건 더 추가하겠습니다.

2. 한컴 호환 검증 자료

한컴 한글 2020 (Office 2020 / HOffice110) 에서 본 PR 의 직렬화 결과 hwpx 를 정상 열람 + PDF 내보내기 검증을 수행했습니다.

• 입력: samples/equation-lim.hwp (한컴 편집기 origin)
• 처리: parse_hwp → serialize_hwpx (본 PR 변경사항 반영)
• 산출물: output/pr400_equation_roundtrip.hwpx (7,145 bytes)
• 한컴 한글 2020 열람 후 PDF 내보내기: 11,662 bytes
• 수식 렌더링: lim_{h→0} {f(2+h) - f(2)} / h (한컴 origin 과 시각적으로 일치)

pr400_equation_roundtrip.pdf

3. devel 기반 rebase

origin/devel 위로 깔끔하게 linear rebase 완료.

ecd7d9a  test: 한컴 origin hwp 기반 수식 HWPX 라운드트립 회귀 추가
f25d986  feat: HWPX 수식 직렬화 보존
d81865f  Merge local/devel: PR #400 검토 문서   ← origin/devel

4. CI 실행

rebase + force-with-lease push 직후 CI 자동 트리거됩니다. 결과 공유드리겠습니다.

5. 이슈 #286 milestone v1.0.0

갱신 감사합니다. 시기상 적합하다는 판단에 동의합니다.

정리

재검토 부탁드립니다.

완료. `devel` 에 cherry-pick (작성자 attribution 보존) 으로 머지됨 (commit `acd114c`).

이슈 #286 close 됩니다.

머지 commit

작성자 attribution 보존:

• `28d6eba` (cherry-pick, @cskwork) — feat: HWPX 수식 직렬화 보존
• `43da8f9` (cherry-pick, @cskwork) — test: 한컴 origin hwp 기반 수식 HWPX 라운드트립 회귀 추가

1차 검토 5 항목 모두 정확 대응

특히 자기검증 한계 보강이 모범적이었습니다. `samples/equation-lim.hwp` (한컴 편집기 origin) 의 수식 IR 을 입력으로 `parse_hwp → serialize_hwpx → parse_hwpx` 의 모든 Equation 필드 라운드트립을 검증하는 패턴은 본 저장소의 메모리 원칙 (`feedback_self_verification_not_hancom.md`) 을 정확히 인지한 결과로 보입니다.

한컴 호환 검증

한컴 한글 2020 (Office 2020 / HOffice110) 에서:

• 입력: `samples/equation-lim.hwp` (한컴 origin)
• rhwp serialize 결과 hwpx (7,145 bytes) 정상 열람
• PDF 내보내기 (11,662 bytes) 의 수식 (`lim_{h→0} {f(2+h) - f(2)} / h`) 가 한컴 origin 과 시각적 일치

→ HWPX 직렬화의 한컴 호환성 입증.

검증

• `cargo test --lib`: 1050 passed (1046 → +4 신규 — 한컴 origin 회귀 + 기타 3건)
• `cargo test --test svg_snapshot`: 6/6 passed
• `cargo test --test issue_418`: 1/1 (Task samples/hwpspec.hwp 20 페이지 이미지 이중 출력 (PR 처리 후 회귀) #418 보존)
• `cargo test --lib serializer::hwpx`: 65 passed (한컴 origin 라운드트립 통과 ✅)
• `cargo clippy --lib -- -D warnings`: warning 0건
• WASM 빌드 (Docker): 4,114,661 bytes
• HWPX 직렬화 (저장 경로) 정정이라 시각 판정 불필요

향후 개선 제안

PR 본문에 명시하신 `samples/hwpx/` 의 수식 포함 hwpx 부재 정황은 별도 task 후보입니다. 한컴 origin hwpx (직접) 회귀 테스트도 가능해지면 본 PR 의 회귀 검증이 한 단계 더 강해질 것입니다.

평가

@cskwork 님은 본 저장소의 첫 외부 컨트리뷰터이시고, PR #397 (수식 ATOP) 에 이어 본 PR (HWPX 수식 직렬화) 까지 두 건 모두 깊이 있는 분석 + 정확한 절차 대응으로 처리됐습니다. 한컴 호환 게이트 (rhwp 자체 라운드트립 ≠ 한컴 호환) 정황을 명확히 인지하고 한컴 한글 2020 으로 직접 검증해 주신 점이 특히 양호합니다.

좋은 기여 감사드립니다. 앞으로의 협업 기대합니다.