Skip to content

Use selected Seedance model limits in phone inputs - #62

Merged
gateway merged 1 commit into
mainfrom
fix/ticket24-seedance-phone-limits
Oct 20, 2026
Merged

gateway merged 1 commit into
mainfrom
fix/ticket24-seedance-phone-limits

Conversation

@gateway

@gateway gateway commented Oct 20, 2026 •

Copy link
Copy Markdown
Owner

Summary

Seedance2.5 phone inputs showed9/3/3 and hid add controls at those limits, while desktop and the existing composer contract allow30/10/10. Pass those same computed model limits into the phone component.

selected model → existing composer limits → desktop and phone
Seedance2.0: 9/3/3
Seedance2.5: 30/10/10

Active rails, markup, layout classes, visible-tile overflow, previews, callbacks and upload acceptance remain unchanged. This is a narrow slice of private local Ticket24/SA20, not GitHub issue24.

Evidence

  • Before: actual installed main desktop/phone comparison reproduces30/10/10 versus9/3/3;22 new regression cases fail.
    **After:**25 new boundary/model-switching/upload/removal cases and73 focused tests pass. All194 safe web files/1338 tests and web typecheck pass.
  • Full local lint passes. Independent Standards and Spec reviews report zero findings on 2f5ef9e against 0c359df.
  • Actual candidate built and verified in visible Codex IAB at measured1440×900 and375×900. Seedance2.5 counters and controls work beyond the former caps, disappear at30/10/10, and return below capacity after removal. Live2.0→2.5 switching retains9/3/3 versus30/10/10. Image/video/audio preview, video playback, audio readiness, close/remove and unchanged start-frame rail pass. Zero browser console errors; viewport override reset and staged inputs cleared.
  • Same populated installation, .env unchanged, paid operations disabled. Repeated selections of three existing QA files through actual file choosers reuse existing reference records:182 before/after, normal usage metadata updates only. Asset/preset/workflow/queue/job/run fingerprints match. Both user-confirmed current original Graph exports preserved; original tabs were not controlled.
  • Existing analyzer identities are reconciled only where surrounding prop/context tokens change:5 web findings and13 clone pairs. Independent reconstruction confirms unchanged metrics/fragments/unaffected rows/metadata; no threshold relaxation.
  • All11 exact-head checks passed before merge, including1036 backend tests,1338 web tests, build and Studio browser smoke. Merged as 036ee6a30e142d77881402ac011caf752155e710; its tree equals the reviewed/browser-tested candidate. All11 checks passed again on that exact main merge. Private evidence: evidence/ticket24-seedance-limits/REPORT.md.

Merge Danger

Door: two-way

Focused code revert; no schema or dependency change.

Blast Radius: inputs

Phone Seedance counter/add-control capacity. Provider limits, backend/API, pricing/options, saved state and other rails are unchanged.

@gateway
gateway merged commit 036ee6a into main Oct 20, 2026
11 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.

1 participant