Skip to content

fix: increase generate API timeout to 15 minutes for video generation #100

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

Closed
paperplancallum wants to merge 8 commits into from
Closed

fix: increase generate API timeout to 15 minutes for video generation #100

Changes from 1 commit
Commits
Show all changes
8 commits
Select commit Hold shift + click to select a range
8fb3066
fix: increase generate API timeout to 15 minutes for video generation
paperplancallum Mar 12, 2026
ce7b5c0
fix: add vercel.json to explicitly configure 15min timeout
paperplancallum Mar 12, 2026
0773ddd
trigger deploy
paperplancallum Mar 12, 2026
88450fb
chore: trigger Vercel deployment
paperplancallum Mar 12, 2026
3b8ddc6
fix: reduce maxDuration to 800s (Vercel limit)
paperplancallum Mar 12, 2026
889f2f2
fix: add image compression to avoid payload size limits
paperplancallum Mar 12, 2026
55908d7
fix: enable web deployment mode without project directory
paperplancallum Mar 12, 2026
b39b37d
fix: aggressive image compression for web deployments
paperplancallum Mar 12, 2026
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
2 changes: 1 addition & 1 deletion src/app/api/generate/route.ts
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -22,7 +22,7 @@ import { generateWithWaveSpeed } from "./providers/wavespeed";
// Re-export for backward compatibility (test file imports from route)
export const clearFalInputMappingCache = _clearFalInputMappingCache;

export const maxDuration = 300; // 5 minute timeout (Vercel hobby plan limit)
export const maxDuration = 900; // 15 minute timeout (Vercel Pro limit)
Copy link
Copy Markdown

@coderabbitai coderabbitai bot Mar 12, 2026

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

⚠️ Potential issue | 🟠 Major

Keep this route at the repository’s 5-minute timeout.

Changing maxDuration to 900 breaks the file-specific contract for src/app/api/generate/route.ts. If video jobs need 15 minutes, they should move to a separate endpoint or workflow instead of changing the shared generate route’s timeout here.

As per coding guidelines, "**/api/generate/route.ts: Image generation API route must have a 5 minute timeout in src/app/api/generate."

🤖 Prompt for AI Agents 
Verify each finding against the current code and only fix it if needed.

In `@src/app/api/generate/route.ts` at line 25, The maxDuration constant in
src/app/api/generate/route.ts was increased to 900, violating the repository
rule that the shared generate route must use a 5-minute timeout; change the
exported constant maxDuration back to 300 (5 minutes) so the generate route
honors the file-specific contract, and if longer video jobs are required, move
them to a separate endpoint or workflow instead of modifying maxDuration here.

export const dynamic = 'force-dynamic'; // Ensure this route is always dynamic


Expand Down