wip

Add /joining, /joined, and /leave endpoints to track user join intent
and trigger presence updates.

Backend:
- Add pending_joins Redis module with 30s TTL
- Add /joining endpoint (before WebRTC handshake)
- Add /joined endpoint (after connection, triggers presence poll)
- Add /leave endpoint (on tab close, triggers presence poll)
- Check for pending joins before deactivating meetings in worker

Frontend:
- Generate unique connectionId per browser tab
- Call /joining before Daily.co join, /joined after connection
- Add beforeunload handler calling /leave via sendBeacon

server/migrations/versions/3aa20b96d963_drop_use_celery_column.py

@@ -1,35 +0,0 @@
-"""drop_use_celery_column
-
-Revision ID: 3aa20b96d963
-Revises: e69f08ead8ea
-Create Date: 2026-02-05 10:12:44.065279
-
-"""
-
-from typing import Sequence, Union
-
-import sqlalchemy as sa
-from alembic import op
-
-# revision identifiers, used by Alembic.
-revision: str = "3aa20b96d963"
-down_revision: Union[str, None] = "e69f08ead8ea"
-branch_labels: Union[str, Sequence[str], None] = None
-depends_on: Union[str, Sequence[str], None] = None
-
-
-def upgrade() -> None:
-    with op.batch_alter_table("room", schema=None) as batch_op:
-        batch_op.drop_column("use_celery")
-
-
-def downgrade() -> None:
-    with op.batch_alter_table("room", schema=None) as batch_op:
-        batch_op.add_column(
-            sa.Column(
-                "use_celery",
-                sa.Boolean(),
-                server_default=sa.text("false"),
-                nullable=False,
-            )
-        )

server/reflector/db/meetings.py

@@ -346,27 +346,6 @@ class MeetingController:
             return None
         return Meeting(**result)
 
-    async def get_by_room_and_time_window(
-        self, room: Room, start_date: datetime, end_date: datetime
-    ) -> Meeting | None:
-        """Check if a meeting already exists for this room with the same time window."""
-        query = (
-            meetings.select()
-            .where(
-                sa.and_(
-                    meetings.c.room_id == room.id,
-                    meetings.c.start_date == start_date,
-                    meetings.c.end_date == end_date,
-                    meetings.c.is_active,
-                )
-            )
-            .limit(1)
-        )
-        result = await get_database().fetch_one(query)
-        if not result:
-            return None
-        return Meeting(**result)
-
     async def update_meeting(self, meeting_id: str, **kwargs):
         query = meetings.update().where(meetings.c.id == meeting_id).values(**kwargs)
         await get_database().execute(query)

server/reflector/db/rooms.py

@@ -57,6 +57,12 @@ rooms = sqlalchemy.Table(
         sqlalchemy.String,
         nullable=False,
     ),
+    sqlalchemy.Column(
+        "use_celery",
+        sqlalchemy.Boolean,
+        nullable=False,
+        server_default=false(),
+    ),
     sqlalchemy.Column(
         "skip_consent",
         sqlalchemy.Boolean,
@@ -91,6 +97,7 @@ class Room(BaseModel):
     ics_last_sync: datetime | None = None
     ics_last_etag: str | None = None
     platform: Platform = Field(default_factory=lambda: settings.DEFAULT_VIDEO_PLATFORM)
+    use_celery: bool = False
     skip_consent: bool = False
 
 

server/reflector/presence/__init__.py

@@ -0,0 +1,17 @@
+"""Presence tracking for meetings."""
+
+from reflector.presence.pending_joins import (
+    PENDING_JOIN_PREFIX,
+    PENDING_JOIN_TTL,
+    create_pending_join,
+    delete_pending_join,
+    has_pending_joins,
+)
+
+__all__ = [
+    "PENDING_JOIN_PREFIX",
+    "PENDING_JOIN_TTL",
+    "create_pending_join",
+    "delete_pending_join",
+    "has_pending_joins",
+]

server/reflector/presence/pending_joins.py

@@ -0,0 +1,59 @@
+"""Track pending join intents in Redis.
+
+When a user signals intent to join a meeting (before WebRTC handshake completes),
+we store a pending join record. This prevents the meeting from being deactivated
+while users are still connecting.
+"""
+
+import time
+
+from redis.asyncio import Redis
+
+from reflector.logger import logger
+
+PENDING_JOIN_TTL = 30  # seconds
+PENDING_JOIN_PREFIX = "pending_join"
+# Max keys to scan per Redis SCAN iteration
+SCAN_BATCH_SIZE = 100
+
+
+async def create_pending_join(redis: Redis, meeting_id: str, user_id: str) -> None:
+    """Create a pending join record. Called before WebRTC handshake."""
+    key = f"{PENDING_JOIN_PREFIX}:{meeting_id}:{user_id}"
+    log = logger.bind(meeting_id=meeting_id, user_id=user_id, key=key)
+    await redis.setex(key, PENDING_JOIN_TTL, str(time.time()))
+    log.debug("Created pending join")
+
+
+async def delete_pending_join(redis: Redis, meeting_id: str, user_id: str) -> None:
+    """Delete pending join. Called after WebRTC connection established."""
+    key = f"{PENDING_JOIN_PREFIX}:{meeting_id}:{user_id}"
+    log = logger.bind(meeting_id=meeting_id, user_id=user_id, key=key)
+    await redis.delete(key)
+    log.debug("Deleted pending join")
+
+
+async def has_pending_joins(redis: Redis, meeting_id: str) -> bool:
+    """Check if meeting has any pending joins.
+
+    Uses Redis SCAN to iterate through all keys matching the pattern.
+    Properly iterates until cursor returns 0 to ensure all keys are checked.
+    """
+    pattern = f"{PENDING_JOIN_PREFIX}:{meeting_id}:*"
+    log = logger.bind(meeting_id=meeting_id, pattern=pattern)
+
+    cursor = 0
+    iterations = 0
+    while True:
+        cursor, keys = await redis.scan(
+            cursor=cursor, match=pattern, count=SCAN_BATCH_SIZE
+        )
+        iterations += 1
+        if keys:
+            log.debug("Found pending joins", count=len(keys), iterations=iterations)
+            return True
+        if cursor == 0:
+            break
+
+    log.debug("No pending joins found", iterations=iterations)
+    return False

server/reflector/services/transcript_process.py

@@ -15,10 +15,14 @@ from hatchet_sdk.clients.rest.exceptions import ApiException, NotFoundException
 from hatchet_sdk.clients.rest.models import V1TaskStatus
 
 from reflector.db.recordings import recordings_controller
+from reflector.db.rooms import rooms_controller
 from reflector.db.transcripts import Transcript, transcripts_controller
 from reflector.hatchet.client import HatchetClientManager
 from reflector.logger import logger
 from reflector.pipelines.main_file_pipeline import task_pipeline_file_process
+from reflector.pipelines.main_multitrack_pipeline import (
+    task_pipeline_multitrack_process,
+)
 from reflector.utils.string import NonEmptyString
 
 
@@ -177,98 +181,124 @@ async def dispatch_transcript_processing(
     Returns AsyncResult for Celery tasks, None for Hatchet workflows.
     """
     if isinstance(config, MultitrackProcessingConfig):
-        # Multitrack processing always uses Hatchet (no Celery fallback)
+        use_celery = False
-        # First check if we can replay (outside transaction since it's read-only)
+        if config.room_id:
-        transcript = await transcripts_controller.get_by_id(config.transcript_id)
+            room = await rooms_controller.get_by_id(config.room_id)
-        if transcript and transcript.workflow_run_id and not force:
+            use_celery = room.use_celery if room else False
-            can_replay = await HatchetClientManager.can_replay(
+
-                transcript.workflow_run_id
+        use_hatchet = not use_celery
+
+        if use_celery:
+            logger.info(
+                "Room uses legacy Celery processing",
+                room_id=config.room_id,
+                transcript_id=config.transcript_id,
             )
-            if can_replay:
+
-                await HatchetClientManager.replay_workflow(transcript.workflow_run_id)
+        if use_hatchet:
-                logger.info(
+            # First check if we can replay (outside transaction since it's read-only)
-                    "Replaying Hatchet workflow",
+            transcript = await transcripts_controller.get_by_id(config.transcript_id)
-                    workflow_id=transcript.workflow_run_id,
+            if transcript and transcript.workflow_run_id and not force:
+                can_replay = await HatchetClientManager.can_replay(
+                    transcript.workflow_run_id
                 )
-                return None
+                if can_replay:
-            else:
+                    await HatchetClientManager.replay_workflow(
-                # Workflow can't replay (CANCELLED, COMPLETED, or 404 deleted)
+                        transcript.workflow_run_id
-                # Log and proceed to start new workflow
+                    )
+                    logger.info(
+                        "Replaying Hatchet workflow",
+                        workflow_id=transcript.workflow_run_id,
+                    )
+                    return None
+                else:
+                    # Workflow can't replay (CANCELLED, COMPLETED, or 404 deleted)
+                    # Log and proceed to start new workflow
+                    try:
+                        status = await HatchetClientManager.get_workflow_run_status(
+                            transcript.workflow_run_id
+                        )
+                        logger.info(
+                            "Old workflow not replayable, starting new",
+                            old_workflow_id=transcript.workflow_run_id,
+                            old_status=status.value,
+                        )
+                    except NotFoundException:
+                        # Workflow deleted from Hatchet but ID still in DB
+                        logger.info(
+                            "Old workflow not found in Hatchet, starting new",
+                            old_workflow_id=transcript.workflow_run_id,
+                        )
+
+            # Force: cancel old workflow if exists
+            if force and transcript and transcript.workflow_run_id:
+                try:
+                    await HatchetClientManager.cancel_workflow(
+                        transcript.workflow_run_id
+                    )
+                    logger.info(
+                        "Cancelled old workflow (--force)",
+                        workflow_id=transcript.workflow_run_id,
+                    )
+                except NotFoundException:
+                    logger.info(
+                        "Old workflow already deleted (--force)",
+                        workflow_id=transcript.workflow_run_id,
+                    )
+                await transcripts_controller.update(
+                    transcript, {"workflow_run_id": None}
+                )
+
+            # Re-fetch and check for concurrent dispatch (optimistic approach).
+            # No database lock - worst case is duplicate dispatch, but Hatchet
+            # workflows are idempotent so this is acceptable.
+            transcript = await transcripts_controller.get_by_id(config.transcript_id)
+            if transcript and transcript.workflow_run_id:
+                # Another process started a workflow between validation and now
                 try:
                     status = await HatchetClientManager.get_workflow_run_status(
                         transcript.workflow_run_id
                     )
-                    logger.info(
+                    if status in (V1TaskStatus.RUNNING, V1TaskStatus.QUEUED):
-                        "Old workflow not replayable, starting new",
+                        logger.info(
-                        old_workflow_id=transcript.workflow_run_id,
+                            "Concurrent workflow detected, skipping dispatch",
-                        old_status=status.value,
+                            workflow_id=transcript.workflow_run_id,
-                    )
+                        )
-                except NotFoundException:
+                        return None
-                    # Workflow deleted from Hatchet but ID still in DB
+                except ApiException:
-                    logger.info(
+                    # Workflow might be gone (404) or API issue - proceed with new workflow
-                        "Old workflow not found in Hatchet, starting new",
+                    pass
-                        old_workflow_id=transcript.workflow_run_id,
-                    )
 
-        # Force: cancel old workflow if exists
+            workflow_id = await HatchetClientManager.start_workflow(
-        if force and transcript and transcript.workflow_run_id:
+                workflow_name="DiarizationPipeline",
-            try:
+                input_data={
-                await HatchetClientManager.cancel_workflow(transcript.workflow_run_id)
+                    "recording_id": config.recording_id,
-                logger.info(
+                    "tracks": [{"s3_key": k} for k in config.track_keys],
-                    "Cancelled old workflow (--force)",
+                    "bucket_name": config.bucket_name,
-                    workflow_id=transcript.workflow_run_id,
+                    "transcript_id": config.transcript_id,
-                )
+                    "room_id": config.room_id,
-            except NotFoundException:
+                },
-                logger.info(
+                additional_metadata={
-                    "Old workflow already deleted (--force)",
+                    "transcript_id": config.transcript_id,
-                    workflow_id=transcript.workflow_run_id,
+                    "recording_id": config.recording_id,
-                )
+                    "daily_recording_id": config.recording_id,
-            await transcripts_controller.update(transcript, {"workflow_run_id": None})
+                },
-
-        # Re-fetch and check for concurrent dispatch (optimistic approach).
-        # No database lock - worst case is duplicate dispatch, but Hatchet
-        # workflows are idempotent so this is acceptable.
-        transcript = await transcripts_controller.get_by_id(config.transcript_id)
-        if transcript and transcript.workflow_run_id:
-            # Another process started a workflow between validation and now
-            try:
-                status = await HatchetClientManager.get_workflow_run_status(
-                    transcript.workflow_run_id
-                )
-                if status in (V1TaskStatus.RUNNING, V1TaskStatus.QUEUED):
-                    logger.info(
-                        "Concurrent workflow detected, skipping dispatch",
-                        workflow_id=transcript.workflow_run_id,
-                    )
-                    return None
-            except ApiException:
-                # Workflow might be gone (404) or API issue - proceed with new workflow
-                pass
-
-        workflow_id = await HatchetClientManager.start_workflow(
-            workflow_name="DiarizationPipeline",
-            input_data={
-                "recording_id": config.recording_id,
-                "tracks": [{"s3_key": k} for k in config.track_keys],
-                "bucket_name": config.bucket_name,
-                "transcript_id": config.transcript_id,
-                "room_id": config.room_id,
-            },
-            additional_metadata={
-                "transcript_id": config.transcript_id,
-                "recording_id": config.recording_id,
-                "daily_recording_id": config.recording_id,
-            },
-        )
-
-        if transcript:
-            await transcripts_controller.update(
-                transcript, {"workflow_run_id": workflow_id}
             )
 
-        logger.info("Hatchet workflow dispatched", workflow_id=workflow_id)
+            if transcript:
-        return None
+                await transcripts_controller.update(
+                    transcript, {"workflow_run_id": workflow_id}
+                )
 
+            logger.info("Hatchet workflow dispatched", workflow_id=workflow_id)
+            return None
+
+        # Celery pipeline (durable workflows disabled)
+        return task_pipeline_multitrack_process.delay(
+            transcript_id=config.transcript_id,
+            bucket_name=config.bucket_name,
+            track_keys=config.track_keys,
+        )
     elif isinstance(config, FileProcessingConfig):
         return task_pipeline_file_process.delay(transcript_id=config.transcript_id)
     else:

server/reflector/settings.py

@@ -1,7 +1,7 @@
 from pydantic.types import PositiveInt
 from pydantic_settings import BaseSettings, SettingsConfigDict
 
-from reflector.schemas.platform import DAILY_PLATFORM, Platform
+from reflector.schemas.platform import WHEREBY_PLATFORM, Platform
 from reflector.utils.string import NonEmptyString
 
 
@@ -155,7 +155,7 @@ class Settings(BaseSettings):
         None  # Webhook UUID for this environment. Not used by production code
     )
     # Platform Configuration
-    DEFAULT_VIDEO_PLATFORM: Platform = DAILY_PLATFORM
+    DEFAULT_VIDEO_PLATFORM: Platform = WHEREBY_PLATFORM
 
     # Zulip integration
     ZULIP_REALM: str | None = None

server/reflector/views/daily.py

@@ -1,4 +1,5 @@
 import json
+from datetime import datetime, timezone
 from typing import assert_never
 
 from fastapi import APIRouter, HTTPException, Request
@@ -12,6 +13,9 @@ from reflector.dailyco_api import (
     RecordingReadyEvent,
     RecordingStartedEvent,
 )
+from reflector.db.daily_participant_sessions import (
+    daily_participant_sessions_controller,
+)
 from reflector.db.meetings import meetings_controller
 from reflector.logger import logger as _logger
 from reflector.settings import settings
@@ -141,15 +145,57 @@ async def _handle_participant_joined(event: ParticipantJoinedEvent):
 
 
 async def _handle_participant_left(event: ParticipantLeftEvent):
-    """Queue poll task for presence reconciliation."""
+    """Close session directly on webhook and update num_clients.
-    await _queue_poll_for_room(
+
-        event.payload.room_name,
+    The webhook IS the authoritative signal that a participant left.
-        "participant.left",
+    We close the session immediately rather than polling Daily.co API,
-        event.payload.user_id,
+    which avoids the race where the API still shows the participant.
-        event.payload.session_id,
+    A delayed reconciliation poll is queued as a safety net.
-        duration=event.payload.duration,
+    """
+    room_name = event.payload.room_name
+    if not room_name:
+        logger.warning("participant.left: no room in payload")
+        return
+
+    meeting = await meetings_controller.get_by_room_name(room_name)
+    if not meeting:
+        logger.warning("participant.left: meeting not found", room_name=room_name)
+        return
+
+    log = logger.bind(
+        meeting_id=meeting.id,
+        room_name=room_name,
+        session_id=event.payload.session_id,
+        user_id=event.payload.user_id,
     )
 
+    existing = await daily_participant_sessions_controller.get_open_session(
+        meeting.id, event.payload.session_id
+    )
+
+    if existing:
+        now = datetime.now(timezone.utc)
+        await daily_participant_sessions_controller.batch_close_sessions(
+            [existing.id], left_at=now
+        )
+        active = await daily_participant_sessions_controller.get_active_by_meeting(
+            meeting.id
+        )
+        await meetings_controller.update_meeting(meeting.id, num_clients=len(active))
+        log.info(
+            "Participant left - session closed",
+            remaining_clients=len(active),
+            duration=event.payload.duration,
+        )
+    else:
+        log.info(
+            "Participant left - no open session found, skipping direct close",
+            duration=event.payload.duration,
+        )
+
+    # Delayed reconciliation poll as safety net
+    poll_daily_room_presence_task.apply_async(args=[meeting.id], countdown=5)
+
 
 async def _handle_recording_started(event: RecordingStartedEvent):
     room_name = event.payload.room_name

server/reflector/views/rooms.py

@@ -1,9 +1,9 @@
-import logging
+import json
 from datetime import datetime, timedelta, timezone
 from enum import Enum
 from typing import Annotated, Any, Literal, Optional
 
-from fastapi import APIRouter, Depends, HTTPException
+from fastapi import APIRouter, Depends, HTTPException, Request
 from fastapi_pagination import Page
 from fastapi_pagination.ext.databases import apaginate
 from pydantic import BaseModel
@@ -12,18 +12,24 @@ from redis.exceptions import LockError
 import reflector.auth as auth
 from reflector.db import get_database
 from reflector.db.calendar_events import calendar_events_controller
+from reflector.db.daily_participant_sessions import (
+    DailyParticipantSession,
+    daily_participant_sessions_controller,
+)
 from reflector.db.meetings import meetings_controller
 from reflector.db.rooms import rooms_controller
-from reflector.redis_cache import RedisAsyncLock
+from reflector.logger import logger
+from reflector.presence.pending_joins import create_pending_join, delete_pending_join
+from reflector.redis_cache import RedisAsyncLock, get_async_redis_client
 from reflector.schemas.platform import Platform
 from reflector.services.ics_sync import ics_sync_service
 from reflector.settings import settings
+from reflector.utils.string import NonEmptyString
 from reflector.utils.url import add_query_param
 from reflector.video_platforms.factory import create_platform_client
+from reflector.worker.process import poll_daily_room_presence_task
 from reflector.worker.webhook import test_webhook
 
-logger = logging.getLogger(__name__)
-
 
 class Room(BaseModel):
     id: str
@@ -597,3 +603,221 @@ async def rooms_join_meeting(
         meeting.room_url = add_query_param(meeting.room_url, "t", token)
 
     return meeting
+
+
+class JoiningRequest(BaseModel):
+    """Request body for /joining endpoint (before WebRTC handshake)."""
+
+    connection_id: NonEmptyString
+    """Unique identifier for this connection. Generated by client via crypto.randomUUID()."""
+
+
+class JoiningResponse(BaseModel):
+    status: Literal["ok"]
+
+
+class JoinedRequest(BaseModel):
+    """Request body for /joined endpoint (after WebRTC connection established)."""
+
+    connection_id: NonEmptyString
+    """Must match the connection_id sent to /joining."""
+
+    session_id: NonEmptyString | None = None
+    """Daily.co session_id for direct session creation. Optional for backward compat."""
+
+    user_name: str | None = None
+    """Display name from Daily.co participant data."""
+
+
+class JoinedResponse(BaseModel):
+    status: Literal["ok"]
+
+
+def _get_pending_join_key(
+    user: Optional[auth.UserInfo], connection_id: NonEmptyString
+) -> str:
+    """Get a unique key for pending join tracking.
+
+    Uses user ID for authenticated users, connection_id for anonymous users.
+    This ensures each browser tab has its own unique pending join record.
+    """
+    if user:
+        return f"{user['sub']}:{connection_id}"
+    return f"anon:{connection_id}"
+
+
+@router.post(
+    "/rooms/{room_name}/meetings/{meeting_id}/joining", response_model=JoiningResponse
+)
+async def meeting_joining(
+    room_name: str,
+    meeting_id: str,
+    body: JoiningRequest,
+    user: Annotated[Optional[auth.UserInfo], Depends(auth.current_user_optional)],
+) -> JoiningResponse:
+    """Signal intent to join meeting. Called before WebRTC handshake starts.
+
+    This creates a pending join record that prevents the meeting from being
+    deactivated while the WebRTC handshake is in progress. The record expires
+    automatically after 30 seconds if the connection is not established.
+    """
+    log = logger.bind(
+        room_name=room_name, meeting_id=meeting_id, connection_id=body.connection_id
+    )
+
+    room = await rooms_controller.get_by_name(room_name)
+    if not room:
+        raise HTTPException(status_code=404, detail="Room not found")
+
+    meeting = await meetings_controller.get_by_id(meeting_id, room=room)
+    if not meeting:
+        raise HTTPException(status_code=404, detail="Meeting not found")
+
+    if not meeting.is_active:
+        raise HTTPException(status_code=400, detail="Meeting is not active")
+
+    join_key = _get_pending_join_key(user, body.connection_id)
+
+    redis = await get_async_redis_client()
+    try:
+        await create_pending_join(redis, meeting_id, join_key)
+        log.debug("Created pending join intent", join_key=join_key)
+    finally:
+        await redis.aclose()
+
+    return JoiningResponse(status="ok")
+
+
+@router.post(
+    "/rooms/{room_name}/meetings/{meeting_id}/joined", response_model=JoinedResponse
+)
+async def meeting_joined(
+    room_name: str,
+    meeting_id: str,
+    body: JoinedRequest,
+    user: Annotated[Optional[auth.UserInfo], Depends(auth.current_user_optional)],
+) -> JoinedResponse:
+    """Signal that WebRTC connection is established.
+
+    This clears the pending join record, confirming the user has successfully
+    connected to the meeting. Safe to call even if meeting was deactivated
+    during the handshake (idempotent cleanup).
+    """
+    log = logger.bind(
+        room_name=room_name, meeting_id=meeting_id, connection_id=body.connection_id
+    )
+
+    room = await rooms_controller.get_by_name(room_name)
+    if not room:
+        raise HTTPException(status_code=404, detail="Room not found")
+
+    meeting = await meetings_controller.get_by_id(meeting_id, room=room)
+    if not meeting:
+        raise HTTPException(status_code=404, detail="Meeting not found")
+
+    # Note: We don't check is_active here - the /joined call is a cleanup operation
+    # and should succeed even if the meeting was deactivated during the handshake
+
+    join_key = _get_pending_join_key(user, body.connection_id)
+
+    redis = await get_async_redis_client()
+    try:
+        await delete_pending_join(redis, meeting_id, join_key)
+        log.debug("Cleared pending join intent", join_key=join_key)
+    finally:
+        await redis.aclose()
+
+    # Create session directly when session_id provided (instant presence update)
+    if body.session_id and meeting.platform == "daily":
+        session = DailyParticipantSession(
+            id=f"{meeting.id}:{body.session_id}",
+            meeting_id=meeting.id,
+            room_id=room.id,
+            session_id=body.session_id,
+            user_id=user["sub"] if user else None,
+            user_name=body.user_name or "Anonymous",
+            joined_at=datetime.now(timezone.utc),
+        )
+        await daily_participant_sessions_controller.batch_upsert_sessions([session])
+
+        active = await daily_participant_sessions_controller.get_active_by_meeting(
+            meeting.id
+        )
+        await meetings_controller.update_meeting(meeting.id, num_clients=len(active))
+        log.info(
+            "Session created directly",
+            session_id=body.session_id,
+            num_clients=len(active),
+        )
+
+    # Trigger presence poll as reconciliation safety net
+    if meeting.platform == "daily":
+        poll_daily_room_presence_task.apply_async(args=[meeting_id], countdown=3)
+
+    return JoinedResponse(status="ok")
+
+
+class LeaveResponse(BaseModel):
+    status: Literal["ok"]
+
+
+@router.post(
+    "/rooms/{room_name}/meetings/{meeting_id}/leave", response_model=LeaveResponse
+)
+async def meeting_leave(
+    room_name: str,
+    meeting_id: str,
+    request: Request,
+    user: Annotated[Optional[auth.UserInfo], Depends(auth.current_user_optional)],
+) -> LeaveResponse:
+    """Trigger presence update when user leaves meeting.
+
+    When session_id is provided in the body, closes the session directly
+    for instant presence update. Falls back to polling when session_id
+    is not available (e.g., sendBeacon without frame access).
+    Called on tab close/navigation via sendBeacon().
+    """
+    # Parse session_id from body (sendBeacon may send text/plain or no body)
+    session_id: str | None = None
+    try:
+        body_bytes = await request.body()
+        if body_bytes:
+            data = json.loads(body_bytes)
+            raw = data.get("session_id")
+            if isinstance(raw, str) and raw.strip():
+                session_id = raw.strip()
+    except Exception:
+        pass
+
+    room = await rooms_controller.get_by_name(room_name)
+    if not room:
+        raise HTTPException(status_code=404, detail="Room not found")
+
+    meeting = await meetings_controller.get_by_id(meeting_id, room=room)
+    if not meeting:
+        raise HTTPException(status_code=404, detail="Meeting not found")
+
+    # Close session directly when session_id provided
+    session_closed = False
+    if session_id and meeting.platform == "daily":
+        existing = await daily_participant_sessions_controller.get_open_session(
+            meeting.id, session_id
+        )
+        if existing:
+            await daily_participant_sessions_controller.batch_close_sessions(
+                [existing.id], left_at=datetime.now(timezone.utc)
+            )
+            active = await daily_participant_sessions_controller.get_active_by_meeting(
+                meeting.id
+            )
+            await meetings_controller.update_meeting(
+                meeting.id, num_clients=len(active)
+            )
+            session_closed = True
+
+    # Only queue poll if we couldn't close directly — the poll runs before
+    # Daily.co API removes the participant, which would undo our correct count
+    if meeting.platform == "daily" and not session_closed:
+        poll_daily_room_presence_task.apply_async(args=[meeting_id], countdown=3)
+
+    return LeaveResponse(status="ok")

server/reflector/worker/ics_sync.py

@@ -5,7 +5,7 @@ from celery import shared_task
 from celery.utils.log import get_task_logger
 
 from reflector.asynctask import asynctask
-from reflector.db.calendar_events import CalendarEvent, calendar_events_controller
+from reflector.db.calendar_events import calendar_events_controller
 from reflector.db.meetings import meetings_controller
 from reflector.db.rooms import Room, rooms_controller
 from reflector.redis_cache import RedisAsyncLock
@@ -83,9 +83,10 @@ def _should_sync(room) -> bool:
     return time_since_sync.total_seconds() >= room.ics_fetch_interval
 
 
-async def create_upcoming_meetings_for_event(
+MEETING_DEFAULT_DURATION = timedelta(hours=1)
-    event: CalendarEvent, create_window: datetime, room: Room
+
-):
+
+async def create_upcoming_meetings_for_event(event, create_window, room: Room):
     if event.start_time <= create_window:
         return
     existing_meeting = await meetings_controller.get_by_calendar_event(event.id, room)
@@ -93,21 +94,6 @@ async def create_upcoming_meetings_for_event(
     if existing_meeting:
         return
 
-    # Prevent duplicate meetings from aggregated calendar feeds
-    # (e.g. same event appears with different UIDs from Cal.com and Google Calendar)
-    end_date = event.end_time
-    existing_by_time = await meetings_controller.get_by_room_and_time_window(
-        room, event.start_time, end_date
-    )
-    if existing_by_time:
-        logger.info(
-            "Skipping duplicate calendar event - meeting already exists for this time window",
-            room_id=room.id,
-            event_id=event.id,
-            existing_meeting_id=existing_by_time.id,
-        )
-        return
-
     logger.info(
         "Pre-creating meeting for calendar event",
         room_id=room.id,
@@ -116,6 +102,8 @@ async def create_upcoming_meetings_for_event(
     )
 
     try:
+        end_date = event.end_time or (event.start_time + MEETING_DEFAULT_DURATION)
+
         client = create_platform_client(room.platform)
 
         meeting_data = await client.create_meeting(

server/reflector/worker/process.py

@@ -27,10 +27,14 @@ from reflector.db.transcripts import (
 from reflector.hatchet.client import HatchetClientManager
 from reflector.pipelines.main_file_pipeline import task_pipeline_file_process
 from reflector.pipelines.main_live_pipeline import asynctask
+from reflector.pipelines.main_multitrack_pipeline import (
+    task_pipeline_multitrack_process,
+)
 from reflector.pipelines.topic_processing import EmptyPipeline
+from reflector.presence.pending_joins import has_pending_joins
 from reflector.processors import AudioFileWriterProcessor
 from reflector.processors.audio_waveform_processor import AudioWaveformProcessor
-from reflector.redis_cache import RedisAsyncLock
+from reflector.redis_cache import RedisAsyncLock, get_async_redis_client
 from reflector.settings import settings
 from reflector.storage import get_transcripts_storage
 from reflector.utils.daily import (
@@ -348,29 +352,49 @@ async def _process_multitrack_recording_inner(
             room_id=room.id,
         )
 
-    # Multitrack processing always uses Hatchet (no Celery fallback)
+    use_celery = room and room.use_celery
-    workflow_id = await HatchetClientManager.start_workflow(
+    use_hatchet = not use_celery
-        workflow_name="DiarizationPipeline",
-        input_data={
-            "recording_id": recording_id,
-            "tracks": [{"s3_key": k} for k in filter_cam_audio_tracks(track_keys)],
-            "bucket_name": bucket_name,
-            "transcript_id": transcript.id,
-            "room_id": room.id,
-        },
-        additional_metadata={
-            "transcript_id": transcript.id,
-            "recording_id": recording_id,
-            "daily_recording_id": recording_id,
-        },
-    )
-    logger.info(
-        "Started Hatchet workflow",
-        workflow_id=workflow_id,
-        transcript_id=transcript.id,
-    )
 
-    await transcripts_controller.update(transcript, {"workflow_run_id": workflow_id})
+    if use_celery:
+        logger.info(
+            "Room uses legacy Celery processing",
+            room_id=room.id,
+            transcript_id=transcript.id,
+        )
+
+    if use_hatchet:
+        workflow_id = await HatchetClientManager.start_workflow(
+            workflow_name="DiarizationPipeline",
+            input_data={
+                "recording_id": recording_id,
+                "tracks": [{"s3_key": k} for k in filter_cam_audio_tracks(track_keys)],
+                "bucket_name": bucket_name,
+                "transcript_id": transcript.id,
+                "room_id": room.id,
+            },
+            additional_metadata={
+                "transcript_id": transcript.id,
+                "recording_id": recording_id,
+                "daily_recording_id": recording_id,
+            },
+        )
+        logger.info(
+            "Started Hatchet workflow",
+            workflow_id=workflow_id,
+            transcript_id=transcript.id,
+        )
+
+        await transcripts_controller.update(
+            transcript, {"workflow_run_id": workflow_id}
+        )
+        return
+
+    # Celery pipeline (runs when durable workflows disabled)
+    task_pipeline_multitrack_process.delay(
+        transcript_id=transcript.id,
+        bucket_name=bucket_name,
+        track_keys=filter_cam_audio_tracks(track_keys),
+    )
 
 
 @shared_task
@@ -846,6 +870,18 @@ async def process_meetings():
                     logger_.debug("Meeting not yet started, keep it")
 
                 if should_deactivate:
+                    # Check for pending joins before deactivating
+                    # Users might be in the process of connecting via WebRTC
+                    redis = await get_async_redis_client()
+                    try:
+                        if await has_pending_joins(redis, meeting.id):
+                            logger_.info(
+                                "Meeting has pending joins, skipping deactivation"
+                            )
+                            continue
+                    finally:
+                        await redis.aclose()
+
                     await meetings_controller.update_meeting(
                         meeting.id, is_active=False
                     )
@@ -1049,43 +1085,66 @@ async def reprocess_failed_daily_recordings():
                 )
                 continue
 
-            # Multitrack reprocessing always uses Hatchet (no Celery fallback)
+            use_celery = room and room.use_celery
-            if not transcript:
+            use_hatchet = not use_celery
-                logger.warning(
+
-                    "No transcript for Hatchet reprocessing, skipping",
+            if use_hatchet:
-                    recording_id=recording.id,
+                if not transcript:
+                    logger.warning(
+                        "No transcript for Hatchet reprocessing, skipping",
+                        recording_id=recording.id,
+                    )
+                    continue
+
+                workflow_id = await HatchetClientManager.start_workflow(
+                    workflow_name="DiarizationPipeline",
+                    input_data={
+                        "recording_id": recording.id,
+                        "tracks": [
+                            {"s3_key": k}
+                            for k in filter_cam_audio_tracks(recording.track_keys)
+                        ],
+                        "bucket_name": bucket_name,
+                        "transcript_id": transcript.id,
+                        "room_id": room.id if room else None,
+                    },
+                    additional_metadata={
+                        "transcript_id": transcript.id,
+                        "recording_id": recording.id,
+                        "reprocess": True,
+                    },
+                )
+                await transcripts_controller.update(
+                    transcript, {"workflow_run_id": workflow_id}
                 )
-                continue
 
-            workflow_id = await HatchetClientManager.start_workflow(
+                logger.info(
-                workflow_name="DiarizationPipeline",
+                    "Queued Daily recording for Hatchet reprocessing",
-                input_data={
+                    recording_id=recording.id,
-                    "recording_id": recording.id,
+                    workflow_id=workflow_id,
-                    "tracks": [
+                    room_name=meeting.room_name,
-                        {"s3_key": k}
+                    track_count=len(recording.track_keys),
-                        for k in filter_cam_audio_tracks(recording.track_keys)
+                )
-                    ],
+            else:
-                    "bucket_name": bucket_name,
+                logger.info(
-                    "transcript_id": transcript.id,
+                    "Queueing Daily recording for Celery reprocessing",
-                    "room_id": room.id if room else None,
+                    recording_id=recording.id,
-                },
+                    room_name=meeting.room_name,
-                additional_metadata={
+                    track_count=len(recording.track_keys),
-                    "transcript_id": transcript.id,
+                    transcript_status=transcript.status if transcript else None,
-                    "recording_id": recording.id,
+                )
-                    "reprocess": True,
-                },
-            )
-            await transcripts_controller.update(
-                transcript, {"workflow_run_id": workflow_id}
-            )
 
-            logger.info(
+                # For reprocessing, pass actual recording time (though it's ignored - see _process_multitrack_recording_inner)
-                "Queued Daily recording for Hatchet reprocessing",
+                # Reprocessing uses recording.meeting_id directly instead of time-based matching
-                recording_id=recording.id,
+                recording_start_ts = int(recording.recorded_at.timestamp())
-                workflow_id=workflow_id,
+
-                room_name=meeting.room_name,
+                process_multitrack_recording.delay(
-                track_count=len(recording.track_keys),
+                    bucket_name=bucket_name,
-            )
+                    daily_room_name=meeting.room_name,
+                    recording_id=recording.id,
+                    track_keys=recording.track_keys,
+                    recording_start_ts=recording_start_ts,
+                )
 
             reprocessed_count += 1
 

server/tests/conftest.py

@@ -4,7 +4,7 @@ from unittest.mock import patch
 
 import pytest
 
-from reflector.schemas.platform import DAILY_PLATFORM, WHEREBY_PLATFORM
+from reflector.schemas.platform import WHEREBY_PLATFORM
 
 
 @pytest.fixture(scope="session", autouse=True)
@@ -14,7 +14,6 @@ def register_mock_platform():
     from reflector.video_platforms.registry import register_platform
 
     register_platform(WHEREBY_PLATFORM, MockPlatformClient)
-    register_platform(DAILY_PLATFORM, MockPlatformClient)
     yield
 
 

server/tests/test_daily_webhook_participant_left.py

@@ -0,0 +1,213 @@
+"""Tests for direct session close on participant.left webhook.
+
+Verifies that _handle_participant_left:
+1. Closes the session directly (authoritative signal)
+2. Updates num_clients from remaining active sessions
+3. Queues a delayed reconciliation poll as safety net
+4. Handles missing session gracefully
+"""
+
+from datetime import datetime, timedelta, timezone
+from unittest.mock import AsyncMock, patch
+
+import pytest
+
+from reflector.dailyco_api.webhooks import ParticipantLeftEvent, ParticipantLeftPayload
+from reflector.db.daily_participant_sessions import DailyParticipantSession
+from reflector.db.meetings import Meeting
+from reflector.views.daily import _handle_participant_left
+
+
+@pytest.fixture
+def mock_meeting():
+    return Meeting(
+        id="meeting-123",
+        room_id="room-456",
+        room_name="test-room-20251118120000",
+        room_url="https://daily.co/test-room-20251118120000",
+        host_room_url="https://daily.co/test-room-20251118120000?t=host-token",
+        platform="daily",
+        num_clients=2,
+        is_active=True,
+        start_date=datetime.now(timezone.utc),
+        end_date=datetime.now(timezone.utc),
+    )
+
+
+@pytest.fixture
+def participant_left_event():
+    now = datetime.now(timezone.utc)
+    return ParticipantLeftEvent(
+        version="1.0.0",
+        type="participant.left",
+        id="evt-left-abc123",
+        payload=ParticipantLeftPayload(
+            room_name="test-room-20251118120000",
+            session_id="session-alice",
+            user_id="user-alice",
+            user_name="Alice",
+            joined_at=int((now - timedelta(minutes=10)).timestamp()),
+            duration=600,
+        ),
+        event_ts=int(now.timestamp()),
+    )
+
+
+@pytest.fixture
+def existing_session():
+    now = datetime.now(timezone.utc)
+    return DailyParticipantSession(
+        id="meeting-123:session-alice",
+        meeting_id="meeting-123",
+        room_id="room-456",
+        session_id="session-alice",
+        user_id="user-alice",
+        user_name="Alice",
+        joined_at=now - timedelta(minutes=10),
+        left_at=None,
+    )
+
+
+@pytest.mark.asyncio
+@patch("reflector.views.daily.poll_daily_room_presence_task")
+@patch("reflector.views.daily.meetings_controller")
+@patch("reflector.views.daily.daily_participant_sessions_controller")
+async def test_closes_session_and_updates_num_clients(
+    mock_sessions_ctrl,
+    mock_meetings_ctrl,
+    mock_poll_task,
+    mock_meeting,
+    participant_left_event,
+    existing_session,
+):
+    """Webhook directly closes session and updates num_clients from remaining active count."""
+    mock_meetings_ctrl.get_by_room_name = AsyncMock(return_value=mock_meeting)
+    mock_sessions_ctrl.get_open_session = AsyncMock(return_value=existing_session)
+    mock_sessions_ctrl.batch_close_sessions = AsyncMock()
+    # One remaining active session after close
+    remaining = DailyParticipantSession(
+        id="meeting-123:session-bob",
+        meeting_id="meeting-123",
+        room_id="room-456",
+        session_id="session-bob",
+        user_id="user-bob",
+        user_name="Bob",
+        joined_at=datetime.now(timezone.utc),
+        left_at=None,
+    )
+    mock_sessions_ctrl.get_active_by_meeting = AsyncMock(return_value=[remaining])
+    mock_meetings_ctrl.update_meeting = AsyncMock()
+
+    await _handle_participant_left(participant_left_event)
+
+    # Session closed
+    mock_sessions_ctrl.batch_close_sessions.assert_called_once()
+    closed_ids = mock_sessions_ctrl.batch_close_sessions.call_args.args[0]
+    assert closed_ids == [existing_session.id]
+
+    # num_clients updated to remaining count
+    mock_meetings_ctrl.update_meeting.assert_called_once_with(
+        mock_meeting.id, num_clients=1
+    )
+
+    # Delayed reconciliation poll queued
+    mock_poll_task.apply_async.assert_called_once()
+    call_kwargs = mock_poll_task.apply_async.call_args.kwargs
+    assert call_kwargs["countdown"] == 5
+    assert call_kwargs["args"] == [mock_meeting.id]
+
+
+@pytest.mark.asyncio
+@patch("reflector.views.daily.poll_daily_room_presence_task")
+@patch("reflector.views.daily.meetings_controller")
+@patch("reflector.views.daily.daily_participant_sessions_controller")
+async def test_handles_missing_session(
+    mock_sessions_ctrl,
+    mock_meetings_ctrl,
+    mock_poll_task,
+    mock_meeting,
+    participant_left_event,
+):
+    """No crash when session not found in DB — still queues reconciliation poll."""
+    mock_meetings_ctrl.get_by_room_name = AsyncMock(return_value=mock_meeting)
+    mock_sessions_ctrl.get_open_session = AsyncMock(return_value=None)
+
+    await _handle_participant_left(participant_left_event)
+
+    # No session close attempted
+    mock_sessions_ctrl.batch_close_sessions.assert_not_called()
+    # No num_clients update (no authoritative data without session)
+    mock_meetings_ctrl.update_meeting.assert_not_called()
+    # Still queues reconciliation poll
+    mock_poll_task.apply_async.assert_called_once()
+
+
+@pytest.mark.asyncio
+@patch("reflector.views.daily.poll_daily_room_presence_task")
+@patch("reflector.views.daily.meetings_controller")
+@patch("reflector.views.daily.daily_participant_sessions_controller")
+async def test_updates_num_clients_to_zero_when_last_participant_leaves(
+    mock_sessions_ctrl,
+    mock_meetings_ctrl,
+    mock_poll_task,
+    mock_meeting,
+    participant_left_event,
+    existing_session,
+):
+    """num_clients set to 0 when no active sessions remain."""
+    mock_meetings_ctrl.get_by_room_name = AsyncMock(return_value=mock_meeting)
+    mock_sessions_ctrl.get_open_session = AsyncMock(return_value=existing_session)
+    mock_sessions_ctrl.batch_close_sessions = AsyncMock()
+    mock_sessions_ctrl.get_active_by_meeting = AsyncMock(return_value=[])
+    mock_meetings_ctrl.update_meeting = AsyncMock()
+
+    await _handle_participant_left(participant_left_event)
+
+    mock_meetings_ctrl.update_meeting.assert_called_once_with(
+        mock_meeting.id, num_clients=0
+    )
+
+
+@pytest.mark.asyncio
+@patch("reflector.views.daily.poll_daily_room_presence_task")
+@patch("reflector.views.daily.meetings_controller")
+async def test_no_room_name_in_event(
+    mock_meetings_ctrl,
+    mock_poll_task,
+):
+    """No crash when room_name is missing from webhook payload."""
+    event = ParticipantLeftEvent(
+        version="1.0.0",
+        type="participant.left",
+        id="evt-left-no-room",
+        payload=ParticipantLeftPayload(
+            room_name=None,
+            session_id="session-x",
+            user_id="user-x",
+            user_name="X",
+            joined_at=int(datetime.now(timezone.utc).timestamp()),
+            duration=0,
+        ),
+        event_ts=int(datetime.now(timezone.utc).timestamp()),
+    )
+
+    await _handle_participant_left(event)
+
+    mock_meetings_ctrl.get_by_room_name.assert_not_called()
+    mock_poll_task.apply_async.assert_not_called()
+
+
+@pytest.mark.asyncio
+@patch("reflector.views.daily.poll_daily_room_presence_task")
+@patch("reflector.views.daily.meetings_controller")
+async def test_meeting_not_found(
+    mock_meetings_ctrl,
+    mock_poll_task,
+    participant_left_event,
+):
+    """No crash when meeting not found for room_name."""
+    mock_meetings_ctrl.get_by_room_name = AsyncMock(return_value=None)
+
+    await _handle_participant_left(participant_left_event)
+
+    mock_poll_task.apply_async.assert_not_called()

server/tests/test_direct_session_management.py

@@ -0,0 +1,339 @@
+"""Tests for direct session management via /joined and /leave endpoints.
+
+Verifies that:
+1. /joined with session_id creates session directly, updates num_clients
+2. /joined without session_id (backward compat) still works, queues poll
+3. /leave with session_id closes session, updates num_clients
+4. /leave without session_id falls back to poll
+5. Duplicate /joined calls are idempotent (upsert)
+6. /leave for already-closed session is a no-op
+"""
+
+from datetime import datetime, timedelta, timezone
+from unittest.mock import AsyncMock, MagicMock, patch
+
+import pytest
+
+from reflector.db.daily_participant_sessions import DailyParticipantSession
+from reflector.db.meetings import Meeting
+from reflector.views.rooms import (
+    JoinedRequest,
+    meeting_joined,
+    meeting_leave,
+)
+
+
+@pytest.fixture
+def mock_room():
+    room = MagicMock()
+    room.id = "room-456"
+    room.name = "test-room"
+    room.platform = "daily"
+    return room
+
+
+@pytest.fixture
+def mock_meeting():
+    return Meeting(
+        id="meeting-123",
+        room_id="room-456",
+        room_name="test-room-20251118120000",
+        room_url="https://daily.co/test-room",
+        host_room_url="https://daily.co/test-room?t=host",
+        platform="daily",
+        num_clients=0,
+        is_active=True,
+        start_date=datetime.now(timezone.utc),
+        end_date=datetime.now(timezone.utc) + timedelta(hours=8),
+    )
+
+
+@pytest.fixture
+def mock_redis():
+    redis = AsyncMock()
+    redis.aclose = AsyncMock()
+    return redis
+
+
+@pytest.fixture
+def mock_request_with_session_id():
+    """Mock Request with session_id in JSON body."""
+    request = AsyncMock()
+    request.body = AsyncMock(return_value=b'{"session_id": "session-abc"}')
+    return request
+
+
+@pytest.fixture
+def mock_request_empty_body():
+    """Mock Request with empty JSON body (old frontend / no frame access)."""
+    request = AsyncMock()
+    request.body = AsyncMock(return_value=b"{}")
+    return request
+
+
+@pytest.mark.asyncio
+@patch("reflector.views.rooms.poll_daily_room_presence_task")
+@patch("reflector.views.rooms.delete_pending_join")
+@patch("reflector.views.rooms.get_async_redis_client")
+@patch("reflector.views.rooms.meetings_controller")
+@patch("reflector.views.rooms.rooms_controller")
+@patch("reflector.views.rooms.daily_participant_sessions_controller")
+async def test_joined_with_session_id_creates_session(
+    mock_sessions_ctrl,
+    mock_rooms_ctrl,
+    mock_meetings_ctrl,
+    mock_redis_client,
+    mock_delete_pending,
+    mock_poll_task,
+    mock_room,
+    mock_meeting,
+    mock_redis,
+):
+    """session_id in /joined -> create session + update num_clients."""
+    mock_rooms_ctrl.get_by_name = AsyncMock(return_value=mock_room)
+    mock_meetings_ctrl.get_by_id = AsyncMock(return_value=mock_meeting)
+    mock_redis_client.return_value = mock_redis
+    mock_sessions_ctrl.batch_upsert_sessions = AsyncMock()
+    mock_sessions_ctrl.get_active_by_meeting = AsyncMock(
+        return_value=[MagicMock()]  # 1 active session
+    )
+    mock_meetings_ctrl.update_meeting = AsyncMock()
+
+    body = JoinedRequest(
+        connection_id="conn-1",
+        session_id="session-abc",
+        user_name="Alice",
+    )
+    result = await meeting_joined(
+        "test-room", "meeting-123", body, user={"sub": "user-1"}
+    )
+
+    assert result.status == "ok"
+
+    # Session created via upsert
+    mock_sessions_ctrl.batch_upsert_sessions.assert_called_once()
+    sessions = mock_sessions_ctrl.batch_upsert_sessions.call_args.args[0]
+    assert len(sessions) == 1
+    assert sessions[0].session_id == "session-abc"
+    assert sessions[0].meeting_id == "meeting-123"
+    assert sessions[0].room_id == "room-456"
+    assert sessions[0].user_name == "Alice"
+    assert sessions[0].user_id == "user-1"
+    assert sessions[0].id == "meeting-123:session-abc"
+
+    # num_clients updated
+    mock_meetings_ctrl.update_meeting.assert_called_once_with(
+        "meeting-123", num_clients=1
+    )
+
+    # Reconciliation poll still queued
+    mock_poll_task.apply_async.assert_called_once()
+
+
+@pytest.mark.asyncio
+@patch("reflector.views.rooms.poll_daily_room_presence_task")
+@patch("reflector.views.rooms.delete_pending_join")
+@patch("reflector.views.rooms.get_async_redis_client")
+@patch("reflector.views.rooms.meetings_controller")
+@patch("reflector.views.rooms.rooms_controller")
+@patch("reflector.views.rooms.daily_participant_sessions_controller")
+async def test_joined_without_session_id_backward_compat(
+    mock_sessions_ctrl,
+    mock_rooms_ctrl,
+    mock_meetings_ctrl,
+    mock_redis_client,
+    mock_delete_pending,
+    mock_poll_task,
+    mock_room,
+    mock_meeting,
+    mock_redis,
+):
+    """No session_id in /joined -> no session create, still queues poll."""
+    mock_rooms_ctrl.get_by_name = AsyncMock(return_value=mock_room)
+    mock_meetings_ctrl.get_by_id = AsyncMock(return_value=mock_meeting)
+    mock_redis_client.return_value = mock_redis
+
+    body = JoinedRequest(connection_id="conn-1")
+    result = await meeting_joined(
+        "test-room", "meeting-123", body, user={"sub": "user-1"}
+    )
+
+    assert result.status == "ok"
+    mock_sessions_ctrl.batch_upsert_sessions.assert_not_called()
+    mock_poll_task.apply_async.assert_called_once()
+
+
+@pytest.mark.asyncio
+@patch("reflector.views.rooms.poll_daily_room_presence_task")
+@patch("reflector.views.rooms.delete_pending_join")
+@patch("reflector.views.rooms.get_async_redis_client")
+@patch("reflector.views.rooms.meetings_controller")
+@patch("reflector.views.rooms.rooms_controller")
+@patch("reflector.views.rooms.daily_participant_sessions_controller")
+async def test_joined_anonymous_user_sets_null_user_id(
+    mock_sessions_ctrl,
+    mock_rooms_ctrl,
+    mock_meetings_ctrl,
+    mock_redis_client,
+    mock_delete_pending,
+    mock_poll_task,
+    mock_room,
+    mock_meeting,
+    mock_redis,
+):
+    """Anonymous user -> session.user_id is None, user_name defaults to 'Anonymous'."""
+    mock_rooms_ctrl.get_by_name = AsyncMock(return_value=mock_room)
+    mock_meetings_ctrl.get_by_id = AsyncMock(return_value=mock_meeting)
+    mock_redis_client.return_value = mock_redis
+    mock_sessions_ctrl.batch_upsert_sessions = AsyncMock()
+    mock_sessions_ctrl.get_active_by_meeting = AsyncMock(return_value=[MagicMock()])
+    mock_meetings_ctrl.update_meeting = AsyncMock()
+
+    body = JoinedRequest(connection_id="conn-1", session_id="session-abc")
+    result = await meeting_joined("test-room", "meeting-123", body, user=None)
+
+    assert result.status == "ok"
+    sessions = mock_sessions_ctrl.batch_upsert_sessions.call_args.args[0]
+    assert sessions[0].user_id is None
+    assert sessions[0].user_name == "Anonymous"
+
+
+@pytest.mark.asyncio
+@patch("reflector.views.rooms.poll_daily_room_presence_task")
+@patch("reflector.views.rooms.meetings_controller")
+@patch("reflector.views.rooms.rooms_controller")
+@patch("reflector.views.rooms.daily_participant_sessions_controller")
+async def test_leave_with_session_id_closes_session(
+    mock_sessions_ctrl,
+    mock_rooms_ctrl,
+    mock_meetings_ctrl,
+    mock_poll_task,
+    mock_room,
+    mock_meeting,
+    mock_request_with_session_id,
+):
+    """session_id in /leave -> close session + update num_clients."""
+    mock_rooms_ctrl.get_by_name = AsyncMock(return_value=mock_room)
+    mock_meetings_ctrl.get_by_id = AsyncMock(return_value=mock_meeting)
+
+    existing_session = DailyParticipantSession(
+        id="meeting-123:session-abc",
+        meeting_id="meeting-123",
+        room_id="room-456",
+        session_id="session-abc",
+        user_id="user-1",
+        user_name="Alice",
+        joined_at=datetime.now(timezone.utc) - timedelta(minutes=5),
+        left_at=None,
+    )
+    mock_sessions_ctrl.get_open_session = AsyncMock(return_value=existing_session)
+    mock_sessions_ctrl.batch_close_sessions = AsyncMock()
+    mock_sessions_ctrl.get_active_by_meeting = AsyncMock(return_value=[])
+    mock_meetings_ctrl.update_meeting = AsyncMock()
+
+    result = await meeting_leave(
+        "test-room", "meeting-123", mock_request_with_session_id, user={"sub": "user-1"}
+    )
+
+    assert result.status == "ok"
+
+    # Session closed
+    mock_sessions_ctrl.batch_close_sessions.assert_called_once()
+    closed_ids = mock_sessions_ctrl.batch_close_sessions.call_args.args[0]
+    assert closed_ids == ["meeting-123:session-abc"]
+
+    # num_clients updated
+    mock_meetings_ctrl.update_meeting.assert_called_once_with(
+        "meeting-123", num_clients=0
+    )
+
+    # No poll — direct close is authoritative, poll would race with API latency
+    mock_poll_task.apply_async.assert_not_called()
+
+
+@pytest.mark.asyncio
+@patch("reflector.views.rooms.poll_daily_room_presence_task")
+@patch("reflector.views.rooms.meetings_controller")
+@patch("reflector.views.rooms.rooms_controller")
+async def test_leave_without_session_id_falls_back_to_poll(
+    mock_rooms_ctrl,
+    mock_meetings_ctrl,
+    mock_poll_task,
+    mock_room,
+    mock_meeting,
+    mock_request_empty_body,
+):
+    """No session_id in /leave -> just queues poll as before."""
+    mock_rooms_ctrl.get_by_name = AsyncMock(return_value=mock_room)
+    mock_meetings_ctrl.get_by_id = AsyncMock(return_value=mock_meeting)
+
+    result = await meeting_leave(
+        "test-room", "meeting-123", mock_request_empty_body, user=None
+    )
+
+    assert result.status == "ok"
+    mock_poll_task.apply_async.assert_called_once()
+
+
+@pytest.mark.asyncio
+@patch("reflector.views.rooms.poll_daily_room_presence_task")
+@patch("reflector.views.rooms.delete_pending_join")
+@patch("reflector.views.rooms.get_async_redis_client")
+@patch("reflector.views.rooms.meetings_controller")
+@patch("reflector.views.rooms.rooms_controller")
+@patch("reflector.views.rooms.daily_participant_sessions_controller")
+async def test_duplicate_joined_is_idempotent(
+    mock_sessions_ctrl,
+    mock_rooms_ctrl,
+    mock_meetings_ctrl,
+    mock_redis_client,
+    mock_delete_pending,
+    mock_poll_task,
+    mock_room,
+    mock_meeting,
+    mock_redis,
+):
+    """Calling /joined twice with same session_id -> upsert both times, no error."""
+    mock_rooms_ctrl.get_by_name = AsyncMock(return_value=mock_room)
+    mock_meetings_ctrl.get_by_id = AsyncMock(return_value=mock_meeting)
+    mock_redis_client.return_value = mock_redis
+    mock_sessions_ctrl.batch_upsert_sessions = AsyncMock()
+    mock_sessions_ctrl.get_active_by_meeting = AsyncMock(return_value=[MagicMock()])
+    mock_meetings_ctrl.update_meeting = AsyncMock()
+
+    body = JoinedRequest(
+        connection_id="conn-1", session_id="session-abc", user_name="Alice"
+    )
+    await meeting_joined("test-room", "meeting-123", body, user={"sub": "user-1"})
+    await meeting_joined("test-room", "meeting-123", body, user={"sub": "user-1"})
+
+    assert mock_sessions_ctrl.batch_upsert_sessions.call_count == 2
+
+
+@pytest.mark.asyncio
+@patch("reflector.views.rooms.poll_daily_room_presence_task")
+@patch("reflector.views.rooms.meetings_controller")
+@patch("reflector.views.rooms.rooms_controller")
+@patch("reflector.views.rooms.daily_participant_sessions_controller")
+async def test_leave_already_closed_session_is_noop(
+    mock_sessions_ctrl,
+    mock_rooms_ctrl,
+    mock_meetings_ctrl,
+    mock_poll_task,
+    mock_room,
+    mock_meeting,
+    mock_request_with_session_id,
+):
+    """/leave for already-closed session -> no close attempted, just poll."""
+    mock_rooms_ctrl.get_by_name = AsyncMock(return_value=mock_room)
+    mock_meetings_ctrl.get_by_id = AsyncMock(return_value=mock_meeting)
+    mock_sessions_ctrl.get_open_session = AsyncMock(return_value=None)
+
+    result = await meeting_leave(
+        "test-room", "meeting-123", mock_request_with_session_id, user=None
+    )
+
+    assert result.status == "ok"
+    mock_sessions_ctrl.batch_close_sessions.assert_not_called()
+    mock_meetings_ctrl.update_meeting.assert_not_called()
+    mock_poll_task.apply_async.assert_called_once()

server/tests/test_ics_dedup.py

@@ -1,190 +0,0 @@
-from datetime import datetime, timedelta, timezone
-from unittest.mock import AsyncMock, patch
-
-import pytest
-
-from reflector.db import get_database
-from reflector.db.calendar_events import CalendarEvent, calendar_events_controller
-from reflector.db.meetings import meetings
-from reflector.db.rooms import rooms_controller
-from reflector.worker.ics_sync import create_upcoming_meetings_for_event
-
-
-@pytest.mark.asyncio
-async def test_duplicate_calendar_event_does_not_create_duplicate_meeting():
-    """When an aggregated ICS feed contains the same event with different UIDs
-    (e.g. Cal.com UID + Google Calendar UUID), only one meeting should be created."""
-
-    room = await rooms_controller.add(
-        name="dedup-test-room",
-        user_id="test-user",
-        zulip_auto_post=False,
-        zulip_stream="",
-        zulip_topic="",
-        is_locked=False,
-        room_mode="normal",
-        recording_type="cloud",
-        recording_trigger="automatic-2nd-participant",
-        is_shared=False,
-        ics_url="https://calendar.example.com/dedup.ics",
-        ics_enabled=True,
-    )
-
-    now = datetime.now(timezone.utc)
-    start_time = now + timedelta(hours=1)
-    end_time = now + timedelta(hours=2)
-
-    # Create first calendar event (Cal.com UID)
-    event1 = await calendar_events_controller.upsert(
-        CalendarEvent(
-            room_id=room.id,
-            ics_uid="abc123@Cal.com",
-            title="Team Standup",
-            start_time=start_time,
-            end_time=end_time,
-        )
-    )
-
-    # create_window must be before start_time for the function to proceed
-    create_window = now - timedelta(minutes=6)
-
-    # Create meeting for event1
-    with patch(
-        "reflector.worker.ics_sync.create_platform_client"
-    ) as mock_platform_factory:
-        mock_client = AsyncMock()
-
-        async def mock_create_meeting_1(room_name_prefix, *, end_date, room):
-            return AsyncMock(
-                meeting_id="meeting-1",
-                room_name="dedup-test-room-abc",
-                room_url="https://mock.video/dedup-test-room-abc",
-                host_room_url="https://mock.video/dedup-test-room-abc?host=true",
-            )
-
-        mock_client.create_meeting = mock_create_meeting_1
-        mock_client.upload_logo = AsyncMock()
-        mock_platform_factory.return_value = mock_client
-
-        await create_upcoming_meetings_for_event(event1, create_window, room)
-
-    # Verify meeting was created
-    results = await get_database().fetch_all(
-        meetings.select().where(meetings.c.room_id == room.id)
-    )
-    assert len(results) == 1, f"Expected 1 meeting, got {len(results)}"
-
-    # Create second calendar event with different UID but same time window (Google Calendar UUID)
-    event2 = await calendar_events_controller.upsert(
-        CalendarEvent(
-            room_id=room.id,
-            ics_uid="550e8400-e29b-41d4-a716-446655440000",
-            title="Team Standup",
-            start_time=start_time,
-            end_time=end_time,
-        )
-    )
-
-    # Try to create meeting for event2 - should be skipped due to dedup
-    with patch(
-        "reflector.worker.ics_sync.create_platform_client"
-    ) as mock_platform_factory:
-        mock_client = AsyncMock()
-        create_meeting_called = False
-
-        async def mock_create_meeting_2(room_name_prefix, *, end_date, room):
-            nonlocal create_meeting_called
-            create_meeting_called = True
-
-        mock_client.create_meeting = mock_create_meeting_2
-        mock_client.upload_logo = AsyncMock()
-        mock_platform_factory.return_value = mock_client
-
-        await create_upcoming_meetings_for_event(event2, create_window, room)
-
-        # Platform client should NOT have been called for the duplicate
-        assert (
-            not create_meeting_called
-        ), "create_meeting should not be called for duplicate"
-
-    # Verify still only 1 meeting
-    results = await get_database().fetch_all(
-        meetings.select().where(meetings.c.room_id == room.id)
-    )
-    assert len(results) == 1, f"Expected 1 meeting after dedup, got {len(results)}"
-
-
-@pytest.mark.asyncio
-async def test_different_time_windows_create_separate_meetings():
-    """Events at different times should create separate meetings, even if titles match."""
-
-    room = await rooms_controller.add(
-        name="dedup-diff-time-room",
-        user_id="test-user",
-        zulip_auto_post=False,
-        zulip_stream="",
-        zulip_topic="",
-        is_locked=False,
-        room_mode="normal",
-        recording_type="cloud",
-        recording_trigger="automatic-2nd-participant",
-        is_shared=False,
-        ics_url="https://calendar.example.com/dedup2.ics",
-        ics_enabled=True,
-    )
-
-    now = datetime.now(timezone.utc)
-    create_window = now - timedelta(minutes=6)
-
-    # Event 1: 1-2pm
-    event1 = await calendar_events_controller.upsert(
-        CalendarEvent(
-            room_id=room.id,
-            ics_uid="event-morning@Cal.com",
-            title="Team Standup",
-            start_time=now + timedelta(hours=1),
-            end_time=now + timedelta(hours=2),
-        )
-    )
-
-    # Event 2: 3-4pm (different time)
-    event2 = await calendar_events_controller.upsert(
-        CalendarEvent(
-            room_id=room.id,
-            ics_uid="event-afternoon@Cal.com",
-            title="Team Standup",
-            start_time=now + timedelta(hours=3),
-            end_time=now + timedelta(hours=4),
-        )
-    )
-
-    with patch(
-        "reflector.worker.ics_sync.create_platform_client"
-    ) as mock_platform_factory:
-        mock_client = AsyncMock()
-
-        call_count = 0
-
-        async def mock_create_meeting(room_name_prefix, *, end_date, room):
-            nonlocal call_count
-            call_count += 1
-            return AsyncMock(
-                meeting_id=f"meeting-{call_count}",
-                room_name=f"dedup-diff-time-room-{call_count}",
-                room_url=f"https://mock.video/dedup-diff-time-room-{call_count}",
-                host_room_url=f"https://mock.video/dedup-diff-time-room-{call_count}?host=true",
-            )
-
-        mock_client.create_meeting = mock_create_meeting
-        mock_client.upload_logo = AsyncMock()
-        mock_platform_factory.return_value = mock_client
-
-        await create_upcoming_meetings_for_event(event1, create_window, room)
-        await create_upcoming_meetings_for_event(event2, create_window, room)
-
-    results = await get_database().fetch_all(
-        meetings.select().where(meetings.c.room_id == room.id)
-    )
-    assert (
-        len(results) == 2
-    ), f"Expected 2 meetings for different times, got {len(results)}"

server/tests/test_joining_endpoint.py

@@ -0,0 +1,367 @@
+"""Integration tests for /joining and /joined endpoints.
+
+Tests for the join intent tracking to prevent race conditions during
+WebRTC handshake when users join meetings.
+"""
+
+from datetime import datetime, timedelta, timezone
+from unittest.mock import AsyncMock, patch
+
+import pytest
+
+from reflector.db.meetings import Meeting
+from reflector.presence.pending_joins import PENDING_JOIN_PREFIX
+
+TEST_CONNECTION_ID = "test-connection-uuid-12345"
+
+
+@pytest.fixture
+def mock_room():
+    """Mock room object."""
+    from reflector.db.rooms import Room
+
+    return Room(
+        id="room-123",
+        name="test-room",
+        user_id="owner-user",
+        created_at=datetime.now(timezone.utc),
+        zulip_auto_post=False,
+        zulip_stream="",
+        zulip_topic="",
+        is_locked=False,
+        room_mode="normal",
+        recording_type="cloud",
+        recording_trigger="automatic",
+        is_shared=True,
+        platform="daily",
+        skip_consent=False,
+    )
+
+
+@pytest.fixture
+def mock_meeting():
+    """Mock meeting object."""
+    now = datetime.now(timezone.utc)
+    return Meeting(
+        id="meeting-456",
+        room_id="room-123",
+        room_name="test-room-20251118120000",
+        room_url="https://daily.co/test-room-20251118120000",
+        host_room_url="https://daily.co/test-room-20251118120000?t=host",
+        platform="daily",
+        num_clients=0,
+        is_active=True,
+        start_date=now,
+        end_date=now + timedelta(hours=1),
+    )
+
+
+@pytest.mark.asyncio
+@patch("reflector.views.rooms.rooms_controller.get_by_name")
+@patch("reflector.views.rooms.meetings_controller.get_by_id")
+@patch("reflector.views.rooms.get_async_redis_client")
+async def test_joining_endpoint_creates_pending_join(
+    mock_get_redis,
+    mock_get_meeting,
+    mock_get_room,
+    mock_room,
+    mock_meeting,
+    client,
+    authenticated_client,
+):
+    """Test that /joining endpoint creates pending join in Redis."""
+    mock_get_room.return_value = mock_room
+    mock_get_meeting.return_value = mock_meeting
+
+    mock_redis = AsyncMock()
+    mock_redis.setex = AsyncMock()
+    mock_redis.aclose = AsyncMock()
+    mock_get_redis.return_value = mock_redis
+
+    response = await client.post(
+        f"/rooms/{mock_room.name}/meetings/{mock_meeting.id}/joining",
+        json={"connection_id": TEST_CONNECTION_ID},
+    )
+
+    assert response.status_code == 200
+    assert response.json() == {"status": "ok"}
+
+    # Verify Redis setex was called with correct key pattern
+    mock_redis.setex.assert_called_once()
+    call_args = mock_redis.setex.call_args[0]
+    assert call_args[0].startswith(f"{PENDING_JOIN_PREFIX}:{mock_meeting.id}:")
+    assert TEST_CONNECTION_ID in call_args[0]
+
+
+@pytest.mark.asyncio
+@patch("reflector.views.rooms.poll_daily_room_presence_task")
+@patch("reflector.views.rooms.rooms_controller.get_by_name")
+@patch("reflector.views.rooms.meetings_controller.get_by_id")
+@patch("reflector.views.rooms.get_async_redis_client")
+async def test_joined_endpoint_deletes_pending_join(
+    mock_get_redis,
+    mock_get_meeting,
+    mock_get_room,
+    mock_poll_task,
+    mock_room,
+    mock_meeting,
+    client,
+    authenticated_client,
+):
+    """Test that /joined endpoint deletes pending join from Redis."""
+    mock_get_room.return_value = mock_room
+    mock_get_meeting.return_value = mock_meeting
+
+    mock_redis = AsyncMock()
+    mock_redis.delete = AsyncMock()
+    mock_redis.aclose = AsyncMock()
+    mock_get_redis.return_value = mock_redis
+
+    response = await client.post(
+        f"/rooms/{mock_room.name}/meetings/{mock_meeting.id}/joined",
+        json={"connection_id": TEST_CONNECTION_ID},
+    )
+
+    assert response.status_code == 200
+    assert response.json() == {"status": "ok"}
+
+    # Verify Redis delete was called with correct key pattern
+    mock_redis.delete.assert_called_once()
+    call_args = mock_redis.delete.call_args[0]
+    assert call_args[0].startswith(f"{PENDING_JOIN_PREFIX}:{mock_meeting.id}:")
+    assert TEST_CONNECTION_ID in call_args[0]
+
+    # Verify presence poll was triggered for Daily meetings
+    mock_poll_task.delay.assert_called_once_with(mock_meeting.id)
+
+
+@pytest.mark.asyncio
+@patch("reflector.views.rooms.rooms_controller.get_by_name")
+async def test_joining_endpoint_room_not_found(
+    mock_get_room,
+    client,
+    authenticated_client,
+):
+    """Test that /joining returns 404 when room not found."""
+    mock_get_room.return_value = None
+
+    response = await client.post(
+        "/rooms/nonexistent-room/meetings/meeting-123/joining",
+        json={"connection_id": TEST_CONNECTION_ID},
+    )
+
+    assert response.status_code == 404
+    assert response.json()["detail"] == "Room not found"
+
+
+@pytest.mark.asyncio
+@patch("reflector.views.rooms.rooms_controller.get_by_name")
+@patch("reflector.views.rooms.meetings_controller.get_by_id")
+async def test_joining_endpoint_meeting_not_found(
+    mock_get_meeting,
+    mock_get_room,
+    mock_room,
+    client,
+    authenticated_client,
+):
+    """Test that /joining returns 404 when meeting not found."""
+    mock_get_room.return_value = mock_room
+    mock_get_meeting.return_value = None
+
+    response = await client.post(
+        f"/rooms/{mock_room.name}/meetings/nonexistent-meeting/joining",
+        json={"connection_id": TEST_CONNECTION_ID},
+    )
+
+    assert response.status_code == 404
+    assert response.json()["detail"] == "Meeting not found"
+
+
+@pytest.mark.asyncio
+@patch("reflector.views.rooms.rooms_controller.get_by_name")
+@patch("reflector.views.rooms.meetings_controller.get_by_id")
+async def test_joining_endpoint_meeting_not_active(
+    mock_get_meeting,
+    mock_get_room,
+    mock_room,
+    mock_meeting,
+    client,
+    authenticated_client,
+):
+    """Test that /joining returns 400 when meeting is not active."""
+    mock_get_room.return_value = mock_room
+    inactive_meeting = mock_meeting.model_copy(update={"is_active": False})
+    mock_get_meeting.return_value = inactive_meeting
+
+    response = await client.post(
+        f"/rooms/{mock_room.name}/meetings/{mock_meeting.id}/joining",
+        json={"connection_id": TEST_CONNECTION_ID},
+    )
+
+    assert response.status_code == 400
+    assert response.json()["detail"] == "Meeting is not active"
+
+
+@pytest.mark.asyncio
+@patch("reflector.views.rooms.rooms_controller.get_by_name")
+@patch("reflector.views.rooms.meetings_controller.get_by_id")
+@patch("reflector.views.rooms.get_async_redis_client")
+async def test_joining_endpoint_anonymous_user(
+    mock_get_redis,
+    mock_get_meeting,
+    mock_get_room,
+    mock_room,
+    mock_meeting,
+    client,
+):
+    """Test that /joining works for anonymous users with unique connection_id."""
+    mock_get_room.return_value = mock_room
+    mock_get_meeting.return_value = mock_meeting
+
+    mock_redis = AsyncMock()
+    mock_redis.setex = AsyncMock()
+    mock_redis.aclose = AsyncMock()
+    mock_get_redis.return_value = mock_redis
+
+    response = await client.post(
+        f"/rooms/{mock_room.name}/meetings/{mock_meeting.id}/joining",
+        json={"connection_id": TEST_CONNECTION_ID},
+    )
+
+    assert response.status_code == 200
+    assert response.json() == {"status": "ok"}
+
+    # Verify Redis setex was called with "anon:" prefix and connection_id
+    call_args = mock_redis.setex.call_args[0]
+    assert ":anon:" in call_args[0]
+    assert TEST_CONNECTION_ID in call_args[0]
+
+
+@pytest.mark.asyncio
+@patch("reflector.views.rooms.rooms_controller.get_by_name")
+@patch("reflector.views.rooms.meetings_controller.get_by_id")
+@patch("reflector.views.rooms.get_async_redis_client")
+async def test_joining_endpoint_redis_closed_on_success(
+    mock_get_redis,
+    mock_get_meeting,
+    mock_get_room,
+    mock_room,
+    mock_meeting,
+    client,
+    authenticated_client,
+):
+    """Test that Redis connection is closed after successful operation."""
+    mock_get_room.return_value = mock_room
+    mock_get_meeting.return_value = mock_meeting
+
+    mock_redis = AsyncMock()
+    mock_redis.setex = AsyncMock()
+    mock_redis.aclose = AsyncMock()
+    mock_get_redis.return_value = mock_redis
+
+    await client.post(
+        f"/rooms/{mock_room.name}/meetings/{mock_meeting.id}/joining",
+        json={"connection_id": TEST_CONNECTION_ID},
+    )
+
+    mock_redis.aclose.assert_called_once()
+
+
+@pytest.mark.asyncio
+@patch("reflector.views.rooms.rooms_controller.get_by_name")
+@patch("reflector.views.rooms.meetings_controller.get_by_id")
+@patch("reflector.views.rooms.get_async_redis_client")
+async def test_joining_endpoint_redis_closed_on_error(
+    mock_get_redis,
+    mock_get_meeting,
+    mock_get_room,
+    mock_room,
+    mock_meeting,
+    client,
+    authenticated_client,
+):
+    """Test that Redis connection is closed even when operation fails."""
+    mock_get_room.return_value = mock_room
+    mock_get_meeting.return_value = mock_meeting
+
+    mock_redis = AsyncMock()
+    mock_redis.setex = AsyncMock(side_effect=Exception("Redis error"))
+    mock_redis.aclose = AsyncMock()
+    mock_get_redis.return_value = mock_redis
+
+    with pytest.raises(Exception):
+        await client.post(
+            f"/rooms/{mock_room.name}/meetings/{mock_meeting.id}/joining",
+            json={"connection_id": TEST_CONNECTION_ID},
+        )
+
+    mock_redis.aclose.assert_called_once()
+
+
+@pytest.mark.asyncio
+async def test_joining_endpoint_requires_connection_id(
+    client,
+):
+    """Test that /joining returns 422 when connection_id is missing."""
+    response = await client.post(
+        "/rooms/test-room/meetings/meeting-123/joining",
+        json={},
+    )
+
+    assert response.status_code == 422  # Validation error
+
+
+@pytest.mark.asyncio
+async def test_joining_endpoint_rejects_empty_connection_id(
+    client,
+):
+    """Test that /joining returns 422 when connection_id is empty string."""
+    response = await client.post(
+        "/rooms/test-room/meetings/meeting-123/joining",
+        json={"connection_id": ""},
+    )
+
+    assert response.status_code == 422  # Validation error (NonEmptyString)
+
+
+@pytest.mark.asyncio
+@patch("reflector.views.rooms.rooms_controller.get_by_name")
+@patch("reflector.views.rooms.meetings_controller.get_by_id")
+@patch("reflector.views.rooms.get_async_redis_client")
+async def test_different_connection_ids_create_different_keys(
+    mock_get_redis,
+    mock_get_meeting,
+    mock_get_room,
+    mock_room,
+    mock_meeting,
+    client,
+):
+    """Test that different connection_ids create different Redis keys."""
+    mock_get_room.return_value = mock_room
+    mock_get_meeting.return_value = mock_meeting
+
+    mock_redis = AsyncMock()
+    mock_redis.setex = AsyncMock()
+    mock_redis.aclose = AsyncMock()
+    mock_get_redis.return_value = mock_redis
+
+    # First connection
+    await client.post(
+        f"/rooms/{mock_room.name}/meetings/{mock_meeting.id}/joining",
+        json={"connection_id": "connection-1"},
+    )
+    key1 = mock_redis.setex.call_args[0][0]
+
+    mock_redis.setex.reset_mock()
+
+    # Second connection (different tab)
+    await client.post(
+        f"/rooms/{mock_room.name}/meetings/{mock_meeting.id}/joining",
+        json={"connection_id": "connection-2"},
+    )
+    key2 = mock_redis.setex.call_args[0][0]
+
+    # Keys should be different
+    assert key1 != key2
+    assert "connection-1" in key1
+    assert "connection-2" in key2

server/tests/test_pending_joins.py

@@ -0,0 +1,153 @@
+"""Tests for pending joins Redis helper functions.
+
+TDD tests for tracking join intent to prevent race conditions during
+WebRTC handshake when users join meetings.
+"""
+
+from unittest.mock import AsyncMock
+
+import pytest
+
+from reflector.presence.pending_joins import (
+    PENDING_JOIN_PREFIX,
+    PENDING_JOIN_TTL,
+    create_pending_join,
+    delete_pending_join,
+    has_pending_joins,
+)
+
+
+@pytest.fixture
+def mock_redis():
+    """Mock async Redis client."""
+    redis = AsyncMock()
+    redis.setex = AsyncMock()
+    redis.delete = AsyncMock()
+    redis.scan = AsyncMock(return_value=(0, []))
+    return redis
+
+
+@pytest.mark.asyncio
+async def test_create_pending_join_sets_key_with_ttl(mock_redis):
+    """Test that create_pending_join stores key with correct TTL."""
+    meeting_id = "meeting-123"
+    user_id = "user-456"
+
+    await create_pending_join(mock_redis, meeting_id, user_id)
+
+    expected_key = f"{PENDING_JOIN_PREFIX}:{meeting_id}:{user_id}"
+    mock_redis.setex.assert_called_once()
+    call_args = mock_redis.setex.call_args
+    assert call_args[0][0] == expected_key
+    assert call_args[0][1] == PENDING_JOIN_TTL
+    # Value should be a timestamp string
+    assert call_args[0][2] is not None
+
+
+@pytest.mark.asyncio
+async def test_delete_pending_join_removes_key(mock_redis):
+    """Test that delete_pending_join removes the key."""
+    meeting_id = "meeting-123"
+    user_id = "user-456"
+
+    await delete_pending_join(mock_redis, meeting_id, user_id)
+
+    expected_key = f"{PENDING_JOIN_PREFIX}:{meeting_id}:{user_id}"
+    mock_redis.delete.assert_called_once_with(expected_key)
+
+
+@pytest.mark.asyncio
+async def test_has_pending_joins_returns_false_when_no_keys(mock_redis):
+    """Test has_pending_joins returns False when no matching keys."""
+    mock_redis.scan.return_value = (0, [])
+
+    result = await has_pending_joins(mock_redis, "meeting-123")
+
+    assert result is False
+    mock_redis.scan.assert_called_once()
+    call_kwargs = mock_redis.scan.call_args.kwargs
+    assert call_kwargs["match"] == f"{PENDING_JOIN_PREFIX}:meeting-123:*"
+
+
+@pytest.mark.asyncio
+async def test_has_pending_joins_returns_true_when_keys_exist(mock_redis):
+    """Test has_pending_joins returns True when matching keys found."""
+    mock_redis.scan.return_value = (0, [b"pending_join:meeting-123:user-1"])
+
+    result = await has_pending_joins(mock_redis, "meeting-123")
+
+    assert result is True
+
+
+@pytest.mark.asyncio
+async def test_has_pending_joins_scans_with_correct_pattern(mock_redis):
+    """Test has_pending_joins uses correct scan pattern."""
+    meeting_id = "meeting-abc-def"
+    mock_redis.scan.return_value = (0, [])
+
+    await has_pending_joins(mock_redis, meeting_id)
+
+    expected_pattern = f"{PENDING_JOIN_PREFIX}:{meeting_id}:*"
+    mock_redis.scan.assert_called_once()
+    call_kwargs = mock_redis.scan.call_args.kwargs
+    assert call_kwargs["match"] == expected_pattern
+    assert call_kwargs["count"] == 100
+
+
+@pytest.mark.asyncio
+async def test_multiple_users_pending_joins(mock_redis):
+    """Test that multiple users can have pending joins for same meeting."""
+    meeting_id = "meeting-123"
+    # Simulate two pending joins
+    mock_redis.scan.return_value = (
+        0,
+        [b"pending_join:meeting-123:user-1", b"pending_join:meeting-123:user-2"],
+    )
+
+    result = await has_pending_joins(mock_redis, meeting_id)
+
+    assert result is True
+
+
+@pytest.mark.asyncio
+async def test_pending_join_ttl_value():
+    """Test that PENDING_JOIN_TTL has expected value."""
+    # 30 seconds should be enough for WebRTC handshake but not too long
+    assert PENDING_JOIN_TTL == 30
+
+
+@pytest.mark.asyncio
+async def test_pending_join_prefix_value():
+    """Test that PENDING_JOIN_PREFIX has expected value."""
+    assert PENDING_JOIN_PREFIX == "pending_join"
+
+
+@pytest.mark.asyncio
+async def test_has_pending_joins_multi_iteration_scan_no_keys(mock_redis):
+    """Test has_pending_joins iterates until cursor returns 0."""
+    # Simulate multi-iteration scan: cursor 100 -> cursor 50 -> cursor 0
+    mock_redis.scan.side_effect = [
+        (100, []),  # First iteration, no keys, continue
+        (50, []),  # Second iteration, no keys, continue
+        (0, []),  # Third iteration, cursor 0, done
+    ]
+
+    result = await has_pending_joins(mock_redis, "meeting-123")
+
+    assert result is False
+    assert mock_redis.scan.call_count == 3
+
+
+@pytest.mark.asyncio
+async def test_has_pending_joins_multi_iteration_finds_key_later(mock_redis):
+    """Test has_pending_joins finds key on second iteration."""
+    # Simulate finding key on second scan iteration
+    mock_redis.scan.side_effect = [
+        (100, []),  # First iteration, no keys
+        (0, [b"pending_join:meeting-123:user-1"]),  # Second iteration, found key
+    ]
+
+    result = await has_pending_joins(mock_redis, "meeting-123")
+
+    assert result is True
+    assert mock_redis.scan.call_count == 2

server/tests/test_process_meetings_pending_joins.py

@@ -0,0 +1,241 @@
+"""Tests for process_meetings pending joins check.
+
+Tests that process_meetings correctly skips deactivation when
+pending joins exist for a meeting.
+"""
+
+from datetime import datetime, timedelta, timezone
+from unittest.mock import AsyncMock, patch
+
+import pytest
+
+from reflector.db.meetings import Meeting
+
+
+def _get_process_meetings_fn():
+    """Get the underlying async function without Celery/asynctask decorators."""
+    from reflector.worker import process
+
+    fn = process.process_meetings
+    # Get through both decorator layers (@shared_task and @asynctask)
+    if hasattr(fn, "__wrapped__"):
+        fn = fn.__wrapped__
+    if hasattr(fn, "__wrapped__"):
+        fn = fn.__wrapped__
+    return fn
+
+
+@pytest.fixture
+def mock_active_meeting():
+    """Mock an active meeting that should be considered for deactivation."""
+    now = datetime.now(timezone.utc)
+    return Meeting(
+        id="meeting-123",
+        room_id="room-456",
+        room_name="test-room-20251118120000",
+        room_url="https://daily.co/test-room-20251118120000",
+        host_room_url="https://daily.co/test-room-20251118120000?t=host",
+        platform="daily",
+        num_clients=0,
+        is_active=True,
+        start_date=now - timedelta(hours=1),
+        end_date=now - timedelta(minutes=30),  # Already ended
+    )
+
+
+@pytest.mark.asyncio
+@patch("reflector.worker.process.meetings_controller.get_all_active")
+@patch("reflector.worker.process.RedisAsyncLock")
+@patch("reflector.worker.process.create_platform_client")
+@patch("reflector.worker.process.get_async_redis_client")
+@patch("reflector.worker.process.has_pending_joins")
+@patch("reflector.worker.process.meetings_controller.update_meeting")
+async def test_process_meetings_skips_deactivation_with_pending_joins(
+    mock_update_meeting,
+    mock_has_pending_joins,
+    mock_get_redis,
+    mock_create_client,
+    mock_redis_lock_class,
+    mock_get_all_active,
+    mock_active_meeting,
+):
+    """Test that process_meetings skips deactivation when pending joins exist."""
+    process_meetings = _get_process_meetings_fn()
+
+    mock_get_all_active.return_value = [mock_active_meeting]
+
+    # Mock lock acquired
+    mock_lock_instance = AsyncMock()
+    mock_lock_instance.acquired = True
+    mock_lock_instance.__aenter__ = AsyncMock(return_value=mock_lock_instance)
+    mock_lock_instance.__aexit__ = AsyncMock()
+    mock_redis_lock_class.return_value = mock_lock_instance
+
+    # Mock platform client - no active sessions, but had sessions (triggers deactivation)
+    mock_daily_client = AsyncMock()
+    mock_session = AsyncMock()
+    mock_session.ended_at = datetime.now(timezone.utc)  # Session ended
+    mock_daily_client.get_room_sessions = AsyncMock(return_value=[mock_session])
+    mock_create_client.return_value = mock_daily_client
+
+    # Mock Redis client
+    mock_redis = AsyncMock()
+    mock_redis.aclose = AsyncMock()
+    mock_get_redis.return_value = mock_redis
+
+    # Mock pending joins exist
+    mock_has_pending_joins.return_value = True
+
+    await process_meetings()
+
+    # Verify has_pending_joins was called
+    mock_has_pending_joins.assert_called_once_with(mock_redis, mock_active_meeting.id)
+
+    # Verify meeting was NOT deactivated
+    mock_update_meeting.assert_not_called()
+
+    # Verify Redis was closed
+    mock_redis.aclose.assert_called_once()
+
+
+@pytest.mark.asyncio
+@patch("reflector.worker.process.meetings_controller.get_all_active")
+@patch("reflector.worker.process.RedisAsyncLock")
+@patch("reflector.worker.process.create_platform_client")
+@patch("reflector.worker.process.get_async_redis_client")
+@patch("reflector.worker.process.has_pending_joins")
+@patch("reflector.worker.process.meetings_controller.update_meeting")
+async def test_process_meetings_deactivates_without_pending_joins(
+    mock_update_meeting,
+    mock_has_pending_joins,
+    mock_get_redis,
+    mock_create_client,
+    mock_redis_lock_class,
+    mock_get_all_active,
+    mock_active_meeting,
+):
+    """Test that process_meetings deactivates when no pending joins."""
+    process_meetings = _get_process_meetings_fn()
+
+    mock_get_all_active.return_value = [mock_active_meeting]
+
+    # Mock lock acquired
+    mock_lock_instance = AsyncMock()
+    mock_lock_instance.acquired = True
+    mock_lock_instance.__aenter__ = AsyncMock(return_value=mock_lock_instance)
+    mock_lock_instance.__aexit__ = AsyncMock()
+    mock_redis_lock_class.return_value = mock_lock_instance
+
+    # Mock platform client - no active sessions, but had sessions
+    mock_daily_client = AsyncMock()
+    mock_session = AsyncMock()
+    mock_session.ended_at = datetime.now(timezone.utc)
+    mock_daily_client.get_room_sessions = AsyncMock(return_value=[mock_session])
+    mock_create_client.return_value = mock_daily_client
+
+    # Mock Redis client
+    mock_redis = AsyncMock()
+    mock_redis.aclose = AsyncMock()
+    mock_get_redis.return_value = mock_redis
+
+    # Mock no pending joins
+    mock_has_pending_joins.return_value = False
+
+    await process_meetings()
+
+    # Verify meeting was deactivated
+    mock_update_meeting.assert_called_once_with(mock_active_meeting.id, is_active=False)
+
+
+@pytest.mark.asyncio
+@patch("reflector.worker.process.meetings_controller.get_all_active")
+@patch("reflector.worker.process.RedisAsyncLock")
+@patch("reflector.worker.process.create_platform_client")
+async def test_process_meetings_no_check_when_active_sessions(
+    mock_create_client,
+    mock_redis_lock_class,
+    mock_get_all_active,
+    mock_active_meeting,
+):
+    """Test that pending joins check is skipped when there are active sessions."""
+    process_meetings = _get_process_meetings_fn()
+
+    mock_get_all_active.return_value = [mock_active_meeting]
+
+    # Mock lock acquired
+    mock_lock_instance = AsyncMock()
+    mock_lock_instance.acquired = True
+    mock_lock_instance.__aenter__ = AsyncMock(return_value=mock_lock_instance)
+    mock_lock_instance.__aexit__ = AsyncMock()
+    mock_redis_lock_class.return_value = mock_lock_instance
+
+    # Mock platform client - has active session
+    mock_daily_client = AsyncMock()
+    mock_session = AsyncMock()
+    mock_session.ended_at = None  # Still active
+    mock_daily_client.get_room_sessions = AsyncMock(return_value=[mock_session])
+    mock_create_client.return_value = mock_daily_client
+
+    with (
+        patch("reflector.worker.process.get_async_redis_client") as mock_get_redis,
+        patch("reflector.worker.process.has_pending_joins") as mock_has_pending_joins,
+        patch(
+            "reflector.worker.process.meetings_controller.update_meeting"
+        ) as mock_update_meeting,
+    ):
+        await process_meetings()
+
+        # Verify pending joins check was NOT called (no need - active sessions exist)
+        mock_has_pending_joins.assert_not_called()
+
+        # Verify meeting was NOT deactivated
+        mock_update_meeting.assert_not_called()
+
+
+@pytest.mark.asyncio
+@patch("reflector.worker.process.meetings_controller.get_all_active")
+@patch("reflector.worker.process.RedisAsyncLock")
+@patch("reflector.worker.process.create_platform_client")
+@patch("reflector.worker.process.get_async_redis_client")
+@patch("reflector.worker.process.has_pending_joins")
+@patch("reflector.worker.process.meetings_controller.update_meeting")
+async def test_process_meetings_closes_redis_even_on_continue(
+    mock_update_meeting,
+    mock_has_pending_joins,
+    mock_get_redis,
+    mock_create_client,
+    mock_redis_lock_class,
+    mock_get_all_active,
+    mock_active_meeting,
+):
+    """Test that Redis connection is always closed, even when skipping deactivation."""
+    process_meetings = _get_process_meetings_fn()
+
+    mock_get_all_active.return_value = [mock_active_meeting]
+
+    # Mock lock acquired
+    mock_lock_instance = AsyncMock()
+    mock_lock_instance.acquired = True
+    mock_lock_instance.__aenter__ = AsyncMock(return_value=mock_lock_instance)
+    mock_lock_instance.__aexit__ = AsyncMock()
+    mock_redis_lock_class.return_value = mock_lock_instance
+
+    # Mock platform client - no active sessions
+    mock_daily_client = AsyncMock()
+    mock_session = AsyncMock()
+    mock_session.ended_at = datetime.now(timezone.utc)
+    mock_daily_client.get_room_sessions = AsyncMock(return_value=[mock_session])
+    mock_create_client.return_value = mock_daily_client
+
+    # Mock Redis client
+    mock_redis = AsyncMock()
+    mock_redis.aclose = AsyncMock()
+    mock_get_redis.return_value = mock_redis
+
+    # Mock pending joins exist (will trigger continue)
+    mock_has_pending_joins.return_value = True
+
+    await process_meetings()
+
+    # Verify Redis was closed
+    mock_redis.aclose.assert_called_once()

server/tests/test_transcripts_process.py

@@ -1,6 +1,6 @@
 import asyncio
 import time
-from unittest.mock import AsyncMock, patch
+from unittest.mock import patch
 
 import pytest
 from httpx import ASGITransport, AsyncClient
@@ -142,17 +142,17 @@ async def test_whereby_recording_uses_file_pipeline(client):
             "reflector.services.transcript_process.task_pipeline_file_process"
         ) as mock_file_pipeline,
         patch(
-            "reflector.services.transcript_process.HatchetClientManager"
+            "reflector.services.transcript_process.task_pipeline_multitrack_process"
-        ) as mock_hatchet,
+        ) as mock_multitrack_pipeline,
     ):
         response = await client.post(f"/transcripts/{transcript.id}/process")
 
         assert response.status_code == 200
         assert response.json()["status"] == "ok"
 
-        # Whereby recordings should use file pipeline, not Hatchet
+        # Whereby recordings should use file pipeline
         mock_file_pipeline.delay.assert_called_once_with(transcript_id=transcript.id)
-        mock_hatchet.start_workflow.assert_not_called()
+        mock_multitrack_pipeline.delay.assert_not_called()
 
 
 @pytest.mark.usefixtures("setup_database")
@@ -177,6 +177,8 @@ async def test_dailyco_recording_uses_multitrack_pipeline(client):
         recording_trigger="automatic-2nd-participant",
         is_shared=False,
     )
+    # Force Celery backend for test
+    await rooms_controller.update(room, {"use_celery": True})
 
     transcript = await transcripts_controller.add(
         "",
@@ -211,23 +213,18 @@ async def test_dailyco_recording_uses_multitrack_pipeline(client):
             "reflector.services.transcript_process.task_pipeline_file_process"
         ) as mock_file_pipeline,
         patch(
-            "reflector.services.transcript_process.HatchetClientManager"
+            "reflector.services.transcript_process.task_pipeline_multitrack_process"
-        ) as mock_hatchet,
+        ) as mock_multitrack_pipeline,
     ):
-        mock_hatchet.start_workflow = AsyncMock(return_value="test-workflow-id")
-
         response = await client.post(f"/transcripts/{transcript.id}/process")
 
         assert response.status_code == 200
         assert response.json()["status"] == "ok"
 
-        # Daily.co multitrack recordings should use Hatchet workflow
+        # Daily.co multitrack recordings should use multitrack pipeline
-        mock_hatchet.start_workflow.assert_called_once()
+        mock_multitrack_pipeline.delay.assert_called_once_with(
-        call_kwargs = mock_hatchet.start_workflow.call_args.kwargs
+            transcript_id=transcript.id,
-        assert call_kwargs["workflow_name"] == "DiarizationPipeline"
+            bucket_name="daily-bucket",
-        assert call_kwargs["input_data"]["transcript_id"] == transcript.id
+            track_keys=track_keys,
-        assert call_kwargs["input_data"]["bucket_name"] == "daily-bucket"
+        )
-        assert call_kwargs["input_data"]["tracks"] == [
-            {"s3_key": k} for k in track_keys
-        ]
         mock_file_pipeline.delay.assert_not_called()

www/app/[roomName]/components/DailyRoom.tsx

@@ -8,7 +8,7 @@ import {
   useRef,
   useState,
 } from "react";
-import { Box, Spinner, Center, Text, Button, VStack } from "@chakra-ui/react";
+import { Box, Spinner, Center, Text } from "@chakra-ui/react";
 import { useRouter, useParams } from "next/navigation";
 import DailyIframe, {
   DailyCall,
@@ -16,18 +16,18 @@ import DailyIframe, {
   DailyCustomTrayButton,
   DailyCustomTrayButtons,
   DailyEventObjectCustomButtonClick,
-  DailyEventObjectFatalError,
-  DailyFatalErrorType,
   DailyFactoryOptions,
   DailyParticipantsObject,
 } from "@daily-co/daily-js";
 import type { components } from "../../reflector-api";
-import { printApiError, ApiError } from "../../api/_error";
 import { useAuth } from "../../lib/AuthProvider";
 import { useConsentDialog } from "../../lib/consent";
 import {
   useRoomJoinMeeting,
   useMeetingStartRecording,
+  useMeetingJoining,
+  useMeetingJoined,
+  buildMeetingLeaveUrl,
 } from "../../lib/apiHooks";
 import { omit } from "remeda";
 import {
@@ -48,63 +48,6 @@ const RAW_TRACKS_NAMESPACE = "a1b2c3d4-e5f6-7890-abcd-ef1234567890";
 const RECORDING_START_DELAY_MS = 2000;
 const RECORDING_START_MAX_RETRIES = 5;
 
-const FATAL_ERROR_MESSAGES: Partial<
-  Record<DailyFatalErrorType, { message: string; rejoinable?: boolean }>
-> = {
-  "connection-error": {
-    message: "Connection lost. Please check your network.",
-    rejoinable: true,
-  },
-  "exp-room": { message: "The meeting time has ended." },
-  "exp-token": { message: "Your session has expired.", rejoinable: true },
-  ejected: { message: "You were removed from this meeting." },
-  "meeting-full": { message: "This meeting is full." },
-  "not-allowed": { message: "You are not allowed to join this meeting." },
-  "nbf-room": { message: "This meeting hasn't started yet." },
-  "nbf-token": { message: "This meeting hasn't started yet." },
-  "no-room": { message: "This room does not exist." },
-  "end-of-life": { message: "This meeting room is no longer available." },
-};
-
-function FatalErrorScreen({
-  error,
-  roomName,
-}: {
-  error: FatalError;
-  roomName: string;
-}) {
-  const router = useRouter();
-  const info =
-    error.type !== "unknown" ? FATAL_ERROR_MESSAGES[error.type] : undefined;
-  const message = info?.message ?? `Something went wrong: ${error.message}`;
-  const rejoinable = info?.rejoinable ?? false;
-
-  return (
-    <Center width="100vw" height="100vh">
-      <VStack gap={4}>
-        <Text color="red.500">{message}</Text>
-        {rejoinable ? (
-          <>
-            <Button onClick={() => window.location.reload()}>
-              Try Rejoining
-            </Button>
-            <Button
-              variant="outline"
-              onClick={() => router.push(`/${roomName}`)}
-            >
-              Leave
-            </Button>
-          </>
-        ) : (
-          <Button onClick={() => router.push(`/${roomName}`)}>
-            Back to Room
-          </Button>
-        )}
-      </VStack>
-    </Center>
-  );
-}
-
 type Meeting = components["schemas"]["Meeting"];
 type Room = components["schemas"]["RoomDetails"];
 
@@ -142,16 +85,13 @@ const USE_FRAME_INIT_STATE = {
   joined: false as boolean,
 } as const;
 
-type FatalError = { type: DailyFatalErrorType | "unknown"; message: string };
-
 // Daily js and not Daily react used right now because daily-js allows for prebuild interface vs. -react is customizable but has no nice defaults
 const useFrame = (
   container: HTMLDivElement | null,
   cbs: {
     onLeftMeeting: () => void;
     onCustomButtonClick: (ev: DailyEventObjectCustomButtonClick) => void;
-    onJoinMeeting: () => void;
+    onJoinMeeting: (sessionId: string | null) => void;
-    onError: (ev: DailyEventObjectFatalError) => void;
   },
 ) => {
   const [{ frame, joined }, setState] = useState(USE_FRAME_INIT_STATE);
@@ -197,19 +137,18 @@ const useFrame = (
     if (!frame) return;
     frame.on("left-meeting", cbs.onLeftMeeting);
     frame.on("custom-button-click", cbs.onCustomButtonClick);
-    frame.on("error", cbs.onError);
     const joinCb = () => {
       if (!frame) {
         console.error("frame is null in joined-meeting callback");
         return;
       }
-      cbs.onJoinMeeting();
+      const local = frame.participants()?.local;
+      cbs.onJoinMeeting(local?.session_id ?? null);
     };
     frame.on("joined-meeting", joinCb);
     return () => {
       frame.off("left-meeting", cbs.onLeftMeeting);
       frame.off("custom-button-click", cbs.onCustomButtonClick);
-      frame.off("error", cbs.onError);
       frame.off("joined-meeting", joinCb);
     };
   }, [frame, cbs]);
@@ -252,8 +191,14 @@ export default function DailyRoom({ meeting, room }: DailyRoomProps) {
   const [container, setContainer] = useState<HTMLDivElement | null>(null);
   const joinMutation = useRoomJoinMeeting();
   const startRecordingMutation = useMeetingStartRecording();
+  const joiningMutation = useMeetingJoining();
+  const joinedMutation = useMeetingJoined();
   const [joinedMeeting, setJoinedMeeting] = useState<Meeting | null>(null);
-  const [fatalError, setFatalError] = useState<FatalError | null>(null);
+  const sessionIdRef = useRef<string | null>(null);
+
+  // Generate a stable connection ID for this component instance
+  // Used to track pending joins per browser tab (prevents key collision for anonymous users)
+  const connectionId = useMemo(() => crypto.randomUUID(), []);
 
   // Generate deterministic instanceIds so all participants use SAME IDs
   const cloudInstanceId = parseNonEmptyString(meeting.id);
@@ -300,18 +245,34 @@ export default function DailyRoom({ meeting, room }: DailyRoomProps) {
   const roomUrl = joinedMeeting?.room_url;
 
   const handleLeave = useCallback(() => {
-    // If a fatal error occurred, don't redirect — let the error UI show
+    if (meeting?.id && roomName) {
-    if (fatalError) return;
+      const payload = sessionIdRef.current
+        ? { session_id: sessionIdRef.current }
+        : {};
+      navigator.sendBeacon(
+        buildMeetingLeaveUrl(roomName, meeting.id),
+        JSON.stringify(payload),
+      );
+    }
     router.push("/browse");
-  }, [router, fatalError]);
+  }, [router, roomName, meeting?.id]);
 
-  const handleError = useCallback((ev: DailyEventObjectFatalError) => {
+  // Trigger presence recheck on dirty disconnects (tab close, navigation away)
-    const error: FatalError = {
+  useEffect(() => {
-      type: ev.error?.type ?? "unknown",
+    if (!meeting?.id || !roomName) return;
-      message: ev.errorMsg,
+
+    const handleBeforeUnload = () => {
+      // sendBeacon guarantees delivery even if tab closes mid-request
+      const url = buildMeetingLeaveUrl(roomName, meeting.id);
+      const payload = sessionIdRef.current
+        ? { session_id: sessionIdRef.current }
+        : {};
+      navigator.sendBeacon(url, JSON.stringify(payload));
     };
-    setFatalError(error);
+
-  }, []);
+    window.addEventListener("beforeunload", handleBeforeUnload);
+    return () => window.removeEventListener("beforeunload", handleBeforeUnload);
+  }, [meeting?.id, roomName]);
 
   const handleCustomButtonClick = useCallback(
     (ev: DailyEventObjectCustomButtonClick) => {
@@ -324,72 +285,106 @@ export default function DailyRoom({ meeting, room }: DailyRoomProps) {
     ],
   );
 
-  const handleFrameJoinMeeting = useCallback(() => {
+  const handleFrameJoinMeeting = useCallback(
-    if (meeting.recording_type === "cloud") {
+    (sessionId: string | null) => {
-      console.log("Starting dual recording via REST API", {
+      sessionIdRef.current = sessionId;
-        cloudInstanceId,
-        rawTracksInstanceId,
-      });
 
-      // Start both cloud and raw-tracks via backend REST API (with retry on 404)
+      // Signal that WebRTC connection is established
-      // Daily.co needs time to register call as "hosting" for REST API
+      // This clears the pending join intent and creates session record directly
-      const startRecordingWithRetry = (
+      joinedMutation.mutate(
-        type: DailyRecordingType,
+        {
-        instanceId: NonEmptyString,
+          params: {
-        attempt: number = 1,
+            path: {
-      ) => {
+              room_name: roomName,
-        setTimeout(() => {
+              meeting_id: meeting.id,
-          startRecordingMutation.mutate(
+            },
-            {
+          },
-              params: {
+          body: {
-                path: {
+            connection_id: connectionId,
-                  meeting_id: meeting.id,
+            session_id: sessionId,
+          },
+        },
+        {
+          onError: (error: unknown) => {
+            // Non-blocking: log but don't fail - this is cleanup, not critical
+            console.warn("Failed to signal joined:", error);
+          },
+        },
+      );
+
+      if (meeting.recording_type === "cloud") {
+        console.log("Starting dual recording via REST API", {
+          cloudInstanceId,
+          rawTracksInstanceId,
+        });
+
+        // Start both cloud and raw-tracks via backend REST API (with retry on 404)
+        // Daily.co needs time to register call as "hosting" for REST API
+        const startRecordingWithRetry = (
+          type: DailyRecordingType,
+          instanceId: NonEmptyString,
+          attempt: number = 1,
+        ) => {
+          setTimeout(() => {
+            startRecordingMutation.mutate(
+              {
+                params: {
+                  path: {
+                    meeting_id: meeting.id,
+                  },
+                },
+                body: {
+                  type,
+                  instanceId,
                 },
               },
-              body: {
+              {
-                type,
+                onError: (error: any) => {
-                instanceId,
+                  const errorText = error?.detail || error?.message || "";
-              },
+                  const is404NotHosting = errorText.includes(
-            },
+                    "does not seem to be hosting a call",
-            {
-              onError: (error: any) => {
-                const errorText = error?.detail || error?.message || "";
-                const is404NotHosting = errorText.includes(
-                  "does not seem to be hosting a call",
-                );
-                const isActiveStream = errorText.includes(
-                  "has an active stream",
-                );
-
-                if (is404NotHosting && attempt < RECORDING_START_MAX_RETRIES) {
-                  console.log(
-                    `${type}: Call not hosting yet, retry ${attempt + 1}/${RECORDING_START_MAX_RETRIES} in ${RECORDING_START_DELAY_MS}ms...`,
                   );
-                  startRecordingWithRetry(type, instanceId, attempt + 1);
+                  const isActiveStream = errorText.includes(
-                } else if (isActiveStream) {
+                    "has an active stream",
-                  console.log(
-                    `${type}: Recording already active (started by another participant)`,
                   );
-                } else {
-                  console.error(`Failed to start ${type} recording:`, error);
-                }
-              },
-            },
-          );
-        }, RECORDING_START_DELAY_MS);
-      };
 
-      // Start both recordings
+                  if (
-      startRecordingWithRetry("cloud", cloudInstanceId);
+                    is404NotHosting &&
-      startRecordingWithRetry("raw-tracks", rawTracksInstanceId);
+                    attempt < RECORDING_START_MAX_RETRIES
-    }
+                  ) {
-  }, [
+                    console.log(
-    meeting.recording_type,
+                      `${type}: Call not hosting yet, retry ${attempt + 1}/${RECORDING_START_MAX_RETRIES} in ${RECORDING_START_DELAY_MS}ms...`,
-    meeting.id,
+                    );
-    startRecordingMutation,
+                    startRecordingWithRetry(type, instanceId, attempt + 1);
-    cloudInstanceId,
+                  } else if (isActiveStream) {
-    rawTracksInstanceId,
+                    console.log(
-  ]);
+                      `${type}: Recording already active (started by another participant)`,
+                    );
+                  } else {
+                    console.error(`Failed to start ${type} recording:`, error);
+                  }
+                },
+              },
+            );
+          }, RECORDING_START_DELAY_MS);
+        };
+
+        // Start both recordings
+        startRecordingWithRetry("cloud", cloudInstanceId);
+        startRecordingWithRetry("raw-tracks", rawTracksInstanceId);
+      }
+    },
+    [
+      meeting.recording_type,
+      meeting.id,
+      roomName,
+      connectionId,
+      joinedMutation,
+      startRecordingMutation,
+      cloudInstanceId,
+      rawTracksInstanceId,
+    ],
+  );
 
   const recordingIconUrl = useMemo(
     () => new URL("/recording-icon.svg", window.location.origin),
@@ -400,13 +395,32 @@ export default function DailyRoom({ meeting, room }: DailyRoomProps) {
     onLeftMeeting: handleLeave,
     onCustomButtonClick: handleCustomButtonClick,
     onJoinMeeting: handleFrameJoinMeeting,
-    onError: handleError,
   });
 
   useEffect(() => {
     if (!frame || !roomUrl) return;
-    frame
+
-      .join({
+    const joinRoom = async () => {
+      // Signal intent to join before WebRTC handshake starts
+      // This prevents race condition where meeting is deactivated during handshake
+      try {
+        await joiningMutation.mutateAsync({
+          params: {
+            path: {
+              room_name: roomName,
+              meeting_id: meeting.id,
+            },
+          },
+          body: {
+            connection_id: connectionId,
+          },
+        });
+      } catch (error) {
+        // Non-blocking: log but continue with join
+        console.warn("Failed to signal joining intent:", error);
+      }
+
+      await frame.join({
         url: roomUrl,
         sendSettings: {
           video: {
@@ -418,9 +432,13 @@ export default function DailyRoom({ meeting, room }: DailyRoomProps) {
           },
           // Note: screenVideo intentionally not configured to preserve full quality for screen shares
         },
-      })
+      });
-      .catch(console.error.bind(console, "Failed to join daily room:"));
+    };
-  }, [frame, roomUrl]);
+
+    joinRoom().catch(console.error.bind(console, "Failed to join daily room:"));
+    // joiningMutation excluded from deps - it's a stable hook reference
+    // eslint-disable-next-line react-hooks/exhaustive-deps
+  }, [frame, roomUrl, roomName, meeting.id, connectionId]);
 
   useEffect(() => {
     setCustomTrayButton(
@@ -457,27 +475,13 @@ export default function DailyRoom({ meeting, room }: DailyRoomProps) {
   }
 
   if (joinMutation.isError) {
-    const apiDetail = printApiError(
-      joinMutation.error as /*ref 095959E6-01CC-4CF0-B3A9-F65F12F046D3*/ ApiError,
-    );
     return (
       <Center width="100vw" height="100vh">
-        <VStack gap={4}>
+        <Text color="red.500">Failed to join meeting. Please try again.</Text>
-          <Text color="red.500">
-            {apiDetail ?? "Failed to join meeting. Please try again."}
-          </Text>
-          <Button onClick={() => router.push(`/${roomName}`)}>
-            Back to Room
-          </Button>
-        </VStack>
       </Center>
     );
   }
 
-  if (fatalError) {
-    return <FatalErrorScreen error={fatalError} roomName={roomName} />;
-  }
-
   if (!roomUrl) {
     return null;
   }

www/app/lib/apiHooks.ts

@@ -1,20 +1,24 @@
 "use client";
 
-import { $api } from "./apiClient";
+import { createFinalURL, createQuerySerializer } from "openapi-fetch";
+import { $api, API_URL } from "./apiClient";
 import { useError } from "../(errors)/errorContext";
 import { QueryClient, useQueryClient } from "@tanstack/react-query";
-import type { components } from "../reflector-api";
+import type { components, paths } from "../reflector-api";
 import { useAuth } from "./AuthProvider";
 import { MeetingId } from "./types";
 import { NonEmptyString } from "./utils";
 
 /*
- * ref 095959E6-01CC-4CF0-B3A9-F65F12F046D3
  * XXX error types returned from the hooks are not always correct; declared types are ValidationError but real type could be string or any other
  * this is either a limitation or incorrect usage of Python json schema generator
  * or, limitation or incorrect usage of .d type generator from json schema
  * */
 
+/*
+if you experience lack of expected endpoints on $api, try to regenerate
+ */
+
 export const useAuthReady = () => {
   const auth = useAuth();
 
@@ -769,6 +773,7 @@ export function useRoomActiveMeetings(roomName: string | null) {
     },
     {
       enabled: !!roomName,
+      refetchInterval: 5000,
     },
   );
 }
@@ -808,6 +813,47 @@ export function useRoomJoinMeeting() {
   );
 }
 
+// Presence race fix endpoints - signal join intent to prevent race conditions during WebRTC handshake
+export function useMeetingJoining() {
+  return $api.useMutation(
+    "post",
+    "/v1/rooms/{room_name}/meetings/{meeting_id}/joining",
+    {},
+  );
+}
+
+export function useMeetingJoined() {
+  return $api.useMutation(
+    "post",
+    "/v1/rooms/{room_name}/meetings/{meeting_id}/joined",
+    {},
+  );
+}
+
+export function useMeetingLeave() {
+  return $api.useMutation(
+    "post",
+    "/v1/rooms/{room_name}/meetings/{meeting_id}/leave",
+    {},
+  );
+}
+
+/**
+ * Build absolute URL for /leave endpoint (for sendBeacon which can't use hooks).
+ */
+export function buildMeetingLeaveUrl(
+  roomName: string,
+  meetingId: string,
+): string {
+  return createFinalURL("/v1/rooms/{room_name}/meetings/{meeting_id}/leave", {
+    baseUrl: API_URL,
+    params: {
+      path: { room_name: roomName, meeting_id: meetingId },
+    },
+    querySerializer: createQuerySerializer(),
+  });
+}
+
 export function useRoomIcsSync() {
   const { setError } = useError();
 

www/app/reflector-api.d.ts

@@ -313,6 +313,79 @@ export interface paths {
     patch?: never;
     trace?: never;
   };
+  "/v1/rooms/{room_name}/meetings/{meeting_id}/joining": {
+    parameters: {
+      query?: never;
+      header?: never;
+      path?: never;
+      cookie?: never;
+    };
+    get?: never;
+    put?: never;
+    /**
+     * Meeting Joining
+     * @description Signal intent to join meeting. Called before WebRTC handshake starts.
+     *
+     *     This creates a pending join record that prevents the meeting from being
+     *     deactivated while the WebRTC handshake is in progress. The record expires
+     *     automatically after 30 seconds if the connection is not established.
+     */
+    post: operations["v1_meeting_joining"];
+    delete?: never;
+    options?: never;
+    head?: never;
+    patch?: never;
+    trace?: never;
+  };
+  "/v1/rooms/{room_name}/meetings/{meeting_id}/joined": {
+    parameters: {
+      query?: never;
+      header?: never;
+      path?: never;
+      cookie?: never;
+    };
+    get?: never;
+    put?: never;
+    /**
+     * Meeting Joined
+     * @description Signal that WebRTC connection is established.
+     *
+     *     This clears the pending join record, confirming the user has successfully
+     *     connected to the meeting. Safe to call even if meeting was deactivated
+     *     during the handshake (idempotent cleanup).
+     */
+    post: operations["v1_meeting_joined"];
+    delete?: never;
+    options?: never;
+    head?: never;
+    patch?: never;
+    trace?: never;
+  };
+  "/v1/rooms/{room_name}/meetings/{meeting_id}/leave": {
+    parameters: {
+      query?: never;
+      header?: never;
+      path?: never;
+      cookie?: never;
+    };
+    get?: never;
+    put?: never;
+    /**
+     * Meeting Leave
+     * @description Trigger presence update when user leaves meeting.
+     *
+     *     When session_id is provided in the body, closes the session directly
+     *     for instant presence update. Falls back to polling when session_id
+     *     is not available (e.g., sendBeacon without frame access).
+     *     Called on tab close/navigation via sendBeacon().
+     */
+    post: operations["v1_meeting_leave"];
+    delete?: never;
+    options?: never;
+    head?: never;
+    patch?: never;
+    trace?: never;
+  };
   "/v1/transcripts": {
     parameters: {
       query?: never;
@@ -1497,6 +1570,56 @@ export interface components {
       /** Reason */
       reason?: string | null;
     };
+    /**
+     * JoinedRequest
+     * @description Request body for /joined endpoint (after WebRTC connection established).
+     */
+    JoinedRequest: {
+      /**
+       * Connection Id
+       * @description A non-empty string
+       */
+      connection_id: string;
+      /** Session Id */
+      session_id?: string | null;
+      /** User Name */
+      user_name?: string | null;
+    };
+    /** JoinedResponse */
+    JoinedResponse: {
+      /**
+       * Status
+       * @constant
+       */
+      status: "ok";
+    };
+    /**
+     * JoiningRequest
+     * @description Request body for /joining endpoint (before WebRTC handshake).
+     */
+    JoiningRequest: {
+      /**
+       * Connection Id
+       * @description A non-empty string
+       */
+      connection_id: string;
+    };
+    /** JoiningResponse */
+    JoiningResponse: {
+      /**
+       * Status
+       * @constant
+       */
+      status: "ok";
+    };
+    /** LeaveResponse */
+    LeaveResponse: {
+      /**
+       * Status
+       * @constant
+       */
+      status: "ok";
+    };
     /** Meeting */
     Meeting: {
       /** Id */
@@ -2687,6 +2810,110 @@ export interface operations {
       };
     };
   };
+  v1_meeting_joining: {
+    parameters: {
+      query?: never;
+      header?: never;
+      path: {
+        room_name: string;
+        meeting_id: string;
+      };
+      cookie?: never;
+    };
+    requestBody: {
+      content: {
+        "application/json": components["schemas"]["JoiningRequest"];
+      };
+    };
+    responses: {
+      /** @description Successful Response */
+      200: {
+        headers: {
+          [name: string]: unknown;
+        };
+        content: {
+          "application/json": components["schemas"]["JoiningResponse"];
+        };
+      };
+      /** @description Validation Error */
+      422: {
+        headers: {
+          [name: string]: unknown;
+        };
+        content: {
+          "application/json": components["schemas"]["HTTPValidationError"];
+        };
+      };
+    };
+  };
+  v1_meeting_joined: {
+    parameters: {
+      query?: never;
+      header?: never;
+      path: {
+        room_name: string;
+        meeting_id: string;
+      };
+      cookie?: never;
+    };
+    requestBody: {
+      content: {
+        "application/json": components["schemas"]["JoinedRequest"];
+      };
+    };
+    responses: {
+      /** @description Successful Response */
+      200: {
+        headers: {
+          [name: string]: unknown;
+        };
+        content: {
+          "application/json": components["schemas"]["JoinedResponse"];
+        };
+      };
+      /** @description Validation Error */
+      422: {
+        headers: {
+          [name: string]: unknown;
+        };
+        content: {
+          "application/json": components["schemas"]["HTTPValidationError"];
+        };
+      };
+    };
+  };
+  v1_meeting_leave: {
+    parameters: {
+      query?: never;
+      header?: never;
+      path: {
+        room_name: string;
+        meeting_id: string;
+      };
+      cookie?: never;
+    };
+    requestBody?: never;
+    responses: {
+      /** @description Successful Response */
+      200: {
+        headers: {
+          [name: string]: unknown;
+        };
+        content: {
+          "application/json": components["schemas"]["LeaveResponse"];
+        };
+      };
+      /** @description Validation Error */
+      422: {
+        headers: {
+          [name: string]: unknown;
+        };
+        content: {
+          "application/json": components["schemas"]["HTTPValidationError"];
+        };
+      };
+    };
+  };
   v1_transcripts_list: {
     parameters: {
       query?: {