Skip to content

feat(video-generator): refresh native provider catalogs and request protocols - #9658

Merged
waleedlatif1 merged 4 commits into
stagingfrom
codex/direct-video-catalogs
Nov 4, 2026
Merged

waleedlatif1 merged 4 commits into
stagingfrom
codex/direct-video-catalogs

Conversation

@waleedlatif1

@waleedlatif1 waleedlatif1 commented Nov 4, 2026 •

Copy link
Copy Markdown
Collaborator

Summary

  • Refresh the Video Generator with all 25 Runway video models plus Google Gemini Omni 1.1 Flash, Luma Ray 3.2, and current MiniMax H3/Hailuo models. Keep supported saved aliases and existing Runway, Luma, and MiniMax defaults.
  • Send each model to its documented API, validate supported settings before submitting a job, and use bounded, DNS-pinned output downloads.
  • Preserve saved setting IDs, Agent audio choices, boolean prompt optimization, default model metadata, and terminal failure handling. Honor published Runway, Ray 3.2, and MiniMax H3 prompt limits before submitting a job. Select the active conditional field definition during workflow state replacement, matching ordinary workflow edits. Keep the fal catalog and its request path intact; record the added client metadata modules in the bundle boundary baseline.
  • Stabilize test fixtures: read large Git messages from a file, and classify only the expected streamed 404 recovery on the two denied-chat E2E cases. Keep all browser error diagnostics and reject unmatched errors. Isolate extension capture scenarios in fresh panels so rejected-send recovery cannot leak into the next scenario.

Type of Change

  • Feature and bug fix

Testing

  • Runway live OpenAPI: 25 models, 49 text/image variants, 1,325 requests through block and tool mapping, including every published maximum prompt length and rejection above that limit.
  • MiniMax and legacy Luma published OpenAPI: 694 request combinations; Google Omni and Ray 3.2 checked against official API documentation and native submit/poll/result regressions. No paid inference jobs were submitted.
  • Existing fal catalog: 112 endpoints and 1,808 schema request variants still pass.
  • Regression checks cover serialization of saved settings with explicit and default models, provider protocols, state replacement, legacy metadata, MiniMax failure/Pro handling, empty and malformed camera controls, cancellation, and output size limits.
  • Replayed the recorded denied-page browser diagnostics through the E2E assertion, with five negative controls for unexpected errors; all seven checks pass.
  • Root tests, all workspace type checks, the separate test-type ratchet, lint, all audits, block registry and generated artifact checks pass.

Checklist

  • Code follows project style guidelines
  • Self-reviewed my changes
  • Tests added/updated and passing (new tests pass the test-audit authoring gate)
  • No new warnings introduced
  • I confirm that I have read and agree to the terms outlined in the Contributor License Agreement (CLA)

@vercel

vercel Bot commented Nov 4, 2026 •

Copy link
Copy Markdown

The latest updates on your projects. Learn more about Vercel for GitHub.

1 Skipped Deployment
Project Deployment Actions Updated
docs Skipped Skipped Nov 4, 2026 4:10am UTC

Request Review

@cubic-dev-ai cubic-dev-ai Bot left a comment •

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

All reported issues were addressed across 18 files

Reply with feedback, questions, or to request a fix.

Turn on auto-fix | Re-trigger cubic

Comment thread apps/sim/tools/video/veo.ts Resolved
Comment thread apps/sim/tools/video/runway.ts Resolved
@greptile-apps

greptile-apps Bot commented Nov 4, 2026 •

Copy link
Copy Markdown
Contributor

RetriggerView in GreptileConfidence Score: 5/5

[Medium impact] The PR appears safe to merge; no new actionable issues were found.

Summary

Refreshes native video models and routes requests through provider-specific APIs.

  • The Video Generator offers current models with settings tailored to each one.
  • Direct video models use their own request formats and return handling.
  • Workflow replacement validates the active conditional field definition.
  • Test scripts isolate browser scenarios and keep expected errors distinct.

All four earlier, unnumbered findings are addressed in the current code. No new actionable issues were found in the changes since the previous review.

Diagram

%%{init: {'theme': 'neutral'}}%%
flowchart LR
  A[Workflow or Agent] --> B[Map selected model settings]
  B --> C{Provider}
  C -->|Native| D[Check settings and prompt]
  D --> E[Submit provider job]
  E --> F[Poll provider job]
  F --> G[Read bounded video output]
  C -->|Fal.ai| H[Existing Fal.ai path]
  G --> I[Store video and return metadata]
  H --> I
Loading

Reviews (4) · Last reviewed commit: "fix(video-generator): retain empty contr..." · Reviewed by Greptile

Comment thread apps/sim/lib/internal/video/client.ts Resolved
@waleedlatif1

Copy link
Copy Markdown
Collaborator Author

@greptile

@waleedlatif1

Copy link
Copy Markdown
Collaborator Author

@cubic-dev-ai review this PR

@cubic-dev-ai

cubic-dev-ai Bot commented Nov 4, 2026

Copy link
Copy Markdown
Contributor

@cubic-dev-ai review this PR

@waleedlatif1 I have started the AI code review. It will take a few minutes to complete.

@cubic-dev-ai cubic-dev-ai Bot left a comment •

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

All reported issues were addressed across 22 files

Reply with feedback, questions, or to request a fix.

Turn on auto-fix | Re-trigger cubic

@waleedlatif1

Copy link
Copy Markdown
Collaborator Author

@greptile

@waleedlatif1

Copy link
Copy Markdown
Collaborator Author

@cubic-dev-ai review this PR

@cubic-dev-ai

cubic-dev-ai Bot commented Nov 4, 2026

Copy link
Copy Markdown
Contributor

@cubic-dev-ai review this PR

@waleedlatif1 I have started the AI code review. It will take a few minutes to complete.

@cubic-dev-ai cubic-dev-ai Bot left a comment •

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

All reported issues were addressed across 22 files

Reply with feedback, questions, or to request a fix.

Turn on auto-fix | Re-trigger cubic

Comment thread apps/sim/lib/internal/video/client.ts Resolved
@waleedlatif1

Copy link
Copy Markdown
Collaborator Author

@greptile

@waleedlatif1

Copy link
Copy Markdown
Collaborator Author

@cubic-dev-ai review this PR

@cubic-dev-ai

cubic-dev-ai Bot commented Nov 4, 2026

Copy link
Copy Markdown
Contributor

@cubic-dev-ai review this PR

@waleedlatif1 I have started the AI code review. It will take a few minutes to complete.

@cubic-dev-ai cubic-dev-ai Bot left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

No issues found across 23 files

Confidence score: 5/5

  • Automated review surfaced no issues in the provided summaries.
  • No files require special attention.

Turn on auto-fix | Re-trigger cubic

@waleedlatif1
waleedlatif1 merged commit b5660ac into staging Nov 4, 2026
83 checks passed
@waleedlatif1
waleedlatif1 deleted the codex/direct-video-catalogs branch November 4, 2026 04:14

This branch was previously deployed

1 inactive deployment
Preview — 213f6514 Deployed Nov 4, 2026 by vercel[bot]
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