[airmang]

요약

2026-11-10 점검에서 나온 효율 항목을 고칩니다. 출력은 바뀌지 않습니다. 커밋은 항목별로 나눴습니다.

항목	전	후
표 격자를 호출마다 다시 만들던 것: 40×40 표의 tables.map()	81.3초	0.14초
같은 문제: form_002.hwpx(131 KB 양식)의 tables.map()	16.0초	0.19초
fill_cells로 800칸을 한 번에 채우기	13.7초	0.33초
스타일 조회 paragraph_property 1회	0.47 ms	0.075 ms
add_paragraph 4,000번	8.3초	0.35초
TextExtractor.extract_text 4,000문단	3,788 ms	59 ms
save_report(131 KB 양식, 문단 1개 추가 뒤)	847 ms	575 ms
tests/test_coverage_ledger.py	592초	128초

• 표 격자: 표 객체에 한 번 만들어 두고 표 XML과 대조해 달라졌으면 버립니다. table.cell()은 같은 래퍼 객체를 돌려줍니다.
• 스타일 조회: 이미 lxml인 요소를 문자열로 썼다 다시 파싱하던 왕복을 없애고 조회 결과를 보관합니다. 헤더가 바뀌면 버립니다.
• 저장: save_report가 직렬화 때 통과한 검증 결과를 넘겨받습니다(바이트의 SHA-256이 같을 때만). 검증 항목은 줄이지 않았습니다. to_bytes 안의 검증기들이 같은 바이트를 각자 푸는 것은 이번에 건드리지 않았습니다.
• CI: Tests 워크플로의 push를 main으로 좁히고 PR 실행에 concurrency를 걸었습니다. 전에는 PR 가지의 커밋마다 두 번 돌았습니다.
• 메일 머지: 행마다의 재검증을 없앴습니다.

측정은 Windows 11에서 다른 작업이 함께 도는 중에 여러 번 재서 낸 중앙값입니다.

출력이 달라지지 않았다는 확인

• tests의 .hwpx 1,210개를 열어 그대로 저장한 바이트의 SHA-256이 수정 전후로 모두 같습니다.
• 44개 문서 × 17개 시나리오(표 격자·탐색·칸 채움·구조 편집·fill_cells·문단 추가·추출·스타일 조회·save_report·lint·쪽 추정·검증·마크다운)의 결과 해시가 전후로 같습니다.
• 1,210개 자료 전부의 스타일 조회 결과가 전후로 같습니다.

병합 전에 볼 것

• PR이 없는 가지와 태그의 push에서는 Tests가 돌지 않게 됩니다.
• 스타일 조회가 돌려주는 모델 객체는 보관본입니다(char_property와 같은 방식). 받은 객체를 고쳐 쓰는 코드는 src 안에서 찾지 못했습니다.
• 표 격자의 좌표 조회(cell(r, c))는 행·칸 수와 돌려줄 칸만 대조합니다. 다른 칸을 XML로 직접 고쳐 조회 대상과 겹치게 만든 깨진 표는 전처럼 ValueError가 나지 않을 수 있습니다.
• 헤더 목록 안에 XML 주석이 있으면 전에는 조회가 TypeError로 실패했고 이제는 조회됩니다.
• 워크플로 변경은 실행해 보지 않았습니다.

관련 이슈

Refs #628

변경 종류

• 버그 수정
• 기능
• 문서
• 리팩터링

체크리스트

• pytest 테스트가 통과합니다. — Windows 11에서 7,690 통과, 14 실패. 실패 14건은 이 변경 전 main에서도 같은 이름으로 실패하던 것입니다(cp949, 심볼릭 링크 권한, CRLF). Linux CI 결과는 이 PR의 체크로 확인해 주세요.
• CHANGELOG.md를 갱신했습니다(해당하는 경우).
• 문서 갱신이 필요한지 확인했습니다.

🤖 Generated with Claude Code

[airmang]

Reviewed the table/style/save/extraction optimizations at 25fc358. All eight exact-head checks pass; the Python 3.12 log records 7,704 passed, 18 skipped, 1 xfailed. Local focused checks pass (27 tests), and the integration with #676 passed 44 tests. An independent baseline/head comparison reproduces the loss of the overlapping-grid rejection described inline, so merge is held. The signed-in account owns this PR, so this COMMENT records the required correction rather than an approval.

[airmang]

[P2] Validate changes to other cells before returning a cached grid entry

The fast path checks only the returned cell. With a warmed 1×2 table, change the second cell's raw hp:cellAddr/@colAddr from 1 to 0, then call table.cell(0, 0). Main 168fe98 raises ValueError: table grid contains overlapping cell spans; this head returns the first cell successfully, while this head's own _build_cell_grid() still raises the same ValueError. same_shape() sees unchanged row/child counts and cell_unchanged() sees an unchanged first cell, so the cache bypasses the existing invalid-grid guard. Raw XML changes are explicitly supported by the new invalidation contract and tests. Please invalidate/revalidate for structural changes to any contributing cell, and add the regression where the changed cell is different from the queried cell, preserving the previous fail-closed lookup behavior.