fix: improve meta migration and warning handling

[vagenas]

Suggests deprecating use_legacy_mode param & deriving whether to use "legacy" annotations (i.e. .annotations) as defined in _should_use_legacy_annotations() (which also takes care of preventing unnecessary warnings).

Through this change, even DoclingDocuments created with enrichments (e.g. visual captioning) and without going through Pydantic validation/migration will be able to have their (legacy) annotations serialized.

[github-actions]

✅ DCO Check Passed

Thanks @vagenas, 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)(?:\(.+\))?(!)?:

🟢 Require two reviewer for test updates

Wonderful, this rule succeeded.

When test data is updated, we require two reviewers

• #approved-reviews-by >= 2

[codecov]

Codecov Report

❌ Patch coverage is 75.38462% with 16 lines in your changes missing coverage. Please review.

Files with missing lines	Patch %	Lines
docling_core/transforms/serializer/doctags.py	57.89%	8 Missing ⚠️
docling_core/transforms/serializer/html.py	60.00%	4 Missing ⚠️
docling_core/transforms/serializer/common.py	86.66%	2 Missing ⚠️
docling_core/transforms/serializer/markdown.py	88.88%	1 Missing ⚠️
docling_core/types/doc/document.py	91.66%	1 Missing ⚠️

📢 Thoughts on this report? Let us know!

[dosubot]

Related Documentation

Checked 3 published document(s) in 1 knowledge base(s). No updates required.

^{How did I do? Any feedback?}  

[cau-git]

LGTM