Skip to content

fix: explicitly re-export DocumentStream from base_models#3183

Closed
stablegenius49 wants to merge 1 commit into
docling-project:mainfrom
stablegenius49:fix-3165-documentstream-export
Closed

fix: explicitly re-export DocumentStream from base_models#3183
stablegenius49 wants to merge 1 commit into
docling-project:mainfrom
stablegenius49:fix-3165-documentstream-export

Conversation

@stablegenius49
Copy link
Copy Markdown
Contributor

Summary

  • explicitly re-export DocumentStream from docling.datamodel.base_models
  • add __all__ to make the module's public exports explicit for static type checkers
  • add a regression test that checks DocumentStream stays part of the public export surface

Testing

  • python3 -m py_compile docling/datamodel/base_models.py tests/test_input_doc.py
  • attempted: python3 -m pytest tests/test_input_doc.py -k 'explicitly_exports_document_stream or valid_buf or invalid_buf'
    • could not run in this workspace because the local environment is missing repo dependencies such as pydantic

Fixes #3165
Related to #614

@mergify
Copy link
Copy Markdown
Contributor

mergify Bot commented Mar 24, 2026

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

@cau-git
Copy link
Copy Markdown
Member

cau-git commented Mar 25, 2026

This appears to be a duplicate of #3181, therefore I am closing this one.

@cau-git cau-git closed this Mar 25, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

DocumentStream not exported from docling.datamodel.base_models for type checkers

2 participants