fix(calls): "Channel Full" counts people, not devices (#342) #351
Merged
jared
merged 1 commits from 2026-10-11 22:57:38 -04:00
fix/voice-full-distinct-users into lotus
No Reviewers
Labels
Clear labels
a11y
area: appearance
area: auth-session
area: build-ci
area: calls
area: desktop
area: media
area: messaging
area: mobile
area: moderation
area: navigation
area: notifications
area: settings
area: threads
bug
dependencies
docs
duplicate
enhancement
help wanted
invalid
needs-human-review
performance
planning
priority: critical
priority: high
priority: low
priority: medium
qa
question
research
security
tech-debt
ux
wontfix
Accessibility: keyboard, screen reader, contrast, motion
Client area: appearance
Client area: auth-session
Client area: build-ci
Client area: calls
Client area: desktop
Client area: media
Client area: messaging
Client area: mobile
Client area: moderation
Client area: navigation
Client area: notifications
Client area: settings
Client area: threads
Something is not working
Third-party package versions and advisories
README / LOTUS_* docs wrong or missing
This issue or pull request already exists
New feature
Need some help
Something is wrong
Re-render storms, leaks, heavy work on hot paths
Data loss, security hole, or crash on a main path
Broken feature or serious usability problem
Minor issue or polish
Wrong behaviour in an edge case or notable degradation
Manual QA: shipped, needs a human in a real environment
More information is needed
XSS, unsafe URLs, data leaks, auth/session
Code health, dead code, fragile patterns
Usability or visual inconsistency
This won't be fixed
No labels
Milestone
No items
No Milestone
Projects
Clear projects
No projects
No Assignees
Notifications
Due Date
No due date set.
Dependencies
No dependencies set.
Reference: LotusGuild/cinny#351
Reference in New Issue
Block a user
Blocking a user prevents them from interacting with repositories, such as opening or commenting on pull requests or issues. Learn more about blocking a user.
Fixes #342.
useVoiceChannelFullcompared MatrixRTC memberships (one per device) withio.lotus.voice_limit. One person on two devices, or a crashed tab whose membership hadn't expired, therefore filled two slots and showed "Channel Full" to everyone else, while the server-side guard (which counts distinct users) would have let them in.The new
utils/voiceChannelFull.ts(4 unit tests) counts distinct senders. Someone already in the call can still add a device, and the header and sidebar counters now show people.Verified on the local stack: limit 2,
@aud_mainin the call on two devices.@aud_bob's prescreen now shows 1/2 with Join enabled (it was "Channel Full (2/2)"), matching the guard's HTTP 200.tscclean · new tests pass · ESLint/Prettier clean.🤖 Generated with Claude Code
https://claude.ai/code/session_017PCb59uY3UKqDqt7B6hzd5