kinemotion 0.62.0__tar.gz → 0.63.0__tar.gz

{kinemotion-0.62.0 → kinemotion-0.63.0}/CHANGELOG.md

@@ -7,6 +7,17 @@ and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0
 
 <!-- version list -->
 
+## kinemotion-v0.63.0 (2025-12-15)
+
+### Feat
+
+- use email as unique identifier for users and storage organization
+
+### Fix
+
+- enhance database debugging with detailed connection logging
+- revert httpx client configuration that broke Supabase compatibility
+
 ## kinemotion-v0.62.0 (2025-12-15)
 
 ### Feat

{kinemotion-0.62.0 → kinemotion-0.63.0}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: kinemotion
-Version: 0.62.0
+Version: 0.63.0
 Summary: Video-based kinematic analysis for athletic performance
 Project-URL: Homepage, https://github.com/feniix/kinemotion
 Project-URL: Repository, https://github.com/feniix/kinemotion

{kinemotion-0.62.0 → kinemotion-0.63.0}/backend/src/kinemotion_backend/database.py

@@ -3,7 +3,6 @@
 import os
 from typing import Any
 
-import httpx
 import structlog
 from supabase import Client, create_client
 
@@ -18,11 +17,14 @@ class DatabaseClient:
         self.supabase_url = os.getenv("SUPABASE_URL", "")
 
         # Prefer modern keys, fall back to legacy for compatibility
+        # Check which key source is being used for debugging
+        supabase_publishable_key = os.getenv("SUPABASE_PUBLISHABLE_KEY")
+        supabase_secret_key = os.getenv("SUPABASE_SECRET_KEY")
+        supabase_anon_key = os.getenv("SUPABASE_ANON_KEY")
+        supabase_key = os.getenv("SUPABASE_KEY")
+
         self.supabase_key = (
-            os.getenv("SUPABASE_PUBLISHABLE_KEY")
-            or os.getenv("SUPABASE_SECRET_KEY")
-            or os.getenv("SUPABASE_ANON_KEY")
-            or os.getenv("SUPABASE_KEY")
+            supabase_publishable_key or supabase_secret_key or supabase_anon_key or supabase_key
         )
 
         if not self.supabase_url:
@@ -35,12 +37,23 @@ class DatabaseClient:
                 "SUPABASE_ANON_KEY, or SUPABASE_KEY"
             )
 
-        # Configure http client with timeout and verify settings to avoid deprecation warnings
-        http_client = httpx.Client(timeout=10.0, verify=True)
-        self.client: Client = create_client(
-            self.supabase_url, self.supabase_key, http_client=http_client
+        # Log which key source is being used (without exposing the actual key)
+        key_source = "unknown"
+        if supabase_publishable_key:
+            key_source = "SUPABASE_PUBLISHABLE_KEY"
+        elif supabase_secret_key:
+            key_source = "SUPABASE_SECRET_KEY"
+        elif supabase_anon_key:
+            key_source = "SUPABASE_ANON_KEY"
+        elif supabase_key:
+            key_source = "SUPABASE_KEY"
+
+        self.client: Client = create_client(self.supabase_url, self.supabase_key)
+        logger.info(
+            "database_client_initialized",
+            supabase_url=self.supabase_url,
+            key_source=key_source,
         )
-        logger.info("database_client_initialized", supabase_url=self.supabase_url)
 
     async def create_analysis_session(
         self,

{kinemotion-0.62.0 → kinemotion-0.63.0}/backend/src/kinemotion_backend/middleware.py

@@ -50,28 +50,26 @@ class RequestLoggingMiddleware(BaseHTTPMiddleware):
             client_ip=request.client.host if request.client else "unknown",
         )
 
-        # Extract and validate user ID from Authorization header
+        # Extract and validate user email from Authorization header
         auth_header = request.headers.get("authorization")
         if auth_header and auth_header.startswith("Bearer ") and supabase_auth:
             token = auth_header.replace("Bearer ", "")
             auth_start = time.time()
             try:
-                user_id = supabase_auth.get_user_id(token)
                 user_email = supabase_auth.get_user_email(token)
                 auth_duration_ms = (time.time() - auth_start) * 1000
 
-                # Bind user info to logging context (auth_duration_ms only in auth event)
+                # Bind user email to logging context
                 structlog.contextvars.bind_contextvars(
-                    user_id=user_id,
-                    user_email=user_email,
+                    email=user_email,
                 )
 
                 # Store in request state for use in endpoints
-                request.state.user_id = user_id
-                request.state.user_email = user_email
+                request.state.email = user_email
 
                 logger.info(
                     "user_authenticated",
+                    email=user_email,
                     auth_duration_ms=round(auth_duration_ms, 2),
                 )
             except Exception as e:

{kinemotion-0.62.0 → kinemotion-0.63.0}/backend/src/kinemotion_backend/routes/analysis.py

@@ -41,27 +41,27 @@ else:
     limiter = RateLimiter()  # type: ignore[operator]
 
 
-async def get_user_id_for_analysis(
+async def get_user_email_for_analysis(
     request: Request,
     x_test_password: str | None = Header(None),  # noqa: B008
 ) -> str:
-    """Extract user ID from JWT token or use test backdoor.
+    """Extract user email from JWT token or use test backdoor.
 
     Args:
         request: HTTP request (to access Authorization header)
         x_test_password: Optional test password for debugging
 
     Returns:
-        User ID (from token or test backdoor)
+        User email (from token or test backdoor)
 
     Raises:
         HTTPException: If authentication fails and test password not provided
     """
     # Allow bypass with test password (for curl testing, debugging)
     if is_test_password_valid(x_test_password):
-        test_user_id = os.getenv("TEST_USER_ID", "test-user-00000000-0000-0000-0000-000000000000")
-        logger.info("analysis_test_password_used", user_id=test_user_id)
-        return test_user_id
+        test_email = os.getenv("TEST_EMAIL", "test@example.com")
+        logger.info("analysis_test_password_used", email=test_email)
+        return test_email
 
     # Otherwise require valid JWT token from Authorization header
     auth_header = request.headers.get("authorization")
@@ -74,12 +74,12 @@ async def get_user_id_for_analysis(
     token = auth_header.replace("Bearer ", "")
     try:
         auth = SupabaseAuth()
-        user_id = auth.get_user_id(token)
-        return user_id
+        email = auth.get_user_email(token)
+        return email
     except HTTPException:
         raise
     except Exception as e:
-        logger.error("user_id_extraction_failed", error=str(e))
+        logger.error("user_email_extraction_failed", error=str(e))
         raise HTTPException(
             status_code=status.HTTP_401_UNAUTHORIZED,
             detail="Invalid authentication credentials",
@@ -96,7 +96,7 @@ async def analyze_video(
     debug: str = Form("false"),  # noqa: B008
     referer: str | None = Header(None),  # noqa: B008
     x_test_password: str | None = Header(None),  # noqa: B008
-    user_id: str = Depends(get_user_id_for_analysis),  # noqa: B008
+    email: str = Depends(get_user_email_for_analysis),  # noqa: B008
 ) -> JSONResponse:
     """Analyze video and return jump metrics.
 
@@ -112,7 +112,7 @@ async def analyze_video(
         jump_type: Type of jump ("drop_jump" or "cmj")
         quality: Analysis quality preset ("fast", "balanced", or "accurate")
         debug: Debug overlay flag ("true" or "false", default "false")
-        user_id: Authenticated user ID (extracted from JWT or test password)
+        email: Authenticated user email (extracted from JWT or test password)
 
     Returns:
         JSON response with metrics or error details
@@ -138,7 +138,7 @@ async def analyze_video(
             jump_type=jump_type,
             quality=quality,
             debug=enable_debug,
-            user_id=user_id,
+            email=email,
         )
 
         # Perform analysis using service layer
@@ -147,7 +147,7 @@ async def analyze_video(
             jump_type=jump_type,
             quality=quality,
             debug=enable_debug,
-            user_id=user_id,
+            user_id=email,
         )
 
         # Log analysis completion

{kinemotion-0.62.0 → kinemotion-0.63.0}/backend/src/kinemotion_backend/routes/health.py

@@ -35,11 +35,23 @@ async def health_check() -> dict[str, str | bool]:
         from kinemotion_backend.database import get_database_client
 
         db_client = get_database_client()
-        # Test database with a simple query
-        db_client.client.table("analysis_sessions").select("id").limit(1).execute()
+        # Try to access analysis_sessions table with select limit to test connectivity
+        # This will return data if accessible, or raise if table doesn't exist or RLS blocks it
+        response = db_client.client.table("analysis_sessions").select("id").limit(1).execute()
+        # If we get here, the database connection worked
+        # Note: response.data could be empty if table is empty, but that's still success
         database_connected = True
+        logger.info(
+            "database_health_check_succeeded",
+            rows_returned=len(response.data or []),
+        )
     except Exception as db_error:
-        logger.warning("database_health_check_failed", error=str(db_error))
+        logger.warning(
+            "database_health_check_failed",
+            error=str(db_error),
+            error_type=type(db_error).__name__,
+            exc_info=True,
+        )
 
     return {
         "status": "healthy",

{kinemotion-0.62.0 → kinemotion-0.63.0}/backend/tests/conftest.py

@@ -41,7 +41,7 @@ def setup_test_environment() -> None:
     os.environ["SUPABASE_ANON_KEY"] = "test-anon-key"
     # Test password for /analyze endpoint authentication bypass
     os.environ["TEST_PASSWORD"] = "test-password-12345"
-    os.environ["TEST_USER_ID"] = "test-user-00000000-0000-0000-0000-000000000000"
+    os.environ["TEST_EMAIL"] = "test@example.com"
 
     yield
 
@@ -55,7 +55,7 @@ def setup_test_environment() -> None:
         "SUPABASE_URL",
         "SUPABASE_ANON_KEY",
         "TEST_PASSWORD",
-        "TEST_USER_ID",
+        "TEST_EMAIL",
     ]
     for key in env_vars:
         os.environ.pop(key, None)

{kinemotion-0.62.0 → kinemotion-0.63.0}/pyproject.toml

@@ -1,6 +1,6 @@
 [project]
 name = "kinemotion"
-version = "0.62.0"
+version = "0.63.0"
 description = "Video-based kinematic analysis for athletic performance"
 readme = "README.md"
 requires-python = ">=3.10,<3.13"

{kinemotion-0.62.0 → kinemotion-0.63.0}/uv.lock

@@ -882,7 +882,7 @@ wheels = [
 
 [[package]]
 name = "kinemotion"
-version = "0.61.0"
+version = "0.62.0"
 source = { editable = "." }
 dependencies = [
     { name = "click" },