UseConversation WebRTC branches

[AngeloGiacco]

Add optional branch parameter to SessionConfig to enable specifying agent branches for WebRTC connections, consistent with WebSocket functionality.

---

Slack Thread

 

[cursor]

Cursor Agent can help with this pull request. Just @cursor in comments and I'll start working on changes in this branch.
_{Learn more about Cursor Agents}

[AngeloGiacco]

@cursor did you read my prompt ? It's a parameter of the livekit token endpoint and parameter of the websocket not an override. Redo it start from scratch and think carefully , check how we currently get the token and add a branch id param nothing else needs to be changed

[cursor]

I've corrected the implementation. Here's what the fix does:

Changes Made:

1. WebRTCConnection.ts - Added branch as a query parameter to the token endpoint URL:

   let url = `${origin}/v1/convai/conversation/token?agent_id=${config.agentId}&source=${source}&version=${version}`;
   if (config.branch) {
     url += `&branch=${encodeURIComponent(config.branch)}`;
   }

2. WebSocketConnection.ts - Added branch as a query parameter to the WebSocket URL:

   url = `${origin}${WSS_API_PATHNAME}${config.agentId}&source=${source}&version=${version}`;
   if (config.branch) {
     url += `&branch=${encodeURIComponent(config.branch)}`;
   }

3. overrides.ts - Reverted the incorrect override changes (branch is NOT an override)

4. BaseConnection.ts - The branch?: string parameter remains in BaseSessionConfig (from the previous commit)

Usage:

const { startSession } = useConversation({
  agentId: "your-agent-id",
  branch: "feature-branch", // Optional - added to URL query params
  connectionType: "webrtc",
});

The branch parameter is now correctly passed as a URL query parameter to both the LiveKit token endpoint and the WebSocket connection URL, properly URL-encoded.