fix: compare .label in predict_merges skip loop, not element object

[Frank-Schruefer]

Bug

In predict_merges, the while loop is intended to skip over interrupting elements (page headers, footers, tables, pictures, captions, footnotes) between two TEXT elements before deciding whether to merge them.

The loop condition compared the PageElement object itself against a list of DocItemLabel enum values:

while ind_p1 < len(sorted_elements) and sorted_elements[ind_p1] in [
    DocItemLabel.PAGE_HEADER,
    DocItemLabel.PAGE_FOOTER,
    ...
]:

Since a PageElement instance is never equal to a DocItemLabel enum value, this comparison always evaluates to False — the skip loop never advances and interrupting elements are never skipped.

Fix

Compare the element's .label attribute instead:

while ind_p1 < len(sorted_elements) and sorted_elements[ind_p1].label in [
    DocItemLabel.PAGE_HEADER,
    ...
]:

[github-actions]

✅ DCO Check Passed

Thanks @Frank-Schruefer, all your commits are properly signed off. 🎉

[mergify]

Merge Protections

Your pull request matches the following merge protections and will not be merged until they are valid.

🟢 Enforce conventional commit

Wonderful, this rule succeeded.

Make sure that we follow https://www.conventionalcommits.org/en/v1.0.0/

• title ~= ^(fix|feat|docs|style|refactor|perf|test|build|ci|chore|revert)(?:\(.+\))?(!)?:

[nikos-livathinos]

Looks good to me. But the code should be properly formatted and pass all code-checks.

[codecov]

Codecov Report

✅ All modified and coverable lines are covered by tests.

📢 Thoughts on this report? Let us know!

[nikos-livathinos]

This PR is superseded by #156