fix(calls): say when ML noise suppression fails or falls back; drop the spoofable listener (#260) #413
Merged
jared
merged 1 commits from 2026-10-18 10:53:40 -04:00
fix/denoise-state-notice 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#413
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.
Refs #260. This covers the denoise half. The issue's audio-output half (
io.lotus.audio_output_state) is in the #310 work, waiting on the fork's lotus.24 package.Problem
io.lotus.denoise_state { active, model, error? }. The host never listened ("Unknown or unsupported from-widget action"), so when ML didn't attach, the call silently published the raw mic while Settings still said ML was on.LotusDenoiseFeature, waited for a{type: 'lotus-denoise-status'}postMessage that nothing sends any more. It also accepted that message from any frame, with no source or origin check, and showed its text as a system toast.Change
useDenoiseNotices(embed), mounted next touseScreenshareNotices. When the user has ML selected:active: false→ "ML noise suppression couldn't start, so your microphone is sending unfiltered audio. You can pick another model in Settings → Calls."model→ "DeepFilterNet 3 couldn't start, so noise suppression is using RNNoise instead."LotusDenoiseFeatureis deleted.Verified with a real failure: on
lotusthe dev server 404s the worklets (#293), with ML / RNNoise selected and joining the Voice Lounge.Unit tests for the texts.
tscclean · unit tests pass · ESLint clean.🤖 Generated with Claude Code
https://claude.ai/code/session_017PCb59uY3UKqDqt7B6hzd5
The fork reports what actually runs as io.lotus.denoise_state, but the host never listened ("unsupported action"), so a failed ML denoiser silently published the raw mic. Handle it: toast when ML did not start or fell back to another model. Remove LotusDenoiseFeature, which waited for a postMessage nothing sends any more and accepted it from any frame. Refs #260 Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_017PCb59uY3UKqDqt7B6hzd5