Skip to content

feat: add live room debate mode#5925

Draft
idoshamun wants to merge 18 commits intomainfrom
flyting
Draft

feat: add live room debate mode#5925
idoshamun wants to merge 18 commits intomainfrom
flyting

Conversation

@idoshamun
Copy link
Copy Markdown
Member

@idoshamun idoshamun commented Apr 27, 2026

What changed

  • Adds the live-room webapp routes and shared live-room GraphQL/hooks/components.
  • Adds the Flyting websocket client protocol/context with mediasoup publishing and subscription support.
  • Adds debate-mode controls for queueing, host moderation, active speaker promotion/removal/kick, and emoji reactions with bottom-to-top animation.
  • Uses usernames for host and visible debate participants, including active speaker and queued participants.

Why

Flyting already exposes the debate-mode websocket behavior. This wires the app side so users can create, join, speak, moderate, react, and watch debate rooms.

Validation

  • node ./scripts/typecheck-strict-changed.js
  • pnpm --filter shared exec eslint ... --max-warnings 0
  • pnpm --filter webapp exec eslint 'pages/live/[id].tsx' pages/live/index.tsx --max-warnings 0\n- git diff --cached --check\n\nNote: targeted Jest for LiveRoomControls.spec.tsx was attempted, but the local shared Jest harness fails before test execution with TypeError: Cannot read properties of undefined (reading 'testEnvironmentOptions').\n

Preview domain

https://flyting.preview.app.daily.dev

@vercel
Copy link
Copy Markdown

vercel Bot commented Apr 27, 2026

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated (UTC)
daily-webapp Ready Ready Preview Apr 29, 2026 8:51pm
1 Skipped Deployment
Project Deployment Actions Updated (UTC)
storybook Ignored Ignored Apr 29, 2026 8:51pm

Request Review

- Show mute indicator on speaker tiles and consolidate mic/camera off
  state with a red strike-through icon in the controls bar.
- Toggle follow/unfollow from speaker tile actions based on the viewer's
  actual content preference status instead of a one-shot local flag.
- Confirm before navigating away from a connected live room, and switch
  the shared exit hook to throw a string so Next.js suppresses the dev
  error overlay when cancelling a route change.
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.

1 participant