Skip to content

fix: limit simultaneous speakers and add mute all - #225

Merged
TheCodeHeist-Coder merged 1 commit into
TheCodeHeist-Coder:mainfrom
dasarisandeepreddy8658:fix/speaker-limit-mute-all
Oct 28, 2026
Merged

TheCodeHeist-Coder merged 1 commit into
TheCodeHeist-Coder:mainfrom
dasarisandeepreddy8658:fix/speaker-limit-mute-all

Conversation

@dasarisandeepreddy8658

Copy link
Copy Markdown
Contributor

What does this PR do?

  • Limits simultaneous speakers to a maximum of three.
  • Prevents hosts from promoting additional speakers when the limit is reached.
  • Adds a host-only Mute All endpoint and button.
  • Displays the current speaker count in the live stage.
  • Adds tests for speaker limits and Mute All permissions.

Related issue

Closes #204

Type of change

  • Bug fix
  • New feature
  • Documentation
  • Refactoring
  • Performance
  • Tests
  • Build / CI / deployment

Which services does this touch?

  • apps/frontend
  • apps/http-server
  • apps/ws-server
  • apps/genAI
  • 'apps/streaming'
  • packages/db (schema or Prisma changes)
  • Docker / CI

How did you test this?

  • Frontend tests: pnpm.cmd --filter frontend test — 84 tests passed.
  • Streaming tests: pnpm.cmd --filter streaming test — 37 tests passed.
  • Frontend type check: pnpm.cmd --filter frontend check-types — passed.
  • Frontend build: pnpm.cmd --filter frontend build — passed.
  • Streaming build: pnpm.cmd --filter streaming build — passed.

Checklist

  • I ran pnpm run lint
  • I ran pnpm run check-types
  • I ran pnpm run build
  • The app still starts with docker-compose up
  • I updated documentation if this changes behaviour or setup
  • I did not commit any .env file, API key or other secret

First pull request here? CONTRIBUTING.md walks through the
whole process, and a checkbox left unticked is not a problem — say so and we
will help.

@TheCodeHeist-Coder
TheCodeHeist-Coder merged commit df96c92 into TheCodeHeist-Coder:main Oct 28, 2026
10 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

A host can let unlimited players speak at once, and cannot mute everyone

2 participants