fix: pass uuid1() instead of video_id as activity_id for view events

video_id is a v4 UUID but user_activity.activity_id is a TimeUUID (v1)
column in Cassandra. Passing a v4 UUID caused INVALID_DATABASE_QUERY errors
on every video view. Fix by dropping the explicit activity_id argument so
record_user_activity auto-generates a uuid1() as intended.

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>

Author: pmcfadin

.gitignore

@@ -1,6 +1,6 @@
 .env
 .python-version
-.DS_Store 
+.DS_Store
 
 # Python
 __pycache__/
@@ -15,6 +15,11 @@ __pycache__/
 
 *.log
 
+# Data files (source of truth is in killrvideo-data project)
+data/
+logs/
+dsbulk.conf
+
 certs/
 
 .vscode/

app/services/video_service.py

@@ -477,7 +477,6 @@ async def record_video_view(
         await record_user_activity(
             userid=effective_user_id,
             activity_type="view",
-            activity_id=video_id,
         )
     except Exception:
         logger.warning("user_activity insert failed for view; ignoring", exc_info=True)

tests/services/test_video_service.py

@@ -286,11 +286,11 @@ async def test_record_video_view_authenticated_user_activity():
             vid, viewer_user_id=viewer_id, db_table=mock_stats_table
         )
 
-        # record_user_activity should be called with the real user ID and video_id
+        # record_user_activity should be called with the real user ID; no activity_id
+        # because view events auto-generate a uuid1() to satisfy the TimeUUID column
         mock_record_user_activity.assert_awaited_once_with(
             userid=viewer_id,
             activity_type="view",
-            activity_id=vid,
         )
 
 
@@ -318,11 +318,11 @@ async def test_record_video_view_anonymous_user_activity():
             vid, viewer_user_id=None, db_table=mock_stats_table
         )
 
-        # record_user_activity should be called with the anonymous sentinel UUID and video_id
+        # record_user_activity should be called with the anonymous sentinel UUID; no activity_id
+        # because view events auto-generate a uuid1() to satisfy the TimeUUID column
         mock_record_user_activity.assert_awaited_once_with(
             userid=ANONYMOUS_USER_ID,
             activity_type="view",
-            activity_id=vid,
         )