Skip to content

fix: make prompt required for lucy-2-v2v (can be empty string)#37

Merged
AdirAmsalem merged 1 commit intomainfrom
fix/lucy-2-v2v-prompt-required
Mar 9, 2026
Merged

fix: make prompt required for lucy-2-v2v (can be empty string)#37
AdirAmsalem merged 1 commit intomainfrom
fix/lucy-2-v2v-prompt-required

Conversation

@AdirAmsalem
Copy link
Contributor

Summary

  • Make prompt a required field for VideoEdit2Input (can be empty string "")
  • Remove min_length=1 to allow empty strings
  • Remove @model_validator for prompt/reference_image validation
  • Update tests to reflect the change

Context

The API spec (openapi.json) changed prompt from optional to "required": ["data", "prompt"] with description: "Text prompt. Send an empty string if you want no text prompt."

Changes

File Change
decart/models.py prompt: str = Field(..., max_length=1000) — required, allow empty
tests/test_queue.py Update ref-image test with prompt: "", remove invalid test

Update VideoEdit2Input to require prompt field (allowing empty strings)
to match API spec. Remove model_validator for prompt/reference_image
since prompt is always required.
@AdirAmsalem AdirAmsalem merged commit 1e93d84 into main Mar 9, 2026
8 checks passed
@AdirAmsalem AdirAmsalem deleted the fix/lucy-2-v2v-prompt-required branch March 9, 2026 17:13
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