Skip to content

Bugfix/mic stream#299

Draft
ariksfaradi wants to merge 4 commits intomainfrom
bugfix/mic-stream
Draft

Bugfix/mic stream#299
ariksfaradi wants to merge 4 commits intomainfrom
bugfix/mic-stream

Conversation

@ariksfaradi
Copy link
Contributor

@ariksfaradi ariksfaradi commented Jan 27, 2026

Fixing mute/unmute microphone issues

Add light functionality that doesn't close the stream publication but replaces the current track with null


Note

Introduces first-class microphone mute/unmute and centralizes mic management.

  • Adds muteMicrophoneStream/unmuteMicrophoneStream across StreamingManager, LiveKit manager, agent-manager, and types; LiveKit implementation uses RTCRtpSender.replaceTrack(null|mediaStreamTrack) to mute/unmute without unpublishing

  • New useMicrophoneStream hook consolidates device selection, acquisition, publish/unpublish, and mute/unmute; demo/app.tsx rewired UI to this hook and simplified legacy mic logic

  • Extends tests in livekit-manager.test.ts to cover mute/unmute flows, error cases, and multiple cycles

  • Minor refactors: remove isMicrophonePublished, streamline device enumeration, and avoid redundant LiveKit imports

Written by Cursor Bugbot for commit 76d0783. Configure here.

Copy link

@cursor cursor bot left a comment

Choose a reason for hiding this comment

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

Cursor Bugbot has reviewed your changes and found 3 potential issues.

Bugbot Autofix is OFF. To automatically fix reported issues with Cloud Agents, enable Autofix in the Cursor dashboard.

This PR is being reviewed by Cursor Bugbot

Details

Your team is on the Bugbot Free tier. On this plan, Bugbot will review limited PRs each billing cycle for each member of your team.

To receive Bugbot reviews on all of your PRs, visit the Cursor dashboard to activate Pro and start your 14-day free trial.

Comment @cursor review or bugbot run to trigger another review on this PR

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