Skip to content

fix: add reference_image to FILE_FIELDS for proper file handling#44

Merged
AdirAmsalem merged 1 commit intomainfrom
pr43-review-comment
Apr 5, 2026
Merged

fix: add reference_image to FILE_FIELDS for proper file handling#44
AdirAmsalem merged 1 commit intomainfrom
pr43-review-comment

Conversation

@AdirAmsalem
Copy link
Copy Markdown
Contributor

@AdirAmsalem AdirAmsalem commented Apr 5, 2026

Summary

Fixes a bug where reference_image was missing from FILE_FIELDS in both the sync and queue clients. This caused reference_image to bypass the file-field handling path, meaning passing a file path or file-like object as reference_image would fail at validation — only raw bytes worked by accident.

Flagged by Cursor Bugbot on #43.


Note

Low Risk
Low risk: a small input-handling change that only affects how reference_image is routed through validation/file processing in the sync and queue clients.

Overview
Fixes reference_image handling in both sync process() and async queue submissions by adding it to the FILE_FIELDS set.

This ensures reference_image goes through the same file-input path (placeholder-bytes validation + final override) as data/start/end, so file paths or file-like objects are accepted consistently.

Reviewed by Cursor Bugbot for commit 8b1bf16. Bugbot is set up for automated code reviews on this repo. Configure here.

@AdirAmsalem AdirAmsalem merged commit fe0324f into main Apr 5, 2026
9 checks passed
@AdirAmsalem AdirAmsalem deleted the pr43-review-comment branch April 5, 2026 08:01
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.

1 participant