Skip to content

Regenerate API and remove Tavus support - #196

Merged
zachm-vapi merged 1 commit into
mainfrom
zachm/regen-api-remove-tavus
Oct 30, 2026
Merged

zachm-vapi merged 1 commit into
mainfrom
zachm/regen-api-remove-tavus

Conversation

@zachm-vapi

Copy link
Copy Markdown
Contributor

Related: VAPICS-1747, PRISM-2325

Problem

CI on main fails right now, and so does CI on every PR. ci.yml regenerates api.ts from the live prod spec before it builds. The backend has since removed the disabled Tavus voice provider from the API, so 'tavus' is no longer a valid voice.provider value and the two isVideoEnabled checks in vapi.ts no longer compile:

vapi.ts(588,30): error TS2367: This comparison appears to be unintentional because the types
'"11labs" | "azure" | ... | "sesame" | undefined' and '"tavus"' have no overlap.

Fix

  • Regenerate api.ts from the current prod spec. It also adds CreateWebCallDTO.videoRecordingStartMode, which Add support for videoRecordingStartMode #193 depends on.
  • Remove the Tavus isVideoEnabled check. It subscribed to the assistant participant's video track when the voice was a Tavus avatar. The API can no longer return that provider, so the check was always false:
    • subscribeToTracks now subscribes to remote video only when video recording is enabled, the same behavior as before.
    • The daily-call-object-creation call-start-progress event no longer includes isVideoEnabled in its metadata. This is the only change that consumers can see.

No version bump. The release goes out through the Update Dependencies and API workflow after #193 merges.

Verification

  • tsc --noEmit is clean.
  • npm test passes 73/73.
  • No isVideoEnabled or tavus references remain in vapi.ts.

Tavus was removed from Vapi, so its no longer in the API, which broke
type checks

@shubham-vapiai shubham-vapiai 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.

LGTM

@zachm-vapi
zachm-vapi merged commit e96dfa9 into main Oct 30, 2026
5 checks passed
@zachm-vapi
zachm-vapi deleted the zachm/regen-api-remove-tavus branch October 30, 2026 23:09
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