Skip to content

feat: Add file-based operations for conversion and chunking #229

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom
Merged
edeandrea merged 1 commit into from
Dec 16, 2025
Merged

feat: Add file-based operations for conversion and chunking #229

edeandrea merged 1 commit into from
Dec 16, 2025
+742 −67

Conversation

@edeandrea
Copy link
Contributor

@edeandrea edeandrea commented Dec 16, 2025

  • Introduced support for file-based conversion and chunking workflows.
  • Added synchronous and asynchronous methods for handling file paths in API and client layers.
  • Enhanced error handling and validation for file-based operations.
  • Updated tests to cover new file-based use cases, including edge cases and null checks.

Fixes #227

@edeandrea
Copy link
Contributor Author

edeandrea commented Dec 16, 2025 •
edited
Loading

So after some examination, really the only difference between "process/chunk file" and "process/chunk url" is that the process/chunk file uses multipart/form-data.

We can do the same thing that /v1/convert/file does using /v1/convert/source (and the chunk variants) by sending the base64 encoded files in the request body.

So all I really did here was add some convenience methods to DoclingServeConvertApi and DoclingServeChunkApi to make constructing the request objects & reading the files easier.

@edeandrea edeandrea force-pushed the process-file-ops branch 2 times, most recently from f7c17fa to 9235db8 Compare December 16, 2025 15:43
@edeandrea
feat: Add file-based operations for conversion and chunking
56be2f2 
- Introduced support for file-based conversion and chunking workflows.
- Added synchronous and asynchronous methods for handling file paths in API and client layers.
- Enhanced error handling and validation for file-based operations.
- Updated tests to cover new file-based use cases, including edge cases and null checks.

Signed-off-by: Eric Deandrea <eric.deandrea@ibm.com>

Fixes docling-project#227

Signed-off-by: Eric Deandrea <eric.deandrea@ibm.com>
@github-actions
Copy link

github-actions bot commented Dec 16, 2025

:java_duke: JaCoCo coverage report

Overall Project 44.7% 🔴

There is no coverage information present for the Files changed

@github-actions
Copy link

github-actions bot commented Dec 16, 2025 •
edited
Loading

TestsPassed ✅SkippedFailed
Gradle Test Results (all modules & JDKs)858 ran858 passed0 skipped0 failed
TestResult
No test annotations available

@github-actions
Copy link

github-actions bot commented Dec 16, 2025

HTML test reports are available as workflow artifacts (zipped HTML).

• Download: Artifacts for this run

@github-actions
Copy link

github-actions bot commented Dec 16, 2025

HTML test reports are available as workflow artifacts (zipped HTML).

• Download: Artifacts for this run

@edeandrea edeandrea merged commit a028d94 into docling-project:main Dec 16, 2025
21 checks passed
@docling-java-ops docling-java-ops bot added the released Issue has been released label Dec 16, 2025
@docling-java-ops
Copy link
Contributor

docling-java-ops bot commented Dec 16, 2025

🎉 This issue has been resolved in v0.4.0 (Release Notes)

@edeandrea edeandrea deleted the process-file-ops branch December 23, 2025 22:14
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@ThomasVitale ThomasVitale ThomasVitale approved these changes

Assignees

No one assigned

Labels

released Issue has been released

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

Implement process file operations

2 participants

@edeandrea @ThomasVitale