fix(web): unify deliverables double shell - #275
Merged
Conversation
Share the inset message-shell primitive with Deliverables. Artifact cards now match status surfaces without duplicated styling.
iamjr15
added a commit
that referenced
this pull request
Aug 13, 2026
## Why Production video generation sent two incompatible Veo options: an explicit audio toggle even though Veo 3.1 audio is native, and forced 1080p for four- and six-second clips even though 1080p requires eight seconds. Successful videos were also presented as generic files despite the product contract promising an in-chat preview. ## What changed - align text, reference-image, and extension requests with their mode-specific Veo 3.1 configuration - reject invalid non-eight-second reference and extension requests before provider billing - keep short text generations on the provider-default 720p path for lower latency - lazy-load generated video previews from signed output URLs and add direct Files access - share viewport preview state between image and video deliverables - retain the shared double-shell Deliverables treatment merged in #275 ## Architecture and migration effects No database migration or new environment variable. The Google AI BYOK boundary and R2 artifact ownership are unchanged. ## Verification - `pnpm lint` - `pnpm typecheck` - `pnpm turbo build --force` - `pnpm deadcode` (existing Knip ignore hint only) - `pnpm architecture:check` - `pnpm turbo skills:build` - schema probe: 4-second text accepted, 8-second extension accepted, 4-second extension rejected - production acceptance test will run after the exact merged SHA is deployed to Cloudflare and Vercel
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
Architecture
MessageDoubleShellowns the outer 2px frame, inset surface, radius relationship, background gradient, and entrance animation.MessageStatusShellsupplies status-specific padding and tone;DeliverablesBlocksupplies artifact-specific spacing.Decisions Made
No schema, migration, environment, Worker, or deployment-topology changes.
Verification
pnpm lintpnpm typecheckpnpm turbo build --forcepnpm deadcode(passes with the existing Knip configuration hint)pnpm architecture:checkpnpm turbo skills:build