dailyco component naming for LLM reading

Author: dearlordylord

IMPLEMENTATION_STATUS.md

@@ -5,7 +5,7 @@
 ### 1. Platform Abstraction Layer (`server/reflector/video_platforms/`)
 - **base.py**: Abstract interface defining all platform operations
 - **whereby.py**: Whereby implementation wrapping existing functionality
-- **dailyco.py**: Daily.co client implementation (ready for testing when credentials available)
+- **daily.py**: Daily client implementation (ready for testing when credentials available)
 - **mock.py**: Mock implementation for unit testing
 - **registry.py**: Platform registration and discovery
 - **factory.py**: Factory methods for creating platform clients
@@ -18,27 +18,27 @@
 ### 3. Configuration
 - **Settings**: Added Daily.co configuration variables
 - **Feature Flags**:
-  - `DAILYCO_MIGRATION_ENABLED`: Master switch for migration
-  - `DAILYCO_MIGRATION_ROOM_IDS`: List of specific rooms to migrate
+  - `DAILY_MIGRATION_ENABLED`: Master switch for migration
+  - `DAILY_MIGRATION_ROOM_IDS`: List of specific rooms to migrate
   - `DEFAULT_VIDEO_PLATFORM`: Default platform when migration enabled
 
 ### 4. Backend API Updates
 - **Room Creation**: Now assigns platform based on feature flags
 - **Meeting Creation**: Uses platform abstraction instead of direct Whereby calls
 - **Response Models**: Include platform field
-- **Webhook Handler**: Added Daily.co webhook endpoint at `/v1/dailyco_webhook`
+- **Webhook Handler**: Added Daily webhook endpoint at `/v1/daily/webhook`
 
 ### 5. Frontend Components (`www/app/[roomName]/components/`)
 - **RoomContainer.tsx**: Platform-agnostic container that routes to appropriate component
 - **WherebyRoom.tsx**: Extracted existing Whereby functionality with consent management
-- **DailyCoRoom.tsx**: Daily.co implementation using DailyIframe
+- **DailyRoom.tsx**: Daily implementation using DailyIframe
 - **Dependencies**: Added `@daily-co/daily-js` and `@daily-co/daily-react`
 
 ## How It Works
 
 1. **Platform Selection**:
-   - If `DAILYCO_MIGRATION_ENABLED=false` → Always use Whereby
-   - If enabled and room ID in `DAILYCO_MIGRATION_ROOM_IDS` → Use Daily.co
+   - If `DAILY_MIGRATION_ENABLED=false` → Always use Whereby
+   - If enabled and room ID in `DAILY_MIGRATION_ROOM_IDS` → Use Daily
    - Otherwise → Use `DEFAULT_VIDEO_PLATFORM`
 
 2. **Meeting Creation Flow**:
@@ -59,11 +59,11 @@
 
 1. **Set Environment Variables**:
    ```bash
-   DAILYCO_API_KEY=your-key
-   DAILYCO_WEBHOOK_SECRET=your-secret
-   DAILYCO_SUBDOMAIN=your-subdomain
-   AWS_DAILYCO_S3_BUCKET=your-bucket
-   AWS_DAILYCO_ROLE_ARN=your-role
+   DAILY_API_KEY=your-key
+   DAILY_WEBHOOK_SECRET=your-secret
+   DAILY_SUBDOMAIN=your-subdomain
+   AWS_DAILY_S3_BUCKET=your-bucket
+   AWS_DAILY_ROLE_ARN=your-role
    ```
 
 2. **Run Database Migration**:
@@ -75,13 +75,13 @@
 3. **Test Platform Creation**:
    ```python
    from reflector.video_platforms.factory import create_platform_client
-   client = create_platform_client("dailyco")
+   client = create_platform_client("daily")
    # Test operations...
    ```
 
 ### 6. Testing & Validation (`server/tests/`)
 - **test_video_platforms.py**: Comprehensive unit tests for all platform clients
-- **test_dailyco_webhook.py**: Integration tests for Daily.co webhook handling
+- **test_daily_webhook.py**: Integration tests for Daily webhook handling
 - **utils/video_platform_test_utils.py**: Testing utilities and helpers
 - **Mock Testing**: Full test coverage using mock platform client
 - **Webhook Testing**: HMAC signature validation and event processing tests
@@ -106,10 +106,10 @@ Daily.co webhooks are configured via API (no dashboard interface). Use the Daily
 ```bash
 # Configure webhook endpoint
 curl -X POST https://api.daily.co/v1/webhook-endpoints \
-  -H "Authorization: Bearer ${DAILYCO_API_KEY}" \
+  -H "Authorization: Bearer ${DAILY_API_KEY}" \
   -H "Content-Type: application/json" \
   -d '{
-    "url": "https://yourdomain.com/v1/dailyco_webhook",
+    "url": "https://yourdomain.com/v1/daily/webhook",
     "events": [
       "participant.joined",
       "participant.left",
@@ -166,7 +166,7 @@ Daily.co uses HMAC-SHA256 for webhook verification:
 import hmac
 import hashlib
 
-def verify_dailyco_webhook(body: bytes, signature: str, secret: str) -> bool:
+def verify_webhook_signature(body: bytes, signature: str) -> bool:
     expected = hmac.new(secret.encode(), body, hashlib.sha256).hexdigest()
     return hmac.compare_digest(expected, signature)
 ```
@@ -200,10 +200,10 @@ async def process_recording_from_url(recording_url: str, meeting_id: str, record
 uv run pytest tests/test_video_platforms.py -v
 
 # Run webhook integration tests
-uv run pytest tests/test_dailyco_webhook.py -v
+uv run pytest tests/test_daily_webhook.py -v
 
 # Run with coverage
-uv run pytest tests/test_video_platforms.py tests/test_dailyco_webhook.py --cov=reflector.video_platforms --cov=reflector.views.dailyco
+uv run pytest tests/test_video_platforms.py tests/test_daily_webhook.py --cov=reflector.video_platforms --cov=reflector.views.daily
 ```
 
 ### Manual Testing with Mock Platform
@@ -231,7 +231,7 @@ print(f"Created meeting: {meeting.room_url}")
 
 ```python
 # Test webhook payload processing
-from reflector.views.dailyco import dailyco_webhook
+from reflector.views.daily import webhook
 from reflector.worker.process import process_recording_from_url
 
 # Simulate webhook event

PLAN.md

@@ -19,10 +19,10 @@ This plan outlines a systematic migration from Whereby to Daily.co, focusing on
 - [ ] Create Daily.co account and obtain API credentials (PENDING - User to provide)
 - [x] Add environment variables to `.env` files:
   ```bash
-  DAILYCO_API_KEY=your-api-key
-  DAILYCO_WEBHOOK_SECRET=your-webhook-secret
-  DAILYCO_SUBDOMAIN=your-subdomain
-  AWS_DAILYCO_ROLE_ARN=arn:aws:iam::xxx:role/dailyco-recording
+  DAILY_API_KEY=your-api-key
+  DAILY_WEBHOOK_SECRET=your-webhook-secret
+  DAILY_SUBDOMAIN=your-subdomain
+  AWS_DAILY_ROLE_ARN=arn:aws:iam::xxx:role/daily-recording
   ```
 - [ ] Set up Daily.co webhook endpoint in dashboard (PENDING - Credentials needed)
 - [ ] Configure S3 bucket permissions for Daily.co (PENDING - Credentials needed)
@@ -45,8 +45,8 @@ This plan outlines a systematic migration from Whereby to Daily.co, focusing on
 
 - [x] Implement feature flag in backend settings:
   ```python
-  DAILYCO_MIGRATION_ENABLED = env.bool("DAILYCO_MIGRATION_ENABLED", False)
-  DAILYCO_MIGRATION_ROOM_IDS = env.list("DAILYCO_MIGRATION_ROOM_IDS", [])
+  DAILY_MIGRATION_ENABLED = env.bool("DAILY_MIGRATION_ENABLED", False)
+  DAILY_MIGRATION_ROOM_IDS = env.list("DAILY_MIGRATION_ROOM_IDS", [])
   ```
 - [x] Add platform selection logic to room creation
 - [ ] Create admin UI to toggle platform per room (FUTURE - Not in Phase 1)
@@ -66,7 +66,7 @@ This plan outlines a systematic migration from Whereby to Daily.co, focusing on
 ### 2.1 Webhook Handler
 **Owner**: Backend Developer
 
-- [x] Create `server/reflector/views/dailyco.py` webhook endpoint
+- [x] Create `server/reflector/views/daily.py` webhook endpoint at `/v1/daily/webhook`
 - [x] Implement HMAC signature verification
 - [x] Handle events:
   - `participant.joined`
@@ -111,13 +111,13 @@ This plan outlines a systematic migration from Whereby to Daily.co, focusing on
   // www/app/[roomName]/components/RoomContainer.tsx
   export default function RoomContainer({ params }) {
     const platform = meeting.response.platform || "whereby";
-    if (platform === 'dailyco') {
-      return <DailyCoRoom meeting={meeting.response} />
+    if (platform === 'daily') {
+      return <DailyRoom meeting={meeting.response} />
     }
     return <WherebyRoom meeting={meeting.response} />
   }
   ```
-- [x] Implement `DailyCoRoom` component with:
+- [x] Implement `DailyRoom` component with:
   - Call initialization using DailyIframe
   - Recording consent flow
   - Leave meeting handling

server/reflector/app.py

@@ -12,7 +12,7 @@
 from reflector.logger import logger
 from reflector.metrics import metrics_init
 from reflector.settings import settings
-from reflector.views.dailyco import router as dailyco_router
+from reflector.views.daily import router as daily_router
 from reflector.views.meetings import router as meetings_router
 from reflector.views.rooms import router as rooms_router
 from reflector.views.rtc_offer import router as rtc_offer_router
@@ -87,7 +87,7 @@ async def lifespan(app: FastAPI):
 app.include_router(user_router, prefix="/v1")
 app.include_router(zulip_router, prefix="/v1")
 app.include_router(whereby_router, prefix="/v1")
-app.include_router(dailyco_router, prefix="/v1")
+app.include_router(daily_router, prefix="/v1/daily")
 add_pagination(app)
 
 # prepare celery

server/reflector/db/meetings.py

@@ -1,17 +1,15 @@
 from datetime import datetime
-from typing import TYPE_CHECKING, Literal
+from typing import Literal
 
 import sqlalchemy as sa
 from fastapi import HTTPException
 from pydantic import BaseModel, Field
 
 from reflector.db import database, metadata
 from reflector.db.rooms import Room
+from reflector.settings import Platform
 from reflector.utils import generate_uuid4
 
-if TYPE_CHECKING:
-    from reflector.video_platforms.base import Platform
-
 meetings = sa.Table(
     "meeting",
     metadata,
@@ -88,7 +86,7 @@ class Meeting(BaseModel):
         "none", "prompt", "automatic", "automatic-2nd-participant"
     ] = "automatic-2nd-participant"
     num_clients: int = 0
-    platform: "Platform" = "whereby"
+    platform: Platform = "whereby"
 
 
 class MeetingController:

server/reflector/settings.py

@@ -1,5 +1,9 @@
+from typing import Literal
+
 from pydantic_settings import BaseSettings, SettingsConfigDict
 
+Platform = Literal["whereby", "daily"]
+
 
 class Settings(BaseSettings):
     model_config = SettingsConfigDict(
@@ -101,18 +105,18 @@ class Settings(BaseSettings):
     AWS_PROCESS_RECORDING_QUEUE_URL: str | None = None
     SQS_POLLING_TIMEOUT_SECONDS: int = 60
 
-    # Daily.co integration
-    DAILYCO_API_KEY: str | None = None
-    DAILYCO_WEBHOOK_SECRET: str | None = None
-    DAILYCO_SUBDOMAIN: str | None = None
-    AWS_DAILYCO_S3_BUCKET: str | None = None
-    AWS_DAILYCO_S3_REGION: str = "us-west-2"
-    AWS_DAILYCO_ROLE_ARN: str | None = None
+    # Daily integration
+    DAILY_API_KEY: str | None = None
+    DAILY_WEBHOOK_SECRET: str | None = None
+    DAILY_SUBDOMAIN: str | None = None
+    AWS_DAILY_S3_BUCKET: str | None = None
+    AWS_DAILY_S3_REGION: str = "us-west-2"
+    AWS_DAILY_ROLE_ARN: str | None = None
 
     # Video platform migration feature flags
-    DAILYCO_MIGRATION_ENABLED: bool = True
-    DAILYCO_MIGRATION_ROOM_IDS: list[str] = []
-    DEFAULT_VIDEO_PLATFORM: str = "dailyco"
+    DAILY_MIGRATION_ENABLED: bool = True
+    DAILY_MIGRATION_ROOM_IDS: list[str] = []
+    DEFAULT_VIDEO_PLATFORM: Platform = "daily"
 
     # Zulip integration
     ZULIP_REALM: str | None = None

server/reflector/video_platforms/base.py

@@ -6,19 +6,18 @@
 
 from reflector.db.rooms import Room
 
-# Supported video platforms
-Platform = Literal["whereby", "dailyco"]
+Platform = Literal["whereby", "daily"]
 
+RecordingType = Literal["none", "local", "cloud"]
 
-class MeetingData(BaseModel):
-    """Standardized meeting data returned by all platforms."""
 
+class MeetingData(BaseModel):
     meeting_id: str
     room_name: str
     room_url: str
     host_room_url: str
     platform: Platform
-    extra_data: Dict[str, Any] = {}  # Platform-specific data
+    extra_data: Dict[str, Any] = {}
 
 
 class VideoPlatformConfig(BaseModel):

server/reflector/video_platforms/dailyco.py renamed to server/reflector/video_platforms/daily.py

@@ -1,21 +1,23 @@
 import hmac
 from datetime import datetime
 from hashlib import sha256
+from http import HTTPStatus
 from typing import Any, Dict, Optional
 
 import httpx
 
 from reflector.db.rooms import Room
 
-from .base import MeetingData, VideoPlatformClient, VideoPlatformConfig
+from .base import MeetingData, Platform, RecordingType, VideoPlatformClient, VideoPlatformConfig
 
 
-class DailyCoClient(VideoPlatformClient):
-    """Daily.co video platform implementation."""
-
-    PLATFORM_NAME = "dailyco"
-    TIMEOUT = 10  # seconds
+class DailyClient(VideoPlatformClient):
+    PLATFORM_NAME: Platform = "daily"
+    TIMEOUT = 10
     BASE_URL = "https://api.daily.co/v1"
+    TIMESTAMP_FORMAT = "%Y%m%d%H%M%S"
+    RECORDING_NONE: RecordingType = "none"
+    RECORDING_CLOUD: RecordingType = "cloud"
 
     def __init__(self, config: VideoPlatformConfig):
         super().__init__(config)
@@ -28,14 +30,14 @@ async def create_meeting(
         self, room_name_prefix: str, end_date: datetime, room: Room
     ) -> MeetingData:
         """Create a Daily.co room."""
-        room_name = f"{room_name_prefix}-{datetime.now().strftime('%Y%m%d%H%M%S')}"
+        room_name = f"{room_name_prefix}-{datetime.now().strftime(self.TIMESTAMP_FORMAT)}"
 
         data = {
             "name": room_name,
             "privacy": "private" if room.is_locked else "public",
             "properties": {
                 "enable_recording": room.recording_type
-                if room.recording_type != "none"
+                if room.recording_type != self.RECORDING_NONE
                 else False,
                 "enable_chat": True,
                 "enable_screenshare": True,
@@ -46,7 +48,7 @@ async def create_meeting(
         }
 
         # Configure S3 bucket for cloud recordings
-        if room.recording_type == "cloud" and self.config.s3_bucket:
+        if room.recording_type == self.RECORDING_CLOUD and self.config.s3_bucket:
             data["properties"]["recordings_bucket"] = {
                 "bucket_name": self.config.s3_bucket,
                 "bucket_region": self.config.s3_region,
@@ -107,7 +109,7 @@ async def delete_room(self, room_name: str) -> bool:
                 timeout=self.TIMEOUT,
             )
             # Daily.co returns 200 for success, 404 if room doesn't exist
-            return response.status_code in (200, 404)
+            return response.status_code in (HTTPStatus.OK, HTTPStatus.NOT_FOUND)
 
     async def upload_logo(self, room_name: str, logo_path: str) -> bool:
         """Daily.co doesn't support custom logos per room - this is a no-op."""

server/reflector/video_platforms/factory.py

@@ -19,14 +19,14 @@ def get_platform_config(platform: Platform) -> VideoPlatformConfig:
             aws_access_key_id=settings.AWS_WHEREBY_ACCESS_KEY_ID,
             aws_access_key_secret=settings.AWS_WHEREBY_ACCESS_KEY_SECRET,
         )
-    elif platform == "dailyco":
+    elif platform == "daily":
         return VideoPlatformConfig(
-            api_key=settings.DAILYCO_API_KEY or "",
-            webhook_secret=settings.DAILYCO_WEBHOOK_SECRET or "",
-            subdomain=settings.DAILYCO_SUBDOMAIN,
-            s3_bucket=settings.AWS_DAILYCO_S3_BUCKET,
-            s3_region=settings.AWS_DAILYCO_S3_REGION,
-            aws_role_arn=settings.AWS_DAILYCO_ROLE_ARN,
+            api_key=settings.DAILY_API_KEY or "",
+            webhook_secret=settings.DAILY_WEBHOOK_SECRET or "",
+            subdomain=settings.DAILY_SUBDOMAIN,
+            s3_bucket=settings.AWS_DAILY_S3_BUCKET,
+            s3_region=settings.AWS_DAILY_S3_REGION,
+            aws_role_arn=settings.AWS_DAILY_ROLE_ARN,
         )
     else:
         raise ValueError(f"Unknown platform: {platform}")
@@ -40,13 +40,13 @@ def create_platform_client(platform: Platform) -> VideoPlatformClient:
 
 def get_platform_for_room(room_id: Optional[str] = None) -> Platform:
     """Determine which platform to use for a room based on feature flags."""
-    # If Daily.co migration is disabled, always use Whereby
-    if not settings.DAILYCO_MIGRATION_ENABLED:
+    # If Daily migration is disabled, always use Whereby
+    if not settings.DAILY_MIGRATION_ENABLED:
         return "whereby"
 
-    # If a specific room is in the migration list, use Daily.co
-    if room_id and room_id in settings.DAILYCO_MIGRATION_ROOM_IDS:
-        return "dailyco"
+    # If a specific room is in the migration list, use Daily
+    if room_id and room_id in settings.DAILY_MIGRATION_ROOM_IDS:
+        return "daily"
 
     # Otherwise use the default platform
     return settings.DEFAULT_VIDEO_PLATFORM