Skip to content

refactor ai video gen#2711

Merged
ProgrammerIn-wonderland merged 3 commits intoHeyPuter:mainfrom
P3il4:ai-video
Mar 24, 2026
Merged

refactor ai video gen#2711
ProgrammerIn-wonderland merged 3 commits intoHeyPuter:mainfrom
P3il4:ai-video

Conversation

@P3il4
Copy link
Copy Markdown
Contributor

@P3il4 P3il4 commented Mar 23, 2026

refactored ai video generation into a single service

  • created main service file similar to image service
  • added two guards against bad duration and dimension values (from wip: ai video gen ai refactor #2698 )
  • changed costs logic from pulling costmaps to having it in models.ts and using costOverrides
  • added duration and seconds values to /models/details of openai models for consistency

smaller stuff:

tested with a mock server implementation of openai video and togetherai video by changing baseurl
.getMonthlyUsage() logged costkeys remain the same
legacy driver hints openai/together-video-generation still work
possibly breaking change for apps: in togetherai durationSeconds is now an array instead of a number
and some togetherai models ids are now lowercase
togetherai:ByteDance/Seedance-1.0-pro -> togetherai:bytedance/seedance-1.0-pro

@ProgrammerIn-wonderland
Copy link
Copy Markdown
Collaborator

GitHub quite literally will not let me merge this PR because it timed out the first time
image

@ProgrammerIn-wonderland
Copy link
Copy Markdown
Collaborator

you might need to just close this one and make another one if it doesn't resolve itself

@ProgrammerIn-wonderland
Copy link
Copy Markdown
Collaborator

but everything seems fine

@ProgrammerIn-wonderland ProgrammerIn-wonderland merged commit bed84ca into HeyPuter:main Mar 24, 2026
4 checks passed
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.

2 participants