Skip to content

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

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)#99
AdirAmsalem merged 1 commit intomainfrom
fix/lucy-2-v2v-prompt-required

Conversation

@AdirAmsalem
Copy link
Contributor

@AdirAmsalem AdirAmsalem commented Mar 9, 2026

Summary

  • Make prompt a required field for lucy-2-v2v model (can be empty string "")
  • Remove .min(1) validation to allow empty strings
  • Remove .refine() validation requiring "at least one of prompt or reference_image"
  • Update VideoEdit2Inputs interface: prompt is now non-optional
  • Update unit tests, e2e tests, and examples 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
src/shared/model.ts Schema: prompt required, allow empty, remove .refine()
src/process/types.ts Interface: prompt: string (non-optional)
tests/unit.test.ts Update ref-image test, remove invalid test
tests/e2e.test.ts Add prompt: "" to ref-image test
examples/sdk-core/video/video-editing.ts Add prompt: "" to Option 2

Note

Medium Risk
Medium risk because it changes the public SDK contract and runtime validation for lucy-2-v2v, potentially breaking callers that previously omitted prompt when using only reference_image. The change is straightforward and covered by updated unit/e2e tests and examples.

Overview
Aligns lucy-2-v2v inputs with the updated API spec by making prompt required while allowing it to be an empty string.

Updates the TypeScript inputs (VideoEdit2Inputs) and Zod validation schema to drop the “at least one of prompt/reference_image” rule, and adjusts tests and examples to pass prompt: "" for reference-image-only edits.

Written by Cursor Bugbot for commit f831e12. This will update automatically on new commits. Configure here.

Update lucy-2-v2v model schema to match API spec change where prompt
is now a required field that accepts empty strings. Remove the
'at least one of prompt or reference_image' validation since prompt
is always required.
@AdirAmsalem AdirAmsalem merged commit cd60293 into main Mar 9, 2026
3 of 4 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