Repository navigation
fix(dom): preserve model-declared blank pages - #958
seonghobae wants to merge 1 commit into
Conversation
|
Important Review skippedAuto reviews are disabled on base/target branches other than the default branch. Please check the settings in the CodeRabbit UI or the ⚙️ Run configurationConfiguration used: Repository: ContextualWisdomLab/newsdom-api/.coderabbit.yaml Review profile: CHILL Plan: Advanced Run ID: You can disable this status message by setting the Use the checkbox below for a quick retry:
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
|
Exact-head admission correction — Ready is review admission only. Fresh audit against base
This PR is moved to Draft/Proposed until the causal owner repair is present on a successor exact head and re-audited. Queued/pending work is neither an additional blocker nor passing evidence. No Close, force push, destructive rebase, manual rerun, synthetic status/approval, merge, auto-merge, or bypass was performed. |
Summary
Preserve model-declared blank pages when other pages contain indexed content. The shared DOM builder previously iterated only content indices, so a three-page document with a blank middle page returned pages 1 and 3.
This PR is stacked on #822 at
927ff2dec160a00ed11ec045727a422ff7ba4cbdto preserve the existing security owner changes. Its unique delta is two production lines and one synthetic regression test. Merge after #822 and retarget to develop; do not bypass its review/security gates.Verification
The test preserves blank-page dimensions and surrounding content. Actual GPU OCR, MinerU 4 migration, model quality, and deployment remain unverified; this repairs the existing shared DOM contract independently.